rulesync 3.33.0 → 3.34.0

package/dist/index.js

@@ -90,6 +90,7 @@ import { globSync } from "fs";
 import { mkdir, readdir, readFile, rm, stat, writeFile } from "fs/promises";
 import os from "os";
 import { basename, dirname, join, relative, resolve } from "path";
+import { kebabCase } from "es-toolkit";
 async function ensureDir(dirPath) {
   try {
     await stat(dirPath);
@@ -217,6 +218,13 @@ function validateBaseDir(baseDir) {
   }
   checkPathTraversal({ relativePath: baseDir, intendedRootDir: process.cwd() });
 }
+function toKebabCaseFilename(filename) {
+  const lastDotIndex = filename.lastIndexOf(".");
+  const extension = lastDotIndex > 0 ? filename.slice(lastDotIndex) : "";
+  const nameWithoutExt = lastDotIndex > 0 ? filename.slice(0, lastDotIndex) : filename;
+  const kebabName = kebabCase(nameWithoutExt);
+  return kebabName + extension;
+}
 
 // src/config/config.ts
 import { optional, z as z3 } from "zod/mini";
@@ -2374,11 +2382,19 @@ var ClaudecodeIgnore = class _ClaudecodeIgnore extends ToolIgnore {
     const exists = await fileExists(filePath);
     const existingFileContent = exists ? await readFileContent(filePath) : "{}";
     const existingJsonValue = JSON.parse(existingFileContent);
+    const existingDenies = existingJsonValue.permissions?.deny ?? [];
+    const preservedDenies = existingDenies.filter((deny) => {
+      const isReadPattern = deny.startsWith("Read(") && deny.endsWith(")");
+      if (isReadPattern) {
+        return deniedValues.includes(deny);
+      }
+      return true;
+    });
     const jsonValue = {
       ...existingJsonValue,
       permissions: {
         ...existingJsonValue.permissions,
-        deny: uniq([...existingJsonValue.permissions?.deny ?? [], ...deniedValues].toSorted())
+        deny: uniq([...preservedDenies, ...deniedValues].toSorted())
       }
     };
     return new _ClaudecodeIgnore({
@@ -4284,7 +4300,7 @@ var McpProcessor = class extends FeatureProcessor {
 // src/features/rules/rules-processor.ts
 import { basename as basename21, join as join82 } from "path";
 import { encode } from "@toon-format/toon";
-import { z as z33 } from "zod/mini";
+import { z as z34 } from "zod/mini";
 
 // src/constants/general.ts
 var SKILL_FILE_NAME = "SKILL.md";
@@ -6150,6 +6166,12 @@ var RulesyncRuleFrontmatterSchema = z28.object({
     z28.object({
       excludeAgent: z28.optional(z28.union([z28.literal("code-review"), z28.literal("coding-agent")]))
     })
+  ),
+  antigravity: z28.optional(
+    z28.looseObject({
+      trigger: z28.optional(z28.string()),
+      globs: z28.optional(z28.array(z28.string()))
+    })
   )
 });
 var RulesyncRule = class _RulesyncRule extends RulesyncFile {
@@ -6519,7 +6541,176 @@ var AmazonQCliRule = class _AmazonQCliRule extends ToolRule {
 
 // src/features/rules/antigravity-rule.ts
 import { join as join65 } from "path";
+import { z as z29 } from "zod/mini";
+var AntigravityRuleFrontmatterSchema = z29.looseObject({
+  trigger: z29.optional(
+    z29.union([
+      z29.literal("always_on"),
+      z29.literal("glob"),
+      z29.literal("manual"),
+      z29.literal("model_decision"),
+      z29.string()
+      // accepts any string for forward compatibility
+    ])
+  ),
+  globs: z29.optional(z29.string()),
+  description: z29.optional(z29.string())
+});
+function parseGlobsString(globs) {
+  if (!globs) {
+    return [];
+  }
+  if (Array.isArray(globs)) {
+    return globs;
+  }
+  if (globs.trim() === "") {
+    return [];
+  }
+  return globs.split(",").map((g) => g.trim());
+}
+function stringifyGlobs(globs) {
+  if (!globs || globs.length === 0) {
+    return void 0;
+  }
+  return globs.join(",");
+}
+function normalizeStoredAntigravity(stored) {
+  if (!stored) {
+    return void 0;
+  }
+  const { globs, ...rest } = stored;
+  return {
+    ...rest,
+    globs: Array.isArray(globs) ? stringifyGlobs(globs) : globs
+  };
+}
+var globStrategy = {
+  canHandle: (trigger) => trigger === "glob",
+  generateFrontmatter: (normalized, rulesyncFrontmatter) => {
+    const effectiveGlobsArray = normalized?.globs ? parseGlobsString(normalized.globs) : rulesyncFrontmatter.globs ?? [];
+    return {
+      ...normalized,
+      trigger: "glob",
+      globs: stringifyGlobs(effectiveGlobsArray)
+    };
+  },
+  exportRulesyncData: ({ description, ...frontmatter }) => ({
+    globs: parseGlobsString(frontmatter.globs),
+    description: description || "",
+    antigravity: frontmatter
+  })
+};
+var manualStrategy = {
+  canHandle: (trigger) => trigger === "manual",
+  generateFrontmatter: (normalized) => ({
+    ...normalized,
+    trigger: "manual"
+  }),
+  exportRulesyncData: ({ description, ...frontmatter }) => ({
+    globs: [],
+    description: description || "",
+    antigravity: frontmatter
+  })
+};
+var alwaysOnStrategy = {
+  canHandle: (trigger) => trigger === "always_on",
+  generateFrontmatter: (normalized) => ({
+    ...normalized,
+    trigger: "always_on"
+  }),
+  exportRulesyncData: ({ description, ...frontmatter }) => ({
+    globs: ["**/*"],
+    description: description || "",
+    antigravity: frontmatter
+  })
+};
+var modelDecisionStrategy = {
+  canHandle: (trigger) => trigger === "model_decision",
+  generateFrontmatter: (normalized, rulesyncFrontmatter) => ({
+    ...normalized,
+    trigger: "model_decision",
+    description: rulesyncFrontmatter.description
+  }),
+  exportRulesyncData: ({ description, ...frontmatter }) => ({
+    globs: [],
+    description: description || "",
+    antigravity: frontmatter
+  })
+};
+var unknownStrategy = {
+  canHandle: (trigger) => trigger !== void 0,
+  generateFrontmatter: (normalized) => {
+    const trigger = typeof normalized?.trigger === "string" ? normalized.trigger : "manual";
+    return {
+      ...normalized,
+      trigger
+    };
+  },
+  exportRulesyncData: ({ description, ...frontmatter }) => ({
+    globs: frontmatter.globs ? parseGlobsString(frontmatter.globs) : ["**/*"],
+    description: description || "",
+    antigravity: frontmatter
+  })
+};
+var inferenceStrategy = {
+  canHandle: (trigger) => trigger === void 0,
+  generateFrontmatter: (normalized, rulesyncFrontmatter) => {
+    const effectiveGlobsArray = normalized?.globs ? parseGlobsString(normalized.globs) : rulesyncFrontmatter.globs ?? [];
+    if (effectiveGlobsArray.length > 0 && !effectiveGlobsArray.includes("**/*") && !effectiveGlobsArray.includes("*")) {
+      return {
+        ...normalized,
+        trigger: "glob",
+        globs: stringifyGlobs(effectiveGlobsArray)
+      };
+    }
+    return {
+      ...normalized,
+      trigger: "always_on"
+    };
+  },
+  exportRulesyncData: ({ description, ...frontmatter }) => ({
+    globs: frontmatter.globs ? parseGlobsString(frontmatter.globs) : ["**/*"],
+    description: description || "",
+    antigravity: frontmatter
+  })
+};
+var STRATEGIES = [
+  globStrategy,
+  manualStrategy,
+  alwaysOnStrategy,
+  modelDecisionStrategy,
+  unknownStrategy,
+  inferenceStrategy
+];
 var AntigravityRule = class _AntigravityRule extends ToolRule {
+  frontmatter;
+  body;
+  /**
+   * Creates an AntigravityRule instance.
+   *
+   * @param params - Rule parameters including frontmatter and body
+   * @param params.frontmatter - Antigravity-specific frontmatter configuration
+   * @param params.body - The markdown body content (without frontmatter)
+   *
+   * Note: Files without frontmatter will default to always_on trigger during fromFile().
+   */
+  constructor({ frontmatter, body, ...rest }) {
+    if (rest.validate !== false) {
+      const result = AntigravityRuleFrontmatterSchema.safeParse(frontmatter);
+      if (!result.success) {
+        throw new Error(
+          `Invalid frontmatter in ${join65(rest.relativeDirPath, rest.relativeFilePath)}: ${formatError(result.error)}`
+        );
+      }
+    }
+    super({
+      ...rest,
+      // Ensure fileContent includes frontmatter when constructed directly
+      fileContent: stringifyFrontmatter(body, frontmatter)
+    });
+    this.frontmatter = frontmatter;
+    this.body = body;
+  }
   static getSettablePaths() {
     return {
       nonRoot: {
@@ -6532,36 +6723,121 @@ var AntigravityRule = class _AntigravityRule extends ToolRule {
     relativeFilePath,
     validate = true
   }) {
-    const fileContent = await readFileContent(
-      join65(baseDir, this.getSettablePaths().nonRoot.relativeDirPath, relativeFilePath)
+    const filePath = join65(
+      baseDir,
+      this.getSettablePaths().nonRoot.relativeDirPath,
+      relativeFilePath
     );
+    const fileContent = await readFileContent(filePath);
+    const { frontmatter, body } = parseFrontmatter(fileContent);
+    let parsedFrontmatter;
+    if (validate) {
+      const result = AntigravityRuleFrontmatterSchema.safeParse(frontmatter);
+      if (result.success) {
+        parsedFrontmatter = result.data;
+      } else {
+        throw new Error(`Invalid frontmatter in ${filePath}: ${formatError(result.error)}`);
+      }
+    } else {
+      parsedFrontmatter = frontmatter;
+    }
     return new _AntigravityRule({
       baseDir,
       relativeDirPath: this.getSettablePaths().nonRoot.relativeDirPath,
       relativeFilePath,
-      fileContent,
+      body,
+      frontmatter: parsedFrontmatter,
       validate,
       root: false
     });
   }
+  /**
+   * Converts a RulesyncRule to an AntigravityRule.
+   *
+   * Trigger inference:
+   * - If antigravity.trigger is set, it's preserved
+   * - If specific globs are set, infers "glob" trigger
+   * - Otherwise, infers "always_on" trigger
+   */
   static fromRulesyncRule({
     baseDir = process.cwd(),
     rulesyncRule,
     validate = true
   }) {
-    return new _AntigravityRule(
-      this.buildToolRuleParamsDefault({
-        baseDir,
-        rulesyncRule,
-        validate,
-        nonRootPath: this.getSettablePaths().nonRoot
-      })
-    );
+    const rulesyncFrontmatter = rulesyncRule.getFrontmatter();
+    const storedAntigravity = rulesyncFrontmatter.antigravity;
+    const normalized = normalizeStoredAntigravity(storedAntigravity);
+    const storedTrigger = storedAntigravity?.trigger;
+    const strategy = STRATEGIES.find((s) => s.canHandle(storedTrigger));
+    if (!strategy) {
+      throw new Error(`No strategy found for trigger: ${storedTrigger}`);
+    }
+    const frontmatter = strategy.generateFrontmatter(normalized, rulesyncFrontmatter);
+    const paths = this.getSettablePaths();
+    const kebabCaseFilename = toKebabCaseFilename(rulesyncRule.getRelativeFilePath());
+    return new _AntigravityRule({
+      baseDir,
+      relativeDirPath: paths.nonRoot.relativeDirPath,
+      relativeFilePath: kebabCaseFilename,
+      frontmatter,
+      body: rulesyncRule.getBody(),
+      validate,
+      root: false
+    });
   }
+  /**
+   * Converts this AntigravityRule to a RulesyncRule.
+   *
+   * The Antigravity configuration is preserved in the RulesyncRule's
+   * frontmatter.antigravity field for round-trip compatibility.
+   *
+   * Note: All Antigravity rules are treated as non-root (root: false),
+   * as they are all placed in the .agent/rules directory.
+   *
+   * @returns RulesyncRule instance with Antigravity config preserved
+   */
   toRulesyncRule() {
-    return this.toRulesyncRuleDefault();
+    const strategy = STRATEGIES.find((s) => s.canHandle(this.frontmatter.trigger));
+    let rulesyncData = {
+      globs: [],
+      description: "",
+      antigravity: this.frontmatter
+    };
+    if (strategy) {
+      rulesyncData = strategy.exportRulesyncData(this.frontmatter);
+    }
+    const antigravityForRulesync = {
+      ...rulesyncData.antigravity,
+      globs: this.frontmatter.globs ? parseGlobsString(this.frontmatter.globs) : void 0
+    };
+    return new RulesyncRule({
+      baseDir: process.cwd(),
+      relativeDirPath: RulesyncRule.getSettablePaths().recommended.relativeDirPath,
+      relativeFilePath: this.getRelativeFilePath(),
+      frontmatter: {
+        root: false,
+        targets: ["*"],
+        ...rulesyncData,
+        antigravity: antigravityForRulesync
+      },
+      // When converting back, we only want the body content
+      body: this.body
+    });
+  }
+  getBody() {
+    return this.body;
+  }
+  // Helper to access raw file content including frontmatter is `this.fileContent` (from ToolFile)
+  // But we might want `body` only for some operations?
+  // ToolFile.getFileContent() returns the whole string.
+  getFrontmatter() {
+    return this.frontmatter;
   }
   validate() {
+    const result = AntigravityRuleFrontmatterSchema.safeParse(this.frontmatter);
+    if (!result.success) {
+      return { success: false, error: new Error(formatError(result.error)) };
+    }
     return { success: true, error: null };
   }
   static isTargetedByRulesyncRule(rulesyncRule) {
@@ -6795,9 +7071,9 @@ var ClaudecodeLegacyRule = class _ClaudecodeLegacyRule extends ToolRule {
 
 // src/features/rules/claudecode-rule.ts
 import { join as join69 } from "path";
-import { z as z29 } from "zod/mini";
-var ClaudecodeRuleFrontmatterSchema = z29.object({
-  paths: z29.optional(z29.string())
+import { z as z30 } from "zod/mini";
+var ClaudecodeRuleFrontmatterSchema = z30.object({
+  paths: z30.optional(z30.string())
 });
 var ClaudecodeRule = class _ClaudecodeRule extends ToolRule {
   frontmatter;
@@ -6987,9 +7263,9 @@ var ClaudecodeRule = class _ClaudecodeRule extends ToolRule {
 
 // src/features/rules/cline-rule.ts
 import { join as join70 } from "path";
-import { z as z30 } from "zod/mini";
-var ClineRuleFrontmatterSchema = z30.object({
-  description: z30.string()
+import { z as z31 } from "zod/mini";
+var ClineRuleFrontmatterSchema = z31.object({
+  description: z31.string()
 });
 var ClineRule = class _ClineRule extends ToolRule {
   static getSettablePaths() {
@@ -7136,11 +7412,11 @@ var CodexcliRule = class _CodexcliRule extends ToolRule {
 
 // src/features/rules/copilot-rule.ts
 import { join as join72 } from "path";
-import { z as z31 } from "zod/mini";
-var CopilotRuleFrontmatterSchema = z31.object({
-  description: z31.optional(z31.string()),
-  applyTo: z31.optional(z31.string()),
-  excludeAgent: z31.optional(z31.union([z31.literal("code-review"), z31.literal("coding-agent")]))
+import { z as z32 } from "zod/mini";
+var CopilotRuleFrontmatterSchema = z32.object({
+  description: z32.optional(z32.string()),
+  applyTo: z32.optional(z32.string()),
+  excludeAgent: z32.optional(z32.union([z32.literal("code-review"), z32.literal("coding-agent")]))
 });
 var CopilotRule = class _CopilotRule extends ToolRule {
   frontmatter;
@@ -7308,11 +7584,11 @@ var CopilotRule = class _CopilotRule extends ToolRule {
 
 // src/features/rules/cursor-rule.ts
 import { basename as basename20, join as join73 } from "path";
-import { z as z32 } from "zod/mini";
-var CursorRuleFrontmatterSchema = z32.object({
-  description: z32.optional(z32.string()),
-  globs: z32.optional(z32.string()),
-  alwaysApply: z32.optional(z32.boolean())
+import { z as z33 } from "zod/mini";
+var CursorRuleFrontmatterSchema = z33.object({
+  description: z33.optional(z33.string()),
+  globs: z33.optional(z33.string()),
+  alwaysApply: z33.optional(z33.boolean())
 });
 var CursorRule = class _CursorRule extends ToolRule {
   frontmatter;
@@ -8030,7 +8306,7 @@ var rulesProcessorToolTargets = [
   "warp",
   "windsurf"
 ];
-var RulesProcessorToolTargetSchema = z33.enum(rulesProcessorToolTargets);
+var RulesProcessorToolTargetSchema = z34.enum(rulesProcessorToolTargets);
 var toolRuleFactories = /* @__PURE__ */ new Map([
   [
     "agentsmd",
@@ -9379,7 +9655,7 @@ import { FastMCP } from "fastmcp";
 
 // src/mcp/commands.ts
 import { basename as basename22, join as join85 } from "path";
-import { z as z34 } from "zod/mini";
+import { z as z35 } from "zod/mini";
 var maxCommandSizeBytes = 1024 * 1024;
 var maxCommandsCount = 1e3;
 async function listCommands() {
@@ -9498,17 +9774,17 @@ async function deleteCommand({ relativePathFromCwd }) {
   }
 }
 var commandToolSchemas = {
-  listCommands: z34.object({}),
-  getCommand: z34.object({
-    relativePathFromCwd: z34.string()
+  listCommands: z35.object({}),
+  getCommand: z35.object({
+    relativePathFromCwd: z35.string()
   }),
-  putCommand: z34.object({
-    relativePathFromCwd: z34.string(),
+  putCommand: z35.object({
+    relativePathFromCwd: z35.string(),
     frontmatter: RulesyncCommandFrontmatterSchema,
-    body: z34.string()
+    body: z35.string()
   }),
-  deleteCommand: z34.object({
-    relativePathFromCwd: z34.string()
+  deleteCommand: z35.object({
+    relativePathFromCwd: z35.string()
   })
 };
 var commandTools = {
@@ -9557,7 +9833,7 @@ var commandTools = {
 
 // src/mcp/ignore.ts
 import { join as join86 } from "path";
-import { z as z35 } from "zod/mini";
+import { z as z36 } from "zod/mini";
 var maxIgnoreFileSizeBytes = 100 * 1024;
 async function getIgnoreFile() {
   const ignoreFilePath = join86(process.cwd(), RULESYNC_AIIGNORE_RELATIVE_FILE_PATH);
@@ -9614,11 +9890,11 @@ async function deleteIgnoreFile() {
   }
 }
 var ignoreToolSchemas = {
-  getIgnoreFile: z35.object({}),
-  putIgnoreFile: z35.object({
-    content: z35.string()
+  getIgnoreFile: z36.object({}),
+  putIgnoreFile: z36.object({
+    content: z36.string()
   }),
-  deleteIgnoreFile: z35.object({})
+  deleteIgnoreFile: z36.object({})
 };
 var ignoreTools = {
   getIgnoreFile: {
@@ -9652,7 +9928,7 @@ var ignoreTools = {
 
 // src/mcp/mcp.ts
 import { join as join87 } from "path";
-import { z as z36 } from "zod/mini";
+import { z as z37 } from "zod/mini";
 var maxMcpSizeBytes = 1024 * 1024;
 async function getMcpFile() {
   const config = await ConfigResolver.resolve({});
@@ -9742,11 +10018,11 @@ async function deleteMcpFile() {
   }
 }
 var mcpToolSchemas = {
-  getMcpFile: z36.object({}),
-  putMcpFile: z36.object({
-    content: z36.string()
+  getMcpFile: z37.object({}),
+  putMcpFile: z37.object({
+    content: z37.string()
   }),
-  deleteMcpFile: z36.object({})
+  deleteMcpFile: z37.object({})
 };
 var mcpTools = {
   getMcpFile: {
@@ -9780,7 +10056,7 @@ var mcpTools = {
 
 // src/mcp/rules.ts
 import { basename as basename23, join as join88 } from "path";
-import { z as z37 } from "zod/mini";
+import { z as z38 } from "zod/mini";
 var maxRuleSizeBytes = 1024 * 1024;
 var maxRulesCount = 1e3;
 async function listRules() {
@@ -9899,17 +10175,17 @@ async function deleteRule({ relativePathFromCwd }) {
   }
 }
 var ruleToolSchemas = {
-  listRules: z37.object({}),
-  getRule: z37.object({
-    relativePathFromCwd: z37.string()
+  listRules: z38.object({}),
+  getRule: z38.object({
+    relativePathFromCwd: z38.string()
   }),
-  putRule: z37.object({
-    relativePathFromCwd: z37.string(),
+  putRule: z38.object({
+    relativePathFromCwd: z38.string(),
     frontmatter: RulesyncRuleFrontmatterSchema,
-    body: z37.string()
+    body: z38.string()
   }),
-  deleteRule: z37.object({
-    relativePathFromCwd: z37.string()
+  deleteRule: z38.object({
+    relativePathFromCwd: z38.string()
   })
 };
 var ruleTools = {
@@ -9958,7 +10234,7 @@ var ruleTools = {
 
 // src/mcp/skills.ts
 import { basename as basename24, dirname as dirname2, join as join89 } from "path";
-import { z as z38 } from "zod/mini";
+import { z as z39 } from "zod/mini";
 var maxSkillSizeBytes = 1024 * 1024;
 var maxSkillsCount = 1e3;
 function aiDirFileToMcpSkillFile(file) {
@@ -10126,23 +10402,23 @@ async function deleteSkill({
     );
   }
 }
-var McpSkillFileSchema = z38.object({
-  name: z38.string(),
-  body: z38.string()
+var McpSkillFileSchema = z39.object({
+  name: z39.string(),
+  body: z39.string()
 });
 var skillToolSchemas = {
-  listSkills: z38.object({}),
-  getSkill: z38.object({
-    relativeDirPathFromCwd: z38.string()
+  listSkills: z39.object({}),
+  getSkill: z39.object({
+    relativeDirPathFromCwd: z39.string()
   }),
-  putSkill: z38.object({
-    relativeDirPathFromCwd: z38.string(),
+  putSkill: z39.object({
+    relativeDirPathFromCwd: z39.string(),
     frontmatter: RulesyncSkillFrontmatterSchema,
-    body: z38.string(),
-    otherFiles: z38.optional(z38.array(McpSkillFileSchema))
+    body: z39.string(),
+    otherFiles: z39.optional(z39.array(McpSkillFileSchema))
   }),
-  deleteSkill: z38.object({
-    relativeDirPathFromCwd: z38.string()
+  deleteSkill: z39.object({
+    relativeDirPathFromCwd: z39.string()
   })
 };
 var skillTools = {
@@ -10192,7 +10468,7 @@ var skillTools = {
 
 // src/mcp/subagents.ts
 import { basename as basename25, join as join90 } from "path";
-import { z as z39 } from "zod/mini";
+import { z as z40 } from "zod/mini";
 var maxSubagentSizeBytes = 1024 * 1024;
 var maxSubagentsCount = 1e3;
 async function listSubagents() {
@@ -10316,17 +10592,17 @@ async function deleteSubagent({ relativePathFromCwd }) {
   }
 }
 var subagentToolSchemas = {
-  listSubagents: z39.object({}),
-  getSubagent: z39.object({
-    relativePathFromCwd: z39.string()
+  listSubagents: z40.object({}),
+  getSubagent: z40.object({
+    relativePathFromCwd: z40.string()
   }),
-  putSubagent: z39.object({
-    relativePathFromCwd: z39.string(),
+  putSubagent: z40.object({
+    relativePathFromCwd: z40.string(),
     frontmatter: RulesyncSubagentFrontmatterSchema,
-    body: z39.string()
+    body: z40.string()
   }),
-  deleteSubagent: z39.object({
-    relativePathFromCwd: z39.string()
+  deleteSubagent: z40.object({
+    relativePathFromCwd: z40.string()
   })
 };
 var subagentTools = {
@@ -10410,7 +10686,7 @@ async function mcpCommand({ version }) {
 }
 
 // src/cli/index.ts
-var getVersion = () => "3.33.0";
+var getVersion = () => "3.34.0";
 var main = async () => {
   const program = new Command();
   const version = getVersion();