CommandCodeAI / CommandCodeAI/command-code
`cmd update` reports success but does not update the installed binary (when installed via bun)
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Sin datos de lenguaje
- Estrellas
- 4k
- Forks
- 350
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
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
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Rastrea updateAction2 y runCliUpdateCommand, comenzando con el updateCommand codificado y la ruta resuelta a partir de process.argv[1]. Usa los casos de pruebas unitarias indicados para cubrir bun, npm, brew y los shims de ruta; después, verifica que el binario resuelto cambie tras una actualización y que las actualizaciones obsoletas fallen sin cambiar el comportamiento de --check-only.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- bun
- Área
- cli
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Tranquilo
- Claridad
- Bastante claro
- Aptitud para principiantes
- 52/100