@leeovery/claude-technical-workflows 2.0.38 → 2.0.39
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
|
@@ -344,31 +344,13 @@ After documenting dependencies, perform a **final comprehensive review** of the
|
|
|
344
344
|
- Error handling, validation rules, or boundary conditions
|
|
345
345
|
- Integration points or data flows mentioned but not elaborated
|
|
346
346
|
|
|
347
|
-
4. **
|
|
347
|
+
4. **Collect what you find** - When you discover potentially missed content, note it for your summary. You'll present all findings together after the review is complete (see "Presenting Review Findings" below).
|
|
348
348
|
|
|
349
|
-
|
|
349
|
+
Categorize each finding:
|
|
350
350
|
|
|
351
|
-
|
|
351
|
+
**Enhancing an existing topic** - Details that belong in an already-documented section. Note which section it would enhance.
|
|
352
352
|
|
|
353
|
-
**
|
|
354
|
-
|
|
355
|
-
> "During my final review, I found additional detail about [existing topic] that isn't captured. From [source]:
|
|
356
|
-
>
|
|
357
|
-
> [quote or summary from source material]
|
|
358
|
-
>
|
|
359
|
-
> I'd add this to the [section name] section. Would you like me to include it, or show you the full section with this addition first?"
|
|
360
|
-
|
|
361
|
-
If the user wants to see context, present the entire section with the new content clearly marked (e.g., with a comment like `<!-- NEW -->` or by calling it out before the block).
|
|
362
|
-
|
|
363
|
-
**An entirely missed topic** - Something that warrants its own section but was glossed over:
|
|
364
|
-
|
|
365
|
-
> "During my final review, I found [topic] discussed in [source] that doesn't have coverage in the specification:
|
|
366
|
-
>
|
|
367
|
-
> [quote or summary from source material]
|
|
368
|
-
>
|
|
369
|
-
> This would be a new section. Should I add it?"
|
|
370
|
-
|
|
371
|
-
In both cases, you know where the content belongs - existing topics get enhanced in place, new topics get added at the end.
|
|
353
|
+
**An entirely missed topic** - Something that warrants its own section but was glossed over. New topics get added at the end.
|
|
372
354
|
|
|
373
355
|
5. **Never fabricate** - Every item you flag must trace back to specific source material. If you can't point to where it came from, don't suggest it. The goal is to catch missed content, not invent new requirements.
|
|
374
356
|
|
|
@@ -380,19 +362,56 @@ After documenting dependencies, perform a **final comprehensive review** of the
|
|
|
380
362
|
- Integration points that seem implicit but aren't specified
|
|
381
363
|
- Behaviors that are ambiguous without clarification
|
|
382
364
|
|
|
383
|
-
|
|
365
|
+
Collect these alongside the missed content from step 4. They'll be presented together in the summary (see below).
|
|
366
|
+
|
|
367
|
+
This should be infrequent - most gaps will be caught from source material. But occasionally the sources themselves have blind spots worth surfacing.
|
|
368
|
+
|
|
369
|
+
### Presenting Review Findings
|
|
370
|
+
|
|
371
|
+
After completing your review (steps 1-7), present findings to the user in two stages:
|
|
384
372
|
|
|
385
|
-
|
|
373
|
+
**Stage 1: Summary of All Findings**
|
|
374
|
+
|
|
375
|
+
First, present a numbered summary of everything you found:
|
|
376
|
+
|
|
377
|
+
> "I've completed my final review against all source material. I found [N] items:
|
|
378
|
+
>
|
|
379
|
+
> 1. **[Brief title]**
|
|
380
|
+
> [2-4 line explanation: what was missed, where it came from, what it affects]
|
|
381
|
+
>
|
|
382
|
+
> 2. **[Brief title]**
|
|
383
|
+
> [2-4 line explanation]
|
|
384
|
+
>
|
|
385
|
+
> 3. **[Brief title]**
|
|
386
|
+
> [2-4 line explanation]
|
|
387
|
+
>
|
|
388
|
+
> Let's work through these one at a time, starting with #1."
|
|
389
|
+
|
|
390
|
+
Each item should have enough context that the user understands what they're about to discuss - not just a label, but clarity on what was missed and why it matters.
|
|
391
|
+
|
|
392
|
+
**Stage 2: Process One Item at a Time**
|
|
393
|
+
|
|
394
|
+
For each item, follow the **same workflow as the main specification process**:
|
|
395
|
+
|
|
396
|
+
1. **Present** the item in detail - what you found, where it came from (source reference), and what you propose to add
|
|
397
|
+
2. **Discuss** if needed - clarify ambiguities, answer questions, refine the content
|
|
398
|
+
3. **Present for approval** - show exactly what will be written to the specification:
|
|
399
|
+
|
|
400
|
+
> "Here's what I'll add to the specification:
|
|
386
401
|
>
|
|
387
|
-
>
|
|
388
|
-
> 2. **[Gap B]** - [brief description]
|
|
389
|
-
> 3. **[Gap C]** - [brief description]
|
|
402
|
+
> [content exactly as it would appear]
|
|
390
403
|
>
|
|
391
|
-
>
|
|
404
|
+
> **To proceed, choose one:**
|
|
405
|
+
> - **"Log it"** - I'll add the above to the specification **verbatim**
|
|
406
|
+
> - **"Adjust"** - Tell me which part to change
|
|
392
407
|
|
|
393
|
-
|
|
408
|
+
4. **Wait for explicit approval** - same rules as always: "Log it" or equivalent before writing
|
|
409
|
+
5. **Log verbatim** when approved
|
|
410
|
+
6. **Move to the next item**: "Moving to #2: [Brief title]..."
|
|
394
411
|
|
|
395
|
-
|
|
412
|
+
> **CHECKPOINT**: Each review item requires the full present → approve → log cycle. Do not batch multiple items together. Do not proceed to the next item until the current one is resolved (approved, adjusted, or explicitly skipped by the user).
|
|
413
|
+
|
|
414
|
+
For potential gaps (items not in source material), you're asking questions rather than proposing content. If the user wants to address a gap, discuss it, then present what you'd add for approval.
|
|
396
415
|
|
|
397
416
|
### What You're NOT Doing
|
|
398
417
|
|