rulesync 8.18.0 → 8.19.0

package/dist/{chunk-KDGHMXTY.js → chunk-66SPWRYP.js}

@@ -1158,11 +1158,18 @@ import * as smolToml from "smol-toml";
 // src/utils/frontmatter.ts
 import matter from "gray-matter";
 import { dump, load } from "js-yaml";
+
+// src/utils/type-guards.ts
+function isRecord(value) {
+  return typeof value === "object" && value !== null && !Array.isArray(value);
+}
 function isPlainObject(value) {
-  if (value === null || typeof value !== "object") return false;
-  const prototype = Object.getPrototypeOf(value);
-  return prototype === Object.prototype || prototype === null;
+  if (!isRecord(value)) return false;
+  const proto = Object.getPrototypeOf(value);
+  return proto === null || proto === Object.prototype;
 }
+
+// src/utils/frontmatter.ts
 function deepRemoveNullishValue(value) {
   if (value === null || value === void 0) {
     return void 0;
@@ -1753,11 +1760,6 @@ var AgentsmdCommand = class _AgentsmdCommand extends SimulatedCommand {
 import { basename, join as join7 } from "path";
 import { z as z6 } from "zod/mini";
 
-// src/utils/type-guards.ts
-function isRecord(value) {
-  return typeof value === "object" && value !== null && !Array.isArray(value);
-}
-
 // src/features/commands/rulesync-command.ts
 import { join as join6 } from "path";
 import { z as z5 } from "zod/mini";
@@ -7760,7 +7762,7 @@ var ToolMcp = class extends ToolFile {
     super({
       ...rest,
       validate: true
-      // Skip validation during construction
+      // ToolMcp runs subclass validation below when requested
     });
     if (rest.validate) {
       const result = this.validate();
@@ -7818,17 +7820,26 @@ var ClaudecodeMcp = class _ClaudecodeMcp extends ToolMcp {
     return this.json;
   }
   /**
-   * In global mode, ~/.claude/.claude.json should not be deleted
-   * as it may contain other user settings.
+   * In global mode, ~/.claude.json should not be deleted as it is the
+   * user's primary Claude Code config and contains many other settings
+   * managed by Claude Code itself (feature flags, project trust list,
+   * hooks, user settings, model selection, etc.).
    * In local mode, .mcp.json can be safely deleted.
    */
   isDeletable() {
     return !this.global;
   }
+  /**
+   * Legacy global path used by rulesync ≤ v8.17.0. The documented store
+   * is `~/.claude.json`; `fromFile` falls back here with a deprecation
+   * warning (mirrors PR #333). Never modified or removed by rulesync.
+   */
+  static LEGACY_GLOBAL_DIR = ".claude";
+  static LEGACY_GLOBAL_FILE = ".claude.json";
   static getSettablePaths({ global } = {}) {
     if (global) {
       return {
-        relativeDirPath: ".claude",
+        relativeDirPath: ".",
         relativeFilePath: ".claude.json"
       };
     }
@@ -7840,19 +7851,52 @@ var ClaudecodeMcp = class _ClaudecodeMcp extends ToolMcp {
   static async fromFile({
     outputRoot = process.cwd(),
     validate = true,
-    global = false
+    global = false,
+    logger
   }) {
     const paths = this.getSettablePaths({ global });
-    const fileContent = await readFileContentOrNull(
-      join49(outputRoot, paths.relativeDirPath, paths.relativeFilePath)
-    ) ?? '{"mcpServers":{}}';
-    const json = JSON.parse(fileContent);
-    const newJson = { ...json, mcpServers: json.mcpServers ?? {} };
+    const recommendedPath = join49(outputRoot, paths.relativeDirPath, paths.relativeFilePath);
+    if (await fileExists(recommendedPath)) {
+      const fileContent = await readFileContent(recommendedPath);
+      const json = JSON.parse(fileContent);
+      const newJson = { ...json, mcpServers: json.mcpServers ?? {} };
+      return new _ClaudecodeMcp({
+        outputRoot,
+        relativeDirPath: paths.relativeDirPath,
+        relativeFilePath: paths.relativeFilePath,
+        fileContent: JSON.stringify(newJson, null, 2),
+        validate,
+        global
+      });
+    }
+    if (global) {
+      const legacyPath = join49(
+        outputRoot,
+        _ClaudecodeMcp.LEGACY_GLOBAL_DIR,
+        _ClaudecodeMcp.LEGACY_GLOBAL_FILE
+      );
+      if (await fileExists(legacyPath)) {
+        logger?.warn(
+          `Warning: using deprecated path "${legacyPath}". Please migrate to "${recommendedPath}"`
+        );
+        const fileContent = await readFileContent(legacyPath);
+        const json = JSON.parse(fileContent);
+        const newJson = { ...json, mcpServers: json.mcpServers ?? {} };
+        return new _ClaudecodeMcp({
+          outputRoot,
+          relativeDirPath: _ClaudecodeMcp.LEGACY_GLOBAL_DIR,
+          relativeFilePath: _ClaudecodeMcp.LEGACY_GLOBAL_FILE,
+          fileContent: JSON.stringify(newJson, null, 2),
+          validate,
+          global
+        });
+      }
+    }
     return new _ClaudecodeMcp({
       outputRoot,
       relativeDirPath: paths.relativeDirPath,
       relativeFilePath: paths.relativeFilePath,
-      fileContent: JSON.stringify(newJson, null, 2),
+      fileContent: JSON.stringify({ mcpServers: {} }, null, 2),
       validate,
       global
     });
@@ -7881,7 +7925,7 @@ var ClaudecodeMcp = class _ClaudecodeMcp extends ToolMcp {
   }
   toRulesyncMcp() {
     return this.toRulesyncMcpDefault({
-      fileContent: JSON.stringify({ mcpServers: this.json.mcpServers }, null, 2)
+      fileContent: JSON.stringify({ mcpServers: this.json.mcpServers ?? {} }, null, 2)
     });
   }
   validate() {
@@ -7985,11 +8029,6 @@ import { join as join51 } from "path";
 import * as smolToml3 from "smol-toml";
 var MAX_REMOVE_EMPTY_ENTRIES_DEPTH = 32;
 var PROTOTYPE_POLLUTION_KEYS = /* @__PURE__ */ new Set(["__proto__", "constructor", "prototype"]);
-function isPlainObject2(value) {
-  if (!isRecord(value)) return false;
-  const proto = Object.getPrototypeOf(value);
-  return proto === null || proto === Object.prototype;
-}
 function convertFromCodexFormat(codexMcp) {
   const result = {};
   for (const [name, config] of Object.entries(codexMcp)) {
@@ -8145,7 +8184,7 @@ var CodexcliMcp = class _CodexcliMcp extends ToolMcp {
     for (const [key, value] of Object.entries(obj)) {
       if (PROTOTYPE_POLLUTION_KEYS.has(key)) continue;
       if (value === null) continue;
-      if (isPlainObject2(value)) {
+      if (isPlainObject(value)) {
         const cleaned = this.removeEmptyEntries(value, depth + 1);
         if (Object.keys(cleaned).length === 0) continue;
         filtered[key] = cleaned;
@@ -9991,7 +10030,8 @@ var McpProcessor = class extends FeatureProcessor {
         await factory.class.fromFile({
           outputRoot: this.outputRoot,
           validate: true,
-          global: this.global
+          global: this.global,
+          logger: this.logger
         })
       ];
       this.logger.debug(`Successfully loaded ${toolMcps.length} ${this.toolTarget} MCP files`);
@@ -19147,15 +19187,52 @@ var OpenCodeStyleSubagent = class extends ToolSubagent {
 // src/features/subagents/kilo-subagent.ts
 var KiloSubagentFrontmatterSchema = z71.looseObject({
   description: z71.optional(z71.string()),
-  // Kilo's documented default for user-defined agents is "all":
-  // available both as a top-level pick and as a subagent.
   mode: z71._default(z71.string(), "all"),
-  name: z71.optional(z71.string())
+  name: z71.optional(z71.string()),
+  displayName: z71.optional(z71.string()),
+  deprecated: z71.optional(z71.boolean()),
+  native: z71.optional(z71.boolean()),
+  hidden: z71.optional(z71.boolean()),
+  top_p: z71.optional(z71.number()),
+  temperature: z71.optional(z71.number()),
+  color: z71.optional(z71.string()),
+  permission: z71.optional(z71.string()),
+  model: z71.optional(z71.string()),
+  variant: z71.optional(z71.string()),
+  prompt: z71.optional(z71.string()),
+  options: z71.optional(z71.looseObject({})),
+  steps: z71.optional(z71.array(z71.looseObject({}))),
+  disable: z71.optional(z71.boolean())
 });
 var KiloSubagent = class _KiloSubagent extends OpenCodeStyleSubagent {
+  constructor(params) {
+    super(params);
+    if (params.validate !== false) {
+      const result = this.validate();
+      if (!result.success) {
+        throw result.error;
+      }
+    }
+  }
   getToolTarget() {
     return "kilo";
   }
+  getFrontmatter() {
+    return this.frontmatter;
+  }
+  validate() {
+    const result = KiloSubagentFrontmatterSchema.safeParse(this.frontmatter);
+    if (result.success) {
+      this.frontmatter = result.data;
+      return { success: true, error: null };
+    }
+    return {
+      success: false,
+      error: new Error(
+        `Invalid frontmatter in ${join118(this.relativeDirPath, this.relativeFilePath)}: ${formatError(result.error)}`
+      )
+    };
+  }
   static getSettablePaths({
     global = false
   } = {}) {
@@ -19171,11 +19248,17 @@ var KiloSubagent = class _KiloSubagent extends OpenCodeStyleSubagent {
   }) {
     const rulesyncFrontmatter = rulesyncSubagent.getFrontmatter();
     const kiloSection = rulesyncFrontmatter.kilo ?? {};
-    const kiloFrontmatter = KiloSubagentFrontmatterSchema.parse({
+    const parseResult = KiloSubagentFrontmatterSchema.safeParse({
       ...kiloSection,
       description: rulesyncFrontmatter.description,
       ...rulesyncFrontmatter.name && { name: rulesyncFrontmatter.name }
     });
+    if (!parseResult.success) {
+      throw new Error(
+        `Invalid frontmatter in ${rulesyncSubagent.getRelativeFilePath()}: ${formatError(parseResult.error)}`
+      );
+    }
+    const kiloFrontmatter = parseResult.data;
     const body = rulesyncSubagent.getBody();
     const fileContent = stringifyFrontmatter(body, kiloFrontmatter);
     const paths = this.getSettablePaths({ global });
@@ -19486,7 +19569,8 @@ var OpenCodeSubagent = class _OpenCodeSubagent extends OpenCodeStyleSubagent {
   static forDeletion({
     outputRoot = process.cwd(),
     relativeDirPath,
-    relativeFilePath
+    relativeFilePath,
+    global = false
   }) {
     return new _OpenCodeSubagent({
       outputRoot,
@@ -19495,7 +19579,8 @@ var OpenCodeSubagent = class _OpenCodeSubagent extends OpenCodeStyleSubagent {
       frontmatter: { description: "", mode: "subagent" },
       body: "",
       fileContent: "",
-      validate: false
+      validate: false,
+      global
     });
   }
 };
@@ -21299,8 +21384,8 @@ var CopilotRuleFrontmatterSchema = z78.object({
   applyTo: z78.optional(z78.string()),
   excludeAgent: z78.optional(z78.union([z78.literal("code-review"), z78.literal("coding-agent")]))
 });
-var normalizeRelativePath = (path4) => path4.replace(/\\/g, "/").replace(/\/+/g, "/");
-var sameRelativePath = (leftDir, leftFile, rightDir, rightFile) => normalizeRelativePath(join133(leftDir, leftFile)) === normalizeRelativePath(join133(rightDir, rightFile));
+var normalizeRelativePath = (p) => toPosixPath(p).replace(/\/+/g, "/");
+var sameRelativePath = (left, right) => normalizeRelativePath(join133(left.dir, left.file)) === normalizeRelativePath(join133(right.dir, right.file));
 var CopilotRule = class _CopilotRule extends ToolRule {
   frontmatter;
   body;
@@ -21419,10 +21504,8 @@ var CopilotRule = class _CopilotRule extends ToolRule {
   }) {
     const paths = this.getSettablePaths({ global });
     const isRoot = relativeDirPath ? sameRelativePath(
-      relativeDirPath,
-      relativeFilePath,
-      paths.root.relativeDirPath,
-      paths.root.relativeFilePath
+      { dir: relativeDirPath, file: relativeFilePath },
+      { dir: paths.root.relativeDirPath, file: paths.root.relativeFilePath }
     ) : relativeFilePath === paths.root.relativeFilePath;
     const resolvedRelativeDirPath = relativeDirPath ?? (isRoot ? paths.root.relativeDirPath : paths.nonRoot?.relativeDirPath ?? paths.root.relativeDirPath);
     if (isRoot) {
@@ -21468,10 +21551,8 @@ var CopilotRule = class _CopilotRule extends ToolRule {
   }) {
     const paths = this.getSettablePaths({ global });
     const isRoot = sameRelativePath(
-      relativeDirPath,
-      relativeFilePath,
-      paths.root.relativeDirPath,
-      paths.root.relativeFilePath
+      { dir: relativeDirPath, file: relativeFilePath },
+      { dir: paths.root.relativeDirPath, file: paths.root.relativeFilePath }
     );
     return new _CopilotRule({
       outputRoot,

package/dist/cli/index.cjs

@@ -1191,11 +1191,18 @@ var smolToml = __toESM(require("smol-toml"), 1);
 // src/utils/frontmatter.ts
 var import_gray_matter = __toESM(require("gray-matter"), 1);
 var import_js_yaml = require("js-yaml");
+
+// src/utils/type-guards.ts
+function isRecord(value) {
+  return typeof value === "object" && value !== null && !Array.isArray(value);
+}
 function isPlainObject(value) {
-  if (value === null || typeof value !== "object") return false;
-  const prototype = Object.getPrototypeOf(value);
-  return prototype === Object.prototype || prototype === null;
+  if (!isRecord(value)) return false;
+  const proto = Object.getPrototypeOf(value);
+  return proto === null || proto === Object.prototype;
 }
+
+// src/utils/frontmatter.ts
 function deepRemoveNullishValue(value) {
   if (value === null || value === void 0) {
     return void 0;
@@ -1786,11 +1793,6 @@ var AgentsmdCommand = class _AgentsmdCommand extends SimulatedCommand {
 var import_node_path10 = require("path");
 var import_mini6 = require("zod/mini");
 
-// src/utils/type-guards.ts
-function isRecord(value) {
-  return typeof value === "object" && value !== null && !Array.isArray(value);
-}
-
 // src/features/commands/rulesync-command.ts
 var import_node_path9 = require("path");
 var import_mini5 = require("zod/mini");
@@ -7793,7 +7795,7 @@ var ToolMcp = class extends ToolFile {
     super({
       ...rest,
       validate: true
-      // Skip validation during construction
+      // ToolMcp runs subclass validation below when requested
     });
     if (rest.validate) {
       const result = this.validate();
@@ -7851,17 +7853,26 @@ var ClaudecodeMcp = class _ClaudecodeMcp extends ToolMcp {
     return this.json;
   }
   /**
-   * In global mode, ~/.claude/.claude.json should not be deleted
-   * as it may contain other user settings.
+   * In global mode, ~/.claude.json should not be deleted as it is the
+   * user's primary Claude Code config and contains many other settings
+   * managed by Claude Code itself (feature flags, project trust list,
+   * hooks, user settings, model selection, etc.).
    * In local mode, .mcp.json can be safely deleted.
    */
   isDeletable() {
     return !this.global;
   }
+  /**
+   * Legacy global path used by rulesync ≤ v8.17.0. The documented store
+   * is `~/.claude.json`; `fromFile` falls back here with a deprecation
+   * warning (mirrors PR #333). Never modified or removed by rulesync.
+   */
+  static LEGACY_GLOBAL_DIR = ".claude";
+  static LEGACY_GLOBAL_FILE = ".claude.json";
   static getSettablePaths({ global } = {}) {
     if (global) {
       return {
-        relativeDirPath: ".claude",
+        relativeDirPath: ".",
         relativeFilePath: ".claude.json"
       };
     }
@@ -7873,19 +7884,52 @@ var ClaudecodeMcp = class _ClaudecodeMcp extends ToolMcp {
   static async fromFile({
     outputRoot = process.cwd(),
     validate = true,
-    global = false
+    global = false,
+    logger: logger5
   }) {
     const paths = this.getSettablePaths({ global });
-    const fileContent = await readFileContentOrNull(
-      (0, import_node_path52.join)(outputRoot, paths.relativeDirPath, paths.relativeFilePath)
-    ) ?? '{"mcpServers":{}}';
-    const json = JSON.parse(fileContent);
-    const newJson = { ...json, mcpServers: json.mcpServers ?? {} };
+    const recommendedPath = (0, import_node_path52.join)(outputRoot, paths.relativeDirPath, paths.relativeFilePath);
+    if (await fileExists(recommendedPath)) {
+      const fileContent = await readFileContent(recommendedPath);
+      const json = JSON.parse(fileContent);
+      const newJson = { ...json, mcpServers: json.mcpServers ?? {} };
+      return new _ClaudecodeMcp({
+        outputRoot,
+        relativeDirPath: paths.relativeDirPath,
+        relativeFilePath: paths.relativeFilePath,
+        fileContent: JSON.stringify(newJson, null, 2),
+        validate,
+        global
+      });
+    }
+    if (global) {
+      const legacyPath = (0, import_node_path52.join)(
+        outputRoot,
+        _ClaudecodeMcp.LEGACY_GLOBAL_DIR,
+        _ClaudecodeMcp.LEGACY_GLOBAL_FILE
+      );
+      if (await fileExists(legacyPath)) {
+        logger5?.warn(
+          `Warning: using deprecated path "${legacyPath}". Please migrate to "${recommendedPath}"`
+        );
+        const fileContent = await readFileContent(legacyPath);
+        const json = JSON.parse(fileContent);
+        const newJson = { ...json, mcpServers: json.mcpServers ?? {} };
+        return new _ClaudecodeMcp({
+          outputRoot,
+          relativeDirPath: _ClaudecodeMcp.LEGACY_GLOBAL_DIR,
+          relativeFilePath: _ClaudecodeMcp.LEGACY_GLOBAL_FILE,
+          fileContent: JSON.stringify(newJson, null, 2),
+          validate,
+          global
+        });
+      }
+    }
     return new _ClaudecodeMcp({
       outputRoot,
       relativeDirPath: paths.relativeDirPath,
       relativeFilePath: paths.relativeFilePath,
-      fileContent: JSON.stringify(newJson, null, 2),
+      fileContent: JSON.stringify({ mcpServers: {} }, null, 2),
       validate,
       global
     });
@@ -7914,7 +7958,7 @@ var ClaudecodeMcp = class _ClaudecodeMcp extends ToolMcp {
   }
   toRulesyncMcp() {
     return this.toRulesyncMcpDefault({
-      fileContent: JSON.stringify({ mcpServers: this.json.mcpServers }, null, 2)
+      fileContent: JSON.stringify({ mcpServers: this.json.mcpServers ?? {} }, null, 2)
     });
   }
   validate() {
@@ -8018,11 +8062,6 @@ var import_node_path54 = require("path");
 var smolToml3 = __toESM(require("smol-toml"), 1);
 var MAX_REMOVE_EMPTY_ENTRIES_DEPTH = 32;
 var PROTOTYPE_POLLUTION_KEYS = /* @__PURE__ */ new Set(["__proto__", "constructor", "prototype"]);
-function isPlainObject2(value) {
-  if (!isRecord(value)) return false;
-  const proto = Object.getPrototypeOf(value);
-  return proto === null || proto === Object.prototype;
-}
 function convertFromCodexFormat(codexMcp) {
   const result = {};
   for (const [name, config] of Object.entries(codexMcp)) {
@@ -8178,7 +8217,7 @@ var CodexcliMcp = class _CodexcliMcp extends ToolMcp {
     for (const [key, value] of Object.entries(obj)) {
       if (PROTOTYPE_POLLUTION_KEYS.has(key)) continue;
       if (value === null) continue;
-      if (isPlainObject2(value)) {
+      if (isPlainObject(value)) {
         const cleaned = this.removeEmptyEntries(value, depth + 1);
         if (Object.keys(cleaned).length === 0) continue;
         filtered[key] = cleaned;
@@ -10024,7 +10063,8 @@ var McpProcessor = class extends FeatureProcessor {
         await factory.class.fromFile({
           outputRoot: this.outputRoot,
           validate: true,
-          global: this.global
+          global: this.global,
+          logger: this.logger
         })
       ];
       this.logger.debug(`Successfully loaded ${toolMcps.length} ${this.toolTarget} MCP files`);
@@ -19180,15 +19220,52 @@ var OpenCodeStyleSubagent = class extends ToolSubagent {
 // src/features/subagents/kilo-subagent.ts
 var KiloSubagentFrontmatterSchema = import_mini71.z.looseObject({
   description: import_mini71.z.optional(import_mini71.z.string()),
-  // Kilo's documented default for user-defined agents is "all":
-  // available both as a top-level pick and as a subagent.
   mode: import_mini71.z._default(import_mini71.z.string(), "all"),
-  name: import_mini71.z.optional(import_mini71.z.string())
+  name: import_mini71.z.optional(import_mini71.z.string()),
+  displayName: import_mini71.z.optional(import_mini71.z.string()),
+  deprecated: import_mini71.z.optional(import_mini71.z.boolean()),
+  native: import_mini71.z.optional(import_mini71.z.boolean()),
+  hidden: import_mini71.z.optional(import_mini71.z.boolean()),
+  top_p: import_mini71.z.optional(import_mini71.z.number()),
+  temperature: import_mini71.z.optional(import_mini71.z.number()),
+  color: import_mini71.z.optional(import_mini71.z.string()),
+  permission: import_mini71.z.optional(import_mini71.z.string()),
+  model: import_mini71.z.optional(import_mini71.z.string()),
+  variant: import_mini71.z.optional(import_mini71.z.string()),
+  prompt: import_mini71.z.optional(import_mini71.z.string()),
+  options: import_mini71.z.optional(import_mini71.z.looseObject({})),
+  steps: import_mini71.z.optional(import_mini71.z.array(import_mini71.z.looseObject({}))),
+  disable: import_mini71.z.optional(import_mini71.z.boolean())
 });
 var KiloSubagent = class _KiloSubagent extends OpenCodeStyleSubagent {
+  constructor(params) {
+    super(params);
+    if (params.validate !== false) {
+      const result = this.validate();
+      if (!result.success) {
+        throw result.error;
+      }
+    }
+  }
   getToolTarget() {
     return "kilo";
   }
+  getFrontmatter() {
+    return this.frontmatter;
+  }
+  validate() {
+    const result = KiloSubagentFrontmatterSchema.safeParse(this.frontmatter);
+    if (result.success) {
+      this.frontmatter = result.data;
+      return { success: true, error: null };
+    }
+    return {
+      success: false,
+      error: new Error(
+        `Invalid frontmatter in ${(0, import_node_path121.join)(this.relativeDirPath, this.relativeFilePath)}: ${formatError(result.error)}`
+      )
+    };
+  }
   static getSettablePaths({
     global = false
   } = {}) {
@@ -19204,11 +19281,17 @@ var KiloSubagent = class _KiloSubagent extends OpenCodeStyleSubagent {
   }) {
     const rulesyncFrontmatter = rulesyncSubagent.getFrontmatter();
     const kiloSection = rulesyncFrontmatter.kilo ?? {};
-    const kiloFrontmatter = KiloSubagentFrontmatterSchema.parse({
+    const parseResult = KiloSubagentFrontmatterSchema.safeParse({
       ...kiloSection,
       description: rulesyncFrontmatter.description,
       ...rulesyncFrontmatter.name && { name: rulesyncFrontmatter.name }
     });
+    if (!parseResult.success) {
+      throw new Error(
+        `Invalid frontmatter in ${rulesyncSubagent.getRelativeFilePath()}: ${formatError(parseResult.error)}`
+      );
+    }
+    const kiloFrontmatter = parseResult.data;
     const body = rulesyncSubagent.getBody();
     const fileContent = stringifyFrontmatter(body, kiloFrontmatter);
     const paths = this.getSettablePaths({ global });
@@ -19519,7 +19602,8 @@ var OpenCodeSubagent = class _OpenCodeSubagent extends OpenCodeStyleSubagent {
   static forDeletion({
     outputRoot = process.cwd(),
     relativeDirPath,
-    relativeFilePath
+    relativeFilePath,
+    global = false
   }) {
     return new _OpenCodeSubagent({
       outputRoot,
@@ -19528,7 +19612,8 @@ var OpenCodeSubagent = class _OpenCodeSubagent extends OpenCodeStyleSubagent {
       frontmatter: { description: "", mode: "subagent" },
       body: "",
       fileContent: "",
-      validate: false
+      validate: false,
+      global
     });
   }
 };
@@ -21332,8 +21417,8 @@ var CopilotRuleFrontmatterSchema = import_mini78.z.object({
   applyTo: import_mini78.z.optional(import_mini78.z.string()),
   excludeAgent: import_mini78.z.optional(import_mini78.z.union([import_mini78.z.literal("code-review"), import_mini78.z.literal("coding-agent")]))
 });
-var normalizeRelativePath = (path5) => path5.replace(/\\/g, "/").replace(/\/+/g, "/");
-var sameRelativePath = (leftDir, leftFile, rightDir, rightFile) => normalizeRelativePath((0, import_node_path136.join)(leftDir, leftFile)) === normalizeRelativePath((0, import_node_path136.join)(rightDir, rightFile));
+var normalizeRelativePath = (p) => toPosixPath(p).replace(/\/+/g, "/");
+var sameRelativePath = (left, right) => normalizeRelativePath((0, import_node_path136.join)(left.dir, left.file)) === normalizeRelativePath((0, import_node_path136.join)(right.dir, right.file));
 var CopilotRule = class _CopilotRule extends ToolRule {
   frontmatter;
   body;
@@ -21452,10 +21537,8 @@ var CopilotRule = class _CopilotRule extends ToolRule {
   }) {
     const paths = this.getSettablePaths({ global });
     const isRoot = relativeDirPath ? sameRelativePath(
-      relativeDirPath,
-      relativeFilePath,
-      paths.root.relativeDirPath,
-      paths.root.relativeFilePath
+      { dir: relativeDirPath, file: relativeFilePath },
+      { dir: paths.root.relativeDirPath, file: paths.root.relativeFilePath }
     ) : relativeFilePath === paths.root.relativeFilePath;
     const resolvedRelativeDirPath = relativeDirPath ?? (isRoot ? paths.root.relativeDirPath : paths.nonRoot?.relativeDirPath ?? paths.root.relativeDirPath);
     if (isRoot) {
@@ -21501,10 +21584,8 @@ var CopilotRule = class _CopilotRule extends ToolRule {
   }) {
     const paths = this.getSettablePaths({ global });
     const isRoot = sameRelativePath(
-      relativeDirPath,
-      relativeFilePath,
-      paths.root.relativeDirPath,
-      paths.root.relativeFilePath
+      { dir: relativeDirPath, file: relativeFilePath },
+      { dir: paths.root.relativeDirPath, file: paths.root.relativeFilePath }
     );
     return new _CopilotRule({
       outputRoot,
@@ -31793,7 +31874,7 @@ function wrapCommand({
 }
 
 // src/cli/index.ts
-var getVersion = () => "8.18.0";
+var getVersion = () => "8.19.0";
 function wrapCommand2(name, errorCode, handler) {
   return wrapCommand({ name, errorCode, handler, getVersion });
 }

package/dist/cli/index.js

@@ -79,7 +79,7 @@ import {
   stringifyFrontmatter,
   toPosixPath,
   writeFileContent
-} from "../chunk-KDGHMXTY.js";
+} from "../chunk-66SPWRYP.js";
 
 // src/cli/index.ts
 import { Command } from "commander";
@@ -6485,7 +6485,7 @@ function wrapCommand({
 }
 
 // src/cli/index.ts
-var getVersion = () => "8.18.0";
+var getVersion = () => "8.19.0";
 function wrapCommand2(name, errorCode, handler) {
   return wrapCommand({ name, errorCode, handler, getVersion });
 }

package/dist/index.cjs

@@ -1064,11 +1064,18 @@ var smolToml = __toESM(require("smol-toml"), 1);
 // src/utils/frontmatter.ts
 var import_gray_matter = __toESM(require("gray-matter"), 1);
 var import_js_yaml = require("js-yaml");
+
+// src/utils/type-guards.ts
+function isRecord(value) {
+  return typeof value === "object" && value !== null && !Array.isArray(value);
+}
 function isPlainObject(value) {
-  if (value === null || typeof value !== "object") return false;
-  const prototype = Object.getPrototypeOf(value);
-  return prototype === Object.prototype || prototype === null;
+  if (!isRecord(value)) return false;
+  const proto = Object.getPrototypeOf(value);
+  return proto === null || proto === Object.prototype;
 }
+
+// src/utils/frontmatter.ts
 function deepRemoveNullishValue(value) {
   if (value === null || value === void 0) {
     return void 0;
@@ -1659,11 +1666,6 @@ var AgentsmdCommand = class _AgentsmdCommand extends SimulatedCommand {
 var import_node_path10 = require("path");
 var import_mini6 = require("zod/mini");
 
-// src/utils/type-guards.ts
-function isRecord(value) {
-  return typeof value === "object" && value !== null && !Array.isArray(value);
-}
-
 // src/features/commands/rulesync-command.ts
 var import_node_path9 = require("path");
 var import_mini5 = require("zod/mini");
@@ -7666,7 +7668,7 @@ var ToolMcp = class extends ToolFile {
     super({
       ...rest,
       validate: true
-      // Skip validation during construction
+      // ToolMcp runs subclass validation below when requested
     });
     if (rest.validate) {
       const result = this.validate();
@@ -7724,17 +7726,26 @@ var ClaudecodeMcp = class _ClaudecodeMcp extends ToolMcp {
     return this.json;
   }
   /**
-   * In global mode, ~/.claude/.claude.json should not be deleted
-   * as it may contain other user settings.
+   * In global mode, ~/.claude.json should not be deleted as it is the
+   * user's primary Claude Code config and contains many other settings
+   * managed by Claude Code itself (feature flags, project trust list,
+   * hooks, user settings, model selection, etc.).
    * In local mode, .mcp.json can be safely deleted.
    */
   isDeletable() {
     return !this.global;
   }
+  /**
+   * Legacy global path used by rulesync ≤ v8.17.0. The documented store
+   * is `~/.claude.json`; `fromFile` falls back here with a deprecation
+   * warning (mirrors PR #333). Never modified or removed by rulesync.
+   */
+  static LEGACY_GLOBAL_DIR = ".claude";
+  static LEGACY_GLOBAL_FILE = ".claude.json";
   static getSettablePaths({ global } = {}) {
     if (global) {
       return {
-        relativeDirPath: ".claude",
+        relativeDirPath: ".",
         relativeFilePath: ".claude.json"
       };
     }
@@ -7746,19 +7757,52 @@ var ClaudecodeMcp = class _ClaudecodeMcp extends ToolMcp {
   static async fromFile({
     outputRoot = process.cwd(),
     validate = true,
-    global = false
+    global = false,
+    logger
   }) {
     const paths = this.getSettablePaths({ global });
-    const fileContent = await readFileContentOrNull(
-      (0, import_node_path52.join)(outputRoot, paths.relativeDirPath, paths.relativeFilePath)
-    ) ?? '{"mcpServers":{}}';
-    const json = JSON.parse(fileContent);
-    const newJson = { ...json, mcpServers: json.mcpServers ?? {} };
+    const recommendedPath = (0, import_node_path52.join)(outputRoot, paths.relativeDirPath, paths.relativeFilePath);
+    if (await fileExists(recommendedPath)) {
+      const fileContent = await readFileContent(recommendedPath);
+      const json = JSON.parse(fileContent);
+      const newJson = { ...json, mcpServers: json.mcpServers ?? {} };
+      return new _ClaudecodeMcp({
+        outputRoot,
+        relativeDirPath: paths.relativeDirPath,
+        relativeFilePath: paths.relativeFilePath,
+        fileContent: JSON.stringify(newJson, null, 2),
+        validate,
+        global
+      });
+    }
+    if (global) {
+      const legacyPath = (0, import_node_path52.join)(
+        outputRoot,
+        _ClaudecodeMcp.LEGACY_GLOBAL_DIR,
+        _ClaudecodeMcp.LEGACY_GLOBAL_FILE
+      );
+      if (await fileExists(legacyPath)) {
+        logger?.warn(
+          `Warning: using deprecated path "${legacyPath}". Please migrate to "${recommendedPath}"`
+        );
+        const fileContent = await readFileContent(legacyPath);
+        const json = JSON.parse(fileContent);
+        const newJson = { ...json, mcpServers: json.mcpServers ?? {} };
+        return new _ClaudecodeMcp({
+          outputRoot,
+          relativeDirPath: _ClaudecodeMcp.LEGACY_GLOBAL_DIR,
+          relativeFilePath: _ClaudecodeMcp.LEGACY_GLOBAL_FILE,
+          fileContent: JSON.stringify(newJson, null, 2),
+          validate,
+          global
+        });
+      }
+    }
     return new _ClaudecodeMcp({
       outputRoot,
       relativeDirPath: paths.relativeDirPath,
       relativeFilePath: paths.relativeFilePath,
-      fileContent: JSON.stringify(newJson, null, 2),
+      fileContent: JSON.stringify({ mcpServers: {} }, null, 2),
       validate,
       global
     });
@@ -7787,7 +7831,7 @@ var ClaudecodeMcp = class _ClaudecodeMcp extends ToolMcp {
   }
   toRulesyncMcp() {
     return this.toRulesyncMcpDefault({
-      fileContent: JSON.stringify({ mcpServers: this.json.mcpServers }, null, 2)
+      fileContent: JSON.stringify({ mcpServers: this.json.mcpServers ?? {} }, null, 2)
     });
   }
   validate() {
@@ -7891,11 +7935,6 @@ var import_node_path54 = require("path");
 var smolToml3 = __toESM(require("smol-toml"), 1);
 var MAX_REMOVE_EMPTY_ENTRIES_DEPTH = 32;
 var PROTOTYPE_POLLUTION_KEYS = /* @__PURE__ */ new Set(["__proto__", "constructor", "prototype"]);
-function isPlainObject2(value) {
-  if (!isRecord(value)) return false;
-  const proto = Object.getPrototypeOf(value);
-  return proto === null || proto === Object.prototype;
-}
 function convertFromCodexFormat(codexMcp) {
   const result = {};
   for (const [name, config] of Object.entries(codexMcp)) {
@@ -8051,7 +8090,7 @@ var CodexcliMcp = class _CodexcliMcp extends ToolMcp {
     for (const [key, value] of Object.entries(obj)) {
       if (PROTOTYPE_POLLUTION_KEYS.has(key)) continue;
       if (value === null) continue;
-      if (isPlainObject2(value)) {
+      if (isPlainObject(value)) {
         const cleaned = this.removeEmptyEntries(value, depth + 1);
         if (Object.keys(cleaned).length === 0) continue;
         filtered[key] = cleaned;
@@ -9897,7 +9936,8 @@ var McpProcessor = class extends FeatureProcessor {
         await factory.class.fromFile({
           outputRoot: this.outputRoot,
           validate: true,
-          global: this.global
+          global: this.global,
+          logger: this.logger
         })
       ];
       this.logger.debug(`Successfully loaded ${toolMcps.length} ${this.toolTarget} MCP files`);
@@ -19053,15 +19093,52 @@ var OpenCodeStyleSubagent = class extends ToolSubagent {
 // src/features/subagents/kilo-subagent.ts
 var KiloSubagentFrontmatterSchema = import_mini71.z.looseObject({
   description: import_mini71.z.optional(import_mini71.z.string()),
-  // Kilo's documented default for user-defined agents is "all":
-  // available both as a top-level pick and as a subagent.
   mode: import_mini71.z._default(import_mini71.z.string(), "all"),
-  name: import_mini71.z.optional(import_mini71.z.string())
+  name: import_mini71.z.optional(import_mini71.z.string()),
+  displayName: import_mini71.z.optional(import_mini71.z.string()),
+  deprecated: import_mini71.z.optional(import_mini71.z.boolean()),
+  native: import_mini71.z.optional(import_mini71.z.boolean()),
+  hidden: import_mini71.z.optional(import_mini71.z.boolean()),
+  top_p: import_mini71.z.optional(import_mini71.z.number()),
+  temperature: import_mini71.z.optional(import_mini71.z.number()),
+  color: import_mini71.z.optional(import_mini71.z.string()),
+  permission: import_mini71.z.optional(import_mini71.z.string()),
+  model: import_mini71.z.optional(import_mini71.z.string()),
+  variant: import_mini71.z.optional(import_mini71.z.string()),
+  prompt: import_mini71.z.optional(import_mini71.z.string()),
+  options: import_mini71.z.optional(import_mini71.z.looseObject({})),
+  steps: import_mini71.z.optional(import_mini71.z.array(import_mini71.z.looseObject({}))),
+  disable: import_mini71.z.optional(import_mini71.z.boolean())
 });
 var KiloSubagent = class _KiloSubagent extends OpenCodeStyleSubagent {
+  constructor(params) {
+    super(params);
+    if (params.validate !== false) {
+      const result = this.validate();
+      if (!result.success) {
+        throw result.error;
+      }
+    }
+  }
   getToolTarget() {
     return "kilo";
   }
+  getFrontmatter() {
+    return this.frontmatter;
+  }
+  validate() {
+    const result = KiloSubagentFrontmatterSchema.safeParse(this.frontmatter);
+    if (result.success) {
+      this.frontmatter = result.data;
+      return { success: true, error: null };
+    }
+    return {
+      success: false,
+      error: new Error(
+        `Invalid frontmatter in ${(0, import_node_path121.join)(this.relativeDirPath, this.relativeFilePath)}: ${formatError(result.error)}`
+      )
+    };
+  }
   static getSettablePaths({
     global = false
   } = {}) {
@@ -19077,11 +19154,17 @@ var KiloSubagent = class _KiloSubagent extends OpenCodeStyleSubagent {
   }) {
     const rulesyncFrontmatter = rulesyncSubagent.getFrontmatter();
     const kiloSection = rulesyncFrontmatter.kilo ?? {};
-    const kiloFrontmatter = KiloSubagentFrontmatterSchema.parse({
+    const parseResult = KiloSubagentFrontmatterSchema.safeParse({
       ...kiloSection,
       description: rulesyncFrontmatter.description,
       ...rulesyncFrontmatter.name && { name: rulesyncFrontmatter.name }
     });
+    if (!parseResult.success) {
+      throw new Error(
+        `Invalid frontmatter in ${rulesyncSubagent.getRelativeFilePath()}: ${formatError(parseResult.error)}`
+      );
+    }
+    const kiloFrontmatter = parseResult.data;
     const body = rulesyncSubagent.getBody();
     const fileContent = stringifyFrontmatter(body, kiloFrontmatter);
     const paths = this.getSettablePaths({ global });
@@ -19392,7 +19475,8 @@ var OpenCodeSubagent = class _OpenCodeSubagent extends OpenCodeStyleSubagent {
   static forDeletion({
     outputRoot = process.cwd(),
     relativeDirPath,
-    relativeFilePath
+    relativeFilePath,
+    global = false
   }) {
     return new _OpenCodeSubagent({
       outputRoot,
@@ -19401,7 +19485,8 @@ var OpenCodeSubagent = class _OpenCodeSubagent extends OpenCodeStyleSubagent {
       frontmatter: { description: "", mode: "subagent" },
       body: "",
       fileContent: "",
-      validate: false
+      validate: false,
+      global
     });
   }
 };
@@ -21205,8 +21290,8 @@ var CopilotRuleFrontmatterSchema = import_mini78.z.object({
   applyTo: import_mini78.z.optional(import_mini78.z.string()),
   excludeAgent: import_mini78.z.optional(import_mini78.z.union([import_mini78.z.literal("code-review"), import_mini78.z.literal("coding-agent")]))
 });
-var normalizeRelativePath = (path4) => path4.replace(/\\/g, "/").replace(/\/+/g, "/");
-var sameRelativePath = (leftDir, leftFile, rightDir, rightFile) => normalizeRelativePath((0, import_node_path136.join)(leftDir, leftFile)) === normalizeRelativePath((0, import_node_path136.join)(rightDir, rightFile));
+var normalizeRelativePath = (p) => toPosixPath(p).replace(/\/+/g, "/");
+var sameRelativePath = (left, right) => normalizeRelativePath((0, import_node_path136.join)(left.dir, left.file)) === normalizeRelativePath((0, import_node_path136.join)(right.dir, right.file));
 var CopilotRule = class _CopilotRule extends ToolRule {
   frontmatter;
   body;
@@ -21325,10 +21410,8 @@ var CopilotRule = class _CopilotRule extends ToolRule {
   }) {
     const paths = this.getSettablePaths({ global });
     const isRoot = relativeDirPath ? sameRelativePath(
-      relativeDirPath,
-      relativeFilePath,
-      paths.root.relativeDirPath,
-      paths.root.relativeFilePath
+      { dir: relativeDirPath, file: relativeFilePath },
+      { dir: paths.root.relativeDirPath, file: paths.root.relativeFilePath }
     ) : relativeFilePath === paths.root.relativeFilePath;
     const resolvedRelativeDirPath = relativeDirPath ?? (isRoot ? paths.root.relativeDirPath : paths.nonRoot?.relativeDirPath ?? paths.root.relativeDirPath);
     if (isRoot) {
@@ -21374,10 +21457,8 @@ var CopilotRule = class _CopilotRule extends ToolRule {
   }) {
     const paths = this.getSettablePaths({ global });
     const isRoot = sameRelativePath(
-      relativeDirPath,
-      relativeFilePath,
-      paths.root.relativeDirPath,
-      paths.root.relativeFilePath
+      { dir: relativeDirPath, file: relativeFilePath },
+      { dir: paths.root.relativeDirPath, file: paths.root.relativeFilePath }
     );
     return new _CopilotRule({
       outputRoot,

package/dist/index.js

@@ -7,7 +7,7 @@ import {
   convertFromTool,
   generate,
   importFromTool
-} from "./chunk-KDGHMXTY.js";
+} from "./chunk-66SPWRYP.js";
 
 // src/index.ts
 async function generate2(options = {}) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "rulesync",
-  "version": "8.18.0",
+  "version": "8.19.0",
   "description": "Unified AI rules management CLI tool that generates configuration files for various AI development tools",
   "keywords": [
     "ai",