cclaw-cli 0.49.0 → 0.51.0

package/dist/content/view-command.js

@@ -1,4 +1,7 @@
 import { RUNTIME_ROOT } from "../constants.js";
+import { diffSubcommandMarkdown } from "./diff-command.js";
+import { statusSubcommandMarkdown } from "./status-command.js";
+import { treeSubcommandMarkdown } from "./tree-command.js";
 const VIEW_SKILL_FOLDER = "flow-view";
 const VIEW_SKILL_NAME = "flow-view";
 export function viewCommandContract() {
@@ -11,20 +14,20 @@ Unified read-only command surface for flow visibility.
 Subcommands:
 - \`/cc-view\` or \`/cc-view status\` -> status snapshot
 - \`/cc-view tree\` -> structural flow tree
-- \`/cc-view diff\` -> baseline delta map
+- \`/cc-view diff\` -> read-only git delta map
 
 ## HARD-GATE
 
 - \`/cc-view\` is strictly read-only at wrapper level.
-- Do not mutate flow-state unless routing to \`diff\` (which updates snapshot baseline by design).
+- Do not mutate flow-state or derived state from any subcommand.
 
 ## Routing
 
 1. Parse subcommand (default \`status\`).
 2. Route:
-   - \`status\` -> load \`${RUNTIME_ROOT}/commands/status.md\` + \`${RUNTIME_ROOT}/skills/flow-status/SKILL.md\`
-   - \`tree\` -> load \`${RUNTIME_ROOT}/commands/tree.md\` + \`${RUNTIME_ROOT}/skills/flow-tree/SKILL.md\`
-   - \`diff\` -> load \`${RUNTIME_ROOT}/commands/diff.md\` + \`${RUNTIME_ROOT}/skills/flow-diff/SKILL.md\`
+   - \`status\` -> use the **Status Subcommand** section in \`${RUNTIME_ROOT}/skills/${VIEW_SKILL_FOLDER}/SKILL.md\`
+   - \`tree\` -> use the **Tree Subcommand** section in \`${RUNTIME_ROOT}/skills/${VIEW_SKILL_FOLDER}/SKILL.md\`
+   - \`diff\` -> use the **Diff Subcommand** section in \`${RUNTIME_ROOT}/skills/${VIEW_SKILL_FOLDER}/SKILL.md\`
 3. Unknown subcommand -> print supported values and stop.
 
 ## Headless mode
@@ -32,7 +35,7 @@ Subcommands:
 For machine orchestration, emit one JSON envelope:
 
 \`\`\`json
-{"version":"1","kind":"stage-output","stage":"review","payload":{"command":"/cc-view","subcommand":"status","summary":"<short>"},"emittedAt":"<ISO-8601>"}
+{"version":"1","kind":"stage-output","stage":"non-flow","payload":{"command":"/cc-view","subcommand":"status","summary":"<short>"},"emittedAt":"<ISO-8601>"}
 \`\`\`
 
 Validate envelopes with:
@@ -44,9 +47,12 @@ Validate envelopes with:
 `;
 }
 export function viewCommandSkillMarkdown() {
+    const status = statusSubcommandMarkdown();
+    const tree = treeSubcommandMarkdown();
+    const diff = diffSubcommandMarkdown();
     return `---
 name: ${VIEW_SKILL_NAME}
-description: "Unified read-only view router for status/tree/diff flow visibility commands."
+description: "Unified read-only view skill for status/tree/diff flow visibility commands."
 ---
 
 # /cc-view
@@ -59,10 +65,22 @@ Wrapper is read-only and dispatch-only. It must not mutate flow state directly.
 
 1. Parse optional subcommand token:
    - missing -> \`status\`
-   - \`status\` -> dispatch to \`${RUNTIME_ROOT}/commands/status.md\`
-   - \`tree\` -> dispatch to \`${RUNTIME_ROOT}/commands/tree.md\`
-   - \`diff\` -> dispatch to \`${RUNTIME_ROOT}/commands/diff.md\`
-2. Execute the target command contract and skill.
+   - \`status\` -> run **Status Subcommand** below
+   - \`tree\` -> run **Tree Subcommand** below
+   - \`diff\` -> run **Diff Subcommand** below
+2. Execute only the chosen subcommand section.
 3. Return concise output and suggest \`/cc-view <subcommand>\` variants for navigation.
+
+## Status Subcommand
+
+${status}
+
+## Tree Subcommand
+
+${tree}
+
+## Diff Subcommand
+
+${diff}
 `;
 }

package/dist/delegation.d.ts

@@ -70,7 +70,7 @@ export type DelegationLedger = {
 };
 /**
  * Heuristic: does a changed file path strongly imply a trust-boundary
- * surface? Used to gate adversarial-reviewer requirements on review.
+ * surface? Used by tests and prompt guidance for risk-triggered review.
  *
  * Matches authN/Z, credentials, crypto, policy, or explicit sanitization
  * or injection handling. Intentionally excludes broad terms like `input`

package/dist/delegation.js

@@ -55,7 +55,7 @@ async function resolveReviewDiffBase(projectRoot) {
 }
 /**
  * Heuristic: does a changed file path strongly imply a trust-boundary
- * surface? Used to gate adversarial-reviewer requirements on review.
+ * surface? Used by tests and prompt guidance for risk-triggered review.
  *
  * Matches authN/Z, credentials, crypto, policy, or explicit sanitization
  * or injection handling. Intentionally excludes broad terms like `input`
@@ -69,8 +69,7 @@ async function detectReviewTriggers(projectRoot) {
     const empty = {
         changedFiles: 0,
         changedLines: 0,
-        trustBoundaryChanged: false,
-        requireAdversarialReviewer: false
+        trustBoundaryChanged: false
     };
     const base = await resolveReviewDiffBase(projectRoot);
     if (!base) {
@@ -94,12 +93,10 @@ async function detectReviewTriggers(projectRoot) {
             .map((line) => line.trim())
             .filter((line) => line.length > 0);
         const trustBoundaryChanged = changedPaths.some((p) => isTrustBoundaryPath(p));
-        const requireAdversarialReviewer = changedLines > 100 || changedFiles > 10 || trustBoundaryChanged;
         return {
             changedFiles,
             changedLines,
-            trustBoundaryChanged,
-            requireAdversarialReviewer
+            trustBoundaryChanged
         };
     }
     catch {
@@ -277,20 +274,13 @@ export async function checkMandatoryDelegations(projectRoot, stage, options = {}
     const harnesses = config?.harnesses ?? [];
     const fallbacks = harnesses.map((h) => HARNESS_ADAPTERS[h].capabilities.subagentFallback);
     const expectedMode = expectedFulfillmentMode(fallbacks);
-    const reviewTriggers = stage === "review" ? await detectReviewTriggers(projectRoot) : null;
     for (const agent of mandatory) {
         const rows = forRun.filter((e) => e.agent === agent);
         const completedRows = rows.filter((e) => e.status === "completed");
         const waivedRows = rows.filter((e) => e.status === "waived");
-        const adversarialReviewerRequired = stage === "review" &&
-            agent === "reviewer" &&
-            reviewTriggers?.requireAdversarialReviewer === true;
-        const requiredCompletedCount = adversarialReviewerRequired ? 2 : 1;
-        const hasCompleted = completedRows.length >= requiredCompletedCount;
+        const hasCompleted = completedRows.length >= 1;
         const hasWaived = waivedRows.length > 0;
-        const hasAdversarialSkill = !adversarialReviewerRequired ||
-            completedRows.some((row) => row.skill === "adversarial-review");
-        const ok = hasWaived || (hasCompleted && hasAdversarialSkill);
+        const ok = hasWaived || hasCompleted;
         if (!ok) {
             missing.push(agent);
             continue;

package/dist/doctor-registry.js

@@ -1,6 +1,5 @@
-import { DOCTOR_REFERENCE_DIR } from "./content/doctor-references.js";
 function ref(fileName) {
-    return `${DOCTOR_REFERENCE_DIR}/${fileName}`;
+    return `docs/${fileName}`;
 }
 const RULES = [
     {
@@ -9,7 +8,7 @@ const RULES = [
             severity: "info",
             summary: "Gate reconciliation status update.",
             fix: "No action required unless subsequent gate checks fail.",
-            docRef: ref("state-and-gates.md")
+            docRef: ref("config.md")
         }
     },
     {
@@ -18,7 +17,7 @@ const RULES = [
             severity: "warning",
             summary: "Advisory signal; runtime can continue with caution.",
             fix: "Address when possible to prevent future drift or degraded behavior.",
-            docRef: ref("README.md")
+            docRef: "README.md"
         }
     },
     {
@@ -27,7 +26,7 @@ const RULES = [
             severity: "warning",
             summary: "Stage skill quality guardrail check.",
             fix: "Tune generated stage skill content and re-run `cclaw sync`.",
-            docRef: ref("runtime-layout.md")
+            docRef: "README.md"
         }
     },
     {
@@ -36,7 +35,7 @@ const RULES = [
             severity: "error",
             summary: "Required runtime tooling availability check.",
             fix: "Install the missing required tool and re-run `cclaw doctor`.",
-            docRef: ref("tooling-capabilities.md")
+            docRef: "README.md"
         }
     },
     {
@@ -45,7 +44,7 @@ const RULES = [
             severity: "error",
             summary: "Generated runtime surface presence check.",
             fix: "Run `cclaw sync` to regenerate runtime files, then re-run doctor.",
-            docRef: ref("runtime-layout.md")
+            docRef: "README.md"
         }
     },
     {
@@ -54,7 +53,7 @@ const RULES = [
             severity: "error",
             summary: "Hook wiring and lifecycle integration check.",
             fix: "Repair hook/plugin wiring (usually via `cclaw sync`) and validate harness config.",
-            docRef: ref("hooks-and-lifecycle.md")
+            docRef: ref("harnesses.md")
         }
     },
     {
@@ -63,16 +62,16 @@ const RULES = [
             severity: "error",
             summary: "Harness shim and routing file consistency check.",
             fix: "Regenerate harness adapters via `cclaw sync`; confirm enabled harness list.",
-            docRef: ref("harness-and-routing.md")
+            docRef: ref("harnesses.md")
         }
     },
     {
-        test: /^(flow_state:|state:|contexts:|gates:)/,
+        test: /^(flow_state:|state:|gates:)/,
         metadata: {
             severity: "error",
             summary: "Flow state and gate evidence consistency check.",
             fix: "Repair flow-state artifacts and gate evidence, then run `cclaw doctor --reconcile-gates`.",
-            docRef: ref("state-and-gates.md")
+            docRef: ref("config.md")
         }
     },
     {
@@ -81,16 +80,16 @@ const RULES = [
             severity: "error",
             summary: "Knowledge and artifact runtime integrity check.",
             fix: "Restore missing runtime files under `.cclaw/` or re-run `cclaw sync`.",
-            docRef: ref("runtime-layout.md")
+            docRef: "README.md"
         }
     },
     {
-        test: /^(meta_skill:|protocol:|stage_skill:|context_mode:)/,
+        test: /^(meta_skill:|protocol:|stage_skill:)/,
         metadata: {
             severity: "error",
             summary: "Routing skill and protocol integrity check.",
-            fix: "Regenerate runtime references and skills via `cclaw sync`, then re-run doctor.",
-            docRef: ref("harness-and-routing.md")
+            fix: "Regenerate runtime skills via `cclaw sync`, then re-run doctor.",
+            docRef: ref("harnesses.md")
         }
     },
     {
@@ -105,7 +104,7 @@ const RULES = [
             severity: "warning",
             summary: "Reference/overview doc integrity (non-blocking).",
             fix: "Run `cclaw sync` to regenerate the reference doc from the canonical source.",
-            docRef: ref("harness-and-routing.md")
+            docRef: ref("harnesses.md")
         }
     },
     {
@@ -114,7 +113,7 @@ const RULES = [
             severity: "error",
             summary: "Mandatory delegation completion check.",
             fix: "Complete or explicitly waive missing mandatory delegations in delegation log.",
-            docRef: ref("delegation-and-preamble.md")
+            docRef: ref("harnesses.md")
         }
     },
     {
@@ -123,7 +122,7 @@ const RULES = [
             severity: "error",
             summary: "Cross-artifact traceability integrity check.",
             fix: "Restore criterion/task/test ID mappings across spec, plan, and tdd artifacts.",
-            docRef: ref("traceability.md")
+            docRef: "README.md"
         }
     },
     {
@@ -132,7 +131,7 @@ const RULES = [
             severity: "error",
             summary: "Config or policy schema consistency check.",
             fix: "Fix config/rules drift, then run `cclaw sync` and re-run doctor.",
-            docRef: ref("config-and-policy.md")
+            docRef: ref("config.md")
         }
     }
 ];
@@ -143,9 +142,9 @@ export function doctorCheckMetadata(checkName) {
         }
     }
     return {
-        severity: "error",
+        severity: "warning",
         summary: "Unclassified doctor check.",
         fix: "Report this check name to cclaw maintainers so doctor-registry can classify it explicitly.",
-        docRef: ref("README.md")
+        docRef: "README.md"
     };
 }