Skip to content

fix(bench): isolate cells from upstream branches - #1621

Merged
AbirAbbas merged 2 commits into
Agent-Field:devfrom
erSpell:fix/fetch-only-benchmark-clone
Sep 30, 2026
Merged

AbirAbbas merged 2 commits into
Agent-Field:devfrom
erSpell:fix/fetch-only-benchmark-clone

Conversation

@erSpell

@erSpell erSpell commented Sep 27, 2026

Copy link
Copy Markdown
Contributor

What changed

• Share one fetch-only working-tree helper across the benchmark runners
• Prevent benchmark cells from exposing upstream branches or newer gold-fix commits
• Add deterministic coverage for pinned commits, remote HEAD, reachable history, and benchmark wiring
• Ensure benchmark-only changes trigger the regression through pr-ready

How it was checked

• bash bench/canary/lib/repo_test.sh
• bash -n bench/run.sh bench/canary/lib/cell.sh bench/canary/lib/repo.sh bench/canary/lib/repo_test.sh bench/oneroad/lib/corpus.sh
• Real fetch of benchmark base 6c978ffa1c49ba600c85eb893958409e37dbedd2: zero remote branches and one reachable commit

Closes #1028

@CLAassistant

CLAassistant commented Sep 27, 2026 •

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@santoshkumarradha

Copy link
Copy Markdown
Member

Hey, thanks for the PR. Could you sign the CLA when you get a chance so we can review it?

@santoshkumarradha santoshkumarradha added bug Something the code does that it should not area:tests The suite itself — flakes, harnesses, laws, CI reds hygiene Tests, laws, dead code, duplication — no person-facing change labels Sep 27, 2026
@santoshkumarradha santoshkumarradha added this to the Tests & tooling milestone Sep 27, 2026
@erSpell

erSpell commented Sep 28, 2026

Copy link
Copy Markdown
Contributor Author

My bad, signed now!

@AbirAbbas
AbirAbbas merged commit 49ad17f into Agent-Field:dev Sep 30, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:tests The suite itself — flakes, harnesses, laws, CI reds bug Something the code does that it should not hygiene Tests, laws, dead code, duplication — no person-facing change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bench: fresh_clone leaks every upstream fix branch into the cell, so real-issue runs can copy the gold patch

4 participants