@fenglimg/fabric-cli 2.0.0-rc.37 → 2.0.0-rc.38

package/dist/{doctor-764NFF3X.js → doctor-EJDSEJSS.js}

@@ -576,6 +576,13 @@ function renderCiteCoverageReport(report, jsonMode, dt) {
   lines.push(`  ${dt("doctor.cite.metric.recalledUnverified")}: ${report.metrics.recalled_unverified}`);
   lines.push(`  ${dt("doctor.cite.metric.expectedButMissed")}: ${report.metrics.expected_but_missed}`);
   lines.push(`  ${dt("doctor.cite.metric.totalTurns")}: ${report.metrics.total_turns}`);
+  const complianceRate = report.metrics.cite_compliance_rate;
+  const complianceStr = complianceRate === null || complianceRate === void 0 ? dt("doctor.cite.metric.complianceNA") : `${(complianceRate * 100).toFixed(1)}% (${report.metrics.compliant_cites ?? 0}/${(report.metrics.compliant_cites ?? 0) + (report.metrics.noncompliant_cites ?? 0)})`;
+  lines.push(`  ${dt("doctor.cite.metric.complianceRate")}: ${complianceStr}`);
+  const uncorrelatable = report.metrics.uncorrelatable_edits ?? 0;
+  if (uncorrelatable > 0) {
+    lines.push(`  ${dt("doctor.cite.metric.uncorrelatableEdits")}: ${uncorrelatable}`);
+  }
   if (report.per_client !== void 0 && Object.keys(report.per_client).length > 1) {
     lines.push("");
     lines.push(`### ${dt("doctor.cite.section.perClient")}`);

package/dist/index.js

@@ -11,11 +11,11 @@ import { defineCommand, runMain } from "citty";
 
 // src/commands/index.ts
 var allCommands = {
-  install: () => import("./install-U7MGIJ2L.js").then((module) => module.default),
-  doctor: () => import("./doctor-764NFF3X.js").then((module) => module.default),
+  install: () => import("./install-E6OEB3V2.js").then((module) => module.default),
+  doctor: () => import("./doctor-EJDSEJSS.js").then((module) => module.default),
   uninstall: () => import("./uninstall-MH7ZIB6M.js").then((module) => module.default),
   config: () => import("./config-XJIPZNUP.js").then((module) => module.default),
-  "plan-context-hint": () => import("./plan-context-hint-UQLRKGBZ.js").then((module) => module.default),
+  "plan-context-hint": () => import("./plan-context-hint-FC6P3WFE.js").then((module) => module.default),
   // v2.0.0-rc.23 TASK-014 (F8c): S5 onboard-slot coverage. Used by the
   // fabric-archive Skill's first-run phase to detect unclaimed slots.
   "onboard-coverage": () => import("./onboard-coverage-MFCAEBDO.js").then((module) => module.default),
@@ -27,7 +27,7 @@ var allCommands = {
 var main = defineCommand({
   meta: {
     name: "fabric",
-    version: "2.0.0-rc.37",
+    version: "2.0.0-rc.38",
     description: t("cli.main.description")
   },
   subCommands: allCommands

package/dist/{install-U7MGIJ2L.js → install-E6OEB3V2.js}

@@ -1350,7 +1350,7 @@ function readProjectName(target) {
   return basename(target);
 }
 function getCliVersion() {
-  return true ? "2.0.0-rc.37" : "unknown";
+  return true ? "2.0.0-rc.38" : "unknown";
 }
 function sortRecord(record) {
   return Object.fromEntries(Object.entries(record).sort(([left], [right]) => left.localeCompare(right)));
@@ -1508,6 +1508,9 @@ async function runInitCommand(args) {
   }
   const result = await executeInitExecutionPlan(plan);
   if (!intent.options.planOnly) {
+    console.log("");
+    console.log(t("cli.install.next-steps"));
+    console.log("");
     console.log(paint.muted("More: docs/surfaces.md explains when to use CLI vs Skill vs MCP."));
   }
   return result;

package/dist/{plan-context-hint-UQLRKGBZ.js → plan-context-hint-FC6P3WFE.js}

@@ -58,24 +58,13 @@ async function runPlanContextHint(opts) {
   const result = await planContext(resolution.target, {
     paths: targetPaths
   });
-  const sharedIndex = result.shared.description_index;
-  const narrowSource = all ? sharedIndex : (
-    // Path mode: union of per-entry description_index across requested
-    // paths, deduped by stable_id. This is identical to `shared` for L0/L1
-    // entries (always included) and additionally captures L2 entries whose
-    // scope_glob matches the requested path.
-    dedupeByStableId(result.entries.flatMap((entry) => entry.description_index))
-  );
-  const entries = narrowSource.map((item) => ({
+  const candidates = result.candidates;
+  const entries = candidates.map((item) => ({
     id: item.stable_id,
-    type: item.type ?? item.description.knowledge_type ?? "",
-    maturity: item.maturity ?? item.description.maturity ?? "",
+    type: item.description.knowledge_type ?? "",
+    maturity: item.description.maturity ?? "",
     summary: item.description.summary,
-    // v2.0.0-rc.27 TASK-002 (§2.5/§2.7): forward the server-side scope.
-    // RuleDescriptionIndexItem already carries this field — knowledge-meta-
-    // builder defaults to "broad" for entries without an explicit
-    // relevance_scope frontmatter, so this read is total and never undefined.
-    relevance_scope: item.relevance_scope ?? "broad"
+    relevance_scope: item.description.relevance_scope ?? "broad"
   }));
   let narrow_count = 0;
   let broad_only_count = 0;
@@ -89,8 +78,8 @@ async function runPlanContextHint(opts) {
     target_paths: targetPaths,
     entries,
     // Legacy field — preserved for v2 consumers that haven't migrated. Value
-    // semantics unchanged from rc.18 (sharedIndex total).
-    broad_count: sharedIndex.length,
+    // semantics unchanged from rc.18 (total candidate count).
+    broad_count: candidates.length,
     narrow_count,
     broad_only_count
   };
@@ -108,16 +97,6 @@ function parsePathsArg(raw) {
   }
   return raw.split(",").map((part) => part.trim()).filter((part) => part.length > 0);
 }
-function dedupeByStableId(items) {
-  const seen = /* @__PURE__ */ new Set();
-  const result = [];
-  for (const item of items) {
-    if (seen.has(item.stable_id)) continue;
-    seen.add(item.stable_id);
-    result.push(item);
-  }
-  return result;
-}
 export {
   plan_context_hint_default as default,
   planContextHintCommand,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fenglimg/fabric-cli",
-  "version": "2.0.0-rc.37",
+  "version": "2.0.0-rc.38",
   "description": "Fabric CLI — installs the MCP server + skills + hooks for Claude Code, Cursor, and Codex CLI; runs doctor / knowledge maintenance.",
   "license": "MIT",
   "author": "wangzhichao <fenglimg90@gmail.com>",
@@ -45,8 +45,8 @@
     "tree-sitter-javascript": "^0.25.0",
     "tree-sitter-typescript": "^0.23.2",
     "web-tree-sitter": "^0.26.8",
-    "@fenglimg/fabric-server": "2.0.0-rc.37",
-    "@fenglimg/fabric-shared": "2.0.0-rc.37"
+    "@fenglimg/fabric-server": "2.0.0-rc.38",
+    "@fenglimg/fabric-shared": "2.0.0-rc.38"
   },
   "devDependencies": {
     "@types/node": "^22.15.0",

package/templates/hooks/knowledge-hint-narrow.cjs

@@ -358,7 +358,7 @@ function extractPaths(toolInput) {
  *     POSIX) is atomic at the OS level, so concurrent PreToolUse fires
  *     from parallel sessions interleave cleanly without partial writes.
  */
-function appendEditIntentToLedger(projectRoot, now, paths, toolName) {
+function appendEditIntentToLedger(projectRoot, now, paths, toolName, sessionId) {
   try {
     const fabricDir = join(projectRoot, EVENTS_LEDGER_DIR_REL);
     // No .fabric/ → project not initialised. Bail before any write.
@@ -383,12 +383,20 @@ function appendEditIntentToLedger(projectRoot, now, paths, toolName) {
     const tsMs = now instanceof Date ? now.getTime() : Number(now);
     const ledgerEntryId = `hook:${randomUUID()}`;
     const intent = typeof toolName === "string" && toolName.length > 0 ? toolName : "edit";
+    // rc.38 UX-8 (C): thread the REAL payload session_id (never the synthetic
+    // fallback) so doctor cite-coverage's expected_but_missed arm can correlate
+    // this edit against the same session's assistant_turn cite lines. Omitting
+    // it (the rc.35 oversight) left the correlation key undefined → missed
+    // permanently 0 → cite_compliance_rate structurally pinned at 100%.
+    const validSessionId =
+      typeof sessionId === "string" && sessionId.length > 0 ? sessionId : null;
     const lines = pathList
       .map((p) => JSON.stringify({
         kind: "fabric-event",
         id: `event:${randomUUID()}`,
         ts: tsMs,
         schema_version: 1,
+        ...(validSessionId ? { session_id: validSessionId } : {}),
         event_type: "edit_intent_checked",
         path: p,
         compliant: true,
@@ -1249,7 +1257,15 @@ function main(env, stdio) {
       // events.jsonl ledger so doctor cite-coverage's editsTouched metric
       // sees actual edit signals. Best-effort — failure is swallowed inside
       // appendEditIntentToLedger and does not block the hook.
-      appendEditIntentToLedger(cwd, now, paths, toolName);
+      // rc.38 UX-8 (C): pass the REAL payload session_id (not resolveSessionId,
+      // which would substitute a synthetic per-process id that matches no
+      // assistant_turn and would inflate expected_but_missed with false
+      // positives under --client=all). null when the client omits session_id.
+      const payloadSessionId =
+        payload && typeof payload === "object" && typeof payload.session_id === "string"
+          ? payload.session_id
+          : null;
+      appendEditIntentToLedger(cwd, now, paths, toolName, payloadSessionId);
     }
 
     // E2 path is conditional on a recognized tool + extractable paths.