Skip to content

Commit aa9fee5

Browse files
johnml1135claude
andcommitted
chore: fix VersionInfoProvider bugs, drop stale RegFree.targets entry, thin opsx prompts
Extracted from PR #964 (phase1-base), where these were bundled in unrelated to the Avalonia migration: - VersionInfoProvider: copyright year no longer freezes at whatever year the constant was last edited, ApplicationVersion resolves from the correct assembly instead of always falling back to the entry assembly, and MajorVersion/ParseInformationalVersion index defensively instead of assuming a fixed part count. Covered by new VersionInfoProviderTests.cs. - RegFree.targets: removes a dangling ManagedVwWindow.dll entry; the project was already retired in #904/#906, so the entry pointed at nothing. - opsx-*.prompt.md: replace inlined instructions with delegation to the existing .claude/skills/openspec-*/SKILL.md files, per this repo's skills-over-inline-prompts convention. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
1 parent fb4c99d commit aa9fee5

14 files changed

Lines changed: 165 additions & 1904 deletions
Lines changed: 4 additions & 144 deletions
Original file line numberDiff line numberDiff line change
@@ -1,149 +1,9 @@
11
---
22
description: Implement tasks from an OpenSpec change (Experimental)
3+
agent: "agent"
4+
argument-hint: "Optional change name"
35
---
46

5-
Implement tasks from an OpenSpec change.
7+
Use the `openspec-apply-change` skill from `.claude/skills/openspec-apply-change/SKILL.md`.
68

7-
**Input**: Optionally specify a change name (e.g., `/opsx:apply add-auth`). If omitted, check if it can be inferred from conversation context. If vague or ambiguous you MUST prompt for available changes.
8-
9-
**Steps**
10-
11-
1. **Select the change**
12-
13-
If a name is provided, use it. Otherwise:
14-
- Infer from conversation context if the user mentioned a change
15-
- Auto-select if only one active change exists
16-
- If ambiguous, run `openspec list --json` to get available changes and use the **AskUserQuestion tool** to let the user select
17-
18-
Always announce: "Using change: <name>" and how to override (e.g., `/opsx:apply <other>`).
19-
20-
2. **Check status to understand the schema**
21-
```bash
22-
openspec status --change "<name>" --json
23-
```
24-
Parse the JSON to understand:
25-
- `schemaName`: The workflow being used (e.g., "spec-driven")
26-
- Which artifact contains the tasks (typically "tasks" for spec-driven, check status for others)
27-
28-
3. **Get apply instructions**
29-
30-
```bash
31-
openspec instructions apply --change "<name>" --json
32-
```
33-
34-
This returns:
35-
- `contextFiles`: artifact ID -> array of concrete file paths (varies by schema)
36-
- Progress (total, complete, remaining)
37-
- Task list with status
38-
- Dynamic instruction based on current state
39-
40-
**Handle states:**
41-
- If `state: "blocked"` (missing artifacts): show message, suggest using `/opsx:continue`
42-
- If `state: "all_done"`: congratulate, suggest archive
43-
- Otherwise: proceed to implementation
44-
45-
4. **Read context files**
46-
47-
Read every file path listed under `contextFiles` from the apply instructions output.
48-
The files depend on the schema being used:
49-
- **spec-driven**: proposal, specs, design, tasks
50-
- Other schemas: follow the contextFiles from CLI output
51-
52-
5. **Show current progress**
53-
54-
Display:
55-
- Schema being used
56-
- Progress: "N/M tasks complete"
57-
- Remaining tasks overview
58-
- Dynamic instruction from CLI
59-
60-
6. **Implement tasks (loop until done or blocked)**
61-
62-
For each pending task:
63-
- Show which task is being worked on
64-
- Make the code changes required
65-
- Keep changes minimal and focused
66-
- Mark task complete in the tasks file: `- [ ]``- [x]`
67-
- Continue to next task
68-
69-
**Pause if:**
70-
- Task is unclear → ask for clarification
71-
- Implementation reveals a design issue → suggest updating artifacts
72-
- Error or blocker encountered → report and wait for guidance
73-
- User interrupts
74-
75-
7. **On completion or pause, show status**
76-
77-
Display:
78-
- Tasks completed this session
79-
- Overall progress: "N/M tasks complete"
80-
- If all done: suggest archive
81-
- If paused: explain why and wait for guidance
82-
83-
**Output During Implementation**
84-
85-
```
86-
## Implementing: <change-name> (schema: <schema-name>)
87-
88-
Working on task 3/7: <task description>
89-
[...implementation happening...]
90-
✓ Task complete
91-
92-
Working on task 4/7: <task description>
93-
[...implementation happening...]
94-
✓ Task complete
95-
```
96-
97-
**Output On Completion**
98-
99-
```
100-
## Implementation Complete
101-
102-
**Change:** <change-name>
103-
**Schema:** <schema-name>
104-
**Progress:** 7/7 tasks complete ✓
105-
106-
### Completed This Session
107-
- [x] Task 1
108-
- [x] Task 2
109-
...
110-
111-
All tasks complete! You can archive this change with `/opsx:archive`.
112-
```
113-
114-
**Output On Pause (Issue Encountered)**
115-
116-
```
117-
## Implementation Paused
118-
119-
**Change:** <change-name>
120-
**Schema:** <schema-name>
121-
**Progress:** 4/7 tasks complete
122-
123-
### Issue Encountered
124-
<description of the issue>
125-
126-
**Options:**
127-
1. <option 1>
128-
2. <option 2>
129-
3. Other approach
130-
131-
What would you like to do?
132-
```
133-
134-
**Guardrails**
135-
- Keep going through tasks until done or blocked
136-
- Always read context files before starting (from the apply instructions output)
137-
- If task is ambiguous, pause and ask before implementing
138-
- If implementation reveals issues, pause and suggest artifact updates
139-
- Keep code changes minimal and scoped to each task
140-
- Update task checkbox immediately after completing each task
141-
- Pause on errors, blockers, or unclear requirements - don't guess
142-
- Use contextFiles from CLI output, don't assume specific file names
143-
144-
**Fluid Workflow Integration**
145-
146-
This skill supports the "actions on a change" model:
147-
148-
- **Can be invoked anytime**: Before all artifacts are done (if tasks exist), after partial implementation, interleaved with other actions
149-
- **Allows artifact updates**: If implementation reveals design issues, suggest updating artifacts - not phase-locked, work fluidly
9+
Treat any text supplied with this prompt as the optional change name. If omitted, the skill will infer from context or prompt for selection.
Lines changed: 4 additions & 152 deletions
Original file line numberDiff line numberDiff line change
@@ -1,157 +1,9 @@
11
---
22
description: Archive a completed change in the experimental workflow
3+
agent: "agent"
4+
argument-hint: "Optional change name"
35
---
46

5-
Archive a completed change in the experimental workflow.
7+
Use the `openspec-archive-change` skill from `.claude/skills/openspec-archive-change/SKILL.md`.
68

7-
**Input**: Optionally specify a change name after `/opsx:archive` (e.g., `/opsx:archive add-auth`). If omitted, check if it can be inferred from conversation context. If vague or ambiguous you MUST prompt for available changes.
8-
9-
**Steps**
10-
11-
1. **If no change name provided, prompt for selection**
12-
13-
Run `openspec list --json` to get available changes. Use the **AskUserQuestion tool** to let the user select.
14-
15-
Show only active changes (not already archived).
16-
Include the schema used for each change if available.
17-
18-
**IMPORTANT**: Do NOT guess or auto-select a change. Always let the user choose.
19-
20-
2. **Check artifact completion status**
21-
22-
Run `openspec status --change "<name>" --json` to check artifact completion.
23-
24-
Parse the JSON to understand:
25-
- `schemaName`: The workflow being used
26-
- `artifacts`: List of artifacts with their status (`done` or other)
27-
28-
**If any artifacts are not `done`:**
29-
- Display warning listing incomplete artifacts
30-
- Prompt user for confirmation to continue
31-
- Proceed if user confirms
32-
33-
3. **Check task completion status**
34-
35-
Read the tasks file (typically `tasks.md`) to check for incomplete tasks.
36-
37-
Count tasks marked with `- [ ]` (incomplete) vs `- [x]` (complete).
38-
39-
**If incomplete tasks found:**
40-
- Display warning showing count of incomplete tasks
41-
- Prompt user for confirmation to continue
42-
- Proceed if user confirms
43-
44-
**If no tasks file exists:** Proceed without task-related warning.
45-
46-
4. **Assess delta spec sync state**
47-
48-
Check for delta specs at `openspec/changes/<name>/specs/`. If none exist, proceed without sync prompt.
49-
50-
**If delta specs exist:**
51-
- Compare each delta spec with its corresponding main spec at `openspec/specs/<capability>/spec.md`
52-
- Determine what changes would be applied (adds, modifications, removals, renames)
53-
- Show a combined summary before prompting
54-
55-
**Prompt options:**
56-
- If changes needed: "Sync now (recommended)", "Archive without syncing"
57-
- If already synced: "Archive now", "Sync anyway", "Cancel"
58-
59-
If user chooses sync, use Task tool (subagent_type: "general-purpose", prompt: "Use Skill tool to invoke openspec-sync-specs for change '<name>'. Delta spec analysis: <include the analyzed delta spec summary>").
60-
- If sync succeeds, continue to archive.
61-
- If sync fails or is interrupted, prompt the user to retry sync or explicitly confirm archiving without syncing.
62-
- Proceed only when sync succeeds or the user explicitly chooses to archive without syncing.
63-
64-
5. **Perform the archive**
65-
66-
Create the archive directory if it doesn't exist:
67-
```bash
68-
mkdir -p openspec/changes/archive
69-
```
70-
71-
Generate target name using current date: `YYYY-MM-DD-<change-name>`
72-
73-
**Check if target already exists:**
74-
- If yes: Fail with error, suggest renaming existing archive or using different date
75-
- If no: Move the change directory to archive
76-
77-
```bash
78-
mv openspec/changes/<name> openspec/changes/archive/YYYY-MM-DD-<name>
79-
```
80-
81-
6. **Display summary**
82-
83-
Show archive completion summary including:
84-
- Change name
85-
- Schema that was used
86-
- Archive location
87-
- Spec sync status (synced / sync skipped / no delta specs)
88-
- Note about any warnings (incomplete artifacts/tasks)
89-
90-
**Output On Success**
91-
92-
```
93-
## Archive Complete
94-
95-
**Change:** <change-name>
96-
**Schema:** <schema-name>
97-
**Archived to:** openspec/changes/archive/YYYY-MM-DD-<name>/
98-
**Specs:** ✓ Synced to main specs
99-
100-
All artifacts complete. All tasks complete.
101-
```
102-
103-
**Output On Success (No Delta Specs)**
104-
105-
```
106-
## Archive Complete
107-
108-
**Change:** <change-name>
109-
**Schema:** <schema-name>
110-
**Archived to:** openspec/changes/archive/YYYY-MM-DD-<name>/
111-
**Specs:** No delta specs
112-
113-
All artifacts complete. All tasks complete.
114-
```
115-
116-
**Output On Success With Warnings**
117-
118-
```
119-
## Archive Complete (with warnings)
120-
121-
**Change:** <change-name>
122-
**Schema:** <schema-name>
123-
**Archived to:** openspec/changes/archive/YYYY-MM-DD-<name>/
124-
**Specs:** Sync skipped (user chose to skip)
125-
126-
**Warnings:**
127-
- Archived with 2 incomplete artifacts
128-
- Archived with 3 incomplete tasks
129-
- Delta spec sync was skipped (user chose to skip)
130-
131-
Review the archive if this was not intentional.
132-
```
133-
134-
**Output On Error (Archive Exists)**
135-
136-
```
137-
## Archive Failed
138-
139-
**Change:** <change-name>
140-
**Target:** openspec/changes/archive/YYYY-MM-DD-<name>/
141-
142-
Target archive directory already exists.
143-
144-
**Options:**
145-
1. Rename the existing archive
146-
2. Delete the existing archive if it's a duplicate
147-
3. Wait until a different date to archive
148-
```
149-
150-
**Guardrails**
151-
- Always prompt for change selection if not provided
152-
- Use artifact graph (openspec status --json) for completion checking
153-
- Don't block archive on warnings - just inform and confirm
154-
- Preserve .openspec.yaml when moving to archive (it moves with the directory)
155-
- Show clear summary of what happened
156-
- If sync is requested, use the Skill tool to invoke `openspec-sync-specs` (agent-driven) and stop for explicit user confirmation if the sync attempt fails
157-
- If delta specs exist, always run the sync assessment and show the combined summary before prompting
9+
Treat any text supplied with this prompt as the optional change name. If omitted, the skill will infer from context or prompt for selection.

0 commit comments

Comments
 (0)