@jspg-ai/coding-bb 0.0.3-beta.3 → 0.0.3-beta.31

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 (145) hide show
  1. package/cbb/dev-standards/rules/cbb-ai-behavior.md +135 -116
  2. package/cbb/lib/external/index.js +228 -0
  3. package/cbb/lib/install/agents-nav.js +92 -0
  4. package/cbb/lib/install/claude-code.js +7 -2
  5. package/cbb/lib/install/command-skill.js +105 -0
  6. package/cbb/lib/install/init.js +502 -168
  7. package/cbb/lib/install/minimax.js +60 -0
  8. package/cbb/lib/install/opencode.js +3 -5
  9. package/cbb/lib/install/qoder.js +1 -3
  10. package/cbb/lib/install/rule-rewrite.js +27 -0
  11. package/cbb/lib/install/setup-decision.js +48 -0
  12. package/cbb/lib/install/tool-selection.js +45 -0
  13. package/cbb/lib/install/trae.js +43 -0
  14. package/cbb/lib/install/workbuddy.js +36 -0
  15. package/cbb/lib/install/workspaces.js +21 -17
  16. package/cbb/lib/openspec/index.js +337 -554
  17. package/cbb/lib/superpowers/index.js +246 -265
  18. package/cbb/lib/utils/check-update.js +17 -2
  19. package/cbb/lib/utils/githook.js +226 -0
  20. package/cbb/lib/utils/gitignore.js +16 -3
  21. package/cbb/lib/utils/plugin-hook.js +92 -0
  22. package/cbb/lib/utils/settings.js +17 -4
  23. package/cbb/lib/utils/tar.js +92 -0
  24. package/cbb/lib/utils/upstream.js +90 -0
  25. package/cbb/worktrees/_shared/scripts/check-local-state.js +114 -0
  26. package/cbb/worktrees/_shared/scripts/find-target-worktree.js +148 -96
  27. package/cbb/worktrees/_shared/scripts/find-workspace-root.js +7 -7
  28. package/cbb/worktrees/_shared/scripts/install-git-hook.js +70 -0
  29. package/cbb/worktrees/_shared/scripts/junction-ai-config.js +127 -0
  30. package/cbb/worktrees/_shared/scripts/start-local.js +180 -0
  31. package/cbb/worktrees/skills/cbb-local-mode/SKILL.md +158 -0
  32. package/cbb/worktrees/skills/cbb-local-mode/scripts/check-local-state.js +114 -0
  33. package/cbb/worktrees/skills/cbb-local-mode/scripts/install-git-hook.js +70 -0
  34. package/cbb/worktrees/skills/cbb-local-mode/scripts/silence-popup.js +23 -0
  35. package/cbb/worktrees/skills/cbb-local-mode/scripts/start-local.js +180 -0
  36. package/cbb/worktrees/skills/cbb-worktree-close/SKILL.md +60 -43
  37. package/cbb/worktrees/skills/cbb-worktree-close/scripts/check-env.js +3 -3
  38. package/cbb/worktrees/skills/cbb-worktree-close/scripts/check-unarchived.js +75 -48
  39. package/cbb/worktrees/skills/cbb-worktree-close/scripts/delete-branches.js +1 -1
  40. package/cbb/worktrees/skills/cbb-worktree-close/scripts/discover-apps.js +14 -5
  41. package/cbb/worktrees/skills/cbb-worktree-close/scripts/find-target-worktree.js +148 -96
  42. package/cbb/worktrees/skills/cbb-worktree-close/scripts/find-workspace-root.js +7 -7
  43. package/cbb/worktrees/skills/cbb-worktree-close/scripts/remove-worktrees.js +13 -4
  44. package/cbb/worktrees/skills/cbb-worktree-close/scripts/safety-check.js +11 -2
  45. package/cbb/worktrees/skills/cbb-worktree-init/SKILL.md +138 -129
  46. package/cbb/worktrees/skills/cbb-worktree-init/scripts/check-env.js +1 -1
  47. package/cbb/worktrees/skills/cbb-worktree-init/scripts/check-repos.js +2 -2
  48. package/cbb/worktrees/skills/cbb-worktree-init/scripts/create-branches.js +33 -8
  49. package/cbb/worktrees/skills/cbb-worktree-init/scripts/create-worktrees.js +60 -46
  50. package/cbb/worktrees/skills/cbb-worktree-init/scripts/find-workspace-root.js +7 -7
  51. package/cbb/worktrees/skills/cbb-worktree-init/scripts/install-git-hook.js +70 -0
  52. package/cbb/worktrees/skills/cbb-worktree-init/scripts/junction-ai-config.js +127 -0
  53. package/cbb/worktrees/skills/cbb-worktree-init/scripts/parse-config.js +7 -7
  54. package/cbb/worktrees/skills/cbb-worktree-init/scripts/push-branches.js +3 -3
  55. package/cbb/worktrees/skills/cbb-worktree-init/scripts/sync-repos.js +11 -7
  56. package/cbb/worktrees/skills/cbb-worktree-init/scripts/update-gitignore.js +37 -9
  57. package/cbb/worktrees/skills/cbb-worktree-push/SKILL.md +140 -71
  58. package/cbb/worktrees/skills/cbb-worktree-push/scripts/commit-worktrees.js +70 -12
  59. package/cbb/worktrees/skills/cbb-worktree-push/scripts/find-target-worktree.js +148 -96
  60. package/cbb/worktrees/skills/cbb-worktree-push/scripts/find-workspace-root.js +7 -7
  61. package/cbb/worktrees/skills/cbb-worktree-push/scripts/make-mr-urls.js +211 -0
  62. package/cbb/worktrees/skills/cbb-worktree-push/scripts/push-worktrees.js +67 -20
  63. package/config/{workspace-config.sample.json → CBB-WORKSPACE.sample.json} +1 -1
  64. package/config/openspec/schemas/spec-driven/schema.yaml +40 -30
  65. package/config/openspec/schemas/spec-driven/templates/design.md +0 -18
  66. package/config/openspec/schemas/spec-driven/templates/proposal.md +4 -3
  67. package/config/upstream-mirrors.json +17 -0
  68. package/config/workspace-agents.sample.md +4 -33
  69. package/external/frontend-design/.version +6 -0
  70. package/external/frontend-design/LICENSE.txt +177 -0
  71. package/external/frontend-design/SKILL.md +71 -0
  72. package/openspec/.version +2 -3
  73. package/openspec/commands/apply.md +189 -175
  74. package/openspec/commands/archive.md +244 -216
  75. package/openspec/commands/bulk-archive.md +362 -327
  76. package/openspec/commands/continue.md +115 -105
  77. package/openspec/commands/explore.md +230 -199
  78. package/openspec/commands/ff.md +115 -104
  79. package/openspec/commands/new.md +74 -63
  80. package/openspec/commands/onboard.md +562 -548
  81. package/openspec/commands/propose.md +161 -150
  82. package/openspec/commands/sync.md +277 -249
  83. package/openspec/commands/update.md +98 -80
  84. package/openspec/commands/verify.md +206 -162
  85. package/openspec/skills/openspec-apply-change/SKILL.md +20 -5
  86. package/openspec/skills/openspec-archive-change/SKILL.md +40 -11
  87. package/openspec/skills/openspec-bulk-archive-change/SKILL.md +46 -9
  88. package/openspec/skills/openspec-continue-change/SKILL.md +14 -3
  89. package/openspec/skills/openspec-explore/SKILL.md +22 -10
  90. package/openspec/skills/openspec-ff-change/SKILL.md +15 -3
  91. package/openspec/skills/openspec-new-change/SKILL.md +13 -1
  92. package/openspec/skills/openspec-onboard/SKILL.md +58 -43
  93. package/openspec/skills/openspec-propose/SKILL.md +15 -3
  94. package/openspec/skills/openspec-sync-specs/SKILL.md +31 -2
  95. package/openspec/skills/openspec-update-change/SKILL.md +34 -15
  96. package/openspec/skills/openspec-verify-change/SKILL.md +72 -26
  97. package/package.json +3 -2
  98. package/superpowers/.version +4 -4
  99. package/superpowers/skills/brainstorming/SKILL.md +47 -12
  100. package/superpowers/skills/brainstorming/scripts/frame-template.html +213 -213
  101. package/superpowers/skills/brainstorming/scripts/server.cjs +723 -723
  102. package/superpowers/skills/brainstorming/visual-companion.md +6 -6
  103. package/superpowers/skills/diagnosing-superpowers/SKILL.md +120 -0
  104. package/superpowers/skills/diagnosing-superpowers/prompts/analyst-common.md +38 -0
  105. package/superpowers/skills/diagnosing-superpowers/prompts/cost-and-time.md +28 -0
  106. package/superpowers/skills/diagnosing-superpowers/prompts/plan-adherence.md +29 -0
  107. package/superpowers/skills/diagnosing-superpowers/prompts/quality-evidence.md +26 -0
  108. package/superpowers/skills/diagnosing-superpowers/prompts/repeated-work.md +30 -0
  109. package/superpowers/skills/diagnosing-superpowers/prompts/request-conflicts.md +20 -0
  110. package/superpowers/skills/diagnosing-superpowers/prompts/scrub-audit.md +33 -0
  111. package/superpowers/skills/diagnosing-superpowers/prompts/scrub.md +29 -0
  112. package/superpowers/skills/diagnosing-superpowers/prompts/similar-session.md +38 -0
  113. package/superpowers/skills/diagnosing-superpowers/prompts/skill-timeline.md +30 -0
  114. package/superpowers/skills/diagnosing-superpowers/prompts/stumbles.md +28 -0
  115. package/superpowers/skills/diagnosing-superpowers/references/context-safety.md +22 -0
  116. package/superpowers/skills/diagnosing-superpowers/references/github-issues.md +47 -0
  117. package/superpowers/skills/diagnosing-superpowers/references/redaction-policy.md +34 -0
  118. package/superpowers/skills/diagnosing-superpowers/references/session-discovery.md +31 -0
  119. package/superpowers/skills/diagnosing-superpowers/templates/bundle-README.md +77 -0
  120. package/superpowers/skills/diagnosing-superpowers/templates/case.md +64 -0
  121. package/superpowers/skills/diagnosing-superpowers/templates/issue.md +51 -0
  122. package/superpowers/skills/diagnosing-superpowers/templates/report.md +82 -0
  123. package/superpowers/skills/executing-plans/SKILL.md +350 -41
  124. package/superpowers/skills/executing-plans/scripts/task-done +52 -0
  125. package/superpowers/skills/executing-plans/scripts/task-start +28 -0
  126. package/superpowers/skills/requesting-code-review/SKILL.md +1 -1
  127. package/superpowers/skills/requesting-code-review/code-reviewer.md +17 -0
  128. package/superpowers/skills/subagent-driven-development/SKILL.md +18 -18
  129. package/superpowers/skills/subagent-driven-development/re-review-prompt.md +1 -1
  130. package/superpowers/skills/subagent-driven-development/scripts/review-package +53 -46
  131. package/superpowers/skills/subagent-driven-development/scripts/sdd-workspace +82 -40
  132. package/superpowers/skills/subagent-driven-development/scripts/task-brief +43 -41
  133. package/superpowers/skills/subagent-driven-development/task-reviewer-prompt.md +2 -2
  134. package/superpowers/skills/systematic-debugging/root-cause-tracing.md +1 -1
  135. package/superpowers/skills/test-driven-development/SKILL.md +10 -0
  136. package/superpowers/skills/using-superpowers/SKILL.md +2 -0
  137. package/superpowers/skills/using-superpowers/references/claude-code-tools.md +29 -0
  138. package/superpowers/skills/using-superpowers/references/muse-tools.md +35 -0
  139. package/superpowers/skills/writing-plans/SKILL.md +62 -29
  140. package/superpowers/skills/writing-skills/SKILL.md +4 -2
  141. package/superpowers/skills/writing-skills/graphviz-conventions.dot +171 -171
  142. package/cbb/worktrees/commands/worktree-close.md +0 -64
  143. package/cbb/worktrees/commands/worktree-init.md +0 -51
  144. package/cbb/worktrees/commands/worktree-push.md +0 -42
  145. package/superpowers/skills/writing-plans/plan-document-reviewer-prompt.md +0 -49
@@ -1,85 +1,103 @@
1
1
  ---
2
2
  name: "OPSX: Update"
3
- description:
3
+ description: Update a change - revise existing planning artifacts and keep them coherent (Experimental)
4
4
  category: Workflow
5
5
  tags: [workflow, artifacts, experimental]
6
6
  ---
7
- Revise a change's existing planning artifacts and keep them coherent. Never edit code.
8
-
9
- **Store selection:** If the user names a store (a store is a standalone OpenSpec repo registered on this machine) or the work lives in one, run `openspec store list --json` to discover registered store ids, then pass `--store <id>` on the commands that read or write specs and changes (`new change`, `status`, `instructions`, `list`, `show`, `validate`, `archive`, `doctor`, `context`). Other commands do not take the flag. Hints printed by commands already carry the flag; keep it on follow-ups. Without a store, commands act on the nearest local `openspec/` root.
10
-
11
- **Input**: Optionally specify a change name after `/opsx:update` (e.g., `/opsx:update add-auth`). If omitted, check if it can be inferred from conversation context. If vague or ambiguous you MUST prompt for available changes.
12
-
13
- `/opsx:continue` is an optional workflow and may not be installed. Before suggesting it anywhere below, verify that it is available. If it is unavailable, `openspec status --change "<name>" --json` shows the next artifact and `openspec instructions "<artifact-id>" --change "<name>" --json` explains how to create it.
14
-
15
- **Steps**
16
-
17
- 1. **Select the change**
18
-
19
- If a name is provided, use it. Otherwise:
20
- - Infer from conversation context if the user mentioned a change
21
- - Auto-select if only one active change exists
22
- - If ambiguous, run `openspec list --json` to get available changes sorted by most recently modified, and ask the user to select one
23
-
24
- When prompting, present the top 3-4 most recently modified changes as options, showing:
25
- - Change name
26
- - Schema (from `schema` field if present, otherwise "spec-driven")
27
- - Status (e.g., "0/5 tasks", "complete", "no tasks")
28
- - How recently it was modified (from `lastModified` field)
29
-
30
- Mark the most recently modified change as "(Recommended)" since it's likely what the user wants to update.
31
-
32
- Always announce: "Using change: <name>" and how to override (e.g., `/opsx:update <other>`).
33
-
34
- 2. **Get the change's artifacts**
35
- ```bash
36
- openspec status --change "<name>" --json
37
- ```
38
- Parse the JSON to understand current state. The response includes:
39
- - `schemaName`: The workflow schema being used (e.g., "spec-driven")
40
- - `artifacts`: Array of artifacts with their status ("done", "skipped", "ready", "blocked")
41
- - `isPlanningComplete`: Boolean indicating if all planning artifacts are complete. Older CLI versions expose the same value as `isComplete`.
42
- - `planningHome`, `changeRoot`, `artifactPaths`, and `actionContext`: path and scope context. Use these instead of assuming repo-local paths.
43
-
44
- The artifact ids and paths come from the active schema - do NOT assume them, and do NOT branch on hardcoded artifact names. Custom schemas must work unchanged.
45
-
46
- The files to edit are `artifactPaths.<id>.existingOutputPaths` - the concrete files that exist on disk, already glob-expanded for glob artifacts (e.g. `specs/**/*.md`). Do NOT write to `resolvedOutputPath`: for a glob artifact it is still the glob pattern, not a real file.
47
-
48
- 3. **Understand the request**
49
- - If the user asked for a specific revision ("the design now uses X"), that is the starting edit.
50
- - If they only said "update" / "make this coherent", treat it as a coherence review: read the existing artifacts and check them against each other for contradictions, gaps, and duplication.
51
-
52
- 4. **Read and reconcile**
53
- - Read the artifact(s) the request touches and the change's other existing artifacts.
54
- - Apply the requested edit. Then check every other existing artifact against it - in ANY direction: an edit to a later artifact may require revising an earlier one, not only the other way around. Build order is a useful reading order, not a constraint on which artifacts may be revised.
55
- - Note everything that is now inconsistent, missing, or contradictory.
56
- - Revise only files that already exist (`existingOutputPaths`). Do NOT create artifacts that don't exist yet, and do NOT invent new files under a glob artifact - note them and point the user to `/opsx:continue` to create them.
57
- - If the change is already coherent, say so and make no edits.
58
-
59
- 5. **Confirm and apply, one artifact at a time**
60
- - Show each proposed revision and why. Write only after the user confirms.
61
- - If the user rejects a revision, do not write it - leave that artifact unchanged.
62
- - When a substantial rewrite is needed, get that artifact's rules and template first:
63
- ```bash
64
- openspec instructions "<artifact-id>" --change "<name>" --json
65
- ```
66
-
67
- 6. **Point to the next step (guidance only - NEVER act on it)**
68
- - Artifacts still missing -> suggest `/opsx:continue` to create them.
69
- - Change already implemented (tasks checked off / already applied) -> the code may no longer match the revised plan; suggest `/opsx:apply` to carry the delta into code.
70
- - Everything done and implemented -> suggest `/opsx:archive`.
71
-
72
- **Output**
73
-
74
- After each invocation, show:
75
- - Which artifacts were revised (and which proposed revisions were rejected)
76
- - Anything deferred to `/opsx:continue` (not-yet-created artifacts or files)
77
- - Where the change stands and the recommended next command
78
-
79
- **Guardrails**
80
- - Planning artifacts only - NEVER edit implementation code. If the revised plan implies code changes, stop and point to `/opsx:apply`.
81
- - Use the artifact ids and paths reported by `openspec status`; never branch on hardcoded artifact names.
82
- - Edit only the concrete files in `existingOutputPaths`; never write to a glob `resolvedOutputPath`.
83
- - Do not advance the build frontier: no new artifacts, no new files under glob artifacts - that is `/opsx:continue`'s job.
84
- - Confirm every edit with the user before writing.
85
- - If the request changes the change's *intent* rather than refining it, first verify whether the optional `/opsx:new` workflow is available. If it is, recommend starting fresh with `/opsx:new` (the "Update vs. Start Fresh" heuristic). If it is unavailable, ask for a distinct unused change name and recommend `openspec new change "<new-change-name>"` instead.
7
+ Revise a change's existing planning artifacts and keep them coherent. Never edit code.
8
+
9
+ **Store selection:** If the user names a store (a store is a standalone OpenSpec repo registered on this machine) or the work lives in one, run `openspec store list --json` to discover registered store ids, then pass `--store <id>` on the commands that read or write specs and changes (`new change`, `status`, `instructions`, `list`, `show`, `validate`, `archive`, `doctor`, `context`, `schemas`, `view`). Once selected, treat `--store <id>` as sticky for the rest of the workflow. Every unscoped example of those commands below is shorthand: before running it, append the flag. For example, run `openspec status --change "<name>" --json --store "<id>"`, not the unscoped form shown below. Other commands do not take the flag. Hints printed by commands already carry the flag; keep it on follow-ups. Without a store, commands act on the nearest local `openspec/` root.
10
+
11
+ **Project check:** These steps expect a project that already uses OpenSpec. Before the first step that writes anything (`new change`, `archive`, `sync specs`, or authoring an artifact file), confirm the project has a root: run `openspec list --json` (with `--store <id>` when a store is selected, since the store is then the root) and read `root`. A root object means the project is set up. `"root": null` means it is not - there is no `openspec/` directory here, and a write such as `openspec new change` would create one as a side effect. The command also exits non-zero, which is that answer rather than a broken CLI, so read the JSON instead of retrying or working around it.
12
+
13
+ One `"root": null` is not about setup: when a `status` error message starts with `Declared in` or `Invalid store declaration in` and names this project's `openspec/config.yaml` (or `config.yml`), the project does use OpenSpec through a store it declares, which this machine cannot resolve (the store is not registered, or the `store:` line is malformed). Do not treat it as uninitialized and skip the branches below: stop before writing and show the user that error's `message` and `fix`.
14
+
15
+ Otherwise, with no root, what happens next depends on how this workflow was reached:
16
+
17
+ - **Auto-selected**: you chose this workflow yourself, without the user naming OpenSpec, naming this skill, or running its slash command. Stop using OpenSpec and answer the request normally, as you would with no OpenSpec installed. Do not ask them to set anything up and do not mention OpenSpec setup.
18
+ - **Explicit OpenSpec request**: the user named OpenSpec, named this skill, or ran its slash command. Stop before writing and ask how to proceed: set this project up (`openspec init`), target a store they already have (`--store <id>`), or continue without OpenSpec for this request. Wait for their answer.
19
+
20
+ In both branches, never create the root as a side effect: do not run `openspec init` until the user asks for it, do not hand-create `openspec/` files, and do not let a command create it.
21
+
22
+ **Input**: Optionally specify a change name after `/opsx:update` (e.g., `/opsx:update add-auth`). If omitted, check if it can be inferred from conversation context. If vague or ambiguous you MUST prompt for available changes.
23
+
24
+ This workflow revises artifacts that already exist; `/opsx:continue` is what creates the ones that do not.
25
+
26
+ **Steps**
27
+
28
+ 1. **Select the change**
29
+
30
+ If a name is provided, use it. Otherwise:
31
+ - Infer from conversation context if the user mentioned a change
32
+ - Auto-select if only one active change exists
33
+ - If ambiguous, run `openspec list --json` to get available changes sorted by most recently modified, and ask the user to select one
34
+
35
+ When prompting, present the top 3-4 most recently modified changes as options, showing:
36
+ - Change name
37
+ - Status (e.g., "0/5 tasks", "complete", "no tasks")
38
+ - How recently it was modified (from `lastModified` field)
39
+
40
+ Mark the most recently modified change as "(Recommended)" since it's likely what the user wants to update.
41
+
42
+ Always announce: "Using change: <name>" and how to override (e.g., `/opsx:update <other>`).
43
+
44
+ 2. **Get the change's artifacts**
45
+ ```bash
46
+ openspec status --change "<name>" --json
47
+ ```
48
+ Parse the JSON to understand current state. The response includes:
49
+ - `schemaName`: The workflow schema being used (e.g., "spec-driven")
50
+ - `artifacts`: Array of artifacts with their status ("done", "skipped", "ready", "blocked")
51
+ - `isPlanningComplete`: Boolean indicating if all planning artifacts are complete. Older CLI versions expose the same value as `isComplete`.
52
+ - `planningHome`, `changeRoot`, `artifactPaths`, and `actionContext`: path and scope context. Use these instead of assuming repo-local paths.
53
+
54
+ The artifact ids and paths come from the active schema - do NOT assume them, and do NOT branch on hardcoded artifact names. Custom schemas must work unchanged.
55
+
56
+ The files to edit are `artifactPaths.<id>.existingOutputPaths` - the concrete files that exist on disk, already glob-expanded for glob artifacts (e.g. `specs/**/*.md`). Do NOT write to `resolvedOutputPath`: for a glob artifact it is still the glob pattern, not a real file.
57
+
58
+ 3. **Understand the request**
59
+ - If the user asked for a specific revision ("the design now uses X"), that is the starting edit.
60
+ - If they only said "update" / "make this coherent", treat it as a coherence review: read the existing artifacts and check them against each other for contradictions, gaps, and duplication.
61
+
62
+ 4. **Read and reconcile**
63
+ - Read the artifact(s) the request touches and the change's other existing artifacts.
64
+ - Draft the requested edit in the conversation, not in files. Work out exactly what it changes; step 5 owns every write. Then check every other existing artifact against the drafted edit - in ANY direction: an edit to a later artifact may require revising an earlier one, not only the other way around. Build order is a useful reading order, not a constraint on which artifacts may be revised.
65
+ - Note everything that is now inconsistent, missing, or contradictory.
66
+ - Propose revisions to files that already exist (`existingOutputPaths`). If an artifact has no existing output files and status `ready` or `blocked`, note it and point the user to `/opsx:continue` to create them. Leave `skipped` artifacts untouched; do not treat them as missing or defer them to the continue workflow.
67
+ - A glob artifact (e.g. `specs/**/*.md`) is marked `done` after at least one file matches, and the continue workflow only handles `ready` artifacts. When reconciliation identifies a missing file for a glob artifact whose `existingOutputPaths` is non-empty:
68
+ 1. Run `openspec instructions "<artifact-id>" --change "<name>" --json` and use its `instruction` and `template`. Treat `context` and `rules` as constraints; do not copy them into the file. If instructions report `skipped: true`, do not create the file. Read current dependency files from disk; if a required non-skipped dependency is missing, stop and ask the user to restore it first.
69
+ 2. Choose a concrete path inside `changeRoot` that matches `artifactPaths.<id>.outputPath` and does not already exist. Verify it remains inside `changeRoot` after resolving any symlinked parent directories. The glob `resolvedOutputPath` is not a valid target.
70
+ 3. Include the new file in step 5's proposed revisions and create it only after the user confirms.
71
+ 4. After confirmation, immediately before creation, refresh status and instructions. Verify the artifact is still in scope, not skipped, and partially populated; repeat the concrete-path checks above.
72
+ 5. Use a create operation that fails if the target already exists. If `instruction` delegates creation to another skill or command, invoke it only if it can honor the confirmed path and these guardrails; otherwise stop. If any check fails or the confirmed draft is no longer valid, stop and reconcile with the user rather than replacing existing content or choosing a different path.
73
+ - If the change is already coherent, say so and propose no revisions.
74
+
75
+ 5. **Confirm and apply, one artifact at a time**
76
+ - This step performs every artifact write in this workflow; no earlier step edits an artifact.
77
+ - Show each proposed revision and why - including the requested edit drafted in step 4. Write only after the user confirms.
78
+ - If the user rejects a revision, do not write it - leave that artifact unchanged.
79
+ - When a substantial rewrite is needed, get that artifact's rules and template first:
80
+ ```bash
81
+ openspec instructions "<artifact-id>" --change "<name>" --json
82
+ ```
83
+
84
+ 6. **Point to the next step (guidance only - NEVER act on it)**
85
+ - Artifacts with empty `existingOutputPaths` and status `ready` or `blocked` -> suggest `/opsx:continue` to create them.
86
+ - Change already implemented (tasks checked off / already applied) -> the code may no longer match the revised plan; suggest `/opsx:apply` to carry the delta into code.
87
+ - Everything done and implemented -> suggest `/opsx:archive`.
88
+
89
+ **Output**
90
+
91
+ After each invocation, show:
92
+ - Which artifacts were revised (and which proposed revisions were rejected)
93
+ - Any file created under a glob artifact that was already partially populated
94
+ - Anything deferred to `/opsx:continue` (artifacts with no files yet and status `ready` or `blocked`, never `skipped` artifacts)
95
+ - Where the change stands and the recommended next command
96
+
97
+ **Guardrails**
98
+ - Planning artifacts only - NEVER edit implementation code. If the revised plan implies code changes, stop and point to `/opsx:apply`.
99
+ - Use the artifact ids and paths reported by `openspec status`; never branch on hardcoded artifact names.
100
+ - Edit only the concrete files in `existingOutputPaths`; never write to a glob `resolvedOutputPath`.
101
+ - Do not advance the build frontier: if an artifact has empty `existingOutputPaths` and status `ready` or `blocked`, that is `/opsx:continue`'s job. Leave `skipped` artifacts untouched. The only new-file scope is a confirmed concrete path under a glob artifact whose `existingOutputPaths` is non-empty.
102
+ - Confirm every edit with the user before writing.
103
+ - If the request changes the change's *intent* rather than refining it, recommend starting fresh with `/opsx:new` (the "Update vs. Start Fresh" heuristic).
@@ -4,166 +4,210 @@ description: Verify implementation matches change artifacts before archiving
4
4
  category: Workflow
5
5
  tags: [workflow, verify, experimental]
6
6
  ---
7
- Verify that an implementation matches the change artifacts (specs, tasks, design).
8
-
9
- **Store selection:** If the user names a store (a store is a standalone OpenSpec repo registered on this machine) or the work lives in one, run `openspec store list --json` to discover registered store ids, then pass `--store <id>` on the commands that read or write specs and changes (`new change`, `status`, `instructions`, `list`, `show`, `validate`, `archive`, `doctor`, `context`). Other commands do not take the flag. Hints printed by commands already carry the flag; keep it on follow-ups. Without a store, commands act on the nearest local `openspec/` root.
10
-
11
- **Input**: Optionally specify a change name after `/opsx:verify` (e.g., `/opsx:verify add-auth`). If omitted, check if it can be inferred from conversation context. If vague or ambiguous you MUST prompt for available changes.
12
-
13
- **Steps**
14
-
15
- 1. **Select the change**
16
-
17
- If a name is provided, use it. Otherwise:
18
- - Infer from conversation context if the user mentioned a change
19
- - Auto-select if only one active change exists
20
- - If ambiguous, run `openspec list --json` to get available changes and ask the user to select one
21
-
22
- When prompting, show changes that have implementation tasks (tasks artifact exists).
23
- Include the schema used for each change if available.
24
- Mark changes with incomplete tasks as "(In Progress)".
25
-
26
- Always announce: "Using change: <name>" and how to override (e.g., `/opsx:verify <other>`).
27
-
28
- 2. **Check status to understand the schema**
29
- ```bash
30
- openspec status --change "<name>" --json
31
- ```
32
- Parse the JSON to understand:
33
- - `schemaName`: The workflow being used (e.g., "spec-driven")
34
- - `planningHome`, `changeRoot`, `artifactPaths`, and `actionContext`: path and scope context
35
- - Which artifacts exist for this change
36
-
37
- 3. **Get planning context and load artifacts**
38
-
39
- ```bash
40
- openspec instructions apply --change "<name>" --json
41
- ```
42
-
43
- This returns the change directory and `contextFiles` (artifact ID -> array of concrete file paths). Read all available artifacts from `contextFiles`.
44
-
45
- 4. **Initialize verification report structure**
46
-
47
- Create a report structure with three dimensions:
48
- - **Completeness**: Track tasks and spec coverage
49
- - **Correctness**: Track requirement implementation and scenario coverage
50
- - **Coherence**: Track design adherence and pattern consistency
51
-
52
- Each dimension can have CRITICAL, WARNING, or SUGGESTION issues.
53
-
54
- 5. **Verify Completeness**
55
-
56
- **Task Completion**:
57
- - If `contextFiles.tasks` exists, read every file path in it
58
- - Parse checkboxes: `- [ ]` (incomplete) vs `- [x]` (complete)
59
- - Count complete vs total tasks
60
- - If incomplete tasks exist:
61
- - Add CRITICAL issue for each incomplete task
62
- - Recommendation: "Complete task: <description>" or "Mark as done if already implemented"
63
-
64
- **Spec Coverage**:
65
- - If delta specs exist in `contextFiles.specs`:
66
- - Extract all requirements (marked with "### Requirement:")
67
- - For each requirement:
68
- - Search codebase for keywords related to the requirement
69
- - Assess if implementation likely exists
70
- - If requirements appear unimplemented:
71
- - Add CRITICAL issue: "Requirement not found: <requirement name>"
72
- - Recommendation: "Implement requirement X: <description>"
73
-
74
- 6. **Verify Correctness**
75
-
76
- **Requirement Implementation Mapping**:
77
- - For each requirement from delta specs:
78
- - Search codebase for implementation evidence
79
- - If found, note file paths and line ranges
80
- - Assess if implementation matches requirement intent
81
- - If divergence detected:
82
- - Add WARNING: "Implementation may diverge from spec: <details>"
83
- - Recommendation: "Review <file>:<lines> against requirement X"
84
-
85
- **Scenario Coverage**:
86
- - For each scenario in delta specs (marked with "#### Scenario:"):
87
- - Check if conditions are handled in code
88
- - Check if tests exist covering the scenario
89
- - If scenario appears uncovered:
90
- - Add WARNING: "Scenario not covered: <scenario name>"
91
- - Recommendation: "Add test or implementation for scenario: <description>"
92
-
93
- 7. **Verify Coherence**
94
-
95
- **Design Adherence**:
96
- - If `contextFiles.design` exists:
97
- - Extract key decisions (look for sections like "Decision:", "Approach:", "Architecture:")
98
- - Verify implementation follows those decisions
99
- - If contradiction detected:
100
- - Add WARNING: "Design decision not followed: <decision>"
101
- - Recommendation: "Update implementation or revise design.md to match reality"
102
- - If no design.md: Skip design adherence check, note "No design.md to verify against"
103
-
104
- **Code Pattern Consistency**:
105
- - Review new code for consistency with project patterns
106
- - Check file naming, directory structure, coding style
107
- - If significant deviations found:
108
- - Add SUGGESTION: "Code pattern deviation: <details>"
109
- - Recommendation: "Consider following project pattern: <example>"
110
-
111
- 8. **Generate Verification Report**
112
-
113
- **Summary Scorecard**:
114
- ```markdown
115
- ## Verification Report: <change-name>
116
-
117
- ### Summary
118
- | Dimension | Status |
119
- |--------------|------------------|
120
- | Completeness | X/Y tasks, N reqs|
121
- | Correctness | M/N reqs covered |
122
- | Coherence | Followed/Issues |
123
- ```
124
-
125
- **Issues by Priority**:
126
-
127
- 1. **CRITICAL** (Must fix before archive):
128
- - Incomplete tasks
129
- - Missing requirement implementations
130
- - Each with specific, actionable recommendation
131
-
132
- 2. **WARNING** (Should fix):
133
- - Spec/design divergences
134
- - Missing scenario coverage
135
- - Each with specific recommendation
136
-
137
- 3. **SUGGESTION** (Nice to fix):
138
- - Pattern inconsistencies
139
- - Minor improvements
140
- - Each with specific recommendation
141
-
142
- **Final Assessment**:
143
- - If CRITICAL issues: "X critical issue(s) found. Fix before archiving."
144
- - If only warnings: "No critical issues. Y warning(s) to consider. Ready for archive (with noted improvements)."
145
- - If all clear: "All checks passed. Ready for archive."
146
-
147
- **Verification Heuristics**
148
-
149
- - **Completeness**: Focus on objective checklist items (checkboxes, requirements list)
150
- - **Correctness**: Use keyword search, file path analysis, reasonable inference - don't require perfect certainty
151
- - **Coherence**: Look for glaring inconsistencies, don't nitpick style
152
- - **False Positives**: When uncertain, prefer SUGGESTION over WARNING, WARNING over CRITICAL
153
- - **Actionability**: Every issue must have a specific recommendation with file/line references where applicable
154
-
155
- **Graceful Degradation**
156
-
157
- - If only tasks.md exists: verify task completion only, skip spec/design checks
158
- - If tasks + specs exist: verify completeness and correctness, skip design
159
- - If full artifacts: verify all three dimensions
160
- - Always note which checks were skipped and why
161
-
162
- **Output Format**
163
-
164
- Use clear markdown with:
165
- - Table for summary scorecard
166
- - Grouped lists for issues (CRITICAL/WARNING/SUGGESTION)
167
- - Code references in format: `file.ts:123`
168
- - Specific, actionable recommendations
7
+ Verify that an implementation matches the change artifacts (specs, tasks, design).
8
+
9
+ **Store selection:** If the user names a store (a store is a standalone OpenSpec repo registered on this machine) or the work lives in one, run `openspec store list --json` to discover registered store ids, then pass `--store <id>` on the commands that read or write specs and changes (`new change`, `status`, `instructions`, `list`, `show`, `validate`, `archive`, `doctor`, `context`, `schemas`, `view`). Once selected, treat `--store <id>` as sticky for the rest of the workflow. Every unscoped example of those commands below is shorthand: before running it, append the flag. For example, run `openspec status --change "<name>" --json --store "<id>"`, not the unscoped form shown below. Other commands do not take the flag. Hints printed by commands already carry the flag; keep it on follow-ups. Without a store, commands act on the nearest local `openspec/` root.
10
+
11
+ **Project check:** These steps expect a project that already uses OpenSpec. Before the first step that writes anything (`new change`, `archive`, `sync specs`, or authoring an artifact file), confirm the project has a root: run `openspec list --json` (with `--store <id>` when a store is selected, since the store is then the root) and read `root`. A root object means the project is set up. `"root": null` means it is not - there is no `openspec/` directory here, and a write such as `openspec new change` would create one as a side effect. The command also exits non-zero, which is that answer rather than a broken CLI, so read the JSON instead of retrying or working around it.
12
+
13
+ One `"root": null` is not about setup: when a `status` error message starts with `Declared in` or `Invalid store declaration in` and names this project's `openspec/config.yaml` (or `config.yml`), the project does use OpenSpec through a store it declares, which this machine cannot resolve (the store is not registered, or the `store:` line is malformed). Do not treat it as uninitialized and skip the branches below: stop before writing and show the user that error's `message` and `fix`.
14
+
15
+ Otherwise, with no root, what happens next depends on how this workflow was reached:
16
+
17
+ - **Auto-selected**: you chose this workflow yourself, without the user naming OpenSpec, naming this skill, or running its slash command. Stop using OpenSpec and answer the request normally, as you would with no OpenSpec installed. Do not ask them to set anything up and do not mention OpenSpec setup.
18
+ - **Explicit OpenSpec request**: the user named OpenSpec, named this skill, or ran its slash command. Stop before writing and ask how to proceed: set this project up (`openspec init`), target a store they already have (`--store <id>`), or continue without OpenSpec for this request. Wait for their answer.
19
+
20
+ In both branches, never create the root as a side effect: do not run `openspec init` until the user asks for it, do not hand-create `openspec/` files, and do not let a command create it.
21
+
22
+ **Input**: Optionally specify a change name after `/opsx:verify` (e.g., `/opsx:verify add-auth`). If omitted, check if it can be inferred from conversation context. If vague or ambiguous you MUST prompt for available changes.
23
+
24
+ **Steps**
25
+
26
+ 1. **Select the change**
27
+
28
+ If a name is provided, use it. Otherwise:
29
+ - Infer from conversation context if the user mentioned a change
30
+ - Auto-select if only one active change exists
31
+ - If ambiguous, run `openspec list --json` to get available changes and ask the user to select one
32
+
33
+ When prompting, show all active changes returned by the list, including changes with `status: "no-tasks"`.
34
+ Include the schema used for each change if available.
35
+ Mark changes with incomplete tasks as "(In Progress)".
36
+
37
+ Always announce: "Using change: <name>" and how to override (e.g., `/opsx:verify <other>`).
38
+
39
+ 2. **Check status to understand the schema**
40
+ ```bash
41
+ openspec status --change "<name>" --json
42
+ ```
43
+ Parse the JSON to understand:
44
+ - `schemaName`: The workflow being used (e.g., "spec-driven")
45
+ - `planningHome`, `changeRoot`, `artifactPaths`, and `actionContext`: path and scope context
46
+ - Which artifacts exist for this change
47
+
48
+ 3. **Get planning context and load artifacts**
49
+
50
+ ```bash
51
+ openspec instructions apply --change "<name>" --json
52
+ ```
53
+
54
+ This returns the change directory, `contextFiles` (artifact ID -> array of concrete file paths), `taskTrackingConfigured`, and top-level `tasks` and `progress` aggregated from every concrete file matched by the schema's `apply.tracks` configuration that could be read. Read all available artifacts from `contextFiles`.
55
+
56
+ Treat apply `state` and `instruction` as context, not a verification verdict. Do not implement tasks or archive the change during verification.
57
+
58
+ 4. **Initialize verification report structure**
59
+
60
+ Create a report structure with three dimensions:
61
+ - **Completeness**: Track tasks and spec coverage
62
+ - **Correctness**: Track requirement implementation and scenario coverage
63
+ - **Coherence**: Track design adherence and pattern consistency
64
+
65
+ Each dimension can have CRITICAL, WARNING, or SUGGESTION issues.
66
+
67
+ Verification is advisory. Respect intentional omissions such as `skip_specs: true`, optional design documents, and schemas without task tracking. Do not require or invent optional or intentionally omitted artifacts to obtain a clean report. `Not verified` describes a limit of this report, not a new archive prerequisite. Archive retains its own checks and user-confirmation behavior.
68
+ Mark checks the schema does not define, or artifacts the status reports as intentionally skipped, as **Not applicable**. The correctness checks of a change whose readable delta specs contain REMOVED or RENAMED requirements but no ADDED or MODIFIED requirements are also **Not applicable** (see step 6). Exclude them from skipped-check counts and the archive-readiness assessment. Reserve **Not verified** for applicable checks whose evidence is missing or unusable.
69
+
70
+ If only task evidence is available for applicable checks, verify task completion only and mark the remaining applicable checks, including **Code Pattern Consistency**, as not verified with the reason "Only task evidence available".
71
+
72
+ If artifacts cannot be read or contain no usable requirements, scenarios, or design decisions, mark the affected checks as not verified with the specific reason. Continue checks supported by the remaining evidence, but a partially checked input set is not a fully verified check. Missing requirements affect Spec Coverage and Requirement Implementation Mapping; missing scenarios affect Scenario Coverage; missing design decisions affect Design Adherence.
73
+
74
+ 5. **Verify Completeness**
75
+
76
+ **Task Completion**:
77
+ - If `taskTrackingConfigured` is false, report **Task Completion** as not applicable. Do not treat empty `tasks` as missing evidence.
78
+ - Otherwise, use the top-level `tasks` and `progress` fields. They already aggregate every readable concrete file matched by `apply.tracks`, regardless of the tracked artifact's ID; do not infer tracking from a `contextFiles` key.
79
+ - If `unavailableTrackingFiles` is nonempty, mark **Task Completion** as not verified and include every unavailable path and reason. Continue using any readable task evidence, but do not infer completion from the partial `tasks` and `progress` fields.
80
+ - If `taskTrackingConfigured` is true and `tasks` is empty, mark **Task Completion** as not verified and record the reason from apply `state` and `instruction`. Nonzero totals alone do not establish evaluable task descriptions.
81
+ - Report complete vs total tasks from `progress`.
82
+ - If `progress.remaining` is greater than 0:
83
+ - Add CRITICAL issue for each listed incomplete task. If the remaining count exceeds the listed incomplete tasks, also report the incomplete checkboxes without descriptions and recommend adding descriptions and completing them. Do not infer completion from the listed tasks alone.
84
+ - Recommendation: "Complete task: <description>" or "Mark as done if already implemented"
85
+
86
+ **Spec Coverage**:
87
+ - If status marks the spec artifact skipped by `skip_specs: true`, or the schema defines no spec artifact, report the spec-dependent checks as not applicable.
88
+ - Otherwise, `contextFiles` is keyed by artifact id, and artifact ids come from the active schema. If `contextFiles.specs` is absent or empty, mark **Spec Coverage**, **Requirement Implementation Mapping**, and **Scenario Coverage** as not verified; do not treat any of them as clean.
89
+ - If delta specs exist in `contextFiles.specs`:
90
+ - Extract all requirements (marked with "### Requirement:", or listed as `FROM:`/`TO:` pairs under `## RENAMED Requirements`) and note the delta section each one sits under: `## ADDED`, `## MODIFIED`, `## REMOVED`, or `## RENAMED Requirements`. The section decides what the check looks for.
91
+ - For each ADDED or MODIFIED requirement (for MODIFIED, check the text in the delta, not the old wording):
92
+ - Search codebase for keywords related to the requirement
93
+ - Assess if implementation likely exists
94
+ - If ADDED or MODIFIED requirements appear unimplemented:
95
+ - Add CRITICAL issue: "Requirement not found: <requirement name>"
96
+ - Recommendation: "Implement requirement X: <description>"
97
+ - For each REMOVED requirement, the change asks for the behavior to be gone, so invert the check:
98
+ - Search codebase for the removed behavior. Matches in `openspec/` artifacts or docs, or in code that serves only the Migration note or an ADDED requirement, are not evidence by themselves. Report any code path that still delivers the removed behavior, including one shared with an ADDED requirement.
99
+ - Finding no implementation is the expected result. Never report a REMOVED requirement as "Requirement not found" or recommend implementing it.
100
+ - If the behavior is still present:
101
+ - Add CRITICAL issue: "Removed requirement still implemented: <requirement name>"
102
+ - Recommendation: "Remove the remaining implementation at <file>:<lines>, following the requirement's Migration note if it has one"
103
+ - For each RENAMED entry (`FROM:`/`TO:`), the name changes but the behavior stays, so check the TO requirement for that unchanged behavior:
104
+ - Do not report the FROM name as missing, and do not require code symbols, identifiers, or file names to be renamed.
105
+ - If the TO name also appears under MODIFIED, its behavior is checked there against the MODIFIED text; skip it here.
106
+ - Otherwise, read the baseline requirement in the main spec at `<planningHome.root>/openspec/specs/<capability-path>/spec.md`, using the same capability path as the delta spec: the requirement under the FROM name, or under the TO name only when the FROM name is absent because the main spec is already synced. Its body and scenarios are the evidence for the behavior the TO requirement keeps.
107
+ - Search codebase for that behavior and assess if it is still implemented.
108
+ - If it appears unimplemented:
109
+ - Add CRITICAL issue: "Renamed requirement not found: <TO name>"
110
+ - Recommendation: "Restore the behavior of <TO name> (renamed from <FROM name>); a rename must not change behavior"
111
+ - If the baseline requirement cannot be found or read, mark **Spec Coverage** as not verified for that entry with the reason. Never count an unchecked rename as passing.
112
+
113
+ 6. **Verify Correctness**
114
+
115
+ If the delta specs are readable and contain at least one REMOVED or RENAMED requirement but no ADDED or MODIFIED requirements (the change only removes or renames requirements), report **Requirement Implementation Mapping** and **Scenario Coverage** as **Not applicable**. The REMOVED and RENAMED checks under Spec Coverage are the evidence for such a change (each RENAMED entry is checked there against its baseline behavior), so do not mark these two checks as not verified. A delta spec with no parseable requirements at all is unusable evidence, not a removal-only change: mark these checks as not verified.
116
+
117
+ **Requirement Implementation Mapping**:
118
+ - For each ADDED or MODIFIED requirement from delta specs (REMOVED entries, and RENAMED entries without a MODIFIED block, were settled under Spec Coverage):
119
+ - Search codebase for implementation evidence
120
+ - If found, note file paths and line ranges
121
+ - Assess if implementation matches requirement intent
122
+ - If divergence detected:
123
+ - Add WARNING: "Implementation may diverge from spec: <details>"
124
+ - Recommendation: "Review <file>:<lines> against requirement X"
125
+
126
+ **Scenario Coverage**:
127
+ - For each scenario under an ADDED or MODIFIED requirement in delta specs (marked with "#### Scenario:"):
128
+ - Check if conditions are handled in code
129
+ - Check if tests exist covering the scenario
130
+ - If scenario appears uncovered:
131
+ - Add WARNING: "Scenario not covered: <scenario name>"
132
+ - Recommendation: "Add test or implementation for scenario: <description>"
133
+ - Skip scenarios under a REMOVED requirement; that behavior is meant to be gone.
134
+
135
+ 7. **Verify Coherence**
136
+
137
+ **Design Adherence**:
138
+ - If the schema defines no design artifact, report **Design Adherence** as not applicable.
139
+ - If `contextFiles.design` exists:
140
+ - Extract key decisions (look for sections like "Decision:", "Approach:", "Architecture:")
141
+ - Verify implementation follows those decisions
142
+ - If contradiction detected:
143
+ - Add WARNING: "Design decision not followed: <decision>"
144
+ - Recommendation: "Update implementation or revise design.md to match reality"
145
+ - Otherwise, if `contextFiles.design` is absent or empty: mark **Design Adherence** as not verified. With other supporting artifacts, **Code Pattern Consistency** still runs; the task-only case remains limited to task completion.
146
+
147
+ **Code Pattern Consistency**:
148
+ - If implementation changes cannot be identified, mark **Code Pattern Consistency** as not verified and explain the missing evidence.
149
+ - Otherwise, review new code for consistency with project patterns
150
+ - Check file naming, directory structure, coding style
151
+ - If significant deviations found:
152
+ - Add SUGGESTION: "Code pattern deviation: <details>"
153
+ - Recommendation: "Consider following project pattern: <example>"
154
+
155
+ 8. **Generate Verification Report**
156
+
157
+ **Summary Scorecard**:
158
+ ```markdown
159
+ ## Verification Report: <change-name>
160
+
161
+ ### Summary
162
+ | Dimension | Status |
163
+ |--------------|------------------|
164
+ | Completeness | X/Y tasks, N reqs|
165
+ | Correctness | M/N reqs covered |
166
+ | Coherence | Followed/Issues |
167
+ ```
168
+
169
+ In each Status cell, report the results of checks that ran and `Not verified (<reason>)` for every skipped check. If all checks in a dimension were skipped, start the cell with `Not verified`. Never score a skipped check as passing. Treat every not verified or partially verified check as skipped in the final assessment. Count only ADDED and MODIFIED requirements in N, and report REMOVED and RENAMED requirements separately (for example, "1 removal confirmed, 1 rename verified"). For a change that only removes or renames requirements, the Correctness cell reads `Not applicable (no ADDED or MODIFIED requirements)`.
170
+
171
+ **Issues by Priority**:
172
+
173
+ 1. **CRITICAL** (Must fix before archive):
174
+ - Incomplete tasks
175
+ - Missing requirement implementations
176
+ - Removed requirements still implemented
177
+ - Renamed requirements whose behavior is no longer implemented
178
+ - Each with specific, actionable recommendation
179
+
180
+ 2. **WARNING** (Should fix):
181
+ - Spec/design divergences
182
+ - Missing scenario coverage
183
+ - Each with specific recommendation
184
+
185
+ 3. **SUGGESTION** (Nice to fix):
186
+ - Pattern inconsistencies
187
+ - Minor improvements
188
+ - Each with specific recommendation
189
+
190
+ **Final Assessment**:
191
+ - If CRITICAL issues: "X critical issue(s) found. Fix before archiving." If any check was skipped, also name every skipped check and its reason.
192
+ - If no CRITICAL issues, one or more warnings, and no checks were skipped: "No critical issues. Y warning(s) to consider. Ready for archive (with noted improvements)."
193
+ - If only suggestions and no checks were skipped: "No critical issues or warnings. Z suggestion(s) to consider. Ready for archive (with noted improvements)."
194
+ - If no issues and no checks were skipped: "All checks passed. Ready for archive."
195
+ - If any check was skipped and there are no CRITICAL issues: do not claim readiness. Say "No critical issues found in the checks that ran. <check(s)> not verified: <reason>." Include the warning count when nonzero.
196
+ - Include the suggestion count when nonzero in every final assessment.
197
+
198
+ **Verification Heuristics**
199
+
200
+ - **Completeness**: Focus on objective checklist items (checkboxes, requirements list)
201
+ - **Correctness**: Use keyword search, file path analysis, reasonable inference - don't require perfect certainty
202
+ - **Coherence**: Look for glaring inconsistencies, don't nitpick style
203
+ - **False Positives**: When uncertain, prefer SUGGESTION over WARNING, WARNING over CRITICAL
204
+ - **Actionability**: Every issue must have a specific recommendation with file/line references where applicable
205
+
206
+ **Output Format**
207
+
208
+ Use clear markdown with:
209
+ - Table for summary scorecard
210
+ - Grouped lists for issues (CRITICAL/WARNING/SUGGESTION)
211
+ - Code references in format: `file.ts:123`
212
+ - Specific, actionable recommendations
169
213
  - No vague suggestions like "consider reviewing"