@fenglimg/fabric-server 2.0.0-rc.11 → 2.0.0-rc.13

package/dist/{chunk-MMGHD42I.js → chunk-3ZBYYPXQ.js}

@@ -88,7 +88,7 @@ import { agentsMetaNodeSchema, agentsMetaSchema as agentsMetaSchema2 } from "@fe
 var AgentsMetaFileMissingError = class extends IOFabricError {
   constructor(metaPath, opts) {
     super(`Fabric agents metadata file is missing: ${metaPath}`, {
-      actionHint: opts?.actionHint ?? "Run `fab init` to scaffold the .fabric/agents.meta.json file"
+      actionHint: opts?.actionHint ?? "Run `fab install` to scaffold the .fabric/agents.meta.json file"
     });
     this.metaPath = metaPath;
   }
@@ -1382,12 +1382,13 @@ async function runDoctorReport(target) {
   const narrowTooFew = inspectNarrowTooFew(projectRoot, lintNow);
   const sessionHintsStale = inspectSessionHintsStale(projectRoot, lintNow);
   const relevanceFieldsMissing = inspectRelevanceFieldsMissing(projectRoot);
+  const skillMdYamlInvalid = inspectSkillMdYamlInvalid(projectRoot);
   const checks = [
     createBootstrapAnchorCheck(bootstrapAnchor),
     createKnowledgeDirMissingCheck(knowledgeDirMissing),
     createForensicCheck(forensic, framework.kind, entryPoints.length),
     // v2.0: removed `createInitContextCheck` — `.fabric/init-context.json`
-    // is owned by the AI-side client init skill, not by `fabric init` CLI.
+    // is owned by the AI-side client init skill, not by `fabric install` CLI.
     // The file's absence is a legitimate post-init state when the skill has
     // not yet run, so flagging it as a doctor manual_error misrepresents
     // ownership.
@@ -1441,6 +1442,9 @@ async function runDoctorReport(target) {
     // Info kind — applies to pending entries only; canonical entries get
     // the fields written verbatim by fab_review.approve/modify.
     createRelevanceFieldsMissingCheck(relevanceFieldsMissing),
+    // rc.12 lint #29: skill_md_yaml_invalid. Warning kind — surfaces
+    // SKILL.md frontmatter that Codex CLI silently drops at load.
+    createSkillMdYamlInvalidCheck(skillMdYamlInvalid),
     createPreexistingRootFilesCheck(preexistingRootFiles)
     // v2.0 / rc.2: `createLegacyClientPathCheck` removed. The schema now
     // rejects retired clientPaths keys (windsurf/rooCode/geminiCLI) at Zod
@@ -2128,7 +2132,7 @@ function createBootstrapAnchorCheck(inspection) {
       "fixable_error",
       "bootstrap_anchor_missing",
       "Neither AGENTS.md nor CLAUDE.md exists at the repo root. Fabric requires a bootstrap anchor file at the project root.",
-      "Run `fabric init` to generate the AGENTS.md / CLAUDE.md bootstrap anchor at the repo root."
+      "Run `fabric install` to generate the AGENTS.md / CLAUDE.md bootstrap anchor at the repo root."
     );
   }
   const present = [
@@ -2173,11 +2177,11 @@ function createForensicCheck(forensic, frameworkKind, entryPointCount) {
       "manual_error",
       "forensic_missing",
       `${forensic.error ?? ".fabric/forensic.json is missing."} Live scan detects ${frameworkKind} with ${entryPointCount} entry point${entryPointCount === 1 ? "" : "s"}.`,
-      "Run `fab init` to regenerate .fabric/forensic.json."
+      "Run `fab install` to regenerate .fabric/forensic.json."
     );
   }
   if (!forensic.valid) {
-    return issueCheck("Scan evidence", "error", "manual_error", "forensic_invalid", forensic.error ?? ".fabric/forensic.json is invalid.", "Run `fab init` to regenerate .fabric/forensic.json.");
+    return issueCheck("Scan evidence", "error", "manual_error", "forensic_invalid", forensic.error ?? ".fabric/forensic.json is invalid.", "Run `fab install` to regenerate .fabric/forensic.json.");
   }
   return okCheck("Scan evidence", `.fabric/forensic.json is valid for ${forensic.report?.framework.kind ?? "unknown"}.`);
 }
@@ -2257,8 +2261,8 @@ function createMcpConfigInWrongFileCheck(inspection) {
       "error",
       "fixable_error",
       "mcp_config_in_wrong_file",
-      `.claude/settings.json contains mcpServers.fabric \u2014 this file is for hooks/permissions only. Run --fix to remove it, then re-run fab init to write .mcp.json.`,
-      "Run `fab doctor --fix` to remove mcpServers.fabric from .claude/settings.json, then run `fab init` to write .mcp.json."
+      `.claude/settings.json contains mcpServers.fabric \u2014 this file is for hooks/permissions only. Run --fix to remove it, then re-run fab install to write .mcp.json.`,
+      "Run `fab doctor --fix` to remove mcpServers.fabric from .claude/settings.json, then run `fab install` to write .mcp.json."
     );
   }
   return okCheck("Claude MCP config location", "mcpServers.fabric is not in .claude/settings.json.");
@@ -3642,6 +3646,93 @@ function createRelevanceFieldsMissingCheck(inspection) {
     "Run `fab doctor --apply-lint` to back-fill the schema defaults (relevance_scope: broad, relevance_paths: [])."
   );
 }
+var SKILL_MD_FRONTMATTER_ROOTS = [".claude/skills", ".codex/skills"];
+var SKILL_FRONTMATTER_KEY_PATTERN = /^([A-Za-z_][A-Za-z0-9_-]*):[ \t]+(.+?)[ \t]*$/u;
+var SKILL_QUOTED_VALUE_LEADS = /* @__PURE__ */ new Set(['"', "'", "[", "{", ">", "|"]);
+function inspectSkillMdYamlInvalid(projectRoot) {
+  const candidates = [];
+  for (const rootRel of SKILL_MD_FRONTMATTER_ROOTS) {
+    const rootAbs = join5(projectRoot, rootRel);
+    if (!existsSync4(rootAbs)) continue;
+    let dirEntries;
+    try {
+      dirEntries = readdirSync(rootAbs, { withFileTypes: true });
+    } catch {
+      continue;
+    }
+    for (const dirEntry of dirEntries) {
+      if (!dirEntry.isDirectory()) continue;
+      const skillFile = join5(rootAbs, dirEntry.name, "SKILL.md");
+      if (!existsSync4(skillFile)) continue;
+      let raw;
+      try {
+        raw = readFileSync(skillFile, "utf8");
+      } catch {
+        continue;
+      }
+      const frontmatter = extractSkillFrontmatterLines(raw);
+      if (frontmatter === null) continue;
+      for (const { line, lineNumber } of frontmatter) {
+        const match = SKILL_FRONTMATTER_KEY_PATTERN.exec(line);
+        if (!match) continue;
+        const [, key, value] = match;
+        if (value.length === 0) continue;
+        if (SKILL_QUOTED_VALUE_LEADS.has(value[0])) continue;
+        const colonSpaceIdx = value.indexOf(": ");
+        const trailingColon = value.endsWith(":");
+        if (colonSpaceIdx < 0 && !trailingColon) continue;
+        const anchor = colonSpaceIdx >= 0 ? colonSpaceIdx : value.length - 1;
+        const previewStart = Math.max(0, anchor - 25);
+        const previewEnd = Math.min(value.length, anchor + 30);
+        const preview = `${previewStart > 0 ? "\u2026" : ""}${value.slice(previewStart, previewEnd)}${previewEnd < value.length ? "\u2026" : ""}`;
+        candidates.push({
+          path: posix.join(rootRel, dirEntry.name, "SKILL.md"),
+          line: lineNumber,
+          key,
+          preview
+        });
+      }
+    }
+  }
+  candidates.sort((a, b) => {
+    const byPath = a.path.localeCompare(b.path);
+    return byPath !== 0 ? byPath : a.line - b.line;
+  });
+  return { candidates };
+}
+function extractSkillFrontmatterLines(raw) {
+  const rawLines = raw.split(/\r?\n/u);
+  if (rawLines.length < 2) return null;
+  if (rawLines[0]?.trim() !== "---") return null;
+  const out = [];
+  for (let i = 1; i < rawLines.length; i++) {
+    const line = rawLines[i];
+    if (line.trim() === "---") {
+      return out;
+    }
+    out.push({ line, lineNumber: i + 1 });
+  }
+  return null;
+}
+function createSkillMdYamlInvalidCheck(inspection) {
+  if (inspection.candidates.length === 0) {
+    return okCheck(
+      "Skill markdown YAML",
+      "All .claude/.codex SKILL.md frontmatter values parse as strict YAML."
+    );
+  }
+  const first = inspection.candidates[0];
+  const detail = `${first.path}:${first.line} (key \`${first.key}\` value contains an unquoted ': ' \u2014 preview: \`${first.preview}\`)`;
+  const plural = inspection.candidates.length === 1;
+  return issueCheck(
+    "Skill markdown YAML",
+    "warn",
+    "warning",
+    "skill_md_yaml_invalid",
+    `${inspection.candidates.length} SKILL.md frontmatter ${plural ? "value contains" : "values contain"} an unquoted ': ' that strict YAML parsers reject (Claude Code tolerates it; Codex CLI drops the skill at load). First: ${detail}.`,
+    'Quote the value with double quotes (`description: "\u2026"`) or rewrite the inner `key: value` token to `key=value`.'
+  );
+}
 function createNarrowTooFewCheck(inspection) {
   const { structural_flagged, telemetry_flagged } = inspection;
   if (!structural_flagged && !telemetry_flagged) {

package/dist/{http-U5RL5Q6P.js → http-B3UKRP5Y.js}

@@ -14,7 +14,7 @@ import {
   readEventLedger,
   runDoctorReport,
   sha256
-} from "./chunk-MMGHD42I.js";
+} from "./chunk-3ZBYYPXQ.js";
 
 // src/http.ts
 import { randomUUID as randomUUID2 } from "crypto";
@@ -942,7 +942,7 @@ function toPosixPath(path) {
 function buildRecommendations(input) {
   const recommendations = [];
   if (!input.hasExistingFabric) {
-    recommendations.push("L0: Run `fab init` to scaffold the .fabric/ knowledge layout (decisions, pitfalls, guidelines, models, processes).");
+    recommendations.push("L0: Run `fab install` to scaffold the .fabric/ knowledge layout (decisions, pitfalls, guidelines, models, processes).");
   }
   if (input.readmeQuality === "stub") {
     recommendations.push("L0: Expand README.md before promoting project facts into Fabric knowledge entries.");

package/dist/index.js

@@ -26,7 +26,7 @@ import {
   sha256,
   stableStringify,
   writeKnowledgeMeta
-} from "./chunk-MMGHD42I.js";
+} from "./chunk-3ZBYYPXQ.js";
 
 // src/index.ts
 import { existsSync as existsSync4 } from "fs";
@@ -1890,7 +1890,7 @@ function formatPreexistingRootMessage(projectRoot) {
 function createFabricServer(tracker) {
   const server = new McpServer({
     name: "fabric-knowledge-server",
-    version: "2.0.0-rc.11"
+    version: "2.0.0-rc.13"
   });
   registerPlanContext(server, tracker);
   registerKnowledgeSections(server, tracker);
@@ -1990,7 +1990,7 @@ function createShutdownHandler(deps) {
   };
 }
 async function startHttpServer(options) {
-  const { createFabricHttpApp } = await import("./http-U5RL5Q6P.js");
+  const { createFabricHttpApp } = await import("./http-B3UKRP5Y.js");
   const { port, projectRoot, host = "127.0.0.1", authToken } = options;
   const app = createFabricHttpApp({ projectRoot, host, authToken });
   return await new Promise((resolveServer, rejectServer) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fenglimg/fabric-server",
-  "version": "2.0.0-rc.11",
+  "version": "2.0.0-rc.13",
   "type": "module",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",
@@ -13,7 +13,7 @@
     "express": "^5.2.1",
     "minimatch": "^10.0.1",
     "zod": "^3.25.0",
-    "@fenglimg/fabric-shared": "2.0.0-rc.11"
+    "@fenglimg/fabric-shared": "2.0.0-rc.13"
   },
   "devDependencies": {
     "@types/express": "^5.0.6",