@simplysm/sd-claude 13.0.36 → 13.0.38
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.
|
@@ -140,7 +140,7 @@ Example: Task 1 creates types.ts, Task 2 uses types.ts, Task 3 independent
|
|
|
140
140
|
- Quality review: `./code-quality-reviewer-prompt.md` -- fill in report + changed files
|
|
141
141
|
- Launch both in a single message (2 Task calls per completed task)
|
|
142
142
|
|
|
143
|
-
**Final review:** `./final-review-prompt.md` -- fill in full plan + all task results
|
|
143
|
+
**Final review:** `./final-review-prompt.md` -- fill in the design doc that corresponds to the current plan (match by topic/date in `docs/plans/`, if exists) + full plan + all task results
|
|
144
144
|
|
|
145
145
|
### Fix-Review Cycle
|
|
146
146
|
|
|
@@ -6,6 +6,10 @@ Run after all batches complete. Fill in all `[bracketed]` sections.
|
|
|
6
6
|
```
|
|
7
7
|
You are performing a final integration review of a multi-task implementation.
|
|
8
8
|
|
|
9
|
+
## Original Design
|
|
10
|
+
|
|
11
|
+
[FULL TEXT of the design document that corresponds to the current plan (match by topic/date in docs/plans/). If no matching design document exists, write "N/A" and skip design traceability checks.]
|
|
12
|
+
|
|
9
13
|
## Original Plan
|
|
10
14
|
|
|
11
15
|
[FULL TEXT of the complete plan]
|
|
@@ -16,7 +20,15 @@ You are performing a final integration review of a multi-task implementation.
|
|
|
16
20
|
|
|
17
21
|
## Your Job
|
|
18
22
|
|
|
19
|
-
Individual tasks already passed spec and quality reviews. Focus on cross-task integration:
|
|
23
|
+
Individual tasks already passed spec and quality reviews. Focus on cross-task integration and design traceability:
|
|
24
|
+
|
|
25
|
+
### Design Traceability (skip if Original Design is N/A)
|
|
26
|
+
|
|
27
|
+
1. **Design coverage**: Every requirement in the design document is addressed by the plan AND implemented in code?
|
|
28
|
+
2. **No drift**: Implementation matches the design intent? (not just the plan — the plan could have missed design requirements)
|
|
29
|
+
3. **Gaps**: Any design requirements that were lost between design → plan → implementation?
|
|
30
|
+
|
|
31
|
+
### Cross-Task Integration
|
|
20
32
|
|
|
21
33
|
1. **Plan completeness**: All tasks from the plan implemented?
|
|
22
34
|
2. **Integration**: Do tasks work together? Import/export chains correct?
|
|
@@ -31,7 +43,8 @@ Run and report results:
|
|
|
31
43
|
|
|
32
44
|
### Report
|
|
33
45
|
|
|
34
|
-
- ✅ APPROVED — all tasks integrated correctly, checks pass
|
|
46
|
+
- ✅ APPROVED — all tasks integrated correctly, design fully implemented, checks pass
|
|
35
47
|
- ❌ ISSUES:
|
|
36
|
-
- [Specific
|
|
48
|
+
- [Specific problems with file:line references]
|
|
49
|
+
- [Design requirements not implemented (if any)]
|
|
37
50
|
```
|