Merge pull request #1264 from sheldonhull/fix/multi-root-per-target

fix: allow multiple root rules with different targets + copilot global nonRoot

Author: dyoshikawa

docs/guide/global-mode.md

@@ -48,6 +48,5 @@ Currently, supports rules generation for Claude Code, GitHub Copilot, and OpenCo
 > Currently, when in the directory enabled global mode:
 >
 > - `rulesync.jsonc` only supports `global`, `features`, `delete` and `verbose`. `Features` can be set `"rules"` and `"commands"`. Other parameters are ignored.
-> - `rules/*.md` only supports single file has `root: true`, and frontmatter parameters without `root` are ignored.
+> - Tools support only a single `root: true` file in global mode as a target, e.g. you can't have 2 root files targeting Claude.
 > - Only Claude Code is supported for global mode commands.
-> - Global mode rules are supported for Claude Code, GitHub Copilot, and OpenCode.

skills/rulesync/global-mode.md

@@ -48,6 +48,5 @@ Currently, supports rules generation for Claude Code, GitHub Copilot, and OpenCo
 > Currently, when in the directory enabled global mode:
 >
 > - `rulesync.jsonc` only supports `global`, `features`, `delete` and `verbose`. `Features` can be set `"rules"` and `"commands"`. Other parameters are ignored.
-> - `rules/*.md` only supports single file has `root: true`, and frontmatter parameters without `root` are ignored.
+> - Tools support only a single `root: true` file in global mode as a target, e.g. you can't have 2 root files targeting Claude.
 > - Only Claude Code is supported for global mode commands.
-> - Global mode rules are supported for Claude Code, GitHub Copilot, and OpenCode.

src/features/rules/copilot-rule.test.ts

@@ -176,7 +176,10 @@ describe("CopilotRule", () => {
         relativeDirPath: ".copilot",
         relativeFilePath: "copilot-instructions.md",
       });
-      expect(paths).not.toHaveProperty("nonRoot");
+      expect(paths).toHaveProperty("nonRoot");
+      expect(paths.nonRoot).toEqual({
+        relativeDirPath: ".copilot/instructions",
+      });
     });
 
     it("should have different paths than regular getSettablePaths", () => {

src/features/rules/copilot-rule.ts

@@ -42,7 +42,11 @@ export type CopilotRuleSettablePaths = Omit<ToolRuleSettablePaths, "root"> & {
   };
 };
 
-export type CopilotRuleSettablePathsGlobal = ToolRuleSettablePathsGlobal;
+export type CopilotRuleSettablePathsGlobal = ToolRuleSettablePathsGlobal & {
+  nonRoot: {
+    relativeDirPath: string;
+  };
+};
 
 /**
  * Rule generator for GitHub Copilot
@@ -70,6 +74,9 @@ export class CopilotRule extends ToolRule {
           relativeDirPath: buildToolPath(".copilot", ".", options.excludeToolDir),
           relativeFilePath: "copilot-instructions.md",
         },
+        nonRoot: {
+          relativeDirPath: buildToolPath(".copilot", "instructions", options.excludeToolDir),
+        },
       };
     }
     return {