@kuznai/inception-engine 0.21.0 → 0.23.0

package/dist/test/unit/manifest.test.js

@@ -554,6 +554,78 @@ describe("loadManifest", () => {
             await rm(dir, { recursive: true });
         }
     });
+    it("agentRules accepts targetDir with scope: 'repo'", async () => {
+        const dir = await makeTmpDir();
+        try {
+            await writeFile(path.join(dir, "inception.json"), JSON.stringify({
+                skills: [],
+                agentRules: [
+                    {
+                        name: "my-rule",
+                        agents: ["claude-code"],
+                        path: "rules/CLAUDE.md",
+                        scope: "repo",
+                        targetDir: "apps/frontend",
+                    },
+                ],
+            }));
+            const manifest = await loadManifest(dir);
+            assert.equal(manifest.agentRules[0]?.targetDir, "apps/frontend");
+        }
+        finally {
+            await rm(dir, { recursive: true });
+        }
+    });
+    it("agentRules rejects targetDir with scope: 'global'", async () => {
+        const dir = await makeTmpDir();
+        try {
+            await writeFile(path.join(dir, "inception.json"), JSON.stringify({
+                skills: [],
+                agentRules: [
+                    {
+                        name: "my-rule",
+                        agents: ["claude-code"],
+                        path: "rules/CLAUDE.md",
+                        scope: "global",
+                        targetDir: "apps/frontend",
+                    },
+                ],
+            }));
+            await assert.rejects(loadManifest(dir), (err) => {
+                assert.ok(err instanceof UserError);
+                assert.match(err.message, /targetDir is only supported for scope "repo" or "workspace"/);
+                return true;
+            });
+        }
+        finally {
+            await rm(dir, { recursive: true });
+        }
+    });
+    it("throws when agentRules targetDir is absolute", async () => {
+        const dir = await makeTmpDir();
+        try {
+            await writeFile(path.join(dir, "inception.json"), JSON.stringify({
+                skills: [],
+                agentRules: [
+                    {
+                        name: "my-rule",
+                        agents: ["claude-code"],
+                        path: "rules/CLAUDE.md",
+                        scope: "repo",
+                        targetDir: "/absolute/path",
+                    },
+                ],
+            }));
+            await assert.rejects(loadManifest(dir), (err) => {
+                assert.ok(err instanceof UserError);
+                assert.match(err.message, /targetDir must be a relative path/);
+                return true;
+            });
+        }
+        finally {
+            await rm(dir, { recursive: true });
+        }
+    });
     it("defaults mcpServers and agentRules to [] when omitted", async () => {
         const dir = await makeTmpDir();
         try {

package/dist/test/unit/preflight.test.js

@@ -125,24 +125,22 @@ describe("runPreflight", () => {
         assert.equal(implementationOnlyWarning, undefined, `expected no implementation-only warning, got: ${implementationOnlyWarning?.message}`);
     });
 });
-describe("github-copilot planned surfaces", () => {
-    it("emits a planned surface info notice for devcontainer MCP when Copilot is detected and MCP is targeted", async () => {
+describe("github-copilot devcontainer support", () => {
+    it("emits no capability warning for devcontainer MCP when Copilot is detected and devcontainer scope is targeted", async () => {
         const manifestWithMcp = {
             ...emptyManifest,
             mcpServers: [
                 {
                     name: "test-mcp",
-                    scope: "repo",
+                    scope: "devcontainer",
                     agents: ["github-copilot"],
                     config: { command: "node", args: ["server.js"] },
                 },
             ],
         };
         const warnings = await runPreflight(baseOptions, manifestWithMcp, "/home/test", ["github-copilot"]);
-        const plannedWarning = warnings.find((w) => w.kind === "info" &&
-            /devcontainer\.json MCP support is planned/.test(w.message));
-        assert.ok(plannedWarning, "expected a planned surface warning for Copilot devcontainer when MCP is targeted");
-        assert.match(plannedWarning.message, /planned/);
+        const capabilityWarning = warnings.find((w) => w.kind === "info" && /mcpServers/.test(w.message));
+        assert.equal(capabilityWarning, undefined, `expected no MCP capability warning for devcontainer scope, got: ${capabilityWarning?.message}`);
     });
 });
 describe("instruction precedence warnings", () => {
@@ -281,6 +279,92 @@ describe("instruction precedence warnings", () => {
             await rm(sourceDir, { recursive: true });
         }
     });
+    it("emits Copilot precedence warning when github-copilot has both shared-via (scope: repo) and native (scope: copilot-repo) entries", async () => {
+        const sourceDir = await makeTmpDir();
+        try {
+            await writeFile(path.join(sourceDir, "shared.md"), "# Shared rules");
+            await writeFile(path.join(sourceDir, "native.md"), "# Native rules");
+            const manifest = {
+                ...emptyManifest,
+                agentRules: [
+                    {
+                        name: "shared-rules",
+                        path: "shared.md",
+                        agents: ["claude-code", "github-copilot"],
+                        scope: "repo",
+                    },
+                    {
+                        name: "native-rules",
+                        path: "native.md",
+                        agents: ["github-copilot"],
+                        scope: "copilot-repo",
+                    },
+                ],
+            };
+            const warnings = await runPreflight({ ...baseOptions, directory: sourceDir }, manifest, "/home/test", ["github-copilot"]);
+            const precedenceWarnings = warnings.filter((w) => w.kind === "precedence");
+            assert.ok(precedenceWarnings.length > 0, "expected a precedence warning");
+            assert.ok(precedenceWarnings.some((w) => w.message.includes("github-copilot") &&
+                w.message.includes("CLAUDE.md-shared") &&
+                w.message.includes("native Copilot")), `expected Copilot precedence warning, got: ${JSON.stringify(precedenceWarnings)}`);
+        }
+        finally {
+            await rm(sourceDir, { recursive: true });
+        }
+    });
+    it("emits Copilot precedence warning when github-copilot has both global (scope: global) and copilot-scoped entries", async () => {
+        const sourceDir = await makeTmpDir();
+        try {
+            await writeFile(path.join(sourceDir, "shared.md"), "# Shared rules");
+            await writeFile(path.join(sourceDir, "scoped.md"), "# Scoped rules");
+            const manifest = {
+                ...emptyManifest,
+                agentRules: [
+                    {
+                        name: "shared-rules",
+                        path: "shared.md",
+                        agents: ["claude-code", "github-copilot"],
+                        scope: "global",
+                    },
+                    {
+                        name: "typescript",
+                        path: "scoped.md",
+                        agents: ["github-copilot"],
+                        scope: "copilot-scoped",
+                    },
+                ],
+            };
+            const warnings = await runPreflight({ ...baseOptions, directory: sourceDir }, manifest, "/home/test", ["github-copilot"]);
+            const precedenceWarnings = warnings.filter((w) => w.kind === "precedence");
+            assert.ok(precedenceWarnings.some((w) => w.message.includes("github-copilot") &&
+                w.message.includes("native Copilot")), `expected Copilot precedence warning, got: ${JSON.stringify(precedenceWarnings)}`);
+        }
+        finally {
+            await rm(sourceDir, { recursive: true });
+        }
+    });
+    it("emits no Copilot precedence warning when github-copilot only has native entries", async () => {
+        const sourceDir = await makeTmpDir();
+        try {
+            await writeFile(path.join(sourceDir, "native.md"), "# Native rules");
+            const manifest = {
+                ...emptyManifest,
+                agentRules: [
+                    {
+                        name: "native-rules",
+                        path: "native.md",
+                        agents: ["github-copilot"],
+                        scope: "copilot-repo",
+                    },
+                ],
+            };
+            const warnings = await runPreflight({ ...baseOptions, directory: sourceDir }, manifest, "/home/test", ["github-copilot"]);
+            assert.equal(warnings.filter((w) => w.kind === "precedence").length, 0);
+        }
+        finally {
+            await rm(sourceDir, { recursive: true });
+        }
+    });
 });
 describe("instruction budget warnings", () => {
     it("emits budget warning for agentRules source file exceeding 50 KB", async () => {

package/dist/test/unit/revert.test.js

@@ -29,6 +29,41 @@ describe("planRevert", () => {
         const actions = planRevert(testManifest, ["codex"], "/home/test");
         assert.equal(actions.length, 0);
     });
+    it("includes hook and execution-config reverts for detected agents", () => {
+        const manifest = {
+            ...testManifest,
+            hooks: [
+                {
+                    name: "pre-exec-check",
+                    agents: ["claude-code"],
+                    config: {
+                        hooks: {
+                            PreToolUse: [
+                                {
+                                    matcher: "Bash",
+                                    hooks: [{ type: "command", command: "scripts/check.sh" }],
+                                },
+                            ],
+                        },
+                    },
+                },
+            ],
+            executionConfigs: [
+                {
+                    name: "gemini-safety",
+                    agents: ["gemini-cli"],
+                    config: { safeMode: true },
+                },
+            ],
+        };
+        const actions = planRevert(manifest, ["claude-code", "gemini-cli"], "/home/test");
+        assert.ok(actions.some((action) => action.kind === "config-patch" &&
+            action.agent === "claude-code" &&
+            action.skill === "pre-exec-check"), "expected hook revert action for Claude Code");
+        assert.ok(actions.some((action) => action.kind === "config-patch" &&
+            action.agent === "gemini-cli" &&
+            action.skill === "gemini-safety"), "expected execution config revert action for Gemini CLI");
+    });
 });
 describe("planRevertAll", () => {
     it("creates revert actions for all agents in manifest regardless of detection", () => {
@@ -46,6 +81,7 @@ describe("planRevertAll", () => {
             agentRules: [],
             permissions: [],
             agentDefinitions: [],
+            hooks: [],
         };
         const actions = planRevertAll(multiAgentManifest, "/home/test");
         assert.equal(actions.length, 3);
@@ -75,6 +111,7 @@ describe("planRevertAll", () => {
             agentRules: [],
             permissions: [],
             agentDefinitions: [],
+            hooks: [],
         };
         const actions = planRevertAll(manifest, "/home/test");
         assert.equal(actions.length, 5);
@@ -85,6 +122,36 @@ describe("planRevertAll", () => {
         assert.match(targets.get("antigravity") ?? "", /\.gemini[\\/]antigravity[\\/]skills[\\/]test-skill$/);
         assert.match(targets.get("opencode") ?? "", /opencode[\\/]skills[\\/]test-skill$/);
     });
+    it("includes hook and execution-config reverts for all manifest agents", () => {
+        const manifest = {
+            ...testManifest,
+            hooks: [
+                {
+                    name: "pre-exec-check",
+                    agents: ["claude-code"],
+                    config: {
+                        hooks: {
+                            Stop: [{ hooks: [{ type: "command", command: "notify.sh" }] }],
+                        },
+                    },
+                },
+            ],
+            executionConfigs: [
+                {
+                    name: "gemini-safety",
+                    agents: ["gemini-cli"],
+                    config: { safeMode: true },
+                },
+            ],
+        };
+        const actions = planRevertAll(manifest, "/home/test");
+        assert.ok(actions.some((action) => action.kind === "config-patch" &&
+            action.agent === "claude-code" &&
+            action.skill === "pre-exec-check"), "expected planRevertAll to include Claude hook revert action");
+        assert.ok(actions.some((action) => action.kind === "config-patch" &&
+            action.agent === "gemini-cli" &&
+            action.skill === "gemini-safety"), "expected planRevertAll to include Gemini execution config revert action");
+    });
 });
 describe("executeRevert", { skip: process.platform === "win32" }, () => {
     it("removes a symlink registered in the deployment registry", async () => {
@@ -712,6 +779,7 @@ describe("planRevert — mcpServers and agentRules", () => {
             agentRules: [],
             permissions: [],
             agentDefinitions: [],
+            hooks: [],
         };
         const actions = planRevert(manifest, ["claude-code"], home);
         assert.equal(actions.length, 1);
@@ -735,6 +803,7 @@ describe("planRevert — mcpServers and agentRules", () => {
             agentRules: [],
             permissions: [],
             agentDefinitions: [],
+            hooks: [],
         };
         const actions = planRevert(manifest, ["codex"], home);
         assert.equal(actions.length, 0);
@@ -755,6 +824,7 @@ describe("planRevert — mcpServers and agentRules", () => {
             ],
             permissions: [],
             agentDefinitions: [],
+            hooks: [],
         };
         const actions = planRevert(manifest, ["claude-code"], home);
         assert.equal(actions.length, 1);
@@ -778,6 +848,7 @@ describe("planRevert — mcpServers and agentRules", () => {
             ],
             permissions: [],
             agentDefinitions: [],
+            hooks: [],
         };
         const actions = planRevert(manifest, ["codex"], home);
         assert.equal(actions.length, 0);
@@ -801,6 +872,7 @@ describe("planRevertAll — mcpServers and agentRules", () => {
             agentRules: [],
             permissions: [],
             agentDefinitions: [],
+            hooks: [],
         };
         const actions = planRevertAll(manifest, home);
         assert.equal(actions.length, 2);
@@ -824,6 +896,7 @@ describe("planRevertAll — mcpServers and agentRules", () => {
             ],
             permissions: [],
             agentDefinitions: [],
+            hooks: [],
         };
         const actions = planRevertAll(manifest, home);
         assert.equal(actions.length, 2);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kuznai/inception-engine",
-  "version": "0.21.0",
+  "version": "0.23.0",
   "description": "Deploy AI agent skills from a git repo to user home directories",
   "license": "MIT",
   "author": "Damian Piątkowski",