@cardor/agent-harness-kit 1.5.1 → 1.6.0

package/dist/agent-templates/reviewer.md

@@ -119,6 +119,7 @@ Then notify lead so the builder can be re-assigned.
 
 - **Run health.sh before approving.** No exceptions.
 - **Check every acceptance criterion.** Not just the obvious ones.
+- **Use `tasks.acceptance.get(taskId)` to retrieve criterion ids.** Call this before `tasks.acceptance.update()` when you do not already have criterion ids from `tasks.get`.
 - **Call `tasks.acceptance.update()` for each criterion.** Never skip this step.
 - **Never self-approve partial work.** All criteria must be met, not most.
 - **Be specific when blocking.** The builder must know exactly what to fix.

package/dist/cli.js

@@ -67,7 +67,7 @@ function mergeClaudeSettingsJson(filePath) {
   };
   writeFileSync2(filePath, JSON.stringify(merged, null, 2) + "\n", "utf8");
 }
-var MCP_PERMISSIONS = [
+var MCP_CLAUDE_PERMISSIONS_LEAD = [
   "mcp__agent-harness-kit__actions_start",
   "mcp__agent-harness-kit__actions_write",
   "mcp__agent-harness-kit__actions_complete",
@@ -76,14 +76,69 @@ var MCP_PERMISSIONS = [
   "mcp__agent-harness-kit__actions_record_tool",
   "mcp__agent-harness-kit__tasks_get",
   "mcp__agent-harness-kit__tasks_claim",
+  "mcp__agent-harness-kit__tasks_add",
   "mcp__agent-harness-kit__tasks_update",
+  "mcp__agent-harness-kit__tasks_edit",
+  "mcp__agent-harness-kit__tasks_archive",
+  "mcp__agent-harness-kit__tasks_unarchive",
+  "mcp__agent-harness-kit__tasks_acceptance_get",
+  "mcp__agent-harness-kit__docs_search"
+];
+var MCP_CLAUDE_PERMISSIONS_EXPLORER = [
+  "mcp__agent-harness-kit__actions_start",
+  "mcp__agent-harness-kit__actions_write",
+  "mcp__agent-harness-kit__actions_complete",
+  "mcp__agent-harness-kit__actions_get",
+  "mcp__agent-harness-kit__actions_record_file",
+  "mcp__agent-harness-kit__actions_record_tool",
+  "mcp__agent-harness-kit__tasks_get",
+  "mcp__agent-harness-kit__tasks_claim",
+  "mcp__agent-harness-kit__tasks_acceptance_get",
+  "mcp__agent-harness-kit__docs_search"
+];
+var MCP_CLAUDE_PERMISSIONS_BUILDER = [
+  "mcp__agent-harness-kit__actions_start",
+  "mcp__agent-harness-kit__actions_write",
+  "mcp__agent-harness-kit__actions_complete",
+  "mcp__agent-harness-kit__actions_get",
+  "mcp__agent-harness-kit__actions_record_file",
+  "mcp__agent-harness-kit__actions_record_tool",
+  "mcp__agent-harness-kit__tasks_get",
+  "mcp__agent-harness-kit__tasks_claim",
   "mcp__agent-harness-kit__tasks_add",
-  "mcp__agent-harness-kit__tasks_acceptance_update",
+  "mcp__agent-harness-kit__tasks_update",
   "mcp__agent-harness-kit__tasks_edit",
   "mcp__agent-harness-kit__tasks_archive",
   "mcp__agent-harness-kit__tasks_unarchive",
+  "mcp__agent-harness-kit__tasks_acceptance_get",
   "mcp__agent-harness-kit__docs_search"
 ];
+var MCP_CLAUDE_PERMISSIONS_REVIEWER = [
+  "mcp__agent-harness-kit__actions_start",
+  "mcp__agent-harness-kit__actions_write",
+  "mcp__agent-harness-kit__actions_complete",
+  "mcp__agent-harness-kit__actions_get",
+  "mcp__agent-harness-kit__actions_record_file",
+  "mcp__agent-harness-kit__actions_record_tool",
+  "mcp__agent-harness-kit__tasks_get",
+  "mcp__agent-harness-kit__tasks_claim",
+  "mcp__agent-harness-kit__tasks_add",
+  "mcp__agent-harness-kit__tasks_update",
+  "mcp__agent-harness-kit__tasks_edit",
+  "mcp__agent-harness-kit__tasks_archive",
+  "mcp__agent-harness-kit__tasks_unarchive",
+  "mcp__agent-harness-kit__tasks_acceptance_update",
+  "mcp__agent-harness-kit__tasks_acceptance_get",
+  "mcp__agent-harness-kit__docs_search"
+];
+var MCP_CLAUDE_PERMISSIONS = [
+  .../* @__PURE__ */ new Set([
+    ...MCP_CLAUDE_PERMISSIONS_LEAD,
+    ...MCP_CLAUDE_PERMISSIONS_EXPLORER,
+    ...MCP_CLAUDE_PERMISSIONS_BUILDER,
+    ...MCP_CLAUDE_PERMISSIONS_REVIEWER
+  ])
+];
 function mergeClaudeSettingsLocalJson(filePath) {
   mkdirSync2(dirname(filePath), { recursive: true });
   let existing = {};
@@ -96,7 +151,7 @@ function mergeClaudeSettingsLocalJson(filePath) {
   const existingPermissions = existing.permissions ?? {};
   const existingAllow = existingPermissions.allow ?? [];
   const existingServers = existing.enabledMcpjsonServers ?? [];
-  const mergedAllow = Array.from(/* @__PURE__ */ new Set([...existingAllow, ...MCP_PERMISSIONS]));
+  const mergedAllow = Array.from(/* @__PURE__ */ new Set([...existingAllow, ...MCP_CLAUDE_PERMISSIONS]));
   const mergedServers = Array.from(/* @__PURE__ */ new Set([...existingServers, "agent-harness-kit"]));
   const merged = {
     ...existing,
@@ -491,47 +546,15 @@ function agentReviewerToml(vars) {
   const { description, body } = stripFrontmatter(loadAgentTemplate("reviewer", vars));
   return toCodexToml("reviewer", description, body, "read-only");
 }
-var CLAUDE_CODE_MCP_TOOLS = {
-  lead: [
-    // 'mcp__agent-harness-kit__actions.start',
-    // 'mcp__agent-harness-kit__actions.write',
-    // 'mcp__agent-harness-kit__actions.complete',
-    // 'mcp__agent-harness-kit__actions.get',
-    // 'mcp__agent-harness-kit__actions.record_tool',
-    // 'mcp__agent-harness-kit__tasks.get',
-    // 'mcp__agent-harness-kit__tasks.claim',
-    // 'mcp__agent-harness-kit__tasks.update',
-    // 'mcp__agent-harness-kit__tasks.add',
-  ],
-  explorer: [
-    // 'mcp__agent-harness-kit__actions.start',
-    // 'mcp__agent-harness-kit__actions.write',
-    // 'mcp__agent-harness-kit__actions.complete',
-    // 'mcp__agent-harness-kit__actions.get',
-    // 'mcp__agent-harness-kit__actions.record_tool',
-    // 'mcp__agent-harness-kit__docs.search',
-  ],
-  builder: [
-    // 'mcp__agent-harness-kit__actions.start',
-    // 'mcp__agent-harness-kit__actions.write',
-    // 'mcp__agent-harness-kit__actions.complete',
-    // 'mcp__agent-harness-kit__actions.get',
-    // 'mcp__agent-harness-kit__actions.record_tool',
-    // 'mcp__agent-harness-kit__actions.record_file',
-  ],
-  reviewer: [
-    // 'mcp__agent-harness-kit__actions.start',
-    // 'mcp__agent-harness-kit__actions.write',
-    // 'mcp__agent-harness-kit__actions.complete',
-    // 'mcp__agent-harness-kit__actions.get',
-    // 'mcp__agent-harness-kit__actions.record_tool',
-    // 'mcp__agent-harness-kit__tasks.acceptance.update',
-    // 'mcp__agent-harness-kit__tasks.update',
-  ]
-};
 function translateFrontmatterForClaudeCode(md, agentName) {
-  const mcpTools = CLAUDE_CODE_MCP_TOOLS[agentName] ?? [];
-  const mcpLines = mcpTools.map((t) => `  - ${t}`).join("\n");
+  const permissionsMap = {
+    lead: MCP_CLAUDE_PERMISSIONS_LEAD,
+    explorer: MCP_CLAUDE_PERMISSIONS_EXPLORER,
+    builder: MCP_CLAUDE_PERMISSIONS_BUILDER,
+    reviewer: MCP_CLAUDE_PERMISSIONS_REVIEWER
+  };
+  const permissions = permissionsMap[agentName] ?? MCP_CLAUDE_PERMISSIONS;
+  const mcpLines = permissions.map((t) => `  - ${t}`).join("\n");
   return md.replace(/(tools:\n(?:  - (?!mcp__)[^\n]+\n)+)/, (match) => {
     const trimmed = match.trimEnd();
     return `${trimmed}
@@ -809,6 +832,30 @@ import { extname, join as join7 } from "path";
 import { serve } from "@hono/node-server";
 import { Hono } from "hono";
 import { WebSocketServer } from "ws";
+
+// src/core/port-utils.ts
+import { createServer } from "net";
+function isPortFree(port) {
+  return new Promise((resolve13) => {
+    const server = createServer();
+    server.once("error", () => resolve13(false));
+    server.once("listening", () => {
+      server.close(() => resolve13(true));
+    });
+    server.listen(port, "127.0.0.1");
+  });
+}
+async function findFreePort(start, maxAttempts = 10) {
+  for (let i = 0; i < maxAttempts; i++) {
+    const port = start + i;
+    if (await isPortFree(port)) return port;
+  }
+  throw new Error(
+    `Could not find a free port after ${maxAttempts} attempts (tried ${start}-${start + maxAttempts - 1}). Please free a port and try again.`
+  );
+}
+
+// src/core/dashboard-server.ts
 var MIME = {
   ".html": "text/html; charset=utf-8",
   ".js": "application/javascript; charset=utf-8",
@@ -829,7 +876,7 @@ function fileResponse(filePath) {
     headers: { "Content-Type": mime, "Cache-Control": "no-cache" }
   });
 }
-function startDashboardServer(db, dbPath, staticPath, port) {
+async function startDashboardServer(db, dbPath, staticPath, port) {
   const app = new Hono();
   const { tasks, actions, stats } = db;
   app.use("/api/*", async (c, next) => {
@@ -871,7 +918,10 @@ function startDashboardServer(db, dbPath, staticPath, port) {
     if (body.description !== void 0) updateParams.description = body.description;
     await db.updateTask(id, updateParams);
     if (body.acceptance !== void 0 && Array.isArray(body.acceptance)) {
-      await db.updateTaskAcceptance(id, body.acceptance.map((a) => a.trim()).filter(Boolean));
+      await db.updateTaskAcceptance(
+        id,
+        body.acceptance.map((a) => a.trim()).filter(Boolean)
+      );
     }
     const updated = await tasks.getById(id);
     const acceptance = await tasks.getAcceptance(id);
@@ -936,7 +986,11 @@ function startDashboardServer(db, dbPath, staticPath, port) {
     }
     return fileResponse(join7(staticPath, "index.html"));
   });
-  const httpServer = serve({ fetch: app.fetch, port });
+  const resolvedPort = await findFreePort(port);
+  if (resolvedPort !== port) {
+    console.log(`Port ${port} in use, using ${resolvedPort}`);
+  }
+  const httpServer = serve({ fetch: app.fetch, port: resolvedPort });
   const wss = new WebSocketServer({ noServer: true });
   httpServer.on("upgrade", (req, socket, head) => {
     if (req.url === "/ws") {
@@ -965,7 +1019,7 @@ function startDashboardServer(db, dbPath, staticPath, port) {
     watcher = watch2(watchTarget, broadcast);
   }
   return {
-    url: `http://localhost:${port}`,
+    url: `http://localhost:${resolvedPort}`,
     close: () => {
       clearTimeout(debounce);
       watcher?.close();
@@ -1188,8 +1242,8 @@ var TaskRepository = class {
   async add(params) {
     const now = (/* @__PURE__ */ new Date()).toISOString();
     return this.driver.insert(
-      `INSERT INTO tasks (slug, title, description, status, created_at) VALUES (?, ?, ?, ?, ?)`,
-      [params.slug, params.title, params.description ?? null, params.status ?? "pending", now]
+      `INSERT INTO tasks (slug, title, description, status, created_at, updated_at) VALUES (?, ?, ?, ?, ?, ?)`,
+      [params.slug, params.title, params.description ?? null, params.status ?? "pending", now, now]
     );
   }
   async addAcceptance(taskId, criteria) {
@@ -1214,7 +1268,7 @@ var TaskRepository = class {
     if (conditions.length > 0) {
       sql += ` WHERE ${conditions.join(" AND ")}`;
     }
-    sql += ` ORDER BY id`;
+    sql += ` ORDER BY CASE status WHEN 'pending' THEN 1 WHEN 'in_progress' THEN 2 WHEN 'blocked' THEN 3 WHEN 'done' THEN 4 ELSE 5 END, updated_at DESC`;
     return this.driver.query(sql, params);
   }
   async getAllWithAcceptanceCounts(includeArchived = false) {
@@ -1228,7 +1282,7 @@ var TaskRepository = class {
     if (!includeArchived) {
       sql += ` WHERE t.archived_at IS NULL`;
     }
-    sql += ` GROUP BY t.id ORDER BY t.id`;
+    sql += ` GROUP BY t.id ORDER BY CASE t.status WHEN 'pending' THEN 1 WHEN 'in_progress' THEN 2 WHEN 'blocked' THEN 3 WHEN 'done' THEN 4 ELSE 5 END, t.updated_at DESC`;
     return this.driver.query(sql);
   }
   async getById(id) {
@@ -1244,23 +1298,25 @@ var TaskRepository = class {
     );
   }
   async setStatus(id, status, extra) {
+    const now = (/* @__PURE__ */ new Date()).toISOString();
     if (extra?.started_at) {
       await this.driver.exec(
-        `UPDATE tasks SET status = ?, started_at = ? WHERE id = ?`,
-        [status, extra.started_at, id]
+        `UPDATE tasks SET status = ?, started_at = ?, updated_at = ? WHERE id = ?`,
+        [status, extra.started_at, now, id]
       );
     } else if (extra?.completed_at) {
       await this.driver.exec(
-        `UPDATE tasks SET status = ?, completed_at = ? WHERE id = ?`,
-        [status, extra.completed_at, id]
+        `UPDATE tasks SET status = ?, completed_at = ?, updated_at = ? WHERE id = ?`,
+        [status, extra.completed_at, now, id]
       );
     } else {
-      await this.driver.exec(`UPDATE tasks SET status = ? WHERE id = ?`, [status, id]);
+      await this.driver.exec(`UPDATE tasks SET status = ?, updated_at = ? WHERE id = ?`, [status, now, id]);
     }
   }
   async update(id, params) {
     const sets = [];
     const vals = [];
+    const now = (/* @__PURE__ */ new Date()).toISOString();
     if (params.title !== void 0) {
       sets.push("title = ?");
       vals.push(params.title);
@@ -1274,6 +1330,8 @@ var TaskRepository = class {
       vals.push(params.slug);
     }
     if (sets.length === 0) return;
+    sets.push("updated_at = ?");
+    vals.push(now);
     vals.push(id);
     await this.driver.exec(`UPDATE tasks SET ${sets.join(", ")} WHERE id = ?`, vals);
   }
@@ -1288,10 +1346,11 @@ var TaskRepository = class {
   }
   async archive(id) {
     const now = (/* @__PURE__ */ new Date()).toISOString();
-    await this.driver.exec(`UPDATE tasks SET archived_at = ? WHERE id = ?`, [now, id]);
+    await this.driver.exec(`UPDATE tasks SET archived_at = ?, updated_at = ? WHERE id = ?`, [now, now, id]);
   }
   async unarchive(id) {
-    await this.driver.exec(`UPDATE tasks SET archived_at = NULL WHERE id = ?`, [id]);
+    const now = (/* @__PURE__ */ new Date()).toISOString();
+    await this.driver.exec(`UPDATE tasks SET archived_at = NULL, updated_at = ? WHERE id = ?`, [now, id]);
   }
   async getArchived() {
     return this.driver.query(
@@ -1300,8 +1359,8 @@ var TaskRepository = class {
   }
   async claim(id, agent, now) {
     return this.driver.exec(
-      `UPDATE tasks SET status = 'in_progress', assigned_to = ?, started_at = ? WHERE id = ? AND status = 'pending'`,
-      [agent, now, id]
+      `UPDATE tasks SET status = 'in_progress', assigned_to = ?, started_at = ?, updated_at = ? WHERE id = ? AND status = 'pending'`,
+      [agent, now, now, id]
     );
   }
   async markAcceptanceMet(criterionId) {
@@ -1577,13 +1636,13 @@ async function openDB(config, cwd2) {
   const dbConfig = config.database;
   let driver;
   if (dbConfig.type === "postgres") {
-    const { PostgresDriver } = await import("./postgres-6BXN7ZH4.js");
+    const { PostgresDriver } = await import("./postgres-IOQE32DM.js");
     driver = new PostgresDriver(dbConfig);
   } else if (dbConfig.type === "mysql") {
-    const { MySQLDriver } = await import("./mysql-NXLYFD2H.js");
+    const { MySQLDriver } = await import("./mysql-THKQOXIS.js");
     driver = new MySQLDriver(dbConfig);
   } else {
-    const { SQLiteDriver } = await import("./sqlite-M65L55DA.js");
+    const { SQLiteDriver } = await import("./sqlite-XBEJJ5T2.js");
     if (dbConfig.type !== "sqlite") {
       throw new Error("Invalid database type");
     }
@@ -1600,7 +1659,7 @@ async function runDashboard(cwd2, opts) {
   const db = await openDB(config, cwd2);
   const dbPath = config.database.type === "sqlite" ? resolve7(cwd2, config.database.path) : null;
   const staticPath = join9(__dirname2, "dashboard-dist");
-  const { url } = startDashboardServer(db, dbPath, staticPath, opts.port);
+  const { url } = await startDashboardServer(db, dbPath, staticPath, opts.port);
   console.log(pc2.green(`\u2713`) + ` Dashboard running at ${pc2.bold(pc2.cyan(url))}`);
   console.log(pc2.dim(`  WebSocket live updates enabled`));
   console.log(pc2.dim(`  Press Ctrl+C to stop`));
@@ -2414,6 +2473,17 @@ var TOOLS = [
       required: ["criterionId"]
     }
   },
+  {
+    name: "tasks.acceptance.get",
+    description: "Given a taskId, returns all acceptance criteria for that task with their id, task_id, criterion text, and met status. Use the returned id values to call tasks.acceptance_update(criterionId).",
+    inputSchema: {
+      type: "object",
+      properties: {
+        taskId: { type: "number", description: "Task ID" }
+      },
+      required: ["taskId"]
+    }
+  },
   {
     name: "tasks.add",
     description: "Create a new task in the harness. Use this when the user describes work in natural language. Infer slug, title, description, and acceptance criteria from the conversation. Ask for missing critical info before calling.",
@@ -2590,6 +2660,11 @@ async function dispatch(name, args, db, docsPath) {
       await db.markAcceptanceMet(criterionId);
       return ok(JSON.stringify({ criterionId, met: true }));
     }
+    case "tasks.acceptance.get": {
+      const taskId = num(args, "taskId");
+      const criteria = await db.getTaskAcceptance(taskId);
+      return ok(JSON.stringify(criteria, null, 2));
+    }
     case "actions.record_tool": {
       const actionId = str(args, "actionId");
       const toolName = str(args, "toolName");