Benchmark progress: socket-patch scan #575
Mikola Lysenko (mikolalysenko)
started this conversation in
General
Replies: 3 comments
|
[agent] Bench: mirror test. The bench/ledger workflow posted this comment. Daily runs add their summaries after it. Generated by Claude Code |
0 replies
|
[agent] Bench: run 2026-10-03, main
Generated by Claude Code |
0 replies
|
[agent] Janitor: state drift. The body still lists #579 as an open regression ( Generated by Claude Code |
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
[agent] Bench: progress log for the
socket-patch scanbenchmark suite (crates/socket-patch-bench,.github/workflows/bench.yml). The daily benchmark steward rewrites this body on every run and posts one comment per run. Machine-readable history is inhistory.jsonon thebench/ledgerbranch. The first run (2026-10-02) was logged in #580 and backfilled there.Last run
045d7ec7(Bound patch API connects and stalled reads, Bound patch API connects and stalled reads (#570) #581)main(Add ascanbenchmark suite and a CI performance gate #485 has landed)compare.1169ae68(the previous run's main), 9 pairs + confirm. No regressions across 37 comparable scenarios.yarn-berry/*has no comparable base, because pins now also rewritepackage.json(Fix yarn berry hosted pin leaking npm auth (#404) #465, intentional).2390c5ffpredates the v5 output (v5 prerelease: scan → vex → vendor workflow, hosted by default #277), so the base is2463257a(v5 prerelease: scan → vex → vendor workflow, hosted by default #277). 7 pairs + confirm. Only the known Fix Bun/vlt bundled copies left unpatched (#469, #471) #472 regressions remain: bun/* ~2.2x and vlt/* ~2.4–2.5x (Perf regression: bun/hosted wall +110% (1169ae68, #472) #578, Perf regression: vlt/hosted wall +127% (1169ae68, #472) #579). The other 33 comparable scenarios are unchanged.203e092b(Fix yarn berry hosted pin leaking npm auth (#404) #465) and0ac5b91a(Fix PyPI vendored to hosted takeover being refused (#328) #503) failedComparebecause Fix yarn berry hosted pin leaking npm auth (#404) #465 changed yarn-berry's rewritten files before the fixture caught up. Stream ZIP member hashes during vendored verification #587 (b8bf049d) aligned the fixture, and the run on045d7ec7is green.Scoreboard
Day: wall ratio vs
1169ae68. Week: vs2463257a(#277). ms/pkg median across*/hostedis 0.1275.npm/hostednpm/rescanpnpm/hostedpnpm/rescanyarn-classic/hostedyarn-classic/rescanyarn-berry/hostedyarn-berry/rescanbun/hostedbun/rescanvlt/hostedvlt/rescanpip/hostedpip/rescanuv/hosteduv/rescanpylock/hostedpylock/rescanpoetry/hostedpoetry/rescanpipenv/hostedpipenv/rescanpdm/hostedpdm/rescanbundler/hostedbundler/rescancomposer/hostedcomposer/rescancargo/hostedcargo/rescangolang/hostedgolang/rescannuget/hostednuget/rescanmaven/hostedmaven/rescannpm/dry-runnpm/public-proxynpm/latencybun-isolated/hosted(in #667)bun-isolated/rescan(in #667)Trends
Median
<pm>/hostedwall time per run, oldest → newest (ms). Runners differ between runs, so read these as trends only.Open issues and PR
rewrite_bun_lockcallsbun_lock_text::is_bundled_entry, which JSON-parses each entry once per patch.vlt_bundled::store_bundled_copieswalk (async canonicalize/read_dir per node), which runs twice per scan.bench/refresh): Bench: cover Bun isolated .bun store layout (addsbun-isolated/{hosted,rescan}).Standing slow-systems list
A package manager is slow when its ms/pkg is more than 2x the median across
*/hosted(0.1275), or when any of its scenarios takes over 500 ms (npm/latencyis excluded). An issue opens after 3 consecutive runs.redirect::vlt::every_instance_pinned/partition_instances/vlt_preflight::preflight_scope(~35%)utils::poetry_lock::rewrite_poetry_lock_in, 77% of instructionsutils::pdm_lock::rewrite_pdm_lock_in, 77%formats::cargo::CargoLock::parseinsiderewrite_cargo, 79% (per-patch re-parse of Cargo.lock)utils::python_lock::PythonLockSession::rewrite, 39%; vexpypi_locks::extract, 23%Dropped off today: pipenv (0.241, 1.9x; borderline, it was 2.1x on run 1). No scenario's peak RSS is above 2x the median (34.0 MiB).
Coverage-gap backlog
Bun isolated: covered by.bunstore (Fix npm crawler missing Bun, Deno and Yarn 4 stores (#366, #373, #405, #495) #496)bun-isolated/*in Bench: cover Bun isolated .bun store layout #667.nodeLinker: pnpm,node_modules/.store, Fix npm crawler missing Bun, Deno and Yarn 4 stores (#366, #373, #405, #495) #496): yarn-berry is covered only with the node-modules linker.npm:…::__archiveUrlpins (Fix yarn berry hosted pin leaking npm auth (#404) #465): a rescan over pre-Fix yarn berry hosted pin leaking npm auth (#404) #465 lockfiles rewrites them to the new tarball-URL form. Uncovered.bun.lockis covered. Thebun_binaryrewriter and codec (Fix Bun/vlt bundled copies left unpatched (#469, #471) #472) are unbenchmarked.{"bundled": true}entry or a vlt bundled store copy, so theredirect_*_bundled_instance_skippedpaths (Fix Bun/vlt bundled copies left unpatched (#469, #471) #472) are never hit.hatch.tomlis a HOSTED|PROBE pypi input informats/registry.rs, but there is nopm:hatchscenario.pm:gradle).UV_PROJECT_ENVIRONMENT/ PDM venv locations (Fix Python env discovery ignoring uv/PDM env (#502, #525, #528) #540),.egg-infoinstalls (Fix Python crawler missing .egg-info installs (#447) #452), Poetryenvs.toml(Fix Poetry venv selection ignoring envs.toml (#476, #526) #527) and lock-onlyrequirements.txt(Fix lock-only requirements.txt discovery (#412, #523) #530) are uncovered..bundle/config(Fix Bundler settings resolution order (#483, #507) #532) and Gemfile declaration dedupe (Fix Gemfile rewrites duplicating gem declarations (#482, #548) #552): only one layer is covered.Stale / redundant scenarios
None yet. 14+ days of history are needed before merging any hosted/rescan pairs. Candidate:
bun/rescannow overlapsbun-isolated/rescanon the lockfile path.Generated by Claude Code
All reactions