CommandCodeAI / CommandCodeAI/command-code
`cmd update` reports success but does not update the installed binary (when installed via bun)
Nessuno ha ancora preso questa issue.
- Lingua principale
- Nessun dato sulla lingua
- Stelle
- 4k
- Fork
- 350
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
Summary
cmd update prints "✔ Updated to v1.17.0" but cmd --version stays on the old version. No error is shown. The self-updater assumes npm
global install and never touches bun-installed binaries, so the update silently no-ops for bun users.
Expected Behavior
cmd updateupgrades the binary that is actually on PATH.- After update,
cmd --versionshows the new version. - If the binary did not change, the command reports failure (non-zero exit) with manual instructions — never a false success.
--check-onlybehavior is preserved.
Actual Behavior
cmd updatealways runsnpm i -g command-code(hardcodedupdateCommandinupdateAction2→runCliUpdateCommand).- On a bun install (
~/.bun/bin/cmd→ bun global node_modules), the new version lands in npm's global dir as an orphan copy (nocmdbin link), while the PATH binary is untouched. - Success is reported from the install command's exit code, not from the on-disk version changing.
Steps to reproduce the issue
- Install via bun:
bun i -g command-code - Run
cmd update→ prints "✔ Updated to v1.17.0. Run cmd to start using the new version." - Run
cmd --version→ still old version (e.g. 1.14.0) - Confirm orphan copy:
ls /opt/homebrew/lib/node_modules/command-code(npm prefix) vsls ~/.bun/install/global/node_modules/command-code(bun) — versions differ.
Command Code Version
1.14.0
Operating System
macOS
Terminal/IDE
ghostty
Shell
zsh
Session file (optional)
No response
Fix prompt (optional)
Fix the self-update logic so it updates the binary that is really on PATH:
- Detect install method before updating. Inspect the resolved binary path (
process.argv[1]; e.g..bun/install/global/...,
/opt/homebrew/bin, npx shim) and/or probe package-manager globals forcommand-code(bun:~/.bun/install/global, npm:npm root -g,
brew:brew list). - Update via the matching manager:
- bun →
bun i -g command-code@latest - npm →
npm i -g command-code@latest - brew →
brew upgrade command-code(if available) - unknown → current behavior + warning.
- bun →
- Verify after update. Compare the version of the resolved install before/after (package.json or
--version). Report success only
if the PATH binary version actually changed. - On stale binary → print failure with manual instructions, exit non-zero.
- Keep
--check-onlysemantics; keep telemetry, addinstall_methodfield if cheap.
Acceptance criteria:
- bun install:
cmd updateupgrades the bun copy;cmd --versionshows new version; no orphan npm copy created. - npm install: behavior unchanged.
- Update command succeeds but version unchanged → failure, non-zero exit, manual instructions.
- Unit tests cover: method detection (bun/npm/brew/path shims), command selection per method, post-update verification (success + stale
cases).
Additional context
OS: macOS Tahoe 26.5.2
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Traccia updateAction2 e runCliUpdateCommand, iniziando dall'updateCommand hardcoded e dal percorso risolto a partire da process.argv[1]. Usa i casi di unit test indicati per coprire bun, npm, brew e gli shim del percorso, quindi verifica che il binario risolto cambi dopo un aggiornamento e che gli aggiornamenti obsoleti falliscano senza modificare il comportamento di --check-only.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- bun
- Ambito
- cli
- Tipo di issue
- Bug
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Tranquilla
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 52/100