Skip to content

Commit 7cff169

Browse files
committed
feat: sub-agent skills
1 parent ddd3ca5 commit 7cff169

7 files changed

Lines changed: 78 additions & 39 deletions

File tree

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
---
2+
id: IMPR-034
3+
name: spec-driver update command
4+
created: "2026-04-17"
5+
updated: "2026-04-17"
6+
status: idea
7+
kind: improvement
8+
---
9+
10+
# spec-driver update command
11+
12+
it'd be nice if it could figure out how to self-update depending on install method, and re-install.
13+
14+
also: it'd be nice to make the npm install less shit / spammy when already installed.
15+
16+
oh and rip out the skills whitelist.
17+

.spec-driver/deltas/DE-132-sub_agent_orchestration_skill_for_parallel_implementation/DR-132.md

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -170,14 +170,16 @@ Isolation is a per-batch rule, not a judgment call:
170170
| Batch type | Isolation | Rationale |
171171
|------------|-----------|-----------|
172172
| Parallel, file-scope disjoint | `isolation="worktree"` | avoid concurrent-write conflicts |
173-
| Sequential, depends on prior batch output | omit (main tree) | must see prior work |
173+
| Sequential, depends on prior batch or phase work | omit (main tree) | must see prior work |
174174
| Single-batch phase | omit (main tree) | no merge step needed |
175175

176-
Sequential batches *that depend on prior output* MUST NOT use
177-
`isolation="worktree"` — the worktree forks from a git ref at spawn time
178-
and cannot see uncommitted prior work (vk field evidence, §9). Independent
179-
batches may still use worktree isolation even in a serial ordering; the
180-
"depends on" clause is what forbids it.
176+
Sequential batches that depend on prior work MUST NOT use
177+
`isolation="worktree"` — regardless of whether that prior work is
178+
committed or in-flight. The worktree forks from a git ref at spawn time;
179+
even committed work can be missing from the fork if the Agent tool's
180+
branch point lags HEAD (vk field evidence, §9). Independent batches may
181+
still use worktree isolation even in a serial ordering; the "depends on"
182+
clause is what forbids it.
181183

182184
**Timing model.** `Agent(isolation="worktree")` forks from the main
183185
branch's HEAD at the moment the Agent tool executes. Uncommitted main-tree

.spec-driver/deltas/DE-132-sub_agent_orchestration_skill_for_parallel_implementation/IP-132.md

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -57,12 +57,16 @@ entries:
5757
- artefact: VA-002
5858
kind: VA
5959
requirement: ""
60-
status: planned
60+
status: verified
6161
notes: >-
6262
Agent review of revised dispatch SKILL.md §6 — answers isolation-selection
6363
questions (sequential batch? single-batch phase? parallel batch depending
6464
on prior phase commit?). Guards against recurrence of the reading-level
65-
ambiguity that caused the vk DE-105 incident.
65+
ambiguity that caused the vk DE-105 incident. PASSED 2026-04-17 (3/3
66+
correct, high confidence, clarity rated "unambiguous"); one narrow-reading
67+
risk identified and resolved in the same session (row-label tightening
68+
from "prior batch output" to "prior batch or phase work"). Evidence in
69+
phase-03.md §9.
6670
- artefact: VH-003
6771
kind: VH
6872
requirement: ""

.spec-driver/deltas/DE-132-sub_agent_orchestration_skill_for_parallel_implementation/phases/phase-03.md

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ Wire skills into the install pipeline and verify end-to-end dispatch.
7878
| [ ] | 3.5 | VH-001: Parallel dispatch test | Deferred to first real use |
7979
| [ ] | 3.6 | VH-002: Worktree isolation test | Re-run post-amendment (§5) |
8080
| [ ] | 3.7 | VH-003: Model routing test | Deferred to first real use |
81-
| [ ] | 3.8 | VA-002: Isolation-rule literacy on revised SKILL.md | See §5 |
81+
| [x] | 3.8 | VA-002: Isolation-rule literacy on revised SKILL.md | Passed 2026-04-17; see §9 |
8282

8383
## 9. Decisions & Outcomes
8484

@@ -91,6 +91,15 @@ Wire skills into the install pipeline and verify end-to-end dispatch.
9191
mode. DR-132 DEC-132-02 clarified (amendment log §9); SKILL.md §3/§6
9292
tightened; VA-002 added. VH-002 re-runs against clarified rules with
9393
concrete acceptance steps in §5.
94+
- 2026-04-17 — **VA-002 PASSED.** Sonnet agent given revised
95+
`.spec-driver/skills/dispatch/SKILL.md` §3+§6 and the three test
96+
questions from §5. All three answered correctly with §6.1 citations,
97+
confidence=high, overall clarity rated "unambiguous". Agent flagged
98+
one narrow-reading risk on the row label "depends on prior batch
99+
output" — tightened the same session to "depends on prior batch or
100+
phase work" with explicit note that the rule applies whether prior
101+
work is committed or in-flight. Mirror in DR-132 §3.5 updated to
102+
match. VH-002 still pending — requires live multi-phase dispatch.
94103

95104
## 11. Wrap-up Checklist
96105

.spec-driver/skills/dispatch/SKILL.md

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -215,13 +215,16 @@ Pick isolation by batch type. This is a rule, not a judgment call:
215215
| Batch type | `isolation` |
216216
|------------|-------------|
217217
| Parallel, verified file-scope disjoint | `"worktree"` |
218-
| Sequential, depends on prior batch output | omit (main tree) |
218+
| Sequential, depends on prior batch or phase work | omit (main tree) |
219219
| Single-batch phase | omit (main tree) |
220220

221-
Sequential batches that depend on prior output MUST NOT use
222-
`isolation="worktree"` — the worktree forks from a git ref and cannot see
223-
uncommitted prior work. Independent-but-ordered batches may use worktree
224-
isolation; the "depends on" clause is what forbids it.
221+
Sequential batches that depend on prior work MUST NOT use
222+
`isolation="worktree"` — regardless of whether that prior work is
223+
committed or in-flight. The worktree forks from a git ref at spawn
224+
time; even committed work can be missing from the fork if the Agent
225+
tool's branch point lags HEAD (the vk DE-105 failure mode).
226+
Independent-but-ordered batches may use worktree isolation; the
227+
"depends on" clause is what forbids it.
225228

226229
**Timing.** A worktree forks from the main branch's HEAD at spawn time.
227230
Uncommitted main-tree changes are invisible to the worker. Therefore,

flake.lock

Lines changed: 21 additions & 20 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

supekku/skills/dispatch/SKILL.md

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -215,13 +215,16 @@ Pick isolation by batch type. This is a rule, not a judgment call:
215215
| Batch type | `isolation` |
216216
|------------|-------------|
217217
| Parallel, verified file-scope disjoint | `"worktree"` |
218-
| Sequential, depends on prior batch output | omit (main tree) |
218+
| Sequential, depends on prior batch or phase work | omit (main tree) |
219219
| Single-batch phase | omit (main tree) |
220220

221-
Sequential batches that depend on prior output MUST NOT use
222-
`isolation="worktree"` — the worktree forks from a git ref and cannot see
223-
uncommitted prior work. Independent-but-ordered batches may use worktree
224-
isolation; the "depends on" clause is what forbids it.
221+
Sequential batches that depend on prior work MUST NOT use
222+
`isolation="worktree"` — regardless of whether that prior work is
223+
committed or in-flight. The worktree forks from a git ref at spawn
224+
time; even committed work can be missing from the fork if the Agent
225+
tool's branch point lags HEAD (the vk DE-105 failure mode).
226+
Independent-but-ordered batches may use worktree isolation; the
227+
"depends on" clause is what forbids it.
225228

226229
**Timing.** A worktree forks from the main branch's HEAD at spawn time.
227230
Uncommitted main-tree changes are invisible to the worker. Therefore,

0 commit comments

Comments
 (0)