Skip to content

Remove root agent instruction files - #79

Merged
dceoy merged 2 commits into
mainfrom
agent/remove-root-agent-instructions
Aug 16, 2026
Merged

Remove root agent instruction files#79
dceoy merged 2 commits into
mainfrom
agent/remove-root-agent-instructions

Conversation

@dceoy

@dceoy dceoy commented Aug 16, 2026

Copy link
Copy Markdown
Owner

Summary

  • remove repository-root AGENTS.md
  • remove repository-root CLAUDE.md

Impact

  • no runtime code changes
  • nested agent instruction files are unchanged

Checks

  • verified only repository-root instruction files are removed

@dceoy dceoy self-assigned this Aug 16, 2026
@dceoy
dceoy marked this pull request as ready for review August 16, 2026 16:45

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your trial has ended. Reactivate Greptile to resume code reviews.

@claude

claude Bot commented Aug 16, 2026

Copy link
Copy Markdown

Reviewed with code-quality, performance, test-coverage, documentation-accuracy, and security passes. The diff itself (deleting AGENTS.md and the CLAUDE.md symlink) is low-risk — no code, tests, or security-sensitive content are touched, and the env vars documented in AGENTS.md are still covered in README.md.

One general observation worth a second look before merging: the PR description says "nested agent instruction files are unchanged," which reads as implying the guidance is preserved elsewhere, but that's not quite accurate. The .agents/skills/*/SKILL.md files are per-skill invocation docs, not a substitute for repo-wide instructions. With this PR, the following guidance has no home anywhere else in the repo (no CONTRIBUTING.md exists):

  • The root-level pointer to run .agents/skills/local-qa/scripts/qa.sh before committing/opening a PR (the script itself still exists, just nothing at the repo root references it anymore)
  • Testing conventions (pytest.mark.parametrize, preferring mocked MT5/pdmt5 interactions)
  • KISS/DRY/YAGNI design principles and commit/PR scoping guidelines

Not blocking, but worth confirming this is an intentional cleanup rather than an accidental loss of contributor guidance — otherwise consider migrating this content into README.md/CONTRIBUTING.md or keeping a thin root AGENTS.md that points to .agents/skills/.

@dceoy
dceoy merged commit 6f4c923 into main Aug 16, 2026
12 checks passed
@dceoy
dceoy deleted the agent/remove-root-agent-instructions branch August 16, 2026 16:58
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