@slowcook-ai/cli 0.16.0-alpha.4 → 0.17.0-alpha.1

package/dist/commands/plate/classify.js

@@ -0,0 +1,194 @@
+/**
+ * PM-comment classifier — 0.16.0-α.7.
+ *
+ * Each element-anchored review-overlay comment on a slowcook-mockup PR
+ * gets categorized so plate knows what to do with it:
+ *
+ *   - "cosmetic"      → amend the mock with minimum diff
+ *   - "spec-altering" → ESCALATE: would invalidate a spec assertion or
+ *                       acceptance scenario; PM must confirm the spec
+ *                       change before plate touches the mock
+ *   - "mock-divergence" → the mock diverged from the spec; align mock
+ *                         to spec; note in summary why the PM ask is
+ *                         being interpreted this way
+ *
+ * α.7 ships a deterministic heuristic. Each classification has a clear
+ * rationale string so the escalation comment can quote the trigger.
+ *
+ * Heuristic structure:
+ *   1. Parse the spec YAML to extract the salient assertion targets:
+ *        - acceptance_scenarios prose lines
+ *        - api_contract response field names
+ *        - invariants prose
+ *        - ui_behavior viewport prose
+ *      → a flat set of "spec terms" (lowercased, normalized words).
+ *   2. Score the comment prose against those terms:
+ *        - any direct mention of an acceptance keyword phrase     → spec-altering
+ *        - mention of a domain noun + a "remove/change/replace"   → spec-altering
+ *        - mentions only adjective/style words (color, padding,   → cosmetic
+ *          font, spacing, alignment, shadow, etc.)
+ *        - else                                                   → mock-divergence
+ *
+ * The heuristic is intentionally conservative on spec-altering: false
+ * positives only cost a PM confirm round; false negatives let plate
+ * silently weaken the spec, which is the failure mode the architecture
+ * is designed to prevent. When in doubt → escalate.
+ *
+ * No LLM dep here — pure functions over inputs. LLM-backed classifier
+ * is a future α.7.1 upgrade if heuristic shows real misses.
+ */
+const COSMETIC_WORDS = [
+    "color", "colour", "shade", "tint", "hue", "rgb", "hex",
+    "padding", "margin", "spacing", "gap", "indent",
+    "font", "typeface", "weight", "italic", "underline",
+    "alignment", "align", "center", "centre", "left-aligned", "right-aligned",
+    "shadow", "border", "outline", "radius", "rounded",
+    "size", "smaller", "bigger", "larger", "tighter", "looser",
+    "background", "bg",
+    "icon", "emoji",
+    "feel", "vibe", "look",
+    "ratio", "scale",
+];
+const STRUCTURAL_VERBS = [
+    "remove", "removed", "delete", "deleted", "drop",
+    "replace", "swap", "switch",
+    "add", "introduce", "include",
+    "change", "rename",
+    "split", "merge", "combine",
+    "block", "prevent", "disallow",
+    "require", "enforce",
+];
+export function classifyComment(args) {
+    const proseNorm = normalize(args.prose);
+    const specTerms = extractSpecTerms(args.specYaml);
+    const matched = [];
+    for (const term of specTerms) {
+        if (proseNorm.includes(term))
+            matched.push(term);
+    }
+    const cosmeticHits = COSMETIC_WORDS.filter((w) => containsWord(proseNorm, w));
+    const structuralVerb = STRUCTURAL_VERBS.find((v) => containsWord(proseNorm, v));
+    // Rule 1 — spec term + structural verb → spec-altering. Highest
+    // priority; this is the failure mode we MUST escalate. "Remove the
+    // pinned strip" / "replace pinned with bookmarked" etc.
+    if (matched.length > 0 && structuralVerb) {
+        return {
+            classification: "spec-altering",
+            rationale: `Mentions spec term${matched.length > 1 ? "s" : ""} (${matched.slice(0, 5).map((m) => `"${m}"`).join(", ")}) ` +
+                `together with structural verb "${structuralVerb}". This would change the spec's contract; ` +
+                `PM must confirm before the mock changes.`,
+            matchedSpecTerms: matched,
+        };
+    }
+    // Rule 2 — cosmetic word present (with or without spec term, but
+    // no structural verb) → cosmetic. A PM saying "the Pinned button
+    // background should be coral" is naming the element AND asking for
+    // a style change; that's still cosmetic. Amend the mock with min diff.
+    if (cosmeticHits.length > 0) {
+        return {
+            classification: "cosmetic",
+            rationale: `Style-only feedback (matched: ${cosmeticHits.slice(0, 5).map((w) => `"${w}"`).join(", ")})` +
+                (matched.length > 0
+                    ? `; spec term${matched.length > 1 ? "s" : ""} (${matched.slice(0, 3).map((m) => `"${m}"`).join(", ")}) named the element but no structural verb present.`
+                    : `; no spec terms triggered.`),
+            matchedSpecTerms: matched,
+        };
+    }
+    // Rule 3 — spec terms with no cosmetic word and no structural verb
+    // → mock-divergence. The PM is talking about something the spec
+    // mentions but neither styling it nor changing the contract.
+    // Likely "mock shows X but spec says Y."
+    if (matched.length > 0) {
+        return {
+            classification: "mock-divergence",
+            rationale: `Mentions spec term${matched.length > 1 ? "s" : ""} (${matched.slice(0, 5).map((m) => `"${m}"`).join(", ")}) ` +
+                `without a structural verb or styling cue. Likely the mock diverged from spec; align mock to spec.`,
+            matchedSpecTerms: matched,
+        };
+    }
+    // Rule 4 — fallthrough. No signal in any direction. Default to
+    // mock-divergence so plate's LLM still gets the chance to reason
+    // about both spec + comment in context (false-positive cost is low).
+    return {
+        classification: "mock-divergence",
+        rationale: `No clear spec or style signal in the prose. Defaulting to mock-divergence so plate has a chance to reconcile against the spec.`,
+        matchedSpecTerms: [],
+    };
+}
+/**
+ * Extract candidate "spec terms" — lowercase, deduplicated significant
+ * tokens from the spec sections plate cares about. Used by the
+ * classifier to detect overlap between PM prose and spec assertions.
+ */
+export function extractSpecTerms(specYaml) {
+    const sections = ["acceptance_scenarios", "invariants", "api_contract"];
+    const terms = new Set();
+    // Also include ui_behavior viewport prose — a comment about
+    // "remove the desktop_light pinned strip" is spec-altering.
+    for (const sec of [...sections, "ui_behavior"]) {
+        const body = extractYamlSection(specYaml, sec);
+        if (!body)
+            continue;
+        for (const tok of tokenize(body)) {
+            if (isStopword(tok))
+                continue;
+            if (tok.length < 4)
+                continue; // skip short noise
+            terms.add(tok);
+        }
+    }
+    return Array.from(terms);
+}
+function extractYamlSection(yaml, sectionName) {
+    // Find the line `sectionName:` at indent 0; capture indented body
+    // until the next zero-indent line.
+    const lines = yaml.split(/\r?\n/);
+    const out = [];
+    let inSection = false;
+    for (const raw of lines) {
+        const m = raw.match(/^(\s*)([a-zA-Z_][\w]*)\s*:/);
+        if (m && m[1].length === 0) {
+            if (m[2] === sectionName) {
+                inSection = true;
+                continue;
+            }
+            else if (inSection) {
+                // hit a sibling top-level key
+                break;
+            }
+        }
+        if (inSection)
+            out.push(raw);
+    }
+    return out.join("\n");
+}
+function normalize(s) {
+    return s.toLowerCase().replace(/\s+/g, " ").trim();
+}
+function tokenize(s) {
+    // Split on non-word; preserve underscores (snake_case identifiers)
+    return s
+        .toLowerCase()
+        .split(/[^a-z0-9_]+/)
+        .filter((w) => w.length > 0);
+}
+function containsWord(haystackNorm, needle) {
+    // Word-boundary check; needle is already lowercase.
+    const re = new RegExp(`(^|[^a-z0-9_])${escapeRe(needle)}([^a-z0-9_]|$)`);
+    return re.test(haystackNorm);
+}
+function escapeRe(s) {
+    return s.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
+}
+const STOPWORDS = new Set([
+    "the", "a", "an", "and", "or", "but", "of", "in", "on", "at", "to", "for", "with", "by", "is", "are", "was", "were", "be", "been", "being", "have", "has", "had", "do", "does", "did", "this", "that", "these", "those", "it", "its", "as", "if", "then", "else", "when", "while", "each", "every", "any", "all", "not", "no", "yes", "true", "false", "null", "can", "will", "may", "must", "should", "would", "also", "only", "very", "more", "most", "such", "than", "into", "from", "over", "under", "between", "through", "because", "since", "once", "yaml", "example", "note", "todo", "null_", "status", "approved", "draft", "paused", "active",
+]);
+function isStopword(w) {
+    if (STOPWORDS.has(w))
+        return true;
+    // Pure-numeric tokens are noise
+    if (/^[0-9]+$/.test(w))
+        return true;
+    return false;
+}
+//# sourceMappingURL=classify.js.map

package/dist/commands/plate/classify.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"classify.js","sourceRoot":"","sources":["../../../src/commands/plate/classify.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAsCG;AAYH,MAAM,cAAc,GAAG;IACrB,OAAO,EAAE,QAAQ,EAAE,OAAO,EAAE,MAAM,EAAE,KAAK,EAAE,KAAK,EAAE,KAAK;IACvD,SAAS,EAAE,QAAQ,EAAE,SAAS,EAAE,KAAK,EAAE,QAAQ;IAC/C,MAAM,EAAE,UAAU,EAAE,QAAQ,EAAE,QAAQ,EAAE,WAAW;IACnD,WAAW,EAAE,OAAO,EAAE,QAAQ,EAAE,QAAQ,EAAE,cAAc,EAAE,eAAe;IACzE,QAAQ,EAAE,QAAQ,EAAE,SAAS,EAAE,QAAQ,EAAE,SAAS;IAClD,MAAM,EAAE,SAAS,EAAE,QAAQ,EAAE,QAAQ,EAAE,SAAS,EAAE,QAAQ;IAC1D,YAAY,EAAE,IAAI;IAClB,MAAM,EAAE,OAAO;IACf,MAAM,EAAE,MAAM,EAAE,MAAM;IACtB,OAAO,EAAE,OAAO;CACjB,CAAC;AAEF,MAAM,gBAAgB,GAAG;IACvB,QAAQ,EAAE,SAAS,EAAE,QAAQ,EAAE,SAAS,EAAE,MAAM;IAChD,SAAS,EAAE,MAAM,EAAE,QAAQ;IAC3B,KAAK,EAAE,WAAW,EAAE,SAAS;IAC7B,QAAQ,EAAE,QAAQ;IAClB,OAAO,EAAE,OAAO,EAAE,SAAS;IAC3B,OAAO,EAAE,SAAS,EAAE,UAAU;IAC9B,SAAS,EAAE,SAAS;CACrB,CAAC;AAaF,MAAM,UAAU,eAAe,CAAC,IAAkB;IAChD,MAAM,SAAS,GAAG,SAAS,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IACxC,MAAM,SAAS,GAAG,gBAAgB,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;IAClD,MAAM,OAAO,GAAa,EAAE,CAAC;IAC7B,KAAK,MAAM,IAAI,IAAI,SAAS,EAAE,CAAC;QAC7B,IAAI,SAAS,CAAC,QAAQ,CAAC,IAAI,CAAC;YAAE,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IACnD,CAAC;IAED,MAAM,YAAY,GAAG,cAAc,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,YAAY,CAAC,SAAS,EAAE,CAAC,CAAC,CAAC,CAAC;IAC9E,MAAM,cAAc,GAAG,gBAAgB,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,YAAY,CAAC,SAAS,EAAE,CAAC,CAAC,CAAC,CAAC;IAEhF,gEAAgE;IAChE,mEAAmE;IACnE,wDAAwD;IACxD,IAAI,OAAO,CAAC,MAAM,GAAG,CAAC,IAAI,cAAc,EAAE,CAAC;QACzC,OAAO;YACL,cAAc,EAAE,eAAe;YAC/B,SAAS,EACP,qBAAqB,OAAO,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,KAAK,OAAO,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI;gBAC9G,kCAAkC,cAAc,4CAA4C;gBAC5F,0CAA0C;YAC5C,gBAAgB,EAAE,OAAO;SAC1B,CAAC;IACJ,CAAC;IAED,iEAAiE;IACjE,iEAAiE;IACjE,mEAAmE;IACnE,uEAAuE;IACvE,IAAI,YAAY,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAC5B,OAAO;YACL,cAAc,EAAE,UAAU;YAC1B,SAAS,EACP,iCAAiC,YAAY,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG;gBAC5F,CAAC,OAAO,CAAC,MAAM,GAAG,CAAC;oBACjB,CAAC,CAAC,cAAc,OAAO,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,KAAK,OAAO,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,qDAAqD;oBAC1J,CAAC,CAAC,4BAA4B,CAAC;YACnC,gBAAgB,EAAE,OAAO;SAC1B,CAAC;IACJ,CAAC;IAED,mEAAmE;IACnE,gEAAgE;IAChE,6DAA6D;IAC7D,yCAAyC;IACzC,IAAI,OAAO,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACvB,OAAO;YACL,cAAc,EAAE,iBAAiB;YACjC,SAAS,EACP,qBAAqB,OAAO,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,KAAK,OAAO,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI;gBAC9G,mGAAmG;YACrG,gBAAgB,EAAE,OAAO;SAC1B,CAAC;IACJ,CAAC;IAED,+DAA+D;IAC/D,iEAAiE;IACjE,qEAAqE;IACrE,OAAO;QACL,cAAc,EAAE,iBAAiB;QACjC,SAAS,EACP,gIAAgI;QAClI,gBAAgB,EAAE,EAAE;KACrB,CAAC;AACJ,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,gBAAgB,CAAC,QAAgB;IAC/C,MAAM,QAAQ,GAAG,CAAC,sBAAsB,EAAE,YAAY,EAAE,cAAc,CAAC,CAAC;IACxE,MAAM,KAAK,GAAG,IAAI,GAAG,EAAU,CAAC;IAChC,4DAA4D;IAC5D,4DAA4D;IAC5D,KAAK,MAAM,GAAG,IAAI,CAAC,GAAG,QAAQ,EAAE,aAAa,CAAC,EAAE,CAAC;QAC/C,MAAM,IAAI,GAAG,kBAAkB,CAAC,QAAQ,EAAE,GAAG,CAAC,CAAC;QAC/C,IAAI,CAAC,IAAI;YAAE,SAAS;QACpB,KAAK,MAAM,GAAG,IAAI,QAAQ,CAAC,IAAI,CAAC,EAAE,CAAC;YACjC,IAAI,UAAU,CAAC,GAAG,CAAC;gBAAE,SAAS;YAC9B,IAAI,GAAG,CAAC,MAAM,GAAG,CAAC;gBAAE,SAAS,CAAC,mBAAmB;YACjD,KAAK,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC;QACjB,CAAC;IACH,CAAC;IACD,OAAO,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;AAC3B,CAAC;AAED,SAAS,kBAAkB,CAAC,IAAY,EAAE,WAAmB;IAC3D,kEAAkE;IAClE,mCAAmC;IACnC,MAAM,KAAK,GAAG,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC;IAClC,MAAM,GAAG,GAAa,EAAE,CAAC;IACzB,IAAI,SAAS,GAAG,KAAK,CAAC;IACtB,KAAK,MAAM,GAAG,IAAI,KAAK,EAAE,CAAC;QACxB,MAAM,CAAC,GAAG,GAAG,CAAC,KAAK,CAAC,4BAA4B,CAAC,CAAC;QAClD,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,CAAE,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;YAC5B,IAAI,CAAC,CAAC,CAAC,CAAC,KAAK,WAAW,EAAE,CAAC;gBACzB,SAAS,GAAG,IAAI,CAAC;gBACjB,SAAS;YACX,CAAC;iBAAM,IAAI,SAAS,EAAE,CAAC;gBACrB,8BAA8B;gBAC9B,MAAM;YACR,CAAC;QACH,CAAC;QACD,IAAI,SAAS;YAAE,GAAG,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;IAC/B,CAAC;IACD,OAAO,GAAG,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AACxB,CAAC;AAED,SAAS,SAAS,CAAC,CAAS;IAC1B,OAAO,CAAC,CAAC,WAAW,EAAE,CAAC,OAAO,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC,IAAI,EAAE,CAAC;AACrD,CAAC;AAED,SAAS,QAAQ,CAAC,CAAS;IACzB,mEAAmE;IACnE,OAAO,CAAC;SACL,WAAW,EAAE;SACb,KAAK,CAAC,aAAa,CAAC;SACpB,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC;AACjC,CAAC;AAED,SAAS,YAAY,CAAC,YAAoB,EAAE,MAAc;IACxD,oDAAoD;IACpD,MAAM,EAAE,GAAG,IAAI,MAAM,CAAC,iBAAiB,QAAQ,CAAC,MAAM,CAAC,gBAAgB,CAAC,CAAC;IACzE,OAAO,EAAE,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC;AAC/B,CAAC;AAED,SAAS,QAAQ,CAAC,CAAS;IACzB,OAAO,CAAC,CAAC,OAAO,CAAC,qBAAqB,EAAE,MAAM,CAAC,CAAC;AAClD,CAAC;AAED,MAAM,SAAS,GAAG,IAAI,GAAG,CAAC;IACxB,KAAK,EAAC,GAAG,EAAC,IAAI,EAAC,KAAK,EAAC,IAAI,EAAC,KAAK,EAAC,IAAI,EAAC,IAAI,EAAC,IAAI,EAAC,IAAI,EAAC,IAAI,EAAC,KAAK,EAAC,MAAM,EAAC,IAAI,EAAC,IAAI,EAAC,KAAK,EAAC,KAAK,EAAC,MAAM,EAAC,IAAI,EAAC,MAAM,EAAC,OAAO,EAAC,MAAM,EAAC,KAAK,EAAC,KAAK,EAAC,IAAI,EAAC,MAAM,EAAC,KAAK,EAAC,MAAM,EAAC,MAAM,EAAC,OAAO,EAAC,OAAO,EAAC,IAAI,EAAC,KAAK,EAAC,IAAI,EAAC,IAAI,EAAC,MAAM,EAAC,MAAM,EAAC,MAAM,EAAC,OAAO,EAAC,MAAM,EAAC,OAAO,EAAC,KAAK,EAAC,KAAK,EAAC,KAAK,EAAC,IAAI,EAAC,KAAK,EAAC,MAAM,EAAC,OAAO,EAAC,MAAM,EAAC,KAAK,EAAC,MAAM,EAAC,KAAK,EAAC,MAAM,EAAC,QAAQ,EAAC,OAAO,EAAC,MAAM,EAAC,MAAM,EAAC,MAAM,EAAC,MAAM,EAAC,MAAM,EAAC,MAAM,EAAC,MAAM,EAAC,MAAM,EAAC,MAAM,EAAC,MAAM,EAAC,OAAO,EAAC,SAAS,EAAC,SAAS,EAAC,SAAS,EAAC,OAAO,EAAC,MAAM,EAAC,MAAM,EAAC,SAAS,EAAC,MAAM,EAAC,MAAM,EAAC,OAAO,EAAC,QAAQ,EAAC,UAAU,EAAC,OAAO,EAAC,QAAQ,EAAC,QAAQ;CACziB,CAAC,CAAC;AAEH,SAAS,UAAU,CAAC,CAAS;IAC3B,IAAI,SAAS,CAAC,GAAG,CAAC,CAAC,CAAC;QAAE,OAAO,IAAI,CAAC;IAClC,gCAAgC;IAChC,IAAI,UAAU,CAAC,IAAI,CAAC,CAAC,CAAC;QAAE,OAAO,IAAI,CAAC;IACpC,OAAO,KAAK,CAAC;AACf,CAAC"}

package/dist/commands/plate/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/commands/plate/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AA8OH,wBAAsB,KAAK,CAAC,IAAI,EAAE,MAAM,EAAE,EAAE,UAAU,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAiM7E"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/commands/plate/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAyQH,wBAAsB,KAAK,CAAC,IAAI,EAAE,MAAM,EAAE,EAAE,UAAU,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAmW7E"}

package/dist/commands/plate/index.js

@@ -12,8 +12,11 @@ import { execSync } from "node:child_process";
 import { existsSync, readFileSync } from "node:fs";
 import { join } from "node:path";
 import { GitHubAdapter } from "@slowcook-ai/forge-github";
+import { parseReviewComment, formatPlateReplyBlock, } from "@slowcook-ai/review-overlay";
 import { runPlate } from "./agent.js";
 import { writeVibeFiles } from "../vibe/emit.js";
+import { classifyComment } from "./classify.js";
+const APPROVED_LABEL = "slowcook-mockup-approved";
 function parseArgs(argv) {
     const args = {
         prNumber: -1,
@@ -149,7 +152,7 @@ function fetchTimelineComments(prNumber) {
     const arr = JSON.parse(json);
     return arr
         .filter((c) => c.user.type !== "Bot")
-        .map((c) => ({ author: c.user.login, body: c.body, createdAt: c.created_at }));
+        .map((c) => ({ id: c.id, author: c.user.login, body: c.body, createdAt: c.created_at }));
 }
 function fetchInlineComments(prNumber) {
     const json = execSync(`gh api repos/{owner}/{repo}/pulls/${prNumber}/comments --paginate`, { encoding: "utf8", stdio: ["ignore", "pipe", "ignore"] });
@@ -157,6 +160,7 @@ function fetchInlineComments(prNumber) {
     return arr
         .filter((c) => c.user.type !== "Bot")
         .map((c) => ({
+        id: c.id,
         author: c.user.login,
         body: c.body,
         createdAt: c.created_at,
@@ -174,31 +178,44 @@ function lastPlateCommitDate(repoRoot, branch) {
     }
 }
 function listBranchFiles(repoRoot, branch) {
-    // Files plate is allowed to amend live under src/ + src/lib/data/*.mock.ts.
-    // We list everything under src/ on the branch and let the agent's own
-    // hard rules govern what to amend.
+    // 0.16.0-α.7 → α.14 — files plate amends now live under mock/, not src/.
+    // The 0.16 architecture keeps mock + production in separate
+    // filesystems; brew + slowcook port handle the src/ side.
+    // Vibe writes: mock/scenarios/story-N.ts (always), mock/src/lib/
+    // scenario-registry.ts (extends), mock/src/components/.../*.tsx
+    // (rarely — only new primitives). Plate amends any of these.
+    //
+    // 0.16.0-α.14 fix: list ALL paths in the branch and filter in code.
+    // Previous version passed `mock/**/*.tsx` as a git ls-tree pathspec,
+    // which doesn't expand `**` globs by default. Result was zero matches
+    // (caught on rewo PR #147 dogfood). The fix is mechanical: drop the
+    // pathspec, list everything, filter via the existing regex predicates.
+    // Negligible perf cost on a per-PR branch with O(thousands) of files.
     const out = [];
-    const patterns = ["src/**/*.tsx", "src/**/*.ts"];
-    for (const pat of patterns) {
-        const lsOut = execSync(`git -C ${JSON.stringify(repoRoot)} ls-tree -r --name-only ${JSON.stringify(branch)} -- ${JSON.stringify(pat)}`, { encoding: "utf8", stdio: ["ignore", "pipe", "ignore"] });
-        for (const line of lsOut.split("\n")) {
-            const trimmed = line.trim();
-            if (!trimmed)
-                continue;
-            // Only include files that vibe would have written: page.tsx,
-            // src/components/**, src/lib/data/*.mock.ts. Skip the brew-target
-            // <domain>.ts stubs (plate must not touch them).
-            const isPage = /\/page\.tsx$/.test(trimmed);
-            const isComponent = trimmed.startsWith("src/components/");
-            const isMockData = /^src\/lib\/data\/[^/]+\.mock\.ts$/.test(trimmed);
-            if (!isPage && !isComponent && !isMockData)
-                continue;
-            try {
-                const contents = execSync(`git -C ${JSON.stringify(repoRoot)} show ${JSON.stringify(branch + ":" + trimmed)}`, { encoding: "utf8", stdio: ["ignore", "pipe", "ignore"] });
-                out.push({ path: trimmed, contents });
-            }
-            catch { /* skip files git can't show (binary, deleted) */ }
+    let lsOut;
+    try {
+        lsOut = execSync(`git -C ${JSON.stringify(repoRoot)} ls-tree -r --name-only ${JSON.stringify(branch)}`, { encoding: "utf8", stdio: ["ignore", "pipe", "ignore"] });
+    }
+    catch {
+        return out;
+    }
+    for (const line of lsOut.split("\n")) {
+        const trimmed = line.trim();
+        if (!trimmed)
+            continue;
+        // Only include files vibe writes / plate may amend.
+        const isScenario = /^mock\/scenarios\/story-[\w-]+\.ts$/.test(trimmed);
+        const isRegistry = trimmed === "mock/src/lib/scenario-registry.ts";
+        const isMockComponent = trimmed.startsWith("mock/src/components/") && /\.tsx$/.test(trimmed);
+        const isMockPage = /^mock\/src\/app\/.*page\.tsx$/.test(trimmed);
+        const isMockLib = trimmed.startsWith("mock/src/lib/") && /\.tsx?$/.test(trimmed);
+        if (!isScenario && !isRegistry && !isMockComponent && !isMockPage && !isMockLib)
+            continue;
+        try {
+            const contents = execSync(`git -C ${JSON.stringify(repoRoot)} show ${JSON.stringify(branch + ":" + trimmed)}`, { encoding: "utf8", stdio: ["ignore", "pipe", "ignore"] });
+            out.push({ path: trimmed, contents });
         }
+        catch { /* skip files git can't show (binary, deleted) */ }
     }
     return out;
 }
@@ -230,6 +247,15 @@ export async function plate(argv, cliVersion) {
         console.error(`PR #${pr.number} head branch '${pr.headBranch}' doesn't match slowcook/mockup/story-N convention. Refusing to act.`);
         process.exit(2);
     }
+    // 0.16.0-α.7 — refuse to amend after the PM applied the
+    // slowcook-mockup-approved label. A stray review-overlay comment
+    // (or accidental /plate) shouldn't bounce a finalized mockup.
+    // PM can remove the label to reopen plate iteration.
+    if (pr.labels.includes(APPROVED_LABEL)) {
+        console.log(`PR #${pr.number} carries label \`${APPROVED_LABEL}\`. Plate refuses to amend approved mockups. ` +
+            `Remove the label to reopen iteration.`);
+        return;
+    }
     const storyId = pr.headBranch.replace(/^slowcook\/mockup\/story-/, "");
     // Fetch the spec from main (the source of truth).
     execSync(`git -C ${JSON.stringify(args.repoRoot)} fetch origin main`, {
@@ -274,14 +300,91 @@ export async function plate(argv, cliVersion) {
         console.log(`Filtering feedback to comments newer than last plate commit at ${cutoffDate}.`);
     }
     const filterByDate = (c) => cutoffDate ? c.createdAt > cutoffDate : true;
+    const rawTimeline = fetchTimelineComments(pr.number).filter(filterByDate);
+    const inlineComments = fetchInlineComments(pr.number).filter(filterByDate);
+    // 0.16.0-α.7 — split timeline comments into review-overlay (structured,
+    // classified) vs free-prose (unstructured, treated as before).
+    const overlayComments = [];
+    const proseTimeline = [];
+    for (const c of rawTimeline) {
+        const payload = parseReviewComment(c.body);
+        if (payload) {
+            const cls = classifyComment({ prose: payload.prose, specYaml });
+            overlayComments.push({
+                payload,
+                classification: cls.classification,
+                rationale: cls.rationale,
+                matchedSpecTerms: cls.matchedSpecTerms,
+                raw: c,
+            });
+        }
+        else {
+            proseTimeline.push(c);
+        }
+    }
+    const specAltering = overlayComments.filter((o) => o.classification === "spec-altering");
+    const cosmeticOrDivergent = overlayComments.filter((o) => o.classification !== "spec-altering");
+    console.log(`Feedback to act on: ${proseTimeline.length} prose comment(s), ${overlayComments.length} review-overlay comment(s) ` +
+        `(${cosmeticOrDivergent.length} amendable, ${specAltering.length} spec-altering — escalating), ` +
+        `${inlineComments.length} inline comment(s).`);
+    // 0.16.0-α.7 — for each spec-altering comment, post an escalation
+    // reply on the PR and EXCLUDE it from the agent's feedback. The PM
+    // can confirm via /plate confirm-spec-change (handled in α.7.1)
+    // OR via amending the spec PR upstream.
+    if (specAltering.length > 0) {
+        const githubAdapter = new GitHubAdapter({ owner, repo, token: githubToken });
+        for (const sa of specAltering) {
+            await githubAdapter.createIssueComment(pr.number, buildEscalationBody({ ...sa, commentId: sa.raw.id }, cliVersion));
+        }
+        console.log(`Posted ${specAltering.length} escalation reply/replies for spec-altering comments. They will NOT influence this plate amendment.`);
+    }
+    // 0.16.0-α.15 — track every overlay comment ID this run touched so the
+    // final plate-reply breadcrumb block lists each one with its outcome.
+    // The overlay's pin layer reads this to render the right status icon
+    // on each pin (no timestamp heuristics).
+    const breadcrumbReplies = [];
+    for (const sa of specAltering) {
+        breadcrumbReplies.push({
+            to_comment_id: sa.raw.id,
+            status: "spec-altering",
+            summary: `Escalated: ${sa.rationale}`,
+        });
+    }
+    // For mock-divergence + cosmetic, render their structured prose into
+    // the timeline-comment shape the agent already understands. The
+    // classification rationale becomes a hint the agent sees in context.
+    const classifiedTimelineForAgent = [
+        ...proseTimeline,
+        ...cosmeticOrDivergent.map((o) => {
+            // 0.5.0+ — payload.element is optional (general comments).
+            // Render an anchor preview when present; "general note" tag
+            // otherwise.
+            const el = o.payload.element;
+            const anchorPreview = el
+                ? `selector \`${el.selector}\` (${el.tag}${el.text_hint ? ` · "${el.text_hint}"` : ""})`
+                : "general note (no element anchor)";
+            return {
+                author: o.raw.author,
+                body: `[${o.classification}] ${anchorPreview}:\n\n` +
+                    o.payload.prose +
+                    `\n\n_(Plate classifier: ${o.rationale})_`,
+                createdAt: o.raw.createdAt,
+            };
+        }),
+    ];
     const feedback = {
-        timelineComments: fetchTimelineComments(pr.number).filter(filterByDate),
-        inlineComments: fetchInlineComments(pr.number).filter(filterByDate),
-        screenshots: [], // α.3.1 will populate from comment attachments
+        timelineComments: classifiedTimelineForAgent,
+        inlineComments,
+        screenshots: [], // α.3.1 / α.7.x will populate from comment attachments
     };
-    console.log(`Feedback to act on: ${feedback.timelineComments.length} timeline comment(s), ${feedback.inlineComments.length} inline comment(s).`);
-    if (feedback.timelineComments.length === 0 && feedback.inlineComments.length === 0) {
-        console.log("Nothing to amend. Exiting cleanly.");
+    if (feedback.timelineComments.length === 0 &&
+        feedback.inlineComments.length === 0) {
+        if (specAltering.length > 0) {
+            console.log("Only spec-altering feedback present — escalations posted; no amendment to make.");
+        }
+        else {
+            console.log("Nothing to amend. Exiting cleanly.");
+        }
         return;
     }
     const ctx = {
@@ -309,7 +412,18 @@ export async function plate(argv, cliVersion) {
     const forge = new GitHubAdapter({ owner, repo, token: githubToken });
     if (result.kind === "no-op") {
         console.log(`Plate decided not to amend (spend $${result.spendUsd.toFixed(4)}). Posting summary as PR reply.`);
-        await forge.createIssueComment(pr.number, plateReplyBody(result.summary, result.spendUsd, cliVersion, []));
+        for (const o of cosmeticOrDivergent) {
+            breadcrumbReplies.push({
+                to_comment_id: o.raw.id,
+                status: "noop",
+                summary: result.summary.split("\n")[0].slice(0, 200),
+            });
+        }
+        await forge.createIssueComment(pr.number, plateReplyBody(result.summary, result.spendUsd, cliVersion, [], {
+            version: cliVersion,
+            amendment_commit: null,
+            replies: breadcrumbReplies,
+        }));
         return;
     }
     // Amended: write files + commit + force-push.
@@ -327,7 +441,18 @@ export async function plate(argv, cliVersion) {
         const changed = await forge.git.hasStagedChanges();
         if (!changed) {
             console.log("No-op amendment (re-emitted byte-identical files). Posting note instead of commit.");
-            await forge.createIssueComment(pr.number, plateReplyBody(`${result.summary}\n\n_(plate produced byte-identical files — no commit. Re-comment with sharper feedback if you wanted a change.)_`, result.spendUsd, cliVersion, result.changeRequests.map((cr) => ({ component: cr.component, path: cr.path, rationale: cr.rationale }))));
+            for (const o of cosmeticOrDivergent) {
+                breadcrumbReplies.push({
+                    to_comment_id: o.raw.id,
+                    status: "noop",
+                    summary: "Plate considered the comment but produced no diff (re-emit byte-identical).",
+                });
+            }
+            await forge.createIssueComment(pr.number, plateReplyBody(`${result.summary}\n\n_(plate produced byte-identical files — no commit. Re-comment with sharper feedback if you wanted a change.)_`, result.spendUsd, cliVersion, result.changeRequests.map((cr) => ({ component: cr.component, path: cr.path, rationale: cr.rationale })), {
+                version: cliVersion,
+                amendment_commit: null,
+                replies: breadcrumbReplies,
+            }));
             return;
         }
     }
@@ -336,10 +461,106 @@ export async function plate(argv, cliVersion) {
     // amended repeatedly across iterations. Same shape as refine's
     // amendment force-push pattern.
     execSync(`git -C ${JSON.stringify(args.repoRoot)} push --force-with-lease origin ${JSON.stringify(pr.headBranch)}`, { stdio: "inherit" });
-    await forge.createIssueComment(pr.number, plateReplyBody(result.summary, result.spendUsd, cliVersion, result.changeRequests.map((cr) => ({ component: cr.component, path: cr.path, rationale: cr.rationale }))));
+    // 0.16.0-α.15 — capture the just-pushed commit SHA for the breadcrumb.
+    let amendmentSha = null;
+    try {
+        amendmentSha = execSync(`git -C ${JSON.stringify(args.repoRoot)} rev-parse HEAD`, { encoding: "utf8", stdio: ["ignore", "pipe", "ignore"] }).trim();
+    }
+    catch { /* best-effort */ }
+    // Each cosmetic/divergent overlay comment gets an "applied" breadcrumb
+    // (the LLM's prose summary attributes specifics; for the pin layer's
+    // status icon this is sufficient).
+    for (const o of cosmeticOrDivergent) {
+        breadcrumbReplies.push({
+            to_comment_id: o.raw.id,
+            status: "applied",
+            summary: shortSummaryFor(o, result.summary),
+            files_touched: result.files.map((f) => f.path),
+        });
+    }
+    await forge.createIssueComment(pr.number, plateReplyBody(result.summary, result.spendUsd, cliVersion, result.changeRequests.map((cr) => ({ component: cr.component, path: cr.path, rationale: cr.rationale })), {
+        version: cliVersion,
+        amendment_commit: amendmentSha,
+        replies: breadcrumbReplies,
+    }));
     console.log(`Plate amendment pushed + summary posted on PR #${pr.number}.`);
 }
-function plateReplyBody(summary, spendUsd, cliVersion, changeRequests) {
+/**
+ * 0.16.0-α.15 — pluck a one-line summary for an overlay comment from
+ * the LLM's run-level summary. Best-effort: when the summary mentions
+ * the comment's author / selector, use that line; otherwise fall back
+ * to a generic "Plate amendment applied" line.
+ */
+function shortSummaryFor(o, runSummary) {
+    const lines = runSummary.split(/\r?\n/);
+    const sel = o.payload.element?.selector ?? null;
+    for (const line of lines) {
+        if ((sel && line.includes(sel)) || line.includes(`@${o.raw.author}`)) {
+            return line.replace(/^[-*]\s*/, "").slice(0, 200);
+        }
+    }
+    return "Plate amendment applied (see commit + summary above).";
+}
+/**
+ * 0.16.0-α.7 — build the escalation reply for a spec-altering review-
+ * overlay comment. The PM either:
+ *   (a) confirms via /plate confirm-spec-change → α.7.1+ (TODO)
+ *   (b) amends the spec via /refine on the spec PR upstream
+ *   (c) discards via /plate keep-spec
+ *
+ * Until α.7.1 lands, options (a) + (c) are advisory; option (b) is
+ * the working path. The PM must amend the spec, re-merge, and the
+ * mockup PR's vibe will get the new spec on next iteration.
+ */
+function buildEscalationBody(sa, cliVersion) {
+    const lines = [];
+    lines.push("### slowcook · plate · spec-altering feedback (escalated)");
+    lines.push("");
+    lines.push(`Plate classified the previous review-overlay comment ` +
+        `${sa.payload.element ? `on \`${sa.payload.element.selector}\` ` : "(general note) "}` +
+        `as **spec-altering** and is NOT amending the mock for it. The reasoning:`);
+    lines.push("");
+    lines.push(`> ${sa.rationale}`);
+    if (sa.matchedSpecTerms.length > 0) {
+        lines.push("");
+        lines.push(`Matched spec terms: ${sa.matchedSpecTerms.map((t) => `\`${t}\``).join(", ")}`);
+    }
+    lines.push("");
+    lines.push("**To proceed, choose one:**");
+    lines.push("");
+    lines.push(`- **Amend the spec** (recommended): \`/refine\` on the spec PR upstream and the next mockup iteration picks up the new contract.`);
+    lines.push(`- **Keep the spec, discard this comment**: comment \`/plate keep-spec\` and plate will skip this on the next iteration.`);
+    lines.push(`- **Confirm the spec change inline**: comment \`/plate confirm-spec-change\` (lands in α.7.1; until then use the spec PR path).`);
+    lines.push("");
+    lines.push(`<!-- slowcook:cost agent=plate type=escalation cli=${cliVersion} -->`);
+    // 0.16.0-α.15 — breadcrumb so the overlay's pin layer correlates
+    // this escalation to the originating overlay comment by id (no
+    // timestamp heuristics). One reply entry per escalation comment.
+    if (sa.commentId !== undefined) {
+        const reply = {
+            version: cliVersion,
+            amendment_commit: null,
+            replies: [
+                {
+                    to_comment_id: sa.commentId,
+                    status: "spec-altering",
+                    summary: `Escalated: ${sa.rationale}`.slice(0, 240),
+                },
+            ],
+        };
+        lines.push("");
+        lines.push(formatPlateReplyBlock(reply));
+    }
+    return lines.join("\n");
+}
+function plateReplyBody(summary, spendUsd, cliVersion, changeRequests, 
+/**
+ * 0.16.0-α.15 — breadcrumb so the overlay's pin layer (review-overlay
+ * 0.3.0+) can correlate replies to their original overlay comments by
+ * id, no timestamp heuristics. When undefined, no breadcrumb block is
+ * emitted (back-compat with non-overlay /plate triggers).
+ */
+plateReply) {
     const parts = [];
     parts.push("### slowcook · plate amendment");
     parts.push("");
@@ -359,6 +580,10 @@ function plateReplyBody(summary, spendUsd, cliVersion, changeRequests) {
     }
     parts.push("");
     parts.push(`<!-- slowcook:cost agent=plate usd=${spendUsd.toFixed(4)} model=claude-opus-4-7 cli=${cliVersion} -->`);
+    if (plateReply && plateReply.replies.length > 0) {
+        parts.push("");
+        parts.push(formatPlateReplyBlock(plateReply));
+    }
     return parts.join("\n");
 }
 //# sourceMappingURL=index.js.map