@chris1807/claude-kit 2.1.43 → 2.1.45

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
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "@chris1807/claude-kit",
3
- "version": "2.1.43",
3
+ "version": "2.1.45",
4
4
  "description": "Claude Code starter kit for Azure DevOps teams — agents, hooks, MCP servers, slash commands, and end-to-end work item → PR → release → deploy workflow automation",
5
5
  "type": "module",
6
6
  "bin": {
@@ -7,14 +7,52 @@ This command walks the **backlog** of a chosen Azure DevOps project, finds user
7
7
  3. **Looks at the code when it makes sense** — to validate the described approach and suggest changes.
8
8
  4. **Proposes a story point estimate** — using the same senior-calibrated rubric as `/quote`.
9
9
  5. **Drafts a comment for the item's creator** when issues are found.
10
- 6. **Suggests a rewrite** of the description/acceptance criteria when they need work — offered to the creator in the comment, or applied directly if the user chooses.
10
+ 6. **Suggests a rewrite of the description, and of existing acceptance criteria,** when they need work — offered to the creator in the comment, or applied directly if the user chooses.
11
+
12
+ **Never author acceptance criteria from nothing.** The rewrite improves AC that are *already there* — it never fills an empty AC field. If the item has no acceptance criteria, the deliverable is a comment telling the creator the AC are missing and the item can't be estimated without them. Rewriting someone's AC is editing their intent; writing AC for a blank field is inventing it.
11
13
 
12
14
  **Hard batch limit: 10 items per run.** If more qualify, process the first 10 (by backlog rank) and report how many remain.
13
15
 
14
- **Nothing is written to Azure DevOps — no points, no comments — until the user has seen the full batch and approved.** This command never modifies code and never reassigns items. The only state change it makes: when approved points are written to an item, that item also moves to **Dev Ready** (see Step 5).
16
+ **Nothing is written to Azure DevOps — no points, no comments — until the user has seen the full batch and approved.** This command never modifies code and never reassigns items. It makes exactly two kinds of state change, both in Step 5:
17
+
18
+ - an item that **gets points** moves to **Dev Ready**;
19
+ - an item that **can't be quoted** because information is missing moves back to **Design Review**, so the next sweep doesn't pick it up again while the creator is still working on it.
15
20
 
16
21
  Treat `$ARGUMENTS` as an optional project name (e.g. `/quote-backlog CSI Development`). If provided, skip the project prompt in Step 1.
17
22
 
23
+ ## `--help`
24
+
25
+ If `$ARGUMENTS` contains `--help` or `-h`, print everything between the two rules below **as markdown — the table must render as a table, not inside a code fence** — then **stop**. Run no query, fetch no work items, write nothing.
26
+
27
+ ---
28
+
29
+ **`/quote-backlog [project]`** — sweep a backlog for Design Approved items with no Story Points.
30
+
31
+ Reviews each item for completeness and duplicate work, proposes points, and drafts feedback for the item's creator. Max 10 items per run, in backlog-rank order.
32
+
33
+ Nothing is written to Azure DevOps until you approve the batch — every option below is a fully reversible choice.
34
+
35
+ | You type | What happens |
36
+ |----------|--------------|
37
+ | `all` | Everything proposed gets applied |
38
+ | `1,2,4` | Only those numbered items get applied; the rest are recorded as skipped |
39
+ | `edit N` | I pause on item N so you can change my proposal, then re-show it |
40
+ | `skip N` | Item N is dropped from the batch — nothing written, so it stays in Design Approved and shows up again next sweep |
41
+ | `apply rewrite N` | Item N's rewrite is written **onto the work item** instead of only suggested |
42
+ | `cancel` | Stop with zero changes |
43
+
44
+ Numbers refer to the `#` column of the summary table, not to AB# ids — so `1,2,10` acts on items 1, 2 and 10 of the batch and leaves everything else untouched.
45
+
46
+ **What approval writes:**
47
+ - **Story Points** on items that got a number — each also moves from Design Approved to **Dev Ready**
48
+ - **Design Review** on items that couldn't be quoted — anything blocked on missing information goes back to the creator's queue so the next sweep skips it
49
+ - **Comments** to the creators of items where something was found
50
+ - **Rewrites** — description, and a tightened version of AC that already exist — only for items you explicitly marked `apply rewrite N`
51
+
52
+ **AC are never written from scratch.** Existing acceptance criteria can be rewritten to be clearer or testable, but an empty AC field is left empty — the creator is told it's missing and that the item can't be estimated without it.
53
+
54
+ ---
55
+
18
56
  ## Step 1: Choose the Azure DevOps Project
19
57
 
20
58
  ### Detect the default project
@@ -85,7 +123,7 @@ Score the item against this checklist:
85
123
  |-------|---------------------------|
86
124
  | **Title prefix** | Follows the project's `PREFIX - Title` convention (e.g. `COM -`, `PAY -`) |
87
125
  | **Description** | States what is being built and why — not just a one-line restatement of the title |
88
- | **Acceptance criteria** | Present, testable, unambiguous. Each AC could become a UAT step |
126
+ | **Acceptance criteria** | Present, testable, unambiguous. Each AC could become a UAT step. Record whether the field is **empty** or merely **weak** — that distinction decides whether AC can be rewritten (3g) or must go back to the creator (3f) |
89
127
  | **Design artifacts** | For UI work: a mockup, screenshot, or design link is attached or referenced (the item is Design Approved — the design should be findable) |
90
128
  | **Scope** | Small enough to point (would land at ≤ 21); no hidden second feature buried in the AC |
91
129
  | **Dependencies** | External dependencies or blockers are named, not implied |
@@ -135,6 +173,13 @@ Use the **modified Fibonacci scale**: `1, 2, 3, 5, 8, 13, 21`. Anything larger t
135
173
 
136
174
  Fold in what 3d found — code reconnaissance that shrinks or grows the work changes the number. For items classified **Blocking gaps** or **appears already implemented**, propose **no points** — the comment is the deliverable for those.
137
175
 
176
+ **An item that gets no points goes back to Design Review.** Whenever the reason for not pointing an item is *we need more information* — missing AC, contradictory description, unclear scope, or a possible duplicate the creator has to confirm — record that the item should move from `Design Approved` back to `Design Review` alongside its comment. That state is what keeps the next `/quote-backlog` run from re-analyzing an item that is still waiting on its creator; leaving it in `Design Approved` guarantees it comes back in the next batch unchanged.
177
+
178
+ Two cases are the exception — no state change, because nothing is missing:
179
+
180
+ - **Needs to be split** (the work is understood, it's just too big) — the item stays in `Design Approved`; the deliverable is the split proposal.
181
+ - The user chooses to leave it alone at the Step 4 gate.
182
+
138
183
  ### 3f. Draft the creator comment (only if issues were found)
139
184
 
140
185
  If 3b–3d surfaced anything — gaps, a duplicate, a suggested approach change — draft a comment addressed to the item's creator (`System.CreatedBy`). Format:
@@ -144,22 +189,28 @@ If 3b–3d surfaced anything — gaps, a duplicate, a suggested approach change
144
189
 
145
190
  {One line per finding, concrete and actionable:}
146
191
  - Acceptance criteria don't cover {X} — what should happen when {Y}?
192
+ - AC #{n} isn't testable as written ({why}) — what does "done" look like for it?
193
+ - The AC are missing the steps for {flow} — {what a tester couldn't verify from them}
147
194
  - This looks already implemented in AB#{id} / PR #{n} ({file or feature}) — can you confirm it's still needed?
148
195
  - Suggested approach change: {what the code shows, what to do instead}
149
196
 
150
- {Closing line: what's needed to make it estimable, or "Estimated at {n} points assuming {assumption} — correct me if that's wrong."}
197
+ {Closing line: what's needed to make it estimable — and, when the item is going back to Design Review, say so plainly: "Moving this back to Design Review until that's answered — ping me and I'll re-quote it." Or, when it was pointed: "Estimated at {n} points assuming {assumption} — correct me if that's wrong."}
151
198
  ```
152
199
 
200
+ **When the AC field is empty, report it — don't fill it.** Say the acceptance criteria are missing and that the item can't be estimated without them, and ask the creator for them. Never follow that with a drafted list, a "here's a starting point:" section, or criteria inferred from the title. Existing AC are a different case — those can be rewritten in 3g.
201
+
153
202
  Keep it professional and brief — findings only, no filler. Items with no issues get **no comment**; don't post "looks good" noise.
154
203
 
155
204
  ### 3g. Draft a suggested rewrite (when needed)
156
205
 
157
- If the completeness review found the **description or acceptance criteria** to be vague, contradictory, or structurally weak — not just missing one detail — draft a full rewrite that preserves the creator's intent:
206
+ If the completeness review found the **description** or the **existing acceptance criteria** to be vague, contradictory, or structurally weak — not just missing one detail — draft a rewrite that preserves the creator's intent:
158
207
 
159
208
  - **Description**: what is being built, why, and for whom — written from what the item, its links, and the code reconnaissance establish. Never invent requirements; where intent is unknowable, leave an explicit `{question for creator}` placeholder instead of guessing.
160
- - **Acceptance criteria**: a numbered, testable list each criterion something UAT could verify.
209
+ - **Acceptance criteria** — **only when the item already has some.** Restructure what's there into a numbered, testable list: split compound criteria, make vague ones verifiable, drop duplicates, and flag contradictions. Every criterion must trace back to something the creator already wrote; a gap the existing AC don't address stays a `{question for creator}` placeholder, not a new criterion you supply.
161
210
  - **Title**: only if it violates the `PREFIX - Title` convention or misdescribes the work.
162
211
 
212
+ **An empty AC field is never filled.** If the item has no acceptance criteria at all, the rewrite covers description and title only — leave AC out of the suggested text entirely and let the comment (3f) ask the creator to write them. Don't route around this by putting criteria in the description as a "should" list.
213
+
163
214
  The rewrite is a *suggestion*: by default it travels inside the creator comment (3f) under a "Suggested rewrite:" heading so the creator stays in control of their item. It is applied directly to the work item only if the user explicitly chooses that in Step 4. Skip this for items that are Complete or only missing a one-line answer — a rewrite should earn its place.
164
215
 
165
216
  ### Ultracode mode (optional fan-out)
@@ -176,17 +227,20 @@ Show the whole batch **before writing anything**. Start with the summary table:
176
227
  Quote sweep — {project} backlog, Design Approved without Story Points
177
228
  Batch: {n} of {total} qualifying items{ — run /quote-backlog again for the next 10}
178
229
 
179
- | # | ID | Title | Completeness | Points | Comment | Rewrite |
180
- |----|----------|----------------------------------------|---------------|--------|---------|---------|
181
- | 1 | AB#4611 | COM - Payment reminder emails | Complete | 5 | — | — |
182
- | 2 | AB#4614 | COM - Bulk close inactive accounts | Minor gaps | 8 | yes | — |
183
- | 3 | AB#4617 | PAY - Refund webhook handling | Blocking gaps | — | yes | yes |
184
- | 4 | AB#4620 | COM - Export audit log | Already done? | — | yes | — |
230
+ | # | ID | Title | Completeness | Points | State | Comment | Rewrite |
231
+ |----|----------|------------------------------------|---------------|--------|-----------------|---------|---------|
232
+ | 1 | AB#4611 | COM - Payment reminder emails | Complete | 5 | → Dev Ready | — | — |
233
+ | 2 | AB#4614 | COM - Bulk close inactive accounts | Minor gaps | 8 | → Dev Ready | yes | — |
234
+ | 3 | AB#4617 | PAY - Refund webhook handling | Blocking gaps | — | → Design Review | yes | yes |
235
+ | 4 | AB#4620 | COM - Export audit log | Already done? | — | → Design Review | yes | — |
236
+ | 5 | AB#4623 | COM - Rebuild the reporting module | Too large | split | unchanged | yes | — |
185
237
  ```
186
238
 
239
+ The **State** column is what will actually be written: `→ Dev Ready` for pointed items, `→ Design Review` for items that can't be quoted until the creator supplies something, and `unchanged` for anything else (a split proposal, or an item already past Dev Ready).
240
+
187
241
  Then a detail block per item — estimate reasoning (2–3 bullets), completeness findings, duplicate evidence with links/IDs, code notes, the **full text of any draft comment**, and the **full text of any suggested rewrite**. The user must be able to read every word that would be posted.
188
242
 
189
- Note above the prompt: **items that get points will also move to Dev Ready** approving the points approves the state change.
243
+ Note above the prompt: **approving an item approves its state change too** — pointed items move to Dev Ready, and items that couldn't be quoted move back to Design Review so the next sweep skips them. Name the items in each group so the user can see exactly which ones leave `Design Approved` and in which direction.
190
244
 
191
245
  Then ask:
192
246
 
@@ -196,11 +250,11 @@ Approve? (all / numbers e.g. "1,2,4" / edit N / skip N / apply rewrite N / cance
196
250
 
197
251
  **Wait for the user.**
198
252
 
199
- - `all` → apply every proposed write (points and comments) in Step 5; rewrites stay inside the comments as suggestions
253
+ - `all` → apply every proposed write (points, state changes, and comments) in Step 5; rewrites stay inside the comments as suggestions
200
254
  - `1,2,4` → apply only those items; the rest are recorded as skipped
201
255
  - `edit N` → ask what to change on item N (points value, comment text, or rewrite text), revise, re-show that item, ask again
202
- - `apply rewrite N` → write item N's rewrite directly onto the work item in Step 5 (instead of only suggesting it in the comment)
203
- - `skip N` → drop item N, re-ask for the rest
256
+ - `apply rewrite N` → write item N's rewrite directly onto the work item in Step 5 (instead of only suggesting it in the comment). This covers the description, the title if the rewrite included one, and rewritten AC **only where the item already had AC** — an empty AC field is never populated, under this or any other option
257
+ - `skip N` → drop item N, re-ask for the rest. A skipped item gets **nothing** written — no comment, and no move to Design Review, so it stays in `Design Approved` and will reappear in the next sweep. Say that out loud when confirming a skip, so the user isn't surprised to see it again
204
258
  - `cancel` → stop with **zero changes** to Azure DevOps
205
259
 
206
260
  ## Step 5: Apply Approved Changes
@@ -208,8 +262,19 @@ Approve? (all / numbers e.g. "1,2,4" / edit N / skip N / apply rewrite N / cance
208
262
  Only for approved items, in batch order:
209
263
 
210
264
  1. **Set Story Points and move to Dev Ready** (items with a proposed number): in one `mcp__azure-devops__wit_update_work_item` call, set `Microsoft.VSTS.Scheduling.StoryPoints` **and** `System.State` = `Dev Ready`. The state change applies only to `User Story`, `Bug`, and `Hot Fix` types, and never moves an item backward — if an item is somehow already past Dev Ready, set the points only and note it. Touch no other field — assignee, iteration, and tags stay as they are.
211
- 2. **Post the comment** (items with an approved draft): add it with `mcp__azure-devops__wit_add_work_item_comment` (or the server's work-item comment tool). Use the mention syntax the server supports so the creator is notified; otherwise lead with their display name as drafted.
212
- 3. **Apply the rewrite** (only items the user marked `apply rewrite N`): update `System.Description` and/or `Microsoft.VSTS.Common.AcceptanceCriteria` via `wit_update_work_item`, and adjust the comment to say the rewrite was applied ("rewrote the description/AC per the above — please review") rather than suggesting it. Never apply a rewrite the user didn't explicitly mark.
265
+ 2. **Move unquotable items back to Design Review** (approved items with **no** proposed points, where the blocker is missing information): set `System.State` = `Design Review` via `mcp__azure-devops__wit_update_work_item`. Do this **before** posting the comment in step 3, so the creator's notification arrives with the item already back in their queue. Rules:
266
+
267
+ - Only for `User Story`, `Bug`, and `Hot Fix` types.
268
+ - Only from `Design Approved` — **never move an item backward past the design stage.** An item at `Dev Ready`, or anything at `Active` or later (`Active`, `Code Review`, `Ready for Testing`, `Testing`, `Ready to Deploy`, …), keeps its state; note it in the summary instead.
269
+ - Skip items whose only finding is **needs to be split** — the work is understood, so nothing is missing; leave those in `Design Approved`.
270
+ - If the item is already in `Design Review`, this is a no-op — post the comment and move on.
271
+ - If the project's process template has no `Design Review` state (the update returns an invalid-state error), fall back in this order: `In Design` → `New` → leave the state alone and **warn the user** that the item will be picked up again by the next sweep. Do not silently swallow the error.
272
+ - Touch no other field — points stay empty, assignee, iteration, and tags stay as they are.
273
+
274
+ 3. **Post the comment** (items with an approved draft): add it with `mcp__azure-devops__wit_add_work_item_comment` (or the server's work-item comment tool). Use the mention syntax the server supports so the creator is notified; otherwise lead with their display name as drafted. For an item that just moved to Design Review, the comment must say so — the creator needs to know why it left their Design Approved column.
275
+ 4. **Apply the rewrite** (only items the user marked `apply rewrite N`): update `System.Description` (and `System.Title` if the rewrite included one) via `wit_update_work_item`, and adjust the comment to say the rewrite was applied ("rewrote the description/AC per the above — please review") rather than suggesting it. Never apply a rewrite the user didn't explicitly mark.
276
+
277
+ **Guard on `Microsoft.VSTS.Common.AcceptanceCriteria`:** write it only if the item's AC field was **non-empty** when fetched in 3a. Re-check the fetched value at write time — if it was blank, drop AC from the update payload and write the other fields. Blank means no criteria at all: empty string, whitespace, or an empty HTML shell like `<div></div>` or `<p><br></p>`.
213
278
 
214
279
  If a write fails, report the failure and ask whether to continue with the remaining items or stop.
215
280
 
@@ -220,22 +285,28 @@ If a write fails, report the failure and ask whether to continue with the remain
220
285
 
221
286
  Items analyzed: {n} (of {total} qualifying — {remaining} left for the next run)
222
287
  ✓ Points set: {n_pointed} (total {sum} pts — each moved to Dev Ready)
288
+ ↩ Design Review: {n_design_review} not quoted — moved back to the creator, out of the next sweep
223
289
  ✓ Comments: {n_comments} posted to creators
224
290
  ✓ Rewrites: {n_rewrites_applied} applied, {n_rewrites_suggested} suggested in comments
225
- Skipped: {n_skipped} ({reasons: user skipped / blocking gaps / possible duplicate})
291
+ AC missing: {n_ac_missing} items sent back to the creator to write their acceptance criteria
292
+ ⏭ Skipped: {n_skipped} left in Design Approved — will reappear next sweep ({reasons})
226
293
 
227
294
  Pointed items:
228
295
  - AB#4611: 5 pts
229
296
  - AB#4614: 8 pts (comment posted)
230
297
  - ...
231
298
 
232
- Flagged for the creator (no points yet):
299
+ Moved back to Design Review (no points — waiting on the creator):
233
300
  - AB#4617: blocking gaps — AC missing failure cases
234
- - AB#4620: possibly already implemented in AB#4102
301
+ - AB#4620: possibly already implemented in AB#4102 — needs confirmation
302
+
303
+ Left in Design Approved:
304
+ - AB#4623: too large to point — split proposal in the comment
305
+ - AB#4625: you skipped it
235
306
 
236
307
  Next steps:
237
308
  /quote-backlog {project} — process the next 10 qualifying items
238
309
  /quote AB#{id} — re-estimate a single item after the creator responds
239
310
  ```
240
311
 
241
- Do not create tasks or assign items — those are downstream decisions. Pointed items are now Dev Ready, so `/plan-backlog` picks them up on its next run. Make no state change other than the points→Dev Ready move described in Step 5.
312
+ Do not create tasks or assign items — those are downstream decisions. Pointed items are now Dev Ready, so `/plan-backlog` picks them up on its next run. Items moved to Design Review are out of the `Design Approved` query, so the next `/quote-backlog` run reaches genuinely new items instead of re-reviewing the ones still waiting on their creator — re-quote one with `/quote AB#{id}` once they respond, or let them move it back to Design Approved themselves. Make no state change other than the two described in Step 5.
@@ -34,6 +34,8 @@ Adjust downward for: pure config changes, mechanical refactors with good test co
34
34
 
35
35
  If the item has child items, size each child as well and present the parent's total as the sum (rounded to the nearest Fibonacci number).
36
36
 
37
+ **If the item can't be estimated at all, don't guess a number.** An item is *not estimable* when the gap is information only its creator can close — no acceptance criteria, a description that contradicts itself, scope you can't bound, or work that looks like it may already be done under another ticket. Report it as **needs more information**, list exactly what's missing, and go to Step 5 instead of Step 4. This is different from **needs to be split**: a split candidate is understood work that's simply too big, and it keeps its current state.
38
+
37
39
  ## Step 3: Display the Estimate
38
40
 
39
41
  Output a compact estimate — not a quote block. Format:
@@ -64,8 +66,20 @@ If an existing `Story Points` value is already set on the work item and your est
64
66
 
65
67
  If the work looks larger than 21 points, do not assign a number — instead report `**Estimate:** needs to be split` and suggest 2–4 candidate split points based on the acceptance criteria.
66
68
 
69
+ If the item isn't estimable, report that instead of a number:
70
+
71
+ ```
72
+ **AB#{id}: {title}**
73
+ **Estimate:** needs more information
74
+
75
+ **Blocking:**
76
+ - {what's missing, and what you'd need to know to size it}
77
+ ```
78
+
67
79
  ## Step 4: Offer to Persist (Points + Dev Ready)
68
80
 
81
+ This step applies when you produced a **number**. If the item was reported as *needs more information*, skip to Step 5.
82
+
69
83
  After displaying the estimate, ask the user: *"Want me to set Story Points = {n} on AB#{id}? (This will also move it to Dev Ready.)"*
70
84
 
71
85
  Only update the work item if the user explicitly says yes. Do not modify anything otherwise.
@@ -76,3 +90,23 @@ When the user agrees, in the **same** `wit_update_work_item` call:
76
90
  2. Set `System.State` to `Dev Ready` — **but only if** the work item type is `User Story`, `Bug`, or `Hot Fix` **and** the item is not already past Dev Ready in the workflow (e.g. `Active`, `Code Review`, `Ready to Deploy`). Never move an item backward — if it's already past Dev Ready, set the points only and mention the state was left alone.
77
91
 
78
92
  **Never change the state of a Feature or a Task** — if the sized item is a Feature, persist points only (or per-child points on the children, each of which does get Dev Ready if it qualifies).
93
+
94
+ ## Step 5: Offer to Send Unquotable Items Back (Design Review)
95
+
96
+ When the item came out of Step 2 as **needs more information**, offer to move it back so it drops out of the estimating queue:
97
+
98
+ > *"I can't size this until {what's missing}. Want me to move AB#{id} back to Design Review so it's out of the next `/quote-backlog` sweep until the creator responds?"*
99
+
100
+ **Wait for the user.** Only update the work item if they say yes.
101
+
102
+ When they agree, set `System.State` = `Design Review` with `mcp__azure-devops__wit_update_work_item`. Rules:
103
+
104
+ - Only for `User Story`, `Bug`, and `Hot Fix` types — **never** a Feature or a Task.
105
+ - **Never move an item backward past the design stage.** Only from `Design Approved`; if it's already `Dev Ready` or anything at `Active` or later (`Active`, `Code Review`, `Ready for Testing`, `Testing`, `Ready to Deploy`, …), leave the state alone and say so — someone is already working on it, and a state bounce there does real damage.
106
+ - Already in `Design Review` → no-op; say it's already there.
107
+ - No `Design Review` state in the process template (the update returns an invalid-state error) → fall back in this order: `In Design` → `New` → leave the state alone and **warn** that the item will keep surfacing in backlog sweeps. Don't swallow the error.
108
+ - Set **no** story points and touch no other field — assignee, iteration, and tags stay as they are.
109
+
110
+ Why it matters: `/quote-backlog` sweeps `Design Approved` items with no points. An item left in `Design Approved` while waiting on its creator gets re-analyzed in every sweep, so moving it to `Design Review` is what keeps the next run reaching new items. Re-quote it with `/quote AB#{id}` once the creator responds.
111
+
112
+ **Items that only need to be split keep their state** — the work is understood, so there's nothing to send back. Same for an item where you got a number: that path ends at Step 4.
@@ -112,6 +112,7 @@ These rules apply to **every** command or flow that creates or estimates work it
112
112
  1. **Every work item Claude creates gets a proposed story point estimate** — User Stories and Bugs are never created unpointed by default. Estimates use the modified Fibonacci scale (`1, 2, 3, 5, 8, 13, 21`), calibrated for a **senior developer working with Claude assistance** — no ramp-up padding; pad only for what seniority + Claude can't shortcut (novel work, missing AC, cross-team coordination, external dependencies).
113
113
  2. **The user must agree before points are written.** Claude proposes the number with a one-line rationale; the user confirms, adjusts (their number wins), or skips. Points are never set silently.
114
114
  3. **Setting points moves the item to Dev Ready.** Any time story points are written to a **User Story**, **Bug**, or **Hot Fix**, `System.State` is set to `Dev Ready` in the same update. Never for Features or Tasks (a Feature's state is never touched; Tasks carry hour estimates, not points), and never backward — an item already past Dev Ready keeps its state, with a note.
115
+ 4. **An item that can't be quoted goes back to Design Review.** When `/quote` or `/quote-backlog` can't produce a number because information is missing — no acceptance criteria, contradictory description, unbounded scope, or a possible duplicate the creator has to confirm — the item moves from `Design Approved` back to `Design Review` along with the feedback comment. This keeps the next backlog sweep from re-analyzing items still waiting on their creator. Same guards as rule 3: `User Story` / `Bug` / `Hot Fix` only, and only from `Design Approved` — an item at `Dev Ready` or anything `Active` or later keeps its state. An item that merely **needs to be split** keeps its state — the work is understood, nothing is missing.
115
116
 
116
117
  ### Branching Strategy
117
118
 
@@ -252,8 +253,8 @@ All deployment and release operations are available as slash commands:
252
253
  | `/status` | `/status release 24` | Check release, pipeline, or work item status |
253
254
  | `/plan-backlog` | `/plan-backlog [project]` | Sweep backlog for Dev Ready stories with points and no tasks → propose child tasks with hours |
254
255
  | `/plan-sprint` | `/plan-sprint [project]` | Sweep the current sprint for stories/bugs with no child tasks → propose one child task with hours per item |
255
- | `/quote-backlog` | `/quote-backlog [project]` | Sweep backlog for Design Approved items without points → review completeness, check for duplicates, suggest rewrites, propose points + creator comments (10 at a time, approval-gated; approved points also move the item to Dev Ready) |
256
- | `/quote` | `/quote AB#1234` | Estimate story points for one work item; on approval, sets the points and moves the item to Dev Ready |
256
+ | `/quote-backlog` | `/quote-backlog [project]` | Sweep backlog for Design Approved items without points → review completeness, check for duplicates, suggest rewrites, propose points + creator comments (10 at a time, approval-gated). Pointed items move to Dev Ready; items that can't be quoted move back to **Design Review** so the next sweep skips them |
257
+ | `/quote` | `/quote AB#1234` | Estimate story points for one work item; on approval, sets the points and moves the item to Dev Ready. If it can't be estimated, offers to move it back to **Design Review** |
257
258
  | `/create-work-item` | `/create-work-item [description]` | Interactively draft and create a Bug or User Story — proposes story points (user must agree) and creates pointed items in Dev Ready |
258
259
  | `/cleanup-branches` | `/cleanup-branches` | Delete merged branches |
259
260
  | `/close-orphan-tasks` | `/close-orphan-tasks [scope] [--dry-run]` | Close open Tasks whose parent is Ready to Deploy / Deployed / Closed |