Perf regression: bun-isolated/rescan wall +21% (cc938aa6..17793173; #1279 candidate)
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 8/100
- Tipo di issue
- Bug
- Chiarezza
- Da chiarire
- Stato di attività
- Attiva
- Stack tecnologico
- bun, rust
- Ambito
- cli, performance
Direzione di ricerca
This is a performance investigation, not a patch request. The bisect points at commit dcdc1043 (#1279, removal of vendoring code) and 4c800ac5 (#1370), with the rest of the regression in eca5d247..17793173. Start by rebuilding the perf profile and running socket-patch-bench compare on bun-isolated/rescan as given in the repro. Done means the wall-time regression is traced to a specific change and fixed, confirmed by a CI that excludes zero.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
[agent] Bench: confirmed weekly regression in bun-isolated/rescan (and a smaller one in bun-isolated/hosted / bun/hosted).
Base cc938aa6 (main, 2026-10-09) → head 657fce2b (main, 2026-10-15). Same machine, interleaved pairs. Daily A/B (e76cb9b3 → 657fce2b) is flat, so this is drift that is already on main.
| scenario | pairs | base wall | head wall | Δ wall [95% CI] | Δ CPU | Δ peak RSS | requests | verdict |
|---|---|---|---|---|---|---|---|---|
bun-isolated/rescan (round 1) |
15+10 | 177.3 ms | 212.2 ms | +22.9% [+12.7, +30.4] | +16.0% | +3.9% | 127 | regression |
bun-isolated/rescan (confirm) |
25+15 | 205.0 ms | 244.7 ms | +21.3% [+12.7, +32.7] | +12.1% | +5.8% | 127 | regression |
bun-isolated/hosted (confirm) |
25+15 | 243.6 ms | 281.1 ms | +15.6% [+8.4, +22.9] | +14.0% | +8.0% | 127 | regression |
bun/hosted (confirm) |
25+15 | 212.9 ms | 234.5 ms | +12.9% [+9.7, +16.5] | +7.0% | +4.3% | 127 | regression |
Request counts are unchanged. The 2026-10-12 daily run already saw bun-isolated/rescan +15% (f3c6313a..17793173); a large bun speedup earlier in that week hid it in the previous weekly windows.
Bisect (3 builds over the 151 commits cc938aa6..17793173, bun-isolated/rescan, 15 pairs each vs cc938aa6):
| build | Δ wall |
|---|---|
1438b1a7 (#1356) |
+5.8% [-0.5, +16.2] ≈ |
a43cf327 (#1332) |
+12.4% [+6.5, +23.2] |
eca5d247 (#1330) |
+11.5% [+2.1, +20.6] |
657fce2b (today) |
+21.3% |
So about half lands in 1438b1a7..a43cf327 (19 commits). The most plausible one there is dcdc1043 (#1279, "remove obsolete vendoring machinery and duplicate crate code"), with 4c800ac5 (#1370) as the other candidate that touches shared npm code. The rest lands after eca5d247, in eca5d247..17793173. Not narrowed further: the bisect budget is spent.
Repro
export CARGO_PROFILE_PERF_INHERITS=release CARGO_PROFILE_PERF_LTO=thin CARGO_PROFILE_PERF_STRIP=none
git worktree add /tmp/base cc938aa6 && (cd /tmp/base && CARGO_TARGET_DIR=/tmp/tb cargo build --locked --profile perf -p socket-patch-cli)
cargo build --locked --profile perf -p socket-patch-cli -p socket-patch-bench
target/perf/socket-patch-bench compare --base /tmp/tb/perf/socket-patch --head target/perf/socket-patch -f '^bun-isolated/' -f '^bun/hosted$' --runs 25
Runner: 4 vCPU, Intel(R) Xeon(R) Processor @ 2.80GHz (cloud sandbox). A/A on the same runner was clean (npm/hosted -3.9%, pip/hosted -3.1%, cargo/hosted +5.1%, all ≈).
Progress log: https://github.com/SocketDev/socket-patch/discussions/575
Generated by Claude Code
- Lingua principale
- Rust
- Stelle
- 8
- Fork
- 1
- Merge medio
- 16h 16m
- PR unite (30g)
- 582
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
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.
Altre issue di SocketDev/socket-patch
-
agent:triaged bug bughunt pm:pipenv priority:p1
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
SocketDev/socket-patch#1608 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
agent:triaged bug bughunt pm:vlt priority:p1
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
SocketDev/socket-patch#1601 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
agent:triaged bug bughunt pm:npm priority:p3 ux
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
SocketDev/socket-patch#1072 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
agent:triaged bug bughunt pm:yarn-berry priority:p1
Difficoltà 2/5 1-3 ore Idoneità per principianti 73/100
SocketDev/socket-patch#783 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Vendored cargo warns cargo_multi_version_old_cargo ("declares no rust-version") when the root package inherits rust-version from [workspace.package]Forse già presa @mikolalysenko l’ha presa 7 giorni fa. Apertaagent:triaged bug bughunt pm:cargo priority:p2
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
SocketDev/socket-patch#651 · 3 commenti ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di SocketDev/socket-patch
Issue simili
-
`[daemons]` fails to resolve standalone file tasks by their display name (`runs unknown task`)Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 64/100
I maintainer di solito rispondono entro 3 giorni
-
[BUG] Filename sequence expansion (`+`) panics on numeric values exceeding `i32::MAX`Forse già presa @muntasir-396 l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
CCExtractor/ccextractor#2402 ·
I maintainer di solito rispondono entro 1 giorno
-
Nushell launcher failure.Apertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100