@skitterbyte/skitterspec-jira 0.0.0-stage → 1.0.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.
Files changed (109) hide show
  1. package/MIGRATION.md +1089 -0
  2. package/README.md +42 -2
  3. package/assets/claude-md-section.md +85 -0
  4. package/assets/commands/allow-main.md +33 -0
  5. package/assets/commands/spec-connect.md +13 -0
  6. package/assets/commands/spec-live.md +14 -0
  7. package/assets/commands/spec-remote-review.md +37 -0
  8. package/assets/commands/spec-skip.md +24 -0
  9. package/assets/core/SETUP.md +192 -0
  10. package/assets/core/env.config.json.example +56 -0
  11. package/assets/core/env.config.md +554 -0
  12. package/assets/core/gating.config.json.example +4 -0
  13. package/assets/core/gating.config.md +81 -0
  14. package/assets/core/jira.config.json.example +18 -0
  15. package/assets/core/jira.config.md +61 -0
  16. package/assets/hooks/main-guard.cjs +162 -0
  17. package/assets/hooks/review-gate.cjs +146 -0
  18. package/assets/review/page.html +3384 -0
  19. package/assets/rules/commit-trailers.md +96 -0
  20. package/assets/rules/negative-checks.md +69 -0
  21. package/assets/rules/offered-unblocks.md +100 -0
  22. package/assets/rules/spec-planning.md +692 -0
  23. package/assets/rules/spec-reports.md +557 -0
  24. package/assets/skills/no-spec/SKILL.md +246 -0
  25. package/assets/skills/spec/SKILL.md +714 -0
  26. package/assets/skills/spec-bug/SKILL.md +583 -0
  27. package/assets/skills/spec-cancel/SKILL.md +261 -0
  28. package/assets/skills/spec-complete/SKILL.md +448 -0
  29. package/assets/skills/spec-diff/SKILL.md +870 -0
  30. package/assets/skills/spec-hotfix/SKILL.md +605 -0
  31. package/assets/skills/spec-init/SKILL.md +178 -0
  32. package/assets/skills/spec-jira-setup/SKILL.md +220 -0
  33. package/assets/skills/spec-list/SKILL.md +173 -0
  34. package/assets/skills/spec-next/SKILL.md +676 -0
  35. package/assets/skills/spec-push/SKILL.md +306 -0
  36. package/assets/skills/spec-review/SKILL.md +256 -0
  37. package/assets/skills/spec-reviewed/SKILL.md +284 -0
  38. package/assets/skills/spec-start/SKILL.md +496 -0
  39. package/assets/skills/spec-status/SKILL.md +112 -0
  40. package/assets/skills/spec-sync/SKILL.md +218 -0
  41. package/assets/skills/spec-to-main/SKILL.md +139 -0
  42. package/bin/skitterspec-jira.js +61 -0
  43. package/package.json +40 -4
  44. package/src/cli.js +5901 -0
  45. package/src/deprecate.js +138 -0
  46. package/src/env/bootstrap.js +188 -0
  47. package/src/env/building.js +143 -0
  48. package/src/env/classify.js +176 -0
  49. package/src/env/commitcmd.js +108 -0
  50. package/src/env/config.js +753 -0
  51. package/src/env/dev.js +63 -0
  52. package/src/env/hooks.js +226 -0
  53. package/src/env/hotfix.js +385 -0
  54. package/src/env/hotfixland.js +302 -0
  55. package/src/env/integrate.js +126 -0
  56. package/src/env/live.js +504 -0
  57. package/src/env/mainguard.js +299 -0
  58. package/src/env/provision.js +466 -0
  59. package/src/env/proxy.js +192 -0
  60. package/src/env/prune.js +119 -0
  61. package/src/env/registry.js +159 -0
  62. package/src/env/render.js +17 -0
  63. package/src/env/resolve.js +716 -0
  64. package/src/env/review.js +2577 -0
  65. package/src/env/reviewers/coderabbit.js +228 -0
  66. package/src/env/reviewers.js +517 -0
  67. package/src/env/serve.js +1044 -0
  68. package/src/env/supervise.js +157 -0
  69. package/src/env/teardown.js +248 -0
  70. package/src/env/trust.js +87 -0
  71. package/src/gating.js +155 -0
  72. package/src/init.js +1150 -0
  73. package/src/lines-diff.js +114 -0
  74. package/src/prompts.js +129 -0
  75. package/src/vendor/jira/adf.js +414 -0
  76. package/src/vendor/jira/api.js +295 -0
  77. package/src/vendor/jira/cli-sync.js +1256 -0
  78. package/src/vendor/jira/commands.js +44 -0
  79. package/src/vendor/jira/config.js +296 -0
  80. package/src/vendor/jira/credentials.js +108 -0
  81. package/src/vendor/jira/dialect.js +45 -0
  82. package/src/vendor/jira/mcp.js +129 -0
  83. package/src/vendor/provider-kit/index.js +41 -0
  84. package/src/vendor/provider-kit/src/adapter.js +23 -0
  85. package/src/vendor/provider-kit/src/apply.js +197 -0
  86. package/src/vendor/provider-kit/src/args.js +68 -0
  87. package/src/vendor/provider-kit/src/config.js +198 -0
  88. package/src/vendor/provider-kit/src/credentials.js +404 -0
  89. package/src/vendor/provider-kit/src/doctor.js +35 -0
  90. package/src/vendor/provider-kit/src/http.js +44 -0
  91. package/src/vendor/provider-kit/src/identity.js +93 -0
  92. package/src/vendor/provider-kit/src/md-compare.js +39 -0
  93. package/src/vendor/provider-kit/src/md-tables.js +94 -0
  94. package/src/vendor/provider-kit/src/prompt.js +50 -0
  95. package/src/vendor/provider-kit/src/released.js +226 -0
  96. package/src/vendor/provider-kit/src/specs.js +166 -0
  97. package/src/vendor/provider-kit/src/transport.js +22 -0
  98. package/src/vendor/sync-core/index.js +92 -0
  99. package/src/vendor/sync-core/src/base.js +81 -0
  100. package/src/vendor/sync-core/src/compare.js +306 -0
  101. package/src/vendor/sync-core/src/dialect.js +51 -0
  102. package/src/vendor/sync-core/src/fields.js +36 -0
  103. package/src/vendor/sync-core/src/legacy.js +100 -0
  104. package/src/vendor/sync-core/src/normalize.js +989 -0
  105. package/src/vendor/sync-core/src/push.js +85 -0
  106. package/src/vendor/sync-core/src/retarget.js +278 -0
  107. package/src/vendor/sync-core/src/task-block.js +308 -0
  108. package/src/vendor/sync-core/src/verify.js +67 -0
  109. package/src/vendor/sync-core/src/write.js +191 -0
@@ -0,0 +1,557 @@
1
+ # Spec Reports
2
+
3
+ Every lifecycle skill ends with the same block. This file is its whole
4
+ definition — a skill's `## Report` section names only which fields it emits and
5
+ any behaviour that was always there.
6
+
7
+ Two things it buys: a run's outcome is scannable in one shape rather than
8
+ composed fresh each time, and "nothing happened" becomes a reported outcome
9
+ instead of an absent one.
10
+
11
+ ## Silence during the run
12
+
13
+ **Do not narrate.** Not what you are about to do, not why, not which step you
14
+ are on. The tool calls are already on screen, so a line per step re-states the
15
+ transcript in words that never vary.
16
+
17
+ Speak mid-run for exactly two reasons:
18
+
19
+ - **A question you cannot answer yourself** — a decision the spec left open, a
20
+ confirmation a guard demands, an ambiguity with more than one defensible
21
+ answer.
22
+ - **A failure, at the moment it happens** — a red test, a refused command, a
23
+ conflict. Say it where it occurs; do not hold it back for the block.
24
+
25
+ Everything else waits.
26
+
27
+ ## The block
28
+
29
+ A verdict sentence, a blank line, then a two-column table:
30
+
31
+ ✅ **Phase 2 built** — `feat-foo`, 2 of 4
32
+
33
+ | | |
34
+ |---|---|
35
+ | **Tracker** | [ABC-88](https://example.invalid/ABC-88) · `feat-foo` · phase 2 moved |
36
+ | **Branch** | `spec/feat-foo` · 3 commits, clean |
37
+ | **Built** | POST /orders handler, orders schema |
38
+ | **Tests** | 128 passed · dotnet test |
39
+ | **Review** | 7 files, +212 −18 · **local** [http://127.0.0.1:7760/…](http://127.0.0.1:7760/…) · **network** [http://192.168.0.136:7760/…](http://192.168.0.136:7760/…) · **remote** off |
40
+ | **Follow-ups** | none |
41
+ | **Next** | `/spec-next` → phase 3 (Auth) |
42
+
43
+ **A table, and never a fenced block.** An aligned label column is only
44
+ achievable inside a fence, because markdown collapses runs of spaces outside
45
+ one — so alignment and being recognised as a report are mutually exclusive, and
46
+ the report wins. A fence renders as the same grey box as a code sample, and a
47
+ reader who has been shown code all session reads one more box as code: the first
48
+ person to receive one of these did not register it as output at all, and missed
49
+ the `Next` row inside it. That is the exact scanning failure this contract exists
50
+ to fix, so producing it in the contract's own format is not a cosmetic mistake.
51
+
52
+ **The verdict sentence** is `<state> **<one clause>** — <spec>, <where it got
53
+ to>`. The clause says what happened; the tail says to what, and how far. Name
54
+ the skill in the clause when the run's identity is not obvious from the
55
+ conversation — a refusal, or a skill invoked by another. A run with no spec to
56
+ name drops that segment rather than inventing one.
57
+
58
+ **The header row is empty.** The labels are the left column, and a header over
59
+ them would only be able to say "field", which no one needs told.
60
+
61
+ **The block is the last thing in the message.** Nothing follows it — not a
62
+ closing line, not a next step, not the offer. Everything the reader must act on
63
+ is a row, which is what earns the block its position.
64
+
65
+ **And that is a rule about the destination, not only the prohibition.** It was
66
+ stated plainly and broken repeatedly, by runs that had something worth saying
67
+ and nowhere in the block to say it — so the paragraph went after the table,
68
+ where it buried the rows it was meant to sit beside. A missing destination reads
69
+ as a rule that does not fit the work, and a rule that does not fit gets ignored.
70
+ So: **if it is worth telling the reader, it is a row.**
71
+ **If it is not a row, it is not worth telling them.**
72
+ `Notes` is the row most of that content belongs in.
73
+
74
+ **Two controls may follow the block, and nothing else ever.** The ban exists
75
+ because *prose* after the block competes with it for the reader's attention and
76
+ loses them the rows. A control does not compete: it is a row made actionable, in
77
+ the place the reader's eye already finishes. Prose after either of them is the
78
+ same violation wearing a control's clothes.
79
+
80
+ **The first is a picker.** Where the run ends in a choice the reader must make,
81
+ the options may be offered after the table.
82
+
83
+ **The second is the review call-to-action**, and only where the run is actually
84
+ **waiting** on a verdict. It is a banner, not a sentence:
85
+
86
+ ---
87
+
88
+ ## ⏸ Review ready — 7 files, +212 −18
89
+
90
+ - **local** — [http://127.0.0.1:7760/7e9e123e7540/feat-orders](http://127.0.0.1:7760/7e9e123e7540/feat-orders)
91
+ - **network** — [http://192.168.0.136:7760/7e9e123e7540/feat-orders](http://192.168.0.136:7760/7e9e123e7540/feat-orders)
92
+ - **remote** — off · `/spec-remote-review` turns it on
93
+
94
+ live: off — /spec-live to put it live
95
+
96
+ I'm holding here until you send a verdict — the wait covers local and network.
97
+
98
+ `/spec-reviewed` picks it up · `spec-env review skip "<reason>"` moves on
99
+
100
+ ---
101
+
102
+ A rule, a heading naming the state and the size, **the stack**, the `live:`
103
+ line, one line saying the run is stopped and which tiers it covers, and the two
104
+ exits. Nothing else, and never a paragraph explaining it.
105
+
106
+ **The `live:` line, and why a rigid contract took a second amendment.** Reading
107
+ a diff is not the only way to judge a change — often the question is whether it
108
+ *works* — and the answer to that lives at the running URL. A reader deciding
109
+ whether to open the page at all wants to know whether it is already running
110
+ somewhere, and finding that out any other way is a trip to a terminal in the
111
+ middle of a review.
112
+
113
+ **Each line names a command a person types**, rather than describing a
114
+ capability. `live: off — the page can put it live` told the reader that a page
115
+ somewhere could do it and left them to find the verb — and the verb is what they
116
+ need when they are in a terminal rather than on the page. So the lines say
117
+ `/spec-live`, `/spec-live main`, `/spec-remote-review`. Making them *clickable*
118
+ was considered and rejected: the only clickable thing markdown has is a URL, and
119
+ a URL that acts when it is **fetched** is one a link previewer or a prefetcher
120
+ fires with nobody involved.
121
+
122
+ It is the engine's line, copied like the stack: `spec-env review` prints it and
123
+ `--json` carries the same answer as `live`, from one function, so the report and
124
+ the command cannot disagree. Four states, and only three of them are a line —
125
+ `on` (with the URL when the project configured one), `off`, `held` (naming the
126
+ spec holding the workbench). The fourth, `unavailable`,
127
+ **prints nothing at all**:
128
+ a project with no isolation and a spec with no worktree both land there,
129
+ and a line about a surface that does not exist is an accusation against a
130
+ healthy repo.
131
+
132
+ **This is the second amendment to this section in two specs**, which is worth
133
+ naming rather than letting pass. The contract earns its rigidity from real
134
+ failures, and two amendments in a row is how a rigid contract quietly becomes a
135
+ negotiable one. Both have carried their reason and both replaced a line rather
136
+ than adding a paragraph.
137
+ **A third should have to argue harder than either did.**
138
+
139
+ **The stack is the engine's, copied rather than composed.** `spec-env review`
140
+ prints one line per tier and `--json` carries the same thing as `tiers`, so take
141
+ it from there in the order it comes — and do not decide which tiers are worth
142
+ mentioning. The whole reason the stack exists is that the engine was being asked
143
+ to guess where the reader was sitting, and it guessed wrong three separate ways
144
+ in one day; a skill re-deciding that in prose is the same guess one layer up.
145
+
146
+ **Three tiers, always all three, in this order.**
147
+
148
+ | Tier | Is | Off when |
149
+ |------|----|----------|
150
+ | `local` | the loopback URL — the machine the run is on | never. With nothing served it is the `file://` page, and says it cannot send a verdict |
151
+ | `network` | the LAN URL — a phone on the same wifi | `review.allowNetwork` is false, or this machine has no network address |
152
+ | `remote` | a published page, for a reader off the network | `review.allowRemote` is false, or nothing has been published |
153
+
154
+ A tier that is off **keeps its line** and names the one command that turns it
155
+ on. Dropping the line is what left a reader who had walked out of the house with
156
+ nothing on screen to tell them a remote page was even possible — they could not
157
+ ask for what they could not see.
158
+
159
+ **Only promise a wait the transport can deliver.** The wait itself is real in
160
+ every case; what varies is what carries it back, not whether the run is
161
+ stopped. Three transports, three true sentences:
162
+
163
+ - **A page the engine SERVES** hands its pass to the local store, and
164
+ `spec-env review wait` returns when it lands. The run is woken by the press:
165
+ *"I'm holding here until you send a verdict."*
166
+
167
+ **That sentence is only true for as long as the wait is actually running**, so
168
+ the wait is the engine's and never one composed for the occasion. A watcher
169
+ written per run is proven by nothing: one was written
170
+ `until [ -f "$P" ] && [ "$x" \> "$y" ]` — valid bash, a syntax error in zsh,
171
+ a predicate that could never be true — and it spun for five minutes while the
172
+ report underneath it said the run was holding.
173
+ **Silence is what made that expensive**:
174
+ a wait that cannot fire and one patiently working look identical,
175
+ so nothing seemed wrong until the reader asked. Give the wait no timeout
176
+ either; it lasts as long as the session, because a reader who walks away from
177
+ a diff is the normal case and a bounded watch once lost a verdict to a lunch
178
+ break.
179
+ - **A `file://` page** has no server to POST to, so the pass is copied and
180
+ pasted — the reader's next message is what carries it. The run is stopped just
181
+ the same, and says so:
182
+ *"I'm holding here — paste the pass when you have it."*
183
+ This is the same wait, carried by the conversation instead of a watch. It is
184
+ not a lesser one, and it is not an excuse to ask without waiting.
185
+ - **A PUBLISHED page** writes to the artifact's own store, and nothing pushes
186
+ from there into the conversation. That is the one case where a verdict is
187
+ genuinely invisible until someone asks for it, so its line in the stack says
188
+ that rather than letting the holding line speak for it:
189
+
190
+ - **remote** — [https://…](https://…) · a verdict here needs `/spec-reviewed`
191
+
192
+ Where `remote` is the only tier a reader can reach, the holding line says so
193
+ too: *press a verdict, then type `/spec-reviewed` — I cannot see it until you
194
+ do.* Where a served tier is reachable as well, the holding line keeps naming
195
+ the tiers the wait covers, and `remote`'s own caveat sits on `remote`'s line
196
+ where it belongs.
197
+
198
+ **A harness with no file-watch does not get an exemption.** It once read
199
+ "change nothing" — keep the row, keep the question, do not wait — and that
200
+ escape hatch is what let a whole class of runs go on asking questions nobody was
201
+ listening for. Where a watch is unavailable the wait is the turn ending, which
202
+ every harness can do.
203
+
204
+ **Run it under a persistent watch primitive where the harness has one** — one
205
+ that lives for the whole session and whose deliberate stop is an explicit act
206
+ (Claude Code: the `Monitor` tool with `persistent: true`, ended by `TaskStop`).
207
+ A backgrounded shell is reclaimed on a long idle — that is the death the next
208
+ paragraph recovers from — where a session-length primitive is designed to
209
+ survive it. Where the harness has none, background the command exactly as
210
+ before. The engine's wait stays the one watcher either way: its heartbeat goes
211
+ to stderr, so stdout carries only the start line and the arrival, and the exit
212
+ ends the watch.
213
+
214
+ **A watch that dies is re-armed, silently, on the same window.** A backgrounded
215
+ wait does not always survive a long idle: three sessions lost theirs over one
216
+ lunch break, and two in another were killed at 43 and 30 minutes. So a wait that
217
+ ends **without a verdict** and **was not stopped by the operator** is started
218
+ again on the **same** `--since`. Not a fresh one — a pass that arrived during
219
+ the gap is still inside the original window, and re-arming on a new timestamp
220
+ would strand exactly that pass.
221
+
222
+ **Say nothing when you do it.** Nothing happened and the window is unchanged; a
223
+ line on every death turns a working loop into a stream of non-events.
224
+
225
+ **An operator who stopped it is not a death** — re-arming over a deliberate
226
+ stop makes the wait unstoppable, which is worse than a wait that stops. But a
227
+ deliberate stop is an **explicit act with positive evidence**: this session
228
+ called TaskStop, or the operator said to stop waiting. A bare killed-task
229
+ notification carries neither and is read as a **death**. This clause once said
230
+ to treat the ambiguous kill as deliberate, and that safe branch ate the
231
+ recovery whole: every observed kill was the harness reclaiming an idle shell
232
+ and none was a person, so the case it protected against does not occur — while
233
+ the `/spec-reviewed` it charged was paid on every idle review. The deliberate
234
+ stop is the positive signal, so its absence is what a death looks like
235
+ (`.claude/rules/negative-checks.md` rule 1).
236
+
237
+ **It is bounded by the age of the window, not by a count of retries.** Keep
238
+ re-arming while the window is younger than **12 hours**, then stop. A count means
239
+ something different depending on whether deaths come five minutes or forty-five
240
+ apart, and the observed interval already varies that much. Twelve hours
241
+ deliberately does not cover an overnight gap — the pass is never lost, and
242
+ `/spec-reviewed` is the designed recovery — because the alternative is waking
243
+ the session every half hour all night for a review nobody is returning to
244
+ before morning.
245
+
246
+ **Past the bound the banner degrades**, to the sentence this file already
247
+ defines for a transport that cannot push into the conversation:
248
+
249
+ *press a verdict, then type `/spec-reviewed` — I cannot see it until you do.*
250
+
251
+ That is a new **state** for an existing sentence, not a new sentence. The
252
+ holding line is only ever printed while a wait is actually running, which is the
253
+ rule this section has always carried: *only promise a wait the transport can
254
+ deliver.*
255
+
256
+ **The `Continue` ending, and why the banner's exits differ mid-run.** A page
257
+ rendered part-way through a run offers `Continue` — *I have read it, carry on* —
258
+ in place of the committing verdicts, because "commit" is the wrong verb for
259
+ unfinished work. Its banner names the same state and the same stack, and
260
+ different exits:
261
+
262
+ …the stack, unchanged…
263
+
264
+ I'm holding here until you send a verdict — the wait covers local and network.
265
+
266
+ `Continue` carries on · `Request changes` works them now
267
+
268
+ The stack is written out once, above, and referred to everywhere else. Two
269
+ copies of it is how the two come to disagree about which tiers exist.
270
+
271
+ **It is not the `none` verdict that was removed.** `none` recorded itself and
272
+ did nothing, which is the record-and-do-nothing ending this contract exists
273
+ against. `Continue` **resumes the run** — it names an action, which is the bar
274
+ every verdict has to clear. What it does not do is commit, and it can never
275
+ clear the gate a finished phase armed: that gate is discharged by a committing
276
+ verdict or a recorded skip, and by nothing else.
277
+
278
+ **So waiting and arming are separate**, and only one of them is about this
279
+ block. *Waiting* is what any offer does, and it is what this section governs.
280
+ *Arming* asserts an obligation that outlives the turn, and belongs only to work
281
+ that is finished. A mid-run render waits without arming — walking away from it
282
+ costs nothing, which is exactly why `Continue` is safe to offer there.
283
+
284
+ **One link per reachable store, each labelled.** This amends
285
+ *"exactly one link, never two"*, and the evidence that rule was written on is
286
+ the reason it still reads the way it does. Two links were handed over, the wait
287
+ stood behind only one of them, and a verdict was pressed on the published page
288
+ three times while each one sat unread under a line claiming the run was waiting.
289
+ Two links was not a convenience with a caveat; the caveat existing was the bug.
290
+
291
+ **What that evidence is about is the store, not the count.** A served page hands
292
+ its pass to the local pending store, which `spec-env review wait` watches; a
293
+ published page writes to the artifact's own store, which nothing here can see.
294
+ Two links into two stores asks the reader to know which door the run is standing
295
+ behind, and they cannot.
296
+
297
+ **Local and network are two doors into one room.** The page POSTs with
298
+ `fetch(location.pathname, …)`, so a page opened at `127.0.0.1:7760/…` and one
299
+ opened at `192.168.0.136:7760/…` reach the same server process and the same
300
+ pending store. One wait covers both, and a verdict pressed on either wakes the
301
+ same run. Between those two the objection does not apply — which is why both are
302
+ offered, and why offering them gives nothing up.
303
+
304
+ **So two things are required, and the second is what the old rule lacked.**
305
+ Every link is **labelled** with its tier, and one line says
306
+ **which tiers the wait covers**.
307
+ Two *unlabelled* links stay forbidden exactly as before, because
308
+ the reader still cannot tell them apart.
309
+
310
+ **`remote` keeps the caveat it always had.** It is listed so a reader off the
311
+ network knows the surface exists and what turns it on — never as a door the wait
312
+ is claimed to cover. Where it is on and published, its line says what is true of
313
+ it: *a verdict here needs `/spec-reviewed`*.
314
+
315
+ **Still forbidden**, in the words of the rule this amends: offering two links
316
+ into **different** stores without saying which one the wait is watching. That is
317
+ the failure that cost three unread verdicts, and nothing here touches it.
318
+
319
+ **This is not the old failure returning**, and the difference is the whole
320
+ justification. The offer used to be two quoted lines in the tail of a long
321
+ report, addressed to nobody, under a closing line that told the reader to move
322
+ on — and it was never once taken. It was moved into a labelled row to make it
323
+ findable, and that worked. What changed since is that the run now *waits*: the
324
+ reader is not being offered something optional, they are being told the work has
325
+ stopped until they answer. A row cannot carry that, because a row is scanned at
326
+ the same weight as every other row. So the state gets the loudest shape on
327
+ screen, at the end, where reading finishes.
328
+
329
+ **ASKING IMPLIES WAITING — one rule, and the two shapes follow from it.** Any
330
+ render that asks the reader for a verdict ends the turn watching for one. A run
331
+ that does not intend to wait does not ask: its `Review` row keeps the counts and
332
+ the link and **loses the question**.
333
+
334
+ So there are two shapes and no third:
335
+
336
+ - **Asking** → the banner above, and the run waits.
337
+ - **Not asking** → the `Review` row: counts and link, no question.
338
+
339
+ **What this guards against is a reader taught that the button is decorative.**
340
+ `/spec-bug` and `/spec-hotfix` used to render the page, emit a row
341
+ asking *"want a written review before you commit?"*, and finish — with nothing
342
+ watching. A verdict pressed on that page landed in the holding area and stayed
343
+ there, because the run had told the reader the page was **ready** rather than
344
+ that it was **waiting**. It happened twice on one spec, and the second press
345
+ existed only because the first appeared to do nothing. An offer that cannot be
346
+ answered costs more than never having offered.
347
+
348
+ **Rejected: keeping the question and adding a caveat** — *"press a verdict, but
349
+ I will not see it until you ask"*. That is exactly the shape that stranded those
350
+ two passes, and a truthful caveat does not make an unanswerable question worth
351
+ asking. Either watch for the answer, or do not ask.
352
+
353
+ **A render nobody is being held for is still worth reporting** — a mid-phase
354
+ `/spec-diff`, a page produced alongside other work. It gets the row, because
355
+ giving it a banner teaches the reader to scroll past banners. The shape marks
356
+ the difference between *here is a page* and *nothing proceeds until you answer*.
357
+
358
+ **Never fence a message to the reader.** A fenced block is for a command to
359
+ run, code, or engine output quoted verbatim — things the reader copies or
360
+ compares. A question, an offer, a hand-off is prose. The same mechanism that
361
+ made the old report unreadable applies to anything else in a grey box: it is
362
+ read as an artefact to skim rather than as something addressed to someone, so a
363
+ question inside one goes unanswered and an instruction inside one goes unread.
364
+
365
+ **It reports this run and nothing else.** Not what else is in flight, not the
366
+ other worktrees, not the backlog, not the state of the repo at large. A reader
367
+ finishing one piece of work cannot tell whether a line about some other spec is
368
+ a consequence of what just happened or an unrelated aside, and has to stop and
369
+ work it out — which is the cost this block exists to remove. If they want the
370
+ wider picture there is a skill that answers it; volunteering it here muddies the
371
+ one thing they asked about.
372
+
373
+ `Next` is the single next action **for this work**, not a menu of what else
374
+ could be done.
375
+
376
+ **And it must be runnable from the state the run actually leaves behind.** Most
377
+ of these skills deliberately do not commit — they finish, verify, and wait — so
378
+ the tree they hand back is dirty, and several of the commands they would name
379
+ next refuse on a dirty tree. A `Next` that omits the commit therefore sends the
380
+ reader to a refusal: `/spec-next` leaves a phase uncommitted and its own §2
381
+ refuses on exactly that, and `/spec-bug` leaves a fix uncommitted where
382
+ `/spec-complete` refuses the same way. Where the skill does not commit, the row
383
+ says `/commit, then <the thing>`.
384
+
385
+ This is not a style rule. `Next` is the one row a reader is meant to act on, and
386
+ it closes the block because the closing row should be the one that moves the
387
+ work on. A row that cannot be run spends the trust the position was designed to
388
+ earn — and the two skills that got it wrong got it wrong independently, which is
389
+ what makes it a missing constraint rather than two slips.
390
+
391
+ ## Field vocabulary, in this order
392
+
393
+ A skill emits only the fields it declares, in this order, skipping the rest.
394
+ `Follow-ups` is the exception: always present, second to last.
395
+
396
+ `Tracker` is the conditional one: a skill may declare it and still never emit
397
+ it, because no ticketing provider is installed. That is an absence with nothing
398
+ behind it — say nothing rather than reporting that there was nothing to report.
399
+
400
+ | Field | Carries |
401
+ |-------|---------|
402
+ | `Tracker` | **First.** The ticket id, linked where the spec has a url, the spec's folder name beside it, and what changed there. |
403
+ | `Why` | **Non-`✅` only.** What stopped it, in one clause. |
404
+ | `Branch` | Branch name, commit count, clean or dirty. |
405
+ | `Spec` | The spec document's own state: status, bucket, phase. |
406
+ | `Cause` | The root cause, for work that diagnosed one. |
407
+ | `Built` | What the run produced — the code, the edits, the spec written. |
408
+ | `Tests` | The result and the command that produced it. |
409
+ | `Notes` | What this run hit and handled — one short paragraph. Named `Snags` once, which announced a problem before the reader had read one; most of what belongs here is neither good nor bad, just worth knowing. A wrong turn, a guard that fired, a check of yours that turned out to be a false negative. Not a caveat on the outcome (`⚠️` and `Why` carry those) and not future work (`Follow-ups` carries that): this is what happened on the way. |
410
+ | `Landed` | A fast-forward, a tag, a cherry-pick. |
411
+ | `Worktree` | A worktree provisioned, entered, or torn down. |
412
+ | `Untouched` | Uncommitted work the run deliberately left alone — whose, and how much. |
413
+ | `Review` | The rendered diff page: files, `+`/`−`, and the stack — the same tiers the banner lists, labelled, run together with `·` because a row is one cell, plus `live` where the engine reported one. **No question** — a row cannot be waited on, so a question in one is unanswerable by construction (see *asking implies waiting*). **Omitted entirely when the run is waiting on a verdict**: the banner after the block carries it instead, and a row saying the same thing beside it splits the reader's attention across two places. |
414
+ | `Follow-ups` | **Always.** `none`, or one line each. |
415
+ | `Next` | **Last.** The single next action for this work — runnable from the state the run leaves behind. |
416
+
417
+ The order runs identity → context → what happened → where it went → what to do.
418
+ **`Tracker` is first** because the id is what addresses this work outside the
419
+ repo, and **`Next` is last** because it is the only row the reader acts on: the
420
+ closing row should be the one that moves the work on.
421
+
422
+ **`Untouched` is a fact, not a caveat.** A run that provisions beside another
423
+ spec's uncommitted work has done nothing wrong and nothing partial — it left
424
+ alone something that was never its business — so it stays `✅` with a row, and
425
+ does not become `⚠️`. Several specs in flight at once is what the worktree mode
426
+ is *for*; spending the "something is worth knowing" verdict on the normal case
427
+ is how that verdict stops meaning anything. It is still reported rather than
428
+ silent, because "I provisioned, and your four files are still sitting there" is
429
+ a different sentence from "I provisioned".
430
+
431
+ **`Review` is one row, not two** — and where the run is waiting, it is no rows
432
+ at all. The page and what to do with it are the same subject, and splitting them
433
+ made the reader resolve a distinction before acting on either. A waiting run
434
+ promotes the whole subject into the banner; keeping the row as well would
435
+ recreate that split with the two halves further apart than ever.
436
+
437
+ **The row carries no question, and that is the rule above applied here.** It
438
+ once ended *"want a written review before you commit?"* — a question addressed
439
+ to a reader the run was not waiting for. Putting it in a findable row fixed
440
+ where it was; it did not fix that nothing was listening. A run that wants that
441
+ question answered waits for it, and waiting means the banner.
442
+
443
+ ## The four verdicts
444
+
445
+ | State | Means |
446
+ |-------|-------|
447
+ | `✅` | Done. Everything the skill set out to do happened. |
448
+ | `⚠️` | Done with caveats — it finished, but something is worth knowing. |
449
+ | `❌` | Failed part-way. It acted, and the repo is mid-something. |
450
+ | `⏸` | Refused before acting. Nothing changed. |
451
+
452
+ **`❌` and `⏸` are different facts about the repo**, and the difference decides
453
+ what happens next. A failed landing leaves a conflicted rebase and a standing
454
+ worktree — there is a mess to clear. A refusal leaves nothing at all, and the
455
+ only thing to do is supply what was missing. Collapsing them sends people
456
+ looking for wreckage that is not there, or past wreckage that is.
457
+
458
+ **A refusal still emits the block.** That is the point of having four states:
459
+ every invocation ends the same way, so an outcome is never inferred from
460
+ silence.
461
+
462
+ ## Where the block may grow
463
+
464
+ One place only. A non-`✅` verdict adds a `Why` row, and quotes the output that
465
+ justifies it — the failing assertion, the conflict, the engine's refusal — in a
466
+ fenced block **above** the table, where a fence means what a fence should mean:
467
+
468
+ ```
469
+ ✖ rejects a negative quantity
470
+ expected 422, got 500
471
+ ```
472
+
473
+ ❌ **Phase 2 stopped — tests red** — `feat-foo`, 2 of 4
474
+
475
+ | | |
476
+ |---|---|
477
+ | **Why** | 3 assertions fail in `test_orders.py`; nothing committed |
478
+ | **Branch** | `spec/feat-foo` · 2 commits, dirty |
479
+ | **Built** | POST /orders handler, orders schema |
480
+ | **Tests** | 125 passed, 3 failed · pytest |
481
+ | **Follow-ups** | none |
482
+ | **Next** | fix the failures, then `/spec-next` to finish phase 2 |
483
+
484
+ Everywhere else the block stays the size it is. Prose after it, an extra row of
485
+ your own invention, a second block — all three are the shape drifting back to
486
+ what this replaced.
487
+
488
+ ## Follow-ups
489
+
490
+ **Always present.** A recorded `none` is a decision; a missing line is an
491
+ oversight — the same logic as the `Gating` header.
492
+
493
+ The bar is deliberately high. A follow-up is something *this* work surfaced
494
+ that *this* spec will not fix — a wrong assumption, a missing guard, a decision
495
+ the spec deferred and the code then leaned on. It is **not** "could be tidier",
496
+ and it is **not** anything already sitting in the spec's Open questions.
497
+
498
+ When there is one, **offer to `/spec` it on the spot**. If the user does not
499
+ take the offer, write one dated line into the spec's Changelog before you
500
+ finish, so it outlives the session:
501
+
502
+ ```
503
+ - 2026-01-09 — Follow-up surfaced: connect's port picker assumes a free
504
+ canonical port; not in scope here.
505
+ ```
506
+
507
+ **The offer needs no caveat about where.** These skills run inside the spec's own
508
+ worktree, and a follow-up spec authored *there* would physically live on that
509
+ branch — invisible from the base branch, and cancelled along with its host. That
510
+ used to make the offer name the primary checkout as the place to write it.
511
+ It no longer does: `/spec` provisions the **new** spec's own worktree before it
512
+ writes anything and moves the session there, so the follow-up lands on its own
513
+ branch and its own ticket without anyone naming a location. Take the offer and
514
+ let `/spec` handle it. Where isolation is absent there is no worktree to move to,
515
+ and the old caveat applies — `spec-planning.md` and `commit-trailers.md` carry
516
+ the reasoning.
517
+
518
+ ## Worked examples
519
+
520
+ **`⚠️` — it finished, with something worth knowing.**
521
+
522
+ ⚠️ **Phase 3 built; tracker not updated** — `feat-foo`, 3 of 4
523
+
524
+ | | |
525
+ |---|---|
526
+ | **Branch** | `spec/feat-foo` · 4 commits, clean |
527
+ | **Built** | Auth middleware, session table |
528
+ | **Tests** | 140 passed · go test ./... |
529
+ | **Follow-ups** | Session expiry is read from two places; phase 4 leans on one |
530
+ | **Next** | `/spec-next` → phase 4 (Docs) |
531
+
532
+ **`⏸` — refused, nothing changed.**
533
+
534
+ ⏸ **`/spec-next` refused — no spec in flight**
535
+
536
+ | | |
537
+ |---|---|
538
+ | **Why** | Not standing in a worktree, and 2 specs are provisioned |
539
+ | **Follow-ups** | none |
540
+ | **Next** | `/spec-start <name>`, or `cd` into `../repo-wt/feat-orders` or `../repo-wt/feat-auth` |
541
+
542
+ **`✅` — a spec finished and landed.**
543
+
544
+ ✅ **Landed and torn down** — `feat-foo`, complete
545
+
546
+ | | |
547
+ |---|---|
548
+ | **Tracker** | [ABC-88](https://example.invalid/ABC-88) · `feat-foo` · moved to its done state |
549
+ | **Spec** | Complete · `specs/complete/feat-foo` |
550
+ | **Tests** | 131 passed · npm test |
551
+ | **Landed** | base fast-forwarded to 4a1c9e2 |
552
+ | **Worktree** | removed · `../repo-wt/feat-foo` |
553
+ | **Follow-ups** | none |
554
+ | **Next** | pick the next spec from `specs/backlog/` |
555
+
556
+ The `❌` example is in **Where the block may grow** above, since the failing
557
+ output it quotes is the thing that example exists to show.