pi-harness-runtime 0.9.25 → 1.1.0

package/CHANGELOG.md

@@ -2,6 +2,48 @@
 
 All notable changes to this project will be documented in this file. See [standard-version](https://github.com/conventional-changelog/standard-version) for commit guidelines.
 
+### [0.10.1](https://github.com/ManotLuijiu/pi-harness-runtime/compare/v0.9.24...v0.10.1) (2026-07-31)
+
+
+### Features
+
+* blackboard-coordinated loop controller with early exit ([f3cf3ea](https://github.com/ManotLuijiu/pi-harness-runtime/commit/f3cf3ea479170dd75b7e9a7c3515dde230bb0da7))
+* herdr inter-agent event bus for autonomous code review ([1490cdd](https://github.com/ManotLuijiu/pi-harness-runtime/commit/1490cddec80dac812065c3c681984524ad3194e1))
+* loop controller for write-review cycles with early exit ([95066ce](https://github.com/ManotLuijiu/pi-harness-runtime/commit/95066ce8cb58c55eaa14d69fd826b5aaff680c16))
+* release adapters, stale reminder fix, tests, and bug fixes ([5fe5a26](https://github.com/ManotLuijiu/pi-harness-runtime/commit/5fe5a260e0d9a005965b43e4973cbe04a371b85e))
+
+
+### Bug Fixes
+
+* **clipboard:** pass UTF-8 bytes directly to xclip via stdin, avoid pipeline corruption ([051ca18](https://github.com/ManotLuijiu/pi-harness-runtime/commit/051ca18fb1deb1169a8a4da16aa9314d2e6ec66d))
+* disable todo-bd auto-reminder (Phase 1 containment) ([e1b16c1](https://github.com/ManotLuijiu/pi-harness-runtime/commit/e1b16c11e34cee8e35ec813081b17dd171671771))
+* herdr-bus refactoring — rename publishers to *Simple variants ([d727591](https://github.com/ManotLuijiu/pi-harness-runtime/commit/d7275913a89deb56a6bcfdb912ea9d17e3a80ce1))
+* import LoopConfig from index.js, remove unused LoopNextAction import ([1100a4e](https://github.com/ManotLuijiu/pi-harness-runtime/commit/1100a4e87433e54416259dfdad41bc5e0def19b3))
+* periodic quota refresh every 15 minutes via setInterval ([f28e700](https://github.com/ManotLuijiu/pi-harness-runtime/commit/f28e7004d089aeada1659f4d55a35461baba76fc))
+* replace all Unicode box-drawing characters with ASCII ([dd4223a](https://github.com/ManotLuijiu/pi-harness-runtime/commit/dd4223abb0e25f746ee09ced05f6b3e40a10eb85))
+* replace Unicode dashes and arrows with ASCII equivalents ([e9027e6](https://github.com/ManotLuijiu/pi-harness-runtime/commit/e9027e689c06fdc1ca2ac4a6365ed6a6a0a384fd))
+* trigger cookie sync on startup so pre-existing cookies aren't ignored ([2914299](https://github.com/ManotLuijiu/pi-harness-runtime/commit/2914299b8caa02acbb06620c7cb8e4870acfa40e))
+
+## [0.10.0](https://github.com/ManotLuijiu/pi-harness-runtime/compare/v0.9.24...v0.10.0) (2026-07-30)
+
+
+### Features
+
+* blackboard-coordinated loop controller with early exit ([f3cf3ea](https://github.com/ManotLuijiu/pi-harness-runtime/commit/f3cf3ea479170dd75b7e9a7c3515dde230bb0da7))
+* herdr inter-agent event bus for autonomous code review ([1490cdd](https://github.com/ManotLuijiu/pi-harness-runtime/commit/1490cddec80dac812065c3c681984524ad3194e1))
+* loop controller for write-review cycles with early exit ([95066ce](https://github.com/ManotLuijiu/pi-harness-runtime/commit/95066ce8cb58c55eaa14d69fd826b5aaff680c16))
+
+
+### Bug Fixes
+
+* **clipboard:** pass UTF-8 bytes directly to xclip via stdin, avoid pipeline corruption ([051ca18](https://github.com/ManotLuijiu/pi-harness-runtime/commit/051ca18fb1deb1169a8a4da16aa9314d2e6ec66d))
+* herdr-bus refactoring — rename publishers to *Simple variants ([d727591](https://github.com/ManotLuijiu/pi-harness-runtime/commit/d7275913a89deb56a6bcfdb912ea9d17e3a80ce1))
+* import LoopConfig from index.js, remove unused LoopNextAction import ([1100a4e](https://github.com/ManotLuijiu/pi-harness-runtime/commit/1100a4e87433e54416259dfdad41bc5e0def19b3))
+* periodic quota refresh every 15 minutes via setInterval ([f28e700](https://github.com/ManotLuijiu/pi-harness-runtime/commit/f28e7004d089aeada1659f4d55a35461baba76fc))
+* replace all Unicode box-drawing characters with ASCII ([dd4223a](https://github.com/ManotLuijiu/pi-harness-runtime/commit/dd4223abb0e25f746ee09ced05f6b3e40a10eb85))
+* replace Unicode dashes and arrows with ASCII equivalents ([e9027e6](https://github.com/ManotLuijiu/pi-harness-runtime/commit/e9027e689c06fdc1ca2ac4a6365ed6a6a0a384fd))
+* trigger cookie sync on startup so pre-existing cookies aren't ignored ([2914299](https://github.com/ManotLuijiu/pi-harness-runtime/commit/2914299b8caa02acbb06620c7cb8e4870acfa40e))
+
 ### [0.9.25](https://github.com/ManotLuijiu/pi-harness-runtime/compare/v0.9.21...v0.9.25) (2026-07-27)
 
 
@@ -117,7 +159,7 @@ All notable changes to this project will be documented in this file. See [standa
 ### Bug Fixes
 
 * publish missing runtime modules ([3fc7058](https://github.com/ManotLuijiu/pi-harness-runtime/commit/3fc70586d0cd10f4d38eab67e4cadaec20aaeb56))
-* **release:** remove redundant Bump version step — was double-bumping version (0.7.1→0.8.0) when tag pushed ([413ad6d](https://github.com/ManotLuijiu/pi-harness-runtime/commit/413ad6d49356ec62b94e9a4e09d3475ef39bebd6))
+* **release:** remove redundant Bump version step — was double-bumping version (0.7.1->0.8.0) when tag pushed ([413ad6d](https://github.com/ManotLuijiu/pi-harness-runtime/commit/413ad6d49356ec62b94e9a4e09d3475ef39bebd6))
 
 ## [0.9.0](https://github.com/ManotLuijiu/pi-harness-runtime/compare/v0.7.1...v0.9.0) (2026-07-13)
 
@@ -127,7 +169,7 @@ All notable changes to this project will be documented in this file. See [standa
 
 ### Bug Fixes
 
-* **release:** remove redundant Bump version step — was double-bumping version (0.7.1→0.8.0) when tag pushed ([413ad6d](https://github.com/ManotLuijiu/pi-harness-runtime/commit/413ad6d49356ec62b94e9a4e09d3475ef39bebd6))
+* **release:** remove redundant Bump version step — was double-bumping version (0.7.1->0.8.0) when tag pushed ([413ad6d](https://github.com/ManotLuijiu/pi-harness-runtime/commit/413ad6d49356ec62b94e9a4e09d3475ef39bebd6))
 
 ### [0.7.1](https://github.com/ManotLuijiu/pi-harness-runtime/compare/v0.6.3...v0.7.1) (2026-07-13)
 
@@ -158,11 +200,11 @@ All notable changes to this project will be documented in this file. See [standa
 
 ### Changed
 
-* **Project rename**: `pi-usage-status` → `pi-harness-runtime`
-  * npm package name: `pi-usage-status` → `pi-harness-runtime`
-  * Repository: `ManotLuijiu/pi-usage-status` → `ManotLuijiu/pi-harness-runtime`
-  * Bundled skill: `usage-status` → `harness-runtime` (file path `skills/usage-status/` → `skills/harness-runtime/`)
-  * Status bar key: `usage-status` → `harness-runtime`
+* **Project rename**: `pi-usage-status` -> `pi-harness-runtime`
+  * npm package name: `pi-usage-status` -> `pi-harness-runtime`
+  * Repository: `ManotLuijiu/pi-usage-status` -> `ManotLuijiu/pi-harness-runtime`
+  * Bundled skill: `usage-status` -> `harness-runtime` (file path `skills/usage-status/` -> `skills/harness-runtime/`)
+  * Status bar key: `usage-status` -> `harness-runtime`
   * **Data directory unchanged**: still `~/.pi/usage-status/` (preserves existing user data)
   * **Env var unchanged**: still `PI_USAGE_DIR` (preserves existing user configs)
   * **Extension symlink unchanged**: still `~/.pi/agent/extensions/pi-usage-status` (repointed to renamed project directory)

package/README.md

@@ -4,7 +4,7 @@
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg?style=for-the-badge)](https://opensource.org/licenses/MIT)
 [![Platform](https://img.shields.io/badge/Platform-macOS%20|Linux-blue?style=for-the-badge)]()
 
-> ⚠️ **Beta Notice — Not Production Ready ⚠️**
+> !️ **Beta Notice — Not Production Ready !️**
 > This project is in active development. Core features work.
 
 **Autonomous AI coding harness for pi: local token tracking + provider mirror + task orchestration.**
@@ -27,6 +27,76 @@ pi install npm:pi-harness-runtime
 
 Requires Pi v0.37.3+.
 
+### Headless Server Setup (herdr clipboard bridge)
+
+If running on a **headless server** via herdr SSH, install the clipboard bridge plugin to enable copy/paste between server and local machine:
+
+```bash
+# 1. Link the plugin (already included in this package)
+cd ~/.config/herdr/plugins
+ln -s ~/.config/herdr/plugins/moocoding.clipboard-bridge moocoding.clipboard-bridge 2>/dev/null || true
+
+# 2. Add keybinding to ~/.config/herdr/config.toml
+cat >> ~/.config/herdr/config.toml << 'EOF'
+
+[[keys.command]]
+key = "prefix+y"
+type = "plugin_action"
+command = "moocoding.clipboard-bridge.copy"
+description = "Copy selected text via bridge"
+EOF
+
+# 3. Reload herdr config
+herdr server reload-config
+```
+
+### todo-bd-sync (Auto Task Tracker)
+
+Automatic two-way sync between rpiv-todo overlay and bd issue tracker.
+
+When you use `bd` for issue tracking, this feature automatically syncs tasks.
+
+> **Important:** `bd` is **not** implemented or bundled by `pi-harness-runtime`.
+> It is the external **Beads CLI** from the upstream project:
+> `https://github.com/gastownhall/beads`
+>
+> Install it system-wide from upstream, then initialize it in your own project:
+
+```bash
+# Install beads CLI (system-wide - don't clone this repo into your project)
+curl -fsSL https://raw.githubusercontent.com/gastownhall/beads/main/scripts/install.sh | bash
+
+# Initialize in YOUR project
+cd your-project
+bd init
+
+# Optional: refresh or install richer instructions for your agent
+bd setup codex    # Codex CLI - installs skill, AGENTS.md guidance, and hooks
+bd setup claude   # Claude Code - installs hooks/settings
+bd setup factory  # Factory.ai Droid - creates/updates AGENTS.md
+```
+
+- Install dependencies:
+
+```bash
+# Install rpiv-todo overlay
+pi install npm:@juicesharp/rpiv-todo
+
+# Verify bd is available from your upstream beads install
+bd --version
+```
+
+- Works automatically: Just use prompts with multiple steps, and tasks sync to bd
+- Manual commands:
+
+```bash
+/bd-todo-sync    # Force sync all mapped tasks
+/bd-todo-status  # Show sync status
+```
+
+> **Note:** Auto-reminder is **disabled by default** (bug fix for transcript growth).
+> See [docs/todo-bd-sync.md](docs/todo-bd-sync.md) for details.
+
 ## Usage Commands
 
 ```bash
@@ -35,6 +105,8 @@ Requires Pi v0.37.3+.
 /usage today       # today's usage + 5h window
 /usage week        # this week's usage + lifetime totals
 /usage reset       # clear provider mirror
+/bd-todo-sync      # Force sync all mapped tasks (todo <-> bd)
+/bd-todo-status    # Show todo-bd sync status and dependencies
 ```
 
 ## Harness Commands
@@ -54,10 +126,10 @@ Requires Pi v0.37.3+.
 /harness start Build a REST API with JWT authentication
 
 # Runtime creates task graph:
-# task-001: Analyze requirements → ready
-# task-002: Implement API → depends on task-001
-# task-003: Write tests → depends on task-002
-# task-004: Code review → depends on task-003
+# task-001: Analyze requirements -> ready
+# task-002: Implement API -> depends on task-001
+# task-003: Write tests -> depends on task-002
+# task-004: Code review -> depends on task-003
 
 /harness status
 # Job: job-123, Status: running
@@ -74,48 +146,48 @@ Requires Pi v0.37.3+.
 # When all tasks complete:
 /harness status
 # Job: job-123, Status: ready_for_client
-# Tasks: 4/4 done ✓
+# Tasks: 4/4 done [x]
 ```
 
 ## Architecture
 
 ```
 pi-harness-runtime/
-├── index.ts                    # Extension entry point
-├── footer-status.ts            # Footer bar renderer (local vs. mirror mode)
-├── mirror.ts                   # Per-provider quota mirror (mirror.json)
-├── harness/
-│   ├── job-state-machine.ts   # 14-state lifecycle machine
-│   ├── task-graph.ts           # DAG-based task management
-│   ├── master-planner.ts       # Requirement → task graph
-│   ├── loop-runtime.ts         # Core execution loop
-│   ├── repair-engine.ts        # Auto-fix failures
-│   ├── blackboard.ts           # Agent coordination
-│   ├── context-window-manager.ts # Context tracking
-│   ├── agent-handoff.ts        # Clean agent transitions
-│   ├── e2e/
-│   │   ├── minimax-quota-scraper.ts  # Playwright MiniMax scraper
-│   │   └── playwright-runner.ts       # Browser automation
-│   └── project-detector/
-│       └── detector.ts         # Auto-detect project type
-├── packages/
-│   ├── cookie-sanitizer/       # Live cookie-file sync via chokidar
-│   ├── providers/              # MiniMax, OpenAI, Codex adapters
-│   ├── quota-manager/          # Quota signal collection
-│   ├── auth/                   # MiniMax browser auth
-│   └── worktree/              # Git worktree per task
-└── skills/
-    └── harness-runtime/
-        └── SKILL.md           # Skill documentation
++-- index.ts                    # Extension entry point
++-- footer-status.ts            # Footer bar renderer (local vs. mirror mode)
++-- mirror.ts                   # Per-provider quota mirror (mirror.json)
++-- harness/
+|   +-- job-state-machine.ts   # 14-state lifecycle machine
+|   +-- task-graph.ts           # DAG-based task management
+|   +-- master-planner.ts       # Requirement -> task graph
+|   +-- loop-runtime.ts         # Core execution loop
+|   +-- repair-engine.ts        # Auto-fix failures
+|   +-- blackboard.ts           # Agent coordination
+|   +-- context-window-manager.ts # Context tracking
+|   +-- agent-handoff.ts        # Clean agent transitions
+|   +-- e2e/
+|   |   +-- minimax-quota-scraper.ts  # Playwright MiniMax scraper
+|   |   +-- playwright-runner.ts       # Browser automation
+|   +-- project-detector/
+|       +-- detector.ts         # Auto-detect project type
++-- packages/
+|   +-- cookie-sanitizer/       # Live cookie-file sync via chokidar
+|   +-- providers/              # MiniMax, OpenAI, Codex adapters
+|   +-- quota-manager/          # Quota signal collection
+|   +-- auth/                   # MiniMax browser auth
+|   +-- worktree/              # Git worktree per task
++-- skills/
+    +-- harness-runtime/
+        +-- SKILL.md           # Skill documentation
 ```
 
 ## Job State Machine
 
 ```
-created → planning → queued → running → testing → reviewing
-    ↓        ↓         ↓        ↓         ↓
+created -> planning -> queued -> running -> testing -> reviewing
+    v        v         v        v         v
 cancelled  blocked   waiting_human  repairing  ready_for_client
-                                        ↓            ↓
+                                        v            v
                                   paused_quota    archived
 ```
 
@@ -136,18 +208,18 @@ All data stored locally in `~/.pi/`:
 
 ```
 ~/.pi/
-├── usage-status/         # /usage data
-│   ├── usage.jsonl
-│   └── mirror.json        # per-provider quota mirror
-├── harness/            # /harness data
-│   └── jobs/
-│       └── <job-id>/
-│           ├── checkpoint.json
-│           ├── events.jsonl
-│           ├── task-graph.json
-│           ├── blackboard/
-│           └── repair-tasks.jsonl
-└── okf/               # (optional) Your custom OKF knowledge
++-- usage-status/         # /usage data
+|   +-- usage.jsonl
+|   +-- mirror.json        # per-provider quota mirror
++-- harness/            # /harness data
+|   +-- jobs/
+|       +-- <job-id>/
+|           +-- checkpoint.json
+|           +-- events.jsonl
+|           +-- task-graph.json
+|           +-- blackboard/
+|           +-- repair-tasks.jsonl
++-- okf/               # (optional) Your custom OKF knowledge
 ```
 
 ## Custom OKF Knowledge (Optional)
@@ -156,9 +228,9 @@ Create `~/.pi/okf/` to add your custom knowledge that the runtime reads during c
 
 ```
 ~/.pi/okf/
-├── <skill-name>.md      # Your custom skill/procedure
-├── <domain>.md          # Domain-specific knowledge
-└── <project>.md        # Project-specific rules
++-- <skill-name>.md      # Your custom skill/procedure
++-- <domain>.md          # Domain-specific knowledge
++-- <project>.md        # Project-specific rules
 ```
 
 The runtime automatically detects and reads these files. If the folder doesn't exist, compilation proceeds without them.
@@ -221,7 +293,7 @@ MiniMax:  cookies not found — drop any cookie file into
 
 The runtime watches `~/.pi-harness-runtime/cookies/` for cookie files. Drop any cookie export there — it syncs within ~1 second:
 
-1. **Install EditThisCookie** Chrome extension → export from `platform.minimax.io`
+1. **Install EditThisCookie** Chrome extension -> export from `platform.minimax.io`
 2. **Create the drop folder:**
 
    ```bash
@@ -245,6 +317,56 @@ The runtime watches `~/.pi-harness-runtime/cookies/` for cookie files. Drop any
 **Override:** set `QUOTA_COOKIE_FILE=/path/to/your/cookies.txt` to bypass the sanitizer
 and read a specific file directly.
 
+## herdr Integration (Clipboard Bridge)
+
+> **⚠️ This only applies when using herdr.dev**
+>
+> If you're using a normal terminal (iTerm2, Terminal.app, etc.), clipboard works fine — no special setup needed.
+
+When running on a **headless server via herdr.dev**, clipboard operations need special setup because:
+
+- herdr's `copy_on_select` uses OSC52 which fails in VS Code's integrated terminal PTY
+- UTF-8 box-drawing characters (`┌─┐│└`) get corrupted to garbled text
+- Server-side clipboard tools (`xclip`, `wl-copy`) don't exist
+
+### Quick Setup
+
+```bash
+# Copy the example config (includes clipboard bridge keybinding)
+cp config.toml.example ~/.config/herdr/config.toml
+
+# Reload herdr config
+herdr server reload-config
+```
+
+### Usage
+
+1. **Select text** with mouse in herdr terminal
+2. Press **`prefix+y`** (prefix is `ctrl+b` by default)
+3. Text is copied via the bridge to your local clipboard
+
+### How It Works
+
+```
+Select text in herdr
+        ↓
+Press prefix+y (plugin action)
+        ↓
+~/.herdr-clipboard (bridge file)
+        ↓
+SSH reverse tunnel or tmux buffer
+        ↓
+Your local machine clipboard
+```
+
+### Files
+
+| File | Purpose |
+|------|---------|
+| `config.toml.example` | Template for `~/.config/herdr/config.toml` |
+| `~/.herdr-clipboard` | Bridge file between herdr and clipboard |
+| `~/.config/herdr/plugins/moocoding.clipboard-bridge/` | herdr plugin |
+
 ### Other Providers
 
 No setup needed. The runtime reads TUI messages and captures quota-exhaustion signals.

package/harness/auto-compact.ts

@@ -256,7 +256,7 @@ export class AutoCompactEngine {
 		return existsSync(join(this.rootDir, "continue_prompt.md"));
 	}
 
-	// ─── Private Methods ────────────────────────────────────────────────
+	// --- Private Methods ------------------------------------------------
 
 	private ensureDir(): void {
 		if (!existsSync(this.rootDir)) {

package/harness/autonomous-review.ts

@@ -0,0 +1,88 @@
+/**
+ * Autonomous Review Agent — polls for code.written events and auto-triggers review.
+ * Human on the loop, not in the loop.
+ */
+import { existsSync, readFileSync, writeFileSync } from "fs";
+import { join } from "path";
+import {
+	createHerdrBus,
+	ensureHerdrWorkspace,
+	publishReviewCompletedSimple,
+	publishReviewRequestedSimple,
+} from "../packages/event-bus/src/herdr-bus.js";
+
+const AGENT_ID = "review-agent";
+const REVIEW_TIMEOUT_MS = 5 * 60 * 1000;
+
+type CodeWrittenPayload = { taskId: string; files: string[]; branch?: string };
+
+async function runReview(
+	bus: ReturnType<typeof createHerdrBus>,
+	codePayload: CodeWrittenPayload,
+): Promise<void> {
+	const { taskId, files } = codePayload;
+	const reportFile = join(
+		ensureHerdrWorkspace().reviews,
+		`${taskId}-review.md`,
+	);
+	console.log(`[${AGENT_ID}] Reviewing task ${taskId}, ${files.length} files`);
+
+	try {
+		publishReviewRequestedSimple(bus, taskId, taskId);
+
+		const fileBlocks = await Promise.all(
+			files.map(async (file) => {
+				const content = existsSync(file)
+					? readFileSync(file, "utf-8").slice(0, 5000)
+					: `[Not found: ${file}]`;
+				return `## ${file}\n\n\`\`\`\n${content}\n\`\`\`\n`;
+			}),
+		);
+
+		const report = `# Code Review — Task ${taskId}
+## Files: ${files.join(", ")}
+
+> Auto-generated. GPT review tab fills this in.
+
+${fileBlocks.join("\n")}
+`;
+		writeFileSync(reportFile, report);
+		console.log(`[${AGENT_ID}] Report: ${reportFile}`);
+		publishReviewCompletedSimple(bus, taskId, reportFile, "changes_requested");
+
+		// Wait for GPT tab to update
+		await waitForGptReview(reportFile);
+	} catch (err) {
+		console.error(`[${AGENT_ID}] Review failed:`, err);
+		publishReviewCompletedSimple(bus, taskId, reportFile, "failed");
+	}
+}
+
+async function waitForGptReview(reportFile: string): Promise<void> {
+	const deadline = Date.now() + REVIEW_TIMEOUT_MS;
+	while (Date.now() < deadline) {
+		if (existsSync(reportFile)) {
+			const content = readFileSync(reportFile, "utf-8");
+			if (!content.includes("Pending")) return;
+		}
+		await new Promise((r) => setTimeout(r, 5000));
+	}
+	console.log(`[${AGENT_ID}] GPT review timed out`);
+}
+
+async function main(): Promise<void> {
+	console.log(`[${AGENT_ID}] Starting...`);
+	const bus = createHerdrBus(AGENT_ID);
+	ensureHerdrWorkspace();
+	bus.subscribe("code.written");
+	bus.startPolling(async (payload) => {
+		if (payload.topic === "code.written") {
+			const data = payload.data as CodeWrittenPayload;
+			if (data.files?.length) runReview(bus, data).catch(console.error);
+		}
+	});
+	console.log(`[${AGENT_ID}] Workspace: ${bus.getWorkspace()}`);
+	console.log(`[${AGENT_ID}] Polling for code.written...`);
+}
+
+main().catch(console.error);

package/harness/context-compact-orchestrator.ts

@@ -6,13 +6,13 @@
  *
  * Architecture:
  *   LoopRuntime.executeTask()
- *     └─> CompactOrchestrator.invokeWithCompact()
- *           ├─> beforeInvoke():
- *           │     1. Estimate token count with buffer awareness
- *           │     2. Check circuit breaker status
- *           │     3. Microcompact: prune old tool results if time-based
- *           │     4. If over autoCompactThreshold → Full compact via forked summarization
- *           └─> afterInvoke():
+ *     +-> CompactOrchestrator.invokeWithCompact()
+ *           +-> beforeInvoke():
+ *           |     1. Estimate token count with buffer awareness
+ *           |     2. Check circuit breaker status
+ *           |     3. Microcompact: prune old tool results if time-based
+ *           |     4. If over autoCompactThreshold → Full compact via forked summarization
+ *           +-> afterInvoke():
  *                 1. Update token stats
  *                 2. Handle reactive 413 (context too long)
  *                 3. Generate continue prompt
@@ -38,12 +38,12 @@ import {
 } from "./context-window-manager.js";
 import { continuePromptGenerator } from "./continue-prompt.js";
 
-// ─── Re-exports ────────────────────────────────────────────────────────────────
+// --- Re-exports ----------------------------------------------------------------
 
 /** Re-export CompactTriggerReason so harness/index.ts can re-export from here */
 export type { CompactTriggerReason } from "../packages/types/src/runtime-types.js";
 
-// ─── Callbacks ────────────────────────────────────────────────────────────────
+// --- Callbacks ----------------------------------------------------------------
 
 /** Callbacks provided by the harness to handle compact side-effects */
 export interface CompactOrchestratorCallbacks {
@@ -74,7 +74,7 @@ export interface CompactOrchestratorCallbacks {
 	) => Promise<{ summary: string; droppedCount: number }>;
 }
 
-// ─── Config ───────────────────────────────────────────────────────────────────
+// --- Config -------------------------------------------------------------------
 
 export interface CompactOrchestratorConfig {
 	jobId: string;
@@ -96,7 +96,7 @@ export interface CompactOrchestratorConfig {
 	contextWindowSize?: number;
 }
 
-// ─── Orchestrator ─────────────────────────────────────────────────────────────
+// --- Orchestrator -------------------------------------------------------------
 
 /**
  * Compact Orchestrator
@@ -166,7 +166,7 @@ export class CompactOrchestrator {
 		const { messages, model, maxOutputTokens } = opts;
 		this.compactAttempts++;
 
-		// ── Estimate tokens with buffer awareness ─────────────────────────────
+		// -- Estimate tokens with buffer awareness -----------------------------
 		const estimate = this.contextWindow.estimateTokensWithBuffer({
 			messages,
 			tools: opts.tools,
@@ -175,7 +175,7 @@ export class CompactOrchestrator {
 			model: model ?? this.model,
 		});
 
-		// ── Circuit breaker check ─────────────────────────────────────────────
+		// -- Circuit breaker check ---------------------------------------------
 		if (this.contextWindow.shouldCircuitBreak()) {
 			return {
 				success: false,
@@ -183,7 +183,7 @@ export class CompactOrchestrator {
 			};
 		}
 
-		// ── Blocking: must compact before API call ───────────────────────────
+		// -- Blocking: must compact before API call ---------------------------
 		if (this.contextWindow.shouldBlockApiCall(estimate)) {
 			callbacks.onPreCompact?.("token_threshold");
 			const compact = await this.runFullCompact(
@@ -206,10 +206,10 @@ export class CompactOrchestrator {
 			this.contextWindow.recordCompactSuccess();
 		}
 
-		// ── Time-based microcompact ─────────────────────────────────────────
+		// -- Time-based microcompact -----------------------------------------
 		this.tryMicrocompact(messages);
 
-		// ── Proactive auto-compact at 85% ──────────────────────────────────
+		// -- Proactive auto-compact at 85% ----------------------------------
 		if (this.contextWindow.shouldProactiveCompact(estimate)) {
 			callbacks.onPreCompact?.("token_threshold");
 			const compact = await this.runFullCompact(
@@ -228,7 +228,7 @@ export class CompactOrchestrator {
 			}
 		}
 
-		// ── Make the LLM call ───────────────────────────────────────────────
+		// -- Make the LLM call -----------------------------------------------
 		const invoke = opts.invokeAgent ?? callbacks.invokeAgent;
 		if (!invoke) {
 			return {
@@ -242,7 +242,7 @@ export class CompactOrchestrator {
 		// Track tool results
 		this.trackToolResults(messages);
 
-		// ── Update stats ─────────────────────────────────────────────────────
+		// -- Update stats -----------------------------------------------------
 		if (result.usage) {
 			this.contextWindow.updateStats({
 				provider: this.provider,
@@ -252,7 +252,7 @@ export class CompactOrchestrator {
 			});
 		}
 
-		// ── Reactive compact on context-too-long error ─────────────────────
+		// -- Reactive compact on context-too-long error ---------------------
 		const isContextError = this.isContextTooLongError(result.error);
 
 		if (isContextError) {
@@ -326,7 +326,7 @@ export class CompactOrchestrator {
 			return { success: false, error: "Nothing to compact" };
 		}
 
-		// ── Generate summary ────────────────────────────────────────────────
+		// -- Generate summary ------------------------------------------------
 		let summary = "";
 		let droppedCount = oldMessages.length;
 
@@ -345,7 +345,7 @@ export class CompactOrchestrator {
 			summary = this.heuristicSummary(oldMessages, reason);
 		}
 
-		// ── Build compact boundary message ───────────────────────────────────
+		// -- Build compact boundary message -----------------------------------
 		const boundaryMsg: CompactableMessage = {
 			role: "system",
 			content: continuePromptGenerator.generateBoundary({
@@ -361,7 +361,7 @@ export class CompactOrchestrator {
 			},
 		};
 
-		// ── Preserve tool results from recent messages ───────────────────────
+		// -- Preserve tool results from recent messages -----------------------
 		const preservedRecent = recentMessages.map((msg) => {
 			if (msg.role !== "assistant" || !msg.toolResults) return msg;
 			return {
@@ -373,7 +373,7 @@ export class CompactOrchestrator {
 			};
 		});
 
-		// ── Rebuild message array ───────────────────────────────────────────
+		// -- Rebuild message array -------------------------------------------
 		messages.length = 0;
 		messages.push(boundaryMsg, ...preservedRecent);