@fenglimg/fabric-shared 2.3.0-rc.12 → 2.3.0-rc.16

package/dist/{chunk-TEITGZY3.js → chunk-5M3HB554.js}

@@ -709,7 +709,18 @@ var _fabReviewModifyChangesSchema = z3.object({
   // impact are flow-arrays.
   must_read_if: z3.string().optional(),
   intent_clues: z3.array(z3.string()).optional(),
-  impact: z3.array(z3.string()).optional()
+  impact: z3.array(z3.string()).optional(),
+  // ISS-20260711-180: propose writes these; without them here zod .strip()
+  // silently drops modify patches (KT-PIT-0005 recurrence).
+  tech_stack: z3.array(z3.string()).optional(),
+  evidence_paths: z3.array(z3.string()).optional(),
+  onboard_slot: z3.enum([
+    "tech-stack-decision",
+    "architecture-pattern",
+    "code-style-tone",
+    "build-system-idiom",
+    "domain-vocabulary"
+  ]).optional()
 });
 var FabReviewInputSchema = z3.discriminatedUnion("action", [
   z3.object({
@@ -863,6 +874,8 @@ var _fabReviewListItemSchema = z3.object({
   tags: z3.array(z3.string()).optional(),
   title: z3.string().optional(),
   summary: z3.string().optional(),
+  // ISS-20260712-011: triage UI needs proposed_reason without a second Read.
+  proposed_reason: z3.string().optional(),
   // Store-only cutover: origin reflects the resolved store audience where the
   // pending file lives; layer reflects the declared classification that will
   // drive approval semantics.
@@ -902,10 +915,16 @@ var _fabReviewSearchItemSchema = z3.object({
   // hits in the same array.
   stable_id: z3.string().optional()
 });
+var _fabReviewFailedItemSchema = z3.object({
+  pending_path: z3.string(),
+  reason: z3.string()
+});
 var FabReviewOutputSchema = z3.discriminatedUnion("action", [
   z3.object({
     action: z3.literal("approve"),
     approved: z3.array(z3.object({ pending_path: z3.string(), stable_id: z3.string() })),
+    // ISS-20260712-012: partial-failure surface so empty approved[] is not false-success.
+    failed: z3.array(_fabReviewFailedItemSchema).optional(),
     warnings: z3.array(structuredWarningSchema).optional()
   }),
   z3.object({
@@ -953,6 +972,8 @@ var FabReviewOutputSchema = z3.discriminatedUnion("action", [
         superseded_by: z3.string().optional()
       })
     ),
+    // ISS-20260712-012: partial-failure surface for skipped/unresolvable paths.
+    failed: z3.array(_fabReviewFailedItemSchema).optional(),
     warnings: z3.array(structuredWarningSchema).optional()
   })
 ]);
@@ -984,6 +1005,9 @@ var FabReviewOutputShape = {
   approved: z3.array(z3.object({ pending_path: z3.string(), stable_id: z3.string() })).optional().describe(
     "Allocated stable ids paired with their original pending paths. Present when action=approve."
   ),
+  failed: z3.array(z3.object({ pending_path: z3.string(), reason: z3.string() })).optional().describe(
+    "Paths that did not apply (approve/retire). Present when any path was skipped or failed."
+  ),
   rejected: z3.array(z3.string()).optional().describe(
     "Pending paths that were rejected (files retained on disk; doctor owns vacuum). Present when action=reject."
   ),

package/dist/{chunk-ASS2KBB7.js → chunk-OX6DWUK4.js}

@@ -1,3 +1,8 @@
+import {
+  atomicWriteJson,
+  withFileLock
+} from "./chunk-C7WZPYZE.js";
+
 // src/i18n/normalize-locale.ts
 function normalizeLocale(raw) {
   if (typeof raw !== "string") {
@@ -189,7 +194,8 @@ var globalConfigSchema = z.object({
 }).passthrough();
 
 // src/store/global-config-io.ts
-import { existsSync, mkdirSync, readFileSync, writeFileSync } from "fs";
+import { existsSync, mkdirSync, readFileSync, renameSync, unlinkSync, writeFileSync } from "fs";
+import { mkdir } from "fs/promises";
 import { homedir } from "os";
 import { join } from "path";
 function isTestRuntime() {
@@ -210,6 +216,9 @@ function resolveGlobalRoot() {
 function globalConfigPath(globalRoot = resolveGlobalRoot()) {
   return join(globalRoot, "fabric-global.json");
 }
+function globalConfigLockPath(globalRoot) {
+  return `${globalConfigPath(globalRoot)}.lock`;
+}
 function loadGlobalConfig(globalRoot = resolveGlobalRoot()) {
   const path = globalConfigPath(globalRoot);
   if (!existsSync(path)) {
@@ -217,11 +226,30 @@ function loadGlobalConfig(globalRoot = resolveGlobalRoot()) {
   }
   return globalConfigSchema.parse(JSON.parse(readFileSync(path, "utf8")));
 }
+async function saveGlobalConfigAsync(config, globalRoot = resolveGlobalRoot()) {
+  const validated = globalConfigSchema.parse(config);
+  await mkdir(globalRoot, { recursive: true });
+  const path = globalConfigPath(globalRoot);
+  await withFileLock(globalConfigLockPath(globalRoot), async () => {
+    await atomicWriteJson(path, validated, { indent: 2 });
+  });
+}
 function saveGlobalConfig(config, globalRoot = resolveGlobalRoot()) {
   const validated = globalConfigSchema.parse(config);
   mkdirSync(globalRoot, { recursive: true });
-  writeFileSync(globalConfigPath(globalRoot), `${JSON.stringify(validated, null, 2)}
+  const path = globalConfigPath(globalRoot);
+  const tmpPath = `${path}.${process.pid}.${Date.now()}.tmp`;
+  try {
+    writeFileSync(tmpPath, `${JSON.stringify(validated, null, 2)}
 `, "utf8");
+    renameSync(tmpPath, path);
+  } catch (error) {
+    try {
+      unlinkSync(tmpPath);
+    } catch {
+    }
+    throw error;
+  }
 }
 
 // src/i18n/resolve-global-locale.ts
@@ -269,6 +297,7 @@ export {
   resolveGlobalRoot,
   globalConfigPath,
   loadGlobalConfig,
+  saveGlobalConfigAsync,
   saveGlobalConfig,
   resolveGlobalLocale
 };

package/dist/{chunk-KFFBQRL5.js → chunk-V7IUN5U2.js}

@@ -1,6 +1,6 @@
 import {
   resolveGlobalLocale
-} from "./chunk-ASS2KBB7.js";
+} from "./chunk-OX6DWUK4.js";
 
 // src/templates/bootstrap-canonical.ts
 var BOOTSTRAP_MARKER_BEGIN = "<!-- fabric:bootstrap:begin -->";
@@ -36,8 +36,9 @@ var BOOTSTRAP_CANONICAL_ZH = `# Fabric Bootstrap
 \u5B8C\u6574 maintainer \u7248\u89C1 \`docs/USER-QUICKSTART.md\`\u3002
 
 ## \u884C\u4E3A\u89C4\u5219
-- **\u4FEE\u6539\u4EFB\u4F55\u6587\u4EF6\u524D**:\u5148 \`fab_recall(paths=[<\u88AB\u6539\u6587\u4EF6>])\` \u2014\u2014 \u4E00\u6B21\u8C03\u7528\u62FF\u56DE\u76F8\u5173 KB \u7684\u63CF\u8FF0 + \u539F\u751F\u8BFB\u53D6\u8DEF\u5F84(\`entries[].read_path\`)\u3002\`fab_recall\` \u4E0D\u518D\u6295\u9012\u6B63\u6587;\u9700\u8981\u67D0\u6761\u6B63\u6587\u65F6\u76F4\u63A5\u5BF9\u5176 \`entries[].read_path\` \u505A\u539F\u751F Read(\`Read <store>/knowledge/<type>/<id>--*.md\`),\u8FD9\u4F1A\u88AB PostToolUse hook \u8BB0\u4E3A \`knowledge_body_read\`\u3002lean \u9ED8\u8BA4:\u63CF\u8FF0+\u7D22\u5F15\u5DF2\u591F\u53D1\u73B0\u6761\u76EE,\u6B63\u6587\u6309\u9700\u8BFB\u4E00\u6B21,\u4E0D\u6BCF\u8F6E\u91CD\u704C(KT-GLD-0005)\u3002
-- **\`.fabric/agents.meta.json\` \u4E25\u7981\u624B\u52A8\u7F16\u8F91**;engine \u4F1A\u81EA\u52A8\u540C\u6B65\u6D3E\u751F\u72B6\u6001,\u663E\u5F0F reconcile \u8DD1 \`fabric doctor --fix\`\u3002
+- **Pre-action gating (\u4FEE\u6539\u4EFB\u4F55\u6587\u4EF6\u524D MUST)**: \u2460 \u5148 \`fab_recall(paths=[<\u88AB\u6539\u6587\u4EF6>], session_id=<client-session-id>)\`\uFF1B\u2461 \u6309\u9700\u539F\u751F Read \u547D\u4E2D\u7684 \`entries[].read_path\` \u6B63\u6587(PostToolUse \u8BB0\u4E3A \`knowledge_body_read\`)\uFF1B\u2462 \u8DF3\u8FC7\u4E0D\u9002\u7528\u547D\u4E2D\u65F6\u8BF4 \`dismissed: <id> (<reason>)\`\u3002\`fab_recall\` \u4E0D\u518D\u6295\u9012\u6B63\u6587;lean \u9ED8\u8BA4\u63CF\u8FF0+\u7D22\u5F15\u5DF2\u591F\u53D1\u73B0\u6761\u76EE(KT-GLD-0005)\u3002PreToolUse \u4EC5\u8F6F nudge,\u5B88 KT-DEC-0007(\u975E permanent gate / \u65E0 \`decision:block\`)\u3002
+- **Archive-as-truth**: \u8FDB canonical **\u552F\u4E00\u8DEF\u5F84** \u662F \`fab_propose\` \u2192 pending \u2192 \`fabric-review\`/\`fab_review\` \u5BA1\u6279\uFF1B\u7981\u6B62\u628A chat/log \u81EA\u52A8 promote \u8FDB canonical\u3002
+- **\`.fabric/agents.meta.json\` \u4E25\u7981\u624B\u52A8\u7F16\u8F91**\uFF08co-location index \u5DF2\u9000\u5F79;\u77E5\u8BC6\u5728 mounted stores\uFF09\u3002\u5F02\u5E38\u65F6\u8DD1 \`fabric doctor\` / \`fabric doctor --fix\` \u81EA\u6108\u53EF\u4FEE\u590D\u9879\u3002
 
 ## \u77E5\u8BC6\u5E93(KB)
 - **Discovery**:SessionStart hook \u5217 broad-scoped \u6761\u76EE(\u6761\u76EE\u6309 \`semantic_scope\` \u5206\u4E09\u5C42:\`team\` \u56E2\u961F\u901A\u7528 / \`project:<id>\` \u672C\u9879\u76EE\u4E13\u5C5E(\u4EC5\u5728\u7ED1\u5B9A\u8BE5\u9879\u76EE\u7684\u4ED3\u5E93\u6D6E\u73B0)/ \`personal\` \u4E2A\u4EBA \`KP-*\`,\u4E09\u8005\u5F15\u7528\u65B9\u5F0F\u76F8\u540C);edit \u6587\u4EF6\u65F6 PreToolUse hook \u53EF\u80FD\u89E6\u53D1 narrow hint\u3002
@@ -46,7 +47,7 @@ var BOOTSTRAP_CANONICAL_ZH = `# Fabric Bootstrap
 - **session_id**: \u8C03\u7528 \`fab_recall\` \u65F6, \u52A1\u5FC5\u628A\u5F53\u524D client session id \u4F5C\u4E3A \`session_id\` \u53C2\u6570\u4F20\u5165(Claude Code \u7684 session id \u5728 stdin payload \u4E2D, Codex \u7684\u5BF9\u5E94 identifier \u540C\u7406)\u3002\u8FD9\u80FD\u8BA9 \`fabric doctor --archive-history\` \u4E0E \`fabric-hint.cjs\` Stop hook \u51C6\u786E\u8BC6\u522B\u8DE8\u4F1A\u8BDD debt \u72B6\u6001\u3002
 - **Skills (4)**:\u5199\u6D41\u7A0B \`fabric-archive\`(\u542B source mode \u51B7\u542F\u52A8\u4ECE git/docs \u56DE\u704C)/ \`fabric-review\`(\u542B retire \u8BED\u4E49\u6DD8\u6C70 + relate \u5173\u8054\u5EFA\u8FB9 \u5B50\u6D41\u7A0B);store \u8FD0\u7EF4 \`fabric-store\` / \`fabric-sync\`\u3002
 - **Language**:\u6E32\u67D3\u6309 \`~/.fabric/fabric-global.json\` \u7684 \`language\` \u5B57\u6BB5(machine-wide tone)\u3002
-- **Archive cadence nudge** (rc.36): \u6BCF\u5B8C\u6210\u4E00\u6279 Edit(\u9ED8\u8BA4 ~20 \u6B21, \u4E0E Stop hook \u9608\u503C config \`archive_edit_threshold\` \u4E00\u81F4)/ \u663E\u8457 decision \u540E,\u5728\u5408\u9002\u56DE\u5408\u4E3B\u52A8 propose \u8C03 \`fabric-archive\` skill \u2014 archive \u6CA1\u5EFA\u7ACB\u9891\u7387\u4F1A\u8BA9 KB \u6162\u901F\u6B7B\u6389\u3002
+- **Archive cadence nudge** (rc.36 / finish\u2192archive): \u663E\u8457 decision \u6536\u53E3\u6216\u4E00\u6279 Edit \u8FBE\u5230 config \`archive_edit_threshold\`(\u9ED8\u8BA4 20) \u540E,\u5728\u5408\u9002\u56DE\u5408\u8F7B\u91CF\u81EA\u8C03 \`fabric-archive\`(\u540C turn \u6700\u591A 1 \u6B21;\u975E task engine / \u975E Stop-hook flood)\u3002Stop hook \u4EC5 soft threshold nudge,\u5B88 KT-DEC-0007 \u2014 archive \u6CA1\u5EFA\u7ACB\u9891\u7387\u4F1A\u8BA9 KB \u6162\u901F\u6B7B\u6389\u3002
 - **Review backlog nudge** (rc.36): \u9700\u8981\u5224\u65AD pending backlog \u65F6\u8D70 \`fab_pending action="list"\` \u6216 \`fabric-review\` \u8FD4\u56DE\u7684 \`pending_path\`;\u4E0D\u8981 glob \u9879\u76EE\u672C\u5730 \`.fabric/knowledge/pending\`\u3002\u5F53\u53EF\u89C1 pending \u7D2F\u79EF >10 \u6761\u65F6,\u5728\u5408\u9002\u56DE\u5408\u4E3B\u52A8 propose \u8C03 \`fabric-review\` skill \u6279\u91CF\u5BA1,\u907F\u514D draft \u5361\u6B7B\u3002
 
 ## Self-archive policy (v2.2 C1: \u7CBE\u7B80\u8BF4\u660E\u4E66)
@@ -95,8 +96,9 @@ As a dev you only need to: run \`fabric install\` once per repo, use \`fabric st
 See \`docs/USER-QUICKSTART.md\` for the full maintainer version.
 
 ## Behavior Rules
-- **Before modifying any file**: first \`fab_recall(paths=[<file-being-edited>])\` \u2014\u2014 a single call returns the relevant KB descriptions + native read paths (\`entries[].read_path\`). \`fab_recall\` no longer delivers bodies; when you need a body, do a native Read of its \`entries[].read_path\` (\`Read <store>/knowledge/<type>/<id>--*.md\`), which the PostToolUse hook records as \`knowledge_body_read\`. Lean default: descriptions + index already suffice to discover entries; read a body once on demand, don't re-inject it every turn (KT-GLD-0005).
-- **Never hand-edit \`.fabric/agents.meta.json\`**; the engine syncs derived state automatically \u2014 run \`fabric doctor --fix\` for an explicit reconcile.
+- **Pre-action gating (MUST before Edit/Write)**: (1) call \`fab_recall(paths=[<file-being-edited>], session_id=<client-session-id>)\`; (2) optionally native-Read selected \`entries[].read_path\` bodies (PostToolUse records \`knowledge_body_read\`); (3) when skipping a hit, say \`dismissed: <id> (<reason>)\`. \`fab_recall\` no longer delivers bodies; lean default is description+index discovery (KT-GLD-0005). PreToolUse is soft nudge only per KT-DEC-0007 (never permanent gate / no \`decision:block\`).
+- **Archive-as-truth**: the **only** path into canonical is \`fab_propose\` \u2192 pending \u2192 \`fabric-review\`/\`fab_review\` approve; never auto-promote chat/logs into canonical.
+- **Never hand-edit \`.fabric/agents.meta.json\`** (co-location index retired; knowledge lives in mounted stores). For anomalies run \`fabric doctor\` / \`fabric doctor --fix\` on fixable state.
 
 ## Knowledge Base (KB)
 - **Discovery**: the SessionStart hook lists broad-scoped entries (scoped by \`semantic_scope\` across three tiers: \`team\` team-wide / \`project:<id>\` this-project-only (surfaces only in repos bound to that project) / \`personal\` \`KP-*\`, all three referenced the same way); editing a file may trigger a narrow hint via the PreToolUse hook.
@@ -105,7 +107,7 @@ See \`docs/USER-QUICKSTART.md\` for the full maintainer version.
 - **session_id**: when calling \`fab_recall\`, always pass the current client session id as the \`session_id\` argument (Claude Code's session id is in the stdin payload; Codex's corresponding identifier likewise). This lets \`fabric doctor --archive-history\` and the \`fabric-hint.cjs\` Stop hook track cross-session debt accurately.
 - **Skills (4)**: write flow \`fabric-archive\` (with source-mode cold-start backfill from git/docs) / \`fabric-review\` (with retire-deprecation + relate-edge sub-flows); store ops \`fabric-store\` / \`fabric-sync\`.
 - **Language**: rendered per the \`language\` field in \`~/.fabric/fabric-global.json\` (machine-wide tone).
-- **Archive cadence nudge** (rc.36): after each batch of edits (default ~20, matching the Stop hook threshold config \`archive_edit_threshold\`) / a significant decision, proactively propose the \`fabric-archive\` skill at a suitable turn \u2014 without an archive cadence the KB slowly dies.
+- **Archive cadence nudge** (rc.36 / finish\u2192archive): after a significant decision lands or an edit batch reaches config \`archive_edit_threshold\` (default 20), lightly self-trigger \`fabric-archive\` at a suitable turn (max 1 per turn; not a task engine / not Stop-hook flood). Stop hook remains a soft threshold nudge per KT-DEC-0007 \u2014 without an archive cadence the KB slowly dies.
 - **Review backlog nudge** (rc.36): to judge the pending backlog, go through \`fab_pending action="list"\` or the \`pending_path\` returned by \`fabric-review\`; don't glob the project-local \`.fabric/knowledge/pending\`. When the visible pending count exceeds 10, proactively propose the \`fabric-review\` skill at a suitable turn to batch-review and avoid draft deadlock.
 
 ## Self-archive policy (v2.2 C1: lean spec)