rulesync 7.24.0 → 7.26.0

package/dist/index.cjs

@@ -115,6 +115,9 @@ async function readOrInitializeFileContent(filePath, initialContent = "") {
     return initialContent;
   }
 }
+function toPosixPath(p) {
+  return p.replace(/\\/g, "/");
+}
 function checkPathTraversal({
   relativePath,
   intendedRootDir
@@ -951,8 +954,11 @@ var AiFile = class {
   getFileContent() {
     return this.fileContent;
   }
+  /**
+   * Returns the relative path from CWD with POSIX separators for consistent cross-platform output.
+   */
   getRelativePathFromCwd() {
-    return import_node_path6.default.join(this.relativeDirPath, this.relativeFilePath).replace(/\\/g, "/");
+    return toPosixPath(import_node_path6.default.join(this.relativeDirPath, this.relativeFilePath));
   }
   setFileContent(newFileContent) {
     this.fileContent = newFileContent;
@@ -6129,7 +6135,7 @@ var import_mini21 = require("zod/mini");
 // src/types/mcp.ts
 var import_mini20 = require("zod/mini");
 var McpServerSchema = import_mini20.z.looseObject({
-  type: import_mini20.z.optional(import_mini20.z.enum(["stdio", "sse", "http"])),
+  type: import_mini20.z.optional(import_mini20.z.enum(["local", "stdio", "sse", "http"])),
   command: import_mini20.z.optional(import_mini20.z.union([import_mini20.z.string(), import_mini20.z.array(import_mini20.z.string())])),
   args: import_mini20.z.optional(import_mini20.z.array(import_mini20.z.string())),
   url: import_mini20.z.optional(import_mini20.z.string()),
@@ -6140,7 +6146,7 @@ var McpServerSchema = import_mini20.z.looseObject({
   timeout: import_mini20.z.optional(import_mini20.z.number()),
   trust: import_mini20.z.optional(import_mini20.z.boolean()),
   cwd: import_mini20.z.optional(import_mini20.z.string()),
-  transport: import_mini20.z.optional(import_mini20.z.enum(["stdio", "sse", "http"])),
+  transport: import_mini20.z.optional(import_mini20.z.enum(["local", "stdio", "sse", "http"])),
   alwaysAllow: import_mini20.z.optional(import_mini20.z.array(import_mini20.z.string())),
   tools: import_mini20.z.optional(import_mini20.z.array(import_mini20.z.string())),
   kiroAutoApprove: import_mini20.z.optional(import_mini20.z.array(import_mini20.z.string())),
@@ -6725,13 +6731,38 @@ var CopilotMcp = class _CopilotMcp extends ToolMcp {
 
 // src/features/mcp/copilotcli-mcp.ts
 var import_node_path51 = require("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;
@@ -6748,10 +6779,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;
@@ -6759,6 +6790,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;
   }
@@ -6781,13 +6816,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"
@@ -8403,8 +8432,11 @@ var AiDir = class {
   getOtherFiles() {
     return this.otherFiles;
   }
+  /**
+   * Returns the relative path from CWD with POSIX separators for consistent cross-platform output.
+   */
   getRelativePathFromCwd() {
-    return import_node_path62.default.join(this.relativeDirPath, this.dirName).replace(/\\/g, "/");
+    return toPosixPath(import_node_path62.default.join(this.relativeDirPath, this.dirName));
   }
   getGlobal() {
     return this.global;
@@ -15762,6 +15794,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
 var import_node_path114 = require("path");
 var import_mini60 = require("zod/mini");
@@ -16947,7 +17015,7 @@ var RooRule = class _RooRule extends ToolRule {
 
 // src/features/rules/rovodev-rule.ts
 var import_node_path126 = require("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.
@@ -16961,7 +17029,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;
       }
     }
@@ -17003,22 +17071,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): ${(0, import_node_path126.join)(overrideDirPath, relativeFilePath)}`
-        );
-      }
-      const fileContent2 = await readFileContent((0, import_node_path126.join)(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): ${(0, import_node_path126.join)(relativeDirPath, relativeFilePath)}`
+      );
+    }
+    const fileContent = await readFileContent((0, import_node_path126.join)(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 ? `${(0, import_node_path126.join)(paths.root.relativeDirPath, paths.root.relativeFilePath)} or project root` : (0, import_node_path126.join)(paths.root.relativeDirPath, paths.root.relativeFilePath);
     if (relativeFilePath !== "AGENTS.md") {
@@ -17307,6 +17407,7 @@ var rulesProcessorToolTargets = [
   "cline",
   "codexcli",
   "copilot",
+  "copilotcli",
   "cursor",
   "deepagents",
   "factorydroid",
@@ -17430,6 +17531,17 @@ var toolRuleFactories = /* @__PURE__ */ new Map([
       }
     }
   ],
+  [
+    "copilotcli",
+    {
+      class: CopilotcliRule,
+      meta: {
+        extension: "md",
+        supportsGlobal: true,
+        ruleDiscoveryMode: "auto"
+      }
+    }
+  ],
   [
     "cursor",
     {
@@ -17949,6 +18061,24 @@ var RulesProcessor = class extends FeatureProcessor {
         const dirName = (0, import_node_path129.dirname)((0, import_node_path129.relative)(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 ? (0, import_node_path129.relative)(effectiveBaseDir, filePath) : (0, import_node_path129.basename)(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) {
@@ -17973,19 +18103,7 @@ var RulesProcessor = class extends FeatureProcessor {
           (alt) => (0, import_node_path129.join)(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: (0, import_node_path129.basename)(filePath),
-              global: this.global
-            });
-          }).filter((rule) => rule.isDeletable());
+          return buildDeletionRulesFromPaths(uniqueRootFilePaths);
         }
         return await Promise.all(
           uniqueRootFilePaths.map((filePath) => {
@@ -18015,19 +18133,7 @@ var RulesProcessor = class extends FeatureProcessor {
           const uniqueLocalRootFilePaths2 = await findFilesByGlobs(
             (0, import_node_path129.join)(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: (0, import_node_path129.basename)(filePath),
-              global: this.global
-            });
-          }).filter((rule) => rule.isDeletable());
+          return buildDeletionRulesFromPaths(uniqueLocalRootFilePaths2);
         }
         if (this.toolTarget !== "claudecode" && this.toolTarget !== "claudecode-legacy") {
           return [];
@@ -18040,19 +18146,7 @@ var RulesProcessor = class extends FeatureProcessor {
           settablePaths.alternativeRoots,
           (alt) => (0, import_node_path129.join)(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: (0, import_node_path129.basename)(filePath),
-            global: this.global
-          });
-        }).filter((rule) => rule.isDeletable());
+        return buildDeletionRulesFromPaths(uniqueLocalRootFilePaths);
       })();
       this.logger.debug(
         `Found ${localRootToolRules.length} local root tool rule files for deletion`
@@ -18066,19 +18160,7 @@ var RulesProcessor = class extends FeatureProcessor {
           return [];
         }
         const mirrorPaths = await findFilesByGlobs((0, import_node_path129.join)(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: (0, import_node_path129.basename)(filePath),
-            global: this.global
-          });
-        }).filter((rule) => rule.isDeletable());
+        return buildDeletionRulesFromPaths(mirrorPaths);
       })();
       const nonRootToolRules = await (async () => {
         if (!settablePaths.nonRoot) {
@@ -18088,8 +18170,14 @@ var RulesProcessor = class extends FeatureProcessor {
         const nonRootFilePaths = await findFilesByGlobs(
           (0, import_node_path129.join)(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 = (0, import_node_path129.relative)(nonRootBaseDir, filePath);
           const ok = RovodevRule.isAllowedModularRulesRelativePath(relativeFilePath);
           if (!ok) {
@@ -18099,21 +18187,6 @@ var RulesProcessor = class extends FeatureProcessor {
           }
           return ok;
         }) : nonRootFilePaths;
-        if (forDeletion) {
-          return nonRootFilePaths.map((filePath) => {
-            const relativeFilePath = (0, import_node_path129.relative)(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 = (0, import_node_path129.relative)(nonRootBaseDir, filePath);

package/dist/index.d.cts

@@ -120,6 +120,9 @@ declare abstract class AiDir {
         frontmatter?: Record<string, unknown>;
     } | undefined;
     getOtherFiles(): AiDirFile[];
+    /**
+     * Returns the relative path from CWD with POSIX separators for consistent cross-platform output.
+     */
     getRelativePathFromCwd(): string;
     getGlobal(): boolean;
     setMainFile(name: string, body: string, frontmatter?: Record<string, unknown>): void;

package/dist/index.d.ts

@@ -120,6 +120,9 @@ declare abstract class AiDir {
         frontmatter?: Record<string, unknown>;
     } | undefined;
     getOtherFiles(): AiDirFile[];
+    /**
+     * Returns the relative path from CWD with POSIX separators for consistent cross-platform output.
+     */
     getRelativePathFromCwd(): string;
     getGlobal(): boolean;
     setMainFile(name: string, body: string, frontmatter?: Record<string, unknown>): void;

package/dist/index.js

@@ -6,7 +6,7 @@ import {
   checkRulesyncDirExists,
   generate,
   importFromTool
-} from "./chunk-A4FZT3JT.js";
+} from "./chunk-MMC5VK27.js";
 
 // src/index.ts
 async function generate2(options = {}) {

package/package.json

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