Perf regression: bun-isolated/rescan wall +21% (cc938aa6..17793173; #1279 candidate)
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức phù hợp với người mới
- 8/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Cần làm rõ
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- bun, rust
- Lĩnh vực
- cli, performance
Hướng nghiên cứu
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.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
[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
- Ngôn ngữ chính
- Rust
- Star
- 8
- Fork
- 1
- Merge trung bình
- 16 giờ 22 phút
- Pull request đã merge (30 ngày)
- 583
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của SocketDev/socket-patch
-
agent:triaged bug bughunt pm:pipenv priority:p1
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
SocketDev/socket-patch#1608 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
agent:triaged bug bughunt pm:vlt priority:p1
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
SocketDev/socket-patch#1601 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
agent:triaged bug bughunt pm:npm priority:p3 ux
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
SocketDev/socket-patch#1072 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
agent:triaged bug bughunt pm:yarn-berry priority:p1
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 73/100
SocketDev/socket-patch#783 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Vendored cargo warns cargo_multi_version_old_cargo ("declares no rust-version") when the root package inherits rust-version from [workspace.package]Có thể đã có người làm @mikolalysenko đã nhận 6 ngày trước. Đang mởagent:triaged bug bughunt pm:cargo priority:p2
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
SocketDev/socket-patch#651 · 3 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của SocketDev/socket-patch
Issue tương tự
-
area:gc-repack-output bug factory:dogfood factory:triaged priority:p3 type:bug type:ux
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
gitbutlerapp/grit#1161 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Optimize the quick add featureĐang mởfeature
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100
-
priority: low
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
storytold/photocraft#7520 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
Maintainer thường phản hồi trong vòng 1 ngày