nccgs 1.1.1 → 1.3.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (212) hide show
  1. package/.claude/agents/nccgs-adversarial-reviewer.md +7 -1
  2. package/.claude/agents/nccgs-engine-programmer.md +2 -1
  3. package/.claude/agents/nccgs-gameplay-programmer.md +2 -1
  4. package/.claude/agents/nccgs-qa-engineer.md +2 -1
  5. package/.claude/agents/nccgs-ui-programmer.md +2 -1
  6. package/.claude/agents/nccgs-unity-build-specialist.md +2 -1
  7. package/.claude/agents/nccgs-unity-implementer.md +2 -1
  8. package/.claude/agents/nccgs-unity-systems-specialist.md +2 -1
  9. package/.claude/agents/nccgs-unity-ui-specialist.md +2 -1
  10. package/.claude/agents/nccgs-verification-engineer.md +2 -1
  11. package/.claude/nccgs/VERSION +1 -1
  12. package/.claude/{agents → nccgs/agent-library}/nccgs-accessibility-specialist.md +0 -1
  13. package/.claude/nccgs/agent-library/nccgs-adversarial-reviewer.md +30 -0
  14. package/.claude/{agents → nccgs/agent-library}/nccgs-ai-programmer.md +0 -1
  15. package/.claude/{agents → nccgs/agent-library}/nccgs-analytics-engineer.md +0 -1
  16. package/.claude/{agents → nccgs/agent-library}/nccgs-art-direction-lead.md +0 -1
  17. package/.claude/{agents → nccgs/agent-library}/nccgs-audio-direction-lead.md +0 -1
  18. package/.claude/{agents → nccgs/agent-library}/nccgs-creative-director.md +0 -1
  19. package/.claude/{agents → nccgs/agent-library}/nccgs-documentation-manager.md +0 -1
  20. package/.claude/{agents → nccgs/agent-library}/nccgs-economy-designer.md +0 -1
  21. package/.claude/nccgs/agent-library/nccgs-engine-programmer.md +25 -0
  22. package/.claude/{agents → nccgs/agent-library}/nccgs-game-design-lead.md +0 -1
  23. package/.claude/{agents → nccgs/agent-library}/nccgs-game-designer.md +0 -1
  24. package/.claude/nccgs/agent-library/nccgs-gameplay-programmer.md +25 -0
  25. package/.claude/{agents → nccgs/agent-library}/nccgs-level-designer.md +0 -1
  26. package/.claude/{agents → nccgs/agent-library}/nccgs-live-ops-designer.md +0 -1
  27. package/.claude/{agents → nccgs/agent-library}/nccgs-localization-lead.md +0 -1
  28. package/.claude/{agents → nccgs/agent-library}/nccgs-narrative-lead.md +0 -1
  29. package/.claude/{agents → nccgs/agent-library}/nccgs-network-programmer.md +0 -1
  30. package/.claude/{agents → nccgs/agent-library}/nccgs-performance-analyst.md +0 -1
  31. package/.claude/{agents → nccgs/agent-library}/nccgs-producer.md +0 -1
  32. package/.claude/{agents → nccgs/agent-library}/nccgs-production-coordinator.md +0 -1
  33. package/.claude/{agents → nccgs/agent-library}/nccgs-programming-lead.md +0 -1
  34. package/.claude/{agents → nccgs/agent-library}/nccgs-prototyper.md +0 -1
  35. package/.claude/nccgs/agent-library/nccgs-qa-engineer.md +25 -0
  36. package/.claude/{agents → nccgs/agent-library}/nccgs-qa-lead.md +0 -1
  37. package/.claude/{agents → nccgs/agent-library}/nccgs-release-engineer.md +0 -1
  38. package/.claude/{agents → nccgs/agent-library}/nccgs-release-lead.md +0 -1
  39. package/.claude/{agents → nccgs/agent-library}/nccgs-security-engineer.md +0 -1
  40. package/.claude/{agents → nccgs/agent-library}/nccgs-sound-designer.md +0 -1
  41. package/.claude/{agents → nccgs/agent-library}/nccgs-systems-designer.md +0 -1
  42. package/.claude/{agents → nccgs/agent-library}/nccgs-technical-architect.md +0 -1
  43. package/.claude/{agents → nccgs/agent-library}/nccgs-technical-artist.md +0 -1
  44. package/.claude/{agents → nccgs/agent-library}/nccgs-technical-director.md +0 -1
  45. package/.claude/{agents → nccgs/agent-library}/nccgs-tools-programmer.md +0 -1
  46. package/.claude/nccgs/agent-library/nccgs-ui-programmer.md +25 -0
  47. package/.claude/nccgs/agent-library/nccgs-unity-build-specialist.md +25 -0
  48. package/.claude/{agents → nccgs/agent-library}/nccgs-unity-content-specialist.md +0 -1
  49. package/.claude/nccgs/agent-library/nccgs-unity-implementer.md +25 -0
  50. package/.claude/{agents → nccgs/agent-library}/nccgs-unity-rendering-specialist.md +0 -1
  51. package/.claude/nccgs/agent-library/nccgs-unity-systems-specialist.md +25 -0
  52. package/.claude/nccgs/agent-library/nccgs-unity-ui-specialist.md +25 -0
  53. package/.claude/{agents → nccgs/agent-library}/nccgs-ux-designer.md +0 -1
  54. package/.claude/nccgs/agent-library/nccgs-verification-engineer.md +25 -0
  55. package/.claude/{agents → nccgs/agent-library}/nccgs-world-builder.md +0 -1
  56. package/.claude/{agents → nccgs/agent-library}/nccgs-writer.md +0 -1
  57. package/.claude/nccgs/constitution.md +13 -0
  58. package/.claude/nccgs/hooks/agent-audit.mjs +6 -0
  59. package/.claude/nccgs/hooks/closure-guard.mjs +34 -0
  60. package/.claude/nccgs/hooks/runtime-observation.mjs +11 -0
  61. package/.claude/nccgs/hooks/session-start.mjs +8 -1
  62. package/.claude/nccgs/pipeline.json +176 -0
  63. package/.claude/nccgs/protocols/agent-contract.md +5 -0
  64. package/.claude/nccgs/protocols/astra-claude.md +77 -0
  65. package/.claude/nccgs/protocols/evidence.md +6 -0
  66. package/.claude/nccgs/protocols/model-routing.md +27 -5
  67. package/.claude/nccgs/protocols/orchestration.md +5 -0
  68. package/.claude/nccgs/protocols/task-gates.md +66 -0
  69. package/.claude/nccgs/routing.generated.md +11 -0
  70. package/.claude/nccgs/settings.fragment.json +73 -9
  71. package/.claude/nccgs/studio.json +187 -96
  72. package/.claude/nccgs/tools/compile-policy.mjs +83 -0
  73. package/.claude/nccgs/tools/configure-models.mjs +75 -35
  74. package/.claude/nccgs/tools/gates.mjs +167 -0
  75. package/.claude/nccgs/tools/policy.mjs +62 -0
  76. package/.claude/nccgs/tools/runtime.mjs +40 -0
  77. package/.claude/nccgs/tools/task.mjs +77 -0
  78. package/.claude/nccgs/workflow-catalog.json +90 -13
  79. package/.claude/skills/accessibility-review/SKILL.md +6 -5
  80. package/.claude/skills/accessibility-review/references/astra.md +17 -0
  81. package/.claude/skills/accessibility-review/references/legacy.md +7 -0
  82. package/.claude/skills/architecture-decision/SKILL.md +6 -5
  83. package/.claude/skills/architecture-decision/references/astra.md +12 -0
  84. package/.claude/skills/architecture-decision/references/legacy.md +7 -0
  85. package/.claude/skills/asset-audit/SKILL.md +6 -5
  86. package/.claude/skills/asset-audit/references/astra.md +17 -0
  87. package/.claude/skills/asset-audit/references/legacy.md +7 -0
  88. package/.claude/skills/audit/SKILL.md +6 -7
  89. package/.claude/skills/audit/references/astra.md +17 -0
  90. package/.claude/skills/audit/references/legacy.md +9 -0
  91. package/.claude/skills/balance-review/SKILL.md +6 -5
  92. package/.claude/skills/balance-review/references/astra.md +17 -0
  93. package/.claude/skills/balance-review/references/legacy.md +7 -0
  94. package/.claude/skills/bug-triage/SKILL.md +6 -5
  95. package/.claude/skills/bug-triage/references/astra.md +20 -0
  96. package/.claude/skills/bug-triage/references/legacy.md +7 -0
  97. package/.claude/skills/closure/SKILL.md +5 -4
  98. package/.claude/skills/closure/references/astra.md +15 -0
  99. package/.claude/skills/closure/references/legacy.md +7 -0
  100. package/.claude/skills/compatibility-review/SKILL.md +6 -5
  101. package/.claude/skills/compatibility-review/references/astra.md +17 -0
  102. package/.claude/skills/compatibility-review/references/legacy.md +7 -0
  103. package/.claude/skills/context-pack/SKILL.md +5 -4
  104. package/.claude/skills/context-pack/references/astra.md +12 -0
  105. package/.claude/skills/context-pack/references/legacy.md +7 -0
  106. package/.claude/skills/dependency-review/SKILL.md +6 -5
  107. package/.claude/skills/dependency-review/references/astra.md +17 -0
  108. package/.claude/skills/dependency-review/references/legacy.md +7 -0
  109. package/.claude/skills/design/SKILL.md +6 -11
  110. package/.claude/skills/design/references/astra.md +12 -0
  111. package/.claude/skills/design/references/legacy.md +13 -0
  112. package/.claude/skills/design-review/SKILL.md +6 -5
  113. package/.claude/skills/design-review/references/astra.md +17 -0
  114. package/.claude/skills/design-review/references/legacy.md +7 -0
  115. package/.claude/skills/evidence-review/SKILL.md +6 -5
  116. package/.claude/skills/evidence-review/references/astra.md +17 -0
  117. package/.claude/skills/evidence-review/references/legacy.md +7 -0
  118. package/.claude/skills/hotfix/SKILL.md +6 -5
  119. package/.claude/skills/hotfix/references/astra.md +23 -0
  120. package/.claude/skills/hotfix/references/legacy.md +7 -0
  121. package/.claude/skills/incident-recovery/SKILL.md +6 -5
  122. package/.claude/skills/incident-recovery/references/astra.md +20 -0
  123. package/.claude/skills/incident-recovery/references/legacy.md +7 -0
  124. package/.claude/skills/localize-game/SKILL.md +6 -5
  125. package/.claude/skills/localize-game/references/astra.md +20 -0
  126. package/.claude/skills/localize-game/references/legacy.md +7 -0
  127. package/.claude/skills/migrate-project/SKILL.md +6 -16
  128. package/.claude/skills/migrate-project/references/astra.md +23 -0
  129. package/.claude/skills/migrate-project/references/legacy.md +19 -0
  130. package/.claude/skills/migrate-project/references/procedure.md +8 -0
  131. package/.claude/skills/milestone-review/SKILL.md +6 -5
  132. package/.claude/skills/milestone-review/references/astra.md +12 -0
  133. package/.claude/skills/milestone-review/references/legacy.md +7 -0
  134. package/.claude/skills/nccgs-code-review/SKILL.md +6 -4
  135. package/.claude/skills/nccgs-code-review/references/astra.md +15 -0
  136. package/.claude/skills/nccgs-code-review/references/legacy.md +7 -0
  137. package/.claude/skills/performance-audit/SKILL.md +6 -5
  138. package/.claude/skills/performance-audit/references/astra.md +17 -0
  139. package/.claude/skills/performance-audit/references/legacy.md +7 -0
  140. package/.claude/skills/plan-feature/SKILL.md +6 -5
  141. package/.claude/skills/plan-feature/references/astra.md +12 -0
  142. package/.claude/skills/plan-feature/references/legacy.md +7 -0
  143. package/.claude/skills/playtest/SKILL.md +5 -4
  144. package/.claude/skills/playtest/references/astra.md +20 -0
  145. package/.claude/skills/playtest/references/legacy.md +7 -0
  146. package/.claude/skills/project-stage/SKILL.md +5 -4
  147. package/.claude/skills/project-stage/references/astra.md +12 -0
  148. package/.claude/skills/project-stage/references/legacy.md +7 -0
  149. package/.claude/skills/prototype-feature/SKILL.md +6 -5
  150. package/.claude/skills/prototype-feature/references/astra.md +20 -0
  151. package/.claude/skills/prototype-feature/references/legacy.md +7 -0
  152. package/.claude/skills/qa-plan/SKILL.md +6 -5
  153. package/.claude/skills/qa-plan/references/astra.md +12 -0
  154. package/.claude/skills/qa-plan/references/legacy.md +7 -0
  155. package/.claude/skills/release/SKILL.md +6 -9
  156. package/.claude/skills/release/references/astra.md +23 -0
  157. package/.claude/skills/release/references/legacy.md +11 -0
  158. package/.claude/skills/release-readiness/SKILL.md +6 -5
  159. package/.claude/skills/release-readiness/references/astra.md +17 -0
  160. package/.claude/skills/release-readiness/references/legacy.md +7 -0
  161. package/.claude/skills/retrospective/SKILL.md +6 -5
  162. package/.claude/skills/retrospective/references/astra.md +12 -0
  163. package/.claude/skills/retrospective/references/legacy.md +7 -0
  164. package/.claude/skills/review/SKILL.md +6 -7
  165. package/.claude/skills/review/references/astra.md +15 -0
  166. package/.claude/skills/review/references/legacy.md +9 -0
  167. package/.claude/skills/security-audit/SKILL.md +6 -5
  168. package/.claude/skills/security-audit/references/astra.md +17 -0
  169. package/.claude/skills/security-audit/references/legacy.md +7 -0
  170. package/.claude/skills/sprint-plan/SKILL.md +6 -5
  171. package/.claude/skills/sprint-plan/references/astra.md +12 -0
  172. package/.claude/skills/sprint-plan/references/legacy.md +7 -0
  173. package/.claude/skills/status/SKILL.md +5 -6
  174. package/.claude/skills/status/references/astra.md +13 -0
  175. package/.claude/skills/status/references/legacy.md +9 -0
  176. package/.claude/skills/story-readiness/SKILL.md +5 -4
  177. package/.claude/skills/story-readiness/references/astra.md +17 -0
  178. package/.claude/skills/story-readiness/references/legacy.md +7 -0
  179. package/.claude/skills/test/SKILL.md +6 -7
  180. package/.claude/skills/test/references/astra.md +20 -0
  181. package/.claude/skills/test/references/legacy.md +9 -0
  182. package/.claude/skills/ui-review/SKILL.md +6 -5
  183. package/.claude/skills/ui-review/references/astra.md +17 -0
  184. package/.claude/skills/ui-review/references/legacy.md +7 -0
  185. package/.claude/skills/work/SKILL.md +6 -27
  186. package/.claude/skills/work/references/astra.md +20 -0
  187. package/.claude/skills/work/references/closure.md +10 -0
  188. package/.claude/skills/work/references/feature-contracts.md +4 -0
  189. package/.claude/skills/work/references/legacy.md +29 -0
  190. package/README.md +59 -25
  191. package/UPGRADING.md +37 -0
  192. package/VERSION +1 -1
  193. package/docs/ARCHITECTURE.md +30 -13
  194. package/docs/HUONG-DAN-MIGRATE-VA-SU-DUNG.md +34 -329
  195. package/docs/MIGRATION-MATRIX.md +1 -1
  196. package/docs/NCCGS-1.3.md +125 -0
  197. package/docs/PROJECT-POLICY.md +18 -3
  198. package/docs/WORKFLOWS.md +19 -13
  199. package/package.json +5 -3
  200. package/scaffold/.nccgs/gates.json +7 -0
  201. package/scaffold/.nccgs/project.yaml +6 -3
  202. package/scaffold/.nccgs/state.md +1 -1
  203. package/scaffold/.nccgs/templates/agent-handoff.md +10 -0
  204. package/scaffold/.nccgs/templates/closure-record.md +11 -0
  205. package/scaffold/.nccgs/templates/feature-contract.md +9 -0
  206. package/scaffold/.nccgs/templates/implementation-report.md +49 -0
  207. package/scripts/cli.mjs +9 -4
  208. package/scripts/doctor.mjs +27 -1
  209. package/scripts/install.mjs +4 -3
  210. package/scripts/validate.mjs +20 -7
  211. package/tests/framework.test.mjs +143 -2
  212. package/tests/gates.test.mjs +170 -0
@@ -1,14 +1,15 @@
1
1
  ---
2
2
  name: dependency-review
3
3
  description: Evaluates a proposed Unity package, SDK, service, or external dependency for necessity, fit, licensing, security, maintenance, compatibility, lock-in, migration, and rollback before adoption.
4
- allowed-tools: Read, Glob, Grep, Bash, Task, WebSearch, WebFetch
4
+ allowed-tools: Read, Glob, Grep, Bash, Agent, WebSearch, WebFetch
5
5
  ---
6
6
 
7
7
  # Dependency Review
8
8
 
9
- Read `.nccgs/project.yaml`, `.nccgs/state.md`, and the active context packet when present. Preserve Product Owner authority and unrelated work.
9
+ Read models.profile in .nccgs/project.yaml; if absent, use the defaultProfile in
10
+ .claude/nccgs/pipeline.json. Load exactly one procedure:
10
11
 
11
- Remain read-only. Define the capability need and compare build, buy, existing-project, and no-change options. Verify current authoritative documentation, supported Unity/platform versions, license, data/privacy, operational cost, update policy, failure behavior, transitive risk, and exit plan. Recommend with evidence. Adoption or replacement requires explicit Product Owner authority and an architecture decision.
12
-
13
- Return outcome, evidence, affected records, blockers or decisions, residual risk, and the exact next workflow. Do not claim DONE unless this is the `closure` skill and every configured gate passes.
12
+ - astra-claude: [Astra/Claude procedure](references/astra.md).
13
+ - quality, balanced, inherit: [Legacy procedure](references/legacy.md).
14
14
 
15
+ Do not load or merge the other branch. Unknown profiles are configuration errors.
@@ -0,0 +1,17 @@
1
+ # Dependency Review — astra-claude
2
+
3
+ Read [the pipeline protocol](../../../nccgs/protocols/astra-claude.md) and
4
+ [generated routing](../../../nccgs/routing.generated.md). Use Astra's supplied plan,
5
+ requirements, and exact context references. Never substitute a legacy procedure.
6
+
7
+ Domain objective: Evaluates a proposed Unity package, SDK, service, or external dependency for necessity, fit, licensing, security, maintenance, compatibility, lock-in, migration, and rollback before adoption.
8
+
9
+ Work only inside the supplied plan and assigned write set. Activate an optional
10
+ specialist with the agents command only when its domain is required. Keep one writer
11
+ per path and use official Unity Skills for Editor/assets/runtime operations.
12
+
13
+ This procedure is read-only: return evidence-backed findings and decisions to Astra
14
+ or the implementer. Supplemental analysis does not replace the mandatory Fable gate.
15
+
16
+ Use [task gate commands](../../../nccgs/protocols/task-gates.md) for evidence and
17
+ state transitions. Report outcome, evidence, pending decisions, and next owner.
@@ -0,0 +1,7 @@
1
+ # Dependency Review
2
+
3
+ Read `.nccgs/project.yaml`, `.nccgs/state.md`, and the active context packet when present. Preserve Product Owner authority and unrelated work.
4
+
5
+ Remain read-only. Define the capability need and compare build, buy, existing-project, and no-change options. Verify current authoritative documentation, supported Unity/platform versions, license, data/privacy, operational cost, update policy, failure behavior, transitive risk, and exit plan. Recommend with evidence. Adoption or replacement requires explicit Product Owner authority and an architecture decision.
6
+
7
+ Return outcome, evidence, affected records, blockers or decisions, residual risk, and the exact next workflow. Do not claim DONE unless this is the `closure` skill and every configured gate passes.
@@ -1,20 +1,15 @@
1
1
  ---
2
2
  name: design
3
3
  description: Designs or revises player-facing mechanics, systems, UX, levels, economy, narrative, content, or accessibility before implementation. Routes to focused design roles and preserves Product Owner control of subjective and canonical decisions.
4
- allowed-tools: Read, Glob, Grep, Write, Edit, Task, AskUserQuestion, WebSearch, WebFetch
4
+ allowed-tools: Read, Glob, Grep, Write, Edit, Agent, AskUserQuestion, WebSearch, WebFetch
5
5
  ---
6
6
 
7
7
  # Design
8
8
 
9
- Produce implementation-ready design truth with the smallest complete set of specialized perspectives.
9
+ Read models.profile in .nccgs/project.yaml; if absent, use the defaultProfile in
10
+ .claude/nccgs/pipeline.json. Load exactly one procedure:
10
11
 
11
- Read policy, state, relevant pillars/canon, interacting requirements, and the code constraints needed for feasibility. Distinguish exploration, focused design, and CONTROLLED cross-system change.
12
-
13
- Select the accountable design role and only the specialists required by the domain: systems, level, economy, UX, accessibility, narrative, art, audio, live ops, or prototyping. Use Opus leadership only for cross-creative conflict or product-wide synthesis.
14
-
15
- Separate established facts from proposals and Product Owner decisions. Batch meaningful choices with real alternatives, tradeoffs, and a recommendation. Do not ask implementation trivia.
16
-
17
- Define applicable rules, states, formulas, feedback, dependencies, edge cases, tuning surfaces, objective acceptance criteria, subjective gates, out-of-scope behavior, and traceable requirement IDs. Consult technical roles for feasibility without letting implementation redefine intended design.
18
-
19
- Persist accepted changes in mapped canonical sources; do not create a parallel GDD tree. Run `/design-review` for STANDARD or CONTROLLED changes. If implementation is included and decisions are closed, hand the accepted contract to `/work`.
12
+ - astra-claude: [Astra/Claude procedure](references/astra.md).
13
+ - quality, balanced, inherit: [Legacy procedure](references/legacy.md).
20
14
 
15
+ Do not load or merge the other branch. Unknown profiles are configuration errors.
@@ -0,0 +1,12 @@
1
+ # Design — astra-claude
2
+
3
+ Read [the pipeline protocol](../../../nccgs/protocols/astra-claude.md) and
4
+ [generated routing](../../../nccgs/routing.generated.md). Use Astra's supplied plan,
5
+ requirements, and exact context references. Never substitute a legacy procedure.
6
+
7
+ Astra owns this workflow and its canonical outputs. Claude only inspects relevant
8
+ inputs and returns factual constraints, missing information, or proposed amendments.
9
+ Do not create a replacement plan/document or sign off as Astra.
10
+
11
+ Use [task gate commands](../../../nccgs/protocols/task-gates.md) for evidence and
12
+ state transitions. Report outcome, evidence, pending decisions, and next owner.
@@ -0,0 +1,13 @@
1
+ # Design
2
+
3
+ Produce implementation-ready design truth with the smallest complete set of specialized perspectives.
4
+
5
+ Read policy, state, relevant pillars/canon, interacting requirements, and the code constraints needed for feasibility. Distinguish exploration, focused design, and CONTROLLED cross-system change.
6
+
7
+ Select the accountable design role and only the specialists required by the domain: systems, level, economy, UX, accessibility, narrative, art, audio, live ops, or prototyping. Use Opus leadership only for cross-creative conflict or product-wide synthesis.
8
+
9
+ Separate established facts from proposals and Product Owner decisions. Batch meaningful choices with real alternatives, tradeoffs, and a recommendation. Do not ask implementation trivia.
10
+
11
+ Define applicable rules, states, formulas, feedback, dependencies, edge cases, tuning surfaces, objective acceptance criteria, subjective gates, out-of-scope behavior, and traceable requirement IDs. Consult technical roles for feasibility without letting implementation redefine intended design.
12
+
13
+ Persist accepted changes in mapped canonical sources; do not create a parallel GDD tree. Run `/design-review` for STANDARD or CONTROLLED changes. If implementation is included and decisions are closed, hand the accepted contract to `/work`.
@@ -1,14 +1,15 @@
1
1
  ---
2
2
  name: design-review
3
3
  description: Reviews a proposed or changed game design for clarity, player intent, systemic interactions, balance, UX, accessibility, canon, feasibility, and testability without approving subjective choices.
4
- allowed-tools: Read, Glob, Grep, Task
4
+ allowed-tools: Read, Glob, Grep, Agent
5
5
  ---
6
6
 
7
7
  # Design Review
8
8
 
9
- Read `.nccgs/project.yaml`, `.nccgs/state.md`, and the active context packet when present. Preserve Product Owner authority and unrelated work.
9
+ Read models.profile in .nccgs/project.yaml; if absent, use the defaultProfile in
10
+ .claude/nccgs/pipeline.json. Load exactly one procedure:
10
11
 
11
- Remain read-only. Review the target against pillars, canonical decisions, interacting systems, platform constraints, and implementation evidence only where relevant. Use focused specialists for economy, UX, accessibility, level, narrative, art, or audio lenses. Separate contradictions, exploitable/degenerate behavior, untestable requirements, and optional improvements. Return prioritized findings, PO decisions, and implementation-readiness verdict.
12
-
13
- Return outcome, evidence, affected records, blockers or decisions, residual risk, and the exact next workflow. Do not claim DONE unless this is the `closure` skill and every configured gate passes.
12
+ - astra-claude: [Astra/Claude procedure](references/astra.md).
13
+ - quality, balanced, inherit: [Legacy procedure](references/legacy.md).
14
14
 
15
+ Do not load or merge the other branch. Unknown profiles are configuration errors.
@@ -0,0 +1,17 @@
1
+ # Design Review — astra-claude
2
+
3
+ Read [the pipeline protocol](../../../nccgs/protocols/astra-claude.md) and
4
+ [generated routing](../../../nccgs/routing.generated.md). Use Astra's supplied plan,
5
+ requirements, and exact context references. Never substitute a legacy procedure.
6
+
7
+ Domain objective: Reviews a proposed or changed game design for clarity, player intent, systemic interactions, balance, UX, accessibility, canon, feasibility, and testability without approving subjective choices.
8
+
9
+ Work only inside the supplied plan and assigned write set. Activate an optional
10
+ specialist with the agents command only when its domain is required. Keep one writer
11
+ per path and use official Unity Skills for Editor/assets/runtime operations.
12
+
13
+ This procedure is read-only: return evidence-backed findings and decisions to Astra
14
+ or the implementer. Supplemental analysis does not replace the mandatory Fable gate.
15
+
16
+ Use [task gate commands](../../../nccgs/protocols/task-gates.md) for evidence and
17
+ state transitions. Report outcome, evidence, pending decisions, and next owner.
@@ -0,0 +1,7 @@
1
+ # Design Review
2
+
3
+ Read `.nccgs/project.yaml`, `.nccgs/state.md`, and the active context packet when present. Preserve Product Owner authority and unrelated work.
4
+
5
+ Remain read-only. Review the target against pillars, canonical decisions, interacting systems, platform constraints, and implementation evidence only where relevant. Use focused specialists for economy, UX, accessibility, level, narrative, art, or audio lenses. Separate contradictions, exploitable/degenerate behavior, untestable requirements, and optional improvements. Return prioritized findings, PO decisions, and implementation-readiness verdict.
6
+
7
+ Return outcome, evidence, affected records, blockers or decisions, residual risk, and the exact next workflow. Do not claim DONE unless this is the `closure` skill and every configured gate passes.
@@ -1,14 +1,15 @@
1
1
  ---
2
2
  name: evidence-review
3
3
  description: Independently evaluates whether accumulated tests, runtime observations, builds, reports, and manual gates actually prove each requirement and support the claimed closure state.
4
- allowed-tools: Read, Glob, Grep, Bash, Task
4
+ allowed-tools: Read, Glob, Grep, Bash, Agent
5
5
  ---
6
6
 
7
7
  # Evidence Review
8
8
 
9
- Read `.nccgs/project.yaml`, `.nccgs/state.md`, and the active context packet when present. Preserve Product Owner authority and unrelated work.
9
+ Read models.profile in .nccgs/project.yaml; if absent, use the defaultProfile in
10
+ .claude/nccgs/pipeline.json. Load exactly one procedure:
10
11
 
11
- Read the [evidence protocol](../../nccgs/protocols/evidence.md), requirements, evidence records, test outputs, runtime/build observations, baseline, and waivers. Map every criterion to PASS, FAIL, NOT PROVEN, PO GATE, or N/A. Detect stale source revisions, unexplained count/skip changes, evidence produced before the final diff, and claims based only on inspection. Return closure impact and exact evidence still required.
12
-
13
- Return outcome, evidence, affected records, blockers or decisions, residual risk, and the exact next workflow. Do not claim DONE unless this is the `closure` skill and every configured gate passes.
12
+ - astra-claude: [Astra/Claude procedure](references/astra.md).
13
+ - quality, balanced, inherit: [Legacy procedure](references/legacy.md).
14
14
 
15
+ Do not load or merge the other branch. Unknown profiles are configuration errors.
@@ -0,0 +1,17 @@
1
+ # Evidence Review — astra-claude
2
+
3
+ Read [the pipeline protocol](../../../nccgs/protocols/astra-claude.md) and
4
+ [generated routing](../../../nccgs/routing.generated.md). Use Astra's supplied plan,
5
+ requirements, and exact context references. Never substitute a legacy procedure.
6
+
7
+ Domain objective: Independently evaluates whether accumulated tests, runtime observations, builds, reports, and manual gates actually prove each requirement and support the claimed closure state.
8
+
9
+ Work only inside the supplied plan and assigned write set. Activate an optional
10
+ specialist with the agents command only when its domain is required. Keep one writer
11
+ per path and use official Unity Skills for Editor/assets/runtime operations.
12
+
13
+ This procedure is read-only: return evidence-backed findings and decisions to Astra
14
+ or the implementer. Supplemental analysis does not replace the mandatory Fable gate.
15
+
16
+ Use [task gate commands](../../../nccgs/protocols/task-gates.md) for evidence and
17
+ state transitions. Report outcome, evidence, pending decisions, and next owner.
@@ -0,0 +1,7 @@
1
+ # Evidence Review
2
+
3
+ Read `.nccgs/project.yaml`, `.nccgs/state.md`, and the active context packet when present. Preserve Product Owner authority and unrelated work.
4
+
5
+ Read the [evidence protocol](../../../nccgs/protocols/evidence.md), requirements, evidence records, test outputs, runtime/build observations, baseline, and waivers. Map every criterion to PASS, FAIL, NOT PROVEN, PO GATE, or N/A. Detect stale source revisions, unexplained count/skip changes, evidence produced before the final diff, and claims based only on inspection. Return closure impact and exact evidence still required.
6
+
7
+ Return outcome, evidence, affected records, blockers or decisions, residual risk, and the exact next workflow. Do not claim DONE unless this is the `closure` skill and every configured gate passes.
@@ -1,14 +1,15 @@
1
1
  ---
2
2
  name: hotfix
3
3
  description: Runs a tightly scoped CONTROLLED workflow for urgent production or release-critical defects with containment, minimal change, accelerated evidence, rollback, and follow-up debt.
4
- allowed-tools: Read, Glob, Grep, Write, Edit, Bash, Task, AskUserQuestion
4
+ allowed-tools: Read, Glob, Grep, Write, Edit, Bash, Agent, AskUserQuestion
5
5
  ---
6
6
 
7
7
  # Hotfix
8
8
 
9
- Read `.nccgs/project.yaml`, `.nccgs/state.md`, and the active context packet when present. Preserve Product Owner authority and unrelated work.
9
+ Read models.profile in .nccgs/project.yaml; if absent, use the defaultProfile in
10
+ .claude/nccgs/pipeline.json. Load exactly one procedure:
10
11
 
11
- Confirm affected version, severity, user/data impact, reproduction, containment, release branch, and rollback. Build the smallest safe fix with one accountable writer; preserve compatibility and avoid unrelated refactors. Run targeted regression plus the release-critical checks justified by blast radius. Record evidence, known gaps, approval, release steps, rollback trigger, and required follow-up. Publication still requires explicit authority.
12
-
13
- Return outcome, evidence, affected records, blockers or decisions, residual risk, and the exact next workflow. Do not claim DONE unless this is the `closure` skill and every configured gate passes.
12
+ - astra-claude: [Astra/Claude procedure](references/astra.md).
13
+ - quality, balanced, inherit: [Legacy procedure](references/legacy.md).
14
14
 
15
+ Do not load or merge the other branch. Unknown profiles are configuration errors.
@@ -0,0 +1,23 @@
1
+ # Hotfix — astra-claude
2
+
3
+ Read [the pipeline protocol](../../../nccgs/protocols/astra-claude.md) and
4
+ [generated routing](../../../nccgs/routing.generated.md). Use Astra's supplied plan,
5
+ requirements, and exact context references. Never substitute a legacy procedure.
6
+
7
+ Domain objective: Runs a tightly scoped CONTROLLED workflow for urgent production or release-critical defects with containment, minimal change, accelerated evidence, rollback, and follow-up debt.
8
+
9
+ Work only inside the supplied plan and assigned write set. Activate an optional
10
+ specialist with the agents command only when its domain is required. Keep one writer
11
+ per path and use official Unity Skills for Editor/assets/runtime operations.
12
+
13
+ For code changes: task init with Astra's plan, bind the session, implement and test,
14
+ record implementation evidence, request independent review, fix and re-review as
15
+ needed, then task check --target report. Only after that check passes write and
16
+ record the report for Astra. Stop at AWAITING_ASTRA_CHECK. Astra checking and the
17
+ Product Owner's explicit acceptance happen outside the Claude implementation session.
18
+
19
+ Publication and other external release actions still require explicit Product Owner
20
+ authority; successful local verification grants no publication permission.
21
+
22
+ Use [task gate commands](../../../nccgs/protocols/task-gates.md) for evidence and
23
+ state transitions. Report outcome, evidence, pending decisions, and next owner.
@@ -0,0 +1,7 @@
1
+ # Hotfix
2
+
3
+ Read `.nccgs/project.yaml`, `.nccgs/state.md`, and the active context packet when present. Preserve Product Owner authority and unrelated work.
4
+
5
+ Confirm affected version, severity, user/data impact, reproduction, containment, release branch, and rollback. Build the smallest safe fix with one accountable writer; preserve compatibility and avoid unrelated refactors. Run targeted regression plus the release-critical checks justified by blast radius. Record evidence, known gaps, approval, release steps, rollback trigger, and required follow-up. Publication still requires explicit authority.
6
+
7
+ Return outcome, evidence, affected records, blockers or decisions, residual risk, and the exact next workflow. Do not claim DONE unless this is the `closure` skill and every configured gate passes.
@@ -1,14 +1,15 @@
1
1
  ---
2
2
  name: incident-recovery
3
3
  description: Coordinates evidence-preserving response to data loss, corruption, broken builds, failed migrations, or production-impacting Unity incidents.
4
- allowed-tools: Read, Glob, Grep, Write, Edit, Bash, Task, AskUserQuestion
4
+ allowed-tools: Read, Glob, Grep, Write, Edit, Bash, Agent, AskUserQuestion
5
5
  ---
6
6
 
7
7
  # Incident Recovery
8
8
 
9
- Read `.nccgs/project.yaml`, `.nccgs/state.md`, and the active context packet when present. Preserve Product Owner authority and unrelated work.
9
+ Read models.profile in .nccgs/project.yaml; if absent, use the defaultProfile in
10
+ .claude/nccgs/pipeline.json. Load exactly one procedure:
10
11
 
11
- Prioritize containment and evidence preservation. Record incident scope, affected versions/data, current state, timeline, hypotheses, and irreversible-action boundaries. Avoid destructive cleanup and speculative rewrites. Build a recovery graph with backups, reproducible diagnostics, validation, and rollback. Apply only authorized recovery changes; maintain an incident record and follow-up prevention actions. Data deletion or production actions require explicit authority.
12
-
13
- Return outcome, evidence, affected records, blockers or decisions, residual risk, and the exact next workflow. Do not claim DONE unless this is the `closure` skill and every configured gate passes.
12
+ - astra-claude: [Astra/Claude procedure](references/astra.md).
13
+ - quality, balanced, inherit: [Legacy procedure](references/legacy.md).
14
14
 
15
+ Do not load or merge the other branch. Unknown profiles are configuration errors.
@@ -0,0 +1,20 @@
1
+ # Incident Recovery — astra-claude
2
+
3
+ Read [the pipeline protocol](../../../nccgs/protocols/astra-claude.md) and
4
+ [generated routing](../../../nccgs/routing.generated.md). Use Astra's supplied plan,
5
+ requirements, and exact context references. Never substitute a legacy procedure.
6
+
7
+ Domain objective: Coordinates evidence-preserving response to data loss, corruption, broken builds, failed migrations, or production-impacting Unity incidents.
8
+
9
+ Work only inside the supplied plan and assigned write set. Activate an optional
10
+ specialist with the agents command only when its domain is required. Keep one writer
11
+ per path and use official Unity Skills for Editor/assets/runtime operations.
12
+
13
+ For code changes: task init with Astra's plan, bind the session, implement and test,
14
+ record implementation evidence, request independent review, fix and re-review as
15
+ needed, then task check --target report. Only after that check passes write and
16
+ record the report for Astra. Stop at AWAITING_ASTRA_CHECK. Astra checking and the
17
+ Product Owner's explicit acceptance happen outside the Claude implementation session.
18
+
19
+ Use [task gate commands](../../../nccgs/protocols/task-gates.md) for evidence and
20
+ state transitions. Report outcome, evidence, pending decisions, and next owner.
@@ -0,0 +1,7 @@
1
+ # Incident Recovery
2
+
3
+ Read `.nccgs/project.yaml`, `.nccgs/state.md`, and the active context packet when present. Preserve Product Owner authority and unrelated work.
4
+
5
+ Prioritize containment and evidence preservation. Record incident scope, affected versions/data, current state, timeline, hypotheses, and irreversible-action boundaries. Avoid destructive cleanup and speculative rewrites. Build a recovery graph with backups, reproducible diagnostics, validation, and rollback. Apply only authorized recovery changes; maintain an incident record and follow-up prevention actions. Data deletion or production actions require explicit authority.
6
+
7
+ Return outcome, evidence, affected records, blockers or decisions, residual risk, and the exact next workflow. Do not claim DONE unless this is the `closure` skill and every configured gate passes.
@@ -1,14 +1,15 @@
1
1
  ---
2
2
  name: localize-game
3
3
  description: Plans, implements, or reviews internationalization and localization across strings, fonts, layout, input, assets, culture, and LQA using the project's supported locales.
4
- allowed-tools: Read, Glob, Grep, Write, Edit, Bash, Task, AskUserQuestion, WebSearch, WebFetch
4
+ allowed-tools: Read, Glob, Grep, Write, Edit, Bash, Agent, AskUserQuestion, WebSearch, WebFetch
5
5
  ---
6
6
 
7
7
  # Localize Game
8
8
 
9
- Read `.nccgs/project.yaml`, `.nccgs/state.md`, and the active context packet when present. Preserve Product Owner authority and unrelated work.
9
+ Read models.profile in .nccgs/project.yaml; if absent, use the defaultProfile in
10
+ .claude/nccgs/pipeline.json. Load exactly one procedure:
10
11
 
11
- Read locale scope, canonical terminology, UI/content sources, existing localization architecture, and platform constraints. Route writing, UI, accessibility, content, and Unity specialists as needed. Preserve stable string IDs, context, placeholders, plurals, fallback, font coverage, layout expansion, and extraction determinism. Use the official `localization` Unity Skill for Unity operation mechanics. Do not invent locale scope or approve translations without review evidence.
12
-
13
- Return outcome, evidence, affected records, blockers or decisions, residual risk, and the exact next workflow. Do not claim DONE unless this is the `closure` skill and every configured gate passes.
12
+ - astra-claude: [Astra/Claude procedure](references/astra.md).
13
+ - quality, balanced, inherit: [Legacy procedure](references/legacy.md).
14
14
 
15
+ Do not load or merge the other branch. Unknown profiles are configuration errors.
@@ -0,0 +1,20 @@
1
+ # Localize Game — astra-claude
2
+
3
+ Read [the pipeline protocol](../../../nccgs/protocols/astra-claude.md) and
4
+ [generated routing](../../../nccgs/routing.generated.md). Use Astra's supplied plan,
5
+ requirements, and exact context references. Never substitute a legacy procedure.
6
+
7
+ Domain objective: Plans, implements, or reviews internationalization and localization across strings, fonts, layout, input, assets, culture, and LQA using the project's supported locales.
8
+
9
+ Work only inside the supplied plan and assigned write set. Activate an optional
10
+ specialist with the agents command only when its domain is required. Keep one writer
11
+ per path and use official Unity Skills for Editor/assets/runtime operations.
12
+
13
+ For code changes: task init with Astra's plan, bind the session, implement and test,
14
+ record implementation evidence, request independent review, fix and re-review as
15
+ needed, then task check --target report. Only after that check passes write and
16
+ record the report for Astra. Stop at AWAITING_ASTRA_CHECK. Astra checking and the
17
+ Product Owner's explicit acceptance happen outside the Claude implementation session.
18
+
19
+ Use [task gate commands](../../../nccgs/protocols/task-gates.md) for evidence and
20
+ state transitions. Report outcome, evidence, pending decisions, and next owner.
@@ -0,0 +1,7 @@
1
+ # Localize Game
2
+
3
+ Read `.nccgs/project.yaml`, `.nccgs/state.md`, and the active context packet when present. Preserve Product Owner authority and unrelated work.
4
+
5
+ Read locale scope, canonical terminology, UI/content sources, existing localization architecture, and platform constraints. Route writing, UI, accessibility, content, and Unity specialists as needed. Preserve stable string IDs, context, placeholders, plurals, fallback, font coverage, layout expansion, and extraction determinism. Use the official `localization` Unity Skill for Unity operation mechanics. Do not invent locale scope or approve translations without review evidence.
6
+
7
+ Return outcome, evidence, affected records, blockers or decisions, residual risk, and the exact next workflow. Do not claim DONE unless this is the `closure` skill and every configured gate passes.
@@ -1,25 +1,15 @@
1
1
  ---
2
2
  name: migrate-project
3
3
  description: Reviews, plans, applies, and verifies adoption or upgrade of NCCGS in an existing Unity project. Preserves canonical truth, ownership, closure safeguards, model policy, local settings, and rollback while replacing obsolete workflow machinery.
4
- allowed-tools: Read, Glob, Grep, Write, Edit, Bash, Task, AskUserQuestion
4
+ allowed-tools: Read, Glob, Grep, Write, Edit, Bash, Agent, AskUserQuestion
5
5
  ---
6
6
 
7
7
  # Migrate project
8
8
 
9
- Treat brownfield adoption and migrations explicitly requested by the installer as CONTROLLED work. Read the complete [migration procedure](references/procedure.md). Default to `full` when no mode is given. Do not require migration merely because NCCGS was updated when the installer reported `No project migration required`.
9
+ Read models.profile in .nccgs/project.yaml; if absent, use the defaultProfile in
10
+ .claude/nccgs/pipeline.json. Load exactly one procedure:
10
11
 
11
- When `.nccgs/framework-overrides.json` exists, treat every `pending_review` entry as preserved project truth, not installer debris. Compare the project version with the current NCCGS version, keep or rebase the customization deliberately, and record the decision in the migration plan. Only after every pending item has been reviewed and the final bytes or intentional absence are correct, run `node .claude/nccgs/tools/approve-overrides.mjs --project . --approve-pending`. Never approve first to make `doctor` green.
12
+ - astra-claude: [Astra/Claude procedure](references/astra.md).
13
+ - quality, balanced, inherit: [Legacy procedure](references/legacy.md).
12
14
 
13
- ## Modes
14
-
15
- - `full` (default): run review, plan, apply, and verify in one invocation. Continue across phase boundaries without asking for ceremonial approval. Pause only for an unresolved Product Owner decision, destructive action, external side effect, or authority outside the current request.
16
- - `review`: read-only inventory and KEEP/ADAPT/MIGRATE/DEPRECATE/REMOVE/MISSING classification.
17
- - `plan`: write a dated reversible plan and proposed schema-2 policy without overwriting project truth.
18
- - `apply`: execute an approved plan in validated batches with backups.
19
- - `verify`: prove policy, routing, hooks, Unity integration, evidence, closure, and a representative work path.
20
-
21
- Migration decisions must cover canonical sources, ownership/protected paths, architecture, Unity tooling, model profile/fallback, enabled departments, context packets, verification baselines, closure safeguards, Git tracking, local settings, preserved framework overrides, obsolete global skills, and rollback.
22
-
23
- In `full` mode, write the plan before applying it, preserve the same evidence and rollback guarantees as the individual modes, and stop safely if review finds a decision that would materially change project truth. A reversible plan created within the same invocation is approved for in-scope application unless project policy explicitly requires a separate approval.
24
-
25
- Never remove reports, registers, checklists, user settings, source assets, or legacy instructions until their live purpose is mapped and the Product Owner approves the replacement. Do not infer canon from code. Return batches, commits, preserved evidence, unresolved decisions, rollback, and final migration state.
15
+ Do not load or merge the other branch. Unknown profiles are configuration errors.
@@ -0,0 +1,23 @@
1
+ # Migrate project — astra-claude
2
+
3
+ Read [the pipeline protocol](../../../nccgs/protocols/astra-claude.md) and
4
+ [generated routing](../../../nccgs/routing.generated.md). Use Astra's supplied plan,
5
+ requirements, and exact context references. Never substitute a legacy procedure.
6
+
7
+ Domain objective: Reviews, plans, applies, and verifies adoption or upgrade of NCCGS in an existing Unity project. Preserves canonical truth, ownership, closure safeguards, model policy, local settings, and rollback while replacing obsolete workflow machinery.
8
+
9
+ Work only inside the supplied plan and assigned write set. Activate an optional
10
+ specialist with the agents command only when its domain is required. Keep one writer
11
+ per path and use official Unity Skills for Editor/assets/runtime operations.
12
+
13
+ For code changes: task init with Astra's plan, bind the session, implement and test,
14
+ record implementation evidence, request independent review, fix and re-review as
15
+ needed, then task check --target report. Only after that check passes write and
16
+ record the report for Astra. Stop at AWAITING_ASTRA_CHECK. Astra checking and the
17
+ Product Owner's explicit acceptance happen outside the Claude implementation session.
18
+
19
+ Default to full inventory/apply/verify using Astra's existing migration plan.
20
+ Preserve project overrides and backups; missing planning inputs go back to Astra.
21
+
22
+ Use [task gate commands](../../../nccgs/protocols/task-gates.md) for evidence and
23
+ state transitions. Report outcome, evidence, pending decisions, and next owner.
@@ -0,0 +1,19 @@
1
+ # Migrate project
2
+
3
+ Treat brownfield adoption and migrations explicitly requested by the installer as CONTROLLED work. Read the complete [migration procedure](../references/procedure.md). Default to `full` when no mode is given. Do not require migration merely because NCCGS was updated when the installer reported `No project migration required`.
4
+
5
+ When `.nccgs/framework-overrides.json` exists, treat every `pending_review` entry as preserved project truth, not installer debris. Compare the project version with the current NCCGS version, keep or rebase the customization deliberately, and record the decision in the migration plan. Only after every pending item has been reviewed and the final bytes or intentional absence are correct, run `node .claude/nccgs/tools/approve-overrides.mjs --project . --approve-pending`. Never approve first to make `doctor` green.
6
+
7
+ ## Modes
8
+
9
+ - `full` (default): run review, plan, apply, and verify in one invocation. Continue across phase boundaries without asking for ceremonial approval. Pause only for an unresolved Product Owner decision, destructive action, external side effect, or authority outside the current request.
10
+ - `review`: read-only inventory and KEEP/ADAPT/MIGRATE/DEPRECATE/REMOVE/MISSING classification.
11
+ - `plan`: write a dated reversible plan and proposed schema-2 policy without overwriting project truth.
12
+ - `apply`: execute an approved plan in validated batches with backups.
13
+ - `verify`: prove policy, routing, hooks, Unity integration, evidence, closure, and a representative work path.
14
+
15
+ Migration decisions must cover canonical sources, ownership/protected paths, architecture, Unity tooling, model profile/fallback, enabled departments, context packets, verification baselines, closure safeguards, Git tracking, local settings, preserved framework overrides, obsolete global skills, and rollback.
16
+
17
+ In `full` mode, write the plan before applying it, preserve the same evidence and rollback guarantees as the individual modes, and stop safely if review finds a decision that would materially change project truth. A reversible plan created within the same invocation is approved for in-scope application unless project policy explicitly requires a separate approval.
18
+
19
+ Never remove reports, registers, checklists, user settings, source assets, or legacy instructions until their live purpose is mapped and the Product Owner approves the replacement. Do not infer canon from code. Return batches, commits, preserved evidence, unresolved decisions, rollback, and final migration state.
@@ -85,3 +85,11 @@ relevant tests, runtime evidence, and subjective-gate routing. Verify that requi
85
85
  linked artifacts block DONE in `linked` or `strict` mode. Use a small representative
86
86
  task or dry run when product changes are not authorized. Run `npx nccgs@latest doctor`
87
87
  and require zero pending framework overrides before declaring migration complete.
88
+
89
+ ## Astra-owned planning exception
90
+
91
+ Under astra-claude, Astra owns the migration plan, policy decisions, and canonical
92
+ document changes. Claude may inventory and implement a supplied migration plan;
93
+ full must pause at missing Astra inputs instead of creating a replacement plan.
94
+ Code changes require Opus 5.5/xhigh, Fable review, a report to Astra, Astra checking,
95
+ and explicit Product Owner acceptance before DONE. See the profile handoff protocol.
@@ -1,14 +1,15 @@
1
1
  ---
2
2
  name: milestone-review
3
3
  description: Evaluates milestone progress, scope, quality, risk, evidence, and release trajectory against approved goals without disguising incomplete work as percentage completion.
4
- allowed-tools: Read, Glob, Grep, Bash, Task
4
+ allowed-tools: Read, Glob, Grep, Bash, Agent
5
5
  ---
6
6
 
7
7
  # Milestone Review
8
8
 
9
- Read `.nccgs/project.yaml`, `.nccgs/state.md`, and the active context packet when present. Preserve Product Owner authority and unrelated work.
9
+ Read models.profile in .nccgs/project.yaml; if absent, use the defaultProfile in
10
+ .claude/nccgs/pipeline.json. Load exactly one procedure:
10
11
 
11
- Remain read-only unless asked to update records. Compare approved milestone outcomes with feature closures, evidence, defects, dependencies, PO gates, and repository/build state. Report achieved outcomes, incomplete gates, scope movement, forecast assumptions, top risks, and decision options. Use evidence-based confidence rather than arbitrary completion percentages.
12
-
13
- Return outcome, evidence, affected records, blockers or decisions, residual risk, and the exact next workflow. Do not claim DONE unless this is the `closure` skill and every configured gate passes.
12
+ - astra-claude: [Astra/Claude procedure](references/astra.md).
13
+ - quality, balanced, inherit: [Legacy procedure](references/legacy.md).
14
14
 
15
+ Do not load or merge the other branch. Unknown profiles are configuration errors.
@@ -0,0 +1,12 @@
1
+ # Milestone Review — astra-claude
2
+
3
+ Read [the pipeline protocol](../../../nccgs/protocols/astra-claude.md) and
4
+ [generated routing](../../../nccgs/routing.generated.md). Use Astra's supplied plan,
5
+ requirements, and exact context references. Never substitute a legacy procedure.
6
+
7
+ Astra owns this workflow and its canonical outputs. Claude only inspects relevant
8
+ inputs and returns factual constraints, missing information, or proposed amendments.
9
+ Do not create a replacement plan/document or sign off as Astra.
10
+
11
+ Use [task gate commands](../../../nccgs/protocols/task-gates.md) for evidence and
12
+ state transitions. Report outcome, evidence, pending decisions, and next owner.
@@ -0,0 +1,7 @@
1
+ # Milestone Review
2
+
3
+ Read `.nccgs/project.yaml`, `.nccgs/state.md`, and the active context packet when present. Preserve Product Owner authority and unrelated work.
4
+
5
+ Remain read-only unless asked to update records. Compare approved milestone outcomes with feature closures, evidence, defects, dependencies, PO gates, and repository/build state. Report achieved outcomes, incomplete gates, scope movement, forecast assumptions, top risks, and decision options. Use evidence-based confidence rather than arbitrary completion percentages.
6
+
7
+ Return outcome, evidence, affected records, blockers or decisions, residual risk, and the exact next workflow. Do not claim DONE unless this is the `closure` skill and every configured gate passes.
@@ -1,13 +1,15 @@
1
1
  ---
2
2
  name: nccgs-code-review
3
3
  description: Performs a read-only code review against requirements, architecture, Unity constraints, ownership, tests, performance, security, and regression risk.
4
- allowed-tools: Read, Glob, Grep, Bash, Task
4
+ allowed-tools: Read, Glob, Grep, Bash, Agent
5
5
  ---
6
6
 
7
7
  # Code Review
8
8
 
9
- Read `.nccgs/project.yaml`, `.nccgs/state.md`, and the active context packet when present. Preserve Product Owner authority and unrelated work.
9
+ Read models.profile in .nccgs/project.yaml; if absent, use the defaultProfile in
10
+ .claude/nccgs/pipeline.json. Load exactly one procedure:
10
11
 
11
- Read the context packet, final diff, governing decisions, affected callers, and tests. Route distinct Unity, networking, security, performance, or UI lenses when needed. Report only actionable findings with severity, exact evidence, impact, and remediation direction. Verify that tests assert requirements and that serialized/lifecycle behavior is addressed. Do not edit or praise generally; state when no material issue is found.
12
+ - astra-claude: [Astra/Claude procedure](references/astra.md).
13
+ - quality, balanced, inherit: [Legacy procedure](references/legacy.md).
12
14
 
13
- Return outcome, evidence, affected records, blockers or decisions, residual risk, and the exact next workflow. Do not claim DONE unless this is the `closure` skill and every configured gate passes.
15
+ Do not load or merge the other branch. Unknown profiles are configuration errors.
@@ -0,0 +1,15 @@
1
+ # Code Review — astra-claude
2
+
3
+ Read [the pipeline protocol](../../../nccgs/protocols/astra-claude.md) and
4
+ [generated routing](../../../nccgs/routing.generated.md). Use Astra's supplied plan,
5
+ requirements, and exact context references. Never substitute a legacy procedure.
6
+
7
+ Run the independent reviewer selected by pipeline.json on the final snapshot.
8
+ Read raw requirements, diff, affected callers, and verification evidence. Return
9
+ PASS, CHANGES_REQUIRED, or BLOCKED with findings, severity, locations, and evidence.
10
+ Do not fix code. BLOCKER/HIGH findings and unmet required criteria prevent PASS.
11
+ Record the review through task record; return fixes to the implementer and review
12
+ again after changes. Review is mandatory even for FAST work.
13
+
14
+ Use [task gate commands](../../../nccgs/protocols/task-gates.md) for evidence and
15
+ state transitions. Report outcome, evidence, pending decisions, and next owner.
@@ -0,0 +1,7 @@
1
+ # Code Review
2
+
3
+ Read `.nccgs/project.yaml`, `.nccgs/state.md`, and the active context packet when present. Preserve Product Owner authority and unrelated work.
4
+
5
+ Read the context packet, final diff, governing decisions, affected callers, and tests. Route distinct Unity, networking, security, performance, or UI lenses when needed. Report only actionable findings with severity, exact evidence, impact, and remediation direction. Verify that tests assert requirements and that serialized/lifecycle behavior is addressed. Do not edit or praise generally; state when no material issue is found.
6
+
7
+ Return outcome, evidence, affected records, blockers or decisions, residual risk, and the exact next workflow. Do not claim DONE unless this is the `closure` skill and every configured gate passes.
@@ -1,14 +1,15 @@
1
1
  ---
2
2
  name: performance-audit
3
3
  description: Profiles or reviews Unity CPU, GPU, memory, loading, allocations, and frame pacing against project budgets and reproducible scenarios.
4
- allowed-tools: Read, Glob, Grep, Bash, Task
4
+ allowed-tools: Read, Glob, Grep, Bash, Agent
5
5
  ---
6
6
 
7
7
  # Performance Audit
8
8
 
9
- Read `.nccgs/project.yaml`, `.nccgs/state.md`, and the active context packet when present. Preserve Product Owner authority and unrelated work.
9
+ Read models.profile in .nccgs/project.yaml; if absent, use the defaultProfile in
10
+ .claude/nccgs/pipeline.json. Load exactly one procedure:
10
11
 
11
- Remain read-only unless explicitly asked to optimize. Define platform, scene, hardware/environment, scenario, baseline, and budget. Use official Unity Skills and profiling tools where available. Measure before attributing cause. Report bottlenecks with evidence, player impact, confidence, and prioritized experiment or remediation. For implemented optimizations, compare identical before/after conditions and regression risk.
12
-
13
- Return outcome, evidence, affected records, blockers or decisions, residual risk, and the exact next workflow. Do not claim DONE unless this is the `closure` skill and every configured gate passes.
12
+ - astra-claude: [Astra/Claude procedure](references/astra.md).
13
+ - quality, balanced, inherit: [Legacy procedure](references/legacy.md).
14
14
 
15
+ Do not load or merge the other branch. Unknown profiles are configuration errors.
@@ -0,0 +1,17 @@
1
+ # Performance Audit — astra-claude
2
+
3
+ Read [the pipeline protocol](../../../nccgs/protocols/astra-claude.md) and
4
+ [generated routing](../../../nccgs/routing.generated.md). Use Astra's supplied plan,
5
+ requirements, and exact context references. Never substitute a legacy procedure.
6
+
7
+ Domain objective: Profiles or reviews Unity CPU, GPU, memory, loading, allocations, and frame pacing against project budgets and reproducible scenarios.
8
+
9
+ Work only inside the supplied plan and assigned write set. Activate an optional
10
+ specialist with the agents command only when its domain is required. Keep one writer
11
+ per path and use official Unity Skills for Editor/assets/runtime operations.
12
+
13
+ This procedure is read-only: return evidence-backed findings and decisions to Astra
14
+ or the implementer. Supplemental analysis does not replace the mandatory Fable gate.
15
+
16
+ Use [task gate commands](../../../nccgs/protocols/task-gates.md) for evidence and
17
+ state transitions. Report outcome, evidence, pending decisions, and next owner.