pi-harness-runtime 0.9.22 → 0.10.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.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)
+
+
+### Bug Fixes
+
+* **ci:** explicitly list test files to exclude cookie auth tests in release workflow ([1c96002](https://github.com/ManotLuijiu/pi-harness-runtime/commit/1c9600232282a1ef9323300fbd763c962aff49f6))
+* **ci:** skip browser auth tests in release workflow ([6117ed5](https://github.com/ManotLuijiu/pi-harness-runtime/commit/6117ed5af11357c6f12d11f4cd8d3441de519043))
+* **ci:** skip tests in release workflow ([8357ecd](https://github.com/ManotLuijiu/pi-harness-runtime/commit/8357ecd0b7d9c0c6ef81680902ca731da59a37c4))
+* **e2e:** add OpenAIUsageResponse type to fix rate_limit TS error ([2ed0891](https://github.com/ManotLuijiu/pi-harness-runtime/commit/2ed08911410ec07815d828028db1a6c9eaf2df2c))
+* **release:** add --workspaces=false to npm publish so only pi-harness-runtime is published ([31e715d](https://github.com/ManotLuijiu/pi-harness-runtime/commit/31e715d97e7fce5b82b503a95b86037d757db906))
+
+### [0.9.24](https://github.com/ManotLuijiu/pi-harness-runtime/compare/v0.9.21...v0.9.24) (2026-07-27)
+
+
+### Bug Fixes
+
+* **ci:** skip browser auth tests in release workflow ([6117ed5](https://github.com/ManotLuijiu/pi-harness-runtime/commit/6117ed5af11357c6f12d11f4cd8d3441de519043))
+* **e2e:** add OpenAIUsageResponse type to fix rate_limit TS error ([2ed0891](https://github.com/ManotLuijiu/pi-harness-runtime/commit/2ed08911410ec07815d828028db1a6c9eaf2df2c))
+* **release:** add --workspaces=false to npm publish so only pi-harness-runtime is published ([31e715d](https://github.com/ManotLuijiu/pi-harness-runtime/commit/31e715d97e7fce5b82b503a95b86037d757db906))
+
+### [0.9.23](https://github.com/ManotLuijiu/pi-harness-runtime/compare/v0.9.21...v0.9.23) (2026-07-27)
+
 ### [0.9.22](https://github.com/ManotLuijiu/pi-harness-runtime/compare/v0.9.21...v0.9.22) (2026-07-27)
 
 ### [0.9.11](https://github.com/ManotLuijiu/pi-harness-runtime/compare/v0.9.8...v0.9.11) (2026-07-23)
@@ -95,7 +137,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)
 
@@ -105,7 +147,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)
 
@@ -136,11 +178,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.**
@@ -54,10 +54,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 +74,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 +136,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 +156,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 +221,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

package/footer-status.ts

@@ -1,11 +1,16 @@
 import type { MirrorRecord, ProviderMirrorRecord } from "./mirror.ts";
-import { formatQuotaStatus } from "./harness/e2e/quota-status.js";
+
 import {
 	parseContextWindowStatusLine,
 	parseQuotaUsageStatusLine,
 	type ParsedContextWindowUsage,
 	type ParsedQuotaUsageData,
 } from "./status-parsers.ts";
+import { getProviderLabel } from "./packages/types/src/ai-providers.js";
+import {
+	providerHasContinuousScrape,
+	providerHasTUISignal,
+} from "./packages/providers/src/provider-id.js";
 
 export type ParsedFooterStatusValue =
 	| { kind: "quota"; value: ParsedQuotaUsageData }
@@ -39,46 +44,52 @@ export function parseFooterStatusValue(value: string): ParsedFooterStatusValue {
 	return { kind: "unknown", value: null };
 }
 
-/** Display name for a provider id. Used as the footer label. */
+/**
+ * Get display label for a provider id.
+ * Uses the canonical getProviderLabel() from packages/types/src/ai-providers.ts
+ * which handles all 9 known providers: minimax, openai, anthropic, glm,
+ * openrouter, openai-codex, deepseek, gemini, kimi.
+ */
 function providerDisplayName(provider: string | null | undefined): string {
 	if (!provider) return "Provider";
-	switch (provider) {
-		case "minimax":
-			return "MiniMax";
-		case "openai":
-			return "OpenAI";
-		case "openai-codex":
-			return "OpenAI Codex";
-		case "glm":
-			return "GLM";
-		case "anthropic":
-			return "Anthropic";
-		case "openrouter":
-			return "OpenRouter";
-		default:
-			return provider;
-	}
+	return getProviderLabel(provider);
 }
 
-/** Hint shown when the provider has no usage data yet. */
+/**
+ * Hint shown when the provider has no usage data yet.
+ * Covers all 9 known providers from KNOWN_AI_PROVIDERS.
+ */
 function missingDataHint(
 	provider: string | null | undefined,
 	hasCookieSource: boolean,
 ): string {
+	if (!provider) return "no usage source configured";
+
+	// MiniMax: needs cookies for continuous scrape
 	if (provider === "minimax") {
-		return "drop minimax cookies into ~/.pi-harness-runtime/cookies/";
+		return hasCookieSource
+			? "no data yet (updates after first scrape)"
+			: "drop minimax cookies into ~/.pi-harness-runtime/cookies/";
 	}
-	if (provider === "openai-codex") {
-		return "will update on first TUI limit signal";
-	}
-	if (
-		provider === "openai" ||
-		provider === "glm" ||
-		provider === "anthropic" ||
-		provider === "openrouter"
-	) {
+
+	// TUI signal providers: one-shot signal on limit hit
+	if (providerHasTUISignal(provider)) {
 		return "no signal yet (updates on first limit hit)";
 	}
+
+	// Providers without any tracking implementation yet
+	// deepseek, gemini, kimi, or any unknown provider
+	if (provider === "deepseek") {
+		return "deepseek usage tracking not yet implemented";
+	}
+	if (provider === "gemini") {
+		return "gemini usage tracking not yet implemented";
+	}
+	if (provider === "kimi") {
+		return "kimi usage tracking not yet implemented";
+	}
+
+	// Generic fallback for any other unknown provider
 	if (!hasCookieSource) return "no usage source configured";
 	return "no data yet";
 }
@@ -114,33 +125,62 @@ export function buildFooterStatusValue(
 	const provider = activeProvider ?? view?.provider ?? null;
 	const label = providerDisplayName(provider);
 
-	// Continuous data path: a fresh record has 5h or weekly percentages.
+	// TUI-signal exhaustion path: fires when limit is hit (one-shot signal).
+	// This takes priority over continuous data to show the most recent state.
 	if (
+		provider &&
+		view &&
+		view.exhausted &&
+		(view.limitType !== undefined ||
+			view.resets_at !== undefined ||
+			view.h5_resets_at !== undefined)
+	) {
+		const reset = view.resets_at ?? view.h5_resets_at ?? "soon";
+		const limitType = view.limitType ?? "tokens";
+		return `${label}: limit hit (${limitType}), reset ${reset}`;
+	}
+
+	// Continuous data path: providers with continuous scrape (MiniMax has 5h+weekly, OpenAI has weekly-only)
+	if (
+		provider &&
+		providerHasContinuousScrape(provider) &&
 		view &&
 		freshness !== "expired" &&
 		(view.h5_used_pct !== undefined || view.weekly_used_pct !== undefined)
 	) {
-		const quotaStatus = formatQuotaStatus({
-			provider: (provider ?? "minimax") as "minimax",
-			h5UsedPct: view.h5_used_pct ?? 0,
-			h5ResetsAt: view.h5_resets_at,
-			weeklyUsedPct: view.weekly_used_pct ?? 0,
-			weeklyResetsAt: view.weekly_resets_at,
-			scrapedAt: view.synced_at,
-		});
+		const weeklyPct = view.weekly_used_pct ?? 0;
+		const weeklyLeft = Math.max(0, 100 - weeklyPct);
+		const weeklyResets = view.weekly_resets_at ?? "soon";
+
+		let statusLine: string;
+		if (view.h5_used_pct !== undefined) {
+			// MiniMax: has both 5h and weekly windows
+			const h5Pct = view.h5_used_pct;
+			const h5Left = Math.max(0, 100 - h5Pct);
+			statusLine = `5h: ${h5Left.toFixed(0)}% left · week: ${weeklyLeft.toFixed(0)}% left`;
+		} else {
+			// OpenAI: weekly-only (no 5h window)
+			statusLine = `week: ${weeklyLeft.toFixed(0)}% left (resets ${weeklyResets})`;
+		}
+
 		const freshnessSuffix =
 			freshness === "fresh" || freshness === "missing" ? "" : ` · ${freshness}`;
-		return `${label}: ${quotaStatus.extended}${freshnessSuffix}`;
+		return `${label}: ${statusLine}${freshnessSuffix}`;
 	}
 
-	// TUI-signal path: an exhausted record from a limit hit.
+	// TUI signal providers (OpenAI, Anthropic, GLM, OpenRouter):
+	// Show monitoring status when we have a record but haven't hit limits.
+	// These providers only emit data when a limit is hit.
 	if (
+		provider &&
+		providerHasTUISignal(provider) &&
 		view &&
-		view.exhausted &&
-		(view.limitType !== undefined || view.resets_at !== undefined)
+		freshness !== "expired"
 	) {
-		const reset = view.resets_at ?? view.h5_resets_at ?? "soon";
-		return `${label}: limit hit (${view.limitType ?? "tokens"}), reset ${reset}`;
+		// If exhausted is not set, we're monitoring normally
+		if (!view.exhausted) {
+			return `${label}: monitoring (no limits hit)`;
+		}
 	}
 
 	// Discoverable hint when we have no data yet.

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/auto-quota-resume.ts

@@ -58,17 +58,20 @@ export function scheduleAutoResume(
 
 	const resetEpoch = record.h5_resets_at_epoch;
 	const now = Date.now();
-	const delayMs = Math.max(resetEpoch - now - RESUME_BUFFER_MS, MIN_RESUME_DELAY_MS);
+	const delayMs = Math.max(
+		resetEpoch - now - RESUME_BUFFER_MS,
+		MIN_RESUME_DELAY_MS,
+	);
 	const resumeAt = now + delayMs;
 	const resumeAtIso = new Date(resumeAt).toISOString();
 
 	// Persist to checkpoint so resumeAt survives worker restart
 	machine.setResumeTime(resumeAtIso);
 
-	console.log(
-		`[auto-quota-resume] ${provider} 5h quota exhausted.` +
-			` Auto-resume scheduled at ${resumeAtIso} (in ${Math.round(delayMs / 1000)}s)`,
-	);
+	// console.log(
+	// 	`[auto-quota-resume] ${provider} 5h quota exhausted.` +
+	// 		` Auto-resume scheduled at ${resumeAtIso} (in ${Math.round(delayMs / 1000)}s)`,
+	// );
 
 	const timeout = setTimeout(async () => {
 		activeTimers.delete(jobId);
@@ -84,10 +87,10 @@ export function scheduleAutoResume(
 		if (updated?.h5_resets_at_epoch) {
 			const stillExhausted = Date.now() < updated.h5_resets_at_epoch;
 			if (stillExhausted) {
-				console.log(
-					`[auto-quota-resume] ${provider} still exhausted at scheduled time.` +
-						` Re-scheduling...`,
-				);
+				// console.log(
+				// 	`[auto-quota-resume] ${provider} still exhausted at scheduled time.` +
+				// 			` Re-scheduling...`,
+				// );
 				scheduleAutoResume(provider, machine, mirrorStore);
 				return;
 			}
@@ -95,9 +98,9 @@ export function scheduleAutoResume(
 
 		const result = await machine.transition("running");
 		if (result.success) {
-			console.log(
-				`[auto-quota-resume] Job ${jobId} auto-resumed after ${provider} 5h quota reset.`,
-			);
+			// console.log(
+			// 	`[auto-quota-resume] Job ${jobId} auto-resumed after ${provider} 5h quota reset.`,
+			// );
 		} else {
 			console.error(
 				`[auto-quota-resume] Auto-resume failed for ${jobId}: ${result.error}`,

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);