Add DOMAIN_CONTEXT, ADRs, and project-scoped agent skills#1118
Open
busbyk wants to merge 3 commits into
Open
Conversation
Scaffold the per-repo setup the Matt Pocock engineering skills assume:
- docs/agents/{issue-tracker,triage-labels,domain}.md — consumer rules
pointing skills at GitHub Issues, canonical triage labels, and a
single-context domain layout
- CLAUDE.md — add an "Agent skills" section linking those docs
- .agents/skills/ — copy 10 workflow skills (to-issues, to-prd, triage,
diagnose, tdd, improve-codebase-architecture, grill-with-docs, grill-me,
zoom-out, setup-matt-pocock-skills), with grill-with-docs' ADR-FORMAT
realigned to this repo's house template
- .claude/skills/ — relative symlinks into .agents/skills
- .gitignore — un-ignore .claude/skills while keeping the rest of .claude ignored
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Produced via a /grill-with-docs session over the codebase: - DOMAIN_CONTEXT.md — domain glossary covering Tenancy, People, Access & Roles, Organizations & Programs, Avalanche Education, Events, and Content & Site Configuration, with canonical terms and _Avoid_ guidance for overloaded language (Tenant vs Avalanche Center, User vs Biography, Course vs Event, Tag vs EventTag, etc.) - docs/decisions/015-national-course-catalog.md — Providers & Courses are national/shared, deliberately outside per-tenant isolation - docs/decisions/016-per-tenant-globals-as-unique-tenant-collections.md — per-tenant singletons modeled as unique-tenant collections, not Payload Globals - drift.lock — bind both ADRs to the source files they describe Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
# Conflicts: # .gitignore
Contributor
|
Preview deployment: https://tooling-domain-context.preview.avy-fx.org |
rchlfryn
approved these changes
Jun 19, 2026
rchlfryn
left a comment
rchlfryn
left a comment
Collaborator
There was a problem hiding this comment.
Excited to see how this works and use this
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
2 participants
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.
Description
Adds a domain glossary, two architecture ADRs, agent-facing process docs, and a set of project-scoped engineering skills + agent config. Docs/tooling only — no runtime code changes.
The most significant change here is the guidance for Claude on issue tracking. So put review emphasis on the issue-tracker docs -- these are from https://github.com/mattpocock/skills/tree/main/skills/engineering/setup-matt-pocock-skills
Related Issues
N/A
Key Changes
DOMAIN_CONTEXT.md— domain glossary015-national-course-catalog.md,016-per-tenant-globals-as-unique-tenant-collections.md(with drift bindings) -- these are both pre-existing decisions but they felt worth defining explicitly.docs/agents/— domain, issue-tracker, and triage-labels process docs.agents/skills/+.claude/skills/and agent config;.gitignoreupdated to track.claude/skillsHow to test
pnpm drift:check→ "docs ok" (the new ADRs are bound). No app/runtime impact.Migration Explanation
N/A — no schema/DB changes.
Future enhancements / Questions
N/A