Global mode misses every Bun global package when BUN_INSTALL_BIN or BUN_INSTALL_GLOBAL_DIR is set: scan -g reports success with nothing found, get -g / vex -g patch and attest nothing
Mantenedores costumam responder em até 1 dia
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 4/5
- Tempo estimado
- 3-5 dias
- Facilidade para iniciantes
- 62/100
Direção de pesquisa
Start in crates/socket-patch-core/src/crawlers/npm_crawler.rs at parse_bun_bin_output (around line 583) and get_global_node_modules_paths (around line 1154), then reproduce the BUN_INSTALL_BIN and BUN_INSTALL_GLOBAL_DIR cases from the issue. Trace how Bun's global directory is selected and verify that scan, get, vex, and rollback find the installed package; an undetectable directory should not produce a silent empty success.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
[agent] Found by the scheduled Bun bug-hunt routine (ledger #306).
Summary
socket-patch doesn't ask Bun where its global packages are. It runs bun pm bin -g, which returns Bun's global bin directory, and then assumes the packages sit at <bin>/../install/global/node_modules. That only holds for Bun's default layout. Bun lets you move each directory on its own, through the documented BUN_INSTALL_BIN and BUN_INSTALL_GLOBAL_DIR environment variables (or globalBinDir / globalDir in bunfig):
- With
BUN_INSTALL_BIN=~/.local/bin(a common way to put Bun's global bins on an existing PATH dir),bun pm bin -gprints~/.local/bin. socket-patch then looks in~/.local/install/global/node_modules, which doesn't exist. The real packages are still in$BUN_INSTALL/install/global/node_modules. - With
BUN_INSTALL_GLOBAL_DIR=<dir>, the packages live in<dir>/node_modules, butbun pm bin -gstill prints$BUN_INSTALL/bin, so socket-patch looks in the default location and finds nothing.
get_global_node_modules_paths drops a non-existent directory without a word (p.is_dir()), so in both cases Bun's globals simply vanish from global mode.
Impact
It's a silent miss. scan -g exits 0 with status: success and no Bun packages, and scan -g --mode agent exits 0 with "No patches available for installed packages." So a user, or a CI job that patches globally installed CLIs, is told there's nothing to patch while a vulnerable global tool stays unpatched. get -g does fail (exit 1, "matched no installed package"), and vex -g can't attest the patch. The only workaround is to pass --global-prefix explicitly.
This isn't a regression: release 4.0.0 behaves the same way.
Repro (Linux; same on macOS and Windows)
The patch API is a local mock (SOCKET_PROXY_URL=http://127.0.0.1:8787) serving a free patch for pkg:npm/[email protected] (bin/semver.js).
export HOME=$(mktemp -d) BUN_INSTALL=$HOME/.bun PATH=$HOME/.bun/bin:$PATH # bun 1.4.2 copied into $BUN_INSTALL/bin
export BUN_INSTALL_BIN=$HOME/.local/bin # or: BUN_INSTALL_GLOBAL_DIR=$HOME/gdir
bun add -g [email protected]
bun pm bin -g # $HOME/.local/bin
ls $BUN_INSTALL/install/global/node_modules # semver (BUN_INSTALL_GLOBAL_DIR case: $HOME/gdir/node_modules/semver)
socket-patch scan -g --json # status success, packages [] <- bug
socket-patch scan -g --mode agent # exit 0, "No patches available for installed packages." <- bug
socket-patch get -g 11111111-1111-4111-8111-111111111111 # exit 1, "targeted manifest patch matched no installed package"
socket-patch scan --global-prefix $BUN_INSTALL/install/global/node_modules --mode agent # patched, works
Without either variable, the same steps find the package, patch it, and vex -g / rollback -g (byte-exact) pass.
Expected vs actual
- Expected: CLI_CONTRACT.md documents
--global/-gas "Operate on globally-installed packages", with--global-prefixdefaulting to(auto). Auto-detection should find the directory Bun actually installs global packages into, the same onebun add -gwrites andbun pm ls -greports (<globalDir> node_modules (N installed)on 1.4.2). If it can't determine that directory, it should say so instead of returning a clean, empty scan. - Actual: the global node_modules path is guessed from the bin dir, so any non-default
BUN_INSTALL_BINorBUN_INSTALL_GLOBAL_DIRmakes every Bun global package disappear, with exit 0.
Matrix (main 2463257)
bun add -g [email protected] [email protected], then scan -g report / scan -g --mode agent / get -g / vex -g:
| OS | Bun | default layout | BUN_INSTALL_BIN set |
BUN_INSTALL_GLOBAL_DIR set |
|---|---|---|---|---|
| Linux (sandbox + ubuntu-latest) | 1.1.45, 1.2.23, 1.3.14, 1.4.2 | pass | FAIL | FAIL |
| macos-latest | 1.1.45, 1.2.23, 1.3.14, 1.4.2 | pass | FAIL | FAIL |
| windows-latest | 1.3.14, 1.4.2 | pass | FAIL | FAIL |
| windows-latest | 1.1.45, 1.2.23 | blocked (bun add -g itself failed in the probe's space + unicode temp dir) |
blocked | blocked |
Release 4.0.0 (Linux, Bun 1.4.2, BUN_INSTALL_BIN): same failure. It's not a regression.
The paths contained a space and ü on every runner. scan -g --mode hosted refused correctly (exit 2) in every cell.
Suspect code
crates/socket-patch-core/src/crawlers/npm_crawler.rs:583(parse_bun_bin_output) derives<bin>/../install/global/node_modulesfrom thebun pm bin -goutput.crates/socket-patch-core/src/crawlers/npm_crawler.rs:1154adds that path only if it exists, so a wrong guess is dropped silently.
Possible fixes: honour BUN_INSTALL_GLOBAL_DIR, then $BUN_INSTALL/install/global, then ~/.bun/install/global, the way Bun resolves it. Or parse the directory from the first line of bun pm ls -g. Also warn when a detected package manager's global dir can't be found.
Open PR #442 changes how this probe is spawned (GlobalProbeRunner), but it leaves this path derivation unchanged.
Probe run
https://github.com/SocketDev/socket-patch/actions/runs/36830650075 (3 OS × Bun 1.1.45 / 1.2.23 / 1.3.14 / 1.4.2; layouts: default, BUN_INSTALL_BIN, BUN_INSTALL_GLOBAL_DIR, npm-installed bun)
- Linguagem predominante
- Rust
- Estrelas
- 8
- Forks
- 0
- Merge médio
- 18h 4min
- PRs com merge (30d)
- 70
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Sem modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de SocketDev/socket-patch
-
agent:triaged bug bughunt pm:composer priority:p2
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 90/100
SocketDev/socket-patch#515 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
agent:triaged bug bughunt pm:npm priority:p1
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
SocketDev/socket-patch#464 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
agent:triaged bug bughunt pm:npm priority:p1
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
SocketDev/socket-patch#433 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
agent:triaged bug bughunt pm:uv priority:p1
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
SocketDev/socket-patch#408 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
agent:triaged bug bughunt pm:yarn-berry priority:p1
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
SocketDev/socket-patch#370 · 2 comentários ·
Mantenedores costumam responder em até 1 dia
Todas as issues de SocketDev/socket-patch
Issues semelhantes
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
Mantenedores costumam responder em até 1 dia
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 78/100
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 86/100
dani-garcia/vaultwarden#7801 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 90/100
boxlite-ai/boxlite#1814 ·
Mantenedores costumam responder em até 1 dia