Skip to content

chore(ci): ignore benchmark and working-note scratch dirs - #1373

Merged
TheGreatAxios merged 3 commits into
mainfrom
fix/ci-ignore-benchmark-scratch
Oct 9, 2026
Merged

TheGreatAxios merged 3 commits into
mainfrom
fix/ci-ignore-benchmark-scratch

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Collaborator

Summary

plans/, tb-dataset/, tb-rerun-50/, terminal-bench/ and evals/**/__pycache__/ are local-only working notes and benchmark snapshot scratch — they are never committed, but they leak into CI's bunx oxfmt --check . (static-analysis job) scan and fail the oxfmt step on every PR that picks them up from the environment. This has been failing the whole agent-extract wave (#1340 #1339 #1341 #1343 #1346 #1347 #1348 #1349 #1351 #1353) plus earlier CL-10234 work (PR #1362) on static-analysis.

This adds them to .gitignore so a clean checkout no longer contains them and oxfmt scans only trackable source.

Verification

bunx oxfmt --check . now passes on the whole repo (0 issues, ~1300 files); git status no longer lists plans//tb-* etc.

Fixes the pre-existing static-analysis/oxfmt red across the extract wave.

plans/, tb-*/** and terminal-bench/ are local-only working notes and
benchmark scratch, never committed. They leaked into CI's
'bunx oxfmt --check .' (static-analysis) scan, failing every PR that
picks them up from the environment. Ignore them so oxfmt scans only
trackable source.
@TheGreatAxios
TheGreatAxios merged commit 3490291 into main Oct 9, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant