Skip to content

fix: use compiler version for standalone CLI distribution versioning [skip chg] - #11157

Open
timotheeguerin with Copilot wants to merge 7 commits into
mainfrom
copilot/fix-cli-package-version-sync
Open

fix: use compiler version for standalone CLI distribution versioning [skip chg]#11157
timotheeguerin with Copilot wants to merge 7 commits into
mainfrom
copilot/fix-cli-package-version-sync

Conversation

Copilot AI commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

@typespec/standalone-cli follows a different version scheme (0.83.0) than @typespec/compiler (1.13.0). The publish pipeline was writing the standalone package version to latest.txt and blob storage paths, while tsp --version reports the compiler version — causing Scoop/winget/Homebrew listings to show a version users never see.

Changes

  • eng/tsp-core/pipelines/jobs/cli/publish-artifacts.yml: Changed workingDirectory for the version-reading step from packages/standalonepackages/compiler, so latest.txt and blob storage paths use the compiler version consistently with tsp --version output.

Co-authored-by: timotheeguerin <1031227+timotheeguerin@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix standalone cli package version mismatch fix: use compiler version for standalone CLI distribution versioning Jul 2, 2026
Copilot AI requested a review from timotheeguerin July 2, 2026 18:27
@timotheeguerin

Copy link
Copy Markdown
Member

@copilot wondering if instead we should just make sure the version match at all time and add standlone cli to the same version policy

…version to 1.13.0

Co-authored-by: timotheeguerin <1031227+timotheeguerin@users.noreply.github.com>

Copilot AI commented Jul 2, 2026

Copy link
Copy Markdown
Contributor Author

@copilot wondering if instead we should just make sure the version match at all time and add standlone cli to the same version policy

Done. Moved @typespec/standalone-cli into the typespec-stable lockstep policy (commit fix: move standalone-cli to typespec-stable version policy and align version to 1.13.0), bumped its version to 1.13.0, and reverted the pipeline change since the versions now stay in sync automatically.

@microsoft-github-policy-service microsoft-github-policy-service Bot added the stale Mark a PR that hasn't been recently updated and will be closed. label Jul 17, 2026
@timotheeguerin

Copy link
Copy Markdown
Member

@copilot rebase

…h main

Co-authored-by: timotheeguerin <1031227+timotheeguerin@users.noreply.github.com>

Copilot AI commented Jul 29, 2026

Copy link
Copy Markdown
Contributor Author

@copilot rebase

Done — the branch is now synced with latest main (version bumped to 1.14.0 to match the current compiler, and @typespec/http-specs/@typespec/graphql added to typespec-preview to pick up those upstream changes).

@timotheeguerin

Copy link
Copy Markdown
Member

@copilot there is sstill conflict, pull main and merge them

Co-authored-by: timotheeguerin <1031227+timotheeguerin@users.noreply.github.com>

Copilot AI commented Jul 29, 2026

Copy link
Copy Markdown
Contributor Author

@copilot there is sstill conflict, pull main and merge them

Done — merged main into the branch and resolved the two conflicts:

  • .chronus/config.yaml: kept @typespec/standalone-cli in typespec-stable (removed the duplicate entry that main had added to typespec-preview)
  • packages/standalone/package.json: kept version 1.14.0 to stay aligned with the compiler

@timotheeguerin
timotheeguerin marked this pull request as ready for review July 29, 2026 18:31
Comment thread .chronus/config.yaml

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@copilot because now the compiler is bundled in the standalone cli it does make sense that it is exactly the same version, should we do things differently to make sure hotfix are applied to both?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Since @typespec/standalone-cli is now in the typespec-stable lockstep policy alongside @typespec/compiler, any version bump — including hotfix patch releases — will automatically apply to both packages at the same time. No additional changes are needed; the lockstep policy already guarantees they stay in sync for every release type.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@copilot no the polcy only lock minor, they would diverge on patch. We probably just want to ignore the version of standalone cli (remove from versioning all together

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done in 4cdde5b. Removed @typespec/standalone-cli from the typespec-stable version policy entirely — its version won't be managed by Chronus at all, so it can never diverge from the compiler on patch releases.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@copilot we also ened to update the version retrieval logic to fetch it from the compiler package.json...

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done in a9f70cb. The pipeline now reads the version from packages/compiler/package.json so latest.txt and blob storage paths always reflect the compiler version.

@azure-sdk-automation

Copy link
Copy Markdown

You can try these changes here

🛝 Playground 🌐 Website 🛝 VSCode Extension

@timotheeguerin timotheeguerin changed the title fix: use compiler version for standalone CLI distribution versioning fix: use compiler version for standalone CLI distribution versioning [skip chg] Jul 29, 2026
@@ -0,0 +1,7 @@
---
changeKind: fix

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
changeKind: fix
changeKind: internal

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Applied in 4cdde5b.

…internal

Co-authored-by: timotheeguerin <1031227+timotheeguerin@users.noreply.github.com>
Co-authored-by: timotheeguerin <1031227+timotheeguerin@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

eng stale Mark a PR that hasn't been recently updated and will be closed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

standalone cli package version is out of sync

2 participants