Skip to content

[playground] Add uploadStandalonePackageAssets and extract uploadAssets helper to bundle-uploader#10775

Open
JennyPng wants to merge 1 commit into
mainfrom
feature/bundle-uploader-asset-upload
Open

[playground] Add uploadStandalonePackageAssets and extract uploadAssets helper to bundle-uploader#10775
JennyPng wants to merge 1 commit into
mainfrom
feature/bundle-uploader-asset-upload

Conversation

@JennyPng
Copy link
Copy Markdown
Member

@JennyPng JennyPng commented May 22, 2026

  • extract the binary asset wheel uploading logic (for the pygen wheel) so that it can be used externally by the typespec-azure repo's upload pipeline
  • needed to enable Python emitter in Azure playground, so that a pygen wheel can be uploaded at a discoverable index by the typespec-python emitter
  • for full context, see description of Add python emitter to azure playground Azure/typespec-azure#4486

@github-actions
Copy link
Copy Markdown
Contributor

No changes needing a change description found.

@azure-sdk-automation
Copy link
Copy Markdown

azure-sdk-automation Bot commented May 22, 2026

You can try these changes here

🛝 Playground 🌐 Website 🛝 VSCode Extension

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the @typespec/bundle-uploader internal helper library to support uploading standalone binary assets (notably the Python pygen wheel) to blob storage, enabling the Azure playground to discover those assets via an index.

Changes:

  • Extracted asset globbing/upload logic into a shared uploadAssets helper.
  • Refactored uploadPlaygroundAssets to reuse the new helper.
  • Added a new exported API uploadStandalonePackageAssets to upload only binary assets without bundling.

Comment thread packages/bundle-uploader/src/index.ts
Comment thread packages/bundle-uploader/src/index.ts
Comment thread packages/bundle-uploader/src/index.ts
@JennyPng JennyPng marked this pull request as ready for review May 26, 2026 21:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants