@rpamis/comet 0.1.4 → 0.1.6
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 +5 -4
- package/assets/manifest.json +19 -15
- package/assets/skills/comet/SKILL.md +206 -204
- package/assets/skills/comet/scripts/comet-yaml-validate.sh +4 -2
- package/assets/skills/comet-archive/SKILL.md +151 -151
- package/assets/skills/comet-build/SKILL.md +203 -161
- package/assets/skills/comet-design/SKILL.md +108 -108
- package/assets/skills/comet-hotfix/SKILL.md +167 -165
- package/assets/skills/comet-open/SKILL.md +123 -123
- package/assets/skills/comet-tweak/SKILL.md +154 -152
- package/assets/skills/comet-verify/SKILL.md +164 -164
- package/assets/skills-zh/comet/SKILL.md +206 -0
- package/assets/skills-zh/comet-archive/SKILL.md +151 -0
- package/assets/skills-zh/comet-build/SKILL.md +203 -0
- package/assets/skills-zh/comet-design/SKILL.md +108 -0
- package/assets/skills-zh/comet-hotfix/SKILL.md +167 -0
- package/assets/skills-zh/comet-open/SKILL.md +123 -0
- package/assets/skills-zh/comet-tweak/SKILL.md +154 -0
- package/assets/skills-zh/comet-verify/SKILL.md +164 -0
- package/dist/core/init.d.ts.map +1 -1
- package/dist/core/init.js +32 -3
- package/dist/core/init.js.map +1 -1
- package/package.json +1 -1
|
@@ -1,152 +1,154 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: comet-tweak
|
|
3
|
-
description: "Comet
|
|
4
|
-
---
|
|
5
|
-
|
|
6
|
-
# Comet
|
|
7
|
-
|
|
8
|
-
Tweak
|
|
9
|
-
|
|
10
|
-
|
|
11
|
-
|
|
12
|
-
|
|
13
|
-
1.
|
|
14
|
-
2.
|
|
15
|
-
3.
|
|
16
|
-
4.
|
|
17
|
-
|
|
18
|
-
|
|
19
|
-
|
|
20
|
-
---
|
|
21
|
-
|
|
22
|
-
##
|
|
23
|
-
|
|
24
|
-
### 0.
|
|
25
|
-
|
|
26
|
-
|
|
27
|
-
|
|
28
|
-
|
|
29
|
-
1. `openspec/changes/<name>/`
|
|
30
|
-
|
|
31
|
-
|
|
32
|
-
- `test -d openspec/changes/<name>`
|
|
33
|
-
-
|
|
34
|
-
-
|
|
35
|
-
|
|
36
|
-
|
|
37
|
-
```
|
|
38
|
-
[HARD STOP] Entry check failed for comet-tweak
|
|
39
|
-
Expected: openspec/changes/<name>/.comet.yaml does not exist (new change)
|
|
40
|
-
Actual: .comet.yaml exists with phase
|
|
41
|
-
Suggestion: Pick a different change name, or check if an existing tweak is in progress.
|
|
42
|
-
```
|
|
43
|
-
|
|
44
|
-
|
|
45
|
-
|
|
46
|
-
|
|
47
|
-
|
|
48
|
-
### 1.
|
|
49
|
-
|
|
50
|
-
|
|
51
|
-
|
|
52
|
-
|
|
53
|
-
|
|
54
|
-
|
|
55
|
-
- `proposal.md` —
|
|
56
|
-
- `design.md` —
|
|
57
|
-
- `tasks.md` —
|
|
58
|
-
-
|
|
59
|
-
|
|
60
|
-
|
|
61
|
-
|
|
62
|
-
```yaml
|
|
63
|
-
workflow: tweak
|
|
64
|
-
phase: build
|
|
65
|
-
design_doc: null
|
|
66
|
-
plan: null
|
|
67
|
-
build_mode: direct
|
|
68
|
-
|
|
69
|
-
|
|
70
|
-
|
|
71
|
-
|
|
72
|
-
|
|
73
|
-
|
|
74
|
-
|
|
75
|
-
|
|
76
|
-
|
|
77
|
-
|
|
78
|
-
|
|
79
|
-
|
|
80
|
-
|
|
81
|
-
|
|
82
|
-
|
|
83
|
-
|
|
84
|
-
|
|
85
|
-
|
|
86
|
-
|
|
87
|
-
|
|
88
|
-
|
|
89
|
-
|
|
90
|
-
|
|
91
|
-
|
|
92
|
-
|
|
93
|
-
|
|
94
|
-
|
|
95
|
-
|
|
96
|
-
|
|
97
|
-
-
|
|
98
|
-
-
|
|
99
|
-
-
|
|
100
|
-
|
|
101
|
-
|
|
102
|
-
|
|
103
|
-
|
|
104
|
-
|
|
105
|
-
|
|
106
|
-
|
|
107
|
-
|
|
108
|
-
|
|
109
|
-
|
|
110
|
-
|
|
111
|
-
|
|
112
|
-
|
|
113
|
-
|
|
114
|
-
|
|
115
|
-
|
|
116
|
-
|
|
117
|
-
|
|
118
|
-
|
|
119
|
-
|
|
120
|
-
|
|
121
|
-
|
|
122
|
-
|
|
123
|
-
|
|
124
|
-
|
|
125
|
-
|
|
126
|
-
|
|
127
|
-
|
|
128
|
-
|
|
129
|
-
|
|
130
|
-
|
|
131
|
-
|
|
132
|
-
|
|
133
|
-
|
|
134
|
-
|
|
135
|
-
|
|
136
|
-
|
|
137
|
-
|
|
138
|
-
|
|
139
|
-
|
|
140
|
-
|
|
141
|
-
|
|
142
|
-
|
|
143
|
-
|
|
144
|
-
|
|
145
|
-
|
|
146
|
-
|
|
147
|
-
|
|
148
|
-
|
|
149
|
-
|
|
150
|
-
|
|
151
|
-
-
|
|
152
|
-
-
|
|
1
|
+
---
|
|
2
|
+
name: comet-tweak
|
|
3
|
+
description: "Comet preset path: Non-bug small changes (tweak). Skip brainstorming and full plan, directly open → lightweight build → light verify → archive. Applicable for copy, configuration, documentation or prompt local optimization."
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# Comet Preset Path: Tweak
|
|
7
|
+
|
|
8
|
+
Tweak is a preset workflow of Comet's five-phase capabilities, not a separate parallel process. It reuses open, build, verify, archive capabilities, only skipping brainstorming and full plan.
|
|
9
|
+
|
|
10
|
+
Applicable for small-scale non-bug changes, such as copy adjustments, configuration adjustments, documentation or prompt local optimization.
|
|
11
|
+
|
|
12
|
+
**Applicable conditions** (all must be met):
|
|
13
|
+
1. No new capability
|
|
14
|
+
2. No architecture changes
|
|
15
|
+
3. No interface changes involved
|
|
16
|
+
4. Usually not exceeding 3 tasks, 5 files
|
|
17
|
+
|
|
18
|
+
**Not applicable**: If change process discovers need for capability, architecture, or interface adjustments, should upgrade to full `/comet` workflow.
|
|
19
|
+
|
|
20
|
+
---
|
|
21
|
+
|
|
22
|
+
## Process (preset workflow, 4 phases)
|
|
23
|
+
|
|
24
|
+
### 0. Entry State Verification (Entry Check)
|
|
25
|
+
|
|
26
|
+
Before performing any operations, verify current state:
|
|
27
|
+
|
|
28
|
+
**Checklist:**
|
|
29
|
+
1. `openspec/changes/<name>/` directory does not exist, or directory exists but `.comet.yaml` does not exist (no conflict)
|
|
30
|
+
|
|
31
|
+
**Verification method:**
|
|
32
|
+
- `test -d openspec/changes/<name>` to check directory
|
|
33
|
+
- If directory exists, `test -f openspec/changes/<name>/.comet.yaml` to check config file
|
|
34
|
+
- If `.comet.yaml` exists, read `phase` to check if it's an incomplete tweak
|
|
35
|
+
|
|
36
|
+
**Failure output (has conflict):**
|
|
37
|
+
```
|
|
38
|
+
[HARD STOP] Entry check failed for comet-tweak
|
|
39
|
+
Expected: openspec/changes/<name>/.comet.yaml does not exist (new change)
|
|
40
|
+
Actual: .comet.yaml exists with phase=<actual-value>
|
|
41
|
+
Suggestion: Pick a different change name, or check if an existing tweak is in progress.
|
|
42
|
+
```
|
|
43
|
+
|
|
44
|
+
Proceed to process steps only after verification passes.
|
|
45
|
+
|
|
46
|
+
Execution chain: open → lightweight build → light verify → archive. Tweak provides default decisions for each phase: streamlined open, lightweight build, lightweight verification, archive after verification passes.
|
|
47
|
+
|
|
48
|
+
### 1. Quick Open (preset open)
|
|
49
|
+
|
|
50
|
+
Reuse Comet open capability to create change, but use tweak defaults: do not execute `openspec-explore` long exploration, directly enter streamlined change creation.
|
|
51
|
+
|
|
52
|
+
**Immediately execute:** Use the Skill tool to load the `openspec-new-change` skill. Skipping this step is prohibited.
|
|
53
|
+
|
|
54
|
+
After the skill loads, follow its guidance to create streamlined artifacts:
|
|
55
|
+
- `proposal.md` — change motivation + goals + scope
|
|
56
|
+
- `design.md` — brief implementation description (no solution comparison needed)
|
|
57
|
+
- `tasks.md` — not exceeding 3 tasks
|
|
58
|
+
- **No delta spec needed** (unless change changes existing spec acceptance scenarios; once delta spec needed, upgrade to full `/comet`)
|
|
59
|
+
|
|
60
|
+
Create independent `.comet.yaml` file under `openspec/changes/<name>/`:
|
|
61
|
+
|
|
62
|
+
```yaml
|
|
63
|
+
workflow: tweak
|
|
64
|
+
phase: build
|
|
65
|
+
design_doc: null
|
|
66
|
+
plan: null
|
|
67
|
+
build_mode: direct
|
|
68
|
+
isolation: branch
|
|
69
|
+
verify_mode: light
|
|
70
|
+
verify_result: pending
|
|
71
|
+
verified_at: null
|
|
72
|
+
archived: false
|
|
73
|
+
```
|
|
74
|
+
|
|
75
|
+
【Write verification】After creation completion, must verify:
|
|
76
|
+
cat openspec/changes/<name>/.comet.yaml
|
|
77
|
+
Confirm workflow line value is "tweak"
|
|
78
|
+
Confirm phase line value is "build"
|
|
79
|
+
Confirm design_doc line value is "null"
|
|
80
|
+
Confirm plan line value is "null"
|
|
81
|
+
Confirm build_mode line value is "direct"
|
|
82
|
+
Confirm isolation line value is "branch"
|
|
83
|
+
Confirm verify_mode line value is "light"
|
|
84
|
+
Confirm verify_result line value is "pending"
|
|
85
|
+
Confirm verified_at line value is "null"
|
|
86
|
+
Confirm archived line value is "false"
|
|
87
|
+
If any field does not match, retry write then verify again. Maximum 2 retries, report error and terminate if still fails.
|
|
88
|
+
|
|
89
|
+
### 2. Lightweight Build (preset build)
|
|
90
|
+
|
|
91
|
+
Use tweak defaults: `build_mode: direct`. Skip `superpowers:brainstorming` and `superpowers:writing-plans`.
|
|
92
|
+
|
|
93
|
+
**Immediately execute:** Execute tasks one by one according to tasks.md:
|
|
94
|
+
|
|
95
|
+
1. Read `openspec/changes/<name>/tasks.md`, get incomplete task list
|
|
96
|
+
2. For each incomplete task:
|
|
97
|
+
- Modify target file according to task description
|
|
98
|
+
- Run `mvn spotless:apply` to format
|
|
99
|
+
- Run related tests to confirm pass
|
|
100
|
+
- Check corresponding `- [ ]` to `- [x]` in tasks.md
|
|
101
|
+
- Commit code, commit message format: `tweak: <brief change description>`
|
|
102
|
+
3. After all tasks complete, enter verification
|
|
103
|
+
|
|
104
|
+
### 3. Lightweight Verification (preset verify)
|
|
105
|
+
|
|
106
|
+
Reuse `/comet-verify`. Tweak must maintain lightweight verification conditions: ≤ 3 tasks, ≤ 5 files, no delta spec, no new capability.
|
|
107
|
+
|
|
108
|
+
**Immediately execute:** Use the Skill tool to load the `comet-verify` skill. Skipping this step is prohibited.
|
|
109
|
+
|
|
110
|
+
If scale assessment enters full verification path, stop tweak, upgrade to full `/comet`.
|
|
111
|
+
|
|
112
|
+
After verification passes, record `.comet.yaml` `verify_result` as `pass` according to `/comet-verify` rules, must not skip this status before archiving.
|
|
113
|
+
|
|
114
|
+
### 4. Archive (preset archive)
|
|
115
|
+
|
|
116
|
+
Reuse `/comet-archive`. Must satisfy `verify_result: pass` in `.comet.yaml` before archiving.
|
|
117
|
+
|
|
118
|
+
**Immediately execute:** Use the Skill tool to load the `comet-archive` skill to archive. Skipping this step is prohibited.
|
|
119
|
+
|
|
120
|
+
---
|
|
121
|
+
|
|
122
|
+
## Continuous Execution Mode
|
|
123
|
+
|
|
124
|
+
<IMPORTANT>
|
|
125
|
+
Tweak workflow is **one-time continuous execution**. After invoking `/comet-tweak`, agent must automatically complete all 4 phases, without pausing to wait for user input mid-way (unless encountering upgrade conditions requiring user confirmation).
|
|
126
|
+
|
|
127
|
+
Execution order: quick open → lightweight build → lightweight verification → archive → complete
|
|
128
|
+
|
|
129
|
+
After each phase completes, immediately enter next phase, no need for user input again. Within each phase, must still call corresponding Comet/OpenSpec/Superpowers skill according to above requirements.
|
|
130
|
+
</IMPORTANT>
|
|
131
|
+
|
|
132
|
+
---
|
|
133
|
+
|
|
134
|
+
## Upgrade Conditions
|
|
135
|
+
|
|
136
|
+
When the following situations occur during execution, stop tweak workflow, upgrade to full `/comet`:
|
|
137
|
+
|
|
138
|
+
1. Need new capability
|
|
139
|
+
2. Need architecture adjustments
|
|
140
|
+
3. Need interface changes
|
|
141
|
+
4. Impact scope expands to > 5 files
|
|
142
|
+
5. Task count exceeds 3
|
|
143
|
+
6. Need delta spec
|
|
144
|
+
|
|
145
|
+
Upgrade method: On current change basis, supplement Design Doc (execute `/comet-design`), then proceed normally with full workflow.
|
|
146
|
+
|
|
147
|
+
---
|
|
148
|
+
|
|
149
|
+
## Exit Conditions
|
|
150
|
+
|
|
151
|
+
- Small change completed, tests pass
|
|
152
|
+
- Change archived
|
|
153
|
+
- No new capability, architecture adjustments, or interface changes
|
|
154
|
+
- **Phase guard**: Before build → verify run `bash $COMET_GUARD <change-name> build`, before verify → archive run `bash $COMET_GUARD <change-name> verify`
|
|
@@ -1,164 +1,164 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: comet-verify
|
|
3
|
-
description: "Comet
|
|
4
|
-
---
|
|
5
|
-
|
|
6
|
-
# Comet
|
|
7
|
-
|
|
8
|
-
##
|
|
9
|
-
|
|
10
|
-
-
|
|
11
|
-
- tasks.md
|
|
12
|
-
|
|
13
|
-
##
|
|
14
|
-
|
|
15
|
-
### 0.
|
|
16
|
-
|
|
17
|
-
|
|
18
|
-
|
|
19
|
-
|
|
20
|
-
1. `openspec/changes/<name>/.comet.yaml`
|
|
21
|
-
2. `phase`
|
|
22
|
-
3. `verify_result`
|
|
23
|
-
|
|
24
|
-
|
|
25
|
-
- `cat openspec/changes/<name>/.comet.yaml`
|
|
26
|
-
-
|
|
27
|
-
|
|
28
|
-
|
|
29
|
-
```
|
|
30
|
-
[HARD STOP] Entry check failed for comet-verify
|
|
31
|
-
Expected: phase=verify, verify_result=pending|null
|
|
32
|
-
Actual: phase
|
|
33
|
-
Suggestion: Run comet-build first, or this change was already verified.
|
|
34
|
-
```
|
|
35
|
-
|
|
36
|
-
|
|
37
|
-
|
|
38
|
-
### 1.
|
|
39
|
-
|
|
40
|
-
|
|
41
|
-
|
|
42
|
-
|
|
|
43
|
-
|
|
44
|
-
| tasks.md
|
|
45
|
-
|
|
|
46
|
-
|
|
|
47
|
-
|
|
|
48
|
-
|
|
49
|
-
|
|
50
|
-
|
|
51
|
-
|
|
52
|
-
|
|
53
|
-
- `light`
|
|
54
|
-
- `full`
|
|
55
|
-
|
|
56
|
-
Few-shot
|
|
57
|
-
|
|
58
|
-
```yaml
|
|
59
|
-
#
|
|
60
|
-
phase: verify
|
|
61
|
-
verify_mode: light
|
|
62
|
-
verify_result: pending
|
|
63
|
-
```
|
|
64
|
-
|
|
65
|
-
```yaml
|
|
66
|
-
#
|
|
67
|
-
phase: verify
|
|
68
|
-
verify_mode: full
|
|
69
|
-
verify_result: pending
|
|
70
|
-
```
|
|
71
|
-
|
|
72
|
-
|
|
73
|
-
cat openspec/changes/<name>/.comet.yaml
|
|
74
|
-
|
|
75
|
-
|
|
76
|
-
|
|
77
|
-
### 2a.
|
|
78
|
-
|
|
79
|
-
|
|
80
|
-
|
|
81
|
-
1. tasks.md
|
|
82
|
-
2.
|
|
83
|
-
3.
|
|
84
|
-
4.
|
|
85
|
-
5.
|
|
86
|
-
|
|
87
|
-
|
|
88
|
-
|
|
89
|
-
|
|
90
|
-
|
|
91
|
-
|
|
92
|
-
- spec scenario
|
|
93
|
-
- design doc
|
|
94
|
-
- code pattern consistency
|
|
95
|
-
- delta spec
|
|
96
|
-
|
|
97
|
-
### 2b.
|
|
98
|
-
|
|
99
|
-
|
|
100
|
-
|
|
101
|
-
|
|
102
|
-
|
|
103
|
-
|
|
104
|
-
1. tasks.md
|
|
105
|
-
2.
|
|
106
|
-
3.
|
|
107
|
-
4.
|
|
108
|
-
5. proposal.md
|
|
109
|
-
6. delta spec
|
|
110
|
-
7. `docs/superpowers/specs/`
|
|
111
|
-
|
|
112
|
-
|
|
113
|
-
|
|
114
|
-
**Spec
|
|
115
|
-
-
|
|
116
|
-
-
|
|
117
|
-
-
|
|
118
|
-
-
|
|
119
|
-
|
|
120
|
-
### 3.
|
|
121
|
-
|
|
122
|
-
|
|
123
|
-
|
|
124
|
-
|
|
125
|
-
|
|
126
|
-
|
|
127
|
-
1.
|
|
128
|
-
2.
|
|
129
|
-
3.
|
|
130
|
-
4.
|
|
131
|
-
|
|
132
|
-
|
|
133
|
-
- Maven
|
|
134
|
-
-
|
|
135
|
-
-
|
|
136
|
-
-
|
|
137
|
-
|
|
138
|
-
##
|
|
139
|
-
|
|
140
|
-
-
|
|
141
|
-
-
|
|
142
|
-
- `.comet.yaml`
|
|
143
|
-
-
|
|
144
|
-
|
|
145
|
-
|
|
146
|
-
|
|
147
|
-
```yaml
|
|
148
|
-
phase: archive
|
|
149
|
-
verify_result: pass
|
|
150
|
-
verified_at: YYYY-MM-DD
|
|
151
|
-
```
|
|
152
|
-
|
|
153
|
-
|
|
154
|
-
cat openspec/changes/<name>/.comet.yaml
|
|
155
|
-
|
|
156
|
-
|
|
157
|
-
|
|
158
|
-
|
|
159
|
-
|
|
160
|
-
##
|
|
161
|
-
|
|
162
|
-
|
|
163
|
-
|
|
164
|
-
> **REQUIRED NEXT SKILL:**
|
|
1
|
+
---
|
|
2
|
+
name: comet-verify
|
|
3
|
+
description: "Comet Phase 4: Verify and Complete. Invoke with /comet-verify. Verify implementation matches design, handle development branch."
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# Comet Phase 4: Verify and Complete (Verify)
|
|
7
|
+
|
|
8
|
+
## Prerequisites
|
|
9
|
+
|
|
10
|
+
- Code has been committed (Phase 3 complete)
|
|
11
|
+
- All tasks in tasks.md are complete
|
|
12
|
+
|
|
13
|
+
## Steps
|
|
14
|
+
|
|
15
|
+
### 0. Entry State Verification (Entry Check)
|
|
16
|
+
|
|
17
|
+
Before performing any operations, read and verify the current state:
|
|
18
|
+
|
|
19
|
+
**Checklist:**
|
|
20
|
+
1. `openspec/changes/<name>/.comet.yaml` exists
|
|
21
|
+
2. `phase` field value is `"verify"`
|
|
22
|
+
3. `verify_result` field is `"pending"` or null (not yet verified)
|
|
23
|
+
|
|
24
|
+
**Verification method:**
|
|
25
|
+
- `cat openspec/changes/<name>/.comet.yaml` to read all fields
|
|
26
|
+
- If `verify_result` is already `"pass"`, this change has already been verified
|
|
27
|
+
|
|
28
|
+
**Failure output:**
|
|
29
|
+
```
|
|
30
|
+
[HARD STOP] Entry check failed for comet-verify
|
|
31
|
+
Expected: phase=verify, verify_result=pending|null
|
|
32
|
+
Actual: phase=<actual-value>, verify_result=<actual-value>
|
|
33
|
+
Suggestion: Run comet-build first, or this change was already verified.
|
|
34
|
+
```
|
|
35
|
+
|
|
36
|
+
Proceed to Step 1 only after verification passes.
|
|
37
|
+
|
|
38
|
+
### 1. Change Scale Assessment
|
|
39
|
+
|
|
40
|
+
Determine change scale based on the following metrics:
|
|
41
|
+
|
|
42
|
+
| Metric | Small (lightweight verification) | Large (full verification) |
|
|
43
|
+
|------|-------------------------------|--------------------------|
|
|
44
|
+
| tasks.md task count | ≤ 3 | > 3 |
|
|
45
|
+
| Changed file count (git diff --stat) | ≤ 5 | > 5 |
|
|
46
|
+
| Has delta spec | No | Yes |
|
|
47
|
+
| New capability added | No | Yes |
|
|
48
|
+
|
|
49
|
+
**Decision rule**: Any metric hitting "large" → full verification. All hitting "small" → lightweight verification.
|
|
50
|
+
|
|
51
|
+
After determination, record actual verification mode in `openspec/changes/<name>/.comet.yaml`. `verify_mode` only allows one of the following values:
|
|
52
|
+
|
|
53
|
+
- `light`
|
|
54
|
+
- `full`
|
|
55
|
+
|
|
56
|
+
Few-shot examples:
|
|
57
|
+
|
|
58
|
+
```yaml
|
|
59
|
+
# All metrics hit "small"
|
|
60
|
+
phase: verify
|
|
61
|
+
verify_mode: light
|
|
62
|
+
verify_result: pending
|
|
63
|
+
```
|
|
64
|
+
|
|
65
|
+
```yaml
|
|
66
|
+
# Any metric hits "large"
|
|
67
|
+
phase: verify
|
|
68
|
+
verify_mode: full
|
|
69
|
+
verify_result: pending
|
|
70
|
+
```
|
|
71
|
+
|
|
72
|
+
【Write verification】After update completion, must verify:
|
|
73
|
+
cat openspec/changes/<name>/.comet.yaml
|
|
74
|
+
Confirm verify_mode line value is "<light or full>"
|
|
75
|
+
If not matching, retry write then verify again. Maximum 2 retries, report error and terminate if still fails.
|
|
76
|
+
|
|
77
|
+
### 2a. Lightweight Verification (Small Changes)
|
|
78
|
+
|
|
79
|
+
When scale assessment result is "small", skip `openspec-verify-change`, directly execute the following checks:
|
|
80
|
+
|
|
81
|
+
1. All tasks in tasks.md completed `[x]`
|
|
82
|
+
2. Changed files consistent with tasks.md description (`git diff --stat`对照 tasks content)
|
|
83
|
+
3. Build passes (for Maven projects, first execute `mvn spotless:apply`, then execute `mvn compile` or equivalent command)
|
|
84
|
+
4. Related tests pass
|
|
85
|
+
5. No obvious security issues (no hardcoded secrets, no new unsafe operations)
|
|
86
|
+
|
|
87
|
+
**Pass standard**: All 5 items OK, no CRITICAL issues.
|
|
88
|
+
|
|
89
|
+
**Report format**: Brief table listing 5 check results + PASS/FAIL.
|
|
90
|
+
|
|
91
|
+
**Skipped items** (not checked in lightweight verification):
|
|
92
|
+
- spec scenario coverage
|
|
93
|
+
- design doc consistency deep comparison
|
|
94
|
+
- code pattern consistency recommendations
|
|
95
|
+
- delta spec and design doc drift detection
|
|
96
|
+
|
|
97
|
+
### 2b. Full Verification (Large Changes)
|
|
98
|
+
|
|
99
|
+
When scale assessment result is "large":
|
|
100
|
+
|
|
101
|
+
**Immediately execute:** Use the Skill tool to load the `openspec-verify-change` skill. Skipping this step is prohibited.
|
|
102
|
+
|
|
103
|
+
After the skill loads, follow its guidance to verify. Check items:
|
|
104
|
+
1. All tasks in tasks.md completed (`[x]`)
|
|
105
|
+
2. Implementation matches design.md design decisions
|
|
106
|
+
3. Implementation matches brainstorming design document
|
|
107
|
+
4. All capability specification scenarios pass
|
|
108
|
+
5. proposal.md goals satisfied
|
|
109
|
+
6. No contradiction between delta spec and design doc (if Build phase had incremental spec modifications, check if design doc has corresponding records)
|
|
110
|
+
7. `docs/superpowers/specs/` associated design document can be located (file exists and relates to current change)
|
|
111
|
+
|
|
112
|
+
When verification fails: report missing items, return to Phase 3 to supplement (invoke `/comet-build`).
|
|
113
|
+
|
|
114
|
+
**Spec drift handling**:
|
|
115
|
+
- If check item 6 finds contradiction (delta spec has content but design doc doesn't reflect it), prompt user:
|
|
116
|
+
- Option A: Append "Implementation Divergence" section to design doc recording deviation reason
|
|
117
|
+
- Option B: Roll back to Build phase, supplement brainstorming to update design doc
|
|
118
|
+
- Option C: Confirm deviation acceptable, continue verification (design doc will be marked as `superseded-by-main-spec` during archiving)
|
|
119
|
+
|
|
120
|
+
### 3. Completion (Superpowers)
|
|
121
|
+
|
|
122
|
+
**Immediately execute:** Use the Skill tool to load the `superpowers:finishing-a-development-branch` skill. Skipping this step is prohibited.
|
|
123
|
+
|
|
124
|
+
If `superpowers:finishing-a-development-branch` is unavailable, stop the process and prompt to install or enable Superpowers skills. Do not substitute this step with normal conversation.
|
|
125
|
+
|
|
126
|
+
After the skill loads, follow its guidance to complete. Branch handling options:
|
|
127
|
+
1. Local merge to main branch
|
|
128
|
+
2. Push and create PR
|
|
129
|
+
3. Keep branch (handle later)
|
|
130
|
+
4. Discard work
|
|
131
|
+
|
|
132
|
+
**Confirmation items**:
|
|
133
|
+
- Maven test or build commands have executed `mvn spotless:apply`
|
|
134
|
+
- All tests pass
|
|
135
|
+
- No remaining spotless formatting issues
|
|
136
|
+
- No hardcoded secrets or security issues
|
|
137
|
+
|
|
138
|
+
## Exit Conditions
|
|
139
|
+
|
|
140
|
+
- Verification report passed
|
|
141
|
+
- Branch handled
|
|
142
|
+
- `.comet.yaml` `verify_result` recorded as `pass`
|
|
143
|
+
- **Phase guard**: Run `bash $COMET_GUARD <change-name> verify`, allow transition only after all PASS
|
|
144
|
+
|
|
145
|
+
Before exit, merge and update the following fields in `.comet.yaml` (keep other fields unchanged):
|
|
146
|
+
|
|
147
|
+
```yaml
|
|
148
|
+
phase: archive
|
|
149
|
+
verify_result: pass
|
|
150
|
+
verified_at: YYYY-MM-DD
|
|
151
|
+
```
|
|
152
|
+
|
|
153
|
+
【Write verification】After update completion, must verify:
|
|
154
|
+
cat openspec/changes/<name>/.comet.yaml
|
|
155
|
+
Confirm phase line value is "archive"
|
|
156
|
+
Confirm verify_result line value is "pass"
|
|
157
|
+
Confirm verified_at line value is non-empty (format YYYY-MM-DD)
|
|
158
|
+
If any field does not match, retry write then verify again. Maximum 2 retries, report error and terminate if still fails.
|
|
159
|
+
|
|
160
|
+
## Automatic Transition
|
|
161
|
+
|
|
162
|
+
After exit conditions are met, **proceed immediately to the next phase without waiting for user input**:
|
|
163
|
+
|
|
164
|
+
> **REQUIRED NEXT SKILL:** Invoke `comet-archive` skill to enter the archiving phase.
|