docs(agents): configure the engineering skills for this repo - #33
Merged
Conversation
Add docs/agents/{issue-tracker,triage-labels,domain}.md so /triage,
/code-review and friends can run here, and link them from CLAUDE.md.
domain.md is adapted rather than stock: it maps the skills' CONTEXT.md +
docs/adr/ model onto this repo's architecture/ + planning/ convention
instead of introducing a second truth home, and points at modern-di's
glossary as the upstream authority for shared terms.
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
/triagecould not run on this repo: there was nodocs/agents/config naming the issuetracker, the triage label vocabulary or the domain-doc homes, and four of the five triage
state labels did not exist. This is the same setup already merged for
modern-python/.github, adapted to this repo's convention.Prompted by #32, which was transferred here from the core tracker during triage and landed
with
enhancementonly, because no state label existed to carry.Changes
docs/agents/issue-tracker.md(GitHub viagh; PRs as a request surface: no).docs/agents/triage-labels.md— the five canonical roles, each mapping to itself.docs/agents/domain.md, adapted rather than stock: it maps the skills'CONTEXT.md+docs/adr/model onto this repo'sarchitecture/+planning/convention instead of introducing a second truth home. It also names
modern-di'sglossary as the upstream authority for shared terms (
Container,Provider,Group,Scope), since this package is a thin adapter and should not redefine them.## Agent skillssection inCLAUDE.md, in the bullet formatthe sibling repos use.
Out of band, on the repo itself: created
needs-triage,needs-info,ready-for-agentand
ready-for-human, mirroring the colors and descriptionsmodern-diuses.Notes for review
but
planning/changes/is empty and the last four PRs (chore: adopt ruff 0.16.0 #28, docs: drop the deprecated validate= argument, validate explicitly #29, docs(release): notes for 3.0.1 #30, Bump planning convention 2.0.0 -> 2.2.0 #31 — including aconvention bump) shipped without one. I followed the repo's actual practice rather than
writing the first-ever change file as a side effect of agent setup. Say the word and
I'll add one.
docs/directory existed before this. There is no MkDocs site here, so unlike.githubno site-exclusion is needed — nothing publishesdocs/.issue-tracker.mdinherits a latent template defect: it documents filtering externalPRs by
authorAssociationongh pr list, a fieldgh2.98.0 rejects. Unreachablewhile the PRs flag is
no, and the same line sits in the sibling repos — worth aseparate sweep.
Verification
just lint-cipasses in full:eof-fixer --check,ruff format --check,ruff check --no-fix,ty check,planning: OK,links: OK.