Skip to content

feat(cli): add toolkit upgrade bridge - #1569

Open
BYK wants to merge 19 commits into
mainfrom
feat/toolkit-bridge-upgrade
Open

feat(cli): add toolkit upgrade bridge#1569
BYK wants to merge 19 commits into
mainfrom
feat/toolkit-bridge-upgrade

Conversation

@BYK

@BYK BYK commented Sep 9, 2026

Copy link
Copy Markdown
Member

Summary

  • check Toolkit before the legacy CLI repositories for stable, nightly, delta, prefetch, and changelog requests
  • fall back only on HTTP 404 and retain one selected source through the complete upgrade operation
  • mirror the ordered source contract in the shell installer and enforce parity in CI

Closes #1244

Validation

  • pnpm run lint
  • pnpm run typecheck
  • pnpm run check:deps
  • pnpm run check:errors
  • pnpm run check:patches
  • pnpm run check:stale-refs
  • pnpm run check:upgrade-sources
  • shellcheck packages/cli/install
  • focused bridge matrix: 458 tests passed
  • isolated rerun of the 10 full-suite failure files: 252 tests passed; the full coverage run had unrelated temp-directory/SQLite contention and one load timeout

@vercel

vercel Bot commented Sep 9, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
cli Ready Ready Preview Sep 10, 2026 10:44am UTC
1 Skipped Deployment
Project Deployment Actions Updated
sentry-local Skipped Skipped Sep 10, 2026 10:44am UTC

Request Review

@cursor cursor Bot left a comment

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.

Stale Bugbot comment from a previous run.

Comment thread packages/cli/install Outdated
Comment thread packages/cli/src/commands/cli/upgrade.ts
@BYK BYK removed the risk: high label Sep 9, 2026

@cursor cursor Bot left a comment

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.

Stale Bugbot comment from a previous run.

Comment thread packages/cli/src/lib/upgrade.ts Outdated
Comment thread packages/cli/src/lib/upgrade.ts
Comment thread packages/cli/src/lib/binary.ts

@cursor cursor Bot left a comment

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.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 5ffa5cd. Configure here.

Comment thread packages/cli/src/lib/upgrade.ts
Comment thread packages/cli/test/commands/cli/upgrade.test.ts Fixed
Comment thread packages/cli/test/commands/cli/upgrade.test.ts Fixed
Comment thread packages/cli/src/lib/ghcr.ts
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.

toolkit merge: bridge CLI release + dual-source upgrade compatibility

2 participants