mirror of
https://github.com/Donchitos/Claude-Code-Game-Studios.git
synced 2026-06-27 04:51:46 +00:00
* Add /vertical-slice skill, prototype overhaul, and workflow integration - Add /vertical-slice skill for pre-production validation (Phase 4 gate) - Overhaul /prototype skill with two-mode design: concept prototype (Phase 1) vs vertical slice (Phase 4), with clearer differentiation and higher standards for VS - Update prototyper agent to own both prototype and vertical-slice workflows - Add prototype-report.md and vertical-slice-report.md output templates - Update WORKFLOW-GUIDE, quick-start, skills-reference, agent-coordination-map, and skill-flow-diagrams to fully integrate both skills into the 7-phase pipeline - Remove orphaned empty quick-prototype/ directory Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * sync v1 counts + polish Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Add entity inventory flow, relax vertical-slice gate, improve UX authoring prompts - /asset-spec: new Phase 0b entity & screen inventory when no argument and no existing inventory — reads GDDs/art-bible, proposes categorized list, writes design/assets/entity-inventory.md collaboratively - /asset-spec: entity/character target falls back to inline user description when no source doc exists, rather than failing - /gate-check: vertical slice changed from blocking to CONCERNS-only when absent; built-but-broken slice still fails; adds entity inventory as gate artifact - /ux-design: convert inline approval prompts to AskUserQuestion for structured option capture at key authoring decision points - workflow-catalog.yaml: entity-inventory step added to pre-production; UX spec min_count raised to 3; vertical-slice and prototype marked required: false with updated descriptions - .gitignore: exclude marrow/ eval tooling directory Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * Add missing AskUserQuestion widgets to 7 skills Audit found 11 decision points across 7 skills where structured option prompts were missing — using plain text, auto-selection, or no gate at all. Skills patched: - create-epics: per-epic approval + producer CONCERNS verdict - sprint-plan: producer CONCERNS verdict with scope/timeline options - milestone-review: AT RISK / OFF TRACK producer verdicts require acknowledgement - retrospective: existing-retro handling converted from plain text [A]/[B] - quick-design: classification confirmation + draft approve/revise/redirect - tech-debt add mode: category (6 options) + effort (S/M/L/XL) structured capture - regression-suite: no-arg mode selection instead of silent auto-detect - hotfix: severity confirmation gate before workflow begins Also added AskUserQuestion to allowed-tools headers for retrospective, quick-design, tech-debt, regression-suite, and hotfix. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * Prep v1 stable: fix WORKFLOW-GUIDE counts, stale agent names, and skill model fields - WORKFLOW-GUIDE.md: correct agent count (48→49), skill count (66/68→73), add 6 missing skills to Appendix B, fix Creative category count (2→4), replace 3 non-existent agent names with correct ue-*/unity-* specialists, add missing godot-csharp/gdextension specialists to hierarchy, fix production/stories/ paths → production/epics/ - coordination-rules.md: replace "not yet used" with opt-in env var note - quick-start.md: rename duplicate "Validate the concept" label → "Prototype the mechanic" - skill-flow-diagrams.md: remove duplicate legacy UX pipeline section - All 62 skills missing model: field now have explicit model: sonnet Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * fix: comprehensive skill audit — consistency, UX, and flow gaps Two-pass audit fixing ~35 bugs across 41 files. Pre-production flow: - Brainstorm next-steps split into Path A (design-first) and Path B (prototype-first) — eliminates "prototype after architecture" confusion - /architecture-review added to pre-production flow in brainstorm and create-architecture handoffs - gate-check traceability check corrected to requirements-traceability.md - dev-story TR registry error now points to /architecture-review (not /create-epics) - start now writes production/stage.txt on first onboarding AskUserQuestion gaps filled: - balance-check, code-review, hotfix, day-one-patch, consistency-check all gain closing widgets and/or missing allowed-tools declarations - hotfix git branch creation now requires user confirmation - sprint-plan review-mode setup moved to Phase 0 (before gates run) - team-combat gains architecture→implementation approval gate - design-review APPROVED path consolidated from 3 widgets to 1 multiSelect All 9 team-* skills: - Phase 0 review-mode resolution added (solo/lean/full now respected) - team-audio output path fixed (design/gdd/ → design/audio/) - team-level final doc compilation delegated to level-designer subagent - team-narrative localization-lead added to composition list - team-qa sprint path fixed (flat files, not directories) - team-release NO-GO override captures written justification - team-live-ops Cancel verdict now explicitly BLOCKED Other fixes: - Art bible path standardized to design/art/art-bible.md (3 wrong refs) - AD-PHASE-GATE added to lean-mode skip list in director-gates.md - design-system duplicate 5d heading fixed; skeleton decline path added; mandatory agent spawns now respect review mode - story-readiness acceptance criteria thresholds now type-aware - create-stories gains multi-ADR and no-ADR handling guidance - consistency-check creates docs/consistency-failures.md on first run - retrospective frontmatter bash injection replaced with explicit Bash call - smoke-check ls -t gains PowerShell fallback - Conventional Commits format documented in coding-standards.md - gate-check: ADR acceptance gate, QA plan check, chain-of-verification tool-action requirement all added Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * fix: expose --review flag in argument-hints for all team-* skills All 9 team-* skills already implement Phase 0 review-mode resolution internally (full/lean/solo), but none advertised [--review full|lean|solo] in their argument-hint. Users had no way to discover the per-run override. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * docs: add SECURITY.md with coordinated disclosure policy Defines scope, reporting process (GitHub private vulnerability reporting), contributor security guidelines for hooks/skills/agents, and 90-day coordinated disclosure timeline. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * docs: add CONTRIBUTING.md with framework contribution guidelines Covers what PRs are welcome, skill/hook/agent technical requirements, the collaborative principle, testing expectations, commit format, and platform compatibility requirements. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * docs: add v1.0.0-beta → v1.0 upgrade section to UPGRADING.md Documents the 17 commits since the beta tag: new /vertical-slice gate, entity inventory flow in /map-systems, AskUserQuestion widgets across 7 skills, --review flag exposure on team-* skills, bug fixes (#21, #36, #42, #43, #45), and the new CONTRIBUTING.md and SECURITY.md. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
240 lines
7.7 KiB
Markdown
240 lines
7.7 KiB
Markdown
---
|
|
name: propagate-design-change
|
|
description: "When a GDD is revised, scans all ADRs and the traceability index to identify which architectural decisions are now potentially stale. Produces a change impact report and guides the user through resolution."
|
|
argument-hint: "[path/to/changed-gdd.md]"
|
|
user-invocable: true
|
|
allowed-tools: Read, Glob, Grep, Write, Bash, Task
|
|
model: sonnet
|
|
agent: technical-director
|
|
---
|
|
|
|
# Propagate Design Change
|
|
|
|
When a GDD changes, architectural decisions written against it may no longer be
|
|
valid. This skill finds every affected ADR, compares what the ADR assumed against
|
|
what the GDD now says, and guides the user through resolution.
|
|
|
|
**Usage:** `/propagate-design-change design/gdd/combat-system.md`
|
|
|
|
---
|
|
|
|
## 1. Validate Argument
|
|
|
|
A GDD path argument is **required**. If missing, fail with:
|
|
> "Usage: `/propagate-design-change design/gdd/[system].md`
|
|
> Provide the path to the GDD that was changed."
|
|
|
|
Verify the file exists. If not, fail with:
|
|
> "[path] not found. Check the path and try again."
|
|
|
|
---
|
|
|
|
## 2. Read the Changed GDD
|
|
|
|
Read the current GDD in full.
|
|
|
|
---
|
|
|
|
## 3. Read the Previous Version
|
|
|
|
Run git to get the previous committed version:
|
|
|
|
```bash
|
|
git show HEAD:design/gdd/[filename].md
|
|
```
|
|
|
|
If the file has no git history (new file), report:
|
|
> "No previous version in git — this appears to be a new GDD, not a revision.
|
|
> Nothing to propagate."
|
|
|
|
If git returns the previous version, do a conceptual diff:
|
|
- Identify sections that changed (new rules, removed rules, modified formulas,
|
|
changed acceptance criteria, changed tuning knobs)
|
|
- Identify sections that are unchanged
|
|
- Produce a change summary:
|
|
|
|
```
|
|
## Change Summary: [GDD filename]
|
|
Date of revision: [today]
|
|
|
|
Changed sections:
|
|
- [Section name]: [what changed — new rule, removed rule, formula modified, etc.]
|
|
|
|
Unchanged sections:
|
|
- [Section name]
|
|
|
|
Key changes affecting architecture:
|
|
- [Change 1 — likely to affect ADRs]
|
|
- [Change 2]
|
|
```
|
|
|
|
---
|
|
|
|
## 4. Load Architecture Inputs
|
|
|
|
Read all ADRs in `docs/architecture/`:
|
|
- For each ADR, read the full file
|
|
- Extract the "GDD Requirements Addressed" table
|
|
- Note which GDD documents and requirement IDs each ADR references
|
|
|
|
Read `docs/architecture/architecture-traceability.md` if it exists.
|
|
|
|
Report: "Loaded [N] ADRs. [M] reference [gdd filename]."
|
|
|
|
---
|
|
|
|
## 5. Impact Analysis
|
|
|
|
For each ADR that references the changed GDD:
|
|
|
|
Compare the ADR's "GDD Requirements Addressed" entries against the changed sections
|
|
of the GDD. For each referenced requirement:
|
|
|
|
1. **Locate the requirement** in the current GDD — does it still exist?
|
|
2. **Compare**: What did the GDD say when the ADR was written vs. what it says now?
|
|
3. **Assess the ADR decision**: Is the architectural decision still valid?
|
|
|
|
Classify each affected ADR as one of:
|
|
|
|
| Status | Meaning |
|
|
|--------|---------|
|
|
| ✅ **Still Valid** | The GDD change doesn't affect what this ADR decided |
|
|
| ⚠️ **Needs Review** | The GDD change may affect this ADR — human judgment needed |
|
|
| 🔴 **Likely Superseded** | The GDD change directly contradicts what this ADR assumed |
|
|
|
|
For each affected ADR, produce an impact entry:
|
|
|
|
```
|
|
### ADR-NNNN: [title]
|
|
Status: [Still Valid / Needs Review / Likely Superseded]
|
|
|
|
What the ADR assumed about this GDD:
|
|
"[relevant quote from the ADR's GDD Requirements Addressed section]"
|
|
|
|
What the GDD now says:
|
|
"[relevant quote from the current GDD]"
|
|
|
|
Assessment:
|
|
[Explanation of whether the ADR decision is still valid, and why]
|
|
|
|
Recommended action:
|
|
[Keep as-is | Review and update | Mark Superseded and write new ADR]
|
|
```
|
|
|
|
---
|
|
|
|
## 6. Present Impact Report
|
|
|
|
Present the full impact report to the user before asking for any action. Format:
|
|
|
|
```
|
|
## Design Change Impact Report
|
|
GDD: [filename]
|
|
Date: [today]
|
|
Changes detected: [N sections changed]
|
|
ADRs referencing this GDD: [M]
|
|
|
|
### Not Affected
|
|
[ADRs referencing this GDD whose decisions remain valid]
|
|
|
|
### Needs Review ([count])
|
|
[ADRs that may need updating]
|
|
|
|
### Likely Superseded ([count])
|
|
[ADRs whose assumptions are now contradicted]
|
|
```
|
|
|
|
---
|
|
|
|
## 6b. Director Gate — Technical Impact Review
|
|
|
|
**Review mode check** — apply before spawning TD-CHANGE-IMPACT:
|
|
- `solo` → skip. Note: "TD-CHANGE-IMPACT skipped — Solo mode." Proceed to Phase 7.
|
|
- `lean` → skip. Note: "TD-CHANGE-IMPACT skipped — Lean mode." Proceed to Phase 7.
|
|
- `full` → spawn as normal.
|
|
|
|
Spawn `technical-director` via Task using gate **TD-CHANGE-IMPACT** (`.claude/docs/director-gates.md`).
|
|
|
|
Pass: the full Design Change Impact Report from Phase 6 (change summary, all affected ADRs with their Still Valid / Needs Review / Likely Superseded classifications, and recommended actions).
|
|
|
|
The technical-director reviews whether:
|
|
- The impact classifications are correct (no ADRs under-classified)
|
|
- The recommended actions are architecturally sound
|
|
- Any cascading effects on other ADRs or systems were missed
|
|
|
|
Apply the verdict:
|
|
- **APPROVE** → proceed to Phase 7 resolution workflow
|
|
- **CONCERNS** → surface the specific ADRs or recommendations flagged; use `AskUserQuestion` with options: `Revise the impact assessment` / `Accept with noted concerns` / `Discuss further`
|
|
- **REJECT** → do not proceed to resolution; re-analyze the impact before continuing
|
|
|
|
---
|
|
|
|
## 7. Resolution Workflow
|
|
|
|
For each ADR marked "Needs Review" or "Likely Superseded", ask the user what to do:
|
|
|
|
Ask for each ADR in turn:
|
|
> "ADR-NNNN ([title]) — [status]. What would you like to do?"
|
|
> Options:
|
|
> - "Mark Superseded (I'll write a new ADR)" — updates ADR status line to `Superseded by: [pending]`
|
|
> - "Update in place (minor revision)" — opens the ADR for editing; note what to revise
|
|
> - "Keep as-is (the change doesn't actually affect this decision)"
|
|
> - "Skip for now (revisit later)"
|
|
|
|
For ADRs marked **Superseded**:
|
|
- Update the ADR's Status field: `Superseded by ADR-[next number] (pending — see change-impact-[date]-[system].md)`
|
|
- Ask: "May I update the status in [ADR filename]?"
|
|
|
|
---
|
|
|
|
## 8. Update Traceability Index
|
|
|
|
If `docs/architecture/architecture-traceability.md` exists:
|
|
- Add the changed GDD requirements to the "Superseded Requirements" table:
|
|
|
|
```markdown
|
|
## Superseded Requirements
|
|
| Date | GDD | Requirement | Changed To | ADRs Affected | Resolution |
|
|
|------|-----|-------------|------------|---------------|------------|
|
|
| [date] | [gdd] | [old requirement text] | [new requirement text] | ADR-NNNN | [Superseded/Updated/Valid] |
|
|
```
|
|
|
|
Ask: "May I update the traceability index?"
|
|
|
|
---
|
|
|
|
## 9. Output Change Impact Document
|
|
|
|
Ask: "May I write the change impact report to `docs/architecture/change-impact-[date]-[system-slug].md`?"
|
|
|
|
The document contains:
|
|
- The change summary from step 3
|
|
- The full impact analysis from step 5
|
|
- Resolution decisions made in step 7
|
|
- List of ADRs that need to be written or updated
|
|
|
|
If user approved: Verdict: **COMPLETE** — change impact report saved.
|
|
If user declined: Verdict: **BLOCKED** — user declined write.
|
|
|
|
---
|
|
|
|
## 10. Follow-Up Actions
|
|
|
|
Based on the resolution decisions, suggest:
|
|
|
|
- **ADRs marked Superseded**: "Run `/architecture-decision [title]` to write the
|
|
replacement ADR. Then re-run `/propagate-design-change` to verify coverage."
|
|
- **ADRs to update in place**: List the specific fields to update in each ADR
|
|
- **If many ADRs affected**: "Run `/architecture-review` after all ADRs are updated
|
|
to verify the full traceability matrix is still coherent."
|
|
|
|
---
|
|
|
|
## Collaborative Protocol
|
|
|
|
1. **Read silently** — compute the full impact before presenting anything
|
|
2. **Show the full report first** — let the user see the scope before asking for action
|
|
3. **Ask per-ADR** — don't batch decisions; each affected ADR may need different treatment
|
|
4. **Ask before writing** — always confirm before modifying any file
|
|
5. **Non-destructive** — never delete ADR content; only add "Superseded by" notes
|