Repository navigation
chore(agents): add agent config, ADRs, and reviewer docs - #186
Merged
Merged
Conversation
- Add docs/agents/ (issue tracker, triage labels, domain docs) following the setup-matt-pocock-skills conventions; add an Agent skills section to AGENTS.md - Add CONTEXT.md (orientation + knowledge index) and REVIEW.md (what failure looks like here, for AI reviewers) - Record five ADRs from the commit history: Yarn 4/oxfmt toolchain, flattened recipe layout, AST-first codemods with AI fixup fallback, convention-based test runner, release-owned README/version generation - Create the five triage labels on the GitHub tracker Config and docs only — no runtime code changed.
- ADR-0001: recipe-local tsconfigs and newer recipes' own typescript devDependency, instead of the categorical shared-config claim - ADR-0003: attribute the d12b8fc/8b709dc/38f173a fixes to tightening AST receiver/type scoping, not to AI-step false positives - REVIEW.md: check:changed runs CI's changed-file gate; CI additionally runs yarn test:scripts and a Changeset Check job
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.
Summary
Extends this repo's agent configuration onto the same standard as the existing
AGENTS.md+.agents/skills/surface, and records the architectural decisions behind the repo's current shape as ADRs.Which issue(s) does this PR fix
None — agentic repo hygiene. All five ADRs were verified against the commit history (no decisions superseded; every cited commit exists on
main).Evidence
AGENTS.mdpointed atCONTRIBUTING.mdand the.agents/skills/; there was no issue-tracker/triage-label/domain config for agent skills, no orientation doc, no reviewer contract, and no record of why the repo looks the way it does.docs/agents/carries the per-repo agent-skill config (GitHub Issues on this repo, default five-label triage vocabulary, single-context domain docs — the five labels now exist on the tracker);CONTEXT.mdorients a fresh agent;REVIEW.mdlists this repo's release-blocking and cross-platform failure modes; ADRs 0001–0005 document the Yarn 4/oxfmt toolchain, the flattened one-recipe-per-release layout, AST-first codemods with AI fixup fallback, the convention-based test runner inpackages/scripts, and release-owned README/version generation.Testing
Config, docs, and skill files only — no runtime code changed.
yarn format,yarn lint,yarn test, andyarn check:changed(the same gate CI runs) all pass on this branch.Merge Danger
Door: two-way
Blast Radius: developer tooling
Nothing is removed and no workflow changes — anyone not using agent tooling is unaffected. Agent harnesses that already read
AGENTS.mdget one new pointer section.How to revert
git revert <commit>— no data or generated artifacts depend on the new layout.How to test changes / Special notes for the reviewer
Review order suggestion:
AGENTS.md(the pointers) →CONTEXT.md+docs/agents/(the layout) →REVIEW.md(reviewer contract) →docs/adr/(records only, no behavior). ADRs cite commits by short hash; spot-check a few withgit show <hash>. No changeset: docs-only, no codemod changed.