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>
227 lines
8.3 KiB
Markdown
227 lines
8.3 KiB
Markdown
---
|
||
name: day-one-patch
|
||
description: "Prepare a day-one patch for a game launch. Scopes, prioritises, implements, and QA-gates a focused patch addressing known issues discovered after gold master but before or immediately after public launch. Treats the patch as a mini-sprint with its own QA gate and rollback plan."
|
||
argument-hint: "[scope: known-bugs | cert-feedback | all]"
|
||
user-invocable: true
|
||
allowed-tools: Read, Glob, Grep, Write, Edit, Bash, Task, AskUserQuestion
|
||
model: sonnet
|
||
---
|
||
|
||
# Day-One Patch
|
||
|
||
Every shipped game has a day-one patch. Planning it before launch day prevents
|
||
chaos. This skill scopes the patch to only what is safe and necessary, gates it
|
||
through a lightweight QA pass, and ensures a rollback plan exists before anything
|
||
ships. It is a mini-sprint — not a hotfix, not a full sprint.
|
||
|
||
**When to run:**
|
||
- After the gold master build is locked (cert approved or launch candidate tagged)
|
||
- When known bugs exist that are too risky to address in the gold master
|
||
- When cert feedback requires minor fixes post-submission
|
||
- When a pre-launch playtest surfaces must-fix issues after the release gate passed
|
||
|
||
**Day-one patch scope rules:**
|
||
- Only P1/P2 bugs that are SAFE to fix quickly
|
||
- No new features — this is fix-only
|
||
- No refactoring — minimum viable change
|
||
- Any fix that requires more than 4 hours of dev time belongs in patch 1.1, not day-one
|
||
|
||
**Output:** `production/releases/day-one-patch-[version].md`
|
||
|
||
---
|
||
|
||
## Phase 1: Load Release Context
|
||
|
||
Read:
|
||
- `production/stage.txt` — confirm project is in Release stage
|
||
- The most recent file in `production/gate-checks/` — read the release gate verdict
|
||
- `production/qa/bugs/*.md` — load all bugs with Status: Open or Fixed — Pending Verification
|
||
- `production/sprints/` most recent — understand what shipped
|
||
- `production/security/security-audit-*.md` most recent — check for any open security items
|
||
|
||
If `production/stage.txt` is not `Release` or `Polish`:
|
||
> "Day-one patch prep is for Release-stage projects. Current stage: [stage]. This skill is not appropriate until you are approaching launch."
|
||
|
||
---
|
||
|
||
## Phase 2: Scope the Patch
|
||
|
||
### Step 2a — Classify open bugs for patch inclusion
|
||
|
||
For each open bug, evaluate:
|
||
|
||
| Criterion | Include in day-one? |
|
||
|-----------|-------------------|
|
||
| S1 or S2 severity | Yes — must include if safe to fix |
|
||
| P1 priority | Yes |
|
||
| Fix estimated < 4 hours | Yes |
|
||
| Fix requires architecture change | No — defer to 1.1 |
|
||
| Fix introduces new code paths | No — too risky |
|
||
| Fix is data/config only (no code change) | Yes — very low risk |
|
||
| Cert feedback requirement | Yes — required for platform approval |
|
||
| S3/S4 severity | Only if trivial config fix; otherwise defer |
|
||
|
||
### Step 2b — Present patch scope to user
|
||
|
||
Use `AskUserQuestion`:
|
||
- Prompt: "Based on open bugs and cert feedback, here is the proposed day-one patch scope. Does this look right?"
|
||
- Show: table of included bugs (ID, severity, description, estimated effort)
|
||
- Show: table of deferred bugs (ID, severity, reason deferred)
|
||
- Options: `[A] Approve this scope` / `[B] Adjust — I want to add or remove items` / `[C] No day-one patch needed`
|
||
|
||
If [C]: output "No day-one patch required. Proceed to `/launch-checklist`." Stop.
|
||
|
||
### Step 2c — Check total scope
|
||
|
||
Sum estimated effort. If total exceeds 1 day of work:
|
||
> "⚠️ Patch scope is [N hours] — this exceeds a safe day-one window. Consider deferring lower-priority items to patch 1.1. A bloated day-one patch introduces more risk than it removes."
|
||
|
||
Use `AskUserQuestion` to confirm proceeding or reduce scope.
|
||
|
||
---
|
||
|
||
## Phase 3: Rollback Plan
|
||
|
||
Before any code is written, define the rollback procedure. This is non-negotiable.
|
||
|
||
Spawn `release-manager` via Task. Ask them to produce a rollback plan covering:
|
||
- How to revert to the gold master build on each target platform
|
||
- Platform-specific rollback constraints (some platforms cannot roll back cert builds)
|
||
- Who is responsible for triggering the rollback
|
||
- What player communication is required if a rollback occurs
|
||
|
||
Present the rollback plan. Ask: "May I write this rollback plan to `production/releases/rollback-plan-[version].md`?"
|
||
|
||
Do not proceed to Phase 4 until the rollback plan is written.
|
||
|
||
---
|
||
|
||
## Phase 4: Implement Fixes
|
||
|
||
For each bug in the approved scope, spawn a focused implementation loop:
|
||
|
||
1. Spawn `lead-programmer` via Task with:
|
||
- The bug report (exact reproduction steps and root cause if known)
|
||
- The constraint: minimum viable fix only, no cleanup
|
||
- The affected files (from bug report Technical Context section)
|
||
|
||
2. The lead-programmer implements and runs targeted tests.
|
||
|
||
3. Spawn `qa-tester` via Task to verify: does the bug reproduce after the fix?
|
||
|
||
For config/data-only fixes: make the change directly (no programmer agent needed). Confirm the value changed and re-run any relevant smoke test.
|
||
|
||
---
|
||
|
||
## Phase 5: Patch QA Gate
|
||
|
||
This is a lightweight QA pass — not a full `/team-qa`. The patch is already QA-approved from the release gate; we are only re-verifying the changed areas.
|
||
|
||
Spawn `qa-lead` via Task with:
|
||
- List of all changed files
|
||
- List of bugs fixed (with verification status from Phase 4)
|
||
- The smoke check scope for the affected systems
|
||
|
||
Ask qa-lead to determine: **Is a targeted smoke check sufficient, or do any fixes touch systems that require a broader regression?**
|
||
|
||
Run the required QA scope:
|
||
- **Targeted smoke check** — run `/smoke-check [affected-systems]`
|
||
- **Broader regression** — run targeted tests in `tests/unit/` and `tests/integration/` for affected systems
|
||
|
||
QA verdict must be PASS or PASS WITH WARNINGS before proceeding. If FAIL: scope the failing fix out of the day-one patch and defer to 1.1.
|
||
|
||
---
|
||
|
||
## Phase 6: Generate Patch Record
|
||
|
||
```markdown
|
||
# Day-One Patch: [Game Name] v[version]
|
||
|
||
**Date prepared**: [date]
|
||
**Target release**: [launch date or "day of launch"]
|
||
**Base build**: [gold master tag or commit]
|
||
**Patch build**: [patch tag or commit]
|
||
|
||
---
|
||
|
||
## Patch Notes (Internal)
|
||
|
||
### Bugs Fixed
|
||
| BUG-ID | Severity | Description | Fix summary |
|
||
|--------|----------|-------------|-------------|
|
||
| BUG-NNN | S[1-4] | [description] | [one-line fix] |
|
||
|
||
### Deferred to 1.1
|
||
| BUG-ID | Severity | Description | Reason deferred |
|
||
|--------|----------|-------------|-----------------|
|
||
| BUG-NNN | S[1-4] | [description] | [reason] |
|
||
|
||
---
|
||
|
||
## QA Sign-Off
|
||
|
||
**QA scope**: [Targeted smoke / Broader regression]
|
||
**Verdict**: [PASS / PASS WITH WARNINGS]
|
||
**QA lead**: qa-lead agent
|
||
**Date**: [date]
|
||
**Warnings (if any)**: [list or "None"]
|
||
|
||
---
|
||
|
||
## Rollback Plan
|
||
|
||
See: `production/releases/rollback-plan-[version].md`
|
||
|
||
**Trigger condition**: If [N] or more S1 bugs are reported within [X] hours of launch, execute rollback.
|
||
**Rollback owner**: [user / producer]
|
||
|
||
---
|
||
|
||
## Approvals Required Before Deploy
|
||
|
||
- [ ] lead-programmer: all fixes reviewed
|
||
- [ ] qa-lead: QA gate PASS confirmed
|
||
- [ ] producer: deployment timing approved
|
||
- [ ] release-manager: platform submission confirmed
|
||
|
||
---
|
||
|
||
## Player-Facing Patch Notes
|
||
|
||
[Draft for community-manager to review before publishing]
|
||
|
||
[list player-facing changes in plain language]
|
||
```
|
||
|
||
Ask: "May I write this patch record to `production/releases/day-one-patch-[version].md`?"
|
||
|
||
---
|
||
|
||
## Phase 7: Next Steps
|
||
|
||
After the patch record is written:
|
||
|
||
1. Run `/patch-notes` to generate the player-facing version of the patch notes
|
||
2. Run `/bug-report verify [BUG-ID]` for each fixed bug after the patch is live
|
||
3. Run `/bug-report close [BUG-ID]` for each verified fix
|
||
4. Schedule a post-launch review 48–72 hours after launch using `/retrospective launch`
|
||
|
||
**If any S1 bugs remain open after the patch:**
|
||
> "⚠️ S1 bugs remain open and were not patched. These are accepted risks. Document them in the rollback plan trigger conditions — if they occur at scale, rollback may be preferable to a follow-up patch."
|
||
|
||
Use `AskUserQuestion`:
|
||
- Prompt: "Day-one patch complete. What's next?"
|
||
- Options:
|
||
- `[A] Run /patch-notes — generate player-facing patch notes`
|
||
- `[B] Run /bug-report to log any issues found post-deploy`
|
||
- `[C] Stop here`
|
||
|
||
---
|
||
|
||
## Collaborative Protocol
|
||
|
||
- **Scope discipline is everything** — resist scope creep; every addition increases risk
|
||
- **Rollback plan first, always** — a patch without a rollback plan is irresponsible
|
||
- **Deferred is not forgotten** — every deferred bug gets a 1.1 ticket automatically
|
||
- **Player communication is part of the patch** — `/patch-notes` is a required output, not optional
|