rulesync 7.24.0 → 7.26.0

package/README.md

@@ -71,7 +71,7 @@ See [Quick Start guide](https://dyoshikawa.github.io/rulesync/getting-started/qu
 | Gemini CLI          | geminicli    | ✅ 🌏 |   ✅   |  ✅ 🌏   |  ✅ 🌏   |    🎮     | ✅ 🌏  | ✅ 🌏 |
 | Goose               | goose        | ✅ 🌏 |   ✅   |          |          |           |        |       |
 | GitHub Copilot      | copilot      | ✅ 🌏 |        |    ✅    |    ✅    |    ✅     |   ✅   |  ✅   |
-| GitHub Copilot CLI  | copilotcli   |       |        |  ✅ 🌏   |          |           |        |       |
+| GitHub Copilot CLI  | copilotcli   | ✅ 🌏 |        |  ✅ 🌏   |          |           |        |       |
 | Cursor              | cursor       |  ✅   |   ✅   |  ✅ 🌏   |  ✅ 🌏   |   ✅ 🌏   | ✅ 🌏  |  ✅   |
 | deepagents-cli      | deepagents   |  ✅   |        |  ✅ 🌏   |          |    ✅     |   ✅   |  🌏   |
 | Factory Droid       | factorydroid | ✅ 🌏 |        |  ✅ 🌏   |    🎮    |    🎮     |   🎮   | ✅ 🌏 |

package/dist/{chunk-A4FZT3JT.js → chunk-MMC5VK27.js}

@@ -137,6 +137,9 @@ async function readOrInitializeFileContent(filePath, initialContent = "") {
     return initialContent;
   }
 }
+function toPosixPath(p) {
+  return p.replace(/\\/g, "/");
+}
 function checkPathTraversal({
   relativePath,
   intendedRootDir
@@ -944,8 +947,11 @@ var AiFile = class {
   getFileContent() {
     return this.fileContent;
   }
+  /**
+   * Returns the relative path from CWD with POSIX separators for consistent cross-platform output.
+   */
   getRelativePathFromCwd() {
-    return path.join(this.relativeDirPath, this.relativeFilePath).replace(/\\/g, "/");
+    return toPosixPath(path.join(this.relativeDirPath, this.relativeFilePath));
   }
   setFileContent(newFileContent) {
     this.fileContent = newFileContent;
@@ -6122,7 +6128,7 @@ import { z as z21 } from "zod/mini";
 // src/types/mcp.ts
 import { z as z20 } from "zod/mini";
 var McpServerSchema = z20.looseObject({
-  type: z20.optional(z20.enum(["stdio", "sse", "http"])),
+  type: z20.optional(z20.enum(["local", "stdio", "sse", "http"])),
   command: z20.optional(z20.union([z20.string(), z20.array(z20.string())])),
   args: z20.optional(z20.array(z20.string())),
   url: z20.optional(z20.string()),
@@ -6133,7 +6139,7 @@ var McpServerSchema = z20.looseObject({
   timeout: z20.optional(z20.number()),
   trust: z20.optional(z20.boolean()),
   cwd: z20.optional(z20.string()),
-  transport: z20.optional(z20.enum(["stdio", "sse", "http"])),
+  transport: z20.optional(z20.enum(["local", "stdio", "sse", "http"])),
   alwaysAllow: z20.optional(z20.array(z20.string())),
   tools: z20.optional(z20.array(z20.string())),
   kiroAutoApprove: z20.optional(z20.array(z20.string())),
@@ -6718,13 +6724,38 @@ var CopilotMcp = class _CopilotMcp extends ToolMcp {
 
 // src/features/mcp/copilotcli-mcp.ts
 import { join as join48 } from "path";
+var isRemoteServerType = (type) => {
+  return type === "http" || type === "sse";
+};
+var resolveCopilotcliServerType = (server) => {
+  if (server.type) {
+    return server.type;
+  }
+  if (server.transport === "http" || server.transport === "sse" || server.transport === "local") {
+    return server.transport;
+  }
+  return "stdio";
+};
 function addTypeField(mcpServers) {
   const result = {};
   for (const [name, server] of Object.entries(mcpServers)) {
     const parsed = McpServerSchema.parse(server);
+    const type = resolveCopilotcliServerType(parsed);
+    if (isRemoteServerType(type)) {
+      if (!parsed.url && !parsed.httpUrl) {
+        throw new Error(
+          `MCP server "${name}" is missing a url or httpUrl. GitHub Copilot CLI ${type} servers require a non-empty url or httpUrl.`
+        );
+      }
+      result[name] = {
+        ...parsed,
+        type
+      };
+      continue;
+    }
     if (!parsed.command) {
       throw new Error(
-        `MCP server "${name}" is missing a command. GitHub Copilot CLI stdio servers require a non-empty command.`
+        `MCP server "${name}" is missing a command. GitHub Copilot CLI ${type} servers require a non-empty command.`
       );
     }
     let command;
@@ -6741,10 +6772,10 @@ function addTypeField(mcpServers) {
       args = cmdArgs.length > 0 ? [...cmdArgs, ...parsed.args ?? []] : parsed.args;
     }
     result[name] = {
-      type: "stdio",
+      ...parsed,
+      type,
       command,
-      ...args && { args },
-      ...parsed.env && { env: parsed.env }
+      ...args && { args }
     };
   }
   return result;
@@ -6752,6 +6783,10 @@ function addTypeField(mcpServers) {
 function removeTypeField(config) {
   const result = {};
   for (const [name, server] of Object.entries(config.mcpServers ?? {})) {
+    if (server.type !== "stdio") {
+      result[name] = server;
+      continue;
+    }
     const { type: _, ...rest } = server;
     result[name] = rest;
   }
@@ -6774,13 +6809,7 @@ var CopilotcliMcp = class _CopilotcliMcp extends ToolMcp {
   isDeletable() {
     return !this.global;
   }
-  static getSettablePaths({ global } = {}) {
-    if (global) {
-      return {
-        relativeDirPath: ".copilot",
-        relativeFilePath: "mcp-config.json"
-      };
-    }
+  static getSettablePaths(_options = {}) {
     return {
       relativeDirPath: ".copilot",
       relativeFilePath: "mcp-config.json"
@@ -8396,8 +8425,11 @@ var AiDir = class {
   getOtherFiles() {
     return this.otherFiles;
   }
+  /**
+   * Returns the relative path from CWD with POSIX separators for consistent cross-platform output.
+   */
   getRelativePathFromCwd() {
-    return path2.join(this.relativeDirPath, this.dirName).replace(/\\/g, "/");
+    return toPosixPath(path2.join(this.relativeDirPath, this.dirName));
   }
   getGlobal() {
     return this.global;
@@ -15755,6 +15787,42 @@ var CopilotRule = class _CopilotRule extends ToolRule {
   }
 };
 
+// src/features/rules/copilotcli-rule.ts
+var CopilotcliRule = class _CopilotcliRule extends CopilotRule {
+  static fromCopilotRule(copilotRule, validate = true) {
+    return new _CopilotcliRule({
+      baseDir: copilotRule.getBaseDir(),
+      relativeDirPath: copilotRule.getRelativeDirPath(),
+      relativeFilePath: copilotRule.getRelativeFilePath(),
+      frontmatter: copilotRule.getFrontmatter(),
+      body: copilotRule.getBody(),
+      validate,
+      root: copilotRule.isRoot()
+    });
+  }
+  static fromRulesyncRule({
+    validate = true,
+    ...rest
+  }) {
+    return this.fromCopilotRule(CopilotRule.fromRulesyncRule({ validate, ...rest }), validate);
+  }
+  static async fromFile({
+    validate = true,
+    ...rest
+  }) {
+    return this.fromCopilotRule(await CopilotRule.fromFile({ validate, ...rest }), validate);
+  }
+  static forDeletion(params) {
+    return this.fromCopilotRule(CopilotRule.forDeletion(params), false);
+  }
+  static isTargetedByRulesyncRule(rulesyncRule) {
+    return this.isTargetedByRulesyncRuleDefault({
+      rulesyncRule,
+      toolTarget: "copilotcli"
+    });
+  }
+};
+
 // src/features/rules/cursor-rule.ts
 import { join as join111 } from "path";
 import { z as z60 } from "zod/mini";
@@ -16940,7 +17008,7 @@ var RooRule = class _RooRule extends ToolRule {
 
 // src/features/rules/rovodev-rule.ts
 import { join as join123 } from "path";
-var DISALLOWED_ROVODEV_MODULAR_RULE_BASENAMES = /* @__PURE__ */ new Set(["AGENTS.md", "AGENTS.local.md"]);
+var DISALLOWED_ROVODEV_MODULAR_RULE_BASENAMES = /* @__PURE__ */ new Set(["agents.md", "agents.local.md"]);
 var RovodevRule = class _RovodevRule extends ToolRule {
   /**
    * Whether `relativePath` (posix-style path relative to modular-rules root) may be imported as a modular rule.
@@ -16954,7 +17022,7 @@ var RovodevRule = class _RovodevRule extends ToolRule {
       if (segment === "" || segment === "." || segment === "..") {
         continue;
       }
-      if (DISALLOWED_ROVODEV_MODULAR_RULE_BASENAMES.has(segment)) {
+      if (DISALLOWED_ROVODEV_MODULAR_RULE_BASENAMES.has(segment.toLowerCase())) {
         return false;
       }
     }
@@ -16996,22 +17064,54 @@ var RovodevRule = class _RovodevRule extends ToolRule {
   }) {
     const paths = this.getSettablePaths({ global });
     if (!global && "nonRoot" in paths && paths.nonRoot && overrideDirPath === paths.nonRoot.relativeDirPath) {
-      if (!this.isAllowedModularRulesRelativePath(relativeFilePath)) {
-        throw new Error(
-          `Reserved Rovodev memory basename under modular-rules (not a modular rule): ${join123(overrideDirPath, relativeFilePath)}`
-        );
-      }
-      const fileContent2 = await readFileContent(join123(baseDir, overrideDirPath, relativeFilePath));
-      return new _RovodevRule({
+      return this.fromModularFile({
         baseDir,
-        relativeDirPath: overrideDirPath,
         relativeFilePath,
-        fileContent: fileContent2,
+        relativeDirPath: overrideDirPath,
         validate,
-        global,
-        root: false
+        global
       });
     }
+    return this.fromRootFile({
+      baseDir,
+      relativeFilePath,
+      overrideDirPath,
+      validate,
+      global,
+      paths
+    });
+  }
+  static async fromModularFile({
+    baseDir,
+    relativeFilePath,
+    relativeDirPath,
+    validate,
+    global
+  }) {
+    if (!this.isAllowedModularRulesRelativePath(relativeFilePath)) {
+      throw new Error(
+        `Reserved Rovodev memory basename under modular-rules (not a modular rule): ${join123(relativeDirPath, relativeFilePath)}`
+      );
+    }
+    const fileContent = await readFileContent(join123(baseDir, relativeDirPath, relativeFilePath));
+    return new _RovodevRule({
+      baseDir,
+      relativeDirPath,
+      relativeFilePath,
+      fileContent,
+      validate,
+      global,
+      root: false
+    });
+  }
+  static async fromRootFile({
+    baseDir,
+    relativeFilePath,
+    overrideDirPath,
+    validate,
+    global,
+    paths
+  }) {
     const relativeDirPath = overrideDirPath ?? paths.root.relativeDirPath;
     const agentsMdExpectedLocationsDescription = "alternativeRoots" in paths && paths.alternativeRoots && paths.alternativeRoots.length > 0 ? `${join123(paths.root.relativeDirPath, paths.root.relativeFilePath)} or project root` : join123(paths.root.relativeDirPath, paths.root.relativeFilePath);
     if (relativeFilePath !== "AGENTS.md") {
@@ -17300,6 +17400,7 @@ var rulesProcessorToolTargets = [
   "cline",
   "codexcli",
   "copilot",
+  "copilotcli",
   "cursor",
   "deepagents",
   "factorydroid",
@@ -17423,6 +17524,17 @@ var toolRuleFactories = /* @__PURE__ */ new Map([
       }
     }
   ],
+  [
+    "copilotcli",
+    {
+      class: CopilotcliRule,
+      meta: {
+        extension: "md",
+        supportsGlobal: true,
+        ruleDiscoveryMode: "auto"
+      }
+    }
+  ],
   [
     "cursor",
     {
@@ -17942,6 +18054,24 @@ var RulesProcessor = class extends FeatureProcessor {
         const dirName = dirname3(relative5(this.baseDir, filePath));
         return dirName === "" ? "." : dirName;
       };
+      const buildDeletionRulesFromPaths = (filePaths, opts) => {
+        const isNonRoot = opts !== void 0;
+        const effectiveBaseDir = isNonRoot ? opts.baseDirOverride : this.baseDir;
+        return filePaths.map((filePath) => {
+          const relativeDirPath = isNonRoot ? opts.relativeDirPathOverride : resolveRelativeDirPath(filePath);
+          const relativeFilePath = isNonRoot ? relative5(effectiveBaseDir, filePath) : basename10(filePath);
+          checkPathTraversal({
+            relativePath: isNonRoot ? relativeFilePath : relativeDirPath,
+            intendedRootDir: effectiveBaseDir
+          });
+          return factory.class.forDeletion({
+            baseDir: this.baseDir,
+            relativeDirPath,
+            relativeFilePath,
+            global: this.global
+          });
+        }).filter((rule) => rule.isDeletable());
+      };
       const findFilesWithFallback = async (primaryGlob, alternativeRoots, buildAltGlob) => {
         const primaryFilePaths = await findFilesByGlobs(primaryGlob);
         if (primaryFilePaths.length > 0) {
@@ -17966,19 +18096,7 @@ var RulesProcessor = class extends FeatureProcessor {
           (alt) => join126(this.baseDir, alt.relativeDirPath, alt.relativeFilePath)
         );
         if (forDeletion) {
-          return uniqueRootFilePaths.map((filePath) => {
-            const relativeDirPath = resolveRelativeDirPath(filePath);
-            checkPathTraversal({
-              relativePath: relativeDirPath,
-              intendedRootDir: this.baseDir
-            });
-            return factory.class.forDeletion({
-              baseDir: this.baseDir,
-              relativeDirPath,
-              relativeFilePath: basename10(filePath),
-              global: this.global
-            });
-          }).filter((rule) => rule.isDeletable());
+          return buildDeletionRulesFromPaths(uniqueRootFilePaths);
         }
         return await Promise.all(
           uniqueRootFilePaths.map((filePath) => {
@@ -18008,19 +18126,7 @@ var RulesProcessor = class extends FeatureProcessor {
           const uniqueLocalRootFilePaths2 = await findFilesByGlobs(
             join126(this.baseDir, "AGENTS.local.md")
           );
-          return uniqueLocalRootFilePaths2.map((filePath) => {
-            const relativeDirPath = resolveRelativeDirPath(filePath);
-            checkPathTraversal({
-              relativePath: relativeDirPath,
-              intendedRootDir: this.baseDir
-            });
-            return factory.class.forDeletion({
-              baseDir: this.baseDir,
-              relativeDirPath,
-              relativeFilePath: basename10(filePath),
-              global: this.global
-            });
-          }).filter((rule) => rule.isDeletable());
+          return buildDeletionRulesFromPaths(uniqueLocalRootFilePaths2);
         }
         if (this.toolTarget !== "claudecode" && this.toolTarget !== "claudecode-legacy") {
           return [];
@@ -18033,19 +18139,7 @@ var RulesProcessor = class extends FeatureProcessor {
           settablePaths.alternativeRoots,
           (alt) => join126(this.baseDir, alt.relativeDirPath, "CLAUDE.local.md")
         );
-        return uniqueLocalRootFilePaths.map((filePath) => {
-          const relativeDirPath = resolveRelativeDirPath(filePath);
-          checkPathTraversal({
-            relativePath: relativeDirPath,
-            intendedRootDir: this.baseDir
-          });
-          return factory.class.forDeletion({
-            baseDir: this.baseDir,
-            relativeDirPath,
-            relativeFilePath: basename10(filePath),
-            global: this.global
-          });
-        }).filter((rule) => rule.isDeletable());
+        return buildDeletionRulesFromPaths(uniqueLocalRootFilePaths);
       })();
       this.logger.debug(
         `Found ${localRootToolRules.length} local root tool rule files for deletion`
@@ -18059,19 +18153,7 @@ var RulesProcessor = class extends FeatureProcessor {
           return [];
         }
         const mirrorPaths = await findFilesByGlobs(join126(this.baseDir, "AGENTS.md"));
-        return mirrorPaths.map((filePath) => {
-          const relativeDirPath = resolveRelativeDirPath(filePath);
-          checkPathTraversal({
-            relativePath: relativeDirPath,
-            intendedRootDir: this.baseDir
-          });
-          return factory.class.forDeletion({
-            baseDir: this.baseDir,
-            relativeDirPath,
-            relativeFilePath: basename10(filePath),
-            global: this.global
-          });
-        }).filter((rule) => rule.isDeletable());
+        return buildDeletionRulesFromPaths(mirrorPaths);
       })();
       const nonRootToolRules = await (async () => {
         if (!settablePaths.nonRoot) {
@@ -18081,8 +18163,14 @@ var RulesProcessor = class extends FeatureProcessor {
         const nonRootFilePaths = await findFilesByGlobs(
           join126(nonRootBaseDir, "**", `*.${factory.meta.extension}`)
         );
+        if (forDeletion) {
+          return buildDeletionRulesFromPaths(nonRootFilePaths, {
+            baseDirOverride: nonRootBaseDir,
+            relativeDirPathOverride: settablePaths.nonRoot.relativeDirPath
+          });
+        }
         const modularRootRelative = settablePaths.nonRoot.relativeDirPath;
-        const nonRootPathsForImport = !forDeletion && this.toolTarget === "rovodev" ? nonRootFilePaths.filter((filePath) => {
+        const nonRootPathsForImport = this.toolTarget === "rovodev" ? nonRootFilePaths.filter((filePath) => {
           const relativeFilePath = relative5(nonRootBaseDir, filePath);
           const ok = RovodevRule.isAllowedModularRulesRelativePath(relativeFilePath);
           if (!ok) {
@@ -18092,21 +18180,6 @@ var RulesProcessor = class extends FeatureProcessor {
           }
           return ok;
         }) : nonRootFilePaths;
-        if (forDeletion) {
-          return nonRootFilePaths.map((filePath) => {
-            const relativeFilePath = relative5(nonRootBaseDir, filePath);
-            checkPathTraversal({
-              relativePath: relativeFilePath,
-              intendedRootDir: nonRootBaseDir
-            });
-            return factory.class.forDeletion({
-              baseDir: this.baseDir,
-              relativeDirPath: settablePaths.nonRoot?.relativeDirPath ?? ".",
-              relativeFilePath,
-              global: this.global
-            });
-          }).filter((rule) => rule.isDeletable());
-        }
         return await Promise.all(
           nonRootPathsForImport.map((filePath) => {
             const relativeFilePath = relative5(nonRootBaseDir, filePath);
@@ -19093,6 +19166,7 @@ export {
   FETCH_CONCURRENCY_LIMIT,
   formatError,
   ensureDir,
+  toPosixPath,
   checkPathTraversal,
   directoryExists,
   readFileContent,