mirror of
https://github.com/Donchitos/Claude-Code-Game-Studios.git
synced 2026-06-27 13:01:50 +00:00
984023ddac
* 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>
123 lines
4.0 KiB
Markdown
123 lines
4.0 KiB
Markdown
# QA Plan: [Sprint/Feature Name]
|
|
|
|
> **Date**: [date]
|
|
> **Generated by**: /qa-plan
|
|
> **Scope**: [N stories across N systems]
|
|
> **Engine**: [engine name and version]
|
|
> **Sprint file**: [path to sprint plan]
|
|
|
|
---
|
|
|
|
## Story Coverage Summary
|
|
|
|
| Story | Type | Automated Test Required | Manual Verification Required |
|
|
|-------|------|------------------------|------------------------------|
|
|
| [story title] | Logic | Unit test — `tests/unit/[system]/` | None |
|
|
| [story title] | Integration | Integration test — `tests/integration/[system]/` | Smoke check |
|
|
| [story title] | Visual/Feel | None (not automatable) | Screenshot + lead sign-off |
|
|
| [story title] | UI | None (not automatable) | Manual step-through |
|
|
| [story title] | Config/Data | Data validation (optional) | Spot-check in-game values |
|
|
|
|
**Totals**: [N] Logic, [N] Integration, [N] Visual/Feel, [N] UI, [N] Config/Data
|
|
|
|
---
|
|
|
|
## Automated Tests Required
|
|
|
|
### [Story Title] — Logic
|
|
|
|
**Test file path**: `tests/unit/[system]/[story-slug]_test.[ext]`
|
|
|
|
**What to test**:
|
|
- [Formula or rule from GDD Formulas section — e.g., "damage = base * multiplier where multiplier ∈ [0.5, 3.0]"]
|
|
- [Each named state transition]
|
|
- [Each side effect that should / should not occur]
|
|
|
|
**Edge cases to cover**:
|
|
- Zero / minimum input values
|
|
- Maximum / boundary input values
|
|
- Invalid or null input
|
|
- [GDD-specified edge cases]
|
|
|
|
**Estimated test count**: ~[N] unit tests
|
|
|
|
---
|
|
|
|
### [Story Title] — Integration
|
|
|
|
**Test file path**: `tests/integration/[system]/[story-slug]_test.[ext]`
|
|
|
|
**What to test**:
|
|
- [Cross-system interaction — e.g., "applying buff updates CharacterStats and triggers UI refresh"]
|
|
- [Round-trip — e.g., "save → load restores all fields"]
|
|
|
|
---
|
|
|
|
## Manual QA Checklist
|
|
|
|
### [Story Title] — Visual/Feel
|
|
|
|
**Verification method**: Screenshot + [designer / art-lead] sign-off
|
|
**Evidence file**: `production/qa/evidence/[story-slug]-evidence.md`
|
|
**Who must sign off**: [designer / lead-programmer / art-lead]
|
|
|
|
- [ ] [Specific observable condition — e.g., "hit flash appears on frame of impact, not the frame after"]
|
|
- [ ] [Another falsifiable condition]
|
|
|
|
### [Story Title] — UI
|
|
|
|
**Verification method**: Manual step-through
|
|
**Evidence file**: `production/qa/evidence/[story-slug]-evidence.md`
|
|
|
|
- [ ] [Every acceptance criterion translated into a manual check item]
|
|
|
|
---
|
|
|
|
## Smoke Test Scope
|
|
|
|
Critical paths to verify before QA hand-off (run via `/smoke-check`):
|
|
|
|
1. Game launches to main menu without crash
|
|
2. New game / session can be started
|
|
3. [Primary mechanic introduced or changed this sprint]
|
|
4. [System with regression risk from this sprint's changes]
|
|
5. Save / load cycle completes without data loss (if save system exists)
|
|
6. Performance is within budget on target hardware
|
|
|
|
---
|
|
|
|
## Playtest Requirements
|
|
|
|
| Story | Playtest Goal | Min Sessions | Target Player Type |
|
|
|-------|--------------|--------------|-------------------|
|
|
| [story] | [What question must be answered?] | [N] | [new player / experienced / etc.] |
|
|
|
|
Sign-off requirement: Playtest notes → `production/session-logs/playtest-[sprint]-[story-slug].md`
|
|
|
|
If no playtest sessions required: *No playtest sessions required for this sprint.*
|
|
|
|
---
|
|
|
|
## Definition of Done — This Sprint
|
|
|
|
A story is DONE when ALL of the following are true:
|
|
|
|
- [ ] All acceptance criteria verified — automated test result OR documented manual evidence
|
|
- [ ] Test file exists for all Logic and Integration stories and passes
|
|
- [ ] Manual evidence document exists for all Visual/Feel and UI stories
|
|
- [ ] Smoke check passes (run `/smoke-check sprint` before QA hand-off)
|
|
- [ ] No regressions introduced — previous sprint's features still pass
|
|
- [ ] Code reviewed (via `/code-review` or documented peer review)
|
|
- [ ] Story file updated to `Status: Complete` via `/story-done`
|
|
|
|
**Stories requiring playtest sign-off before close**: [list, or "None"]
|
|
|
|
---
|
|
|
|
---
|
|
|
|
*Results and sign-off are tracked in the QA sign-off report generated by `/team-qa` — not in this plan file.*
|
|
|
|
*Template: `.claude/docs/templates/test-plan.md`*
|
|
*Generated by: `/qa-plan` — do not edit this line*
|