Skip to content

chore: gitignore orchestrator-workflow run state#156

Merged
LanNguyenSi merged 1 commit into
masterfrom
chore/gitignore-ai-runs
Jul 27, 2026
Merged

chore: gitignore orchestrator-workflow run state#156
LanNguyenSi merged 1 commit into
masterfrom
chore/gitignore-ai-runs

Conversation

@LanNguyenSi

Copy link
Copy Markdown
Owner

Untracked .ai/runs/ files from an earlier run sit in this work tree and fail
preflight's clean-worktree check, which gates push. They are local working
notes, not repo content.

Root-anchored /.ai/ so a .ai/ fixture nested inside a package or test tree
stays tracked normally. An unanchored .ai/ would swallow those silently.

Same fix class as agent-memory PR #66.

Verification

  • git ls-files .ai returns 0 results, so nothing tracked is masked.
  • git check-ignore -v .ai/runs resolves to .gitignore:11:/.ai/, the intended rule.
  • harness preflight at this HEAD: clean-worktree flips from fail to pass, which is the point of the change; typecheck, tests and secret-detection stay green.

Review

Reviewed by the orchestrator rather than a reviewer subagent, per the AGENTS.md
"Scaling delegation" clause for a trivial change. Recorded in the evidence
ledger.

The 20 pre-existing dev-side npm-audit highs in this repo are unrelated to this
change and separately tracked; the preflight waiver for this branch is recorded
in the ledger with its rationale.

Refs: operator batch run 2026-07-27

Untracked .ai/runs/ files from a previous run sit in this work tree and fail
preflight's clean-worktree check, which gates push. They are local working
notes, not repo content.

Root-anchored so a .ai/ fixture nested inside a package or test tree is still
tracked normally. Same fix class as agent-memory PR #66.
@LanNguyenSi LanNguyenSi added review:tests-pass merge-approval prereq review:checklist-complete merge-approval prereq review:comments-resolved merge-approval prereq review:scope-matches-task merge-approval prereq review:evidence-logged merge-approval prereq labels Jul 27, 2026
@LanNguyenSi
LanNguyenSi merged commit bd977c4 into master Jul 27, 2026
4 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review:checklist-complete merge-approval prereq review:comments-resolved merge-approval prereq review:evidence-logged merge-approval prereq review:scope-matches-task merge-approval prereq review:tests-pass merge-approval prereq

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant