Add Goose skills support (.goose/skills/)

README.md

@@ -69,7 +69,7 @@ See [Quick Start guide](https://dyoshikawa.github.io/rulesync/getting-started/qu
 | Claude Code        | claudecode   | ✅ 🌏 |   ✅   |  ✅ 🌏   |  ✅ 🌏   |   ✅ 🌏   | ✅ 🌏  |  ✅   |
 | Codex CLI          | codexcli     | ✅ 🌏 |        | ✅ 🌏 🔧 |    🌏    |    ✅     | ✅ 🌏  |       |
 | Gemini CLI         | geminicli    | ✅ 🌏 |   ✅   |  ✅ 🌏   |  ✅ 🌏   |    🎮     | ✅ 🌏  |       |
-| Goose              | goose        | ✅ 🌏 |        |          |          |           |        |       |
+| Goose              | goose        | ✅ 🌏 |        |          |          |           |   ✅   |       |
 | GitHub Copilot     | copilot      | ✅ 🌏 |        |    ✅    |    ✅    |    ✅     |   ✅   |       |
 | Cursor             | cursor       |  ✅   |   ✅   |    ✅    |  ✅ 🌏   |   ✅ 🌏   | ✅ 🌏  |  ✅   |
 | Factory Droid      | factorydroid | ✅ 🌏 |        |  ✅ 🌏   |  ✅ 🌏   |   ✅ 🌏   | ✅ 🌏  |       |

docs/reference/file-formats.md

@@ -176,6 +176,22 @@ The skill can include:
 Skills are directory-based and can include additional files alongside SKILL.md.
 ```
 
+## `.goose/skills/*/SKILL.md`
+
+Example:
+
+```md
+---
+name: example-skill # must match the directory name
+description: >- # skill description
+  A sample Goose skill
+---
+
+This is the skill body content.
+
+Goose skills live under `.goose/skills/` and must use matching `name` frontmatter.
+```
+
 ## `.rulesync/mcp.json`
 
 Example:

docs/reference/supported-tools.md

@@ -10,7 +10,7 @@ Rulesync supports both **generation** and **import** for All of the major AI cod
 | Codex CLI          | codexcli     | ✅ 🌏 |        | ✅ 🌏 🔧 |    🌏    |    ✅     | ✅ 🌏  |       |
 | Gemini CLI         | geminicli    | ✅ 🌏 |   ✅   |  ✅ 🌏   |  ✅ 🌏   |    🎮     | ✅ 🌏  |       |
 | GitHub Copilot     | copilot      | ✅ 🌏 |        |    ✅    |    ✅    |    ✅     |   ✅   |       |
-| Goose              | goose        | ✅ 🌏 |        |          |          |           |        |       |
+| Goose              | goose        | ✅ 🌏 |        |          |          |           |   ✅   |       |
 | Cursor             | cursor       |  ✅   |   ✅   |    ✅    |  ✅ 🌏   |   ✅ 🌏   | ✅ 🌏  |  ✅   |
 | Factory Droid      | factorydroid | ✅ 🌏 |        |  ✅ 🌏   |  ✅ 🌏   |   ✅ 🌏   | ✅ 🌏  |       |
 | OpenCode           | opencode     | ✅ 🌏 |        |  ✅ 🔧   |  ✅ 🌏   |   ✅ 🌏   | ✅ 🌏  | ✅ 🌏 |

skills/rulesync/file-formats.md

@@ -176,6 +176,22 @@ The skill can include:
 Skills are directory-based and can include additional files alongside SKILL.md.
 ```
 
+## `.goose/skills/*/SKILL.md`
+
+Example:
+
+```md
+---
+name: example-skill # must match the directory name
+description: >- # skill description
+  A sample Goose skill
+---
+
+This is the skill body content.
+
+Goose skills live under `.goose/skills/` and must use matching `name` frontmatter.
+```
+
 ## `.rulesync/mcp.json`
 
 Example:

skills/rulesync/supported-tools.md

@@ -10,7 +10,7 @@ Rulesync supports both **generation** and **import** for All of the major AI cod
 | Codex CLI          | codexcli     | ✅ 🌏 |        | ✅ 🌏 🔧 |    🌏    |    ✅     | ✅ 🌏  |       |
 | Gemini CLI         | geminicli    | ✅ 🌏 |   ✅   |  ✅ 🌏   |  ✅ 🌏   |    🎮     | ✅ 🌏  |       |
 | GitHub Copilot     | copilot      | ✅ 🌏 |        |    ✅    |    ✅    |    ✅     |   ✅   |       |
-| Goose              | goose        | ✅ 🌏 |        |          |          |           |        |       |
+| Goose              | goose        | ✅ 🌏 |        |          |          |           |   ✅   |       |
 | Cursor             | cursor       |  ✅   |   ✅   |    ✅    |  ✅ 🌏   |   ✅ 🌏   | ✅ 🌏  |  ✅   |
 | Factory Droid      | factorydroid | ✅ 🌏 |        |  ✅ 🌏   |  ✅ 🌏   |   ✅ 🌏   | ✅ 🌏  |       |
 | OpenCode           | opencode     | ✅ 🌏 |        |  ✅ 🔧   |  ✅ 🌏   |   ✅ 🌏   | ✅ 🌏  | ✅ 🌏 |

src/features/skills/goose-skill.test.ts

@@ -0,0 +1,263 @@
+import { join } from "node:path";
+
+import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
+
+import { SKILL_FILE_NAME } from "../../constants/general.js";
+import { RULESYNC_SKILLS_RELATIVE_DIR_PATH } from "../../constants/rulesync-paths.js";
+import { setupTestDirectory } from "../../test-utils/test-directories.js";
+import { ensureDir, writeFileContent } from "../../utils/file.js";
+import { GooseSkill } from "./goose-skill.js";
+import { RulesyncSkill } from "./rulesync-skill.js";
+
+describe("GooseSkill", () => {
+  let testDir: string;
+  let cleanup: () => Promise<void>;
+
+  beforeEach(async () => {
+    const testSetup = await setupTestDirectory();
+    testDir = testSetup.testDir;
+    cleanup = testSetup.cleanup;
+    vi.spyOn(process, "cwd").mockReturnValue(testDir);
+  });
+
+  afterEach(async () => {
+    await cleanup();
+    vi.restoreAllMocks();
+  });
+
+  describe("getSettablePaths", () => {
+    it("should return .goose/skills as relativeDirPath", () => {
+      const paths = GooseSkill.getSettablePaths();
+      expect(paths.relativeDirPath).toBe(join(".goose", "skills"));
+    });
+
+    it("should throw error when global is true", () => {
+      expect(() => GooseSkill.getSettablePaths({ global: true })).toThrow(
+        "GooseSkill does not support global mode.",
+      );
+    });
+  });
+
+  describe("constructor", () => {
+    it("should create instance with valid content", () => {
+      const skill = new GooseSkill({
+        baseDir: testDir,
+        relativeDirPath: join(".goose", "skills"),
+        dirName: "test-skill",
+        frontmatter: {
+          name: "test-skill",
+          description: "Test skill description",
+        },
+        body: "This is the body of the goose skill.",
+        validate: true,
+      });
+
+      expect(skill).toBeInstanceOf(GooseSkill);
+      expect(skill.getBody()).toBe("This is the body of the goose skill.");
+      expect(skill.getFrontmatter()).toEqual({
+        name: "test-skill",
+        description: "Test skill description",
+      });
+    });
+  });
+
+  describe("fromDir", () => {
+    it("should create instance from valid skill directory", async () => {
+      const skillDir = join(testDir, ".goose", "skills", "test-skill");
+      await ensureDir(skillDir);
+      const skillContent = `---
+name: test-skill
+description: Test skill description
+---
+
+This is the body of the goose skill.`;
+      await writeFileContent(join(skillDir, SKILL_FILE_NAME), skillContent);
+
+      const skill = await GooseSkill.fromDir({
+        baseDir: testDir,
+        dirName: "test-skill",
+      });
+
+      expect(skill).toBeInstanceOf(GooseSkill);
+      expect(skill.getBody()).toBe("This is the body of the goose skill.");
+      expect(skill.getFrontmatter()).toEqual({
+        name: "test-skill",
+        description: "Test skill description",
+      });
+    });
+
+    it("should throw error when SKILL.md not found", async () => {
+      const skillDir = join(testDir, ".goose", "skills", "empty-skill");
+      await ensureDir(skillDir);
+
+      await expect(
+        GooseSkill.fromDir({
+          baseDir: testDir,
+          dirName: "empty-skill",
+        }),
+      ).rejects.toThrow(/SKILL\.md not found/);
+    });
+
+    it("should throw error when frontmatter name does not match dir", async () => {
+      const skillDir = join(testDir, ".goose", "skills", "mismatch-skill");
+      await ensureDir(skillDir);
+      const skillContent = `---
+name: Different Skill
+description: Mismatch example
+---
+
+This is the body of the goose skill.`;
+      await writeFileContent(join(skillDir, SKILL_FILE_NAME), skillContent);
+
+      await expect(
+        GooseSkill.fromDir({
+          baseDir: testDir,
+          dirName: "mismatch-skill",
+        }),
+      ).rejects.toThrow(/name must match directory name/);
+    });
+  });
+
+  describe("fromRulesyncSkill", () => {
+    it("should create instance from RulesyncSkill", () => {
+      const rulesyncSkill = new RulesyncSkill({
+        baseDir: testDir,
+        relativeDirPath: RULESYNC_SKILLS_RELATIVE_DIR_PATH,
+        dirName: "test-skill",
+        frontmatter: {
+          name: "Different Name",
+          description: "Test skill description",
+        },
+        body: "Test body content",
+        validate: true,
+      });
+
+      const gooseSkill = GooseSkill.fromRulesyncSkill({
+        rulesyncSkill,
+        validate: true,
+      });
+
+      expect(gooseSkill).toBeInstanceOf(GooseSkill);
+      expect(gooseSkill.getBody()).toBe("Test body content");
+      expect(gooseSkill.getFrontmatter()).toEqual({
+        name: "test-skill",
+        description: "Test skill description",
+      });
+    });
+  });
+
+  describe("isTargetedByRulesyncSkill", () => {
+    it("should return true when targets includes '*'", () => {
+      const rulesyncSkill = new RulesyncSkill({
+        baseDir: testDir,
+        relativeDirPath: RULESYNC_SKILLS_RELATIVE_DIR_PATH,
+        dirName: "all-targets-skill",
+        frontmatter: {
+          name: "All Targets Skill",
+          description: "Skill for all targets",
+          targets: ["*"],
+        },
+        body: "Test body",
+        validate: true,
+      });
+
+      expect(GooseSkill.isTargetedByRulesyncSkill(rulesyncSkill)).toBe(true);
+    });
+
+    it("should return true when targets includes 'goose'", () => {
+      const rulesyncSkill = new RulesyncSkill({
+        baseDir: testDir,
+        relativeDirPath: RULESYNC_SKILLS_RELATIVE_DIR_PATH,
+        dirName: "goose-skill",
+        frontmatter: {
+          name: "Goose Skill",
+          description: "Skill for goose",
+          targets: ["copilot", "goose"],
+        },
+        body: "Test body",
+        validate: true,
+      });
+
+      expect(GooseSkill.isTargetedByRulesyncSkill(rulesyncSkill)).toBe(true);
+    });
+
+    it("should return false when targets does not include 'goose'", () => {
+      const rulesyncSkill = new RulesyncSkill({
+        baseDir: testDir,
+        relativeDirPath: RULESYNC_SKILLS_RELATIVE_DIR_PATH,
+        dirName: "claudecode-only-skill",
+        frontmatter: {
+          name: "ClaudeCode Only Skill",
+          description: "Skill for claudecode only",
+          targets: ["claudecode"],
+        },
+        body: "Test body",
+        validate: true,
+      });
+
+      expect(GooseSkill.isTargetedByRulesyncSkill(rulesyncSkill)).toBe(false);
+    });
+  });
+
+  describe("toRulesyncSkill", () => {
+    it("should convert to RulesyncSkill", () => {
+      const skill = new GooseSkill({
+        baseDir: testDir,
+        relativeDirPath: join(".goose", "skills"),
+        dirName: "test-skill",
+        frontmatter: {
+          name: "test-skill",
+          description: "Test description",
+        },
+        body: "Test body",
+        validate: true,
+      });
+
+      const rulesyncSkill = skill.toRulesyncSkill();
+
+      expect(rulesyncSkill).toBeInstanceOf(RulesyncSkill);
+      expect(rulesyncSkill.getFrontmatter()).toEqual({
+        name: "test-skill",
+        description: "Test description",
+        targets: ["*"],
+      });
+      expect(rulesyncSkill.getBody()).toBe("Test body");
+    });
+  });
+
+  describe("forDeletion", () => {
+    it("should create minimal instance for deletion", () => {
+      const skill = GooseSkill.forDeletion({
+        dirName: "cleanup",
+        relativeDirPath: join(".goose", "skills"),
+      });
+
+      expect(skill.getDirName()).toBe("cleanup");
+      expect(skill.getRelativeDirPath()).toBe(join(".goose", "skills"));
+      expect(skill.getGlobal()).toBe(false);
+    });
+
+    it("should use process.cwd() as default baseDir", () => {
+      const skill = GooseSkill.forDeletion({
+        dirName: "cleanup",
+        relativeDirPath: join(".goose", "skills"),
+      });
+
+      expect(skill).toBeInstanceOf(GooseSkill);
+      expect(skill.getBaseDir()).toBe(testDir);
+    });
+
+    it("should create instance with empty frontmatter for deletion", () => {
+      const skill = GooseSkill.forDeletion({
+        dirName: "to-delete",
+        relativeDirPath: join(".goose", "skills"),
+      });
+
+      expect(skill.getFrontmatter()).toEqual({
+        name: "",
+        description: "",
+      });
+      expect(skill.getBody()).toBe("");
+    });
+  });
+});