@codewalla_india/openspec 1.2.0 → 1.3.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/README.md +49 -88
- package/dist/cli/index.js +8 -2
- package/dist/commands/workflow/index.d.ts +2 -2
- package/dist/commands/workflow/index.js +1 -1
- package/dist/commands/workflow/instructions.d.ts +21 -1
- package/dist/commands/workflow/instructions.js +124 -3
- package/dist/core/artifact-graph/graph.d.ts +9 -0
- package/dist/core/artifact-graph/graph.js +37 -7
- package/dist/core/change-status-policy.js +1 -1
- package/dist/core/completions/command-registry.js +20 -0
- package/dist/core/profile-sync-drift.js +1 -0
- package/dist/core/profiles.d.ts +2 -2
- package/dist/core/profiles.js +2 -1
- package/dist/core/shared/skill-generation.js +3 -1
- package/dist/core/shared/tool-detection.d.ts +2 -2
- package/dist/core/shared/tool-detection.js +2 -0
- package/dist/core/templates/skill-templates.d.ts +1 -0
- package/dist/core/templates/skill-templates.js +1 -0
- package/dist/core/templates/workflows/apply-change.js +18 -117
- package/dist/core/templates/workflows/comprehension-guidance.d.ts +1 -1
- package/dist/core/templates/workflows/comprehension-guidance.js +2 -0
- package/dist/core/templates/workflows/modify-change.d.ts +7 -0
- package/dist/core/templates/workflows/modify-change.js +132 -0
- package/dist/core/templates/workflows/user-prompt-guidance.d.ts +1 -0
- package/dist/core/templates/workflows/user-prompt-guidance.js +5 -0
- package/dist/telemetry/index.d.ts +1 -1
- package/dist/telemetry/index.js +1 -1
- package/dist/telemetry/marker.d.ts +6 -0
- package/dist/telemetry/workflow.d.ts +10 -0
- package/dist/telemetry/workflow.js +28 -0
- package/package.json +20 -18
package/README.md
CHANGED
|
@@ -1,46 +1,25 @@
|
|
|
1
1
|
<p align="center">
|
|
2
2
|
<a href="https://github.com/codewalla-engineering/OpenSpec">
|
|
3
3
|
<picture>
|
|
4
|
-
<source srcset="assets/
|
|
5
|
-
<img src="assets/
|
|
4
|
+
<source srcset="assets/codewalla_bg.png">
|
|
5
|
+
<img src="assets/codewalla_bg.png" alt="Codewalla OpenSpec" width="100%">
|
|
6
6
|
</picture>
|
|
7
7
|
</a>
|
|
8
8
|
</p>
|
|
9
9
|
|
|
10
|
+
<p align="center"><strong>Codewalla OpenSpec — spec-driven development for AI coding assistants</strong></p>
|
|
11
|
+
|
|
10
12
|
<p align="center">
|
|
11
13
|
<a href="https://github.com/codewalla-engineering/OpenSpec/actions/workflows/ci.yml"><img alt="CI" src="https://github.com/codewalla-engineering/OpenSpec/actions/workflows/ci.yml/badge.svg" /></a>
|
|
12
14
|
<a href="https://www.npmjs.com/package/@codewalla_india/openspec"><img alt="npm version" src="https://img.shields.io/npm/v/@codewalla_india/openspec?style=flat-square" /></a>
|
|
13
15
|
<a href="./LICENSE"><img alt="License: MIT" src="https://img.shields.io/badge/License-MIT-blue.svg?style=flat-square" /></a>
|
|
14
|
-
<a href="https://discord.gg/YctCnvvshC"><img alt="Discord" src="https://img.shields.io/discord/1411657095639601154?style=flat-square&logo=discord&logoColor=white&label=Discord&suffix=%20online" /></a>
|
|
15
16
|
</p>
|
|
16
17
|
|
|
17
|
-
<details>
|
|
18
|
-
<summary><strong>The most loved spec framework.</strong></summary>
|
|
19
|
-
|
|
20
|
-
[](https://github.com/codewalla-engineering/OpenSpec/stargazers)
|
|
21
|
-
[](https://www.npmjs.com/package/@codewalla_india/openspec)
|
|
22
|
-
[](https://github.com/codewalla-engineering/OpenSpec/graphs/contributors)
|
|
23
|
-
|
|
24
|
-
</details>
|
|
25
|
-
<p></p>
|
|
26
|
-
Our philosophy:
|
|
27
|
-
|
|
28
|
-
```text
|
|
29
|
-
→ fluid not rigid
|
|
30
|
-
→ iterative not waterfall
|
|
31
|
-
→ easy not complex
|
|
32
|
-
→ built for brownfield not just greenfield
|
|
33
|
-
→ scalable from personal projects to enterprises
|
|
34
|
-
```
|
|
35
|
-
|
|
36
18
|
> [!TIP]
|
|
37
|
-
> **
|
|
19
|
+
> **Codewalla workflow:** Propose from a Jira ticket or plain description, revise with `/opsx:modify`, then apply.
|
|
38
20
|
>
|
|
39
|
-
> Run `/opsx:propose "your idea"` to get started.
|
|
40
|
-
|
|
41
|
-
<p align="center">
|
|
42
|
-
Follow <a href="https://x.com/0xTab">@0xTab on X</a> for updates · Join the <a href="https://discord.gg/YctCnvvshC">OpenSpec Discord</a> for help and questions.
|
|
43
|
-
</p>
|
|
21
|
+
> Run `/opsx:propose CW-1234 add dark mode` or `/opsx:propose "your idea"` to get started.
|
|
22
|
+
> → [Jira integration](docs/workflows.md#jira-integration) · [MCP setup](docs/mcp-setup.md)
|
|
44
23
|
|
|
45
24
|
<!-- TODO: Add GIF demo of /opsx:propose → /opsx:archive workflow -->
|
|
46
25
|
|
|
@@ -55,17 +34,23 @@ AI: Let me look at your styling setup...
|
|
|
55
34
|
with system-preference detection. No new dependencies. Scope it?
|
|
56
35
|
You: Yes, let's do it.
|
|
57
36
|
|
|
58
|
-
You: /opsx:propose add-dark-mode
|
|
59
|
-
AI:
|
|
60
|
-
✓ proposal.md
|
|
61
|
-
✓ specs/
|
|
37
|
+
You: /opsx:propose CW-1234 add-dark-mode
|
|
38
|
+
AI: Found CW-1234: 'Add dark mode'. Creating change from Jira ticket...
|
|
39
|
+
✓ proposal.md (Impact: Jira: CW-1234)
|
|
40
|
+
✓ specs/ui/ (requirements from Jira ACs)
|
|
62
41
|
✓ design.md — technical approach
|
|
42
|
+
✓ plan.md — file-level code map
|
|
63
43
|
✓ tasks.md — implementation checklist
|
|
64
44
|
Ready for implementation!
|
|
65
45
|
|
|
46
|
+
You: /opsx:modify add-dark-mode design use CSS variables instead of hardcoded colors
|
|
47
|
+
AI: Updated design.md → propagated to plan.md and tasks.md
|
|
48
|
+
Run /opsx:apply when ready.
|
|
49
|
+
|
|
66
50
|
You: /opsx:apply
|
|
67
|
-
AI:
|
|
68
|
-
|
|
51
|
+
AI: Jira context: CW-1234 — all ACs covered ✓
|
|
52
|
+
Comprehension check — 7 questions on proposal, design, specs, plan, and tasks...
|
|
53
|
+
✓ 86% — ready to implement.
|
|
69
54
|
Implementing tasks...
|
|
70
55
|
✓ 1.1 Add theme context provider
|
|
71
56
|
✓ 1.2 Create toggle component
|
|
@@ -78,15 +63,6 @@ AI: Archived to openspec/changes/archive/2025-01-23-add-dark-mode/
|
|
|
78
63
|
Specs updated. Ready for the next feature.
|
|
79
64
|
```
|
|
80
65
|
|
|
81
|
-
<details>
|
|
82
|
-
<summary><strong>OpenSpec Dashboard</strong></summary>
|
|
83
|
-
|
|
84
|
-
<p align="center">
|
|
85
|
-
<img src="assets/openspec_dashboard.png" alt="OpenSpec dashboard preview" width="90%">
|
|
86
|
-
</p>
|
|
87
|
-
|
|
88
|
-
</details>
|
|
89
|
-
|
|
90
66
|
## Quick Start
|
|
91
67
|
|
|
92
68
|
**Requires Node.js 20.19.0 or higher.**
|
|
@@ -104,19 +80,36 @@ cd your-project
|
|
|
104
80
|
openspec init
|
|
105
81
|
```
|
|
106
82
|
|
|
83
|
+
> [!NOTE]
|
|
84
|
+
> **Codewalla identity:** `openspec init` prompts for your Codewalla email or username. All commands require telemetry identity. CI runners should pre-provision `~/.config/openspec/telemetry-identity.json` or set `OPENSPEC_TELEMETRY_USER`.
|
|
85
|
+
|
|
107
86
|
Now talk to your AI:
|
|
108
87
|
|
|
109
|
-
- **Not sure what to build yet?** Start with `/opsx:explore`, a no-stakes thinking partner that reads your code, weighs options, and shapes a plan before anything is written. ([Explore guide](docs/explore.md))
|
|
110
|
-
- **Already know what you want?** Go straight to `/opsx:propose <what-you-want-to-build
|
|
111
|
-
- **
|
|
88
|
+
- **Not sure what you want to build yet?** Start with `/opsx:explore`, a no-stakes thinking partner that reads your code, weighs options, and shapes a plan before anything is written. ([Explore guide](docs/explore.md))
|
|
89
|
+
- **Already know what you want?** Go straight to `/opsx:propose <what-you-want-to-build>` or `/opsx:propose CW-1234 <summary>` to import from Jira.
|
|
90
|
+
- **Need to revise the plan before coding?** Run `/opsx:modify` — pre-apply only; propagates changes to downstream artifacts. ([Editing a change](docs/editing-changes.md))
|
|
91
|
+
- **Ready to implement?** Run `/opsx:apply` — a short comprehension quiz checks you understand the proposal, design, specs, plan, and tasks before any code is written.
|
|
112
92
|
|
|
113
|
-
The default `core` profile includes `/opsx:explore`, `/opsx:propose`, `/opsx:apply`, `/opsx:sync`, and `/opsx:archive`. If you want the expanded workflow (`/opsx:new`, `/opsx:continue`, `/opsx:ff`, `/opsx:verify`, `/opsx:bulk-archive`, `/opsx:onboard`), select it with `openspec config profile` and apply with `openspec update`.
|
|
93
|
+
The default `core` profile includes `/opsx:explore`, `/opsx:propose`, `/opsx:modify`, `/opsx:apply`, `/opsx:sync`, and `/opsx:archive`. If you want the expanded workflow (`/opsx:new`, `/opsx:continue`, `/opsx:ff`, `/opsx:verify`, `/opsx:bulk-archive`, `/opsx:onboard`), select it with `openspec config profile` and apply with `openspec update`.
|
|
114
94
|
|
|
115
95
|
> [!NOTE]
|
|
116
96
|
> Not sure if your tool is supported? [View the full list](docs/supported-tools.md) – we support 25+ tools and growing.
|
|
117
97
|
>
|
|
118
98
|
> Also works with pnpm, yarn, bun, and nix. [See installation options](docs/installation.md).
|
|
119
99
|
|
|
100
|
+
## Codewalla workflow & MCPs
|
|
101
|
+
|
|
102
|
+
OpenSpec workflows integrate with MCP servers in your AI tool. MCPs are optional — workflows degrade gracefully when a server is unavailable.
|
|
103
|
+
|
|
104
|
+
| MCP | Workflow | Behavior |
|
|
105
|
+
|-----|----------|----------|
|
|
106
|
+
| **Atlassian** | `/opsx:propose`, `/opsx:apply` | Import Jira tickets; enrich and cross-check ACs vs tasks; skips if unavailable |
|
|
107
|
+
| **Context7** | `/opsx:apply` | Fetch current library docs when tasks reference packages; max 3 calls per session |
|
|
108
|
+
| **Browser** | `/opsx:verify` (expanded profile) | Screenshots, a11y snapshot, console errors; not run during apply |
|
|
109
|
+
|
|
110
|
+
→ **[MCP Setup guide](docs/mcp-setup.md)** — enable servers in Cursor or your AI tool<br>
|
|
111
|
+
→ **[Jira integration](docs/workflows.md#jira-integration)** — naming conventions and ticket flows
|
|
112
|
+
|
|
120
113
|
## Docs
|
|
121
114
|
|
|
122
115
|
**Start here:** the **[Documentation Home](docs/README.md)** maps everything. New to OpenSpec? Read [Getting Started](docs/getting-started.md), then [How Commands Work](docs/how-commands-work.md) (where you actually type `/opsx:propose`).
|
|
@@ -127,6 +120,7 @@ The default `core` profile includes `/opsx:explore`, `/opsx:propose`, `/opsx:app
|
|
|
127
120
|
→ **[Core Concepts at a Glance](docs/overview.md)**: the whole mental model, one page<br>
|
|
128
121
|
→ **[Examples & Recipes](docs/examples.md)**: real changes, start to finish<br>
|
|
129
122
|
→ **[Workflows](docs/workflows.md)**: combos and patterns<br>
|
|
123
|
+
→ **[MCP Setup](docs/mcp-setup.md)**: Atlassian, Context7, and browser MCPs<br>
|
|
130
124
|
→ **[Existing Projects](docs/existing-projects.md)**: adopt OpenSpec on a brownfield codebase<br>
|
|
131
125
|
→ **[Editing a Change](docs/editing-changes.md)**: update artifacts, go back, reconcile manual edits<br>
|
|
132
126
|
→ **[Commands](docs/commands.md)**: slash commands & skills<br>
|
|
@@ -151,18 +145,12 @@ Third-party schema bundles distributed via standalone repositories — these pro
|
|
|
151
145
|
AI coding assistants are powerful but unpredictable when requirements live only in chat history. OpenSpec adds a lightweight spec layer so you agree on what to build before any code is written.
|
|
152
146
|
|
|
153
147
|
- **Agree before you build** — human and AI align on specs before code gets written
|
|
154
|
-
- **Stay organized** — each change gets its own folder with proposal, specs, design, and tasks
|
|
155
|
-
- **Work fluidly** — update any artifact anytime
|
|
148
|
+
- **Stay organized** — each change gets its own folder with proposal, specs, design, plan, and tasks
|
|
149
|
+
- **Work fluidly** — update any artifact anytime with `/opsx:modify`; no rigid phase gates
|
|
150
|
+
- **Jira-native workflow** — propose from ticket keys, enrich from Jira during apply, traceability in proposal Impact
|
|
151
|
+
- **MCP-powered** — Atlassian, Context7, and browser MCPs built into generated skills
|
|
156
152
|
- **Use your tools** — works with 20+ AI assistants via slash commands
|
|
157
153
|
|
|
158
|
-
### How we compare
|
|
159
|
-
|
|
160
|
-
**vs. [Spec Kit](https://github.com/github/spec-kit)** (GitHub) — Thorough but heavyweight. Rigid phase gates, lots of Markdown, Python setup. OpenSpec is lighter and lets you iterate freely.
|
|
161
|
-
|
|
162
|
-
**vs. [Kiro](https://kiro.dev)** (AWS) — Powerful but you're locked into their IDE and limited to Claude models. OpenSpec works with the tools you already use.
|
|
163
|
-
|
|
164
|
-
**vs. nothing** — AI coding without specs means vague prompts and unpredictable results. OpenSpec brings predictability without the ceremony.
|
|
165
|
-
|
|
166
154
|
## Updating OpenSpec
|
|
167
155
|
|
|
168
156
|
**Upgrade the package**
|
|
@@ -173,7 +161,7 @@ npm install -g @codewalla_india/openspec@latest
|
|
|
173
161
|
|
|
174
162
|
**Refresh agent instructions**
|
|
175
163
|
|
|
176
|
-
Run this inside each project to regenerate AI guidance and ensure the latest slash commands are active:
|
|
164
|
+
Run this inside each project to regenerate AI guidance and ensure the latest slash commands and MCP guidance are active:
|
|
177
165
|
|
|
178
166
|
```bash
|
|
179
167
|
openspec update
|
|
@@ -181,29 +169,11 @@ openspec update
|
|
|
181
169
|
|
|
182
170
|
## Usage Notes
|
|
183
171
|
|
|
184
|
-
**Model selection**: OpenSpec works best with high-reasoning models. We recommend Codex 5.5 and Opus 4.7 for both planning and implementation.
|
|
185
|
-
|
|
186
172
|
**Context hygiene**: OpenSpec benefits from a clean context window. Clear your context before starting implementation and maintain good context hygiene throughout your session.
|
|
187
173
|
|
|
188
|
-
**
|
|
189
|
-
|
|
190
|
-
## Contributing
|
|
191
|
-
|
|
192
|
-
**Small fixes** — Bug fixes, typo corrections, and minor improvements can be submitted directly as PRs.
|
|
174
|
+
**Modify before apply**: `/opsx:modify` revises planning artifacts and propagates changes downstream. It is pre-apply only — once tasks are checked off in `/opsx:apply`, use manual edits or start a new change.
|
|
193
175
|
|
|
194
|
-
**
|
|
195
|
-
|
|
196
|
-
When writing proposals, keep the OpenSpec philosophy in mind: we serve a wide variety of users across different coding agents, models, and use cases. Changes should work well for everyone.
|
|
197
|
-
|
|
198
|
-
**AI-generated code is welcome** — as long as it's been tested and verified. PRs containing AI-generated code should mention the coding agent and model used (e.g., "Generated with Claude Code using claude-opus-4-5-20251101").
|
|
199
|
-
|
|
200
|
-
### Development
|
|
201
|
-
|
|
202
|
-
- Install dependencies: `pnpm install`
|
|
203
|
-
- Build: `pnpm run build`
|
|
204
|
-
- Test: `pnpm test`
|
|
205
|
-
- Develop CLI locally: `pnpm run dev` or `pnpm run dev:cli`
|
|
206
|
-
- Conventional commits (one-line): `type(scope): subject`
|
|
176
|
+
**Comprehension check**: `/opsx:apply` runs a short quiz (enabled by default) on proposal, design, specs, plan, and pending tasks before implementation. Questions test holistic understanding of the change, not task numbers or checklist trivia; plan receives the majority of questions when present. You need ≥ 80% to proceed. Disable with `comprehension.enabled: false` in `openspec/config.yaml`. See [Workflows](docs/workflows.md#comprehension-quiz-before-apply).
|
|
207
177
|
|
|
208
178
|
## Other
|
|
209
179
|
|
|
@@ -212,19 +182,10 @@ When writing proposals, keep the OpenSpec philosophy in mind: we serve a wide va
|
|
|
212
182
|
|
|
213
183
|
Codewalla OpenSpec collects mandatory usage analytics tied to your email or username. Identity is collected during interactive `openspec init` or `openspec update` and stored at `~/.config/openspec/telemetry-identity.json` (never committed). All other commands require identity. CI runners should pre-provision that file or set `OPENSPEC_TELEMETRY_USER`.
|
|
214
184
|
|
|
215
|
-
Events include command names, workflow metrics, change names,
|
|
185
|
+
Events include command names, workflow metrics, change names, workflow input text (via `--workflow-input` on `new change`), and modify requests (`artifact_modify_requested` with modify input) — not file paths or artifact/spec body content.
|
|
216
186
|
|
|
217
187
|
</details>
|
|
218
188
|
|
|
219
|
-
<details>
|
|
220
|
-
<summary><strong>Maintainers & Advisors</strong></summary>
|
|
221
|
-
|
|
222
|
-
See [MAINTAINERS.md](MAINTAINERS.md) for the list of core maintainers and advisors who help guide the project.
|
|
223
|
-
|
|
224
|
-
</details>
|
|
225
|
-
|
|
226
|
-
|
|
227
|
-
|
|
228
189
|
## License
|
|
229
190
|
|
|
230
191
|
MIT
|
package/dist/cli/index.js
CHANGED
|
@@ -23,7 +23,7 @@ import { registerStoreCommand } from '../commands/store.js';
|
|
|
23
23
|
import { registerDoctorCommand } from '../commands/doctor.js';
|
|
24
24
|
import { registerContextCommand } from '../commands/context.js';
|
|
25
25
|
import { registerWorksetCommand } from '../commands/workset.js';
|
|
26
|
-
import { statusCommand, instructionsCommand, applyInstructionsCommand, templatesCommand, schemasCommand, newChangeCommand, DEFAULT_SCHEMA, } from '../commands/workflow/index.js';
|
|
26
|
+
import { statusCommand, instructionsCommand, applyInstructionsCommand, modifyInstructionsCommand, templatesCommand, schemasCommand, newChangeCommand, DEFAULT_SCHEMA, } from '../commands/workflow/index.js';
|
|
27
27
|
import { requireTelemetryIdentity, TelemetryIdentityRequiredError, trackCommand, shutdown } from '../telemetry/index.js';
|
|
28
28
|
import { buildCommandTelemetryContext, resolveTelemetryCommandPath, } from '../telemetry/command-context.js';
|
|
29
29
|
import { COMMON_FLAGS } from '../core/completions/shared-flags.js';
|
|
@@ -505,14 +505,20 @@ program
|
|
|
505
505
|
.option('--score <percent>', 'Quiz score 0-100 (required with --record-comprehension-pass)', parseInt)
|
|
506
506
|
.option('--attempt <n>', 'Quiz attempt number', parseInt)
|
|
507
507
|
.option('--question-count <n>', 'Number of quiz questions taken', parseInt)
|
|
508
|
+
.option('--artifact <id>', 'Source artifact to modify (use with: instructions modify)')
|
|
509
|
+
.option('--workflow-input <text>', 'User modify request for telemetry (use with: instructions modify)')
|
|
510
|
+
.option('--workflow-input-file <path>', 'Read modify request from a file (use with: instructions modify)')
|
|
511
|
+
.option('--editor <tool>', 'AI editor used (cursor, windsurf, claude)')
|
|
508
512
|
.option('--store <id>', STORE_OPTION_DESCRIPTION)
|
|
509
513
|
.addOption(hiddenStorePathOption())
|
|
510
514
|
.action(async (artifactId, options) => {
|
|
511
515
|
try {
|
|
512
|
-
// Special case: "apply" is not an artifact, but a command to get apply instructions
|
|
513
516
|
if (artifactId === 'apply') {
|
|
514
517
|
await applyInstructionsCommand(options);
|
|
515
518
|
}
|
|
519
|
+
else if (artifactId === 'modify') {
|
|
520
|
+
await modifyInstructionsCommand(options);
|
|
521
|
+
}
|
|
516
522
|
else {
|
|
517
523
|
await instructionsCommand(artifactId, options);
|
|
518
524
|
}
|
|
@@ -5,8 +5,8 @@
|
|
|
5
5
|
*/
|
|
6
6
|
export { statusCommand } from './status.js';
|
|
7
7
|
export type { StatusOptions } from './status.js';
|
|
8
|
-
export { instructionsCommand, applyInstructionsCommand } from './instructions.js';
|
|
9
|
-
export type { InstructionsOptions } from './instructions.js';
|
|
8
|
+
export { instructionsCommand, applyInstructionsCommand, modifyInstructionsCommand } from './instructions.js';
|
|
9
|
+
export type { InstructionsOptions, ModifyInstructionsOptions } from './instructions.js';
|
|
10
10
|
export { templatesCommand } from './templates.js';
|
|
11
11
|
export type { TemplatesOptions } from './templates.js';
|
|
12
12
|
export { schemasCommand } from './schemas.js';
|
|
@@ -4,7 +4,7 @@
|
|
|
4
4
|
* Commands for the artifact-driven workflow: status, instructions, templates, schemas, new change.
|
|
5
5
|
*/
|
|
6
6
|
export { statusCommand } from './status.js';
|
|
7
|
-
export { instructionsCommand, applyInstructionsCommand } from './instructions.js';
|
|
7
|
+
export { instructionsCommand, applyInstructionsCommand, modifyInstructionsCommand } from './instructions.js';
|
|
8
8
|
export { templatesCommand } from './templates.js';
|
|
9
9
|
export { schemasCommand } from './schemas.js';
|
|
10
10
|
export { newChangeCommand } from './new-change.js';
|
|
@@ -4,7 +4,7 @@
|
|
|
4
4
|
* Generates enriched instructions for creating artifacts or applying tasks.
|
|
5
5
|
* Includes both artifact instructions and apply instructions.
|
|
6
6
|
*/
|
|
7
|
-
import { type ArtifactInstructions } from '../../core/artifact-graph/index.js';
|
|
7
|
+
import { type ArtifactInstructions, type ArtifactPathSummary } from '../../core/artifact-graph/index.js';
|
|
8
8
|
import { type PlanningHome } from '../../core/planning-home.js';
|
|
9
9
|
import { type ReferenceIndexEntry } from '../../core/references.js';
|
|
10
10
|
import { type ProjectConfig } from '../../core/project-config.js';
|
|
@@ -27,6 +27,24 @@ export interface ApplyInstructionsOptions {
|
|
|
27
27
|
attempt?: number;
|
|
28
28
|
questionCount?: number;
|
|
29
29
|
}
|
|
30
|
+
export interface ModifyInstructionsOptions extends InstructionsOptions {
|
|
31
|
+
artifact?: string;
|
|
32
|
+
workflowInput?: string;
|
|
33
|
+
workflowInputFile?: string;
|
|
34
|
+
editor?: string;
|
|
35
|
+
}
|
|
36
|
+
export interface ModifyInstructions {
|
|
37
|
+
changeName: string;
|
|
38
|
+
schemaName: string;
|
|
39
|
+
sourceArtifact: string;
|
|
40
|
+
modifyInput?: string;
|
|
41
|
+
downstreamArtifacts: string[];
|
|
42
|
+
artifactsToUpdate: string[];
|
|
43
|
+
changeRoot: string;
|
|
44
|
+
artifactPaths: Record<string, ArtifactPathSummary>;
|
|
45
|
+
phase: 'pre_apply';
|
|
46
|
+
instruction: string;
|
|
47
|
+
}
|
|
30
48
|
export declare function instructionsCommand(artifactId: string | undefined, options: InstructionsOptions): Promise<void>;
|
|
31
49
|
export declare function printInstructionsText(instructions: ArtifactInstructions, isBlocked: boolean): void;
|
|
32
50
|
export interface GenerateApplyInstructionsOptions {
|
|
@@ -42,4 +60,6 @@ export interface GenerateApplyInstructionsOptions {
|
|
|
42
60
|
export declare function generateApplyInstructions(projectRoot: string, changeName: string, schemaName?: string, options?: GenerateApplyInstructionsOptions): Promise<ApplyInstructions>;
|
|
43
61
|
export declare function applyInstructionsCommand(options: ApplyInstructionsOptions): Promise<void>;
|
|
44
62
|
export declare function printApplyInstructionsText(instructions: ApplyInstructions): void;
|
|
63
|
+
export declare function modifyInstructionsCommand(options: ModifyInstructionsOptions): Promise<void>;
|
|
64
|
+
export declare function printModifyInstructionsText(instructions: ModifyInstructions): void;
|
|
45
65
|
//# sourceMappingURL=instructions.d.ts.map
|
|
@@ -7,14 +7,15 @@
|
|
|
7
7
|
import ora from 'ora';
|
|
8
8
|
import path from 'path';
|
|
9
9
|
import * as fs from 'fs';
|
|
10
|
-
import { loadChangeContext, generateInstructions, resolveSchema, resolveArtifactOutputs, } from '../../core/artifact-graph/index.js';
|
|
10
|
+
import { loadChangeContext, generateInstructions, formatChangeStatus, resolveSchema, resolveArtifactOutputs, } from '../../core/artifact-graph/index.js';
|
|
11
11
|
import { getChangeDir, resolveCurrentPlanningHomeSync, } from '../../core/planning-home.js';
|
|
12
|
-
import { resolveRootForCommand, withStoreFlag, toPlanningHome, toRootOutput, } from '../../core/root-selection.js';
|
|
12
|
+
import { resolveRootForCommand, withStoreFlag, toPlanningHome, toRootOutput, isStoreSelectedRoot, } from '../../core/root-selection.js';
|
|
13
13
|
import { assembleReferenceIndex, renderReferencedStoresBlock, renderReferencedStoresSection, } from '../../core/references.js';
|
|
14
14
|
import { readRegistrySnapshot } from '../../core/store/registry.js';
|
|
15
15
|
import { readProjectConfig } from '../../core/project-config.js';
|
|
16
16
|
import { checkComprehensionGate, ComprehensionPassError, computeSpecStats, recordComprehensionPass, resolveComprehensionConfig, } from '../../core/comprehension/index.js';
|
|
17
|
-
import { maybeEmitProposalReady, maybeEmitApplyReady, trackArtifactInstructions, trackArtifactContentChanges, incrementComprehensionAttempt, incrementComprehensionFailureCount, trackComprehensionAttempt, trackComprehensionGateChecked, trackComprehensionRetakeRequired, } from '../../telemetry/index.js';
|
|
17
|
+
import { maybeEmitProposalReady, maybeEmitApplyReady, trackArtifactInstructions, trackArtifactContentChanges, trackArtifactModifyRequested, incrementComprehensionAttempt, incrementComprehensionFailureCount, trackComprehensionAttempt, trackComprehensionGateChecked, trackComprehensionRetakeRequired, } from '../../telemetry/index.js';
|
|
18
|
+
import { normalizeEditor, resolveWorkflowInputAsync, } from '../../telemetry/input.js';
|
|
18
19
|
import { validateChangeExists, validateSchemaExists, } from './shared.js';
|
|
19
20
|
function buildArtifactPresence(contextFiles, pendingTaskCount) {
|
|
20
21
|
return {
|
|
@@ -634,4 +635,124 @@ export function printApplyInstructionsText(instructions) {
|
|
|
634
635
|
console.log('### Instruction');
|
|
635
636
|
console.log(instruction);
|
|
636
637
|
}
|
|
638
|
+
// -----------------------------------------------------------------------------
|
|
639
|
+
// Modify Instructions Command (pre-apply artifact revision)
|
|
640
|
+
// -----------------------------------------------------------------------------
|
|
641
|
+
function buildModifyInstruction(sourceArtifact, downstreamArtifacts, modifyInput) {
|
|
642
|
+
const downstreamLine = downstreamArtifacts.length > 0
|
|
643
|
+
? `Downstream artifacts to refresh: ${downstreamArtifacts.join(', ')}.`
|
|
644
|
+
: 'No downstream artifacts depend on this source.';
|
|
645
|
+
const requestLine = modifyInput
|
|
646
|
+
? `User modify request: ${modifyInput}`
|
|
647
|
+
: 'Apply the user modify request from the conversation.';
|
|
648
|
+
return [
|
|
649
|
+
`Revise the "${sourceArtifact}" artifact for this change (pre-apply only).`,
|
|
650
|
+
requestLine,
|
|
651
|
+
downstreamLine,
|
|
652
|
+
'For each artifact in artifactsToUpdate (in order): run openspec instructions <id> --change <name> --json, read upstream dependencies, apply a surgical edit aligned with the modify request, and write to resolvedOutputPath.',
|
|
653
|
+
'When proposal capabilities change, add/remove/rename specs under specs/ accordingly.',
|
|
654
|
+
'After all updates, run openspec status --change <name> --json, then hand off to /opsx:apply.',
|
|
655
|
+
].join('\n');
|
|
656
|
+
}
|
|
657
|
+
export async function modifyInstructionsCommand(options) {
|
|
658
|
+
const root = await resolveRootForCommand(options, { json: options.json });
|
|
659
|
+
if (!root) {
|
|
660
|
+
return;
|
|
661
|
+
}
|
|
662
|
+
const spinner = options.json ? undefined : ora('Generating modify instructions...').start();
|
|
663
|
+
try {
|
|
664
|
+
const planningHome = toPlanningHome(root);
|
|
665
|
+
const projectRoot = root.path;
|
|
666
|
+
const changeName = await validateChangeExists(options.change, projectRoot, root.changesDir, { newChangeHint: withStoreFlag(root, 'openspec new change <name>') });
|
|
667
|
+
if (options.schema) {
|
|
668
|
+
validateSchemaExists(options.schema, projectRoot);
|
|
669
|
+
}
|
|
670
|
+
const sourceArtifact = options.artifact?.trim();
|
|
671
|
+
if (!sourceArtifact) {
|
|
672
|
+
spinner?.stop();
|
|
673
|
+
throw new Error('Missing required option --artifact. Specify the artifact to modify (e.g., design, proposal, tasks).');
|
|
674
|
+
}
|
|
675
|
+
const modifyInput = await resolveWorkflowInputAsync({
|
|
676
|
+
workflowInput: options.workflowInput,
|
|
677
|
+
workflowInputFile: options.workflowInputFile,
|
|
678
|
+
});
|
|
679
|
+
const editor = normalizeEditor(options.editor);
|
|
680
|
+
const context = loadChangeContext(projectRoot, changeName, options.schema, {
|
|
681
|
+
changeDir: getChangeDir(planningHome, changeName),
|
|
682
|
+
planningHome,
|
|
683
|
+
});
|
|
684
|
+
const artifact = context.graph.getArtifact(sourceArtifact);
|
|
685
|
+
if (!artifact) {
|
|
686
|
+
spinner?.stop();
|
|
687
|
+
const validIds = context.graph.getAllArtifacts().map((a) => a.id);
|
|
688
|
+
throw new Error(`Artifact '${sourceArtifact}' not found in schema '${context.schemaName}'. Valid artifacts:\n ${validIds.join('\n ')}`);
|
|
689
|
+
}
|
|
690
|
+
const applyInstructions = await generateApplyInstructions(projectRoot, changeName, options.schema, {
|
|
691
|
+
planningHome,
|
|
692
|
+
});
|
|
693
|
+
if (applyInstructions.progress.complete > 0) {
|
|
694
|
+
spinner?.stop();
|
|
695
|
+
throw new Error(`Apply has already started (${applyInstructions.progress.complete}/${applyInstructions.progress.total} tasks complete). /opsx:modify is pre-apply only. Edit artifacts manually or start a new change.`);
|
|
696
|
+
}
|
|
697
|
+
if (applyInstructions.missingArtifacts && applyInstructions.missingArtifacts.length > 0) {
|
|
698
|
+
spinner?.stop();
|
|
699
|
+
throw new Error(`Change is not ready to modify. Missing artifacts: ${applyInstructions.missingArtifacts.join(', ')}. Use /opsx:continue or /opsx:propose to create them first.`);
|
|
700
|
+
}
|
|
701
|
+
const sourceOutputs = resolveArtifactOutputs(context.changeDir, artifact.generates);
|
|
702
|
+
if (sourceOutputs.length === 0) {
|
|
703
|
+
spinner?.stop();
|
|
704
|
+
throw new Error(`Artifact '${sourceArtifact}' does not exist yet. Use /opsx:continue to create it first.`);
|
|
705
|
+
}
|
|
706
|
+
const downstreamArtifacts = context.graph.getTransitiveDependents(sourceArtifact);
|
|
707
|
+
const artifactsToUpdate = [sourceArtifact, ...downstreamArtifacts];
|
|
708
|
+
const status = formatChangeStatus(context, isStoreSelectedRoot(root) ? { storeId: root.storeId } : {});
|
|
709
|
+
await trackArtifactModifyRequested({
|
|
710
|
+
changeDir: context.changeDir,
|
|
711
|
+
changeName,
|
|
712
|
+
schema: context.schemaName,
|
|
713
|
+
sourceArtifactId: sourceArtifact,
|
|
714
|
+
downstreamArtifactIds: downstreamArtifacts,
|
|
715
|
+
artifactsToUpdate,
|
|
716
|
+
modifyInput,
|
|
717
|
+
editor,
|
|
718
|
+
});
|
|
719
|
+
const payload = {
|
|
720
|
+
changeName,
|
|
721
|
+
schemaName: context.schemaName,
|
|
722
|
+
sourceArtifact,
|
|
723
|
+
...(modifyInput ? { modifyInput } : {}),
|
|
724
|
+
downstreamArtifacts,
|
|
725
|
+
artifactsToUpdate,
|
|
726
|
+
changeRoot: context.changeDir,
|
|
727
|
+
artifactPaths: status.artifactPaths,
|
|
728
|
+
phase: 'pre_apply',
|
|
729
|
+
instruction: buildModifyInstruction(sourceArtifact, downstreamArtifacts, modifyInput),
|
|
730
|
+
};
|
|
731
|
+
spinner?.stop();
|
|
732
|
+
if (options.json) {
|
|
733
|
+
console.log(JSON.stringify({ ...payload, root: toRootOutput(root) }, null, 2));
|
|
734
|
+
return;
|
|
735
|
+
}
|
|
736
|
+
printModifyInstructionsText(payload);
|
|
737
|
+
}
|
|
738
|
+
catch (error) {
|
|
739
|
+
spinner?.stop();
|
|
740
|
+
throw error;
|
|
741
|
+
}
|
|
742
|
+
}
|
|
743
|
+
export function printModifyInstructionsText(instructions) {
|
|
744
|
+
console.log(`## Modify: ${instructions.changeName}`);
|
|
745
|
+
console.log(`Schema: ${instructions.schemaName}`);
|
|
746
|
+
console.log(`Phase: ${instructions.phase}`);
|
|
747
|
+
console.log();
|
|
748
|
+
console.log(`Source artifact: ${instructions.sourceArtifact}`);
|
|
749
|
+
if (instructions.modifyInput) {
|
|
750
|
+
console.log(`Modify request: ${instructions.modifyInput}`);
|
|
751
|
+
}
|
|
752
|
+
console.log(`Downstream: ${instructions.downstreamArtifacts.length > 0 ? instructions.downstreamArtifacts.join(', ') : '(none)'}`);
|
|
753
|
+
console.log(`Artifacts to update: ${instructions.artifactsToUpdate.join(', ')}`);
|
|
754
|
+
console.log();
|
|
755
|
+
console.log('### Instruction');
|
|
756
|
+
console.log(instructions.instruction);
|
|
757
|
+
}
|
|
637
758
|
//# sourceMappingURL=instructions.js.map
|
|
@@ -35,6 +35,15 @@ export declare class ArtifactGraph {
|
|
|
35
35
|
* Gets the schema version.
|
|
36
36
|
*/
|
|
37
37
|
getVersion(): number;
|
|
38
|
+
/**
|
|
39
|
+
* Builds reverse adjacency: artifact ID -> IDs of artifacts that depend on it.
|
|
40
|
+
*/
|
|
41
|
+
private buildDependentsMap;
|
|
42
|
+
/**
|
|
43
|
+
* Returns all transitive downstream artifact IDs that depend on the given artifact,
|
|
44
|
+
* sorted by build order (excludes the source artifact itself).
|
|
45
|
+
*/
|
|
46
|
+
getTransitiveDependents(artifactId: string): string[];
|
|
38
47
|
/**
|
|
39
48
|
* Computes the topological build order using Kahn's algorithm.
|
|
40
49
|
* Returns artifact IDs in the order they should be built.
|
|
@@ -55,23 +55,53 @@ export class ArtifactGraph {
|
|
|
55
55
|
return this.schema.version;
|
|
56
56
|
}
|
|
57
57
|
/**
|
|
58
|
-
*
|
|
59
|
-
* Returns artifact IDs in the order they should be built.
|
|
58
|
+
* Builds reverse adjacency: artifact ID -> IDs of artifacts that depend on it.
|
|
60
59
|
*/
|
|
61
|
-
|
|
62
|
-
const inDegree = new Map();
|
|
60
|
+
buildDependentsMap() {
|
|
63
61
|
const dependents = new Map();
|
|
64
|
-
// Initialize all artifacts
|
|
65
62
|
for (const artifact of this.artifacts.values()) {
|
|
66
|
-
inDegree.set(artifact.id, artifact.requires.length);
|
|
67
63
|
dependents.set(artifact.id, []);
|
|
68
64
|
}
|
|
69
|
-
// Build reverse adjacency (who depends on whom)
|
|
70
65
|
for (const artifact of this.artifacts.values()) {
|
|
71
66
|
for (const req of artifact.requires) {
|
|
72
67
|
dependents.get(req).push(artifact.id);
|
|
73
68
|
}
|
|
74
69
|
}
|
|
70
|
+
return dependents;
|
|
71
|
+
}
|
|
72
|
+
/**
|
|
73
|
+
* Returns all transitive downstream artifact IDs that depend on the given artifact,
|
|
74
|
+
* sorted by build order (excludes the source artifact itself).
|
|
75
|
+
*/
|
|
76
|
+
getTransitiveDependents(artifactId) {
|
|
77
|
+
if (!this.artifacts.has(artifactId)) {
|
|
78
|
+
return [];
|
|
79
|
+
}
|
|
80
|
+
const dependents = this.buildDependentsMap();
|
|
81
|
+
const collected = new Set();
|
|
82
|
+
const queue = [artifactId];
|
|
83
|
+
while (queue.length > 0) {
|
|
84
|
+
const current = queue.shift();
|
|
85
|
+
for (const dependentId of dependents.get(current) ?? []) {
|
|
86
|
+
if (!collected.has(dependentId)) {
|
|
87
|
+
collected.add(dependentId);
|
|
88
|
+
queue.push(dependentId);
|
|
89
|
+
}
|
|
90
|
+
}
|
|
91
|
+
}
|
|
92
|
+
return this.getBuildOrder().filter((id) => collected.has(id));
|
|
93
|
+
}
|
|
94
|
+
/**
|
|
95
|
+
* Computes the topological build order using Kahn's algorithm.
|
|
96
|
+
* Returns artifact IDs in the order they should be built.
|
|
97
|
+
*/
|
|
98
|
+
getBuildOrder() {
|
|
99
|
+
const inDegree = new Map();
|
|
100
|
+
const dependents = this.buildDependentsMap();
|
|
101
|
+
// Initialize in-degrees
|
|
102
|
+
for (const artifact of this.artifacts.values()) {
|
|
103
|
+
inDegree.set(artifact.id, artifact.requires.length);
|
|
104
|
+
}
|
|
75
105
|
// Start with roots (in-degree 0), sorted for determinism
|
|
76
106
|
const queue = [...this.artifacts.keys()]
|
|
77
107
|
.filter(id => inDegree.get(id) === 0)
|
|
@@ -28,7 +28,7 @@ export function buildNextSteps(input) {
|
|
|
28
28
|
steps.push(`Run openspec instructions ${readyArtifact.id} --change "${input.changeName}"${storeFlag} --json before writing that artifact.`);
|
|
29
29
|
}
|
|
30
30
|
else if (input.allArtifactsComplete) {
|
|
31
|
-
steps.push('
|
|
31
|
+
steps.push('Review planning artifacts; use /opsx:modify to revise or /opsx:apply to implement.');
|
|
32
32
|
}
|
|
33
33
|
return steps;
|
|
34
34
|
}
|
|
@@ -210,6 +210,26 @@ export const COMMAND_REGISTRY = [
|
|
|
210
210
|
description: 'Number of quiz questions taken',
|
|
211
211
|
takesValue: true,
|
|
212
212
|
},
|
|
213
|
+
{
|
|
214
|
+
name: 'artifact',
|
|
215
|
+
description: 'Source artifact to modify (use with: instructions modify)',
|
|
216
|
+
takesValue: true,
|
|
217
|
+
},
|
|
218
|
+
{
|
|
219
|
+
name: 'workflow-input',
|
|
220
|
+
description: 'User modify request for telemetry (use with: instructions modify)',
|
|
221
|
+
takesValue: true,
|
|
222
|
+
},
|
|
223
|
+
{
|
|
224
|
+
name: 'workflow-input-file',
|
|
225
|
+
description: 'Read modify request from a file (use with: instructions modify)',
|
|
226
|
+
takesValue: true,
|
|
227
|
+
},
|
|
228
|
+
{
|
|
229
|
+
name: 'editor',
|
|
230
|
+
description: 'AI editor used (cursor, windsurf, claude)',
|
|
231
|
+
takesValue: true,
|
|
232
|
+
},
|
|
213
233
|
COMMON_FLAGS.json,
|
|
214
234
|
COMMON_FLAGS.store,
|
|
215
235
|
],
|
|
@@ -11,6 +11,7 @@ export const WORKFLOW_TO_SKILL_DIR = {
|
|
|
11
11
|
'explore': 'openspec-explore',
|
|
12
12
|
'new': 'openspec-new-change',
|
|
13
13
|
'continue': 'openspec-continue-change',
|
|
14
|
+
'modify': 'openspec-modify-change',
|
|
14
15
|
'apply': 'openspec-apply-change',
|
|
15
16
|
'ff': 'openspec-ff-change',
|
|
16
17
|
'sync': 'openspec-sync-specs',
|
package/dist/core/profiles.d.ts
CHANGED
|
@@ -9,11 +9,11 @@ import type { Profile } from './global-config.js';
|
|
|
9
9
|
* Core workflows included in the 'core' profile.
|
|
10
10
|
* These provide the streamlined experience for new users.
|
|
11
11
|
*/
|
|
12
|
-
export declare const CORE_WORKFLOWS: readonly ["propose", "explore", "apply", "sync", "archive"];
|
|
12
|
+
export declare const CORE_WORKFLOWS: readonly ["propose", "explore", "modify", "apply", "sync", "archive"];
|
|
13
13
|
/**
|
|
14
14
|
* All available workflows in the system.
|
|
15
15
|
*/
|
|
16
|
-
export declare const ALL_WORKFLOWS: readonly ["propose", "explore", "new", "continue", "apply", "ff", "sync", "archive", "bulk-archive", "verify", "onboard"];
|
|
16
|
+
export declare const ALL_WORKFLOWS: readonly ["propose", "explore", "modify", "new", "continue", "apply", "ff", "sync", "archive", "bulk-archive", "verify", "onboard"];
|
|
17
17
|
export type WorkflowId = (typeof ALL_WORKFLOWS)[number];
|
|
18
18
|
export type CoreWorkflowId = (typeof CORE_WORKFLOWS)[number];
|
|
19
19
|
/**
|
package/dist/core/profiles.js
CHANGED
|
@@ -8,13 +8,14 @@
|
|
|
8
8
|
* Core workflows included in the 'core' profile.
|
|
9
9
|
* These provide the streamlined experience for new users.
|
|
10
10
|
*/
|
|
11
|
-
export const CORE_WORKFLOWS = ['propose', 'explore', 'apply', 'sync', 'archive'];
|
|
11
|
+
export const CORE_WORKFLOWS = ['propose', 'explore', 'modify', 'apply', 'sync', 'archive'];
|
|
12
12
|
/**
|
|
13
13
|
* All available workflows in the system.
|
|
14
14
|
*/
|
|
15
15
|
export const ALL_WORKFLOWS = [
|
|
16
16
|
'propose',
|
|
17
17
|
'explore',
|
|
18
|
+
'modify',
|
|
18
19
|
'new',
|
|
19
20
|
'continue',
|
|
20
21
|
'apply',
|
|
@@ -3,7 +3,7 @@
|
|
|
3
3
|
*
|
|
4
4
|
* Shared utilities for generating skill and command files.
|
|
5
5
|
*/
|
|
6
|
-
import { getExploreSkillTemplate, getNewChangeSkillTemplate, getContinueChangeSkillTemplate, getApplyChangeSkillTemplate, getFfChangeSkillTemplate, getSyncSpecsSkillTemplate, getArchiveChangeSkillTemplate, getBulkArchiveChangeSkillTemplate, getVerifyChangeSkillTemplate, getOnboardSkillTemplate, getOpsxProposeSkillTemplate, getOpsxExploreCommandTemplate, getOpsxNewCommandTemplate, getOpsxContinueCommandTemplate, getOpsxApplyCommandTemplate, getOpsxFfCommandTemplate, getOpsxSyncCommandTemplate, getOpsxArchiveCommandTemplate, getOpsxBulkArchiveCommandTemplate, getOpsxVerifyCommandTemplate, getOpsxOnboardCommandTemplate, getOpsxProposeCommandTemplate, } from '../templates/skill-templates.js';
|
|
6
|
+
import { getExploreSkillTemplate, getNewChangeSkillTemplate, getContinueChangeSkillTemplate, getApplyChangeSkillTemplate, getFfChangeSkillTemplate, getSyncSpecsSkillTemplate, getArchiveChangeSkillTemplate, getBulkArchiveChangeSkillTemplate, getVerifyChangeSkillTemplate, getOnboardSkillTemplate, getOpsxProposeSkillTemplate, getModifyChangeSkillTemplate, getOpsxExploreCommandTemplate, getOpsxNewCommandTemplate, getOpsxContinueCommandTemplate, getOpsxApplyCommandTemplate, getOpsxFfCommandTemplate, getOpsxSyncCommandTemplate, getOpsxArchiveCommandTemplate, getOpsxBulkArchiveCommandTemplate, getOpsxVerifyCommandTemplate, getOpsxOnboardCommandTemplate, getOpsxProposeCommandTemplate, getOpsxModifyCommandTemplate, } from '../templates/skill-templates.js';
|
|
7
7
|
/**
|
|
8
8
|
* Gets skill templates with their directory names, optionally filtered by workflow IDs.
|
|
9
9
|
*
|
|
@@ -15,6 +15,7 @@ export function getSkillTemplates(workflowFilter) {
|
|
|
15
15
|
{ template: getNewChangeSkillTemplate(), dirName: 'openspec-new-change', workflowId: 'new' },
|
|
16
16
|
{ template: getContinueChangeSkillTemplate(), dirName: 'openspec-continue-change', workflowId: 'continue' },
|
|
17
17
|
{ template: getApplyChangeSkillTemplate(), dirName: 'openspec-apply-change', workflowId: 'apply' },
|
|
18
|
+
{ template: getModifyChangeSkillTemplate(), dirName: 'openspec-modify-change', workflowId: 'modify' },
|
|
18
19
|
{ template: getFfChangeSkillTemplate(), dirName: 'openspec-ff-change', workflowId: 'ff' },
|
|
19
20
|
{ template: getSyncSpecsSkillTemplate(), dirName: 'openspec-sync-specs', workflowId: 'sync' },
|
|
20
21
|
{ template: getArchiveChangeSkillTemplate(), dirName: 'openspec-archive-change', workflowId: 'archive' },
|
|
@@ -39,6 +40,7 @@ export function getCommandTemplates(workflowFilter) {
|
|
|
39
40
|
{ template: getOpsxNewCommandTemplate(), id: 'new' },
|
|
40
41
|
{ template: getOpsxContinueCommandTemplate(), id: 'continue' },
|
|
41
42
|
{ template: getOpsxApplyCommandTemplate(), id: 'apply' },
|
|
43
|
+
{ template: getOpsxModifyCommandTemplate(), id: 'modify' },
|
|
42
44
|
{ template: getOpsxFfCommandTemplate(), id: 'ff' },
|
|
43
45
|
{ template: getOpsxSyncCommandTemplate(), id: 'sync' },
|
|
44
46
|
{ template: getOpsxArchiveCommandTemplate(), id: 'archive' },
|
|
@@ -6,12 +6,12 @@
|
|
|
6
6
|
/**
|
|
7
7
|
* Names of skill directories created by openspec init.
|
|
8
8
|
*/
|
|
9
|
-
export declare const SKILL_NAMES: readonly ["openspec-explore", "openspec-new-change", "openspec-continue-change", "openspec-apply-change", "openspec-ff-change", "openspec-sync-specs", "openspec-archive-change", "openspec-bulk-archive-change", "openspec-verify-change", "openspec-onboard", "openspec-propose"];
|
|
9
|
+
export declare const SKILL_NAMES: readonly ["openspec-explore", "openspec-new-change", "openspec-continue-change", "openspec-modify-change", "openspec-apply-change", "openspec-ff-change", "openspec-sync-specs", "openspec-archive-change", "openspec-bulk-archive-change", "openspec-verify-change", "openspec-onboard", "openspec-propose"];
|
|
10
10
|
export type SkillName = (typeof SKILL_NAMES)[number];
|
|
11
11
|
/**
|
|
12
12
|
* IDs of command templates created by openspec init.
|
|
13
13
|
*/
|
|
14
|
-
export declare const COMMAND_IDS: readonly ["explore", "new", "continue", "apply", "ff", "sync", "archive", "bulk-archive", "verify", "onboard", "propose"];
|
|
14
|
+
export declare const COMMAND_IDS: readonly ["explore", "new", "continue", "modify", "apply", "ff", "sync", "archive", "bulk-archive", "verify", "onboard", "propose"];
|
|
15
15
|
export type CommandId = (typeof COMMAND_IDS)[number];
|
|
16
16
|
/**
|
|
17
17
|
* Status of skill configuration for a tool.
|
|
@@ -13,6 +13,7 @@ export const SKILL_NAMES = [
|
|
|
13
13
|
'openspec-explore',
|
|
14
14
|
'openspec-new-change',
|
|
15
15
|
'openspec-continue-change',
|
|
16
|
+
'openspec-modify-change',
|
|
16
17
|
'openspec-apply-change',
|
|
17
18
|
'openspec-ff-change',
|
|
18
19
|
'openspec-sync-specs',
|
|
@@ -29,6 +30,7 @@ export const COMMAND_IDS = [
|
|
|
29
30
|
'explore',
|
|
30
31
|
'new',
|
|
31
32
|
'continue',
|
|
33
|
+
'modify',
|
|
32
34
|
'apply',
|
|
33
35
|
'ff',
|
|
34
36
|
'sync',
|
|
@@ -15,5 +15,6 @@ export { getBulkArchiveChangeSkillTemplate, getOpsxBulkArchiveCommandTemplate }
|
|
|
15
15
|
export { getVerifyChangeSkillTemplate, getOpsxVerifyCommandTemplate } from './workflows/verify-change.js';
|
|
16
16
|
export { getOnboardSkillTemplate, getOpsxOnboardCommandTemplate } from './workflows/onboard.js';
|
|
17
17
|
export { getOpsxProposeSkillTemplate, getOpsxProposeCommandTemplate } from './workflows/propose.js';
|
|
18
|
+
export { getModifyChangeSkillTemplate, getOpsxModifyCommandTemplate } from './workflows/modify-change.js';
|
|
18
19
|
export { getFeedbackSkillTemplate } from './workflows/feedback.js';
|
|
19
20
|
//# sourceMappingURL=skill-templates.d.ts.map
|
|
@@ -14,5 +14,6 @@ export { getBulkArchiveChangeSkillTemplate, getOpsxBulkArchiveCommandTemplate }
|
|
|
14
14
|
export { getVerifyChangeSkillTemplate, getOpsxVerifyCommandTemplate } from './workflows/verify-change.js';
|
|
15
15
|
export { getOnboardSkillTemplate, getOpsxOnboardCommandTemplate } from './workflows/onboard.js';
|
|
16
16
|
export { getOpsxProposeSkillTemplate, getOpsxProposeCommandTemplate } from './workflows/propose.js';
|
|
17
|
+
export { getModifyChangeSkillTemplate, getOpsxModifyCommandTemplate } from './workflows/modify-change.js';
|
|
17
18
|
export { getFeedbackSkillTemplate } from './workflows/feedback.js';
|
|
18
19
|
//# sourceMappingURL=skill-templates.js.map
|
|
@@ -181,16 +181,13 @@ export function getOpsxApplyCommandTemplate() {
|
|
|
181
181
|
|
|
182
182
|
${STORE_SELECTION_GUIDANCE}
|
|
183
183
|
|
|
184
|
-
**Input**: Optionally specify a change name (e.g., \`/opsx:apply add-auth\`). If omitted,
|
|
184
|
+
**Input**: Optionally specify a change name (e.g., \`/opsx:apply add-auth\`). If omitted, infer from conversation context; if ambiguous you MUST prompt for available changes.
|
|
185
185
|
|
|
186
186
|
**Steps**
|
|
187
187
|
|
|
188
188
|
1. **Select the change**
|
|
189
189
|
|
|
190
|
-
If a name is provided, use it. Otherwise
|
|
191
|
-
- Infer from conversation context if the user mentioned a change
|
|
192
|
-
- Auto-select if only one active change exists
|
|
193
|
-
- If ambiguous, ${PROMPT_SELECT_CHANGE}
|
|
190
|
+
If a name is provided, use it. Otherwise infer from conversation context, auto-select if only one active change exists, or if ambiguous, ${PROMPT_SELECT_CHANGE}
|
|
194
191
|
|
|
195
192
|
Always announce: "Using change: <name>" and how to override (e.g., \`/opsx:apply <other>\`).
|
|
196
193
|
|
|
@@ -198,28 +195,19 @@ ${STORE_SELECTION_GUIDANCE}
|
|
|
198
195
|
\`\`\`bash
|
|
199
196
|
openspec status --change "<name>" --json
|
|
200
197
|
\`\`\`
|
|
201
|
-
Parse the
|
|
202
|
-
- \`schemaName\`: The workflow being used (e.g., "spec-driven")
|
|
203
|
-
- \`planningHome\`, \`changeRoot\`, and \`actionContext\`: planning scope and edit constraints
|
|
204
|
-
- Which artifact contains the tasks (typically "tasks" for spec-driven, check status for others)
|
|
198
|
+
Parse \`schemaName\`, \`planningHome\`/\`changeRoot\`/\`actionContext\` (planning scope and edit constraints), and which artifact holds the tasks (typically "tasks" for spec-driven).
|
|
205
199
|
|
|
206
200
|
3. **Get apply instructions**
|
|
207
|
-
|
|
208
201
|
\`\`\`bash
|
|
209
202
|
openspec instructions apply --change "<name>" --json
|
|
210
203
|
\`\`\`
|
|
211
|
-
|
|
212
|
-
This returns:
|
|
213
|
-
- \`contextFiles\`: artifact ID -> array of concrete file paths (varies by schema)
|
|
214
|
-
- Progress (total, complete, remaining)
|
|
215
|
-
- Task list with status
|
|
216
|
-
- Dynamic instruction based on current state
|
|
204
|
+
Returns \`contextFiles\` (artifact ID -> file paths, varies by schema), progress, task list, and a dynamic instruction.
|
|
217
205
|
|
|
218
206
|
**Handle states:**
|
|
219
|
-
-
|
|
220
|
-
-
|
|
221
|
-
-
|
|
222
|
-
-
|
|
207
|
+
- \`blocked\` + \`missingArtifacts\`: show message, suggest \`/opsx:continue\`
|
|
208
|
+
- \`blocked\` + \`missingComprehension\`: proceed to step 4 (comprehension quiz) — do NOT implement
|
|
209
|
+
- \`all_done\`: congratulate, suggest archive
|
|
210
|
+
- \`ready\`: proceed to step 5
|
|
223
211
|
|
|
224
212
|
${ATLASSIAN_ENRICHMENT_GUIDANCE}
|
|
225
213
|
|
|
@@ -227,116 +215,29 @@ ${COMPREHENSION_QUIZ_GUIDANCE}
|
|
|
227
215
|
|
|
228
216
|
5. **Read context files**
|
|
229
217
|
|
|
230
|
-
After comprehension
|
|
231
|
-
The files depend on the schema being used:
|
|
232
|
-
- **spec-driven**: proposal, specs, design, plan, tasks
|
|
233
|
-
- Other schemas: follow the contextFiles from CLI output
|
|
234
|
-
|
|
235
|
-
6. **Show current progress**
|
|
218
|
+
After comprehension passes (or isn't required), read every path under \`contextFiles\` from the apply instructions output (varies by schema; spec-driven: proposal, specs, design, plan, tasks).
|
|
236
219
|
|
|
237
|
-
|
|
238
|
-
- Schema being used
|
|
239
|
-
- Progress: "N/M tasks complete"
|
|
240
|
-
- Remaining tasks overview
|
|
241
|
-
- Dynamic instruction from CLI
|
|
220
|
+
6. **Show current progress** — schema, progress ("N/M tasks complete"), remaining tasks, and the dynamic instruction from the CLI.
|
|
242
221
|
|
|
243
222
|
7. **Implement tasks (loop until done or blocked)**
|
|
244
223
|
|
|
245
|
-
For each pending task
|
|
246
|
-
- Show which task is being worked on
|
|
224
|
+
For each pending task, announce which task is being worked on.
|
|
247
225
|
|
|
248
226
|
${CONTEXT7_LOOKUP_GUIDANCE}
|
|
249
227
|
|
|
250
|
-
- Make the code changes
|
|
251
|
-
-
|
|
252
|
-
-
|
|
253
|
-
- Continue to next task
|
|
254
|
-
|
|
255
|
-
**After editing artifacts:** run \`openspec status --change "<name>" --json\` so revision tracking records content changes.
|
|
256
|
-
|
|
257
|
-
**Pause if:**
|
|
258
|
-
- Task is unclear → ask for clarification
|
|
259
|
-
- Implementation reveals a design issue → suggest updating artifacts
|
|
260
|
-
- Error or blocker encountered → report and wait for guidance
|
|
261
|
-
- User interrupts
|
|
262
|
-
|
|
263
|
-
8. **On completion or pause, show status**
|
|
264
|
-
|
|
265
|
-
Display:
|
|
266
|
-
- Tasks completed this session
|
|
267
|
-
- Overall progress: "N/M tasks complete"
|
|
268
|
-
- If all done: suggest archive
|
|
269
|
-
- If paused: explain why and wait for guidance
|
|
270
|
-
|
|
271
|
-
**Output During Implementation**
|
|
272
|
-
|
|
273
|
-
\`\`\`
|
|
274
|
-
## Implementing: <change-name> (schema: <schema-name>)
|
|
275
|
-
|
|
276
|
-
Working on task 3/7: <task description>
|
|
277
|
-
[...implementation happening...]
|
|
278
|
-
✓ Task complete
|
|
279
|
-
|
|
280
|
-
Working on task 4/7: <task description>
|
|
281
|
-
[...implementation happening...]
|
|
282
|
-
✓ Task complete
|
|
283
|
-
\`\`\`
|
|
284
|
-
|
|
285
|
-
**Output On Completion**
|
|
286
|
-
|
|
287
|
-
\`\`\`
|
|
288
|
-
## Implementation Complete
|
|
289
|
-
|
|
290
|
-
**Change:** <change-name>
|
|
291
|
-
**Schema:** <schema-name>
|
|
292
|
-
**Progress:** 7/7 tasks complete ✓
|
|
293
|
-
|
|
294
|
-
### Completed This Session
|
|
295
|
-
- [x] Task 1
|
|
296
|
-
- [x] Task 2
|
|
297
|
-
...
|
|
298
|
-
|
|
299
|
-
All tasks complete! You can archive this change with \`/opsx:archive\`.
|
|
300
|
-
\`\`\`
|
|
301
|
-
|
|
302
|
-
**Output On Pause (Issue Encountered)**
|
|
303
|
-
|
|
304
|
-
\`\`\`
|
|
305
|
-
## Implementation Paused
|
|
306
|
-
|
|
307
|
-
**Change:** <change-name>
|
|
308
|
-
**Schema:** <schema-name>
|
|
309
|
-
**Progress:** 4/7 tasks complete
|
|
310
|
-
|
|
311
|
-
### Issue Encountered
|
|
312
|
-
<description of the issue>
|
|
228
|
+
- Make the required code changes (minimal and focused), mark the task \`- [ ]\` → \`- [x]\`, and continue.
|
|
229
|
+
- **After editing artifacts:** run \`openspec status --change "<name>" --json\` so revision tracking records content changes.
|
|
230
|
+
- **Pause if** a task is unclear, implementation reveals a design issue, you hit an error/blocker, or the user interrupts.
|
|
313
231
|
|
|
314
|
-
**
|
|
315
|
-
1. <option 1>
|
|
316
|
-
2. <option 2>
|
|
317
|
-
3. Other approach
|
|
318
|
-
|
|
319
|
-
What would you like to do?
|
|
320
|
-
\`\`\`
|
|
232
|
+
8. **On completion or pause, show status** — announce each task as you start it, mark it \`✓\` when done, and report progress ("N/M tasks complete"). On completion suggest \`/opsx:archive\`; on pause state the issue and options and ask how to proceed.
|
|
321
233
|
|
|
322
234
|
**Guardrails**
|
|
323
|
-
- Keep going
|
|
324
|
-
-
|
|
325
|
-
- If task is ambiguous, pause and ask before implementing
|
|
326
|
-
- If implementation reveals issues, pause and suggest artifact updates
|
|
327
|
-
- Keep code changes minimal and scoped to each task
|
|
328
|
-
- Update task checkbox immediately after completing each task
|
|
329
|
-
- Pause on errors, blockers, or unclear requirements - don't guess
|
|
330
|
-
- Use contextFiles from CLI output, don't assume specific file names
|
|
235
|
+
- Keep going until done or blocked; pause on ambiguity, errors, or design issues and ask before guessing
|
|
236
|
+
- Read context files before starting (don't assume file names); keep changes minimal and scoped; update each checkbox immediately
|
|
331
237
|
${COMPREHENSION_APPLY_GUARDRAIL}
|
|
332
238
|
${PLAYWRIGHT_APPLY_GUARDRAIL}
|
|
333
239
|
|
|
334
|
-
**Fluid Workflow Integration**
|
|
335
|
-
|
|
336
|
-
This skill supports the "actions on a change" model:
|
|
337
|
-
|
|
338
|
-
- **Can be invoked anytime**: Before all artifacts are done (if tasks exist), after partial implementation, interleaved with other actions
|
|
339
|
-
- **Allows artifact updates**: If implementation reveals design issues, suggest updating artifacts - not phase-locked, work fluidly`
|
|
240
|
+
**Fluid Workflow Integration** — invoke anytime (before all artifacts are done if tasks exist, after partial implementation, or interleaved). If implementation reveals design issues, suggest updating artifacts; not phase-locked.`
|
|
340
241
|
};
|
|
341
242
|
}
|
|
342
243
|
//# sourceMappingURL=apply-change.js.map
|
|
@@ -4,6 +4,6 @@
|
|
|
4
4
|
* Interpolated into apply skill and slash command templates so agents run
|
|
5
5
|
* a spec comprehension gate before implementation.
|
|
6
6
|
*/
|
|
7
|
-
export declare const COMPREHENSION_QUIZ_GUIDANCE = "4. **Comprehension quiz (required before implementation)**\n\n After `openspec instructions apply --change \"<name>\" --json`, check comprehension status:\n\n - If `missingComprehension` is true OR `comprehension.required && !comprehension.passed`:\n - Do NOT edit application source code or mark task checkboxes yet\n - Read `contextFiles.proposal`, `contextFiles.design`, `contextFiles.specs`, `contextFiles.plan`, and `contextFiles.tasks` (or the `tasks` array in apply JSON)\n - Use `comprehension.questionCount` and `comprehension.questionAllocation` from the JSON\n\n **Generate questions**\n - Create exactly `comprehension.questionCount` multiple-choice questions\n - **Follow `comprehension.questionAllocation`** \u2014 generate the exact count per category (e.g. plan\u00D74, specs\u00D71); do not invent your own split\n - Each question maps to one artifact category:\n - **Proposal**: motivation, scope, or impact from `proposal.md`\n - **Design**: decisions, trade-offs, or approach from `design.md`\n - **Specs**: a `### Requirement:` or `#### Scenario:` from delta specs\n - **Plan**: code map, file targets, test plan, sequencing, or alignment with design from `plan.md`\n - **Tasks**: conceptual understanding of the implementation approach from pending (unchecked) tasks\n - Do NOT use completed tasks as question sources\n - Each question: **3 options** (`comprehension.optionsPerQuestion`, default 3) \u2014 1 correct from source substance, 2 plausible distractors from other proposal/design/spec/plan/task substance in the change\n\n **Plan question quality**\n - Test code map, file targets, test plan, sequencing, or alignment with design\n - **Forbidden**: section numbers, verbatim headings, trivia answerable without reading plan substance\n\n **Task question quality**\n - Test scope, approach, dependencies, sequencing rationale, or alignment with proposal/design/plan\n - **Forbidden**: task numbers, checklist order, \"which task says X verbatim\", or answers identifiable only by task index or checkbox position\n - Good: \"What is the primary file where quiz rules are centralized?\" (answer from task substance)\n - Bad: \"Which task number updates `comprehension-guidance.ts`?\" or \"What is the exact text of task 2.1?\"\n\n **Present and grade**\n - Present each question in chat with labeled options (A/B/C/D or 1\u20134)\n - Ask ONE question at a time; after each, STOP and wait for the user's answer before the next question\n - NEVER select answers yourself, infer what the user would pick, or call `--record-comprehension-pass` until the user has answered every question\n - On Cursor, you may use the **AskUserQuestion tool** instead of plain chat for this step.\n - Grade: `score_percent = round(correct / question_count * 100)`\n - Pass when `score_percent >= comprehension.thresholdPercent` (default 80)\n\n **On failure (score below threshold)**\n - Announce score and that a new quiz is required\n - Update `.comprehension-session.yaml` in the change dir with `used_sources` from this attempt\n - Generate a NEW question set using different proposal/design/spec/plan/task sources (avoid `used_sources`)\n - Retry until pass\n\n **On pass**\n ```bash\n openspec instructions apply --change \"<name>\" --record-comprehension-pass --score <score> --attempt <n> --question-count <count> --json\n ```\n - Re-run `openspec instructions apply --change \"<name>\" --json`\n - Confirm `state` is `\"ready\"` and `comprehension.passed` is true before continuing\n\n **Output template**\n ```\n ## Applying: <change-name> \u2014 comprehension check\n\n plan\u00D7N, specs\u00D7N, design\u00D7N, proposal\u00D7N, tasks\u00D7N \u2192 <questionCount> questions (3 options each)\n\n Question 1/N: ...\n ...\n \u2713 Comprehension passed (<score>%, attempt <n>)\n ```\n\n Then continue to step 5 (show progress) and implementation.";
|
|
7
|
+
export declare const COMPREHENSION_QUIZ_GUIDANCE = "4. **Comprehension quiz (required before implementation)**\n\n **STOP \u2014 the human developer answers every question.** NEVER answer, infer, or select answers on the user's behalf, and NEVER run `--record-comprehension-pass` until the user has answered every question. Ask one question, then end your turn and wait for their reply.\n\n After `openspec instructions apply --change \"<name>\" --json`, check comprehension status:\n\n - If `missingComprehension` is true OR `comprehension.required && !comprehension.passed`:\n - Do NOT edit application source code or mark task checkboxes yet\n - Read `contextFiles.proposal`, `contextFiles.design`, `contextFiles.specs`, `contextFiles.plan`, and `contextFiles.tasks` (or the `tasks` array in apply JSON)\n - Use `comprehension.questionCount` and `comprehension.questionAllocation` from the JSON\n\n **Generate questions**\n - Create exactly `comprehension.questionCount` multiple-choice questions\n - **Follow `comprehension.questionAllocation`** \u2014 generate the exact count per category (e.g. plan\u00D74, specs\u00D71); do not invent your own split\n - Each question maps to one artifact category:\n - **Proposal**: motivation, scope, or impact from `proposal.md`\n - **Design**: decisions, trade-offs, or approach from `design.md`\n - **Specs**: a `### Requirement:` or `#### Scenario:` from delta specs\n - **Plan**: code map, file targets, test plan, sequencing, or alignment with design from `plan.md`\n - **Tasks**: conceptual understanding of the implementation approach from pending (unchecked) tasks\n - Do NOT use completed tasks as question sources\n - Each question: **3 options** (`comprehension.optionsPerQuestion`, default 3) \u2014 1 correct from source substance, 2 plausible distractors from other proposal/design/spec/plan/task substance in the change\n\n **Plan question quality**\n - Test code map, file targets, test plan, sequencing, or alignment with design\n - **Forbidden**: section numbers, verbatim headings, trivia answerable without reading plan substance\n\n **Task question quality**\n - Test scope, approach, dependencies, sequencing rationale, or alignment with proposal/design/plan\n - **Forbidden**: task numbers, checklist order, \"which task says X verbatim\", or answers identifiable only by task index or checkbox position\n - Good: \"What is the primary file where quiz rules are centralized?\" (answer from task substance)\n - Bad: \"Which task number updates `comprehension-guidance.ts`?\" or \"What is the exact text of task 2.1?\"\n\n **Present and grade**\n - Present each question in chat with labeled options (A/B/C/D or 1\u20134)\n - Ask ONE question at a time; after each, STOP and wait for the user's answer before the next question\n - NEVER select answers yourself, infer what the user would pick, or call `--record-comprehension-pass` until the user has answered every question\n - On Cursor, you may use the **AskUserQuestion tool** instead of plain chat for this step.\n - Grade: `score_percent = round(correct / question_count * 100)`\n - Pass when `score_percent >= comprehension.thresholdPercent` (default 80)\n\n **On failure (score below threshold)**\n - Announce score and that a new quiz is required\n - Update `.comprehension-session.yaml` in the change dir with `used_sources` from this attempt\n - Generate a NEW question set using different proposal/design/spec/plan/task sources (avoid `used_sources`)\n - Retry until pass\n\n **On pass**\n ```bash\n openspec instructions apply --change \"<name>\" --record-comprehension-pass --score <score> --attempt <n> --question-count <count> --json\n ```\n - Re-run `openspec instructions apply --change \"<name>\" --json`\n - Confirm `state` is `\"ready\"` and `comprehension.passed` is true before continuing\n\n **Output template**\n ```\n ## Applying: <change-name> \u2014 comprehension check\n\n plan\u00D7N, specs\u00D7N, design\u00D7N, proposal\u00D7N, tasks\u00D7N \u2192 <questionCount> questions (3 options each)\n\n Question 1/N: ...\n ...\n \u2713 Comprehension passed (<score>%, attempt <n>)\n ```\n\n Then continue to step 5 (show progress) and implementation.";
|
|
8
8
|
export declare const COMPREHENSION_APPLY_GUARDRAIL = "- NEVER implement code or mark tasks while `missingComprehension` is true\n- NEVER skip the comprehension quiz when the apply JSON requires it\n- NEVER answer comprehension quiz questions yourself \u2014 the human developer must answer every question\n- NEVER call `--record-comprehension-pass` until the user has answered every question\n- If the user asks to skip the quiz, refuse and explain they must pass or set comprehension.enabled: false in openspec/config.yaml";
|
|
9
9
|
//# sourceMappingURL=comprehension-guidance.d.ts.map
|
|
@@ -7,6 +7,8 @@
|
|
|
7
7
|
import { COMPREHENSION_PRESENT_AND_GRADE } from './user-prompt-guidance.js';
|
|
8
8
|
export const COMPREHENSION_QUIZ_GUIDANCE = `4. **Comprehension quiz (required before implementation)**
|
|
9
9
|
|
|
10
|
+
**STOP — the human developer answers every question.** NEVER answer, infer, or select answers on the user's behalf, and NEVER run \`--record-comprehension-pass\` until the user has answered every question. Ask one question, then end your turn and wait for their reply.
|
|
11
|
+
|
|
10
12
|
After \`openspec instructions apply --change "<name>" --json\`, check comprehension status:
|
|
11
13
|
|
|
12
14
|
- If \`missingComprehension\` is true OR \`comprehension.required && !comprehension.passed\`:
|
|
@@ -0,0 +1,7 @@
|
|
|
1
|
+
/**
|
|
2
|
+
* Skill Template Workflow Modules — pre-apply artifact revision.
|
|
3
|
+
*/
|
|
4
|
+
import type { SkillTemplate, CommandTemplate } from '../types.js';
|
|
5
|
+
export declare function getModifyChangeSkillTemplate(): SkillTemplate;
|
|
6
|
+
export declare function getOpsxModifyCommandTemplate(): CommandTemplate;
|
|
7
|
+
//# sourceMappingURL=modify-change.d.ts.map
|
|
@@ -0,0 +1,132 @@
|
|
|
1
|
+
import { STORE_SELECTION_GUIDANCE } from './store-selection.js';
|
|
2
|
+
import { PROMPT_CLARIFY, PROMPT_SELECT_CHANGE_RECENT, TELEMETRY_MODIFY_GUIDANCE, } from './user-prompt-guidance.js';
|
|
3
|
+
export function getModifyChangeSkillTemplate() {
|
|
4
|
+
return {
|
|
5
|
+
name: 'openspec-modify-change',
|
|
6
|
+
description: 'Revise planning artifacts before implementation. Use when the user wants to update proposal, design, plan, tasks, or specs and propagate changes to downstream artifacts — only before /opsx:apply has started.',
|
|
7
|
+
instructions: `Revise planning artifacts on an existing change before implementation.
|
|
8
|
+
|
|
9
|
+
${STORE_SELECTION_GUIDANCE}
|
|
10
|
+
|
|
11
|
+
**Input**: Optionally specify change name, artifact to modify, and what to change. Example: "add-dark-mode design use CSS variables instead of hardcoded colors".
|
|
12
|
+
|
|
13
|
+
**Pre-apply only**: Do NOT use after /opsx:apply has started (any tasks checked off).
|
|
14
|
+
|
|
15
|
+
**Steps**
|
|
16
|
+
|
|
17
|
+
1. **Select the change and artifact**
|
|
18
|
+
|
|
19
|
+
If not provided:
|
|
20
|
+
- ${PROMPT_SELECT_CHANGE_RECENT}
|
|
21
|
+
- Ask which artifact to modify: proposal, specs, design, plan, or tasks
|
|
22
|
+
- ${PROMPT_CLARIFY} if the modify request is unclear
|
|
23
|
+
|
|
24
|
+
2. **Resolve scope and validate pre-apply**
|
|
25
|
+
\`\`\`bash
|
|
26
|
+
openspec instructions modify --change "<name>" --artifact "<id>" \\
|
|
27
|
+
--workflow-input "<user request verbatim>" --editor cursor --json
|
|
28
|
+
\`\`\`
|
|
29
|
+
${TELEMETRY_MODIFY_GUIDANCE}
|
|
30
|
+
|
|
31
|
+
If the command fails because apply has started, stop and tell the user modify is pre-apply only.
|
|
32
|
+
|
|
33
|
+
3. **Confirm scope**
|
|
34
|
+
|
|
35
|
+
Show source artifact, downstream artifacts, and full \`artifactsToUpdate\` list from JSON.
|
|
36
|
+
Ask the user to confirm before editing unless they already gave explicit approval.
|
|
37
|
+
|
|
38
|
+
4. **Update artifacts in order**
|
|
39
|
+
|
|
40
|
+
Loop through \`artifactsToUpdate\` from the modify JSON:
|
|
41
|
+
|
|
42
|
+
a. Get revision instructions:
|
|
43
|
+
\`\`\`bash
|
|
44
|
+
openspec instructions <artifact-id> --change "<name>" --json
|
|
45
|
+
\`\`\`
|
|
46
|
+
b. Read the current file and upstream dependency artifacts
|
|
47
|
+
c. Apply a surgical edit aligned with the modify request — do not rewrite unrelated sections
|
|
48
|
+
d. Write to \`resolvedOutputPath\`
|
|
49
|
+
e. For \`specs\`: add/remove/rename capability folders when proposal capabilities change
|
|
50
|
+
|
|
51
|
+
5. **Show final status**
|
|
52
|
+
\`\`\`bash
|
|
53
|
+
openspec status --change "<name>" --json
|
|
54
|
+
\`\`\`
|
|
55
|
+
|
|
56
|
+
**Output**
|
|
57
|
+
|
|
58
|
+
Summarize what changed, which artifacts were updated, and prompt: "Run \`/opsx:apply\` when ready to implement."
|
|
59
|
+
|
|
60
|
+
**Guardrails**
|
|
61
|
+
- Pre-apply only — never use after tasks have been checked off
|
|
62
|
+
- Update all artifacts in \`artifactsToUpdate\` unless the user explicitly opts out of downstream propagation
|
|
63
|
+
- Preserve task checkbox format (\`- [ ]\`) in tasks.md
|
|
64
|
+
- Do NOT skip calling \`openspec instructions\` before each artifact write (telemetry)`,
|
|
65
|
+
license: 'MIT',
|
|
66
|
+
compatibility: 'Requires openspec CLI.',
|
|
67
|
+
metadata: { author: 'openspec', version: '1.0' },
|
|
68
|
+
};
|
|
69
|
+
}
|
|
70
|
+
export function getOpsxModifyCommandTemplate() {
|
|
71
|
+
return {
|
|
72
|
+
name: 'OPSX: Modify',
|
|
73
|
+
description: 'Revise planning artifacts before implementation and propagate to downstream artifacts',
|
|
74
|
+
category: 'Workflow',
|
|
75
|
+
tags: ['workflow', 'artifacts', 'experimental'],
|
|
76
|
+
content: `Revise planning artifacts on an existing change before implementation.
|
|
77
|
+
|
|
78
|
+
${STORE_SELECTION_GUIDANCE}
|
|
79
|
+
|
|
80
|
+
**Input**: \`/opsx:modify [change-name] [artifact] <what to change>\`
|
|
81
|
+
|
|
82
|
+
Example: \`/opsx:modify add-dark-mode design use CSS variables instead of hardcoded colors\`
|
|
83
|
+
|
|
84
|
+
**Pre-apply only**: Do NOT use after \`/opsx:apply\` has started (any tasks checked off).
|
|
85
|
+
|
|
86
|
+
**Steps**
|
|
87
|
+
|
|
88
|
+
1. **Select the change and artifact**
|
|
89
|
+
|
|
90
|
+
Parse change name and artifact from input, or prompt:
|
|
91
|
+
- ${PROMPT_SELECT_CHANGE_RECENT}
|
|
92
|
+
- Ask which artifact to modify if not specified
|
|
93
|
+
- ${PROMPT_CLARIFY} if the modify request is unclear
|
|
94
|
+
|
|
95
|
+
2. **Resolve scope and validate pre-apply**
|
|
96
|
+
\`\`\`bash
|
|
97
|
+
openspec instructions modify --change "<name>" --artifact "<id>" \\
|
|
98
|
+
--workflow-input "<user request verbatim>" --editor cursor --json
|
|
99
|
+
\`\`\`
|
|
100
|
+
${TELEMETRY_MODIFY_GUIDANCE}
|
|
101
|
+
|
|
102
|
+
If apply has started, stop — modify is pre-apply only.
|
|
103
|
+
|
|
104
|
+
3. **Confirm scope**
|
|
105
|
+
|
|
106
|
+
Show source artifact, downstream list, and \`artifactsToUpdate\`. Confirm with user before editing.
|
|
107
|
+
|
|
108
|
+
4. **Update artifacts in order**
|
|
109
|
+
|
|
110
|
+
For each ID in \`artifactsToUpdate\`:
|
|
111
|
+
- \`openspec instructions <id> --change "<name>" --json\`
|
|
112
|
+
- Read current file + upstream deps; apply surgical edit
|
|
113
|
+
- Write to \`resolvedOutputPath\`
|
|
114
|
+
- For \`specs\`: sync capability folders with proposal changes
|
|
115
|
+
|
|
116
|
+
5. **Final status**
|
|
117
|
+
\`\`\`bash
|
|
118
|
+
openspec status --change "<name>" --json
|
|
119
|
+
\`\`\`
|
|
120
|
+
|
|
121
|
+
**Output**
|
|
122
|
+
|
|
123
|
+
Summarize changes and prompt: "Run \`/opsx:apply\` when ready to implement."
|
|
124
|
+
|
|
125
|
+
**Guardrails**
|
|
126
|
+
- Pre-apply only
|
|
127
|
+
- Propagate to downstream artifacts unless user opts out
|
|
128
|
+
- Call \`openspec instructions\` before each write (telemetry)
|
|
129
|
+
- Hand off to \`/opsx:apply\`, not continued implementation`,
|
|
130
|
+
};
|
|
131
|
+
}
|
|
132
|
+
//# sourceMappingURL=modify-change.js.map
|
|
@@ -12,4 +12,5 @@ export declare const PROMPT_OPEN_ENDED = "Ask the user an open-ended question in
|
|
|
12
12
|
export declare const PROMPT_CLARIFY = "Ask the user a clarifying question in chat:\n - STOP and wait for the user's reply before continuing. NEVER answer, infer, or choose on the user's behalf.\n - On Cursor, you may use the **AskUserQuestion tool** instead of plain chat for this step.";
|
|
13
13
|
export declare const COMPREHENSION_PRESENT_AND_GRADE = "**Present and grade**\n - Present each question in chat with labeled options (A/B/C/D or 1\u20134)\n - Ask ONE question at a time; after each, STOP and wait for the user's answer before the next question\n - NEVER select answers yourself, infer what the user would pick, or call `--record-comprehension-pass` until the user has answered every question\n - On Cursor, you may use the **AskUserQuestion tool** instead of plain chat for this step.\n - Grade: `score_percent = round(correct / question_count * 100)`\n - Pass when `score_percent >= comprehension.thresholdPercent` (default 80)";
|
|
14
14
|
export declare const TELEMETRY_WORKFLOW_INPUT_GUIDANCE = "**Telemetry**: When running `openspec new change`, ALWAYS pass:\n - `--workflow-input \"<user request verbatim>\"` \u2014 slash-command args or the user's open-ended answer\n - `--editor <cursor|windsurf|claude>` \u2014 the AI tool you are running in\n - For long or heavily quoted text, write a temp file and use `--workflow-input-file <path>` instead";
|
|
15
|
+
export declare const TELEMETRY_MODIFY_GUIDANCE = "**Telemetry**: When running `openspec instructions modify`, ALWAYS pass:\n - `--artifact <id>` \u2014 the source artifact being modified\n - `--workflow-input \"<user request verbatim>\"` \u2014 the user's modify request\n - `--editor <cursor|windsurf|claude>` \u2014 the AI tool you are running in\n - For long or heavily quoted text, use `--workflow-input-file <path>` instead";
|
|
15
16
|
//# sourceMappingURL=user-prompt-guidance.d.ts.map
|
|
@@ -40,4 +40,9 @@ export const TELEMETRY_WORKFLOW_INPUT_GUIDANCE = `**Telemetry**: When running \`
|
|
|
40
40
|
- \`--workflow-input "<user request verbatim>"\` — slash-command args or the user's open-ended answer
|
|
41
41
|
- \`--editor <cursor|windsurf|claude>\` — the AI tool you are running in
|
|
42
42
|
- For long or heavily quoted text, write a temp file and use \`--workflow-input-file <path>\` instead`;
|
|
43
|
+
export const TELEMETRY_MODIFY_GUIDANCE = `**Telemetry**: When running \`openspec instructions modify\`, ALWAYS pass:
|
|
44
|
+
- \`--artifact <id>\` — the source artifact being modified
|
|
45
|
+
- \`--workflow-input "<user request verbatim>"\` — the user's modify request
|
|
46
|
+
- \`--editor <cursor|windsurf|claude>\` — the AI tool you are running in
|
|
47
|
+
- For long or heavily quoted text, use \`--workflow-input-file <path>\` instead`;
|
|
43
48
|
//# sourceMappingURL=user-prompt-guidance.js.map
|
|
@@ -11,7 +11,7 @@ export declare function trackCommandFailed(command: string, error: unknown, erro
|
|
|
11
11
|
export declare function shutdown(): Promise<void>;
|
|
12
12
|
/** @internal Test helper */
|
|
13
13
|
export declare function resetTelemetryForTests(): void;
|
|
14
|
-
export { trackWorkflowStarted, maybeEmitProposalReady, maybeEmitApplyReady, trackArtifactInstructions, trackArtifactContentChanges, trackChangeArchived, buildSpecDeltasFromUpdates, } from './workflow.js';
|
|
14
|
+
export { trackWorkflowStarted, maybeEmitProposalReady, maybeEmitApplyReady, trackArtifactInstructions, trackArtifactContentChanges, trackArtifactModifyRequested, trackChangeArchived, buildSpecDeltasFromUpdates, } from './workflow.js';
|
|
15
15
|
export { trackComprehensionAttempt, trackComprehensionGateChecked, trackComprehensionRetakeRequired, incrementComprehensionAttempt, incrementComprehensionFailureCount, enrichFromMarker, } from './comprehension.js';
|
|
16
16
|
export type { EntryPoint } from './marker.js';
|
|
17
17
|
//# sourceMappingURL=index.d.ts.map
|
package/dist/telemetry/index.js
CHANGED
|
@@ -40,6 +40,6 @@ export async function shutdown() {
|
|
|
40
40
|
export function resetTelemetryForTests() {
|
|
41
41
|
resetTelemetryClientForTests();
|
|
42
42
|
}
|
|
43
|
-
export { trackWorkflowStarted, maybeEmitProposalReady, maybeEmitApplyReady, trackArtifactInstructions, trackArtifactContentChanges, trackChangeArchived, buildSpecDeltasFromUpdates, } from './workflow.js';
|
|
43
|
+
export { trackWorkflowStarted, maybeEmitProposalReady, maybeEmitApplyReady, trackArtifactInstructions, trackArtifactContentChanges, trackArtifactModifyRequested, trackChangeArchived, buildSpecDeltasFromUpdates, } from './workflow.js';
|
|
44
44
|
export { trackComprehensionAttempt, trackComprehensionGateChecked, trackComprehensionRetakeRequired, incrementComprehensionAttempt, incrementComprehensionFailureCount, enrichFromMarker, } from './comprehension.js';
|
|
45
45
|
//# sourceMappingURL=index.js.map
|
|
@@ -1,5 +1,10 @@
|
|
|
1
1
|
export declare const CHANGE_TELEMETRY_FILENAME = ".openspec-telemetry.yaml";
|
|
2
2
|
export type EntryPoint = 'propose' | 'new' | 'ff' | 'manual';
|
|
3
|
+
export interface ModifyHistoryEntry {
|
|
4
|
+
at: string;
|
|
5
|
+
source_artifact: string;
|
|
6
|
+
modify_input?: string;
|
|
7
|
+
}
|
|
3
8
|
export interface ChangeTelemetryMarker {
|
|
4
9
|
started_at?: string;
|
|
5
10
|
entry_point?: EntryPoint;
|
|
@@ -12,6 +17,7 @@ export interface ChangeTelemetryMarker {
|
|
|
12
17
|
artifact_hashes?: Record<string, string>;
|
|
13
18
|
artifact_body_cache?: Record<string, string>;
|
|
14
19
|
revision_counts?: Record<string, number>;
|
|
20
|
+
modify_history?: ModifyHistoryEntry[];
|
|
15
21
|
comprehension_attempt_count?: number;
|
|
16
22
|
comprehension_failure_count?: number;
|
|
17
23
|
comprehension_gate_last_emitted?: {
|
|
@@ -32,6 +32,16 @@ export declare function trackArtifactInstructions(params: {
|
|
|
32
32
|
artifactWasDone: boolean;
|
|
33
33
|
artifactPaths?: string[];
|
|
34
34
|
}): Promise<void>;
|
|
35
|
+
export declare function trackArtifactModifyRequested(params: {
|
|
36
|
+
changeDir: string;
|
|
37
|
+
changeName: string;
|
|
38
|
+
schema: string;
|
|
39
|
+
sourceArtifactId: string;
|
|
40
|
+
downstreamArtifactIds: string[];
|
|
41
|
+
artifactsToUpdate: string[];
|
|
42
|
+
modifyInput?: string;
|
|
43
|
+
editor?: string;
|
|
44
|
+
}): Promise<void>;
|
|
35
45
|
export declare function trackArtifactContentChanges(params: {
|
|
36
46
|
changeDir: string;
|
|
37
47
|
changeName: string;
|
|
@@ -124,6 +124,34 @@ export async function trackArtifactInstructions(params) {
|
|
|
124
124
|
});
|
|
125
125
|
await captureEvent('artifact_revision_requested', revisionProps);
|
|
126
126
|
}
|
|
127
|
+
export async function trackArtifactModifyRequested(params) {
|
|
128
|
+
const modifyInput = params.modifyInput
|
|
129
|
+
? sanitizeWorkflowInput(params.modifyInput)
|
|
130
|
+
: undefined;
|
|
131
|
+
const now = new Date().toISOString();
|
|
132
|
+
await updateMarker(params.changeDir, (current) => ({
|
|
133
|
+
...current,
|
|
134
|
+
modify_history: [
|
|
135
|
+
...(current.modify_history ?? []),
|
|
136
|
+
{
|
|
137
|
+
at: now,
|
|
138
|
+
source_artifact: params.sourceArtifactId,
|
|
139
|
+
...(modifyInput ? { modify_input: modifyInput } : {}),
|
|
140
|
+
},
|
|
141
|
+
],
|
|
142
|
+
}));
|
|
143
|
+
const props = await enrichFromMarker(params.changeDir, {
|
|
144
|
+
change_name: params.changeName,
|
|
145
|
+
schema: params.schema,
|
|
146
|
+
source_artifact_id: params.sourceArtifactId,
|
|
147
|
+
downstream_artifact_ids: params.downstreamArtifactIds,
|
|
148
|
+
artifacts_to_update: params.artifactsToUpdate,
|
|
149
|
+
phase: 'pre_apply',
|
|
150
|
+
...(modifyInput ? { modify_input: modifyInput } : {}),
|
|
151
|
+
...(params.editor ? { editor: params.editor } : {}),
|
|
152
|
+
});
|
|
153
|
+
await captureEvent('artifact_modify_requested', props);
|
|
154
|
+
}
|
|
127
155
|
async function hashArtifactFiles(changeDir, contextFiles) {
|
|
128
156
|
const hashes = {};
|
|
129
157
|
for (const artifactId of TRACKED_ARTIFACT_IDS) {
|
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "@codewalla_india/openspec",
|
|
3
|
-
"version": "1.
|
|
3
|
+
"version": "1.3.1",
|
|
4
4
|
"description": "AI-native system for spec-driven development",
|
|
5
5
|
"keywords": [
|
|
6
6
|
"openspec",
|
|
@@ -38,6 +38,24 @@
|
|
|
38
38
|
"!dist/**/__tests__",
|
|
39
39
|
"!dist/**/*.map"
|
|
40
40
|
],
|
|
41
|
+
"scripts": {
|
|
42
|
+
"lint": "eslint src/",
|
|
43
|
+
"build": "node build.js",
|
|
44
|
+
"dev": "tsc --watch",
|
|
45
|
+
"dev:cli": "pnpm build && node bin/openspec.js",
|
|
46
|
+
"test": "vitest run",
|
|
47
|
+
"test:watch": "vitest",
|
|
48
|
+
"test:ui": "vitest --ui",
|
|
49
|
+
"test:coverage": "vitest --coverage",
|
|
50
|
+
"test:postinstall": "node scripts/postinstall.js",
|
|
51
|
+
"prepare": "pnpm run build",
|
|
52
|
+
"prepublishOnly": "pnpm run build",
|
|
53
|
+
"postinstall": "node scripts/postinstall.js",
|
|
54
|
+
"check:pack-version": "node scripts/pack-version-check.mjs",
|
|
55
|
+
"release": "pnpm run release:ci",
|
|
56
|
+
"release:ci": "pnpm run check:pack-version && pnpm exec changeset publish",
|
|
57
|
+
"changeset": "changeset"
|
|
58
|
+
},
|
|
41
59
|
"engines": {
|
|
42
60
|
"node": ">=20.19.0"
|
|
43
61
|
},
|
|
@@ -62,21 +80,5 @@
|
|
|
62
80
|
"posthog-node": "^5.20.0",
|
|
63
81
|
"yaml": "^2.8.2",
|
|
64
82
|
"zod": "^4.0.17"
|
|
65
|
-
},
|
|
66
|
-
"scripts": {
|
|
67
|
-
"lint": "eslint src/",
|
|
68
|
-
"build": "node build.js",
|
|
69
|
-
"dev": "tsc --watch",
|
|
70
|
-
"dev:cli": "pnpm build && node bin/openspec.js",
|
|
71
|
-
"test": "vitest run",
|
|
72
|
-
"test:watch": "vitest",
|
|
73
|
-
"test:ui": "vitest --ui",
|
|
74
|
-
"test:coverage": "vitest --coverage",
|
|
75
|
-
"test:postinstall": "node scripts/postinstall.js",
|
|
76
|
-
"postinstall": "node scripts/postinstall.js",
|
|
77
|
-
"check:pack-version": "node scripts/pack-version-check.mjs",
|
|
78
|
-
"release": "pnpm run release:ci",
|
|
79
|
-
"release:ci": "pnpm run check:pack-version && pnpm exec changeset publish",
|
|
80
|
-
"changeset": "changeset"
|
|
81
83
|
}
|
|
82
|
-
}
|
|
84
|
+
}
|