@kbdavis07/openspec 1.5.0-nov.1
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.
- package/LICENSE +22 -0
- package/README.md +239 -0
- package/bin/openspec.js +5 -0
- package/dist/cli/index.d.ts +10 -0
- package/dist/cli/index.js +544 -0
- package/dist/commands/change.d.ts +38 -0
- package/dist/commands/change.js +245 -0
- package/dist/commands/completion.d.ts +72 -0
- package/dist/commands/completion.js +264 -0
- package/dist/commands/config.d.ts +36 -0
- package/dist/commands/config.js +552 -0
- package/dist/commands/context.d.ts +3 -0
- package/dist/commands/context.js +155 -0
- package/dist/commands/doctor.d.ts +8 -0
- package/dist/commands/doctor.js +163 -0
- package/dist/commands/feedback.d.ts +9 -0
- package/dist/commands/feedback.js +183 -0
- package/dist/commands/schema.d.ts +6 -0
- package/dist/commands/schema.js +869 -0
- package/dist/commands/shared-gather.d.ts +14 -0
- package/dist/commands/shared-gather.js +31 -0
- package/dist/commands/shared-output.d.ts +18 -0
- package/dist/commands/shared-output.js +61 -0
- package/dist/commands/show.d.ts +19 -0
- package/dist/commands/show.js +177 -0
- package/dist/commands/spec.d.ts +19 -0
- package/dist/commands/spec.js +236 -0
- package/dist/commands/store.d.ts +3 -0
- package/dist/commands/store.js +547 -0
- package/dist/commands/validate.d.ts +34 -0
- package/dist/commands/validate.js +342 -0
- package/dist/commands/workflow/index.d.ts +17 -0
- package/dist/commands/workflow/index.js +12 -0
- package/dist/commands/workflow/instructions.d.ts +39 -0
- package/dist/commands/workflow/instructions.js +393 -0
- package/dist/commands/workflow/new-change.d.ts +20 -0
- package/dist/commands/workflow/new-change.js +109 -0
- package/dist/commands/workflow/schemas.d.ts +10 -0
- package/dist/commands/workflow/schemas.js +34 -0
- package/dist/commands/workflow/shared.d.ts +74 -0
- package/dist/commands/workflow/shared.js +170 -0
- package/dist/commands/workflow/status.d.ts +16 -0
- package/dist/commands/workflow/status.js +92 -0
- package/dist/commands/workflow/templates.d.ts +16 -0
- package/dist/commands/workflow/templates.js +69 -0
- package/dist/commands/workset-input.d.ts +19 -0
- package/dist/commands/workset-input.js +112 -0
- package/dist/commands/workset-prompts.d.ts +12 -0
- package/dist/commands/workset-prompts.js +143 -0
- package/dist/commands/workset.d.ts +25 -0
- package/dist/commands/workset.js +446 -0
- package/dist/core/archive.d.ts +22 -0
- package/dist/core/archive.js +496 -0
- package/dist/core/artifact-graph/graph.d.ts +56 -0
- package/dist/core/artifact-graph/graph.js +141 -0
- package/dist/core/artifact-graph/index.d.ts +9 -0
- package/dist/core/artifact-graph/index.js +14 -0
- package/dist/core/artifact-graph/instruction-loader.d.ts +188 -0
- package/dist/core/artifact-graph/instruction-loader.js +233 -0
- package/dist/core/artifact-graph/outputs.d.ts +14 -0
- package/dist/core/artifact-graph/outputs.js +39 -0
- package/dist/core/artifact-graph/resolver.d.ts +81 -0
- package/dist/core/artifact-graph/resolver.js +257 -0
- package/dist/core/artifact-graph/schema.d.ts +13 -0
- package/dist/core/artifact-graph/schema.js +108 -0
- package/dist/core/artifact-graph/state.d.ts +12 -0
- package/dist/core/artifact-graph/state.js +31 -0
- package/dist/core/artifact-graph/types.d.ts +40 -0
- package/dist/core/artifact-graph/types.js +29 -0
- package/dist/core/available-tools.d.ts +17 -0
- package/dist/core/available-tools.js +43 -0
- package/dist/core/change-metadata/index.d.ts +2 -0
- package/dist/core/change-metadata/index.js +2 -0
- package/dist/core/change-metadata/schema.d.ts +19 -0
- package/dist/core/change-metadata/schema.js +30 -0
- package/dist/core/change-status-policy.d.ts +37 -0
- package/dist/core/change-status-policy.js +35 -0
- package/dist/core/command-generation/adapters/amazon-q.d.ts +13 -0
- package/dist/core/command-generation/adapters/amazon-q.js +26 -0
- package/dist/core/command-generation/adapters/antigravity.d.ts +13 -0
- package/dist/core/command-generation/adapters/antigravity.js +26 -0
- package/dist/core/command-generation/adapters/auggie.d.ts +13 -0
- package/dist/core/command-generation/adapters/auggie.js +27 -0
- package/dist/core/command-generation/adapters/bob.d.ts +14 -0
- package/dist/core/command-generation/adapters/bob.js +32 -0
- package/dist/core/command-generation/adapters/claude.d.ts +13 -0
- package/dist/core/command-generation/adapters/claude.js +39 -0
- package/dist/core/command-generation/adapters/cline.d.ts +14 -0
- package/dist/core/command-generation/adapters/cline.js +27 -0
- package/dist/core/command-generation/adapters/codebuddy.d.ts +13 -0
- package/dist/core/command-generation/adapters/codebuddy.js +28 -0
- package/dist/core/command-generation/adapters/codex.d.ts +16 -0
- package/dist/core/command-generation/adapters/codex.js +39 -0
- package/dist/core/command-generation/adapters/continue.d.ts +13 -0
- package/dist/core/command-generation/adapters/continue.js +28 -0
- package/dist/core/command-generation/adapters/costrict.d.ts +13 -0
- package/dist/core/command-generation/adapters/costrict.js +27 -0
- package/dist/core/command-generation/adapters/crush.d.ts +13 -0
- package/dist/core/command-generation/adapters/crush.js +30 -0
- package/dist/core/command-generation/adapters/cursor.d.ts +14 -0
- package/dist/core/command-generation/adapters/cursor.js +31 -0
- package/dist/core/command-generation/adapters/factory.d.ts +13 -0
- package/dist/core/command-generation/adapters/factory.js +27 -0
- package/dist/core/command-generation/adapters/gemini.d.ts +13 -0
- package/dist/core/command-generation/adapters/gemini.js +26 -0
- package/dist/core/command-generation/adapters/github-copilot.d.ts +13 -0
- package/dist/core/command-generation/adapters/github-copilot.js +26 -0
- package/dist/core/command-generation/adapters/iflow.d.ts +13 -0
- package/dist/core/command-generation/adapters/iflow.js +29 -0
- package/dist/core/command-generation/adapters/index.d.ts +34 -0
- package/dist/core/command-generation/adapters/index.js +34 -0
- package/dist/core/command-generation/adapters/junie.d.ts +13 -0
- package/dist/core/command-generation/adapters/junie.js +26 -0
- package/dist/core/command-generation/adapters/kilocode.d.ts +14 -0
- package/dist/core/command-generation/adapters/kilocode.js +23 -0
- package/dist/core/command-generation/adapters/kiro.d.ts +13 -0
- package/dist/core/command-generation/adapters/kiro.js +26 -0
- package/dist/core/command-generation/adapters/lingma.d.ts +13 -0
- package/dist/core/command-generation/adapters/lingma.js +30 -0
- package/dist/core/command-generation/adapters/oh-my-pi.d.ts +21 -0
- package/dist/core/command-generation/adapters/oh-my-pi.js +44 -0
- package/dist/core/command-generation/adapters/opencode.d.ts +13 -0
- package/dist/core/command-generation/adapters/opencode.js +29 -0
- package/dist/core/command-generation/adapters/pi.d.ts +18 -0
- package/dist/core/command-generation/adapters/pi.js +42 -0
- package/dist/core/command-generation/adapters/qoder.d.ts +13 -0
- package/dist/core/command-generation/adapters/qoder.js +30 -0
- package/dist/core/command-generation/adapters/qwen.d.ts +13 -0
- package/dist/core/command-generation/adapters/qwen.js +26 -0
- package/dist/core/command-generation/adapters/roocode.d.ts +14 -0
- package/dist/core/command-generation/adapters/roocode.js +27 -0
- package/dist/core/command-generation/adapters/trae.d.ts +13 -0
- package/dist/core/command-generation/adapters/trae.js +48 -0
- package/dist/core/command-generation/adapters/windsurf.d.ts +14 -0
- package/dist/core/command-generation/adapters/windsurf.js +38 -0
- package/dist/core/command-generation/generator.d.ts +21 -0
- package/dist/core/command-generation/generator.js +27 -0
- package/dist/core/command-generation/index.d.ts +22 -0
- package/dist/core/command-generation/index.js +24 -0
- package/dist/core/command-generation/registry.d.ts +36 -0
- package/dist/core/command-generation/registry.js +102 -0
- package/dist/core/command-generation/types.d.ts +56 -0
- package/dist/core/command-generation/types.js +8 -0
- package/dist/core/command-generation/yaml.d.ts +22 -0
- package/dist/core/command-generation/yaml.js +38 -0
- package/dist/core/completions/command-registry.d.ts +3 -0
- package/dist/core/completions/command-registry.js +759 -0
- package/dist/core/completions/completion-provider.d.ts +71 -0
- package/dist/core/completions/completion-provider.js +129 -0
- package/dist/core/completions/factory.d.ts +64 -0
- package/dist/core/completions/factory.js +75 -0
- package/dist/core/completions/generators/bash-generator.d.ts +35 -0
- package/dist/core/completions/generators/bash-generator.js +230 -0
- package/dist/core/completions/generators/fish-generator.d.ts +32 -0
- package/dist/core/completions/generators/fish-generator.js +160 -0
- package/dist/core/completions/generators/powershell-generator.d.ts +36 -0
- package/dist/core/completions/generators/powershell-generator.js +266 -0
- package/dist/core/completions/generators/zsh-generator.d.ts +47 -0
- package/dist/core/completions/generators/zsh-generator.js +276 -0
- package/dist/core/completions/installers/bash-installer.d.ts +87 -0
- package/dist/core/completions/installers/bash-installer.js +321 -0
- package/dist/core/completions/installers/fish-installer.d.ts +43 -0
- package/dist/core/completions/installers/fish-installer.js +151 -0
- package/dist/core/completions/installers/powershell-installer.d.ts +102 -0
- package/dist/core/completions/installers/powershell-installer.js +415 -0
- package/dist/core/completions/installers/zsh-installer.d.ts +117 -0
- package/dist/core/completions/installers/zsh-installer.js +424 -0
- package/dist/core/completions/shared-flags.d.ts +13 -0
- package/dist/core/completions/shared-flags.js +33 -0
- package/dist/core/completions/templates/bash-templates.d.ts +6 -0
- package/dist/core/completions/templates/bash-templates.js +30 -0
- package/dist/core/completions/templates/fish-templates.d.ts +7 -0
- package/dist/core/completions/templates/fish-templates.js +45 -0
- package/dist/core/completions/templates/powershell-templates.d.ts +6 -0
- package/dist/core/completions/templates/powershell-templates.js +34 -0
- package/dist/core/completions/templates/zsh-templates.d.ts +6 -0
- package/dist/core/completions/templates/zsh-templates.js +45 -0
- package/dist/core/completions/types.d.ts +101 -0
- package/dist/core/completions/types.js +2 -0
- package/dist/core/config-prompts.d.ts +9 -0
- package/dist/core/config-prompts.js +34 -0
- package/dist/core/config-schema.d.ts +87 -0
- package/dist/core/config-schema.js +239 -0
- package/dist/core/config.d.ts +18 -0
- package/dist/core/config.js +40 -0
- package/dist/core/converters/json-converter.d.ts +6 -0
- package/dist/core/converters/json-converter.js +51 -0
- package/dist/core/file-state.d.ts +36 -0
- package/dist/core/file-state.js +112 -0
- package/dist/core/global-config.d.ts +51 -0
- package/dist/core/global-config.js +124 -0
- package/dist/core/id.d.ts +17 -0
- package/dist/core/id.js +30 -0
- package/dist/core/index.d.ts +6 -0
- package/dist/core/index.js +7 -0
- package/dist/core/init.d.ts +37 -0
- package/dist/core/init.js +614 -0
- package/dist/core/legacy-cleanup.d.ts +162 -0
- package/dist/core/legacy-cleanup.js +514 -0
- package/dist/core/list.d.ts +11 -0
- package/dist/core/list.js +214 -0
- package/dist/core/migration.d.ts +23 -0
- package/dist/core/migration.js +108 -0
- package/dist/core/openers.d.ts +77 -0
- package/dist/core/openers.js +251 -0
- package/dist/core/openspec-root.d.ts +45 -0
- package/dist/core/openspec-root.js +195 -0
- package/dist/core/parsers/change-parser.d.ts +13 -0
- package/dist/core/parsers/change-parser.js +198 -0
- package/dist/core/parsers/markdown-parser.d.ts +23 -0
- package/dist/core/parsers/markdown-parser.js +173 -0
- package/dist/core/parsers/requirement-blocks.d.ts +48 -0
- package/dist/core/parsers/requirement-blocks.js +235 -0
- package/dist/core/parsers/requirement-text.d.ts +55 -0
- package/dist/core/parsers/requirement-text.js +136 -0
- package/dist/core/parsers/spec-structure.d.ts +9 -0
- package/dist/core/parsers/spec-structure.js +88 -0
- package/dist/core/planning-home.d.ts +16 -0
- package/dist/core/planning-home.js +67 -0
- package/dist/core/profile-sync-drift.d.ts +38 -0
- package/dist/core/profile-sync-drift.js +201 -0
- package/dist/core/profiles.d.ts +26 -0
- package/dist/core/profiles.js +41 -0
- package/dist/core/project-config.d.ts +107 -0
- package/dist/core/project-config.js +373 -0
- package/dist/core/references.d.ts +63 -0
- package/dist/core/references.js +310 -0
- package/dist/core/relationship-health.d.ts +65 -0
- package/dist/core/relationship-health.js +64 -0
- package/dist/core/root-selection.d.ts +122 -0
- package/dist/core/root-selection.js +337 -0
- package/dist/core/schemas/base.schema.d.ts +13 -0
- package/dist/core/schemas/base.schema.js +18 -0
- package/dist/core/schemas/change.schema.d.ts +73 -0
- package/dist/core/schemas/change.schema.js +31 -0
- package/dist/core/schemas/index.d.ts +4 -0
- package/dist/core/schemas/index.js +4 -0
- package/dist/core/schemas/spec.schema.d.ts +18 -0
- package/dist/core/schemas/spec.schema.js +15 -0
- package/dist/core/shared/allowed-tools.d.ts +12 -0
- package/dist/core/shared/allowed-tools.js +12 -0
- package/dist/core/shared/index.d.ts +8 -0
- package/dist/core/shared/index.js +8 -0
- package/dist/core/shared/skill-generation.d.ts +49 -0
- package/dist/core/shared/skill-generation.js +100 -0
- package/dist/core/shared/tool-detection.d.ts +71 -0
- package/dist/core/shared/tool-detection.js +160 -0
- package/dist/core/specs-apply.d.ts +78 -0
- package/dist/core/specs-apply.js +428 -0
- package/dist/core/store/errors.d.ts +20 -0
- package/dist/core/store/errors.js +22 -0
- package/dist/core/store/foundation.d.ts +56 -0
- package/dist/core/store/foundation.js +251 -0
- package/dist/core/store/git.d.ts +23 -0
- package/dist/core/store/git.js +137 -0
- package/dist/core/store/index.d.ts +5 -0
- package/dist/core/store/index.js +5 -0
- package/dist/core/store/operations.d.ts +114 -0
- package/dist/core/store/operations.js +803 -0
- package/dist/core/store/registry.d.ts +58 -0
- package/dist/core/store/registry.js +275 -0
- package/dist/core/styles/palette.d.ts +7 -0
- package/dist/core/styles/palette.js +8 -0
- package/dist/core/templates/index.d.ts +8 -0
- package/dist/core/templates/index.js +9 -0
- package/dist/core/templates/skill-templates.d.ts +20 -0
- package/dist/core/templates/skill-templates.js +19 -0
- package/dist/core/templates/types.d.ts +19 -0
- package/dist/core/templates/types.js +5 -0
- package/dist/core/templates/workflows/apply-change.d.ts +10 -0
- package/dist/core/templates/workflows/apply-change.js +315 -0
- package/dist/core/templates/workflows/archive-change.d.ts +10 -0
- package/dist/core/templates/workflows/archive-change.js +278 -0
- package/dist/core/templates/workflows/bulk-archive-change.d.ts +10 -0
- package/dist/core/templates/workflows/bulk-archive-change.js +493 -0
- package/dist/core/templates/workflows/continue-change.d.ts +10 -0
- package/dist/core/templates/workflows/continue-change.js +239 -0
- package/dist/core/templates/workflows/explore.d.ts +10 -0
- package/dist/core/templates/workflows/explore.js +464 -0
- package/dist/core/templates/workflows/feedback.d.ts +9 -0
- package/dist/core/templates/workflows/feedback.js +108 -0
- package/dist/core/templates/workflows/ff-change.d.ts +10 -0
- package/dist/core/templates/workflows/ff-change.js +205 -0
- package/dist/core/templates/workflows/new-change.d.ts +10 -0
- package/dist/core/templates/workflows/new-change.js +148 -0
- package/dist/core/templates/workflows/onboard.d.ts +10 -0
- package/dist/core/templates/workflows/onboard.js +566 -0
- package/dist/core/templates/workflows/propose.d.ts +10 -0
- package/dist/core/templates/workflows/propose.js +223 -0
- package/dist/core/templates/workflows/store-selection.d.ts +8 -0
- package/dist/core/templates/workflows/store-selection.js +8 -0
- package/dist/core/templates/workflows/sync-specs.d.ts +10 -0
- package/dist/core/templates/workflows/sync-specs.js +291 -0
- package/dist/core/templates/workflows/update-change.d.ts +10 -0
- package/dist/core/templates/workflows/update-change.js +167 -0
- package/dist/core/templates/workflows/verify-change.d.ts +10 -0
- package/dist/core/templates/workflows/verify-change.js +339 -0
- package/dist/core/update.d.ts +82 -0
- package/dist/core/update.js +557 -0
- package/dist/core/validation/constants.d.ts +34 -0
- package/dist/core/validation/constants.js +40 -0
- package/dist/core/validation/types.d.ts +18 -0
- package/dist/core/validation/types.js +2 -0
- package/dist/core/validation/validator.d.ts +51 -0
- package/dist/core/validation/validator.js +498 -0
- package/dist/core/view.d.ts +8 -0
- package/dist/core/view.js +168 -0
- package/dist/core/working-set.d.ts +47 -0
- package/dist/core/working-set.js +43 -0
- package/dist/core/worksets.d.ts +75 -0
- package/dist/core/worksets.js +245 -0
- package/dist/core/zod-issues.d.ts +4 -0
- package/dist/core/zod-issues.js +10 -0
- package/dist/index.d.ts +3 -0
- package/dist/index.js +3 -0
- package/dist/prompts/searchable-multi-select.d.ts +28 -0
- package/dist/prompts/searchable-multi-select.js +159 -0
- package/dist/telemetry/config.d.ts +38 -0
- package/dist/telemetry/config.js +136 -0
- package/dist/telemetry/index.d.ts +31 -0
- package/dist/telemetry/index.js +164 -0
- package/dist/ui/ascii-patterns.d.ts +16 -0
- package/dist/ui/ascii-patterns.js +133 -0
- package/dist/ui/welcome-screen.d.ts +10 -0
- package/dist/ui/welcome-screen.js +146 -0
- package/dist/utils/change-metadata.d.ts +55 -0
- package/dist/utils/change-metadata.js +141 -0
- package/dist/utils/change-utils.d.ts +85 -0
- package/dist/utils/change-utils.js +173 -0
- package/dist/utils/command-references.d.ts +18 -0
- package/dist/utils/command-references.js +20 -0
- package/dist/utils/file-system.d.ts +41 -0
- package/dist/utils/file-system.js +320 -0
- package/dist/utils/index.d.ts +6 -0
- package/dist/utils/index.js +9 -0
- package/dist/utils/interactive.d.ts +18 -0
- package/dist/utils/interactive.js +21 -0
- package/dist/utils/item-discovery.d.ts +4 -0
- package/dist/utils/item-discovery.js +72 -0
- package/dist/utils/match.d.ts +3 -0
- package/dist/utils/match.js +22 -0
- package/dist/utils/shell-detection.d.ts +20 -0
- package/dist/utils/shell-detection.js +41 -0
- package/dist/utils/task-progress.d.ts +17 -0
- package/dist/utils/task-progress.js +100 -0
- package/package.json +85 -0
- package/schemas/spec-driven/schema.yaml +153 -0
- package/schemas/spec-driven/templates/design.md +19 -0
- package/schemas/spec-driven/templates/proposal.md +23 -0
- package/schemas/spec-driven/templates/spec.md +8 -0
- package/schemas/spec-driven/templates/tasks.md +9 -0
- package/scripts/postinstall.js +83 -0
|
@@ -0,0 +1,167 @@
|
|
|
1
|
+
import { STORE_SELECTION_GUIDANCE } from './store-selection.js';
|
|
2
|
+
export function getUpdateChangeSkillTemplate() {
|
|
3
|
+
return {
|
|
4
|
+
name: 'openspec-update-change',
|
|
5
|
+
description: "Update an OpenSpec change by revising its existing planning artifacts and keeping them coherent with one another. Use when the user wants to revise a change's plan, fold new decisions into it, or reconcile its artifacts after an edit. Never edits code.",
|
|
6
|
+
instructions: `Revise a change's existing planning artifacts and keep them coherent. Never edit code.
|
|
7
|
+
|
|
8
|
+
${STORE_SELECTION_GUIDANCE}
|
|
9
|
+
|
|
10
|
+
**Input**: Optionally specify a change name. If omitted, check if it can be inferred from conversation context. If vague or ambiguous you MUST prompt for available changes.
|
|
11
|
+
|
|
12
|
+
**Steps**
|
|
13
|
+
|
|
14
|
+
1. **If no change name provided, prompt for selection**
|
|
15
|
+
|
|
16
|
+
Run \`openspec list --json\` to get available changes sorted by most recently modified. Then use the **AskUserQuestion tool** to let the user select which change to update.
|
|
17
|
+
|
|
18
|
+
Present the top 3-4 most recently modified changes as options, showing:
|
|
19
|
+
- Change name
|
|
20
|
+
- Schema (from \`schema\` field if present, otherwise "spec-driven")
|
|
21
|
+
- Status (e.g., "0/5 tasks", "complete", "no tasks")
|
|
22
|
+
- How recently it was modified (from \`lastModified\` field)
|
|
23
|
+
|
|
24
|
+
Mark the most recently modified change as "(Recommended)" since it's likely what the user wants to update.
|
|
25
|
+
|
|
26
|
+
**IMPORTANT**: Do NOT guess or auto-select a change. Always let the user choose.
|
|
27
|
+
|
|
28
|
+
2. **Get the change's artifacts**
|
|
29
|
+
\`\`\`bash
|
|
30
|
+
openspec status --change "<name>" --json
|
|
31
|
+
\`\`\`
|
|
32
|
+
Parse the JSON to understand current state. The response includes:
|
|
33
|
+
- \`schemaName\`: The workflow schema being used (e.g., "spec-driven")
|
|
34
|
+
- \`artifacts\`: Array of artifacts with their status ("done", "ready", "blocked")
|
|
35
|
+
- \`isComplete\`: Boolean indicating if all artifacts are complete
|
|
36
|
+
- \`planningHome\`, \`changeRoot\`, \`artifactPaths\`, and \`actionContext\`: path and scope context. Use these instead of assuming repo-local paths.
|
|
37
|
+
|
|
38
|
+
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.
|
|
39
|
+
|
|
40
|
+
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.
|
|
41
|
+
|
|
42
|
+
3. **Understand the request**
|
|
43
|
+
- If the user asked for a specific revision ("the design now uses X"), that is the starting edit.
|
|
44
|
+
- 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.
|
|
45
|
+
|
|
46
|
+
4. **Read and reconcile**
|
|
47
|
+
- Read the artifact(s) the request touches and the change's other existing artifacts.
|
|
48
|
+
- 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.
|
|
49
|
+
- Note everything that is now inconsistent, missing, or contradictory.
|
|
50
|
+
- 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.
|
|
51
|
+
- If the change is already coherent, say so and make no edits.
|
|
52
|
+
|
|
53
|
+
5. **Confirm and apply, one artifact at a time**
|
|
54
|
+
- Show each proposed revision and why. Write only after the user confirms.
|
|
55
|
+
- If the user rejects a revision, do not write it - leave that artifact unchanged.
|
|
56
|
+
- When a substantial rewrite is needed, get that artifact's rules and template first:
|
|
57
|
+
\`\`\`bash
|
|
58
|
+
openspec instructions <artifact-id> --change "<name>" --json
|
|
59
|
+
\`\`\`
|
|
60
|
+
|
|
61
|
+
6. **Point to the next step (guidance only - NEVER act on it)**
|
|
62
|
+
- Artifacts still missing -> suggest \`/opsx:continue\` to create them.
|
|
63
|
+
- 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.
|
|
64
|
+
- Everything done and implemented -> suggest \`/opsx:archive\`.
|
|
65
|
+
|
|
66
|
+
**Output**
|
|
67
|
+
|
|
68
|
+
After each invocation, show:
|
|
69
|
+
- Which artifacts were revised (and which proposed revisions were rejected)
|
|
70
|
+
- Anything deferred to \`/opsx:continue\` (not-yet-created artifacts or files)
|
|
71
|
+
- Where the change stands and the recommended next command
|
|
72
|
+
|
|
73
|
+
**Guardrails**
|
|
74
|
+
- Planning artifacts only - NEVER edit implementation code. If the revised plan implies code changes, stop and point to \`/opsx:apply\`.
|
|
75
|
+
- Use the artifact ids and paths reported by \`openspec status\`; never branch on hardcoded artifact names.
|
|
76
|
+
- Edit only the concrete files in \`existingOutputPaths\`; never write to a glob \`resolvedOutputPath\`.
|
|
77
|
+
- Do not advance the build frontier: no new artifacts, no new files under glob artifacts - that is \`/opsx:continue\`'s job.
|
|
78
|
+
- Confirm every edit with the user before writing.
|
|
79
|
+
- If the request changes the change's *intent* rather than refining it, recommend starting fresh with \`/opsx:new\` (the "Update vs. Start Fresh" heuristic).`,
|
|
80
|
+
license: 'MIT',
|
|
81
|
+
compatibility: 'Requires openspec CLI.',
|
|
82
|
+
metadata: { author: 'openspec', version: '1.0' },
|
|
83
|
+
};
|
|
84
|
+
}
|
|
85
|
+
export function getOpsxUpdateCommandTemplate() {
|
|
86
|
+
return {
|
|
87
|
+
name: 'OPSX: Update',
|
|
88
|
+
description: "Update a change - revise existing planning artifacts and keep them coherent (Experimental)",
|
|
89
|
+
category: 'Workflow',
|
|
90
|
+
tags: ['workflow', 'artifacts', 'experimental'],
|
|
91
|
+
content: `Revise a change's existing planning artifacts and keep them coherent. Never edit code.
|
|
92
|
+
|
|
93
|
+
${STORE_SELECTION_GUIDANCE}
|
|
94
|
+
|
|
95
|
+
**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.
|
|
96
|
+
|
|
97
|
+
**Steps**
|
|
98
|
+
|
|
99
|
+
1. **If no change name provided, prompt for selection**
|
|
100
|
+
|
|
101
|
+
Run \`openspec list --json\` to get available changes sorted by most recently modified. Then use the **AskUserQuestion tool** to let the user select which change to update.
|
|
102
|
+
|
|
103
|
+
Present the top 3-4 most recently modified changes as options, showing:
|
|
104
|
+
- Change name
|
|
105
|
+
- Schema (from \`schema\` field if present, otherwise "spec-driven")
|
|
106
|
+
- Status (e.g., "0/5 tasks", "complete", "no tasks")
|
|
107
|
+
- How recently it was modified (from \`lastModified\` field)
|
|
108
|
+
|
|
109
|
+
Mark the most recently modified change as "(Recommended)" since it's likely what the user wants to update.
|
|
110
|
+
|
|
111
|
+
**IMPORTANT**: Do NOT guess or auto-select a change. Always let the user choose.
|
|
112
|
+
|
|
113
|
+
2. **Get the change's artifacts**
|
|
114
|
+
\`\`\`bash
|
|
115
|
+
openspec status --change "<name>" --json
|
|
116
|
+
\`\`\`
|
|
117
|
+
Parse the JSON to understand current state. The response includes:
|
|
118
|
+
- \`schemaName\`: The workflow schema being used (e.g., "spec-driven")
|
|
119
|
+
- \`artifacts\`: Array of artifacts with their status ("done", "ready", "blocked")
|
|
120
|
+
- \`isComplete\`: Boolean indicating if all artifacts are complete
|
|
121
|
+
- \`planningHome\`, \`changeRoot\`, \`artifactPaths\`, and \`actionContext\`: path and scope context. Use these instead of assuming repo-local paths.
|
|
122
|
+
|
|
123
|
+
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.
|
|
124
|
+
|
|
125
|
+
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.
|
|
126
|
+
|
|
127
|
+
3. **Understand the request**
|
|
128
|
+
- If the user asked for a specific revision ("the design now uses X"), that is the starting edit.
|
|
129
|
+
- 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.
|
|
130
|
+
|
|
131
|
+
4. **Read and reconcile**
|
|
132
|
+
- Read the artifact(s) the request touches and the change's other existing artifacts.
|
|
133
|
+
- 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.
|
|
134
|
+
- Note everything that is now inconsistent, missing, or contradictory.
|
|
135
|
+
- 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.
|
|
136
|
+
- If the change is already coherent, say so and make no edits.
|
|
137
|
+
|
|
138
|
+
5. **Confirm and apply, one artifact at a time**
|
|
139
|
+
- Show each proposed revision and why. Write only after the user confirms.
|
|
140
|
+
- If the user rejects a revision, do not write it - leave that artifact unchanged.
|
|
141
|
+
- When a substantial rewrite is needed, get that artifact's rules and template first:
|
|
142
|
+
\`\`\`bash
|
|
143
|
+
openspec instructions <artifact-id> --change "<name>" --json
|
|
144
|
+
\`\`\`
|
|
145
|
+
|
|
146
|
+
6. **Point to the next step (guidance only - NEVER act on it)**
|
|
147
|
+
- Artifacts still missing -> suggest \`/opsx:continue\` to create them.
|
|
148
|
+
- 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.
|
|
149
|
+
- Everything done and implemented -> suggest \`/opsx:archive\`.
|
|
150
|
+
|
|
151
|
+
**Output**
|
|
152
|
+
|
|
153
|
+
After each invocation, show:
|
|
154
|
+
- Which artifacts were revised (and which proposed revisions were rejected)
|
|
155
|
+
- Anything deferred to \`/opsx:continue\` (not-yet-created artifacts or files)
|
|
156
|
+
- Where the change stands and the recommended next command
|
|
157
|
+
|
|
158
|
+
**Guardrails**
|
|
159
|
+
- Planning artifacts only - NEVER edit implementation code. If the revised plan implies code changes, stop and point to \`/opsx:apply\`.
|
|
160
|
+
- Use the artifact ids and paths reported by \`openspec status\`; never branch on hardcoded artifact names.
|
|
161
|
+
- Edit only the concrete files in \`existingOutputPaths\`; never write to a glob \`resolvedOutputPath\`.
|
|
162
|
+
- Do not advance the build frontier: no new artifacts, no new files under glob artifacts - that is \`/opsx:continue\`'s job.
|
|
163
|
+
- Confirm every edit with the user before writing.
|
|
164
|
+
- If the request changes the change's *intent* rather than refining it, recommend starting fresh with \`/opsx:new\` (the "Update vs. Start Fresh" heuristic).`
|
|
165
|
+
};
|
|
166
|
+
}
|
|
167
|
+
//# sourceMappingURL=update-change.js.map
|
|
@@ -0,0 +1,10 @@
|
|
|
1
|
+
/**
|
|
2
|
+
* Skill Template Workflow Modules
|
|
3
|
+
*
|
|
4
|
+
* This file is generated by splitting the legacy monolithic
|
|
5
|
+
* templates file into workflow-focused modules.
|
|
6
|
+
*/
|
|
7
|
+
import type { SkillTemplate, CommandTemplate } from '../types.js';
|
|
8
|
+
export declare function getVerifyChangeSkillTemplate(): SkillTemplate;
|
|
9
|
+
export declare function getOpsxVerifyCommandTemplate(): CommandTemplate;
|
|
10
|
+
//# sourceMappingURL=verify-change.d.ts.map
|
|
@@ -0,0 +1,339 @@
|
|
|
1
|
+
import { STORE_SELECTION_GUIDANCE } from './store-selection.js';
|
|
2
|
+
export function getVerifyChangeSkillTemplate() {
|
|
3
|
+
return {
|
|
4
|
+
name: 'openspec-verify-change',
|
|
5
|
+
description: 'Verify implementation matches change artifacts. Use when the user wants to validate that implementation is complete, correct, and coherent before archiving.',
|
|
6
|
+
instructions: `Verify that an implementation matches the change artifacts (specs, tasks, design).
|
|
7
|
+
|
|
8
|
+
${STORE_SELECTION_GUIDANCE}
|
|
9
|
+
|
|
10
|
+
**Input**: Optionally specify a change name. If omitted, check if it can be inferred from conversation context. If vague or ambiguous you MUST prompt for available changes.
|
|
11
|
+
|
|
12
|
+
**Steps**
|
|
13
|
+
|
|
14
|
+
1. **If no change name provided, prompt for selection**
|
|
15
|
+
|
|
16
|
+
Run \`openspec list --json\` to get available changes. Use the **AskUserQuestion tool** to let the user select.
|
|
17
|
+
|
|
18
|
+
Show changes that have implementation tasks (tasks artifact exists).
|
|
19
|
+
Include the schema used for each change if available.
|
|
20
|
+
Mark changes with incomplete tasks as "(In Progress)".
|
|
21
|
+
|
|
22
|
+
**IMPORTANT**: Do NOT guess or auto-select a change. Always let the user choose.
|
|
23
|
+
|
|
24
|
+
2. **Check status to understand the schema**
|
|
25
|
+
\`\`\`bash
|
|
26
|
+
openspec status --change "<name>" --json
|
|
27
|
+
\`\`\`
|
|
28
|
+
Parse the JSON to understand:
|
|
29
|
+
- \`schemaName\`: The workflow being used (e.g., "spec-driven")
|
|
30
|
+
- \`planningHome\`, \`changeRoot\`, \`artifactPaths\`, and \`actionContext\`: path and scope context
|
|
31
|
+
- Which artifacts exist for this change
|
|
32
|
+
|
|
33
|
+
3. **Get planning context and load artifacts**
|
|
34
|
+
|
|
35
|
+
\`\`\`bash
|
|
36
|
+
openspec instructions apply --change "<name>" --json
|
|
37
|
+
\`\`\`
|
|
38
|
+
|
|
39
|
+
This returns the change directory and \`contextFiles\` (artifact ID -> array of concrete file paths). Read all available artifacts from \`contextFiles\`.
|
|
40
|
+
|
|
41
|
+
4. **Initialize verification report structure**
|
|
42
|
+
|
|
43
|
+
Create a report structure with three dimensions:
|
|
44
|
+
- **Completeness**: Track tasks and spec coverage
|
|
45
|
+
- **Correctness**: Track requirement implementation and scenario coverage
|
|
46
|
+
- **Coherence**: Track design adherence and pattern consistency
|
|
47
|
+
|
|
48
|
+
Each dimension can have CRITICAL, WARNING, or SUGGESTION issues.
|
|
49
|
+
|
|
50
|
+
5. **Verify Completeness**
|
|
51
|
+
|
|
52
|
+
**Task Completion**:
|
|
53
|
+
- If \`contextFiles.tasks\` exists, read every file path in it
|
|
54
|
+
- Parse checkboxes: \`- [ ]\` (incomplete) vs \`- [x]\` (complete)
|
|
55
|
+
- Count complete vs total tasks
|
|
56
|
+
- If incomplete tasks exist:
|
|
57
|
+
- Add CRITICAL issue for each incomplete task
|
|
58
|
+
- Recommendation: "Complete task: <description>" or "Mark as done if already implemented"
|
|
59
|
+
|
|
60
|
+
**Spec Coverage**:
|
|
61
|
+
- If delta specs exist in \`contextFiles.specs\`:
|
|
62
|
+
- Extract all requirements (marked with "### Requirement:")
|
|
63
|
+
- For each requirement:
|
|
64
|
+
- Search codebase for keywords related to the requirement
|
|
65
|
+
- Assess if implementation likely exists
|
|
66
|
+
- If requirements appear unimplemented:
|
|
67
|
+
- Add CRITICAL issue: "Requirement not found: <requirement name>"
|
|
68
|
+
- Recommendation: "Implement requirement X: <description>"
|
|
69
|
+
|
|
70
|
+
6. **Verify Correctness**
|
|
71
|
+
|
|
72
|
+
**Requirement Implementation Mapping**:
|
|
73
|
+
- For each requirement from delta specs:
|
|
74
|
+
- Search codebase for implementation evidence
|
|
75
|
+
- If found, note file paths and line ranges
|
|
76
|
+
- Assess if implementation matches requirement intent
|
|
77
|
+
- If divergence detected:
|
|
78
|
+
- Add WARNING: "Implementation may diverge from spec: <details>"
|
|
79
|
+
- Recommendation: "Review <file>:<lines> against requirement X"
|
|
80
|
+
|
|
81
|
+
**Scenario Coverage**:
|
|
82
|
+
- For each scenario in delta specs (marked with "#### Scenario:"):
|
|
83
|
+
- Check if conditions are handled in code
|
|
84
|
+
- Check if tests exist covering the scenario
|
|
85
|
+
- If scenario appears uncovered:
|
|
86
|
+
- Add WARNING: "Scenario not covered: <scenario name>"
|
|
87
|
+
- Recommendation: "Add test or implementation for scenario: <description>"
|
|
88
|
+
|
|
89
|
+
7. **Verify Coherence**
|
|
90
|
+
|
|
91
|
+
**Design Adherence**:
|
|
92
|
+
- If \`contextFiles.design\` exists:
|
|
93
|
+
- Extract key decisions (look for sections like "Decision:", "Approach:", "Architecture:")
|
|
94
|
+
- Verify implementation follows those decisions
|
|
95
|
+
- If contradiction detected:
|
|
96
|
+
- Add WARNING: "Design decision not followed: <decision>"
|
|
97
|
+
- Recommendation: "Update implementation or revise design.md to match reality"
|
|
98
|
+
- If no design.md: Skip design adherence check, note "No design.md to verify against"
|
|
99
|
+
|
|
100
|
+
**Code Pattern Consistency**:
|
|
101
|
+
- Review new code for consistency with project patterns
|
|
102
|
+
- Check file naming, directory structure, coding style
|
|
103
|
+
- If significant deviations found:
|
|
104
|
+
- Add SUGGESTION: "Code pattern deviation: <details>"
|
|
105
|
+
- Recommendation: "Consider following project pattern: <example>"
|
|
106
|
+
|
|
107
|
+
8. **Generate Verification Report**
|
|
108
|
+
|
|
109
|
+
**Summary Scorecard**:
|
|
110
|
+
\`\`\`
|
|
111
|
+
## Verification Report: <change-name>
|
|
112
|
+
|
|
113
|
+
### Summary
|
|
114
|
+
| Dimension | Status |
|
|
115
|
+
|--------------|------------------|
|
|
116
|
+
| Completeness | X/Y tasks, N reqs|
|
|
117
|
+
| Correctness | M/N reqs covered |
|
|
118
|
+
| Coherence | Followed/Issues |
|
|
119
|
+
\`\`\`
|
|
120
|
+
|
|
121
|
+
**Issues by Priority**:
|
|
122
|
+
|
|
123
|
+
1. **CRITICAL** (Must fix before archive):
|
|
124
|
+
- Incomplete tasks
|
|
125
|
+
- Missing requirement implementations
|
|
126
|
+
- Each with specific, actionable recommendation
|
|
127
|
+
|
|
128
|
+
2. **WARNING** (Should fix):
|
|
129
|
+
- Spec/design divergences
|
|
130
|
+
- Missing scenario coverage
|
|
131
|
+
- Each with specific recommendation
|
|
132
|
+
|
|
133
|
+
3. **SUGGESTION** (Nice to fix):
|
|
134
|
+
- Pattern inconsistencies
|
|
135
|
+
- Minor improvements
|
|
136
|
+
- Each with specific recommendation
|
|
137
|
+
|
|
138
|
+
**Final Assessment**:
|
|
139
|
+
- If CRITICAL issues: "X critical issue(s) found. Fix before archiving."
|
|
140
|
+
- If only warnings: "No critical issues. Y warning(s) to consider. Ready for archive (with noted improvements)."
|
|
141
|
+
- If all clear: "All checks passed. Ready for archive."
|
|
142
|
+
|
|
143
|
+
**Verification Heuristics**
|
|
144
|
+
|
|
145
|
+
- **Completeness**: Focus on objective checklist items (checkboxes, requirements list)
|
|
146
|
+
- **Correctness**: Use keyword search, file path analysis, reasonable inference - don't require perfect certainty
|
|
147
|
+
- **Coherence**: Look for glaring inconsistencies, don't nitpick style
|
|
148
|
+
- **False Positives**: When uncertain, prefer SUGGESTION over WARNING, WARNING over CRITICAL
|
|
149
|
+
- **Actionability**: Every issue must have a specific recommendation with file/line references where applicable
|
|
150
|
+
|
|
151
|
+
**Graceful Degradation**
|
|
152
|
+
|
|
153
|
+
- If only tasks.md exists: verify task completion only, skip spec/design checks
|
|
154
|
+
- If tasks + specs exist: verify completeness and correctness, skip design
|
|
155
|
+
- If full artifacts: verify all three dimensions
|
|
156
|
+
- Always note which checks were skipped and why
|
|
157
|
+
|
|
158
|
+
**Output Format**
|
|
159
|
+
|
|
160
|
+
Use clear markdown with:
|
|
161
|
+
- Table for summary scorecard
|
|
162
|
+
- Grouped lists for issues (CRITICAL/WARNING/SUGGESTION)
|
|
163
|
+
- Code references in format: \`file.ts:123\`
|
|
164
|
+
- Specific, actionable recommendations
|
|
165
|
+
- No vague suggestions like "consider reviewing"`,
|
|
166
|
+
license: 'MIT',
|
|
167
|
+
compatibility: 'Requires openspec CLI.',
|
|
168
|
+
metadata: { author: 'openspec', version: '1.0' },
|
|
169
|
+
};
|
|
170
|
+
}
|
|
171
|
+
export function getOpsxVerifyCommandTemplate() {
|
|
172
|
+
return {
|
|
173
|
+
name: 'OPSX: Verify',
|
|
174
|
+
description: 'Verify implementation matches change artifacts before archiving',
|
|
175
|
+
category: 'Workflow',
|
|
176
|
+
tags: ['workflow', 'verify', 'experimental'],
|
|
177
|
+
content: `Verify that an implementation matches the change artifacts (specs, tasks, design).
|
|
178
|
+
|
|
179
|
+
${STORE_SELECTION_GUIDANCE}
|
|
180
|
+
|
|
181
|
+
**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.
|
|
182
|
+
|
|
183
|
+
**Steps**
|
|
184
|
+
|
|
185
|
+
1. **If no change name provided, prompt for selection**
|
|
186
|
+
|
|
187
|
+
Run \`openspec list --json\` to get available changes. Use the **AskUserQuestion tool** to let the user select.
|
|
188
|
+
|
|
189
|
+
Show changes that have implementation tasks (tasks artifact exists).
|
|
190
|
+
Include the schema used for each change if available.
|
|
191
|
+
Mark changes with incomplete tasks as "(In Progress)".
|
|
192
|
+
|
|
193
|
+
**IMPORTANT**: Do NOT guess or auto-select a change. Always let the user choose.
|
|
194
|
+
|
|
195
|
+
2. **Check status to understand the schema**
|
|
196
|
+
\`\`\`bash
|
|
197
|
+
openspec status --change "<name>" --json
|
|
198
|
+
\`\`\`
|
|
199
|
+
Parse the JSON to understand:
|
|
200
|
+
- \`schemaName\`: The workflow being used (e.g., "spec-driven")
|
|
201
|
+
- \`planningHome\`, \`changeRoot\`, \`artifactPaths\`, and \`actionContext\`: path and scope context
|
|
202
|
+
- Which artifacts exist for this change
|
|
203
|
+
|
|
204
|
+
3. **Get planning context and load artifacts**
|
|
205
|
+
|
|
206
|
+
\`\`\`bash
|
|
207
|
+
openspec instructions apply --change "<name>" --json
|
|
208
|
+
\`\`\`
|
|
209
|
+
|
|
210
|
+
This returns the change directory and \`contextFiles\` (artifact ID -> array of concrete file paths). Read all available artifacts from \`contextFiles\`.
|
|
211
|
+
|
|
212
|
+
4. **Initialize verification report structure**
|
|
213
|
+
|
|
214
|
+
Create a report structure with three dimensions:
|
|
215
|
+
- **Completeness**: Track tasks and spec coverage
|
|
216
|
+
- **Correctness**: Track requirement implementation and scenario coverage
|
|
217
|
+
- **Coherence**: Track design adherence and pattern consistency
|
|
218
|
+
|
|
219
|
+
Each dimension can have CRITICAL, WARNING, or SUGGESTION issues.
|
|
220
|
+
|
|
221
|
+
5. **Verify Completeness**
|
|
222
|
+
|
|
223
|
+
**Task Completion**:
|
|
224
|
+
- If \`contextFiles.tasks\` exists, read every file path in it
|
|
225
|
+
- Parse checkboxes: \`- [ ]\` (incomplete) vs \`- [x]\` (complete)
|
|
226
|
+
- Count complete vs total tasks
|
|
227
|
+
- If incomplete tasks exist:
|
|
228
|
+
- Add CRITICAL issue for each incomplete task
|
|
229
|
+
- Recommendation: "Complete task: <description>" or "Mark as done if already implemented"
|
|
230
|
+
|
|
231
|
+
**Spec Coverage**:
|
|
232
|
+
- If delta specs exist in \`contextFiles.specs\`:
|
|
233
|
+
- Extract all requirements (marked with "### Requirement:")
|
|
234
|
+
- For each requirement:
|
|
235
|
+
- Search codebase for keywords related to the requirement
|
|
236
|
+
- Assess if implementation likely exists
|
|
237
|
+
- If requirements appear unimplemented:
|
|
238
|
+
- Add CRITICAL issue: "Requirement not found: <requirement name>"
|
|
239
|
+
- Recommendation: "Implement requirement X: <description>"
|
|
240
|
+
|
|
241
|
+
6. **Verify Correctness**
|
|
242
|
+
|
|
243
|
+
**Requirement Implementation Mapping**:
|
|
244
|
+
- For each requirement from delta specs:
|
|
245
|
+
- Search codebase for implementation evidence
|
|
246
|
+
- If found, note file paths and line ranges
|
|
247
|
+
- Assess if implementation matches requirement intent
|
|
248
|
+
- If divergence detected:
|
|
249
|
+
- Add WARNING: "Implementation may diverge from spec: <details>"
|
|
250
|
+
- Recommendation: "Review <file>:<lines> against requirement X"
|
|
251
|
+
|
|
252
|
+
**Scenario Coverage**:
|
|
253
|
+
- For each scenario in delta specs (marked with "#### Scenario:"):
|
|
254
|
+
- Check if conditions are handled in code
|
|
255
|
+
- Check if tests exist covering the scenario
|
|
256
|
+
- If scenario appears uncovered:
|
|
257
|
+
- Add WARNING: "Scenario not covered: <scenario name>"
|
|
258
|
+
- Recommendation: "Add test or implementation for scenario: <description>"
|
|
259
|
+
|
|
260
|
+
7. **Verify Coherence**
|
|
261
|
+
|
|
262
|
+
**Design Adherence**:
|
|
263
|
+
- If \`contextFiles.design\` exists:
|
|
264
|
+
- Extract key decisions (look for sections like "Decision:", "Approach:", "Architecture:")
|
|
265
|
+
- Verify implementation follows those decisions
|
|
266
|
+
- If contradiction detected:
|
|
267
|
+
- Add WARNING: "Design decision not followed: <decision>"
|
|
268
|
+
- Recommendation: "Update implementation or revise design.md to match reality"
|
|
269
|
+
- If no design.md: Skip design adherence check, note "No design.md to verify against"
|
|
270
|
+
|
|
271
|
+
**Code Pattern Consistency**:
|
|
272
|
+
- Review new code for consistency with project patterns
|
|
273
|
+
- Check file naming, directory structure, coding style
|
|
274
|
+
- If significant deviations found:
|
|
275
|
+
- Add SUGGESTION: "Code pattern deviation: <details>"
|
|
276
|
+
- Recommendation: "Consider following project pattern: <example>"
|
|
277
|
+
|
|
278
|
+
8. **Generate Verification Report**
|
|
279
|
+
|
|
280
|
+
**Summary Scorecard**:
|
|
281
|
+
\`\`\`
|
|
282
|
+
## Verification Report: <change-name>
|
|
283
|
+
|
|
284
|
+
### Summary
|
|
285
|
+
| Dimension | Status |
|
|
286
|
+
|--------------|------------------|
|
|
287
|
+
| Completeness | X/Y tasks, N reqs|
|
|
288
|
+
| Correctness | M/N reqs covered |
|
|
289
|
+
| Coherence | Followed/Issues |
|
|
290
|
+
\`\`\`
|
|
291
|
+
|
|
292
|
+
**Issues by Priority**:
|
|
293
|
+
|
|
294
|
+
1. **CRITICAL** (Must fix before archive):
|
|
295
|
+
- Incomplete tasks
|
|
296
|
+
- Missing requirement implementations
|
|
297
|
+
- Each with specific, actionable recommendation
|
|
298
|
+
|
|
299
|
+
2. **WARNING** (Should fix):
|
|
300
|
+
- Spec/design divergences
|
|
301
|
+
- Missing scenario coverage
|
|
302
|
+
- Each with specific recommendation
|
|
303
|
+
|
|
304
|
+
3. **SUGGESTION** (Nice to fix):
|
|
305
|
+
- Pattern inconsistencies
|
|
306
|
+
- Minor improvements
|
|
307
|
+
- Each with specific recommendation
|
|
308
|
+
|
|
309
|
+
**Final Assessment**:
|
|
310
|
+
- If CRITICAL issues: "X critical issue(s) found. Fix before archiving."
|
|
311
|
+
- If only warnings: "No critical issues. Y warning(s) to consider. Ready for archive (with noted improvements)."
|
|
312
|
+
- If all clear: "All checks passed. Ready for archive."
|
|
313
|
+
|
|
314
|
+
**Verification Heuristics**
|
|
315
|
+
|
|
316
|
+
- **Completeness**: Focus on objective checklist items (checkboxes, requirements list)
|
|
317
|
+
- **Correctness**: Use keyword search, file path analysis, reasonable inference - don't require perfect certainty
|
|
318
|
+
- **Coherence**: Look for glaring inconsistencies, don't nitpick style
|
|
319
|
+
- **False Positives**: When uncertain, prefer SUGGESTION over WARNING, WARNING over CRITICAL
|
|
320
|
+
- **Actionability**: Every issue must have a specific recommendation with file/line references where applicable
|
|
321
|
+
|
|
322
|
+
**Graceful Degradation**
|
|
323
|
+
|
|
324
|
+
- If only tasks.md exists: verify task completion only, skip spec/design checks
|
|
325
|
+
- If tasks + specs exist: verify completeness and correctness, skip design
|
|
326
|
+
- If full artifacts: verify all three dimensions
|
|
327
|
+
- Always note which checks were skipped and why
|
|
328
|
+
|
|
329
|
+
**Output Format**
|
|
330
|
+
|
|
331
|
+
Use clear markdown with:
|
|
332
|
+
- Table for summary scorecard
|
|
333
|
+
- Grouped lists for issues (CRITICAL/WARNING/SUGGESTION)
|
|
334
|
+
- Code references in format: \`file.ts:123\`
|
|
335
|
+
- Specific, actionable recommendations
|
|
336
|
+
- No vague suggestions like "consider reviewing"`
|
|
337
|
+
};
|
|
338
|
+
}
|
|
339
|
+
//# sourceMappingURL=verify-change.js.map
|
|
@@ -0,0 +1,82 @@
|
|
|
1
|
+
/**
|
|
2
|
+
* Update Command
|
|
3
|
+
*
|
|
4
|
+
* Refreshes OpenSpec skills and commands for configured tools.
|
|
5
|
+
* Supports profile-aware updates, delivery changes, migration, and smart update detection.
|
|
6
|
+
*/
|
|
7
|
+
/**
|
|
8
|
+
* Options for the update command.
|
|
9
|
+
*/
|
|
10
|
+
export interface UpdateCommandOptions {
|
|
11
|
+
/** Force update even when tools are up to date */
|
|
12
|
+
force?: boolean;
|
|
13
|
+
}
|
|
14
|
+
/**
|
|
15
|
+
* Scans installed workflow artifacts (skills and managed commands) across all configured tools.
|
|
16
|
+
* Returns the union of detected workflow IDs that match ALL_WORKFLOWS.
|
|
17
|
+
*
|
|
18
|
+
* Wrapper around the shared migration module's scanInstalledWorkflows that accepts tool IDs.
|
|
19
|
+
*/
|
|
20
|
+
export declare function scanInstalledWorkflows(projectPath: string, toolIds: string[]): string[];
|
|
21
|
+
export declare class UpdateCommand {
|
|
22
|
+
private readonly force;
|
|
23
|
+
constructor(options?: UpdateCommandOptions);
|
|
24
|
+
execute(projectPath: string): Promise<void>;
|
|
25
|
+
/**
|
|
26
|
+
* Display message when all tools are up to date.
|
|
27
|
+
*/
|
|
28
|
+
private displayUpToDateMessage;
|
|
29
|
+
/**
|
|
30
|
+
* Display the update plan showing which tools need updating.
|
|
31
|
+
*/
|
|
32
|
+
private displayUpdatePlan;
|
|
33
|
+
/**
|
|
34
|
+
* Detects new tool directories that aren't currently configured and displays a hint.
|
|
35
|
+
*/
|
|
36
|
+
private detectNewTools;
|
|
37
|
+
/**
|
|
38
|
+
* Displays a note about extra workflows installed that aren't in the current profile.
|
|
39
|
+
*/
|
|
40
|
+
private displayExtraWorkflowsNote;
|
|
41
|
+
/**
|
|
42
|
+
* Suggest opting back into core when a custom profile still matches the old
|
|
43
|
+
* pre-sync core set. Keep custom profiles user-owned; do not mutate them.
|
|
44
|
+
*/
|
|
45
|
+
private displayOldCoreCustomProfileNote;
|
|
46
|
+
/**
|
|
47
|
+
* Removes skill directories for workflows when delivery changed to commands-only.
|
|
48
|
+
* Returns the number of directories removed.
|
|
49
|
+
*/
|
|
50
|
+
private removeSkillDirs;
|
|
51
|
+
/**
|
|
52
|
+
* Removes skill directories for workflows that are no longer selected in the active profile.
|
|
53
|
+
* Returns the number of directories removed.
|
|
54
|
+
*/
|
|
55
|
+
private removeUnselectedSkillDirs;
|
|
56
|
+
/**
|
|
57
|
+
* Removes command files for workflows when delivery changed to skills-only.
|
|
58
|
+
* Returns the number of files removed.
|
|
59
|
+
*/
|
|
60
|
+
private removeCommandFiles;
|
|
61
|
+
/**
|
|
62
|
+
* Removes command files for workflows that are no longer selected in the active profile.
|
|
63
|
+
* Returns the number of files removed.
|
|
64
|
+
*/
|
|
65
|
+
private removeUnselectedCommandFiles;
|
|
66
|
+
/**
|
|
67
|
+
* Detect and handle legacy OpenSpec artifacts.
|
|
68
|
+
* Unlike init, update warns but continues if legacy files found in non-interactive mode.
|
|
69
|
+
* Returns array of tool IDs that were newly configured during legacy upgrade.
|
|
70
|
+
*/
|
|
71
|
+
private handleLegacyCleanup;
|
|
72
|
+
/**
|
|
73
|
+
* Perform cleanup of legacy artifacts.
|
|
74
|
+
*/
|
|
75
|
+
private performLegacyCleanup;
|
|
76
|
+
/**
|
|
77
|
+
* Upgrade legacy tools to new skills system.
|
|
78
|
+
* Returns array of tool IDs that were newly configured.
|
|
79
|
+
*/
|
|
80
|
+
private upgradeLegacyTools;
|
|
81
|
+
}
|
|
82
|
+
//# sourceMappingURL=update.d.ts.map
|