@fenglimg/fabric-cli 2.0.0-rc.10 → 2.0.0-rc.13

package/dist/{scan-ELSNCSKS.js → scan-VHKZPT2W.js}

@@ -3,11 +3,12 @@ import {
   __testing__,
   createScanReport,
   deriveTagsFromForensic,
+  detectExistingLanguage,
   formatKnowledgeId,
   runInitScan,
   scanCommand,
   scan_default
-} from "./chunk-MT3R57VG.js";
+} from "./chunk-FDRLV5PL.js";
 import "./chunk-WWNXR34K.js";
 import "./chunk-OBQU6NHO.js";
 import "./chunk-6ICJICVU.js";
@@ -16,6 +17,7 @@ export {
   createScanReport,
   scan_default as default,
   deriveTagsFromForensic,
+  detectExistingLanguage,
   formatKnowledgeId,
   runInitScan,
   scanCommand

package/dist/{uninstall-DBAR2JBS.js → uninstall-JHUSFENL.js}

@@ -2,18 +2,16 @@
 import {
   detectClientSupports,
   resolveClients
-} from "./chunk-HQLEHH4O.js";
+} from "./chunk-OHWQNSLH.js";
 import {
   FABRIC_HOOK_COMMAND_PATHS,
+  FABRIC_SECTION_REGEX,
   HOOK_CONFIG_ARRAY_PATHS,
   HOOK_CONFIG_TARGETS,
   HOOK_SCRIPT_DESTINATIONS,
-  IMPORT_POINTER_LINE,
-  POINTER_LINE,
-  POINTER_TARGETS,
-  REVIEW_POINTER_LINE,
+  SECTION_TARGETS,
   SKILL_DESTINATIONS
-} from "./chunk-AW3G7ZH5.js";
+} from "./chunk-X7QPY5KH.js";
 import {
   paint
 } from "./chunk-WWNXR34K.js";
@@ -115,12 +113,12 @@ async function unmergeCursorHookConfig(projectRoot, opts = {}) {
     cleanEmpties: opts.cleanEmpties === true
   });
 }
-async function stripArchiveSkillPointers(projectRoot) {
+async function stripFabricKnowledgeBaseSection(projectRoot) {
   const results = [];
-  for (const rel of POINTER_TARGETS) {
+  for (const rel of SECTION_TARGETS) {
     const target = join(projectRoot, rel);
     if (!existsSync(target)) {
-      results.push({ step: "pointer", path: target, status: "skipped", message: "absent" });
+      results.push({ step: "section", path: target, status: "skipped", message: "absent" });
       continue;
     }
     let existing;
@@ -128,30 +126,41 @@ async function stripArchiveSkillPointers(projectRoot) {
       existing = await readFile(target, "utf8");
     } catch (error) {
       results.push({
-        step: "pointer",
+        step: "section",
         path: target,
         status: "error",
         message: error instanceof Error ? error.message : String(error)
       });
       continue;
     }
-    const pointerLiterals = [POINTER_LINE, REVIEW_POINTER_LINE, IMPORT_POINTER_LINE];
-    const filtered = existing.split("\n").filter((line) => !pointerLiterals.some((literal) => line.includes(literal))).join("\n");
+    const match = existing.match(FABRIC_SECTION_REGEX);
+    if (match === null) {
+      results.push({
+        step: "section",
+        path: target,
+        status: "skipped",
+        message: "no-fabric-section"
+      });
+      continue;
+    }
+    const before = existing.slice(0, match.index ?? 0);
+    const after = existing.slice((match.index ?? 0) + match[0].length);
+    const filtered = `${before}${after.replace(/^\r?\n/, "")}`;
     if (filtered === existing) {
       results.push({
-        step: "pointer",
+        step: "section",
         path: target,
         status: "skipped",
-        message: "no-fabric-pointers"
+        message: "no-fabric-section"
       });
       continue;
     }
     try {
       await atomicWriteText(target, filtered);
-      results.push({ step: "pointer", path: target, status: "removed" });
+      results.push({ step: "section", path: target, status: "removed" });
     } catch (error) {
       results.push({
-        step: "pointer",
+        step: "section",
         path: target,
         status: "error",
         message: error instanceof Error ? error.message : String(error)
@@ -164,9 +173,9 @@ async function uninstallBootstrapStage(projectRoot, opts = {}) {
   const results = [];
   await runAndCollect(
     results,
-    "pointer",
+    "section",
     projectRoot,
-    () => stripArchiveSkillPointers(projectRoot)
+    () => stripFabricKnowledgeBaseSection(projectRoot)
   );
   await runAndCollectOne(
     results,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fenglimg/fabric-cli",
-  "version": "2.0.0-rc.10",
+  "version": "2.0.0-rc.13",
   "type": "module",
   "bin": {
     "fab": "dist/index.js",
@@ -20,8 +20,8 @@
     "tree-sitter-javascript": "^0.25.0",
     "tree-sitter-typescript": "^0.23.2",
     "web-tree-sitter": "^0.26.8",
-    "@fenglimg/fabric-shared": "2.0.0-rc.10",
-    "@fenglimg/fabric-server": "2.0.0-rc.10"
+    "@fenglimg/fabric-shared": "2.0.0-rc.13",
+    "@fenglimg/fabric-server": "2.0.0-rc.13"
   },
   "devDependencies": {
     "@types/node": "^22.15.0",

package/templates/hooks/configs/README.md

@@ -1,6 +1,6 @@
 # Client hook config templates
 
-These JSON files are **fragment templates** consumed by `fabric init` and
+These JSON files are **fragment templates** consumed by `fabric install` and
 `fabric hooks install`. They are not standalone client config files.
 
 The supported clients are pinned by `packages/shared/src/schemas/fabric-config.ts`

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

@@ -49,41 +49,22 @@
 const { spawnSync } = require("node:child_process");
 const {
   existsSync,
-  mkdirSync,
   readdirSync,
   readFileSync,
-  writeFileSync,
 } = require("node:fs");
-const { dirname, join } = require("node:path");
+const { join } = require("node:path");
 
 // -----------------------------------------------------------------------------
-// rc.7 T8: SessionStart revision_hash gating.
-//
-// Q-14 problem: every SessionStart re-dumped the full broad knowledge list,
-// causing banner blindness. Solution: hash-of-canonical-graph gating — record
-// the last-emitted `payload.revision_hash` to a sidecar; on subsequent
-// SessionStart fires, compare. Match → silent exit 0 (no re-dump). Mismatch
-// (canonical/ corpus changed → planContext bumps revision_hash) → emit AND
-// update sidecar.
-//
-// The revision_hash is supplied by `fabric plan-context-hint --all`'s JSON
-// payload (carried in payload.revision_hash since rc.5). Reusing the existing
-// hash primitive keeps the gating predicate exactly aligned with the "is the
-// knowledge graph different from last time?" question — no second hashing
-// scheme to maintain. computeRevisionHash() is not needed at this layer; we
-// compare the strings the CLI hands us.
-//
-// rc.8 underseed self-check: the retired `.fabric/.import-requested` sentinel
-// mechanism is replaced by a deterministic three-condition probe in
-// shouldRecommendImport(). When the probe says "recommend", a one-line
-// `/fabric-import` banner is appended to the broad-injection output and
-// the revision_hash gate is bypassed FOR THE BANNER ONLY (the broad-summary
-// body itself remains hash-gated). See shouldRecommendImport() below for
-// the full truth table.
+// rc.12: SessionStart broad-menu is now unconditionally emitted on every
+// SessionStart fire (matching Skill-style progressive disclosure). Prior
+// versions (rc.5-rc.11) wrote `.fabric/.cache/sessionstart-last-hash` as a
+// revision_hash cooldown sidecar to suppress re-emission on unchanged
+// knowledge graphs; that gate was removed in rc.12. Orphaned sidecar files
+// on existing dogfood repos are harmless dead state and are intentionally
+// NOT cleaned up (zero-user clean-slate — no migration logic needed).
 // -----------------------------------------------------------------------------
 
 const FABRIC_DIR_REL = ".fabric";
-const SESSIONSTART_HASH_CACHE_FILE = join(".fabric", ".cache", "sessionstart-last-hash");
 
 // rc.8 underseed self-check constants (mirror fabric-hint.cjs ~line 76 / 83).
 // Intentionally duplicated inline — hooks are independent .cjs files and
@@ -102,41 +83,6 @@ const KNOWLEDGE_CANONICAL_TYPES = [
 ];
 const DEFAULT_UNDERSEED_NODE_THRESHOLD = 10;
 
-/**
- * Read the previously-emitted revision_hash from
- * `.fabric/.cache/sessionstart-last-hash`. Missing file / read failure /
- * empty file → null (treat as "no prior emit", forces re-emit).
- *
- * NEVER throws — best-effort read.
- */
-function readSessionStartLastHash(projectRoot) {
-  try {
-    const p = join(projectRoot, SESSIONSTART_HASH_CACHE_FILE);
-    if (!existsSync(p)) return null;
-    const raw = readFileSync(p, "utf8").trim();
-    return raw.length > 0 ? raw : null;
-  } catch {
-    return null;
-  }
-}
-
-/**
- * Write `hash` to `.fabric/.cache/sessionstart-last-hash` so subsequent
- * SessionStart fires can compare. Creates the directory if missing.
- * Best-effort: any write failure is swallowed so a read-only .fabric/
- * never blocks session start.
- */
-function writeSessionStartLastHash(projectRoot, hash) {
-  try {
-    if (typeof hash !== "string" || hash.length === 0) return;
-    const p = join(projectRoot, SESSIONSTART_HASH_CACHE_FILE);
-    mkdirSync(dirname(p), { recursive: true });
-    writeFileSync(p, hash, "utf8");
-  } catch {
-    // Silent — sidecar failure must never block session start.
-  }
-}
-
 // -----------------------------------------------------------------------------
 // rc.8 underseed self-check helpers.
 //
@@ -488,15 +434,21 @@ function renderTruncated(narrow) {
  * (empty narrow set) so callers know to stay silent.
  */
 function renderSummary(payload) {
-  const narrow = Array.isArray(payload && payload.narrow) ? payload.narrow : [];
-  if (narrow.length === 0) return [];
-
-  const truncated = narrow.length > TRUNCATION_THRESHOLD;
+  // Local rebind: `payload.narrow` in `--all` mode degenerates to the full
+  // shared index (every broad-scoped entry), so the field name `narrow` is
+  // misleading at this rendering layer. We rename the local variable to
+  // `entries` to avoid name confusion when reading renderSummary in isolation.
+  // The CLI protocol field name (`payload.narrow`) is unchanged — a wire-shape
+  // rename is a deferred independent task.
+  const entries = Array.isArray(payload && payload.narrow) ? payload.narrow : [];
+  if (entries.length === 0) return [];
+
+  const truncated = entries.length > TRUNCATION_THRESHOLD;
   const banner = truncated
-    ? `[fabric] Session start — ${narrow.length} broad-scoped knowledge entries available (truncated):`
-    : `[fabric] Session start — ${narrow.length} broad-scoped knowledge entries available:`;
+    ? `[fabric] Session start — ${entries.length} broad-scoped knowledge entries available (truncated):`
+    : `[fabric] Session start — ${entries.length} broad-scoped knowledge entries available:`;
 
-  const body = truncated ? renderTruncated(narrow) : renderFull(narrow);
+  const body = truncated ? renderTruncated(entries) : renderFull(entries);
 
   const lines = [banner, ...body];
   const revHash = typeof payload.revision_hash === "string" ? payload.revision_hash : null;
@@ -524,32 +476,15 @@ function main(env, stdio) {
     if (payload === null || payload === undefined) return; // silent
 
     // rc.8 underseed self-check: decide whether to surface the one-line
-    // `/fabric-import` recommendation. The decision is taken BEFORE the
-    // revision_hash gate so the banner can bypass it (an unchanged
-    // knowledge graph would otherwise hide the recommendation forever).
-    // The broad-summary BODY itself remains hash-gated below — only the
-    // banner line is unconditionally emitted when the probe says so.
+    // `/fabric-import` recommendation banner alongside the broad summary.
     const recommendImport = shouldRecommendImport(cwd);
 
-    // rc.7 T8: revision_hash gate. If the CLI payload carries a stable
-    // revision_hash and it matches the previously-emitted hash recorded in
-    // the sidecar, the knowledge graph is unchanged since last session →
-    // suppress the broad-summary body. The import-recommendation banner
-    // (when applicable) is still emitted below regardless of this gate.
-    const currentHash =
-      typeof payload.revision_hash === "string" ? payload.revision_hash : "";
-    let bodySuppressed = false;
-    if (currentHash.length > 0) {
-      const lastHash = readSessionStartLastHash(cwd);
-      if (lastHash !== null && lastHash === currentHash) {
-        bodySuppressed = true;
-      }
-    }
-
-    // Build emitted lines. When the body is hash-suppressed we skip the
-    // broad summary entirely; only the import banner (if applicable) goes
-    // to stderr in that case.
-    const lines = bodySuppressed ? [] : renderSummary(payload);
+    // rc.12: broad-summary body is unconditionally rendered on every
+    // SessionStart fire (Skill-style progressive disclosure). The prior
+    // revision_hash cooldown gate (rc.7 T8 — rc.11) was removed because
+    // compact/clear-triggered SessionStart re-fires must re-inject the menu
+    // for the agent's working memory.
+    const lines = renderSummary(payload);
 
     if (recommendImport) {
       lines.push(IMPORT_RECOMMENDATION_BANNER);
@@ -560,16 +495,6 @@ function main(env, stdio) {
     for (const line of lines) {
       err.write(`${line}\n`);
     }
-
-    // Update sidecar AFTER successful emit. We only persist the hash when
-    // the broad-summary body actually went out (i.e. the gate let the body
-    // through). If the body was suppressed but the banner emitted on its
-    // own, we deliberately do NOT bump the sidecar — the next session
-    // should still get to compare against the prior canonical-graph hash
-    // and re-emit the body when the graph actually changes.
-    if (!bodySuppressed && currentHash.length > 0) {
-      writeSessionStartLastHash(cwd, currentHash);
-    }
   } catch {
     // Silent — never block session start on hook failure.
   }
@@ -583,9 +508,6 @@ module.exports = {
   renderTruncated,
   renderSummary,
   truncateSummary,
-  // rc.7 T8: revision_hash gating sidecar helpers (exported for unit testing).
-  readSessionStartLastHash,
-  writeSessionStartLastHash,
   // rc.8 underseed self-check helpers (exported for unit testing).
   countCanonicalNodes,
   readUnderseedThreshold,
@@ -599,7 +521,6 @@ module.exports = {
     MATURITY_PROVEN,
     MATURITY_VERIFIED,
     MATURITY_DRAFT,
-    SESSIONSTART_HASH_CACHE_FILE,
     DEFAULT_UNDERSEED_NODE_THRESHOLD,
     KNOWLEDGE_CANONICAL_TYPES,
     IMPORT_RECOMMENDATION_BANNER,