Skip to content

fix: remove PATCH+1 version heuristic from VSIX builds - #85

Merged
Dongbum(DB) Lee (Dongbumlee) merged 1 commit into
developfrom
fix/vsix-version-sync
Apr 15, 2026
Merged

Dongbum(DB) Lee (Dongbumlee) merged 1 commit into
developfrom
fix/vsix-version-sync

Conversation

@Dongbumlee

Copy link
Copy Markdown

Problem

The VSIX version sync logic in ci.yml and staging.yml used a PATCH+1 heuristic that auto-bumped the version when HEAD was not an exact tag match. This caused VSIX 0.1.8 to be published to the Marketplace before 0.1.8 was officially released (PyPI is still at 0.1.7).

How each workflow now derives the VSIX version

Workflow Source Example
ci.yml Committed package.json (dry-run only) 0.1.7 as-is
staging.yml Branch name: release/v0.1.8 -> 0.1.8 Explicit, no guessing
release.yml Tag name: v0.1.8 -> 0.1.8 Already correct (no change)

Changes

  • staging.yml: Replace tag-based PATCH+1 with branch name derivation (GITHUB_REF_NAME)
  • ci.yml: Remove version sync entirely - CI only validates the VSIX builds, never publishes
  • release.yml: No change (already uses tag-direct logic)

The old logic in ci.yml and staging.yml auto-bumped PATCH+1 when HEAD
was not an exact tag match, causing VSIX versions to leak ahead of the
actual release (e.g. 0.1.8 published to Marketplace before 0.1.8 was
planned).

- staging.yml: derive version from branch name (release/v0.1.8 → 0.1.8)
- ci.yml: use committed package.json as-is (dry-run only, no publish)
- release.yml: already uses tag-direct logic (no change needed)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@Dongbumlee
Dongbum(DB) Lee (Dongbumlee) merged commit facd4cd into develop Apr 15, 2026
12 checks passed
@placerda
Paulo Lacerda (placerda) deleted the fix/vsix-version-sync branch June 9, 2026 20:00
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.

1 participant