test(perf): anchor provenance expectations to checkout - #3659
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
GJC receiptRepaired the exact post-merge perf provenance failures from run Root cause: both tests computed their expected Git HEAD with Change: Evidence:
— |
2ee717e to
359d318
Compare
Rebase receiptRebased the single test-only fix onto current immutable
Fresh exact CI is required for this successor. The stale workflow was not rerun or cancelled. — |
Inherited hold receipt — exact successor
|
|
Current-dev route update: preserve #3659 ownership Run The current PR head — |
The provenance runner already resolves Git HEAD from the benchmark checkout, but these expectations resolved HEAD from the caller cwd. CI invokes the shard outside the repository, so both contract tests failed before exercising the runner.\n\nLore-id: 30630819543\nConstraint: benchmark checkout HEAD and cwd must dominate workflow SHA and caller cwd\nRejected: changing runner provenance | current source already uses import.meta.dir-derived checkout root\nConfidence: high\nScope-risk: narrow\nReversibility: easy\nTested: focused provenance tests twice; full perf corpus test; coding-agent check; native build\nNot-tested: workflow rerun (explicitly prohibited)
359d318 to
5cce992
Compare
Summary
GITHUB_SHAand caller cwdimport.meta.dirVerification
packages/coding-agent/test/perf-corpus.test.ts: 45 passbun --cwd=packages/coding-agent run check: passFailure run: 30630819543
Failure commit: 71e9c95
No workflow reruns or cancellations were performed.
—
[repo owner's gaebal-gajae (clawdbot) 🦞]