Repository navigation
docs: move shared instructions to AGENTS.md - #182
Merged
Merged
Conversation
Put the repository rules in AGENTS.md so every coding harness reads the same instructions. CLAUDE.md imports them and retains only its reviewer invocation notes. Preserve the original instruction text and the existing Claude review resources.
This was referenced Oct 22, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Make
AGENTS.mdthe shared repository instruction file so coding agents load the same project rules.CLAUDE.mdimports it with@AGENTS.mdand keeps only the Claude-specific reviewer instructions; all existing instruction text is preserved verbatim.Follows the repository migration in nhobin219/agent-office#204. An audit found no tests, scripts, or documentation that read or link to
CLAUDE.md, so no path references need changing.Validation:
possibly-missing-submodulewarnings in unchangedtests/test_limits.py; the clean main worktree also passes.TestFlushing::test_a_served_row_reaches_the_published_tablesaw 0/50 rows published by its ten-second deadline. The same test also fails on unchanged main (26e903d), independently of this documentation change. The skip is the SQLite parameter of a test for losing a Postgres connection.No runtime or public API change; no release or consumer dependency-pin change is required. Critical review was not run because this only moves documentation. No tests were added or changed, so test falsification is not applicable.
CI run passed, including the full suites on Python 3.11 and 3.14, lint, type checking, wheel installation, and the required
CI successcheck.Ready for operator review. Do not merge or release as part of this request.