Perf regression: bun-isolated/rescan wall +21% (cc938aa6..17793173; #1279 candidate)
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Anfängerfreundlichkeit
- 8/100
- Issue-Typ
- Bug
- Klarheit
- Muss geklärt werden
- Aktivitätsstatus
- Aktiv
- Tech-Stack
- bun, rust
- Bereich
- cli, performance
Rechercherichtung
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.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
[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
- Vorherrschende Sprache
- Rust
- Sterne
- 8
- Forks
- 1
- Ø Merge
- 16 Std. 19 Min.
- Gemergte PRs (30 T.)
- 586
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus SocketDev/socket-patch
-
agent:triaged bug bughunt pm:pipenv priority:p1
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
SocketDev/socket-patch#1608 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
agent:triaged bug bughunt pm:vlt priority:p1
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
SocketDev/socket-patch#1601 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
agent:triaged bug bughunt pm:npm priority:p3 ux
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
SocketDev/socket-patch#1072 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
agent:triaged bug bughunt pm:yarn-berry priority:p1
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 73/100
SocketDev/socket-patch#783 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Vendored cargo warns cargo_multi_version_old_cargo ("declares no rust-version") when the root package inherits rust-version from [workspace.package]Evtl. vergeben @mikolalysenko hat das vor 5 Tagen übernommen. Offenagent:triaged bug bughunt pm:cargo priority:p2
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100
SocketDev/socket-patch#651 · 3 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in SocketDev/socket-patch
Ähnliche Issues
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 88/100
pop-os/cosmic-comp#2956 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
Maintainer antworten meist innerhalb von 1 Tag
-
area:release criticality:p3 documentation
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
registrystack/registry-stack#2085 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 85/100
libp2p/rust-libp2p#6694 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
EmbarkStudios/cargo-deny#910 ·