Skip to content

Commit 8708d67

Browse files
committed
fix(quick-260507-esn): repair extract_learnings skill ref + drift baseline
Three fixes flushed out by the post-sync CI run: 1. skills/extract_learnings/SKILL.md was still pointing at workflows/extract_learnings.md (the old underscore name) after Task 1 renamed the workflow to extract-learnings.md. Updated both <execution_context> and the process delegation line. This was the sole Category B (genuinely-missing) drift entry CI flagged. 2. workflows/mvp-phase.md and references/planner-mvp-mode.md (new upstream files) used the legacy @~/.claude/get-shit-done/ ref form. Rewrote 7 occurrences to the plugin-runtime form @${CLAUDE_PLUGIN_ROOT}/ for correct CC ${CLAUDE_PLUGIN_ROOT} substitution. 3. Regenerated tests/drift-baseline.json to 131/131/0 (was 123/123/0). The +8 increase is the legitimate result of the upstream sync adding more plugin-counterpart refs; the drift detector's policy is to ratchet baseline when the increase is reviewed (as it is here). Category B remains 0.
1 parent 48e4cc5 commit 8708d67

4 files changed

Lines changed: 12 additions & 12 deletions

File tree

references/planner-mvp-mode.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ When the orchestrator sets `WALKING_SKELETON=true` (Phase 1 of a new project und
3838
- One real DB read/write
3939
- One real UI interaction wired to the API
4040
- Deployment to a dev environment (or a documented local-run command that exercises the full stack)
41-
- The plan **must produce** `SKELETON.md` in the phase directory alongside `PLAN.md`. Use the template at `@~/.claude/get-shit-done/references/skeleton-template.md`. `SKELETON.md` records the architectural decisions that subsequent phases will build on (chosen framework, DB, deployment target, auth approach, directory layout).
41+
- The plan **must produce** `SKELETON.md` in the phase directory alongside `PLAN.md`. Use the template at `@${CLAUDE_PLUGIN_ROOT}/references/skeleton-template.md`. `SKELETON.md` records the architectural decisions that subsequent phases will build on (chosen framework, DB, deployment target, auth approach, directory layout).
4242

4343
`SKELETON.md` is the architectural backbone for every later vertical slice; treat it as a contract, not a scratchpad.
4444

skills/extract_learnings/SKILL.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ Extract structured learnings from completed phase artifacts (PLAN.md, SUMMARY.md
1616
</objective>
1717

1818
<execution_context>
19-
@${CLAUDE_PLUGIN_ROOT}/workflows/extract_learnings.md
19+
@${CLAUDE_PLUGIN_ROOT}/workflows/extract-learnings.md
2020
</execution_context>
2121

22-
Execute the extract-learnings workflow from @${CLAUDE_PLUGIN_ROOT}/workflows/extract_learnings.md end-to-end.
22+
Execute the extract-learnings workflow from @${CLAUDE_PLUGIN_ROOT}/workflows/extract-learnings.md end-to-end.

tests/drift-baseline.json

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
11
{
22
"file_layout": {
3-
"total_dangling": 123,
4-
"has_plugin_counterpart": 123,
3+
"total_dangling": 131,
4+
"has_plugin_counterpart": 131,
55
"genuinely_missing": 0,
6-
"generated_at": "2026-05-01",
6+
"generated_at": "2026-05-07",
77
"note": "Counts-based ratchet per Phase 7 CONTEXT D-07. Regenerate with --write-baseline when drift is legitimately reduced."
88
}
99
}

workflows/mvp-phase.md

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -3,9 +3,9 @@ Guide the user through MVP-mode planning for a phase. Prompts for an "As a / I w
33
</purpose>
44

55
<required_reading>
6-
@~/.claude/get-shit-done/references/user-story-template.md
7-
@~/.claude/get-shit-done/references/spidr-splitting.md
8-
@~/.claude/get-shit-done/references/planner-mvp-mode.md
6+
@${CLAUDE_PLUGIN_ROOT}/references/user-story-template.md
7+
@${CLAUDE_PLUGIN_ROOT}/references/spidr-splitting.md
8+
@${CLAUDE_PLUGIN_ROOT}/references/planner-mvp-mode.md
99
</required_reading>
1010

1111
<runtime_note>
@@ -29,7 +29,7 @@ Example: /gsd mvp-phase 2.1
2929
```
3030
Exit.
3131

32-
Normalize per `@~/.claude/get-shit-done/references/phase-argument-parsing.md` (zero-pad integer phases to two digits).
32+
Normalize per `@${CLAUDE_PLUGIN_ROOT}/references/phase-argument-parsing.md` (zero-pad integer phases to two digits).
3333

3434
## 2. Validate phase exists and check status
3535

@@ -73,7 +73,7 @@ Use `AskUserQuestion` with options [Re-prompt / Abort]. On Abort, exit cleanly.
7373

7474
## 3. User story prompts
7575

76-
Run three sequential `AskUserQuestion` calls. Each is free-text. After all three, assemble into the canonical sentence per `@~/.claude/get-shit-done/references/user-story-template.md`:
76+
Run three sequential `AskUserQuestion` calls. Each is free-text. After all three, assemble into the canonical sentence per `@${CLAUDE_PLUGIN_ROOT}/references/user-story-template.md`:
7777

7878
**Prompt 1 — As a:**
7979
> "As a [user role]?"
@@ -112,7 +112,7 @@ If `RE_PROMPT_USER_STORY=true`, re-run only the offending prompt field(s), rebui
112112

113113
## 4. SPIDR splitting check
114114

115-
Run the SPIDR rules from `@~/.claude/get-shit-done/references/spidr-splitting.md`. Briefly:
115+
Run the SPIDR rules from `@${CLAUDE_PLUGIN_ROOT}/references/spidr-splitting.md`. Briefly:
116116

117117
**Trigger evaluation.** Check the assembled `USER_STORY` against the four size signals from the reference (compound capabilities, multi-actor, length > 120 chars, vague capability). If none fire, **skip SPIDR** entirely — go to step 5.
118118

0 commit comments

Comments
 (0)