Skip to content

feat(cli): add explicit self-update command and update notices - #7753

Merged
waleedlatif1 merged 3 commits into
stagingfrom
feat/cli-update
Sep 11, 2026
Merged

waleedlatif1 merged 3 commits into
stagingfrom
feat/cli-update

Conversation

@TheodoreSpeaks

@TheodoreSpeaks TheodoreSpeaks commented Sep 11, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Add sim update for verified global npm, pnpm, Bun, and Yarn Classic installations, preserving the release channel, refusing downgrades, and surfacing installation failures.
  • Point daily update notices to sim update. Normal commands continue without installing updates.

Type of Change

  • Feature

Testing

CLI suite: 1,087 passed, 1 skipped. Build, type-check, repository lint, block-registry validation, full repository audits, and docs-manifest check passed. Manually verified local command help and the update notice.

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@TheodoreSpeaks
TheodoreSpeaks requested a review from a team as a code owner September 11, 2026 01:13
@vercel

vercel Bot commented Sep 11, 2026

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated
docs Skipped Skipped Sep 11, 2026 1:52am UTC

Request Review

@greptile-apps

greptile-apps Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

The PR appears safe to merge; no actionable new failure remains, and all previous findings are resolved.

Summary

  • Detects and verifies the active global npm, pnpm, Bun, or Yarn Classic installation.
  • Resolves and validates the target release before installing an exact version.
  • Refuses downgrades and concurrent updates while sanitizing expected CLI failures.
  • Adds command, unit, bundled-process, and documentation coverage.
  • The latest revision preserves the original registry or installer failure when lock cleanup also fails.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart TD
  Command[sim update] --> Detect[Detect selected package manager]
  Detect --> Verify[Verify active global installation]
  Verify --> Lock[Acquire installation lock]
  Lock --> Resolve[Resolve channel release]
  Resolve --> Compare{Compare versions}
  Compare -->|Older| Refuse[Refuse downgrade]
  Compare -->|Equivalent| Current[Report already current]
  Compare -->|Newer| Install[Install exact version]
  Install --> Validate[Verify installed manifest]
  Refuse --> Cleanup[Release lock]
  Current --> Cleanup
  Validate --> Cleanup
  Cleanup --> Result[Report success or preserved failure]
Loading

Reviews (3) · Last reviewed commit: "fix(cli): preserve update failures durin..."

Comment thread packages/sim-cli/src/update/install.ts Outdated
Comment thread packages/sim-cli/src/update/install.ts Outdated
@TheodoreSpeaks

Copy link
Copy Markdown
Collaborator Author

@greptile

@TheodoreSpeaks

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@TheodoreSpeaks I have started the AI code review. It will take a few minutes to complete.

Comment thread packages/sim-cli/src/update/install.ts
@TheodoreSpeaks

Copy link
Copy Markdown
Collaborator Author

@greptile

@TheodoreSpeaks

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@TheodoreSpeaks I have started the AI code review. It will take a few minutes to complete.

@waleedlatif1
waleedlatif1 merged commit f45cab0 into staging Sep 11, 2026
34 checks passed
@waleedlatif1
waleedlatif1 deleted the feat/cli-update branch September 11, 2026 01:59
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