cclaw-cli 0.51.2 → 0.51.4

package/dist/artifact-linter.js

@@ -1633,25 +1633,34 @@ export async function lintArtifact(projectRoot, stage, track = "standard") {
         // IDs all fail the lint; absence of the section remains advisory so
         // scope stays optional for small/quick tracks.
         if (headingPresent(sections, "Locked Decisions (D-XX)")) {
-            const decisionIds = extractDecisionIds(lockedDecisionsBody);
-            const bulletLines = lockedDecisionsBody
+            const listDecisionLines = lockedDecisionsBody
                 .split(/\r?\n/u)
                 .map((line) => line.trim())
-                .filter((line) => /^(?:[-*]|\|)\s+\S/u.test(line));
-            const orphanBullets = bulletLines.filter((line) => !/\bD-\d+\b/u.test(line));
+                .filter((line) => /^[-*]\s+\S/u.test(line));
+            const tableDecisionRows = getMarkdownTableRows(lockedDecisionsBody);
+            const tableDecisionLines = tableDecisionRows.map((row) => row.join(" | "));
+            const decisionLines = [...listDecisionLines, ...tableDecisionLines];
+            const orphanDecisionLines = decisionLines.filter((line) => !/\bD-\d+\b/u.test(line));
+            const rowDecisionIds = [
+                ...listDecisionLines.map((line) => /\bD-\d+\b/u.exec(line)?.[0]),
+                ...tableDecisionRows.map((row) => /\bD-\d+\b/u.exec(row[0] ?? "")?.[0])
+            ].filter((id) => typeof id === "string");
             const duplicateIds = (() => {
-                const all = lockedDecisionsBody.match(/\bD-\d+\b/gu) ?? [];
                 const counts = new Map();
-                for (const id of all)
+                for (const id of rowDecisionIds)
                     counts.set(id, (counts.get(id) ?? 0) + 1);
                 return [...counts.entries()].filter(([, n]) => n > 1).map(([id]) => id);
             })();
             const issues = [];
-            if (decisionIds.length === 0 && bulletLines.length === 0) {
+            if (rowDecisionIds.length === 0 && decisionLines.length === 0) {
                 issues.push("section is empty");
             }
-            if (orphanBullets.length > 0) {
-                issues.push(`${orphanBullets.length} bullet(s) missing a D-XX ID`);
+            if (orphanDecisionLines.length > 0) {
+                const examples = orphanDecisionLines
+                    .slice(0, 3)
+                    .map((line) => `\`${line.slice(0, 120)}\``)
+                    .join(", ");
+                issues.push(`${orphanDecisionLines.length} decision row(s) missing a D-XX ID${examples.length > 0 ? `: ${examples}` : ""}`);
             }
             if (duplicateIds.length > 0) {
                 issues.push(`duplicate IDs: ${duplicateIds.join(", ")}`);
@@ -1662,7 +1671,7 @@ export async function lintArtifact(projectRoot, stage, track = "standard") {
                 rule: "Locked Decisions section must list each decision with a unique stable D-XX ID.",
                 found: issues.length === 0,
                 details: issues.length === 0
-                    ? `${decisionIds.length} decision ID(s) recorded with no duplicates.`
+                    ? `${rowDecisionIds.length} decision ID(s) recorded with no duplicates.`
                     : issues.join("; ")
             });
         }

package/dist/content/stages/brainstorm.js

@@ -48,7 +48,7 @@ export const BRAINSTORM = {
         ],
         interactionProtocol: [
             "Start from observed project context; if the idea is vague, first narrow the project type.",
-            "Ask at most one question per turn, only when decision-changing; prefer multiple choice, and summarize after 2-3 answers.",
+            "Ask at most one question per turn, only when decision-changing; if using a structured question tool, send exactly one question object, not a multi-question form.",
             "If likely answers do not change architecture or scope boundaries, choose the default and state the assumption.",
             "Show approaches before the recommendation; include a higher-upside challenger and gather reaction first.",
             "State exactly what is being approved, then **STOP** until the user explicitly approves the artifact."
@@ -119,9 +119,9 @@ export const BRAINSTORM = {
             { section: "Clarifying Questions", required: false, validationRule: "Must capture question, answer, and decision impact for each clarifying question." },
             { section: "Approach Tier", required: true, validationRule: "Must classify depth as Lightweight/Standard/Deep and explain why." },
             { section: "Short-Circuit Decision", required: false, validationRule: "Must include Status/Why/Scope handoff lines when short-circuit is discussed; compact stubs are valid for concrete asks." },
-            { section: "Approaches", required: true, validationRule: "Must compare 2-3 architecturally distinct options with real trade-offs and include one row labeled `challenger: higher-upside`." },
-            { section: "Approach Reaction", required: true, validationRule: "Must summarize user reaction before recommendation, including concerns that changed direction." },
-            { section: "Selected Direction", required: true, validationRule: "Must include the selected approach, rationale tied to user reaction/feedback, and explicit approval marker." },
+            { section: "Approaches", required: true, validationRule: "Must compare 2-3 architecturally distinct options with real trade-offs; include at least one table/bullet row containing both `challenger` and `higher-upside`." },
+            { section: "Approach Reaction", required: true, validationRule: "Must appear before Selected Direction and summarize user reaction before recommendation, including `Closest option`, `Concerns`, and what changed after reaction." },
+            { section: "Selected Direction", required: true, validationRule: "Must include the selected approach, an explicit approval marker, and rationale tied to user reaction/feedback/concerns so the recommendation traces to the user response." },
             { section: "Design", required: false, validationRule: "Must cover architecture, key components, and data flow scaled to complexity." },
             { section: "Visual Companion", required: false, validationRule: "If architecture/data-flow complexity is medium+, include compact ASCII/Mermaid diagram or explicitly justify omission." },
             { section: "Assumptions and Open Questions", required: false, validationRule: "Must capture unresolved assumptions/open questions, or explicitly state none." }

package/dist/content/templates.js

@@ -58,7 +58,7 @@ export const ARTIFACT_TEMPLATES = {
 
 ## Selected Direction
 - **Approach:**
-- **Rationale:**
+- **Rationale:** Based on user reaction/feedback/concerns, ...
 - **Approval:** pending
 
 ${SEED_SHELF_SECTION}

package/dist/gate-evidence.js

@@ -215,11 +215,21 @@ export async function verifyCurrentStageGateEvidence(projectRoot, flowState) {
     if (shouldValidateArtifact) {
         const lint = await lintArtifact(projectRoot, stage, flowState.track);
         if (!lint.passed) {
-            const failedRequired = lint.findings
-                .filter((finding) => finding.required && !finding.found)
-                .map((finding) => finding.section);
+            const failedRequiredFindings = lint.findings
+                .filter((finding) => finding.required && !finding.found);
+            const failedRequired = failedRequiredFindings.map((finding) => finding.section);
             if (failedRequired.length > 0) {
-                issues.push(`artifact validation failed for required sections: ${failedRequired.join(", ")}.`);
+                const failureDetails = failedRequiredFindings
+                    .map((finding) => {
+                    const details = finding.details?.trim();
+                    const rule = finding.rule?.trim();
+                    const explanation = details && details.length > 0 ? details : rule;
+                    return explanation && explanation.length > 0
+                        ? `${finding.section}: ${explanation}`
+                        : finding.section;
+                })
+                    .join("; ");
+                issues.push(`artifact validation failed for required sections: ${failedRequired.join(", ")}. ${failureDetails}`);
             }
         }
         if (stage === "review") {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cclaw-cli",
-  "version": "0.51.2",
+  "version": "0.51.4",
   "description": "Installer-first flow toolkit for coding agents",
   "type": "module",
   "bin": {