Perf regression: bun-isolated/rescan wall +21% (cc938aa6..17793173; #1279 candidate)
Mantenedores costumam responder em até 1 dia
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 5/5
- Tempo estimado
- Mais de uma semana
- Facilidade para iniciantes
- 8/100
- Tipo de issue
- Bug
- Clareza
- Precisa de esclarecimento
- Status de atividade
- Ativa
- Stack de tecnologia
- bun, rust
- Domínio
- cli, performance
Direção de pesquisa
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.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
[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
- Linguagem predominante
- Rust
- Estrelas
- 8
- Forks
- 1
- Merge médio
- 16h 18min
- PRs com merge (30d)
- 587
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:pipenv priority:p1
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
SocketDev/socket-patch#1608 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
agent:triaged bug bughunt pm:vlt priority:p1
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 62/100
SocketDev/socket-patch#1601 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
agent:triaged bug bughunt pm:npm priority:p3 ux
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
SocketDev/socket-patch#1072 · 2 comentários ·
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 73/100
SocketDev/socket-patch#783 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
Vendored cargo warns cargo_multi_version_old_cargo ("declares no rust-version") when the root package inherits rust-version from [workspace.package]Talvez já em andamento @mikolalysenko assumiu há 4 dias. Abertaagent:triaged bug bughunt pm:cargo priority:p2
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 84/100
SocketDev/socket-patch#651 · 3 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
EmbarkStudios/cargo-deny#910 ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
tursodatabase/turso#10054 ·
Mantenedores costumam responder em até 1 dia
-
build: Update rust versionTalvez já em andamento Um pull request vinculado a esta issue está aberto ou já foi mesclado. Abertadependencies
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 67/100
ibm-granite-community/granite-cli#229 ·
Mantenedores costumam responder em até 1 dia
-
setTargetAtTime booked ahead of time starts its curve early (values blow up before start_time)Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
orottier/web-audio-api-rs#658 ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 84/100
aosama/astronomical#1318 ·
Mantenedores costumam responder em até 1 dia