Skip to content

feat(vscode): publish release VSIX assets - #171

Merged
peternhale merged 4 commits into
mainfrom
sm/publish-release-vsix
Aug 14, 2026
Merged

feat(vscode): publish release VSIX assets#171
peternhale merged 4 commits into
mainfrom
sm/publish-release-vsix

Conversation

@mshanemc

@mshanemcmshanemc commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add separate reusable workflows that publish every VSIX attached to a caller repository release to VS Code Marketplace or Open VSX.
  • Share release download, recursive VSIX discovery, and prerelease detection in a composite action.
  • Preserve the GitHub release pre-release state, skip duplicate marketplace versions, and support safe workflow-level dry runs.
  • Document both release-asset publisher workflows.

Verification

  • actionlint .github/workflows/vscode-publish-release-vsix.yml .github/workflows/openvsx-publish-release-vsix.yml
  • YAML parsing and Prettier checks for the new action and workflows.
  • Downloaded and inspected the 17 VSIX assets from forcedotcom/salesforcedx-vscode release v67.10.0; confirmed its stable release state and flat asset layout.
  • Tested recursive nested VSIX discovery and both dry-run paths; neither invokes a publish CLI, and the Open VSX token is not logged.

What issues does this PR fix or reference?

No related issues or discussions.

[skip-validate-pr]

@mshanemc
mshanemc marked this pull request as ready for review August 14, 2026 04:54
@mshanemc
mshanemc requested a review from a team as a code ownerAugust 14, 2026 04:54
required: true

outputs:
pre-release:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The vsix has a property within the manifest that marks the artifact as pre-release. Providing a flag to influence which bucket it lands in is ineffective and may cause a failure if there is a mistmatch.

If the goal of this workflow is to simply publish it might be best to follow what the vsix says rather than imposing a bucket.

@peternhale
peternhale merged commit 946341a into mainAug 14, 2026
3 checks passed
Sign up for freeto 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

@mshanemc@peternhale