@cardor/agent-harness-kit 1.7.2 → 1.7.5

package/README.md

@@ -174,7 +174,7 @@ Regenerates `AGENTS.md` and provider-specific files from your `agent-harness-kit
 ```bash
 ahk build
 ahk build --watch    # watch mode: rebuilds automatically on config changes
-ahk build --sync     # sync tools: frontmatter in .claude/agents/*.md to match current permission constants
+ahk build --sync     # sync tools: frontmatter in agent files to match current permission constants (claude-code only; no-op for opencode/codex-cli)
 ```
 
 ---

package/dist/cli.js

@@ -12,7 +12,7 @@ import * as p from "@clack/prompts";
 import pc from "picocolors";
 
 // src/core/materializer/claude-code.ts
-import { existsSync as existsSync3, mkdirSync as mkdirSync4, writeFileSync as writeFileSync4 } from "fs";
+import { existsSync as existsSync3, mkdirSync as mkdirSync4, readFileSync as readFileSync4, writeFileSync as writeFileSync4 } from "fs";
 import { join as join4, resolve as resolve3 } from "path";
 
 // src/utils/file.ts
@@ -672,38 +672,6 @@ function appendGitignore(cwd2) {
 function slugify(title) {
   return title.toLowerCase().replace(/[^a-z0-9]+/g, "-").replace(/^-+|-+$/g, "").slice(0, 64);
 }
-var AGENT_TOOLS = {
-  lead: [...MCP_CLAUDE_PERMISSIONS_LEAD],
-  explorer: [...MCP_CLAUDE_PERMISSIONS_EXPLORER],
-  consultant: [...MCP_CLAUDE_PERMISSIONS_CONSULTANT],
-  builder: [...MCP_CLAUDE_PERMISSIONS_BUILDER],
-  reviewer: [...MCP_CLAUDE_PERMISSIONS_REVIEWER]
-};
-async function syncAgentPermissions(cwd2) {
-  for (const [agent, tools] of Object.entries(AGENT_TOOLS)) {
-    const filePath = join3(cwd2, ".claude", "agents", `${agent}.md`);
-    if (!existsSync2(filePath)) {
-      console.log(`  ${agent}.md not found \u2014 skipping`);
-      continue;
-    }
-    const content = readFileSync3(filePath, "utf-8");
-    const updated = content.replace(
-      /(tools:\n)((?:  - [^\n]+\n)*)/m,
-      (_match, header, toolsSection) => {
-        const nativeLines = toolsSection.split("\n").filter((line) => line.trim() && !line.includes("mcp__"));
-        const nativeSection = nativeLines.length ? nativeLines.join("\n") + "\n" : "";
-        const mcpSection = tools.map((t) => `  - ${t}`).join("\n") + "\n";
-        return header + nativeSection + mcpSection;
-      }
-    );
-    if (updated === content) {
-      console.log(`  ${agent}.md already in sync`);
-    } else {
-      writeFileSync3(filePath, updated, "utf-8");
-      console.log(`  ${agent}.md updated`);
-    }
-  }
-}
 
 // src/core/materializer/claude-code.ts
 var ClaudeCodeMaterializer = class {
@@ -765,6 +733,38 @@ No tasks in progress.
   async migrate(config, _to, _cwd) {
     void config;
   }
+  async syncPermissions(cwd2) {
+    const AGENT_TOOLS = {
+      lead: [...MCP_CLAUDE_PERMISSIONS_LEAD],
+      explorer: [...MCP_CLAUDE_PERMISSIONS_EXPLORER],
+      consultant: [...MCP_CLAUDE_PERMISSIONS_CONSULTANT],
+      builder: [...MCP_CLAUDE_PERMISSIONS_BUILDER],
+      reviewer: [...MCP_CLAUDE_PERMISSIONS_REVIEWER]
+    };
+    for (const [agent, tools] of Object.entries(AGENT_TOOLS)) {
+      const filePath = join4(cwd2, ".claude", "agents", `${agent}.md`);
+      if (!existsSync3(filePath)) {
+        console.log(`  ${agent}.md not found \u2014 skipping`);
+        continue;
+      }
+      const content = readFileSync4(filePath, "utf-8");
+      const updated = content.replace(
+        /(tools:\n)((?:  - [^\n]+\n)*)/m,
+        (_match, header, toolsSection) => {
+          const nativeLines = toolsSection.split("\n").filter((line) => line.trim() && !line.includes("mcp__"));
+          const nativeSection = nativeLines.length ? nativeLines.join("\n") + "\n" : "";
+          const mcpSection = tools.map((t) => `  - ${t}`).join("\n") + "\n";
+          return header + nativeSection + mcpSection;
+        }
+      );
+      if (updated === content) {
+        console.log(`  ${agent}.md already in sync`);
+      } else {
+        writeFileSync4(filePath, updated, "utf-8");
+        console.log(`  ${agent}.md updated`);
+      }
+    }
+  }
 };
 
 // src/core/materializer/codex-cli.ts
@@ -829,6 +829,9 @@ No tasks in progress.
   async migrate(config, _to, _cwd) {
     void config;
   }
+  async syncPermissions(_cwd) {
+    console.log("  Permissions sync not needed for codex-cli \u2014 skipping");
+  }
 };
 
 // src/core/materializer/opencode.ts
@@ -891,6 +894,9 @@ No tasks in progress.
   async migrate(config, _to, _cwd) {
     void config;
   }
+  async syncPermissions(_cwd) {
+    console.log("  Permissions sync not needed for opencode \u2014 skipping");
+  }
 };
 
 // src/core/materializer/index.ts
@@ -911,7 +917,10 @@ function getMaterializer(provider) {
 async function runBuild(cwd2, opts) {
   await buildOnce(cwd2);
   if (opts.sync) {
-    await syncAgentPermissions(cwd2);
+    p.log.step("Syncing agent permissions...");
+    const config = await loadConfig(cwd2);
+    const materializer = getMaterializer(config.provider);
+    await materializer.syncPermissions(cwd2);
   }
   if (opts.watch) {
     p.log.info(`Watching agent-harness-kit.config.ts for changes...`);
@@ -951,7 +960,7 @@ import pc2 from "picocolors";
 
 // src/core/dashboard-server.ts
 import { watch as watch2 } from "fs";
-import { existsSync as existsSync6, readFileSync as readFileSync4 } from "fs";
+import { existsSync as existsSync6, readFileSync as readFileSync5 } from "fs";
 import { extname, join as join7 } from "path";
 import { serve } from "@hono/node-server";
 import { Hono } from "hono";
@@ -994,7 +1003,7 @@ var MIME = {
   ".ttf": "font/ttf"
 };
 function fileResponse(filePath) {
-  const content = readFileSync4(filePath);
+  const content = readFileSync5(filePath);
   const mime = MIME[extname(filePath)] ?? "application/octet-stream";
   return new Response(content, {
     headers: { "Content-Type": mime, "Cache-Control": "no-cache" }
@@ -1978,14 +1987,14 @@ var cliFormWithRetry = async (formFn, schema) => {
 };
 
 // src/commands/init-helpers.ts
-import { existsSync as existsSync8, readFileSync as readFileSync5 } from "fs";
+import { existsSync as existsSync8, readFileSync as readFileSync6 } from "fs";
 import { join as join11 } from "path";
 import pc5 from "picocolors";
 function readProjectNameFromPackageJson(cwd2) {
   try {
     const pkgPath2 = join11(cwd2, "package.json");
     if (!existsSync8(pkgPath2)) return null;
-    const content = readFileSync5(pkgPath2, "utf8");
+    const content = readFileSync6(pkgPath2, "utf8");
     const pkg2 = JSON.parse(content);
     const name = pkg2?.name;
     if (typeof name === "string" && name.trim()) return name.trim();
@@ -1999,7 +2008,7 @@ function detectConfigExtension(cwd2) {
     if (existsSync8(join11(cwd2, "tsconfig.json"))) return "ts";
     const pkgPath2 = join11(cwd2, "package.json");
     if (!existsSync8(pkgPath2)) return "mjs";
-    const pkg2 = JSON.parse(readFileSync5(pkgPath2, "utf8"));
+    const pkg2 = JSON.parse(readFileSync6(pkgPath2, "utf8"));
     if (pkg2?.type === "module") return "mjs";
   } catch {
   }
@@ -2459,7 +2468,7 @@ async function runReset(cwd2, opts) {
 }
 
 // src/core/mcp-server.ts
-import { existsSync as existsSync11, mkdirSync as mkdirSync9, readdirSync as readdirSync2, readFileSync as readFileSync7, statSync, writeFileSync as writeFileSync10 } from "fs";
+import { existsSync as existsSync11, mkdirSync as mkdirSync9, readdirSync as readdirSync2, readFileSync as readFileSync8, statSync, writeFileSync as writeFileSync10 } from "fs";
 import { join as join15, resolve as resolve10 } from "path";
 import { Server } from "@modelcontextprotocol/sdk/server";
 import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js";
@@ -2469,7 +2478,7 @@ import {
 } from "@modelcontextprotocol/sdk/types.js";
 
 // src/core/permissions-check.ts
-import { existsSync as existsSync10, readFileSync as readFileSync6 } from "fs";
+import { existsSync as existsSync10, readFileSync as readFileSync7 } from "fs";
 import { join as join14 } from "path";
 var CANONICAL = {
   lead: [...MCP_CLAUDE_PERMISSIONS_LEAD],
@@ -2486,7 +2495,10 @@ function parseToolsFromFrontmatter(content) {
   if (!toolsMatch) return [];
   return toolsMatch[1].split("\n").map((l) => l.trim().replace(/^- /, "")).filter((l) => l.startsWith("mcp__"));
 }
-function checkPermissionsSync(cwd2) {
+function checkPermissionsSync(cwd2, config) {
+  if (config.provider !== "claude-code") {
+    return { in_sync: true };
+  }
   const agents = {};
   let in_sync = true;
   for (const agent of ["lead", "explorer", "consultant", "builder", "reviewer"]) {
@@ -2497,7 +2509,7 @@ function checkPermissionsSync(cwd2) {
       in_sync = false;
       continue;
     }
-    const content = readFileSync6(filePath, "utf-8");
+    const content = readFileSync7(filePath, "utf-8");
     const installed = parseToolsFromFrontmatter(content);
     const canonical = CANONICAL[agent];
     const missing = canonical.filter((t) => !installed.includes(t));
@@ -2646,7 +2658,10 @@ var TOOLS = [
     inputSchema: {
       type: "object",
       properties: {
-        criterionId: { type: "number", description: "The id of the acceptance criterion to mark as met" }
+        criterionId: {
+          type: "number",
+          description: "The id of the acceptance criterion to mark as met"
+        }
       },
       required: ["criterionId"]
     }
@@ -2669,7 +2684,10 @@ var TOOLS = [
       type: "object",
       properties: {
         title: { type: "string", description: "Short human-readable title for the task" },
-        slug: { type: "string", description: "URL-safe identifier (lowercase, hyphens). Auto-derived from title if omitted." },
+        slug: {
+          type: "string",
+          description: "URL-safe identifier (lowercase, hyphens). Auto-derived from title if omitted."
+        },
         description: { type: "string", description: "Longer description of the task goal" },
         acceptance: {
           type: "array",
@@ -2687,8 +2705,14 @@ var TOOLS = [
       type: "object",
       properties: {
         actionId: { type: "string", description: "UUID returned by actions.start" },
-        toolName: { type: "string", description: "Name of the tool that was called (e.g. Read, Bash, Edit)" },
-        argsJson: { type: "string", description: "Optional JSON string of the arguments passed to the tool" },
+        toolName: {
+          type: "string",
+          description: "Name of the tool that was called (e.g. Read, Bash, Edit)"
+        },
+        argsJson: {
+          type: "string",
+          description: "Optional JSON string of the arguments passed to the tool"
+        },
         resultSummary: { type: "string", description: "Optional short summary of the tool result" }
       },
       required: ["actionId", "toolName"]
@@ -2762,7 +2786,7 @@ async function startMcpServer(config, cwd2) {
     const { name, arguments: args } = request.params;
     const a = args ?? {};
     try {
-      const result = await dispatch(name, a, db, docsPath, cwd2);
+      const result = await dispatch(name, a, db, docsPath, cwd2, config);
       return result;
     } catch (err) {
       return ok(`Error: ${err instanceof Error ? err.message : String(err)}`, true);
@@ -2771,7 +2795,7 @@ async function startMcpServer(config, cwd2) {
   const transport = new StdioServerTransport();
   await server.connect(transport);
 }
-async function dispatch(name, args, db, docsPath, cwd2) {
+async function dispatch(name, args, db, docsPath, cwd2, config) {
   switch (name) {
     case "actions.start": {
       const taskId = num(args, "taskId");
@@ -2790,7 +2814,9 @@ async function dispatch(name, args, db, docsPath, cwd2) {
       const actionId = str(args, "actionId");
       const summary = str(args, "summary");
       const action = await db.completeAction(actionId, summary);
-      return ok(JSON.stringify({ actionId, status: action.status, completedAt: action.completed_at }));
+      return ok(
+        JSON.stringify({ actionId, status: action.status, completedAt: action.completed_at })
+      );
     }
     case "actions.get": {
       const taskId = num(args, "taskId");
@@ -2873,7 +2899,10 @@ async function dispatch(name, args, db, docsPath, cwd2) {
       const acceptance = args["acceptance"];
       const task2 = await db.getTaskById(id);
       if (!task2) return ok(JSON.stringify({ error: "Task not found", taskId: id }), true);
-      await db.updateTask(id, { title, description: description !== void 0 ? description : void 0 });
+      await db.updateTask(id, {
+        title,
+        description: description !== void 0 ? description : void 0
+      });
       if (acceptance !== void 0 && acceptance !== null) {
         await db.updateTaskAcceptance(id, acceptance);
       }
@@ -2891,7 +2920,7 @@ async function dispatch(name, args, db, docsPath, cwd2) {
       return ok(JSON.stringify(task2));
     }
     case "permissions.check": {
-      const result = checkPermissionsSync(cwd2);
+      const result = checkPermissionsSync(cwd2, config);
       return ok(JSON.stringify(result, null, 2));
     }
     case "deps.snapshot": {
@@ -2899,7 +2928,7 @@ async function dispatch(name, args, db, docsPath, cwd2) {
       if (!existsSync11(pkgPath2)) {
         return ok("package.json not found in project root", true);
       }
-      const pkg2 = JSON.parse(readFileSync7(pkgPath2, "utf8"));
+      const pkg2 = JSON.parse(readFileSync8(pkgPath2, "utf8"));
       const snapshot = {
         capturedAt: (/* @__PURE__ */ new Date()).toISOString(),
         dependencies: pkg2.dependencies ?? {},
@@ -2908,7 +2937,12 @@ async function dispatch(name, args, db, docsPath, cwd2) {
       const harnessDir = join15(cwd2, ".harness");
       mkdirSync9(harnessDir, { recursive: true });
       writeFileSync10(join15(harnessDir, "deps-lock.json"), JSON.stringify(snapshot, null, 2), "utf8");
-      return ok(JSON.stringify({ message: "Snapshot saved to .harness/deps-lock.json", capturedAt: snapshot.capturedAt }));
+      return ok(
+        JSON.stringify({
+          message: "Snapshot saved to .harness/deps-lock.json",
+          capturedAt: snapshot.capturedAt
+        })
+      );
     }
     case "deps.check": {
       const pkgPath2 = join15(cwd2, "package.json");
@@ -2917,10 +2951,15 @@ async function dispatch(name, args, db, docsPath, cwd2) {
         return ok("package.json not found in project root", true);
       }
       if (!existsSync11(lockPath)) {
-        return ok(JSON.stringify({ status: "no-snapshot", message: "No deps-lock.json found. Run deps.snapshot first to establish a baseline." }));
+        return ok(
+          JSON.stringify({
+            status: "no-snapshot",
+            message: "No deps-lock.json found. Run deps.snapshot first to establish a baseline."
+          })
+        );
       }
-      const pkg2 = JSON.parse(readFileSync7(pkgPath2, "utf8"));
-      const lock = JSON.parse(readFileSync7(lockPath, "utf8"));
+      const pkg2 = JSON.parse(readFileSync8(pkgPath2, "utf8"));
+      const lock = JSON.parse(readFileSync8(lockPath, "utf8"));
       const current = { ...pkg2.dependencies ?? {}, ...pkg2.devDependencies ?? {} };
       const previous = { ...lock.dependencies ?? {}, ...lock.devDependencies ?? {} };
       const added = [];
@@ -2930,7 +2969,10 @@ async function dispatch(name, args, db, docsPath, cwd2) {
         if (!(name2 in previous)) {
           added.push(`${name2}@${version}`);
         } else {
-          const prevMajor = parseInt(previous[name2].replace(/^[\^~>=v]/, "").split(".")[0] ?? "0", 10);
+          const prevMajor = parseInt(
+            previous[name2].replace(/^[\^~>=v]/, "").split(".")[0] ?? "0",
+            10
+          );
           const curMajor = parseInt(version.replace(/^[\^~>=v]/, "").split(".")[0] ?? "0", 10);
           if (!isNaN(prevMajor) && !isNaN(curMajor) && curMajor > prevMajor) {
             majorBumps.push({ name: name2, from: previous[name2], to: version });
@@ -2944,7 +2986,16 @@ async function dispatch(name, args, db, docsPath, cwd2) {
       }
       const significant = added.length > 0 || removed.length > 0 || majorBumps.length > 0;
       const advisory = significant ? "Significant dependency changes detected. Consider running `pnpx autoskills` (or `npx autoskills` if pnpm is unavailable) to refresh agent skills. Clearing stale skills before re-running is recommended." : "No significant dependency changes detected.";
-      return ok(JSON.stringify({ significant, added, removed, majorBumps, advisory, snapshotDate: lock.capturedAt }));
+      return ok(
+        JSON.stringify({
+          significant,
+          added,
+          removed,
+          majorBumps,
+          advisory,
+          snapshotDate: lock.capturedAt
+        })
+      );
     }
     default:
       return ok(`Unknown tool: ${name}`, true);
@@ -2958,12 +3009,16 @@ function searchDocs(docsPath, query, maxResults = 10) {
     for (const file of files) {
       if (results.length >= maxResults) break;
       try {
-        const content = readFileSync7(file, "utf8");
+        const content = readFileSync8(file, "utf8");
         const lines = content.split("\n");
         for (let i = 0; i < lines.length; i++) {
           const lower = lines[i].toLowerCase();
           if (terms.every((t) => lower.includes(t))) {
-            results.push({ file: file.replace(docsPath + "/", ""), line: i + 1, text: lines[i].trim() });
+            results.push({
+              file: file.replace(docsPath + "/", ""),
+              line: i + 1,
+              text: lines[i].trim()
+            });
             if (results.length >= maxResults) break;
           }
         }
@@ -3013,8 +3068,8 @@ async function runServe(cwd2, opts) {
   }
   process.stderr.write(`[agent-harness-kit] MCP server starting (stdio)
 `);
-  const syncResult = checkPermissionsSync(cwd2);
-  if (!syncResult.in_sync) {
+  const syncResult = checkPermissionsSync(cwd2, config);
+  if (!syncResult.in_sync && syncResult.agents) {
     const affected = Object.entries(syncResult.agents).filter(([, r]) => !r.ok).map(([name, r]) => {
       const parts = [];
       if (r.missing.length) parts.push(`missing: ${r.missing.map((t) => t.replace("mcp__agent-harness-kit__", "")).join(", ")}`);
@@ -3103,7 +3158,7 @@ async function runStatus(cwd2, opts) {
 }
 
 // src/commands/sync.ts
-import { existsSync as existsSync12, readFileSync as readFileSync8 } from "fs";
+import { existsSync as existsSync12, readFileSync as readFileSync9 } from "fs";
 import { join as join16, resolve as resolve11 } from "path";
 import pc10 from "picocolors";
 async function runSync(cwd2, opts) {
@@ -3129,7 +3184,7 @@ async function syncIn(featureListPath, db, dryRun) {
   }
   let seeds;
   try {
-    seeds = JSON.parse(readFileSync8(featureListPath, "utf8"));
+    seeds = JSON.parse(readFileSync9(featureListPath, "utf8"));
   } catch (err) {
     console.error(pc10.red(`Failed to parse feature_list.json: ${err}`));
     process.exit(1);