rulesync 7.3.0 → 7.4.0

package/dist/cli/index.cjs

@@ -12175,14 +12175,22 @@ var CopilotRuleFrontmatterSchema = import_mini46.z.object({
 var CopilotRule = class _CopilotRule extends ToolRule {
   frontmatter;
   body;
-  static getSettablePaths(_options = {}) {
+  static getSettablePaths(options = {}) {
+    if (options.global) {
+      return {
+        root: {
+          relativeDirPath: buildToolPath(".copilot", ".", options.excludeToolDir),
+          relativeFilePath: "copilot-instructions.md"
+        }
+      };
+    }
     return {
       root: {
-        relativeDirPath: buildToolPath(".github", ".", _options.excludeToolDir),
+        relativeDirPath: buildToolPath(".github", ".", options.excludeToolDir),
         relativeFilePath: "copilot-instructions.md"
       },
       nonRoot: {
-        relativeDirPath: buildToolPath(".github", "instructions", _options.excludeToolDir)
+        relativeDirPath: buildToolPath(".github", "instructions", options.excludeToolDir)
       }
     };
   }
@@ -12233,10 +12241,12 @@ var CopilotRule = class _CopilotRule extends ToolRule {
   static fromRulesyncRule({
     baseDir = process.cwd(),
     rulesyncRule,
-    validate = true
+    validate = true,
+    global = false
   }) {
     const rulesyncFrontmatter = rulesyncRule.getFrontmatter();
     const root = rulesyncFrontmatter.root;
+    const paths = this.getSettablePaths({ global });
     const copilotFrontmatter = {
       description: rulesyncFrontmatter.description,
       applyTo: rulesyncFrontmatter.globs?.length ? rulesyncFrontmatter.globs.join(",") : void 0,
@@ -12248,12 +12258,15 @@ var CopilotRule = class _CopilotRule extends ToolRule {
         baseDir,
         frontmatter: copilotFrontmatter,
         body,
-        relativeDirPath: this.getSettablePaths().root.relativeDirPath,
-        relativeFilePath: this.getSettablePaths().root.relativeFilePath,
+        relativeDirPath: paths.root.relativeDirPath,
+        relativeFilePath: paths.root.relativeFilePath,
         validate,
         root
       });
     }
+    if (!paths.nonRoot) {
+      throw new Error(`nonRoot path is not set for ${rulesyncRule.getRelativeFilePath()}`);
+    }
     const originalFileName = rulesyncRule.getRelativeFilePath();
     const nameWithoutExt = originalFileName.replace(/\.md$/, "");
     const newFileName = `${nameWithoutExt}.instructions.md`;
@@ -12261,7 +12274,7 @@ var CopilotRule = class _CopilotRule extends ToolRule {
       baseDir,
       frontmatter: copilotFrontmatter,
       body,
-      relativeDirPath: this.getSettablePaths().nonRoot.relativeDirPath,
+      relativeDirPath: paths.nonRoot.relativeDirPath,
       relativeFilePath: newFileName,
       validate,
       root
@@ -12270,25 +12283,29 @@ var CopilotRule = class _CopilotRule extends ToolRule {
   static async fromFile({
     baseDir = process.cwd(),
     relativeFilePath,
-    validate = true
+    validate = true,
+    global = false
   }) {
-    const isRoot = relativeFilePath === "copilot-instructions.md";
-    const relativePath = isRoot ? (0, import_node_path95.join)(
-      this.getSettablePaths().root.relativeDirPath,
-      this.getSettablePaths().root.relativeFilePath
-    ) : (0, import_node_path95.join)(this.getSettablePaths().nonRoot.relativeDirPath, relativeFilePath);
-    const fileContent = await readFileContent((0, import_node_path95.join)(baseDir, relativePath));
+    const paths = this.getSettablePaths({ global });
+    const isRoot = relativeFilePath === paths.root.relativeFilePath;
     if (isRoot) {
+      const relativePath2 = (0, import_node_path95.join)(paths.root.relativeDirPath, paths.root.relativeFilePath);
+      const fileContent2 = await readFileContent((0, import_node_path95.join)(baseDir, relativePath2));
       return new _CopilotRule({
         baseDir,
-        relativeDirPath: this.getSettablePaths().root.relativeDirPath,
-        relativeFilePath: this.getSettablePaths().root.relativeFilePath,
+        relativeDirPath: paths.root.relativeDirPath,
+        relativeFilePath: paths.root.relativeFilePath,
         frontmatter: {},
-        body: fileContent.trim(),
+        body: fileContent2.trim(),
         validate,
         root: isRoot
       });
     }
+    if (!paths.nonRoot) {
+      throw new Error(`nonRoot path is not set for ${relativeFilePath}`);
+    }
+    const relativePath = (0, import_node_path95.join)(paths.nonRoot.relativeDirPath, relativeFilePath);
+    const fileContent = await readFileContent((0, import_node_path95.join)(baseDir, relativePath));
     const { frontmatter, body: content } = parseFrontmatter(fileContent);
     const result = CopilotRuleFrontmatterSchema.safeParse(frontmatter);
     if (!result.success) {
@@ -12298,7 +12315,7 @@ var CopilotRule = class _CopilotRule extends ToolRule {
     }
     return new _CopilotRule({
       baseDir,
-      relativeDirPath: this.getSettablePaths().nonRoot.relativeDirPath,
+      relativeDirPath: paths.nonRoot.relativeDirPath,
       relativeFilePath: relativeFilePath.endsWith(".instructions.md") ? relativeFilePath : relativeFilePath.replace(/\.md$/, ".instructions.md"),
       frontmatter: result.data,
       body: content.trim(),
@@ -12309,9 +12326,11 @@ var CopilotRule = class _CopilotRule extends ToolRule {
   static forDeletion({
     baseDir = process.cwd(),
     relativeDirPath,
-    relativeFilePath
+    relativeFilePath,
+    global = false
   }) {
-    const isRoot = relativeFilePath === this.getSettablePaths().root.relativeFilePath;
+    const paths = this.getSettablePaths({ global });
+    const isRoot = relativeFilePath === paths.root.relativeFilePath;
     return new _CopilotRule({
       baseDir,
       relativeDirPath,
@@ -12991,46 +13010,78 @@ var KiroRule = class _KiroRule extends ToolRule {
 // src/features/rules/opencode-rule.ts
 var import_node_path102 = require("path");
 var OpenCodeRule = class _OpenCodeRule extends ToolRule {
-  static getSettablePaths(_options = {}) {
+  static getSettablePaths({
+    global,
+    excludeToolDir
+  } = {}) {
+    if (global) {
+      return {
+        root: {
+          relativeDirPath: buildToolPath(".config/opencode", ".", excludeToolDir),
+          relativeFilePath: "AGENTS.md"
+        }
+      };
+    }
     return {
       root: {
         relativeDirPath: ".",
         relativeFilePath: "AGENTS.md"
       },
       nonRoot: {
-        relativeDirPath: buildToolPath(".opencode", "memories", _options.excludeToolDir)
+        relativeDirPath: buildToolPath(".opencode", "memories", excludeToolDir)
       }
     };
   }
   static async fromFile({
     baseDir = process.cwd(),
     relativeFilePath,
-    validate = true
+    validate = true,
+    global = false
   }) {
-    const isRoot = relativeFilePath === "AGENTS.md";
-    const relativePath = isRoot ? "AGENTS.md" : (0, import_node_path102.join)(".opencode", "memories", relativeFilePath);
+    const paths = this.getSettablePaths({ global });
+    const isRoot = relativeFilePath === paths.root.relativeFilePath;
+    if (isRoot) {
+      const relativePath2 = paths.root.relativeFilePath;
+      const fileContent2 = await readFileContent(
+        (0, import_node_path102.join)(baseDir, paths.root.relativeDirPath, relativePath2)
+      );
+      return new _OpenCodeRule({
+        baseDir,
+        relativeDirPath: paths.root.relativeDirPath,
+        relativeFilePath: paths.root.relativeFilePath,
+        fileContent: fileContent2,
+        validate,
+        root: true
+      });
+    }
+    if (!paths.nonRoot) {
+      throw new Error(`nonRoot path is not set for ${relativeFilePath}`);
+    }
+    const relativePath = (0, import_node_path102.join)(paths.nonRoot.relativeDirPath, relativeFilePath);
     const fileContent = await readFileContent((0, import_node_path102.join)(baseDir, relativePath));
     return new _OpenCodeRule({
       baseDir,
-      relativeDirPath: isRoot ? this.getSettablePaths().root.relativeDirPath : this.getSettablePaths().nonRoot.relativeDirPath,
-      relativeFilePath: isRoot ? "AGENTS.md" : relativeFilePath,
+      relativeDirPath: paths.nonRoot.relativeDirPath,
+      relativeFilePath,
+      fileContent,
       validate,
-      root: isRoot,
-      fileContent
+      root: false
     });
   }
   static fromRulesyncRule({
     baseDir = process.cwd(),
     rulesyncRule,
-    validate = true
+    validate = true,
+    global = false
   }) {
+    const paths = this.getSettablePaths({ global });
     return new _OpenCodeRule(
       this.buildToolRuleParamsDefault({
         baseDir,
         rulesyncRule,
         validate,
-        rootPath: this.getSettablePaths().root,
-        nonRootPath: this.getSettablePaths().nonRoot
+        rootPath: paths.root,
+        nonRootPath: paths.nonRoot
       })
     );
   }
@@ -13043,9 +13094,11 @@ var OpenCodeRule = class _OpenCodeRule extends ToolRule {
   static forDeletion({
     baseDir = process.cwd(),
     relativeDirPath,
-    relativeFilePath
+    relativeFilePath,
+    global = false
   }) {
-    const isRoot = relativeFilePath === "AGENTS.md" && relativeDirPath === ".";
+    const paths = this.getSettablePaths({ global });
+    const isRoot = relativeFilePath === paths.root.relativeFilePath;
     return new _OpenCodeRule({
       baseDir,
       relativeDirPath,
@@ -13503,42 +13556,66 @@ var toolRuleFactories = /* @__PURE__ */ new Map([
     "antigravity",
     {
       class: AntigravityRule,
-      meta: { extension: "md", supportsGlobal: false, ruleDiscoveryMode: "auto" }
+      meta: {
+        extension: "md",
+        supportsGlobal: false,
+        ruleDiscoveryMode: "auto"
+      }
     }
   ],
   [
     "augmentcode",
     {
       class: AugmentcodeRule,
-      meta: { extension: "md", supportsGlobal: false, ruleDiscoveryMode: "auto" }
+      meta: {
+        extension: "md",
+        supportsGlobal: false,
+        ruleDiscoveryMode: "auto"
+      }
     }
   ],
   [
     "augmentcode-legacy",
     {
       class: AugmentcodeLegacyRule,
-      meta: { extension: "md", supportsGlobal: false, ruleDiscoveryMode: "toon" }
+      meta: {
+        extension: "md",
+        supportsGlobal: false,
+        ruleDiscoveryMode: "toon"
+      }
     }
   ],
   [
     "claudecode",
     {
       class: ClaudecodeRule,
-      meta: { extension: "md", supportsGlobal: true, ruleDiscoveryMode: "auto" }
+      meta: {
+        extension: "md",
+        supportsGlobal: true,
+        ruleDiscoveryMode: "auto"
+      }
     }
   ],
   [
     "claudecode-legacy",
     {
       class: ClaudecodeLegacyRule,
-      meta: { extension: "md", supportsGlobal: true, ruleDiscoveryMode: "claudecode-legacy" }
+      meta: {
+        extension: "md",
+        supportsGlobal: true,
+        ruleDiscoveryMode: "claudecode-legacy"
+      }
     }
   ],
   [
     "cline",
     {
       class: ClineRule,
-      meta: { extension: "md", supportsGlobal: false, ruleDiscoveryMode: "auto" }
+      meta: {
+        extension: "md",
+        supportsGlobal: false,
+        ruleDiscoveryMode: "auto"
+      }
     }
   ],
   [
@@ -13561,7 +13638,7 @@ var toolRuleFactories = /* @__PURE__ */ new Map([
       class: CopilotRule,
       meta: {
         extension: "md",
-        supportsGlobal: false,
+        supportsGlobal: true,
         ruleDiscoveryMode: "auto"
       }
     }
@@ -13611,42 +13688,66 @@ var toolRuleFactories = /* @__PURE__ */ new Map([
     "junie",
     {
       class: JunieRule,
-      meta: { extension: "md", supportsGlobal: false, ruleDiscoveryMode: "toon" }
+      meta: {
+        extension: "md",
+        supportsGlobal: false,
+        ruleDiscoveryMode: "toon"
+      }
     }
   ],
   [
     "kilo",
     {
       class: KiloRule,
-      meta: { extension: "md", supportsGlobal: true, ruleDiscoveryMode: "auto" }
+      meta: {
+        extension: "md",
+        supportsGlobal: true,
+        ruleDiscoveryMode: "auto"
+      }
     }
   ],
   [
     "kiro",
     {
       class: KiroRule,
-      meta: { extension: "md", supportsGlobal: false, ruleDiscoveryMode: "toon" }
+      meta: {
+        extension: "md",
+        supportsGlobal: false,
+        ruleDiscoveryMode: "toon"
+      }
     }
   ],
   [
     "opencode",
     {
       class: OpenCodeRule,
-      meta: { extension: "md", supportsGlobal: false, ruleDiscoveryMode: "toon" }
+      meta: {
+        extension: "md",
+        supportsGlobal: true,
+        ruleDiscoveryMode: "toon"
+      }
     }
   ],
   [
     "qwencode",
     {
       class: QwencodeRule,
-      meta: { extension: "md", supportsGlobal: false, ruleDiscoveryMode: "toon" }
+      meta: {
+        extension: "md",
+        supportsGlobal: false,
+        ruleDiscoveryMode: "toon"
+      }
     }
   ],
   [
     "replit",
     {
       class: ReplitRule,
-      meta: { extension: "md", supportsGlobal: false, ruleDiscoveryMode: "auto" }
+      meta: {
+        extension: "md",
+        supportsGlobal: false,
+        ruleDiscoveryMode: "auto"
+      }
     }
   ],
   [
@@ -13668,14 +13769,22 @@ var toolRuleFactories = /* @__PURE__ */ new Map([
     "warp",
     {
       class: WarpRule,
-      meta: { extension: "md", supportsGlobal: false, ruleDiscoveryMode: "toon" }
+      meta: {
+        extension: "md",
+        supportsGlobal: false,
+        ruleDiscoveryMode: "toon"
+      }
     }
   ],
   [
     "windsurf",
     {
       class: WindsurfRule,
-      meta: { extension: "md", supportsGlobal: false, ruleDiscoveryMode: "auto" }
+      meta: {
+        extension: "md",
+        supportsGlobal: false,
+        ruleDiscoveryMode: "auto"
+      }
     }
   ]
 ]);
@@ -13823,7 +13932,9 @@ var RulesProcessor = class extends FeatureProcessor {
         })
       );
     } else if (this.toolTarget === "claudecode-legacy") {
-      const paths = ClaudecodeLegacyRule.getSettablePaths({ global: this.global });
+      const paths = ClaudecodeLegacyRule.getSettablePaths({
+        global: this.global
+      });
       toolRules.push(
         new ClaudecodeLegacyRule({
           baseDir: this.baseDir,
@@ -13909,7 +14020,10 @@ var RulesProcessor = class extends FeatureProcessor {
     const rulesyncRules = await Promise.all(
       files.map((file) => {
         const relativeFilePath = (0, import_node_path108.relative)(rulesyncBaseDir, file);
-        checkPathTraversal({ relativePath: relativeFilePath, intendedRootDir: rulesyncBaseDir });
+        checkPathTraversal({
+          relativePath: relativeFilePath,
+          intendedRootDir: rulesyncBaseDir
+        });
         return RulesyncRule.fromFile({
           relativeFilePath
         });
@@ -13960,7 +14074,9 @@ var RulesProcessor = class extends FeatureProcessor {
   } = {}) {
     try {
       const factory = this.getFactory(this.toolTarget);
-      const settablePaths = factory.class.getSettablePaths({ global: this.global });
+      const settablePaths = factory.class.getSettablePaths({
+        global: this.global
+      });
       const rootToolRules = await (async () => {
         if (!settablePaths.root) {
           return [];
@@ -14042,7 +14158,10 @@ var RulesProcessor = class extends FeatureProcessor {
         return await Promise.all(
           nonRootFilePaths.map((filePath) => {
             const relativeFilePath = (0, import_node_path108.relative)(nonRootBaseDir, filePath);
-            checkPathTraversal({ relativePath: relativeFilePath, intendedRootDir: nonRootBaseDir });
+            checkPathTraversal({
+              relativePath: relativeFilePath,
+              intendedRootDir: nonRootBaseDir
+            });
             return factory.class.fromFile({
               baseDir: this.baseDir,
               relativeFilePath,
@@ -16737,19 +16856,10 @@ async function resolveAndFetchSources(params) {
   let lock = options.updateSources ? createEmptyLock() : await readLockFile({ baseDir });
   if (options.frozen) {
     const missingKeys = [];
-    const missingSkills = [];
-    const curatedDir = (0, import_node_path115.join)(baseDir, RULESYNC_CURATED_SKILLS_RELATIVE_DIR_PATH);
     for (const source of sources) {
       const locked = getLockedSource(lock, source.source);
       if (!locked) {
         missingKeys.push(source.source);
-        continue;
-      }
-      const skillNames = getLockedSkillNames(locked);
-      for (const skillName of skillNames) {
-        if (!await directoryExists((0, import_node_path115.join)(curatedDir, skillName))) {
-          missingSkills.push(`${source.source}:${skillName}`);
-        }
       }
     }
     if (missingKeys.length > 0) {
@@ -16757,11 +16867,6 @@ async function resolveAndFetchSources(params) {
         `Frozen install failed: lockfile is missing entries for: ${missingKeys.join(", ")}. Run 'rulesync install' to update the lockfile.`
       );
     }
-    if (missingSkills.length > 0) {
-      throw new Error(
-        `Frozen install failed: locked skills missing from disk: ${missingSkills.join(", ")}. Run 'rulesync install' to fetch missing skills.`
-      );
-    }
   }
   const originalLockJson = JSON.stringify(lock);
   const token = GitHubClient.resolveToken(options.token);
@@ -18782,7 +18887,7 @@ async function updateCommand(currentVersion, options) {
 }
 
 // src/cli/index.ts
-var getVersion = () => "7.3.0";
+var getVersion = () => "7.4.0";
 var main = async () => {
   const program = new import_commander.Command();
   const version = getVersion();
@@ -18855,7 +18960,10 @@ var main = async () => {
       process.exit(1);
     }
   });
-  program.command("install").description("Install skills from declarative sources in rulesync.jsonc").option("--update", "Force re-resolve all source refs, ignoring lockfile").option("--frozen", "Fail if lockfile is missing or out of sync (for CI)").option("--token <token>", "GitHub token for private repos").option("-c, --config <path>", "Path to configuration file").option("-V, --verbose", "Verbose output").option("-s, --silent", "Suppress all output").action(async (options) => {
+  program.command("install").description("Install skills from declarative sources in rulesync.jsonc").option("--update", "Force re-resolve all source refs, ignoring lockfile").option(
+    "--frozen",
+    "Fail if lockfile is missing or out of sync (for CI); fetches missing skills using locked refs"
+  ).option("--token <token>", "GitHub token for private repos").option("-c, --config <path>", "Path to configuration file").option("-V, --verbose", "Verbose output").option("-s, --silent", "Suppress all output").action(async (options) => {
     try {
       await installCommand({
         update: options.update,

package/dist/cli/index.js

@@ -60,7 +60,7 @@ import {
   removeTempDirectory,
   stringifyFrontmatter,
   writeFileContent
-} from "../chunk-NRNUPCXY.js";
+} from "../chunk-UCC3WPDL.js";
 
 // src/cli/index.ts
 import { Command } from "commander";
@@ -1759,19 +1759,10 @@ async function resolveAndFetchSources(params) {
   let lock = options.updateSources ? createEmptyLock() : await readLockFile({ baseDir });
   if (options.frozen) {
     const missingKeys = [];
-    const missingSkills = [];
-    const curatedDir = join5(baseDir, RULESYNC_CURATED_SKILLS_RELATIVE_DIR_PATH);
     for (const source of sources) {
       const locked = getLockedSource(lock, source.source);
       if (!locked) {
         missingKeys.push(source.source);
-        continue;
-      }
-      const skillNames = getLockedSkillNames(locked);
-      for (const skillName of skillNames) {
-        if (!await directoryExists(join5(curatedDir, skillName))) {
-          missingSkills.push(`${source.source}:${skillName}`);
-        }
       }
     }
     if (missingKeys.length > 0) {
@@ -1779,11 +1770,6 @@ async function resolveAndFetchSources(params) {
         `Frozen install failed: lockfile is missing entries for: ${missingKeys.join(", ")}. Run 'rulesync install' to update the lockfile.`
       );
     }
-    if (missingSkills.length > 0) {
-      throw new Error(
-        `Frozen install failed: locked skills missing from disk: ${missingSkills.join(", ")}. Run 'rulesync install' to fetch missing skills.`
-      );
-    }
   }
   const originalLockJson = JSON.stringify(lock);
   const token = GitHubClient.resolveToken(options.token);
@@ -3804,7 +3790,7 @@ async function updateCommand(currentVersion, options) {
 }
 
 // src/cli/index.ts
-var getVersion = () => "7.3.0";
+var getVersion = () => "7.4.0";
 var main = async () => {
   const program = new Command();
   const version = getVersion();
@@ -3877,7 +3863,10 @@ var main = async () => {
       process.exit(1);
     }
   });
-  program.command("install").description("Install skills from declarative sources in rulesync.jsonc").option("--update", "Force re-resolve all source refs, ignoring lockfile").option("--frozen", "Fail if lockfile is missing or out of sync (for CI)").option("--token <token>", "GitHub token for private repos").option("-c, --config <path>", "Path to configuration file").option("-V, --verbose", "Verbose output").option("-s, --silent", "Suppress all output").action(async (options) => {
+  program.command("install").description("Install skills from declarative sources in rulesync.jsonc").option("--update", "Force re-resolve all source refs, ignoring lockfile").option(
+    "--frozen",
+    "Fail if lockfile is missing or out of sync (for CI); fetches missing skills using locked refs"
+  ).option("--token <token>", "GitHub token for private repos").option("-c, --config <path>", "Path to configuration file").option("-V, --verbose", "Verbose output").option("-s, --silent", "Suppress all output").action(async (options) => {
     try {
       await installCommand({
         update: options.update,