Skip to content

[python] Refactor Python emitter node imports to be browser bundleable#10844

Draft
JennyPng wants to merge 3 commits into
microsoft:mainfrom
JennyPng:refactor-node-python-emitter
Draft

[python] Refactor Python emitter node imports to be browser bundleable#10844
JennyPng wants to merge 3 commits into
microsoft:mainfrom
JennyPng:refactor-node-python-emitter

Conversation

@JennyPng
Copy link
Copy Markdown
Member

@JennyPng JennyPng commented May 29, 2026 •
edited
Loading

  • after polyfill plugin was removed, Python emitter is no longer browser bundleable due to Node or filesystem imports
  • extract Node calls into a separate file and swap for browser vs. non-browser

@microsoft-github-policy-service microsoft-github-policy-service Bot added the emitter:client:python Issue for the Python client emitter: @typespec/http-client-python label May 29, 2026
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented May 29, 2026 •
edited
Loading

Open in StackBlitz

npm i https://pkg.pr.new/@typespec/http-client-python@10844

commit: c2a2391

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 29, 2026 •
edited
Loading

All changed packages have been documented.

  • ✅ @typespec/http-client-python
Show changes

@typespec/http-client-python - internal ✏️

Refactor Node usage in Python emitter to make it browser bundleable

@azure-sdk-automation
Copy link
Copy Markdown

azure-sdk-automation Bot commented May 29, 2026 •
edited
Loading

You can try these changes here

🛝 Playground 🌐 Website 🛝 VSCode Extension

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@iscai-msft iscai-msft Awaiting requested review from iscai-msft iscai-msft will be requested when the pull request is marked ready for review iscai-msft is a code owner

@tadelesh tadelesh Awaiting requested review from tadelesh tadelesh will be requested when the pull request is marked ready for review tadelesh is a code owner

@msyyc msyyc Awaiting requested review from msyyc msyyc will be requested when the pull request is marked ready for review msyyc is a code owner

@timotheeguerin timotheeguerin Awaiting requested review from timotheeguerin timotheeguerin will be requested when the pull request is marked ready for review timotheeguerin is a code owner

@lmazuel lmazuel Awaiting requested review from lmazuel lmazuel will be requested when the pull request is marked ready for review lmazuel is a code owner

@catalinaperalta catalinaperalta Awaiting requested review from catalinaperalta catalinaperalta will be requested when the pull request is marked ready for review catalinaperalta is a code owner

@ChenxiJiang333 ChenxiJiang333 Awaiting requested review from ChenxiJiang333 ChenxiJiang333 will be requested when the pull request is marked ready for review ChenxiJiang333 is a code owner

@l0lawrence l0lawrence Awaiting requested review from l0lawrence l0lawrence will be requested when the pull request is marked ready for review l0lawrence is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

emitter:client:python Issue for the Python client emitter: @typespec/http-client-python

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@JennyPng