@cardor/agent-harness-kit 1.6.5 → 1.6.7

package/README.md

@@ -107,7 +107,7 @@ Everything is stored locally in a SQLite database (`.harness/harness.db`). No cl
 ## Features
 
 - **Provider-agnostic** — works with Claude Code, OpenCode, Codex CLI, or any MCP-compatible AI tool. Switch providers without losing your task history or reconfiguring your workflow.
-- **Structured 4-agent workflow** — Lead, Explorer, Builder, and Reviewer each have defined responsibilities and can only act within their role.
+- **Structured 5-agent workflow** — Lead, Explorer, Consultant, Builder, and Reviewer each have defined responsibilities and can only act within their role.
 - **Atomic task claiming** — agents use `tasks.claim()` which uses a SQLite transaction to prevent two agents from picking up the same task at the same time.
 - **Full audit trail** — every action, file touched, tool used, and section written is stored in SQLite and queryable.
 - **Health gate** — agents must run `health.sh` and get a green exit before starting or closing any task. You define what "healthy" means.

package/dist/cli.js

@@ -334,7 +334,7 @@ docs.search          query                                  \u2192 search ${docs
    - tasks.get('in_progress') \u2192 resume if something is in progress
    - tasks.get('pending') \u2192 pick lowest id
 
-2. WORK  (lead \u2192 explorer \u2192 builder \u2192 reviewer)
+2. WORK  (lead \u2192 explorer \u2192 consultant \u2192 builder \u2192 reviewer)
    - Each agent calls actions.start(taskId, agentName) \u2192 actionId
    - After EVERY tool call: actions.record_tool(actionId, toolName, args, summary)
    - After EVERY file change: actions.record_file(actionId, filePath, operation, notes)
@@ -351,6 +351,7 @@ docs.search          query                                  \u2192 search ${docs
 |-------|---------------|
 | lead | Decomposes the task into a plan, assigns sub-agents |
 | explorer | Reads and maps relevant code, never writes |
+| consultant | Technical advisor, runs after explorer, before builder. Never writes code. |
 | builder | Implements the plan, writes files |
 | reviewer | Verifies acceptance criteria, approves or blocks |
 
@@ -418,7 +419,7 @@ docs.search          query                                  \u2192 search ${docs
    - tasks.get('pending') \u2192 pick lowest id
    - No pending tasks? \u2192 ask user, infer fields, call tasks.add, then tasks.claim
 
-2. WORK  (lead \u2192 explorer \u2192 builder \u2192 reviewer)
+2. WORK  (lead \u2192 explorer \u2192 consultant \u2192 builder \u2192 reviewer)
    - Each agent calls actions.start(taskId, agentName) \u2192 actionId
    - After EVERY tool call: actions.record_tool(actionId, toolName, args, summary)
    - After EVERY file change: actions.record_file(actionId, filePath, operation, notes)
@@ -435,6 +436,7 @@ docs.search          query                                  \u2192 search ${docs
 |-------|---------------|
 | lead | Decomposes the task into a plan, assigns sub-agents |
 | explorer | Reads and maps relevant code, never writes |
+| consultant | Technical advisor, runs after explorer, before builder. Never writes code. |
 | builder | Implements the plan, writes files |
 | reviewer | Verifies acceptance criteria, approves or blocks |
 
@@ -564,6 +566,10 @@ function agentReviewerToml(vars) {
   const { description, body } = stripFrontmatter(loadAgentTemplate("reviewer", vars));
   return toCodexToml("reviewer", description, body, "read-only");
 }
+function agentConsultantToml(vars) {
+  const { description, body } = stripFrontmatter(loadAgentTemplate("consultant", vars));
+  return toCodexToml("consultant", description, body, "read-only");
+}
 function translateFrontmatterForClaudeCode(md, agentName) {
   const permissionsMap = {
     lead: [...MCP_CLAUDE_PERMISSIONS_LEAD],
@@ -629,10 +635,15 @@ async function syncAgentPermissions(cwd2) {
       continue;
     }
     const content = readFileSync3(filePath, "utf-8");
-    const toolsBlock = `tools:
-${tools.map((t) => `  - ${t}`).join("\n")}
-`;
-    const updated = content.replace(/tools:\n(?:  - [^\n]+\n)*/m, toolsBlock);
+    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 {
@@ -738,6 +749,7 @@ No tasks in progress.
     const writablePaths = (config.agents.builder.writablePaths ?? []).join(", ");
     writeAgentFile(cwd2, ".codex/agents/lead.toml", agentLeadToml({ projectName }));
     writeAgentFile(cwd2, ".codex/agents/explorer.toml", agentExplorerToml({ projectName, allowedPaths }));
+    writeAgentFile(cwd2, ".codex/agents/consultant.toml", agentConsultantToml({ projectName }));
     writeAgentFile(cwd2, ".codex/agents/builder.toml", agentBuilderToml({ projectName, writablePaths }));
     writeAgentFile(cwd2, ".codex/agents/reviewer.toml", agentReviewerToml({ projectName }));
     writeAgentFile(cwd2, ".codex/agents/default.toml", agentLeadAsDefaultToml({ projectName }));
@@ -756,6 +768,7 @@ No tasks in progress.
     const writablePaths = (config.agents.builder.writablePaths ?? []).join(", ");
     writeAgentFile(cwd2, ".codex/agents/lead.toml", agentLeadToml({ projectName }));
     writeAgentFile(cwd2, ".codex/agents/explorer.toml", agentExplorerToml({ projectName, allowedPaths }));
+    writeAgentFile(cwd2, ".codex/agents/consultant.toml", agentConsultantToml({ projectName }));
     writeAgentFile(cwd2, ".codex/agents/builder.toml", agentBuilderToml({ projectName, writablePaths }));
     writeAgentFile(cwd2, ".codex/agents/reviewer.toml", agentReviewerToml({ projectName }));
     writeAgentFile(cwd2, ".codex/agents/default.toml", agentLeadAsDefaultToml({ projectName }));
@@ -800,6 +813,7 @@ No tasks in progress.
     const writablePaths = (config.agents.builder.writablePaths ?? []).join(", ");
     writeAgentFile(cwd2, ".opencode/agents/lead.md", translateFrontmatterForOpenCode(agentLead({ projectName })));
     writeAgentFile(cwd2, ".opencode/agents/explorer.md", translateFrontmatterForOpenCode(agentExplorer({ projectName, allowedPaths })));
+    writeAgentFile(cwd2, ".opencode/agents/consultant.md", translateFrontmatterForOpenCode(agentConsultant({ projectName })));
     writeAgentFile(cwd2, ".opencode/agents/builder.md", translateFrontmatterForOpenCode(agentBuilder({ projectName, writablePaths })));
     writeAgentFile(cwd2, ".opencode/agents/reviewer.md", translateFrontmatterForOpenCode(agentReviewer({ projectName })));
     mergeOpencodeJson(join6(cwd2, "opencode.json"), config.tools.mcp.port);
@@ -817,6 +831,7 @@ No tasks in progress.
     const writablePaths = (config.agents.builder.writablePaths ?? []).join(", ");
     writeAgentFile(cwd2, ".opencode/agents/lead.md", translateFrontmatterForOpenCode(agentLead({ projectName })));
     writeAgentFile(cwd2, ".opencode/agents/explorer.md", translateFrontmatterForOpenCode(agentExplorer({ projectName, allowedPaths })));
+    writeAgentFile(cwd2, ".opencode/agents/consultant.md", translateFrontmatterForOpenCode(agentConsultant({ projectName })));
     writeAgentFile(cwd2, ".opencode/agents/builder.md", translateFrontmatterForOpenCode(agentBuilder({ projectName, writablePaths })));
     writeAgentFile(cwd2, ".opencode/agents/reviewer.md", translateFrontmatterForOpenCode(agentReviewer({ projectName })));
     mergeOpencodeJson(join6(cwd2, "opencode.json"), config.tools.mcp.port);
@@ -1792,7 +1807,7 @@ async function runHealth(cwd2) {
   if (!dbOk) allOk = false;
   const providerFiles = getProviderHealthFiles(config.provider);
   const agentsDir = providerFiles.agentsDir;
-  const agentNames = ["lead", "explorer", "builder", "reviewer"];
+  const agentNames = ["lead", "explorer", "consultant", "builder", "reviewer"];
   const agentsLabelWidth = "[checking agents] ".length;
   for (let i = 0; i < agentNames.length; i++) {
     const name = agentNames[i];
@@ -2250,7 +2265,7 @@ import { existsSync as existsSync9, readdirSync, rmSync } from "fs";
 import { join as join13, resolve as resolve9 } from "path";
 import * as p5 from "@clack/prompts";
 import pc8 from "picocolors";
-var AGENT_MD_FILES = ["lead", "explorer", "builder", "reviewer"];
+var AGENT_MD_FILES = ["lead", "explorer", "consultant", "builder", "reviewer"];
 async function resetAgentMds(cwd2, provider) {
   const agentDir = provider === "claude-code" ? ".claude/agents" : ".opencode/agents";
   const agentDirPath = resolve9(cwd2, agentDir);