Skip to content

AK-508: add .gitignore for agent review-findings dirs - #4

Draft
benw5483 wants to merge 1 commit into
mainfrom
ignore-agent-review-findings-dirs
Draft

AK-508: add .gitignore for agent review-findings dirs#4
benw5483 wants to merge 1 commit into
mainfrom
ignore-agent-review-findings-dirs

Conversation

@benw5483

Copy link
Copy Markdown
Contributor

Summary

  • Adds a .gitignore to this repo, which had none, so automated review tooling's findings directories can never be committed here.
  • Ignores .architect/ and .reviewer/, the gate-findings directories agent tooling writes at the repo root. They carry internal run identifiers and raw review commentary, so a broad git add -A must not sweep them into a PR.
  • Adds .DS_Store as the ordinary baseline. No node_modules/ or Python entries, because this repo tracks only Markdown, JSON, YAML, one shell script, and one image. Neither language is present.

Verified before committing

git ls-files | grep -E '^\.(architect|reviewer)/' returns nothing on main, so no findings file was ever committed to this repo. The new rule is purely preventive and untracks nothing.

The diff is one new file and eight added lines. Nothing is removed, reordered, or otherwise touched.

Test plan

  • git check-ignore -v .architect/round-1.findings.json .reviewer/round-1.findings.json names .gitignore as the matching source for both paths.
  • git ls-files | grep -E '^\.(architect|reviewer)/' is empty, confirming nothing matching was already tracked.
  • Create a scratch .architect/ directory in a clone, then confirm git status --porcelain stays clean.

Generated by the operator's software factory.
• City: factory-main · Agent: local-core__builder-fm-session-c20ba9dca735092c301bd842f4e87b44
• On behalf of: @benw5483

Automated review tooling writes `.architect/` and `.reviewer/` at the repo root of a working tree. Those directories hold internal run identifiers and raw review commentary, so a broad `git add -A` could sweep them into a PR. This repo had no `.gitignore` at all, so this adds one covering both patterns plus a `.DS_Store` baseline.

Verified before committing: no tracked file matched either pattern, so the rule is purely preventive and untracks nothing.

Generated by the operator's software factory.
City: factory-main · Agent: local-core__builder-fm-session-c20ba9dca735092c301bd842f4e87b44
On behalf of: @benw5483
Co-Authored-By: operator-factory-bot <factory-bot@actual.ai.invalid>
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