flight-rules 0.11.0 → 0.12.0
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/package.json
CHANGED
package/payload/AGENTS.md
CHANGED
|
@@ -300,7 +300,8 @@ After applying changes:
|
|
|
300
300
|
> - Defined [X] initial tasks (all 🔵 Planned)
|
|
301
301
|
>
|
|
302
302
|
> **Next Steps:**
|
|
303
|
-
> - Run `/impl.create [area]` to add
|
|
303
|
+
> - Run `/impl.create [area]` to add new task groups or tasks
|
|
304
|
+
> - Run `/impl.clarify [area]` to refine existing task definitions
|
|
304
305
|
> - Run `/dev-session.start` to begin implementing this feature
|
|
305
306
|
> - Run `/feature.add` to add another feature
|
|
306
307
|
|
|
@@ -0,0 +1,183 @@
|
|
|
1
|
+
# Clarify Implementation
|
|
2
|
+
|
|
3
|
+
When the user invokes this command, help them refine and clarify existing implementation specs through targeted questions.
|
|
4
|
+
|
|
5
|
+
## 1. Read Existing Implementation
|
|
6
|
+
|
|
7
|
+
Read `docs/implementation/overview.md`. If it doesn't exist or is empty:
|
|
8
|
+
|
|
9
|
+
> "I couldn't find existing implementation specs at `docs/implementation/overview.md`. Would you like me to create an outline first with `/impl.outline`?"
|
|
10
|
+
|
|
11
|
+
Stop and wait for the user's response.
|
|
12
|
+
|
|
13
|
+
Also read `docs/prd.md` for context on goals and constraints.
|
|
14
|
+
|
|
15
|
+
## 2. Determine Scope
|
|
16
|
+
|
|
17
|
+
Check if the user specified a scope with the command:
|
|
18
|
+
- `/impl.clarify` → Ask what scope they want
|
|
19
|
+
- `/impl.clarify 1-authentication` → Clarify that area
|
|
20
|
+
- `/impl.clarify 1.2` → Clarify that task group
|
|
21
|
+
- `/impl.clarify 1.2.3` → Clarify that specific task
|
|
22
|
+
|
|
23
|
+
If no argument provided, first analyze the existing specs to identify areas that could benefit from clarification, then ask:
|
|
24
|
+
|
|
25
|
+
> **Current Implementation Structure:**
|
|
26
|
+
>
|
|
27
|
+
> [List areas with brief assessment of clarity]
|
|
28
|
+
>
|
|
29
|
+
> **Areas that may need clarification:**
|
|
30
|
+
> - [Area/task group with vague goals or missing acceptance criteria]
|
|
31
|
+
> - [Area/task group with unclear dependencies]
|
|
32
|
+
>
|
|
33
|
+
> Which would you like to clarify? You can specify:
|
|
34
|
+
> - An area (e.g., `1-authentication`)
|
|
35
|
+
> - A task group (e.g., `1.2`)
|
|
36
|
+
> - A specific task (e.g., `1.2.3`)
|
|
37
|
+
|
|
38
|
+
Wait for the user to specify what to clarify.
|
|
39
|
+
|
|
40
|
+
## 3. Analyze Current Content
|
|
41
|
+
|
|
42
|
+
Once a scope is identified, read the relevant files and analyze for clarity issues:
|
|
43
|
+
|
|
44
|
+
### For Area Scope
|
|
45
|
+
|
|
46
|
+
Read `docs/implementation/{N}-{area}/index.md` and all task group files within.
|
|
47
|
+
|
|
48
|
+
Look for:
|
|
49
|
+
- Vague or missing area goals
|
|
50
|
+
- Task groups without clear scope boundaries
|
|
51
|
+
- Missing or incomplete key considerations
|
|
52
|
+
- Unclear relationships between task groups
|
|
53
|
+
|
|
54
|
+
### For Task Group Scope
|
|
55
|
+
|
|
56
|
+
Read `docs/implementation/{N}-{area}/{N}.{M}-topic.md`.
|
|
57
|
+
|
|
58
|
+
Look for:
|
|
59
|
+
- Vague task group goals
|
|
60
|
+
- Tasks without clear acceptance criteria
|
|
61
|
+
- Missing or unclear approach descriptions
|
|
62
|
+
- Undocumented dependencies
|
|
63
|
+
- Tasks that seem too large or too small
|
|
64
|
+
|
|
65
|
+
### For Task Scope
|
|
66
|
+
|
|
67
|
+
Read the specific task within its task group file.
|
|
68
|
+
|
|
69
|
+
Look for:
|
|
70
|
+
- Goal that doesn't clearly state what "done" means
|
|
71
|
+
- Missing or vague acceptance criteria
|
|
72
|
+
- Approach that lacks technical specificity
|
|
73
|
+
- Missing dependencies on other tasks
|
|
74
|
+
|
|
75
|
+
## 4. Present Findings
|
|
76
|
+
|
|
77
|
+
Summarize the current content and what needs clarification:
|
|
78
|
+
|
|
79
|
+
> **Current [Area/Task Group/Task]:**
|
|
80
|
+
>
|
|
81
|
+
> [Quote or summarize the current content]
|
|
82
|
+
>
|
|
83
|
+
> **Areas needing clarification:**
|
|
84
|
+
> - [Specific issue 1]
|
|
85
|
+
> - [Specific issue 2]
|
|
86
|
+
|
|
87
|
+
## 5. Ask Targeted Questions
|
|
88
|
+
|
|
89
|
+
Ask 1-3 specific questions to draw out more detail. Tailor questions to what's unclear:
|
|
90
|
+
|
|
91
|
+
**Goals:**
|
|
92
|
+
- "What specific outcome does this [area/task group/task] produce?"
|
|
93
|
+
- "How does this map to the PRD goals?"
|
|
94
|
+
- "What would be different in the codebase when this is complete?"
|
|
95
|
+
|
|
96
|
+
**Acceptance Criteria:**
|
|
97
|
+
- "How would you verify this task is complete?"
|
|
98
|
+
- "What specific behavior should be testable?"
|
|
99
|
+
- "Are there edge cases that need to be handled?"
|
|
100
|
+
|
|
101
|
+
**Approach:**
|
|
102
|
+
- "What's the high-level technical strategy here?"
|
|
103
|
+
- "Are there specific patterns or libraries you want to use?"
|
|
104
|
+
- "What files or modules will this touch?"
|
|
105
|
+
|
|
106
|
+
**Dependencies:**
|
|
107
|
+
- "Does this require any other tasks to be completed first?"
|
|
108
|
+
- "What other tasks depend on this being done?"
|
|
109
|
+
- "Are there external dependencies (APIs, services) to consider?"
|
|
110
|
+
|
|
111
|
+
**Scope:**
|
|
112
|
+
- "This task seems broad — should it be broken into smaller tasks?"
|
|
113
|
+
- "This seems very specific — is it part of a larger task group?"
|
|
114
|
+
|
|
115
|
+
## 6. Push Back on Vagueness
|
|
116
|
+
|
|
117
|
+
If the user's answers are vague or unmeasurable, push back:
|
|
118
|
+
|
|
119
|
+
- ❌ "Implement authentication" → ✅ "Create login endpoint that validates credentials against user table and returns JWT token"
|
|
120
|
+
- ❌ "Make it work" → ✅ "Function returns correct result for inputs X, Y, Z as verified by unit tests"
|
|
121
|
+
- ❌ "Handle errors" → ✅ "Invalid input returns 400 with error message; server errors return 500 and log to error tracking"
|
|
122
|
+
|
|
123
|
+
> "That's a good direction, but it's hard to verify completion. What specific behavior or output would you check to confirm this is done?"
|
|
124
|
+
|
|
125
|
+
## 7. Check for Conflicts
|
|
126
|
+
|
|
127
|
+
As you clarify, watch for conflicts:
|
|
128
|
+
|
|
129
|
+
- Dependencies that create circular references
|
|
130
|
+
- Acceptance criteria that contradict other tasks
|
|
131
|
+
- Scope that overlaps with other task groups
|
|
132
|
+
- Approaches that conflict with stated constraints in the PRD
|
|
133
|
+
|
|
134
|
+
If you spot a conflict:
|
|
135
|
+
|
|
136
|
+
> "I noticed a potential conflict: [describe conflict]. Should we adjust one of these?"
|
|
137
|
+
|
|
138
|
+
## 8. Propose Updated Content
|
|
139
|
+
|
|
140
|
+
After gathering more detail, propose the updated spec:
|
|
141
|
+
|
|
142
|
+
> **Proposed [Area/Task Group/Task]:**
|
|
143
|
+
>
|
|
144
|
+
> [Show the proposed updated content in full]
|
|
145
|
+
>
|
|
146
|
+
> Does this capture what we discussed, or would you like to adjust it?
|
|
147
|
+
|
|
148
|
+
Wait for user approval before proceeding.
|
|
149
|
+
|
|
150
|
+
## 9. Update the Specs
|
|
151
|
+
|
|
152
|
+
After the user approves the clarified content:
|
|
153
|
+
|
|
154
|
+
1. Update the relevant file(s) with the new content
|
|
155
|
+
2. Preserve sections that weren't modified
|
|
156
|
+
3. Update any related files if dependencies changed
|
|
157
|
+
|
|
158
|
+
## 10. Report Changes
|
|
159
|
+
|
|
160
|
+
Summarize what was changed:
|
|
161
|
+
|
|
162
|
+
> **Implementation Specs Updated:**
|
|
163
|
+
>
|
|
164
|
+
> **Files modified:**
|
|
165
|
+
> - [File 1]: [brief description of changes]
|
|
166
|
+
>
|
|
167
|
+
> **Sections clarified:**
|
|
168
|
+
> - [Section 1]: [what was improved]
|
|
169
|
+
> - [Section 2]: [what was improved]
|
|
170
|
+
>
|
|
171
|
+
> Would you like to clarify anything else?
|
|
172
|
+
|
|
173
|
+
## Key Behaviors
|
|
174
|
+
|
|
175
|
+
Throughout this command, maintain these behaviors:
|
|
176
|
+
|
|
177
|
+
- **Analyze before asking** — Review existing specs to identify what's actually unclear
|
|
178
|
+
- **Be specific in questions** — Ask about concrete details, not abstract concepts
|
|
179
|
+
- **Push back on vagueness** — Testable > aspirational
|
|
180
|
+
- **Watch for conflicts** — Dependencies and acceptance criteria should be consistent
|
|
181
|
+
- **Propose, don't impose** — Show proposed changes and wait for approval
|
|
182
|
+
- **One scope at a time** — Don't overwhelm with too many changes at once
|
|
183
|
+
- **Map to PRD** — Ensure implementation details align with stated goals and constraints
|