Repository navigation
chore: scaffold agent-skills config (issue tracker, triage labels, domain docs) - #206
Open
HeyItsGilbert wants to merge 1 commit into
Open
HeyItsGilbert wants to merge 1 commit into
HeyItsGilbert wants to merge 1 commit into
Conversation
…main docs) Adds the per-repo configuration the engineering skills (to-tickets, triage, to-spec, wayfinder, domain-modeling, grill-with-docs, improve-codebase-architecture) read from: - docs/agents/issue-tracker.md: GitHub, PowerShellOrg/PSDepend, via gh. PRs-as-triage-surface left off (default). - docs/agents/triage-labels.md: canonical label defaults (needs-triage, needs-info, ready-for-agent, ready-for-human, wontfix) unchanged. - docs/agents/domain.md: single-context (GLOSSARY.md + docs/adr/ at the repo root), noting a legacy adr/ directory with two older entries still pending consolidation into docs/adr/. - CLAUDE.md: new "## Agent skills" section pointing at the three files above.
Test Results 3 files 90 suites 1m 30s ⏱️ Results for commit c231dbe. |
Contributor
There was a problem hiding this comment.
🟡 Changes recommended
Several documented commands and repository-specific mappings do not match the current tracker and repository layout.
4 open findings
What changed in this PR
Adds repository configuration for engineering agent skills covering issue tracking, triage labels, and domain documentation.
Changes:
- Documents GitHub issue and wayfinding workflows.
- Defines triage-label mappings and domain-document locations.
- Links agent guidance from
CLAUDE.md.
| File | Description |
|---|---|
CLAUDE.md |
Links agent configuration documents. |
docs/agents/issue-tracker.md |
Defines issue-tracker and wayfinding operations. |
docs/agents/triage-labels.md |
Maps canonical triage roles to labels. |
docs/agents/domain.md |
Defines glossary and ADR discovery conventions. |
🧠 Review effort: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| ## Conventions | ||
|
|
||
| - **Create an issue**: `gh issue create --title "..." --body "..."`. Use a heredoc for multi-line bodies. | ||
| - **Read an issue**: `gh issue view <number> --comments`, filtering comments by `jq` and also fetching labels. |
Comment on lines
+41
to
+42
| - **Map**: a single issue labelled `wayfinder:map`, holding the Notes / Decisions-so-far / Fog body. `gh issue create --label wayfinder:map`. | ||
| - **Child ticket**: an issue linked to the map as a GitHub sub-issue (`gh api` on the sub-issues endpoint). Where sub-issues aren't enabled, add the child to a task list in the map body and put `Part of #<map>` at the top of the child body. Labels: `wayfinder:<type>` (`research`/`prototype`/`grilling`/`task`). Once claimed, the ticket is assigned to the driving dev. |
Comment on lines
+7
to
+10
| | `needs-triage` | `needs-triage` | Maintainer needs to evaluate this issue | | ||
| | `needs-info` | `needs-info` | Waiting on reporter for more information | | ||
| | `ready-for-agent` | `ready-for-agent` | Fully specified, ready for an AFK agent | | ||
| | `ready-for-human` | `ready-for-human` | Requires human implementation | |
Comment on lines
+13
to
+16
| > **Known split (as of this setup):** two ADR directories currently exist — | ||
| > `docs/adr/` (canonical, going forward) and a legacy `adr/` at the repo root | ||
| > with two older entries. Read both until they're consolidated, but write new | ||
| > ADRs only to `docs/adr/`. |
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
Adds the per-repo configuration that the Matt Pocock engineering skills (
to-tickets,triage,to-spec,wayfinder,domain-modeling,grill-with-docs,improve-codebase-architecture) read from, persetup-matt-pocock-skills.docs/agents/issue-tracker.md— GitHub (PowerShellOrg/PSDepend) via theghCLI. PRs-as-triage-surface left off (the default).docs/agents/triage-labels.md— canonical defaults unchanged (needs-triage,needs-info,ready-for-agent,ready-for-human,wontfix).docs/agents/domain.md— single-context layout (GLOSSARY.md+docs/adr/at the repo root). Flags a pre-existing split: a legacyadr/directory at the repo root still holds two older ADRs (0001-nuget-version-ranges.md,0002-lock-resolution-model.md) that haven't been consolidated intodocs/adr/. Not touched by this PR — only documented as a known gap for whoever owns that cleanup.CLAUDE.md— new## Agent skillssection pointing at the three files above.Testing
Documentation-only change; no code/tests affected.