Skip to content

#646 - Adopt Matt Pocock agent skills and restructure Claude Code config#647

Merged
bracyw merged 27 commits into
developfrom
646-adopt-matt-pocock-skills
Jun 7, 2026
Merged

#646 - Adopt Matt Pocock agent skills and restructure Claude Code config#647
bracyw merged 27 commits into
developfrom
646-adopt-matt-pocock-skills

Conversation

@bracyw

@bracyw bracyw commented Jun 1, 2026

Copy link
Copy Markdown
Collaborator

Changes

Adopts Matt Pocock's agent-skill framework as Argos's AI-driven path for issue authoring and engineering work, and restructures the repo's Claude Code config around it. Lands in three groups: docs (agent conventions under docs/agents/, ADRs 0001 and 0002, and a root CONTEXT.md domain glossary), skills (19 skill dirs under .claude/skills/ — Matt's engineering and issue-authoring set plus Argos's own workflow skills), and code rules (per-subdirectory CLAUDE.md for angular-client and scylla-server, plus the first project-level root CLAUDE.md).

Issue authoring is two-path: AI files via Matt's body shapes through the gh CLI, humans keep the YAML form templates, and the ready-for-agent label marks AI-filed issues. The rejected single-path alternative and its rationale live in docs/adr/0002. Five triage labels (needs-triage, needs-info, ready-for-agent, ready-for-human, wontfix) are created on the repo; spike and other issue templates are removed, leaving task, bug-form, feature-request, epic.

Notes

  • First committed CLAUDE.md in the repo: project-focused (overview, local dev, testing, conventions, safety) with pointers to the subdir files.
  • The two subdir CLAUDE.md files port the former .claude/rules frontend and backend guidelines, minus the Cursor-style glob frontmatter that Claude Code ignores.
  • Caveman terse mode ships as an opt-in repo-wide pilot — default-on, toggle off per session.
  • Setup conventions live in docs (issue-tracker.md, triage-labels.md) rather than a setup skill; the labels are seeded once via gh label create.
  • Follow-ups out of scope: revisiting individual skills (tdd, verify-graph, prototype, diagnose) after real-world usage, tracked separately.

Test Cases

  • Verified skill and docs cross-references resolve, with no dangling skill invocations.
  • Confirmed the five triage labels exist on the repo with the intended colors and descriptions.
  • Confirmed no merge conflicts against develop.
  • Spot-checked the migrated skill files and the pruned issue templates (four survivors).

Checklist

  • All commits are tagged with the ticket number
  • No linting errors / newline at end of file warnings
  • All code follows repository-configured prettier formatting
  • No merge conflicts
  • All checks passing
  • Remove any non-applicable sections of this template
  • Assign the PR to yourself
  • No package-lock.json changes (unless dependencies have changed)
  • Request reviewers & ping on Slack
  • PR is linked to the ticket (fill in the closes line below)

Closes #646

@bracyw bracyw self-assigned this Jun 1, 2026
@bracyw bracyw requested a review from TheJeffreyKuo June 1, 2026 02:40
Comment thread .claude/skills/verify-telemetry/SKILL.md Outdated
Comment thread .claude/skills/verify-graph/SKILL.md
Comment thread .claude/skills/verify-telemetry/SKILL.md
Comment thread .claude/skills/setup-matt-pocock-skills/SKILL.md Outdated
Comment thread .gitignore
Comment thread .claude/skills/address-pr-comments/SKILL.md
Comment thread .claude/skills/caveman/SKILL.md
Comment thread .claude/skills/commit/SKILL.md
Comment thread .claude/skills/diagnose/scripts/hitl-loop.template.sh
Comment thread .claude/skills/setup-matt-pocock-skills/SKILL.md Outdated
Comment thread .claude/skills/setup-matt-pocock-skills/triage-labels.md Outdated
Comment thread .claude/skills/tdd/SKILL.md
Comment thread .claude/skills/verify-graph/SKILL.md
Comment thread CLAUDE.md Outdated
Comment thread .claude/skills/open-pr/SKILL.md Outdated
Comment thread .claude/skills/prototype/UI.md
Comment thread .claude/skills/run-local/SKILL.md Outdated
@bracyw bracyw requested a review from TheJeffreyKuo June 7, 2026 05:07
@bracyw bracyw marked this pull request as ready for review June 7, 2026 05:18
Comment thread .claude/skills/to-issues/SKILL.md Outdated
Comment thread .claude/skills/to-prd/SKILL.md Outdated
Comment thread .claude/skills/triage/SKILL.md Outdated
Comment thread .gitignore
@bracyw bracyw requested a review from TheJeffreyKuo June 7, 2026 06:46

@TheJeffreyKuo TheJeffreyKuo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm!

@bracyw bracyw merged commit 8f38107 into develop Jun 7, 2026
4 checks passed
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.

Adopt Matt Pocock agent skills and restructure Claude Code config

3 participants