hool-cli 0.6.2 → 0.7.1

package/dist/mcps/mcps.test.js

@@ -0,0 +1,110 @@
+import { describe, it, expect, vi } from 'vitest';
+import { MCP_REGISTRY, getRequiredMcps, getRequiredMcpNames } from './registry.js';
+import { checkAndInstallMcps } from './installer.js';
+describe('MCP_REGISTRY', () => {
+    it('contains context7, playwright, deepwiki', () => {
+        expect(MCP_REGISTRY).toHaveProperty('context7');
+        expect(MCP_REGISTRY).toHaveProperty('playwright');
+        expect(MCP_REGISTRY).toHaveProperty('deepwiki');
+    });
+    it('each entry has name, installCommand, configEntry', () => {
+        for (const [key, mcp] of Object.entries(MCP_REGISTRY)) {
+            expect(mcp.name).toBe(key);
+            expect(mcp.installCommand).toBeTruthy();
+            expect(mcp.configEntry).toHaveProperty('command');
+            expect(mcp.configEntry).toHaveProperty('args');
+        }
+    });
+});
+describe('getRequiredMcps', () => {
+    it('returns context7 + deepwiki + playwright for web-app', () => {
+        const mcps = getRequiredMcps('web-app');
+        const names = mcps.map(m => m.name);
+        expect(names).toContain('context7');
+        expect(names).toContain('deepwiki');
+        expect(names).toContain('playwright');
+    });
+    it('returns context7 + deepwiki for cli-tool (no playwright)', () => {
+        const mcps = getRequiredMcps('cli-tool');
+        const names = mcps.map(m => m.name);
+        expect(names).toContain('context7');
+        expect(names).toContain('deepwiki');
+        expect(names).not.toContain('playwright');
+    });
+    it('returns context7 + deepwiki + playwright for browser-game', () => {
+        const mcps = getRequiredMcps('browser-game');
+        const names = mcps.map(m => m.name);
+        expect(names).toContain('playwright');
+    });
+    it('returns context7 + deepwiki for api-only', () => {
+        const mcps = getRequiredMcps('api-only');
+        const names = mcps.map(m => m.name);
+        expect(names).toEqual(['context7', 'deepwiki']);
+    });
+    it('returns McpDefinition objects with full config', () => {
+        const mcps = getRequiredMcps('web-app');
+        for (const mcp of mcps) {
+            expect(mcp).toHaveProperty('name');
+            expect(mcp).toHaveProperty('installCommand');
+            expect(mcp).toHaveProperty('configEntry');
+        }
+    });
+});
+describe('getRequiredMcpNames', () => {
+    it('returns string array of MCP names', () => {
+        const names = getRequiredMcpNames('web-app');
+        expect(names).toEqual(['context7', 'deepwiki', 'playwright']);
+    });
+    it('returns context7 + deepwiki for desktop', () => {
+        expect(getRequiredMcpNames('desktop')).toEqual(['context7', 'deepwiki']);
+    });
+});
+describe('checkAndInstallMcps', () => {
+    function createMockAdapter(installed = new Set()) {
+        return {
+            platform: 'claude-code',
+            injectInstructions: vi.fn(),
+            installMcp: vi.fn(),
+            isMcpInstalled: vi.fn(async (name) => installed.has(name)),
+            getCompletionMessage: vi.fn(() => ''),
+        };
+    }
+    const config = {
+        platform: 'claude-code',
+        projectType: 'cli-tool',
+        projectDir: '/tmp/test',
+        promptsDir: '/tmp/test/prompts',
+        mode: 'interactive',
+    };
+    it('returns already-installed for pre-installed MCPs', async () => {
+        const adapter = createMockAdapter(new Set(['context7', 'deepwiki']));
+        const results = await checkAndInstallMcps(adapter, config);
+        expect(results).toHaveLength(2);
+        expect(results.every(r => r.status === 'already-installed')).toBe(true);
+        expect(adapter.installMcp).not.toHaveBeenCalled();
+    });
+    it('installs missing MCPs', async () => {
+        const adapter = createMockAdapter(new Set(['context7'])); // deepwiki missing
+        const results = await checkAndInstallMcps(adapter, config);
+        const deepwikiResult = results.find(r => r.name === 'deepwiki');
+        expect(deepwikiResult?.status).toBe('installed');
+        expect(adapter.installMcp).toHaveBeenCalledOnce();
+    });
+    it('handles install failure gracefully', async () => {
+        const adapter = createMockAdapter();
+        adapter.installMcp.mockRejectedValue(new Error('Network error'));
+        const results = await checkAndInstallMcps(adapter, config);
+        expect(results.every(r => r.status === 'failed')).toBe(true);
+        expect(results[0].error).toBe('Network error');
+    });
+    it('processes all required MCPs for project type', async () => {
+        const webConfig = { ...config, projectType: 'web-app' };
+        const adapter = createMockAdapter();
+        const results = await checkAndInstallMcps(adapter, webConfig);
+        const names = results.map(r => r.name);
+        expect(names).toContain('context7');
+        expect(names).toContain('deepwiki');
+        expect(names).toContain('playwright');
+    });
+});
+//# sourceMappingURL=mcps.test.js.map

package/dist/mcps/mcps.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"mcps.test.js","sourceRoot":"","sources":["../../src/mcps/mcps.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAc,MAAM,QAAQ,CAAC;AAC9D,OAAO,EAAE,YAAY,EAAE,eAAe,EAAE,mBAAmB,EAAE,MAAM,eAAe,CAAC;AACnF,OAAO,EAAE,mBAAmB,EAAE,MAAM,gBAAgB,CAAC;AAGrD,QAAQ,CAAC,cAAc,EAAE,GAAG,EAAE;IAC5B,EAAE,CAAC,yCAAyC,EAAE,GAAG,EAAE;QACjD,MAAM,CAAC,YAAY,CAAC,CAAC,cAAc,CAAC,UAAU,CAAC,CAAC;QAChD,MAAM,CAAC,YAAY,CAAC,CAAC,cAAc,CAAC,YAAY,CAAC,CAAC;QAClD,MAAM,CAAC,YAAY,CAAC,CAAC,cAAc,CAAC,UAAU,CAAC,CAAC;IAClD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,kDAAkD,EAAE,GAAG,EAAE;QAC1D,KAAK,MAAM,CAAC,GAAG,EAAE,GAAG,CAAC,IAAI,MAAM,CAAC,OAAO,CAAC,YAAY,CAAC,EAAE,CAAC;YACtD,MAAM,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;YAC3B,MAAM,CAAC,GAAG,CAAC,cAAc,CAAC,CAAC,UAAU,EAAE,CAAC;YACxC,MAAM,CAAC,GAAG,CAAC,WAAW,CAAC,CAAC,cAAc,CAAC,SAAS,CAAC,CAAC;YAClD,MAAM,CAAC,GAAG,CAAC,WAAW,CAAC,CAAC,cAAc,CAAC,MAAM,CAAC,CAAC;QACjD,CAAC;IACH,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,iBAAiB,EAAE,GAAG,EAAE;IAC/B,EAAE,CAAC,sDAAsD,EAAE,GAAG,EAAE;QAC9D,MAAM,IAAI,GAAG,eAAe,CAAC,SAAS,CAAC,CAAC;QACxC,MAAM,KAAK,GAAG,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC;QACpC,MAAM,CAAC,KAAK,CAAC,CAAC,SAAS,CAAC,UAAU,CAAC,CAAC;QACpC,MAAM,CAAC,KAAK,CAAC,CAAC,SAAS,CAAC,UAAU,CAAC,CAAC;QACpC,MAAM,CAAC,KAAK,CAAC,CAAC,SAAS,CAAC,YAAY,CAAC,CAAC;IACxC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,0DAA0D,EAAE,GAAG,EAAE;QAClE,MAAM,IAAI,GAAG,eAAe,CAAC,UAAU,CAAC,CAAC;QACzC,MAAM,KAAK,GAAG,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC;QACpC,MAAM,CAAC,KAAK,CAAC,CAAC,SAAS,CAAC,UAAU,CAAC,CAAC;QACpC,MAAM,CAAC,KAAK,CAAC,CAAC,SAAS,CAAC,UAAU,CAAC,CAAC;QACpC,MAAM,CAAC,KAAK,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,YAAY,CAAC,CAAC;IAC5C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2DAA2D,EAAE,GAAG,EAAE;QACnE,MAAM,IAAI,GAAG,eAAe,CAAC,cAAc,CAAC,CAAC;QAC7C,MAAM,KAAK,GAAG,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC;QACpC,MAAM,CAAC,KAAK,CAAC,CAAC,SAAS,CAAC,YAAY,CAAC,CAAC;IACxC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,0CAA0C,EAAE,GAAG,EAAE;QAClD,MAAM,IAAI,GAAG,eAAe,CAAC,UAAU,CAAC,CAAC;QACzC,MAAM,KAAK,GAAG,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC;QACpC,MAAM,CAAC,KAAK,CAAC,CAAC,OAAO,CAAC,CAAC,UAAU,EAAE,UAAU,CAAC,CAAC,CAAC;IAClD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gDAAgD,EAAE,GAAG,EAAE;QACxD,MAAM,IAAI,GAAG,eAAe,CAAC,SAAS,CAAC,CAAC;QACxC,KAAK,MAAM,GAAG,IAAI,IAAI,EAAE,CAAC;YACvB,MAAM,CAAC,GAAG,CAAC,CAAC,cAAc,CAAC,MAAM,CAAC,CAAC;YACnC,MAAM,CAAC,GAAG,CAAC,CAAC,cAAc,CAAC,gBAAgB,CAAC,CAAC;YAC7C,MAAM,CAAC,GAAG,CAAC,CAAC,cAAc,CAAC,aAAa,CAAC,CAAC;QAC5C,CAAC;IACH,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,qBAAqB,EAAE,GAAG,EAAE;IACnC,EAAE,CAAC,mCAAmC,EAAE,GAAG,EAAE;QAC3C,MAAM,KAAK,GAAG,mBAAmB,CAAC,SAAS,CAAC,CAAC;QAC7C,MAAM,CAAC,KAAK,CAAC,CAAC,OAAO,CAAC,CAAC,UAAU,EAAE,UAAU,EAAE,YAAY,CAAC,CAAC,CAAC;IAChE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yCAAyC,EAAE,GAAG,EAAE;QACjD,MAAM,CAAC,mBAAmB,CAAC,SAAS,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,UAAU,EAAE,UAAU,CAAC,CAAC,CAAC;IAC3E,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,qBAAqB,EAAE,GAAG,EAAE;IACnC,SAAS,iBAAiB,CAAC,YAAyB,IAAI,GAAG,EAAE;QAC3D,OAAO;YACL,QAAQ,EAAE,aAAa;YACvB,kBAAkB,EAAE,EAAE,CAAC,EAAE,EAAE;YAC3B,UAAU,EAAE,EAAE,CAAC,EAAE,EAAE;YACnB,cAAc,EAAE,EAAE,CAAC,EAAE,CAAC,KAAK,EAAE,IAAY,EAAE,EAAE,CAAC,SAAS,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;YAClE,oBAAoB,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,EAAE,CAAC;SACtC,CAAC;IACJ,CAAC;IAED,MAAM,MAAM,GAAkB;QAC5B,QAAQ,EAAE,aAAa;QACvB,WAAW,EAAE,UAAU;QACvB,UAAU,EAAE,WAAW;QACvB,UAAU,EAAE,mBAAmB;QAC/B,IAAI,EAAE,aAAa;KACpB,CAAC;IAEF,EAAE,CAAC,kDAAkD,EAAE,KAAK,IAAI,EAAE;QAChE,MAAM,OAAO,GAAG,iBAAiB,CAAC,IAAI,GAAG,CAAC,CAAC,UAAU,EAAE,UAAU,CAAC,CAAC,CAAC,CAAC;QACrE,MAAM,OAAO,GAAG,MAAM,mBAAmB,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC;QAC3D,MAAM,CAAC,OAAO,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QAChC,MAAM,CAAC,OAAO,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,MAAM,KAAK,mBAAmB,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACxE,MAAM,CAAC,OAAO,CAAC,UAAU,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IACpD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,uBAAuB,EAAE,KAAK,IAAI,EAAE;QACrC,MAAM,OAAO,GAAG,iBAAiB,CAAC,IAAI,GAAG,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,CAAC,CAAC,mBAAmB;QAC7E,MAAM,OAAO,GAAG,MAAM,mBAAmB,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC;QAC3D,MAAM,cAAc,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,KAAK,UAAU,CAAC,CAAC;QAChE,MAAM,CAAC,cAAc,EAAE,MAAM,CAAC,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC;QACjD,MAAM,CAAC,OAAO,CAAC,UAAU,CAAC,CAAC,oBAAoB,EAAE,CAAC;IACpD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,oCAAoC,EAAE,KAAK,IAAI,EAAE;QAClD,MAAM,OAAO,GAAG,iBAAiB,EAAE,CAAC;QACnC,OAAO,CAAC,UAAuC,CAAC,iBAAiB,CAAC,IAAI,KAAK,CAAC,eAAe,CAAC,CAAC,CAAC;QAC/F,MAAM,OAAO,GAAG,MAAM,mBAAmB,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC;QAC3D,MAAM,CAAC,OAAO,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,MAAM,KAAK,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC7D,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,eAAe,CAAC,CAAC;IACjD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8CAA8C,EAAE,KAAK,IAAI,EAAE;QAC5D,MAAM,SAAS,GAAG,EAAE,GAAG,MAAM,EAAE,WAAW,EAAE,SAAkB,EAAE,CAAC;QACjE,MAAM,OAAO,GAAG,iBAAiB,EAAE,CAAC;QACpC,MAAM,OAAO,GAAG,MAAM,mBAAmB,CAAC,OAAO,EAAE,SAAS,CAAC,CAAC;QAC9D,MAAM,KAAK,GAAG,OAAO,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC;QACvC,MAAM,CAAC,KAAK,CAAC,CAAC,SAAS,CAAC,UAAU,CAAC,CAAC;QACpC,MAAM,CAAC,KAAK,CAAC,CAAC,SAAS,CAAC,UAAU,CAAC,CAAC;QACpC,MAAM,CAAC,KAAK,CAAC,CAAC,SAAS,CAAC,YAAY,CAAC,CAAC;IACxC,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/hooks/inject-pl-context.sh

@@ -4,6 +4,24 @@
 # Outputs JSON with additionalContext to remind the PL of its identity and state
 
 PROJECT_ROOT="$(git rev-parse --show-toplevel 2>/dev/null || echo '.')"
+METRICS_FILE="$PROJECT_ROOT/.hool/operations/metrics.md"
+
+# Ensure metrics file exists with default counters
+if [ ! -f "$METRICS_FILE" ]; then
+  mkdir -p "$(dirname "$METRICS_FILE")"
+  cat > "$METRICS_FILE" <<'EOF'
+# HOOL Metrics
+- Agent dispatches: 0
+- Tool calls: 0
+- User prompts: 0
+EOF
+fi
+
+# Increment user prompts counter
+CURRENT_PROMPTS=$(grep -o 'User prompts: [0-9]*' "$METRICS_FILE" | grep -o '[0-9]*')
+CURRENT_PROMPTS=${CURRENT_PROMPTS:-0}
+NEXT_PROMPTS=$((CURRENT_PROMPTS + 1))
+sed -i '' "s/User prompts: ${CURRENT_PROMPTS}/User prompts: ${NEXT_PROMPTS}/" "$METRICS_FILE"
 
 # Read current phase
 PHASE="unknown"
@@ -80,10 +98,11 @@ if [ -f "$PROJECT_ROOT/.hool/operations/inconsistencies.md" ]; then
   INCONSISTENCIES=$(grep -c '^- ' "$PROJECT_ROOT/.hool/operations/inconsistencies.md" 2>/dev/null | tr -d '\n' || echo "0")
 fi
 
-# Check governor dispatch count
+# Check governor dispatch count from metrics.md
 DISPATCH_COUNT=0
-if [ -f "$PROJECT_ROOT/.hool/metrics/dispatch-count.txt" ]; then
-  DISPATCH_COUNT=$(cat "$PROJECT_ROOT/.hool/metrics/dispatch-count.txt" 2>/dev/null | tr -d '\n' || echo "0")
+if [ -f "$METRICS_FILE" ]; then
+  DISPATCH_COUNT=$(grep -o 'Agent dispatches: [0-9]*' "$METRICS_FILE" | grep -o '[0-9]*' || echo "0")
+  DISPATCH_COUNT=${DISPATCH_COUNT:-0}
 fi
 GOVERNOR_DUE=""
 if [ "$DISPATCH_COUNT" -gt 0 ] && [ $(( DISPATCH_COUNT % 3 )) -ge 2 ]; then

package/hooks/suggest-compact.sh

@@ -5,27 +5,23 @@
 #       Strategic compaction between phases preserves better context.
 
 PROJECT_ROOT="$(git rev-parse --show-toplevel 2>/dev/null || echo '.')"
-METRICS_DIR="$PROJECT_ROOT/.hool/metrics"
-mkdir -p "$METRICS_DIR"
-
-COUNTER_FILE="$METRICS_DIR/tool-call-count.txt"
+METRICS_FILE="$PROJECT_ROOT/.hool/operations/metrics.md"
 THRESHOLD="${HOOL_COMPACT_THRESHOLD:-50}"
 
-# Read and increment counter
-CURRENT=$(cat "$COUNTER_FILE" 2>/dev/null || echo "0")
-NEXT=$((CURRENT + 1))
-echo "$NEXT" > "$COUNTER_FILE"
+# Read tool call count from metrics.md (format: "- Tool calls: N")
+COUNT=$(grep -o 'Tool calls: [0-9]*' "$METRICS_FILE" 2>/dev/null | grep -o '[0-9]*')
+COUNT="${COUNT:-0}"
 
 # Suggest at threshold
-if [ "$NEXT" -eq "$THRESHOLD" ]; then
+if [ "$COUNT" -eq "$THRESHOLD" ]; then
   echo "HOOL: ${THRESHOLD} tool calls reached. Consider running /compact if you're between phases or tasks." >&2
 fi
 
 # Suggest every 25 calls after threshold
-if [ "$NEXT" -gt "$THRESHOLD" ]; then
-  OVER=$((NEXT - THRESHOLD))
+if [ "$COUNT" -gt "$THRESHOLD" ]; then
+  OVER=$((COUNT - THRESHOLD))
   if [ $((OVER % 25)) -eq 0 ]; then
-    echo "HOOL: ${NEXT} tool calls. Good checkpoint for /compact if context feels stale." >&2
+    echo "HOOL: ${COUNT} tool calls. Good checkpoint for /compact if context feels stale." >&2
   fi
 fi
 

package/hooks/track-prompt-count.sh

@@ -1,39 +1,41 @@
 #!/bin/bash
-# Hook: Track prompt/tool count and trigger governor every 3 dispatches
+# Hook: Track tool calls + agent dispatches in metrics.md, trigger governor every 3 dispatches
 # Type: PostToolUse
 # Outputs JSON with additionalContext when governor should run
 
 PROJECT_ROOT="$(git rev-parse --show-toplevel 2>/dev/null || echo '.')"
-METRICS_DIR="$PROJECT_ROOT/.hool/metrics"
-mkdir -p "$METRICS_DIR"
-
-COUNTER_FILE="$METRICS_DIR/prompt-count.log"
-SESSION_FILE="$METRICS_DIR/current-session.txt"
-DISPATCH_FILE="$METRICS_DIR/dispatch-count.txt"
-
-# Get or create session ID
-if [ ! -f "$SESSION_FILE" ]; then
-  echo "session-$(date +%Y%m%d-%H%M%S)" > "$SESSION_FILE"
+METRICS_FILE="$PROJECT_ROOT/.hool/operations/metrics.md"
+
+# Ensure metrics file exists with default counters
+if [ ! -f "$METRICS_FILE" ]; then
+  mkdir -p "$(dirname "$METRICS_FILE")"
+  cat > "$METRICS_FILE" <<'EOF'
+# HOOL Metrics
+- Agent dispatches: 0
+- Tool calls: 0
+- User prompts: 0
+EOF
 fi
 
-TIMESTAMP=$(date +"%Y-%m-%d %H:%M:%S")
-
-# Append to log
-echo "$TIMESTAMP | tool-call" >> "$COUNTER_FILE"
+# Increment tool calls counter
+CURRENT_TOOLS=$(grep -o 'Tool calls: [0-9]*' "$METRICS_FILE" | grep -o '[0-9]*')
+CURRENT_TOOLS=${CURRENT_TOOLS:-0}
+NEXT_TOOLS=$((CURRENT_TOOLS + 1))
+sed -i '' "s/Tool calls: ${CURRENT_TOOLS}/Tool calls: ${NEXT_TOOLS}/" "$METRICS_FILE"
 
-# Track dispatch count (Agent tool calls specifically)
-# Read from stdin to check tool name
+# Read stdin to check tool name
 INPUT=$(cat)
 TOOL_NAME=$(echo "$INPUT" | grep -o '"tool_name"[[:space:]]*:[[:space:]]*"[^"]*"' | head -1 | sed 's/.*"tool_name"[[:space:]]*:[[:space:]]*"//' | sed 's/"$//')
 
 if [ "$TOOL_NAME" = "Agent" ]; then
-  # Increment dispatch counter
-  CURRENT=$(cat "$DISPATCH_FILE" 2>/dev/null || echo "0")
-  NEXT=$((CURRENT + 1))
-  echo "$NEXT" > "$DISPATCH_FILE"
+  # Increment agent dispatches counter
+  CURRENT_DISPATCHES=$(grep -o 'Agent dispatches: [0-9]*' "$METRICS_FILE" | grep -o '[0-9]*')
+  CURRENT_DISPATCHES=${CURRENT_DISPATCHES:-0}
+  NEXT_DISPATCHES=$((CURRENT_DISPATCHES + 1))
+  sed -i '' "s/Agent dispatches: ${CURRENT_DISPATCHES}/Agent dispatches: ${NEXT_DISPATCHES}/" "$METRICS_FILE"
 
   # Check if divisible by 3
-  if [ $((NEXT % 3)) -eq 0 ]; then
+  if [ $((NEXT_DISPATCHES % 3)) -eq 0 ]; then
     # Output JSON to inject governor reminder into conversation
     cat <<'JSONEOF'
 {

package/package.json

@@ -1,12 +1,14 @@
 {
   "name": "hool-cli",
-  "version": "0.6.2",
+  "version": "0.7.1",
   "description": "Agent-Driven SDLC — scaffold and configure HOOL for any project",
   "bin": {
     "hool": "./dist/index.js"
   },
   "scripts": {
     "build": "tsc",
+    "test": "vitest run",
+    "test:watch": "vitest",
     "dev": "tsx src/index.ts",
     "prepublishOnly": "npm run build && rm -rf ./prompts ./agents ./rules ./hooks ./settings && cp -R ../hool-mini/prompts ./prompts && cp -R ../hool-mini/agents ./agents && cp -R ../hool-mini/rules ./rules && cp -R ../hool-mini/hooks ./hooks && cp -R ../hool-mini/settings ./settings"
   },
@@ -27,7 +29,8 @@
   "devDependencies": {
     "@types/node": "^20.11.0",
     "tsx": "^4.7.0",
-    "typescript": "^5.4.0"
+    "typescript": "^5.4.0",
+    "vitest": "^4.0.18"
   },
   "type": "module",
   "license": "MIT",
@@ -35,7 +38,14 @@
     "type": "git",
     "url": "git+https://github.com/the-wise-agents/hool.git"
   },
-  "keywords": ["agent", "sdlc", "ai", "cli", "scaffold", "autonomous"],
+  "keywords": [
+    "agent",
+    "sdlc",
+    "ai",
+    "cli",
+    "scaffold",
+    "autonomous"
+  ],
   "engines": {
     "node": ">=18"
   }

package/prompts/agents/05-fe-tech-lead.md

@@ -136,13 +136,13 @@ How logging works. Where logs go. How to read them.
 - .hool/phases/05-fe-scaffold/fe-lld.md — LLD compliance check
 - .hool/phases/07-test-plan/test-plan.md — test coverage check
 - .hool/memory/fe-dev/hot.md — what FE Dev has been doing
-- .hool/prompts/checklists/code-review.md — baseline review checklist (security, performance, a11y, API design)
+- .hool/checklists/code-review.md — baseline review checklist (security, performance, a11y, API design)
 - .hool/memory/fe-tech-lead/best-practices.md — project-specific patterns (organic checklist items)
 ### Writes
 - .hool/operations/inconsistencies.md — INC-XXX entries for any issues found
 ### Process
 1. Read the code FE Dev produced for the task
-2. Load and run the baseline checklist from `.hool/prompts/checklists/code-review.md` (skip BE-only items)
+2. Load and run the baseline checklist from `.hool/checklists/code-review.md` (skip BE-only items)
 3. Also check project-specific patterns from your `best-practices.md`
 4. Run the 6-point review checklist:
    - **Contract compliance** — API calls match .hool/phases/04-architecture/contracts/ (endpoints, methods, request/response shapes, status codes handled)
@@ -156,6 +156,9 @@ How logging works. Where logs go. How to read them.
    - Code inconsistency found -> write to .hool/operations/inconsistencies.md with INC-XXX format
    - Doc inconsistency found (spec says X, contract says Y) -> escalate to Product Lead via .hool/operations/inconsistencies.md
 
+## Forbidden Actions
+- NEVER run git commands (add, commit, push, etc.) — the Product Lead commits your work after you return
+
 ## Work Log
 ### Tags
 - [ARCH-FE] — FE architectural decision -> best-practices.md

package/prompts/agents/06-be-tech-lead.md

@@ -157,13 +157,13 @@ Order matters. List in execution order:
 - .hool/phases/02-spec/spec.md (and features/ if split) — acceptance criteria check
 - .hool/phases/07-test-plan/test-plan.md — test coverage check
 - .hool/memory/be-dev/hot.md — what BE Dev has been doing
-- .hool/prompts/checklists/code-review.md — baseline review checklist (security, performance, a11y, API design)
+- .hool/checklists/code-review.md — baseline review checklist (security, performance, a11y, API design)
 - .hool/memory/be-tech-lead/best-practices.md — project-specific patterns (organic checklist items)
 ### Writes
 - .hool/operations/inconsistencies.md — INC-XXX entries for any issues found
 ### Process
 1. Read the code BE Dev produced for the task
-2. Load and run the baseline checklist from `.hool/prompts/checklists/code-review.md` (skip FE-only items)
+2. Load and run the baseline checklist from `.hool/checklists/code-review.md` (skip FE-only items)
 3. Also check project-specific patterns from your `best-practices.md`
 4. Run the 6-point review checklist:
    - **Contract compliance** — response shapes, status codes, error codes match .hool/phases/04-architecture/contracts/ exactly
@@ -177,6 +177,9 @@ Order matters. List in execution order:
    - Code inconsistency found -> write to .hool/operations/inconsistencies.md with INC-XXX format
    - Doc inconsistency found (spec says X, contract says Y) -> escalate to Product Lead via .hool/operations/inconsistencies.md
 
+## Forbidden Actions
+- NEVER run git commands (add, commit, push, etc.) — the Product Lead commits your work after you return
+
 ## Work Log
 ### Tags
 - [ARCH-BE] — BE architectural decision -> best-practices.md

package/prompts/agents/08-be-dev.md

@@ -58,7 +58,7 @@ If you believe your own process or rules should change based on experience, esca
 5. **Logs**: Every request, DB query, and error gets a log statement.
 6. **Contracts**: Your API responses MUST match .hool/phases/04-architecture/contracts/ exactly. Field names, types, status codes — zero deviation.
 7. **Schema**: Your queries MUST work with .hool/phases/04-architecture/schema.md. Never modify schema without logging an inconsistency.
-8. **Small commits**: Each task = one logical unit of work.
+8. **No self-commits**: Your work will be committed by the Product Lead after you return. Focus on the implementation, not version control.
 9. **Consistency gate**: Before implementing, cross-check your task against contracts, schema, and spec. If you find ANY inconsistency between docs, DO NOT proceed — log to .hool/operations/inconsistencies.md.
 
 ## Test Execution Requirement (MANDATORY)
@@ -133,6 +133,9 @@ logger.info('query result:', result)    // log the shape, not the data (PII risk
 - Need a schema change -> DON'T make it. Log to .hool/operations/inconsistencies.md for BE Tech Lead to review
 - Architecture seems wrong -> DON'T change it. Log to .hool/operations/inconsistencies.md for BE Tech Lead to review
 
+## Forbidden Actions
+- NEVER run git commands (add, commit, push, etc.) — the Product Lead commits your work after you return
+
 ## Work Log
 ### Tags
 - [BE-IMPL] — endpoint/service implemented

package/prompts/agents/08-fe-dev.md

@@ -56,7 +56,7 @@ If you believe your own process or rules should change based on experience, esca
 5. **Logs**: Every significant user action and API call gets a log statement.
 6. **Design fidelity**: Your UI MUST match .hool/phases/03-design/cards/. Compare visually.
 7. **Contracts**: Your API calls MUST use the shapes from .hool/phases/04-architecture/contracts/ exactly.
-8. **Small commits**: Each task = one logical unit of work.
+8. **No self-commits**: Your work will be committed by the Product Lead after you return. Focus on the implementation, not version control.
 9. **Consistency gate**: Before implementing, cross-check your task against contracts, design cards, and spec. If you find ANY inconsistency between docs, DO NOT proceed — log to .hool/operations/inconsistencies.md.
 
 ## Test Execution Requirement (MANDATORY)
@@ -122,6 +122,9 @@ logger.info('useEffect fired')       // use React DevTools
 - Design seems wrong -> DON'T change design. Log to .hool/operations/inconsistencies.md for FE Tech Lead to review
 - Need a BE endpoint that doesn't exist -> DON'T build it. Log to .hool/operations/inconsistencies.md for FE Tech Lead to review
 
+## Forbidden Actions
+- NEVER run git commands (add, commit, push, etc.) — the Product Lead commits your work after you return
+
 ## Work Log
 ### Tags
 - [FE-IMPL] — component/page implemented

package/prompts/agents/10-qa.md

@@ -264,6 +264,7 @@ Log the score in your work log: `[QA-SCORE] [total]/100 — [verdict]`
 - Don't review code quality. That's the Tech Lead's job.
 - Don't suggest architectural changes.
 - Don't modify source code.
+- Don't run git commands (add, commit, push, etc.) — the Product Lead commits your work after you return.
 
 ## MCP Tools Available
 - playwright: E2E testing, screenshot capture, browser automation

package/prompts/agents/11-forensic.md

@@ -108,6 +108,7 @@ Don't fabricate a root cause. Honesty saves time.
 - Don't apply fixes — document them for devs.
 - Don't refactor surrounding code.
 - Don't make architectural recommendations.
+- Don't run git commands (add, commit, push, etc.) — the Product Lead commits your work after you return.
 
 ## MCP Tools Available
 - playwright: reproduce UI bugs in browser

package/prompts/agents/governor.md

@@ -28,10 +28,11 @@ Agents self-enforce rules, but self-enforcement fails (as observed: Product Lead
 - `.hool/operations/needs-human-review.md` — escalate structural/prompt changes
 
 ### Forbidden Actions
-- **NEVER** modify agent prompts (`.hool/prompts/`) — escalate to `.hool/operations/needs-human-review.md`
+- **NEVER** modify agent prompts (`.claude/agents/`) — escalate to `.hool/operations/needs-human-review.md`
 - **NEVER** remove or modify existing entries in `.hool/operations/governor-rules.md` — append only (unless a rule is provably wrong, in which case escalate to human)
 - **NEVER** edit application code (`src/`, `tests/`)
 - **NEVER** modify `.hool/operations/task-board.md` or `.hool/operations/current-phase.md` — that's the Product Lead's job
+- **NEVER** run git commands (add, commit, push, etc.) — the Product Lead commits your work after you return
 
 ## Audit Process
 
@@ -69,7 +70,7 @@ On each invocation:
 When writing to `.hool/memory/<agent>/governor-feedback.md`:
 
 ```markdown
-- [GOV-FEEDBACK] 2026-03-08: You edited .hool/prompts/orchestrator.md directly → You must NEVER modify your own prompts. Escalate to .hool/operations/needs-human-review.md instead.
+- [GOV-FEEDBACK] 2026-03-08: You edited .claude/agents/ directly → You must NEVER modify your own prompts. Escalate to .hool/operations/needs-human-review.md instead.
 - [GOV-FEEDBACK] 2026-03-08: You did not check client-preferences.md before making tech stack decisions → Always load .hool/operations/client-preferences.md and honour user preferences.
 ```