@deskwork/studio 0.17.1 → 0.19.0

package/dist/pages/dashboard/affordances.d.ts

@@ -6,37 +6,32 @@
  * carry a `data-copy` payload that the existing studio client copies
  * to the clipboard. No new backend handlers are wired here; the
  * universal-verb skills are the canonical action path.
+ *
+ * Per DESKWORK-STATE-MACHINE.md (v5):
+ * - Verbs are stage-gated, not state-gated. The iterate button shows
+ *   on stages-that-permit-edits (Ideas/Planned/Outlining/Drafting),
+ *   not on Final (locked) or Published (immutable).
+ * - Revisions (the iteration counter) are bookkeeping. They do not
+ *   surface in routine UI; the operator only sees them via revision
+ *   history / revert flows. The previous "iteration: N" per-row
+ *   display has been removed.
  */
 import { type RawHtml } from '../html.ts';
-import type { Entry, ReviewState } from '@deskwork/core/schema/entry';
-/**
- * Render the reviewState badge. When the entry has no reviewState
- * (most pre-review stages), render an em-dash placeholder so the row
- * stays grid-aligned with sibling rows that DO carry a badge.
- */
-export declare function renderReviewStateBadge(state: ReviewState | undefined): RawHtml;
-/**
- * Iteration count for the entry's current stage. The sidecar's
- * `iterationByStage` records every stage the entry has touched; this
- * surfaces the count for the stage the entry is currently in. Defaults
- * to 0 when the bucket is missing (a brand-new entry on its first tick
- * before any iterate has fired).
- */
-export declare function iterationForCurrentStage(entry: Entry): number;
+import type { Entry } from '@deskwork/core/schema/entry';
 /**
  * Build the per-row action strip. Affordances vary by stage:
  *
- * - Linear pipeline stages (Ideas / Planned / Outlining / Drafting /
- *   Final): "open →" link to the review surface, plus an "iterate"
- *   copy-CLI button when reviewState is `iterating` and an
- *   "approve" copy-CLI button when reviewState is `approved`.
- * - Published: "view →" (read-only review surface).
+ * - Stages-that-permit-edits (Ideas / Planned / Outlining / Drafting):
+ *   "open →" link, "iterate →" copy-CLI button (always; no state gate),
+ *   "approve →" copy-CLI button, "cancel ⊘" copy-CLI button.
+ * - Final: "open →", "approve →" (graduates to Published; assigns a
+ *   version per the operator's scheme), "cancel ⊘". NO iterate
+ *   (Final locks content per the spec).
+ * - Published: "view →" (read-only review surface). Future enhancement:
+ *   surface a "revise" affordance that inducts back to Drafting.
  * - Blocked / Cancelled: "induct →" copy-CLI to bring the entry back.
  *
- * All non-terminal stages also get a `scrapbook ↗` link (#157) — the
- * scrapbook viewer is the primary surface for entry-attached research
- * notes / images / config / drafts and was previously unreachable
- * from the dashboard.
+ * All non-terminal stages also get a `scrapbook ↗` link (#157).
  *
  * Each button's behavior is parked behind a `data-copy` attribute so
  * the existing studio client (editorial-studio-client.ts) handles

package/dist/pages/dashboard/affordances.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"affordances.d.ts","sourceRoot":"","sources":["../../../src/pages/dashboard/affordances.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,OAAO,EAAgB,KAAK,OAAO,EAAE,MAAM,YAAY,CAAC;AAExD,OAAO,KAAK,EAAE,KAAK,EAAS,WAAW,EAAE,MAAM,6BAA6B,CAAC;AAQ7E;;;;GAIG;AACH,wBAAgB,sBAAsB,CAAC,KAAK,EAAE,WAAW,GAAG,SAAS,GAAG,OAAO,CAK9E;AAED;;;;;;GAMG;AACH,wBAAgB,wBAAwB,CAAC,KAAK,EAAE,KAAK,GAAG,MAAM,CAE7D;AAED;;;;;;;;;;;;;;;;;;GAkBG;AACH,wBAAgB,gBAAgB,CAAC,KAAK,EAAE,KAAK,EAAE,WAAW,EAAE,MAAM,GAAG,OAAO,CAkD3E"}
+{"version":3,"file":"affordances.d.ts","sourceRoot":"","sources":["../../../src/pages/dashboard/affordances.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;GAiBG;AAEH,OAAO,EAAgB,KAAK,OAAO,EAAE,MAAM,YAAY,CAAC;AAExD,OAAO,KAAK,EAAE,KAAK,EAAS,MAAM,6BAA6B,CAAC;AAEhE;;;;;;;;;;;;;;;;;;GAkBG;AACH,wBAAgB,gBAAgB,CAAC,KAAK,EAAE,KAAK,EAAE,WAAW,EAAE,MAAM,GAAG,OAAO,CAkE3E"}

package/dist/pages/dashboard/affordances.js

@@ -6,49 +6,32 @@
  * carry a `data-copy` payload that the existing studio client copies
  * to the clipboard. No new backend handlers are wired here; the
  * universal-verb skills are the canonical action path.
+ *
+ * Per DESKWORK-STATE-MACHINE.md (v5):
+ * - Verbs are stage-gated, not state-gated. The iterate button shows
+ *   on stages-that-permit-edits (Ideas/Planned/Outlining/Drafting),
+ *   not on Final (locked) or Published (immutable).
+ * - Revisions (the iteration counter) are bookkeeping. They do not
+ *   surface in routine UI; the operator only sees them via revision
+ *   history / revert flows. The previous "iteration: N" per-row
+ *   display has been removed.
  */
 import { html, unsafe } from "../html.js";
 import { scrapbookViewerUrl } from "../../components/scrapbook-item.js";
-const REVIEW_STATE_LABEL = {
-    'in-review': 'in review',
-    iterating: 'iterating',
-    approved: 'approved',
-};
-/**
- * Render the reviewState badge. When the entry has no reviewState
- * (most pre-review stages), render an em-dash placeholder so the row
- * stays grid-aligned with sibling rows that DO carry a badge.
- */
-export function renderReviewStateBadge(state) {
-    if (state === undefined) {
-        return unsafe('<span class="er-stamp er-stamp-none" data-review-state="none">—</span>');
-    }
-    return unsafe(html `<span class="er-stamp er-stamp-${state}" data-review-state="${state}">${REVIEW_STATE_LABEL[state]}</span>`);
-}
-/**
- * Iteration count for the entry's current stage. The sidecar's
- * `iterationByStage` records every stage the entry has touched; this
- * surfaces the count for the stage the entry is currently in. Defaults
- * to 0 when the bucket is missing (a brand-new entry on its first tick
- * before any iterate has fired).
- */
-export function iterationForCurrentStage(entry) {
-    return entry.iterationByStage[entry.currentStage] ?? 0;
-}
 /**
  * Build the per-row action strip. Affordances vary by stage:
  *
- * - Linear pipeline stages (Ideas / Planned / Outlining / Drafting /
- *   Final): "open →" link to the review surface, plus an "iterate"
- *   copy-CLI button when reviewState is `iterating` and an
- *   "approve" copy-CLI button when reviewState is `approved`.
- * - Published: "view →" (read-only review surface).
+ * - Stages-that-permit-edits (Ideas / Planned / Outlining / Drafting):
+ *   "open →" link, "iterate →" copy-CLI button (always; no state gate),
+ *   "approve →" copy-CLI button, "cancel ⊘" copy-CLI button.
+ * - Final: "open →", "approve →" (graduates to Published; assigns a
+ *   version per the operator's scheme), "cancel ⊘". NO iterate
+ *   (Final locks content per the spec).
+ * - Published: "view →" (read-only review surface). Future enhancement:
+ *   surface a "revise" affordance that inducts back to Drafting.
  * - Blocked / Cancelled: "induct →" copy-CLI to bring the entry back.
  *
- * All non-terminal stages also get a `scrapbook ↗` link (#157) — the
- * scrapbook viewer is the primary surface for entry-attached research
- * notes / images / config / drafts and was previously unreachable
- * from the dashboard.
+ * All non-terminal stages also get a `scrapbook ↗` link (#157).
  *
  * Each button's behavior is parked behind a `data-copy` attribute so
  * the existing studio client (editorial-studio-client.ts) handles
@@ -78,16 +61,32 @@ export function renderRowActions(entry, defaultSite) {
     if (isLinearActiveStage(stage)) {
         buttons.push(html `<a class="er-btn er-btn-small" href="${reviewLink}"
       title="open the review surface for ${entry.slug}">open →</a>`);
-        if (entry.reviewState === 'iterating') {
+        // Iterate — stage-gated per DESKWORK-STATE-MACHINE.md Commandment II:
+        // verbs are stage-gated, never state-gated. Iterate runs on stages
+        // that permit edits (Ideas / Planned / Outlining / Drafting); Final
+        // locks content so iterate is hidden there. The previous
+        // `reviewState === 'iterating'` gate was a Commandment II violation;
+        // removed in the v0.19 spec-conformance sweep.
+        if (stagePermitsEdits(stage)) {
             buttons.push(html `<button class="er-btn er-btn-small er-btn-primary er-copy-btn" type="button"
         data-copy="/deskwork:iterate ${entry.slug}"
         title="operator clicked Iterate — run the iterate skill in Claude Code">iterate →</button>`);
         }
-        if (entry.reviewState === 'approved') {
-            buttons.push(html `<button class="er-btn er-btn-small er-btn-approve er-copy-btn" type="button"
-        data-copy="/deskwork:approve ${entry.slug}"
-        title="approved — graduate to the next stage">approve →</button>`);
-        }
+        // Approve (#244) — unconditional on active-pipeline rows. Mirrors
+        // the review-surface decision strip (Approve is always visible
+        // there); the dashboard previously gated this button on
+        // `reviewState === 'approved'`, which is a transient state most
+        // rows never carry, leaving the button effectively invisible. The
+        // operator should be able to advance any active-pipeline entry
+        // from the dashboard without first opening the review surface.
+        buttons.push(html `<button class="er-btn er-btn-small er-btn-approve er-copy-btn" type="button"
+      data-copy="/deskwork:approve ${entry.slug}"
+      title="advance this entry to the next stage">approve →</button>`);
+        // Cancel (#242) — pull this entry off-pipeline. Same clipboard
+        // routing as approve / iterate; the skill mutates sidecar + journal.
+        buttons.push(html `<button class="er-btn er-btn-small er-btn-reject er-copy-btn" type="button"
+      data-copy="/deskwork:cancel ${entry.slug}"
+      title="pull this entry off-pipeline (Cancelled). Reversible via /deskwork:induct.">cancel ⊘</button>`);
     }
     else if (stage === 'Published') {
         buttons.push(html `<a class="er-btn er-btn-small" href="${reviewLink}"
@@ -111,4 +110,16 @@ function isLinearActiveStage(stage) {
         stage === 'Drafting' ||
         stage === 'Final');
 }
+/**
+ * Stages that permit content edits (and therefore iterate). Final
+ * locks content per DESKWORK-STATE-MACHINE.md — iterate refuses there;
+ * the operator must induct backward to a stage that permits edits if
+ * they want to revise a Final entry.
+ */
+function stagePermitsEdits(stage) {
+    return (stage === 'Ideas' ||
+        stage === 'Planned' ||
+        stage === 'Outlining' ||
+        stage === 'Drafting');
+}
 //# sourceMappingURL=affordances.js.map

package/dist/pages/dashboard/affordances.js.map

@@ -1 +1 @@
-{"version":3,"file":"affordances.js","sourceRoot":"","sources":["../../../src/pages/dashboard/affordances.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,OAAO,EAAE,IAAI,EAAE,MAAM,EAAgB,MAAM,YAAY,CAAC;AACxD,OAAO,EAAE,kBAAkB,EAAE,MAAM,oCAAoC,CAAC;AAGxE,MAAM,kBAAkB,GAAgC;IACtD,WAAW,EAAE,WAAW;IACxB,SAAS,EAAE,WAAW;IACtB,QAAQ,EAAE,UAAU;CACrB,CAAC;AAEF;;;;GAIG;AACH,MAAM,UAAU,sBAAsB,CAAC,KAA8B;IACnE,IAAI,KAAK,KAAK,SAAS,EAAE,CAAC;QACxB,OAAO,MAAM,CAAC,wEAAwE,CAAC,CAAC;IAC1F,CAAC;IACD,OAAO,MAAM,CAAC,IAAI,CAAA,kCAAkC,KAAK,wBAAwB,KAAK,KAAK,kBAAkB,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC;AACjI,CAAC;AAED;;;;;;GAMG;AACH,MAAM,UAAU,wBAAwB,CAAC,KAAY;IACnD,OAAO,KAAK,CAAC,gBAAgB,CAAC,KAAK,CAAC,YAAY,CAAC,IAAI,CAAC,CAAC;AACzD,CAAC;AAED;;;;;;;;;;;;;;;;;;GAkBG;AACH,MAAM,UAAU,gBAAgB,CAAC,KAAY,EAAE,WAAmB;IAChE,MAAM,OAAO,GAAa,EAAE,CAAC;IAC7B,MAAM,KAAK,GAAG,KAAK,CAAC,YAAY,CAAC;IACjC,MAAM,UAAU,GAAG,+BAA+B,KAAK,CAAC,IAAI,EAAE,CAAC;IAC/D,qEAAqE;IACrE,qEAAqE;IACrE,qEAAqE;IACrE,oEAAoE;IACpE,kCAAkC;IAClC,EAAE;IACF,oEAAoE;IACpE,4DAA4D;IAC5D,sEAAsE;IACtE,oDAAoD;IACpD,kEAAkE;IAClE,gDAAgD;IAChD,MAAM,SAAS,GAAG,kBAAkB,CAAC;QACnC,IAAI,EAAE,WAAW;QACjB,IAAI,EAAE,KAAK,CAAC,IAAI;QAChB,OAAO,EAAE,KAAK,CAAC,IAAI;KACpB,CAAC,CAAC;IAEH,IAAI,mBAAmB,CAAC,KAAK,CAAC,EAAE,CAAC;QAC/B,OAAO,CAAC,IAAI,CAAC,IAAI,CAAA,wCAAwC,UAAU;2CAC5B,KAAK,CAAC,IAAI,cAAc,CAAC,CAAC;QACjE,IAAI,KAAK,CAAC,WAAW,KAAK,WAAW,EAAE,CAAC;YACtC,OAAO,CAAC,IAAI,CAAC,IAAI,CAAA;uCACgB,KAAK,CAAC,IAAI;mGACkD,CAAC,CAAC;QACjG,CAAC;QACD,IAAI,KAAK,CAAC,WAAW,KAAK,UAAU,EAAE,CAAC;YACrC,OAAO,CAAC,IAAI,CAAC,IAAI,CAAA;uCACgB,KAAK,CAAC,IAAI;yEACwB,CAAC,CAAC;QACvE,CAAC;IACH,CAAC;SAAM,IAAI,KAAK,KAAK,WAAW,EAAE,CAAC;QACjC,OAAO,CAAC,IAAI,CAAC,IAAI,CAAA,wCAAwC,UAAU;0EACG,CAAC,CAAC;IAC1E,CAAC;SAAM,IAAI,KAAK,KAAK,SAAS,IAAI,KAAK,KAAK,WAAW,EAAE,CAAC;QACxD,OAAO,CAAC,IAAI,CAAC,IAAI,CAAA;oCACe,KAAK,CAAC,IAAI;wEAC0B,CAAC,CAAC;IACxE,CAAC;IAED,uDAAuD;IACvD,OAAO,CAAC,IAAI,CAAC,IAAI,CAAA;YACP,SAAS;sFACiE,CAAC,CAAC;IAEtF,OAAO,MAAM,CAAC,oCAAoC,OAAO,CAAC,IAAI,CAAC,EAAE,CAAC,SAAS,CAAC,CAAC;AAC/E,CAAC;AAED,SAAS,mBAAmB,CAAC,KAAY;IACvC,OAAO,CACL,KAAK,KAAK,OAAO;QACjB,KAAK,KAAK,SAAS;QACnB,KAAK,KAAK,WAAW;QACrB,KAAK,KAAK,UAAU;QACpB,KAAK,KAAK,OAAO,CAClB,CAAC;AACJ,CAAC"}
+{"version":3,"file":"affordances.js","sourceRoot":"","sources":["../../../src/pages/dashboard/affordances.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;GAiBG;AAEH,OAAO,EAAE,IAAI,EAAE,MAAM,EAAgB,MAAM,YAAY,CAAC;AACxD,OAAO,EAAE,kBAAkB,EAAE,MAAM,oCAAoC,CAAC;AAGxE;;;;;;;;;;;;;;;;;;GAkBG;AACH,MAAM,UAAU,gBAAgB,CAAC,KAAY,EAAE,WAAmB;IAChE,MAAM,OAAO,GAAa,EAAE,CAAC;IAC7B,MAAM,KAAK,GAAG,KAAK,CAAC,YAAY,CAAC;IACjC,MAAM,UAAU,GAAG,+BAA+B,KAAK,CAAC,IAAI,EAAE,CAAC;IAC/D,qEAAqE;IACrE,qEAAqE;IACrE,qEAAqE;IACrE,oEAAoE;IACpE,kCAAkC;IAClC,EAAE;IACF,oEAAoE;IACpE,4DAA4D;IAC5D,sEAAsE;IACtE,oDAAoD;IACpD,kEAAkE;IAClE,gDAAgD;IAChD,MAAM,SAAS,GAAG,kBAAkB,CAAC;QACnC,IAAI,EAAE,WAAW;QACjB,IAAI,EAAE,KAAK,CAAC,IAAI;QAChB,OAAO,EAAE,KAAK,CAAC,IAAI;KACpB,CAAC,CAAC;IAEH,IAAI,mBAAmB,CAAC,KAAK,CAAC,EAAE,CAAC;QAC/B,OAAO,CAAC,IAAI,CAAC,IAAI,CAAA,wCAAwC,UAAU;2CAC5B,KAAK,CAAC,IAAI,cAAc,CAAC,CAAC;QACjE,sEAAsE;QACtE,mEAAmE;QACnE,oEAAoE;QACpE,yDAAyD;QACzD,qEAAqE;QACrE,+CAA+C;QAC/C,IAAI,iBAAiB,CAAC,KAAK,CAAC,EAAE,CAAC;YAC7B,OAAO,CAAC,IAAI,CAAC,IAAI,CAAA;uCACgB,KAAK,CAAC,IAAI;mGACkD,CAAC,CAAC;QACjG,CAAC;QACD,kEAAkE;QAClE,+DAA+D;QAC/D,wDAAwD;QACxD,gEAAgE;QAChE,kEAAkE;QAClE,+DAA+D;QAC/D,+DAA+D;QAC/D,OAAO,CAAC,IAAI,CAAC,IAAI,CAAA;qCACgB,KAAK,CAAC,IAAI;sEACuB,CAAC,CAAC;QACpE,+DAA+D;QAC/D,qEAAqE;QACrE,OAAO,CAAC,IAAI,CAAC,IAAI,CAAA;oCACe,KAAK,CAAC,IAAI;2GAC6D,CAAC,CAAC;IAC3G,CAAC;SAAM,IAAI,KAAK,KAAK,WAAW,EAAE,CAAC;QACjC,OAAO,CAAC,IAAI,CAAC,IAAI,CAAA,wCAAwC,UAAU;0EACG,CAAC,CAAC;IAC1E,CAAC;SAAM,IAAI,KAAK,KAAK,SAAS,IAAI,KAAK,KAAK,WAAW,EAAE,CAAC;QACxD,OAAO,CAAC,IAAI,CAAC,IAAI,CAAA;oCACe,KAAK,CAAC,IAAI;wEAC0B,CAAC,CAAC;IACxE,CAAC;IAED,uDAAuD;IACvD,OAAO,CAAC,IAAI,CAAC,IAAI,CAAA;YACP,SAAS;sFACiE,CAAC,CAAC;IAEtF,OAAO,MAAM,CAAC,oCAAoC,OAAO,CAAC,IAAI,CAAC,EAAE,CAAC,SAAS,CAAC,CAAC;AAC/E,CAAC;AAED,SAAS,mBAAmB,CAAC,KAAY;IACvC,OAAO,CACL,KAAK,KAAK,OAAO;QACjB,KAAK,KAAK,SAAS;QACnB,KAAK,KAAK,WAAW;QACrB,KAAK,KAAK,UAAU;QACpB,KAAK,KAAK,OAAO,CAClB,CAAC;AACJ,CAAC;AAED;;;;;GAKG;AACH,SAAS,iBAAiB,CAAC,KAAY;IACrC,OAAO,CACL,KAAK,KAAK,OAAO;QACjB,KAAK,KAAK,SAAS;QACnB,KAAK,KAAK,WAAW;QACrB,KAAK,KAAK,UAAU,CACrB,CAAC;AACJ,CAAC"}

package/dist/pages/dashboard/affordances.ts

@@ -6,55 +6,35 @@
  * carry a `data-copy` payload that the existing studio client copies
  * to the clipboard. No new backend handlers are wired here; the
  * universal-verb skills are the canonical action path.
+ *
+ * Per DESKWORK-STATE-MACHINE.md (v5):
+ * - Verbs are stage-gated, not state-gated. The iterate button shows
+ *   on stages-that-permit-edits (Ideas/Planned/Outlining/Drafting),
+ *   not on Final (locked) or Published (immutable).
+ * - Revisions (the iteration counter) are bookkeeping. They do not
+ *   surface in routine UI; the operator only sees them via revision
+ *   history / revert flows. The previous "iteration: N" per-row
+ *   display has been removed.
  */
 
 import { html, unsafe, type RawHtml } from '../html.ts';
 import { scrapbookViewerUrl } from '../../components/scrapbook-item.ts';
-import type { Entry, Stage, ReviewState } from '@deskwork/core/schema/entry';
-
-const REVIEW_STATE_LABEL: Record<ReviewState, string> = {
-  'in-review': 'in review',
-  iterating: 'iterating',
-  approved: 'approved',
-};
-
-/**
- * Render the reviewState badge. When the entry has no reviewState
- * (most pre-review stages), render an em-dash placeholder so the row
- * stays grid-aligned with sibling rows that DO carry a badge.
- */
-export function renderReviewStateBadge(state: ReviewState | undefined): RawHtml {
-  if (state === undefined) {
-    return unsafe('<span class="er-stamp er-stamp-none" data-review-state="none">—</span>');
-  }
-  return unsafe(html`<span class="er-stamp er-stamp-${state}" data-review-state="${state}">${REVIEW_STATE_LABEL[state]}</span>`);
-}
-
-/**
- * Iteration count for the entry's current stage. The sidecar's
- * `iterationByStage` records every stage the entry has touched; this
- * surfaces the count for the stage the entry is currently in. Defaults
- * to 0 when the bucket is missing (a brand-new entry on its first tick
- * before any iterate has fired).
- */
-export function iterationForCurrentStage(entry: Entry): number {
-  return entry.iterationByStage[entry.currentStage] ?? 0;
-}
+import type { Entry, Stage } from '@deskwork/core/schema/entry';
 
 /**
  * Build the per-row action strip. Affordances vary by stage:
  *
- * - Linear pipeline stages (Ideas / Planned / Outlining / Drafting /
- *   Final): "open →" link to the review surface, plus an "iterate"
- *   copy-CLI button when reviewState is `iterating` and an
- *   "approve" copy-CLI button when reviewState is `approved`.
- * - Published: "view →" (read-only review surface).
+ * - Stages-that-permit-edits (Ideas / Planned / Outlining / Drafting):
+ *   "open →" link, "iterate →" copy-CLI button (always; no state gate),
+ *   "approve →" copy-CLI button, "cancel ⊘" copy-CLI button.
+ * - Final: "open →", "approve →" (graduates to Published; assigns a
+ *   version per the operator's scheme), "cancel ⊘". NO iterate
+ *   (Final locks content per the spec).
+ * - Published: "view →" (read-only review surface). Future enhancement:
+ *   surface a "revise" affordance that inducts back to Drafting.
  * - Blocked / Cancelled: "induct →" copy-CLI to bring the entry back.
  *
- * All non-terminal stages also get a `scrapbook ↗` link (#157) — the
- * scrapbook viewer is the primary surface for entry-attached research
- * notes / images / config / drafts and was previously unreachable
- * from the dashboard.
+ * All non-terminal stages also get a `scrapbook ↗` link (#157).
  *
  * Each button's behavior is parked behind a `data-copy` attribute so
  * the existing studio client (editorial-studio-client.ts) handles
@@ -85,16 +65,32 @@ export function renderRowActions(entry: Entry, defaultSite: string): RawHtml {
   if (isLinearActiveStage(stage)) {
     buttons.push(html`<a class="er-btn er-btn-small" href="${reviewLink}"
       title="open the review surface for ${entry.slug}">open →</a>`);
-    if (entry.reviewState === 'iterating') {
+    // Iterate — stage-gated per DESKWORK-STATE-MACHINE.md Commandment II:
+    // verbs are stage-gated, never state-gated. Iterate runs on stages
+    // that permit edits (Ideas / Planned / Outlining / Drafting); Final
+    // locks content so iterate is hidden there. The previous
+    // `reviewState === 'iterating'` gate was a Commandment II violation;
+    // removed in the v0.19 spec-conformance sweep.
+    if (stagePermitsEdits(stage)) {
       buttons.push(html`<button class="er-btn er-btn-small er-btn-primary er-copy-btn" type="button"
         data-copy="/deskwork:iterate ${entry.slug}"
         title="operator clicked Iterate — run the iterate skill in Claude Code">iterate →</button>`);
     }
-    if (entry.reviewState === 'approved') {
-      buttons.push(html`<button class="er-btn er-btn-small er-btn-approve er-copy-btn" type="button"
-        data-copy="/deskwork:approve ${entry.slug}"
-        title="approved — graduate to the next stage">approve →</button>`);
-    }
+    // Approve (#244) — unconditional on active-pipeline rows. Mirrors
+    // the review-surface decision strip (Approve is always visible
+    // there); the dashboard previously gated this button on
+    // `reviewState === 'approved'`, which is a transient state most
+    // rows never carry, leaving the button effectively invisible. The
+    // operator should be able to advance any active-pipeline entry
+    // from the dashboard without first opening the review surface.
+    buttons.push(html`<button class="er-btn er-btn-small er-btn-approve er-copy-btn" type="button"
+      data-copy="/deskwork:approve ${entry.slug}"
+      title="advance this entry to the next stage">approve →</button>`);
+    // Cancel (#242) — pull this entry off-pipeline. Same clipboard
+    // routing as approve / iterate; the skill mutates sidecar + journal.
+    buttons.push(html`<button class="er-btn er-btn-small er-btn-reject er-copy-btn" type="button"
+      data-copy="/deskwork:cancel ${entry.slug}"
+      title="pull this entry off-pipeline (Cancelled). Reversible via /deskwork:induct.">cancel ⊘</button>`);
   } else if (stage === 'Published') {
     buttons.push(html`<a class="er-btn er-btn-small" href="${reviewLink}"
       title="read-only review surface for the published entry">view →</a>`);
@@ -121,3 +117,18 @@ function isLinearActiveStage(stage: Stage): boolean {
     stage === 'Final'
   );
 }
+
+/**
+ * Stages that permit content edits (and therefore iterate). Final
+ * locks content per DESKWORK-STATE-MACHINE.md — iterate refuses there;
+ * the operator must induct backward to a stage that permits edits if
+ * they want to revise a Final entry.
+ */
+function stagePermitsEdits(stage: Stage): boolean {
+  return (
+    stage === 'Ideas' ||
+    stage === 'Planned' ||
+    stage === 'Outlining' ||
+    stage === 'Drafting'
+  );
+}

package/dist/pages/dashboard/header.d.ts

@@ -2,9 +2,15 @@
  * Dashboard masthead + filter strip.
  *
  * Pipeline-redesign Task 34. The masthead reads from sidecar-derived
- * counts (total entries, in-review entries) instead of the legacy
- * workflow store. The filter strip exposes one chip per stage so
- * operators can collapse to a single section quickly.
+ * counts (total entries) instead of the legacy workflow store. The
+ * filter strip is a search-only row; stage chips were removed in v0.19
+ * (they were never used and added chrome noise). The collapsible stage
+ * tiles on mobile now serve the per-stage navigation role chips used to.
+ *
+ * Review-state-derived stats ("X in review", "X approved") were removed
+ * in v0.19 per operator: review state isn't user-facing data and is
+ * slated for backend removal. The masthead surfaces only stage-shape
+ * counts (date + total) now.
  */
 import type { DashboardData } from './data.ts';
 import { type RawHtml } from '../html.ts';

package/dist/pages/dashboard/header.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"header.d.ts","sourceRoot":"","sources":["../../../src/pages/dashboard/header.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAEH,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,WAAW,CAAC;AAE/C,OAAO,EAAgB,KAAK,OAAO,EAAE,MAAM,YAAY,CAAC;AAwBxD,wBAAgB,YAAY,CAC1B,IAAI,EAAE,aAAa,EACnB,WAAW,EAAE,MAAM,EACnB,GAAG,EAAE,IAAI,GACR,OAAO,CA4BT;AAED,wBAAgB,iBAAiB,IAAI,OAAO,CAa3C"}
+{"version":3,"file":"header.d.ts","sourceRoot":"","sources":["../../../src/pages/dashboard/header.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;GAaG;AAEH,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,WAAW,CAAC;AAC/C,OAAO,EAAgB,KAAK,OAAO,EAAE,MAAM,YAAY,CAAC;AAQxD,wBAAgB,YAAY,CAC1B,IAAI,EAAE,aAAa,EACnB,WAAW,EAAE,MAAM,EACnB,GAAG,EAAE,IAAI,GACR,OAAO,CAwBT;AAED,wBAAgB,iBAAiB,IAAI,OAAO,CAM3C"}

package/dist/pages/dashboard/header.js

@@ -2,39 +2,28 @@
  * Dashboard masthead + filter strip.
  *
  * Pipeline-redesign Task 34. The masthead reads from sidecar-derived
- * counts (total entries, in-review entries) instead of the legacy
- * workflow store. The filter strip exposes one chip per stage so
- * operators can collapse to a single section quickly.
+ * counts (total entries) instead of the legacy workflow store. The
+ * filter strip is a search-only row; stage chips were removed in v0.19
+ * (they were never used and added chrome noise). The collapsible stage
+ * tiles on mobile now serve the per-stage navigation role chips used to.
+ *
+ * Review-state-derived stats ("X in review", "X approved") were removed
+ * in v0.19 per operator: review state isn't user-facing data and is
+ * slated for backend removal. The masthead surfaces only stage-shape
+ * counts (date + total) now.
  */
-import { DASHBOARD_STAGE_ORDER } from "./data.js";
 import { html, unsafe } from "../html.js";
 import { getStudioVersion } from "../../lib/version.js";
 const MONTH_NAMES = [
     'January', 'February', 'March', 'April', 'May', 'June',
     'July', 'August', 'September', 'October', 'November', 'December',
 ];
-function reviewActiveCount(data) {
-    let n = 0;
-    for (const entry of data.entries) {
-        if (entry.reviewState === 'in-review' || entry.reviewState === 'iterating')
-            n++;
-    }
-    return n;
-}
-function approvedCount(data) {
-    let n = 0;
-    for (const entry of data.entries) {
-        if (entry.reviewState === 'approved')
-            n++;
-    }
-    return n;
-}
 export function renderHeader(data, projectRoot, now) {
     const volume = '01';
     const issueNum = String(data.entries.length).padStart(2, '0');
     const issueDate = `${now.getDate()} ${MONTH_NAMES[now.getMonth()]} ${now.getFullYear()}`;
     return unsafe(html `
-  <header class="er-pagehead er-pagehead--centered">
+  <header class="er-pagehead er-pagehead--centered er-pagehead--dashboard">
     <p class="er-pagehead__kicker">
       Vol. ${volume} &middot; № ${issueNum}
     </p>
@@ -51,10 +40,6 @@ export function renderHeader(data, projectRoot, now) {
       <span>${issueDate}</span>
       <span class="sep">·</span>
       <span>${data.entries.length} on the calendar</span>
-      <span class="sep">·</span>
-      <span>${reviewActiveCount(data)} in review</span>
-      <span class="sep">·</span>
-      <span>${approvedCount(data)} approved</span>
     </p>
   </header>`);
 }
@@ -63,11 +48,6 @@ export function renderFilterStrip() {
     <section class="er-filter" data-filter-strip>
       <span class="er-filter-label">Find</span>
       <input type="search" data-filter-input placeholder="slug, title…" autocomplete="off" />
-      <span class="er-filter-label er-filter-label--gap">Stage</span>
-      <div class="er-chips" role="tablist">
-        <button class="er-chip" aria-pressed="true" data-stage-chip="all">all</button>
-        ${DASHBOARD_STAGE_ORDER.map((s) => unsafe(html `<button class="er-chip" data-stage-chip="${s}">${s.toLowerCase()}</button>`))}
-      </div>
     </section>`);
 }
 //# sourceMappingURL=header.js.map

package/dist/pages/dashboard/header.js.map

@@ -1 +1 @@
-{"version":3,"file":"header.js","sourceRoot":"","sources":["../../../src/pages/dashboard/header.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAGH,OAAO,EAAE,qBAAqB,EAAE,MAAM,WAAW,CAAC;AAClD,OAAO,EAAE,IAAI,EAAE,MAAM,EAAgB,MAAM,YAAY,CAAC;AACxD,OAAO,EAAE,gBAAgB,EAAE,MAAM,sBAAsB,CAAC;AAExD,MAAM,WAAW,GAAG;IAClB,SAAS,EAAE,UAAU,EAAE,OAAO,EAAE,OAAO,EAAE,KAAK,EAAE,MAAM;IACtD,MAAM,EAAE,QAAQ,EAAE,WAAW,EAAE,SAAS,EAAE,UAAU,EAAE,UAAU;CACjE,CAAC;AAEF,SAAS,iBAAiB,CAAC,IAAmB;IAC5C,IAAI,CAAC,GAAG,CAAC,CAAC;IACV,KAAK,MAAM,KAAK,IAAI,IAAI,CAAC,OAAO,EAAE,CAAC;QACjC,IAAI,KAAK,CAAC,WAAW,KAAK,WAAW,IAAI,KAAK,CAAC,WAAW,KAAK,WAAW;YAAE,CAAC,EAAE,CAAC;IAClF,CAAC;IACD,OAAO,CAAC,CAAC;AACX,CAAC;AAED,SAAS,aAAa,CAAC,IAAmB;IACxC,IAAI,CAAC,GAAG,CAAC,CAAC;IACV,KAAK,MAAM,KAAK,IAAI,IAAI,CAAC,OAAO,EAAE,CAAC;QACjC,IAAI,KAAK,CAAC,WAAW,KAAK,UAAU;YAAE,CAAC,EAAE,CAAC;IAC5C,CAAC;IACD,OAAO,CAAC,CAAC;AACX,CAAC;AAED,MAAM,UAAU,YAAY,CAC1B,IAAmB,EACnB,WAAmB,EACnB,GAAS;IAET,MAAM,MAAM,GAAG,IAAI,CAAC;IACpB,MAAM,QAAQ,GAAG,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,QAAQ,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC;IAC9D,MAAM,SAAS,GAAG,GAAG,GAAG,CAAC,OAAO,EAAE,IAAI,WAAW,CAAC,GAAG,CAAC,QAAQ,EAAE,CAAC,IAAI,GAAG,CAAC,WAAW,EAAE,EAAE,CAAC;IACzF,OAAO,MAAM,CAAC,IAAI,CAAA;;;aAGP,MAAM,eAAe,QAAQ;;;;;;uBAMnB,WAAW;;;4CAGU,gBAAgB,EAAE;;;cAGhD,SAAS;;cAET,IAAI,CAAC,OAAO,CAAC,MAAM;;cAEnB,iBAAiB,CAAC,IAAI,CAAC;;cAEvB,aAAa,CAAC,IAAI,CAAC;;YAErB,CAAC,CAAC;AACd,CAAC;AAED,MAAM,UAAU,iBAAiB;IAC/B,OAAO,MAAM,CAAC,IAAI,CAAA;;;;;;;UAOV,qBAAqB,CAAC,GAAG,CACzB,CAAC,CAAC,EAAE,EAAE,CAAC,MAAM,CAAC,IAAI,CAAA,4CAA4C,CAAC,KAAK,CAAC,CAAC,WAAW,EAAE,WAAW,CAAC,CAChG;;eAEM,CAAC,CAAC;AACjB,CAAC"}
+{"version":3,"file":"header.js","sourceRoot":"","sources":["../../../src/pages/dashboard/header.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;GAaG;AAGH,OAAO,EAAE,IAAI,EAAE,MAAM,EAAgB,MAAM,YAAY,CAAC;AACxD,OAAO,EAAE,gBAAgB,EAAE,MAAM,sBAAsB,CAAC;AAExD,MAAM,WAAW,GAAG;IAClB,SAAS,EAAE,UAAU,EAAE,OAAO,EAAE,OAAO,EAAE,KAAK,EAAE,MAAM;IACtD,MAAM,EAAE,QAAQ,EAAE,WAAW,EAAE,SAAS,EAAE,UAAU,EAAE,UAAU;CACjE,CAAC;AAEF,MAAM,UAAU,YAAY,CAC1B,IAAmB,EACnB,WAAmB,EACnB,GAAS;IAET,MAAM,MAAM,GAAG,IAAI,CAAC;IACpB,MAAM,QAAQ,GAAG,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,QAAQ,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC;IAC9D,MAAM,SAAS,GAAG,GAAG,GAAG,CAAC,OAAO,EAAE,IAAI,WAAW,CAAC,GAAG,CAAC,QAAQ,EAAE,CAAC,IAAI,GAAG,CAAC,WAAW,EAAE,EAAE,CAAC;IACzF,OAAO,MAAM,CAAC,IAAI,CAAA;;;aAGP,MAAM,eAAe,QAAQ;;;;;;uBAMnB,WAAW;;;4CAGU,gBAAgB,EAAE;;;cAGhD,SAAS;;cAET,IAAI,CAAC,OAAO,CAAC,MAAM;;YAErB,CAAC,CAAC;AACd,CAAC;AAED,MAAM,UAAU,iBAAiB;IAC/B,OAAO,MAAM,CAAC,IAAI,CAAA;;;;eAIL,CAAC,CAAC;AACjB,CAAC"}

package/dist/pages/dashboard/header.ts

@@ -2,13 +2,18 @@
  * Dashboard masthead + filter strip.
  *
  * Pipeline-redesign Task 34. The masthead reads from sidecar-derived
- * counts (total entries, in-review entries) instead of the legacy
- * workflow store. The filter strip exposes one chip per stage so
- * operators can collapse to a single section quickly.
+ * counts (total entries) instead of the legacy workflow store. The
+ * filter strip is a search-only row; stage chips were removed in v0.19
+ * (they were never used and added chrome noise). The collapsible stage
+ * tiles on mobile now serve the per-stage navigation role chips used to.
+ *
+ * Review-state-derived stats ("X in review", "X approved") were removed
+ * in v0.19 per operator: review state isn't user-facing data and is
+ * slated for backend removal. The masthead surfaces only stage-shape
+ * counts (date + total) now.
  */
 
 import type { DashboardData } from './data.ts';
-import { DASHBOARD_STAGE_ORDER } from './data.ts';
 import { html, unsafe, type RawHtml } from '../html.ts';
 import { getStudioVersion } from '../../lib/version.ts';
 
@@ -17,22 +22,6 @@ const MONTH_NAMES = [
   'July', 'August', 'September', 'October', 'November', 'December',
 ];
 
-function reviewActiveCount(data: DashboardData): number {
-  let n = 0;
-  for (const entry of data.entries) {
-    if (entry.reviewState === 'in-review' || entry.reviewState === 'iterating') n++;
-  }
-  return n;
-}
-
-function approvedCount(data: DashboardData): number {
-  let n = 0;
-  for (const entry of data.entries) {
-    if (entry.reviewState === 'approved') n++;
-  }
-  return n;
-}
-
 export function renderHeader(
   data: DashboardData,
   projectRoot: string,
@@ -42,7 +31,7 @@ export function renderHeader(
   const issueNum = String(data.entries.length).padStart(2, '0');
   const issueDate = `${now.getDate()} ${MONTH_NAMES[now.getMonth()]} ${now.getFullYear()}`;
   return unsafe(html`
-  <header class="er-pagehead er-pagehead--centered">
+  <header class="er-pagehead er-pagehead--centered er-pagehead--dashboard">
     <p class="er-pagehead__kicker">
       Vol. ${volume} &middot; № ${issueNum}
     </p>
@@ -59,10 +48,6 @@ export function renderHeader(
       <span>${issueDate}</span>
       <span class="sep">·</span>
       <span>${data.entries.length} on the calendar</span>
-      <span class="sep">·</span>
-      <span>${reviewActiveCount(data)} in review</span>
-      <span class="sep">·</span>
-      <span>${approvedCount(data)} approved</span>
     </p>
   </header>`);
 }
@@ -72,12 +57,5 @@ export function renderFilterStrip(): RawHtml {
     <section class="er-filter" data-filter-strip>
       <span class="er-filter-label">Find</span>
       <input type="search" data-filter-input placeholder="slug, title…" autocomplete="off" />
-      <span class="er-filter-label er-filter-label--gap">Stage</span>
-      <div class="er-chips" role="tablist">
-        <button class="er-chip" aria-pressed="true" data-stage-chip="all">all</button>
-        ${DASHBOARD_STAGE_ORDER.map(
-          (s) => unsafe(html`<button class="er-chip" data-stage-chip="${s}">${s.toLowerCase()}</button>`),
-        )}
-      </div>
     </section>`);
 }

package/dist/pages/dashboard/section.d.ts

@@ -1,12 +1,17 @@
 /**
  * Single-stage section renderer.
  *
- * Pipeline-redesign Task 34. Each of the eight stage sections renders
- * with a section heading (stage name + entry count) and either a list
- * of rows or an empty-state placeholder. Per-row HTML carries the
- * sidecar-derived state inline (iteration count + reviewState badge)
- * so an operator can see at a glance where each entry sits without
- * opening it.
+ * Each of the eight stage sections (plus the Distribution placeholder)
+ * renders with a section heading (stage name + entry count) and either
+ * a list of rows or an empty-state placeholder. Each row carries the
+ * entry's slug, title, updated-at timestamp, and stage-gated verb
+ * buttons. Per DESKWORK-STATE-MACHINE.md Commandment III, rows do NOT
+ * surface iteration counts or reviewState — those were retired in
+ * v0.19 along with the legacy reviewState concept.
+ *
+ * On mobile, each section is fronted by a collapsible tile (see
+ * `renderStageTile`); on desktop the tiles are display:none and the
+ * `<h2 class="er-section-head">` heading carries the stage name.
  */
 import { type RawHtml } from '../html.ts';
 import type { Entry, Stage } from '@deskwork/core/schema/entry';
@@ -14,27 +19,49 @@ import type { Entry, Stage } from '@deskwork/core/schema/entry';
  * Render one entry as a single dashboard row. Carries inline:
  *   - slug (linked to the review surface)
  *   - title
- *   - iteration count for the entry's currentStage
- *   - reviewState badge (or an em-dash placeholder)
  *   - updatedAt timestamp
  *   - per-stage action buttons
+ *
+ * Per DESKWORK-STATE-MACHINE.md (v5): revisions (the iteration counter)
+ * are bookkeeping and do NOT surface in routine UI. The previous
+ * "iteration: N" inline display was a violation — operators see
+ * revisions only via the View History surface and revert flows.
+ * reviewState badges are likewise retired (Commandment III).
  */
 export declare function renderRow(entry: Entry, index: number, defaultSite: string): RawHtml;
 /**
  * Render one full stage section: heading + ornaments + count + rows.
  *
- * Empty stages render compact (just the heading, no placeholder body)
- * — keeps the operator's sense of pipeline shape without padding the
- * dashboard with multi-line empty placeholders for low-volume
+ * The output is wrapped in a `.er-stage-block` div that pairs a mobile-
+ * only stage tile (the collapsible head) with the existing section. On
+ * desktop, the tile is `display: none` and the section's `<h2>` head
+ * carries the heading as before. On mobile, the section's head is hidden
+ * and the tile is shown; tapping the tile toggles a `data-collapsed`
+ * attribute on the section that hides/shows its rows. Single-expand
+ * (tapping one tile collapses the others) is handled by
+ * `dashboard/stage-tiles.ts`.
+ *
+ * Empty stages still render their tile (so the pipeline shape is visible
+ * at-rest on phone) but the empty section body itself is hidden on mobile.
+ *
+ * Empty stages on desktop render compact (just the heading, no placeholder
+ * body) — keeps the operator's sense of pipeline shape without padding
+ * the dashboard with multi-line empty placeholders for low-volume
  * calendars (#112). The hover title still surfaces the stage's
  * "what to run next" hint when the operator points at the heading.
  */
 export declare function renderStageSection(stage: Stage, entries: readonly Entry[], defaultSite: string): RawHtml;
 /**
- * Render the reserved Distribution placeholder. Stays a separate
- * sibling of the stage sections — distribution records (shortform
- * cross-posts) are tracked under their own model and the dashboard
- * surfaces only a placeholder here until that integration lands.
+ * Render the reserved Distribution placeholder. Distribution isn't a
+ * pipeline stage in the formal sense (no entries flow through it; it
+ * lives under its own model when shortform cross-posts arrive), but on
+ * the mobile dashboard it renders as a stage tile alongside the rest
+ * so the operator's pipeline-shape scan stays uniform — see operator
+ * feedback on 2026-05-09. The tile is `is-empty` + `disabled` until
+ * DistributionRecords land in the data layer.
+ *
+ * On desktop, the existing section + heading + placeholder text render
+ * as before; the tile is `display: none` per dashboard-mobile.css.
  */
 export declare function renderDistributionPlaceholder(): RawHtml;
 //# sourceMappingURL=section.d.ts.map

package/dist/pages/dashboard/section.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"section.d.ts","sourceRoot":"","sources":["../../../src/pages/dashboard/section.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAEH,OAAO,EAAgB,KAAK,OAAO,EAAE,MAAM,YAAY,CAAC;AACxD,OAAO,KAAK,EAAE,KAAK,EAAE,KAAK,EAAE,MAAM,6BAA6B,CAAC;AA6BhE;;;;;;;;GAQG;AACH,wBAAgB,SAAS,CAAC,KAAK,EAAE,KAAK,EAAE,KAAK,EAAE,MAAM,EAAE,WAAW,EAAE,MAAM,GAAG,OAAO,CAgCnF;AAED;;;;;;;;GAQG;AACH,wBAAgB,kBAAkB,CAChC,KAAK,EAAE,KAAK,EACZ,OAAO,EAAE,SAAS,KAAK,EAAE,EACzB,WAAW,EAAE,MAAM,GAClB,OAAO,CA0BT;AAED;;;;;GAKG;AACH,wBAAgB,6BAA6B,IAAI,OAAO,CAWvD"}
+{"version":3,"file":"section.d.ts","sourceRoot":"","sources":["../../../src/pages/dashboard/section.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;GAcG;AAEH,OAAO,EAAgB,KAAK,OAAO,EAAE,MAAM,YAAY,CAAC;AACxD,OAAO,KAAK,EAAE,KAAK,EAAE,KAAK,EAAE,MAAM,6BAA6B,CAAC;AAyBhE;;;;;;;;;;;;GAYG;AACH,wBAAgB,SAAS,CAAC,KAAK,EAAE,KAAK,EAAE,KAAK,EAAE,MAAM,EAAE,WAAW,EAAE,MAAM,GAAG,OAAO,CA2BnF;AAgCD;;;;;;;;;;;;;;;;;;;;GAoBG;AACH,wBAAgB,kBAAkB,CAChC,KAAK,EAAE,KAAK,EACZ,OAAO,EAAE,SAAS,KAAK,EAAE,EACzB,WAAW,EAAE,MAAM,GAClB,OAAO,CAkCT;AAED;;;;;;;;;;;GAWG;AACH,wBAAgB,6BAA6B,IAAI,OAAO,CAsBvD"}