pi-harness-runtime 1.1.49 → 1.1.50

package/harness/langchain/agents.test.ts

@@ -1,4 +1,3 @@
-
 import { describe, it } from "node:test";
 import assert from "node:assert";
 import { isAutonomyRequest } from "./agents.js";

package/harness/langchain/daemon.test.ts

@@ -23,6 +23,7 @@ import { randomUUID } from "node:crypto";
 
 import { LoopDaemon, writeAck } from "./daemon.js";
 import type { DaemonConfig } from "./daemon.js";
+import type { LoopDeps } from "./graph.js";
 
 // ─── Test fixtures ────────────────────────────────────────────────────────────
 
@@ -301,25 +302,79 @@ describe("T5 — single-writer (exactly one lease winner)", () => {
 describe("T6 — maxIterations cap", () => {
 	it("with maxIterations=2, exactly 2 code.written events appear", async () => {
 		cleanEvents();
+		// Mock reviewer: always changes_requested with a rotating file to avoid smart-stop.
+		// Use a mutable ref so review() sees the CURRENT writeCount (after increment).
+		const counter = { n: 0 };
+		const mockDeps: LoopDeps = {
+			maxIterations: 2,
+			onStep: undefined,
+			plan: async () => "# Stub plan\n1. Do thing",
+			write: async () => {
+				counter.n++;
+				return `// code pass ${counter.n}`;
+			},
+			review: async () => ({
+				verdict: "changes_requested" as const,
+				summary: "need changes",
+				comments: [{ file: `file${counter.n}.ts`, comment: "fix this", severity: "major" }],
+			}),
+		};
 		const daemon = new LoopDaemon(
-			testConfig({ sources: ["inbox"], maxIterations: 2 }),
+			testConfig({ sources: ["inbox"], maxIterations: 2, deps: mockDeps }),
 		);
 		daemon.start();
 		try {
 			const filename = `test-T6-${randomUUID().slice(0, 6)}.md`;
 			writeFileSync(join(WORKSPACE, filename), "Test iteration cap", "utf8");
 			await waitForEvent("loop.finished", 15_000);
-			// Dry-run: writeCount=1 → changes_requested → writeCount=2 → approved
-			// With maxIterations=2, loop exits after iteration 2
+			// Mock: writeCount=1 → review(changes, file1.ts) → writeCount=2 → review(changes, file2.ts)
+			// With maxIterations=2, the hard cap kicks in after iteration 2 → loop exits
+			// Smart-stop: different files (file1 vs file2) → no stuck-file trigger
 			assert.strictEqual(
-				countEvents("code.written"),
+				counter.n,
 				2,
-				`Expected exactly 2 code.written`,
+				`Expected exactly 2 writes (maxIterations=2, smart-stop overridden by different files)`,
 			);
 		} finally {
 			daemon.stop();
 		}
 	});
+
+	it("smart-stop: converged (only minor comments) terminates early", async () => {
+		cleanEvents();
+		let writeCount = 0;
+		// Reviewer returns only minor comments → smart-stop should trigger at iteration 1
+		const mockDeps: LoopDeps = {
+			maxIterations: 5,
+			onStep: undefined,
+			plan: async () => "# Stub plan",
+			write: async () => {
+				writeCount++;
+				return `// code ${writeCount}`;
+			},
+			review: async () => ({
+				verdict: "changes_requested" as const,
+				summary: "minor",
+				comments: [
+					{ file: "a.ts", comment: "nitpick", severity: "minor" },
+					{ file: "b.ts", comment: "typo", severity: "minor" },
+				],
+			}),
+		};
+		const daemon = new LoopDaemon(
+			testConfig({ sources: ["inbox"], maxIterations: 5, deps: mockDeps }),
+		);
+		daemon.start();
+		try {
+			const filename = `test-T6b-${randomUUID().slice(0, 6)}.md`;
+			writeFileSync(join(WORKSPACE, filename), "Test smart-stop", "utf8");
+			await waitForEvent("loop.finished", 15_000);
+			// Smart-stop: hasOnlyMinor=true (2 minor comments) → stop at iteration 1
+			assert.strictEqual(writeCount, 1, "converged: should stop after 1 write");
+		} finally {
+			daemon.stop();
+		}
+	});
 });
 
 // ─── T7: Crash resilience ───────────────────────────────────────────────────

package/harness/langchain/daemon.ts

@@ -824,6 +824,7 @@ export class LoopDaemon {
 			notificationConfig: config.notificationConfig ?? undefined,
 			tunnelCommand: config.tunnelCommand,
 			widget: config.widget,
+			deps: config.deps,
 		} as Required<DaemonConfig>;
 
 		this.agentId = this.config.agentId;

package/harness/langchain/file-tools.ts

@@ -46,7 +46,9 @@ export const readFile = tool(
 			}
 			const stats = statSync(resolved);
 			if (stats.isDirectory()) {
-				return JSON.stringify({ error: `Path is a directory, not a file: ${resolved}` });
+				return JSON.stringify({
+					error: `Path is a directory, not a file: ${resolved}`,
+				});
 			}
 			const content = readFileSync(resolved, encoding as BufferEncoding);
 			return JSON.stringify({
@@ -70,7 +72,9 @@ export const readFile = tool(
 			"Use this before editing a file so you know its current contents. " +
 			"Returns JSON with path, size, lines, and content fields.",
 		schema: z.object({
-			path: z.string().describe("Absolute path or path relative to the repository root"),
+			path: z
+				.string()
+				.describe("Absolute path or path relative to the repository root"),
 			encoding: z
 				.enum(["utf-8", "utf-16", "ascii", "base64"])
 				.optional()
@@ -126,12 +130,16 @@ export const writeFile = tool(
 			"Set append=true to add to the end of an existing file. " +
 			"Returns JSON with ok, path, size, and action fields.",
 		schema: z.object({
-			path: z.string().describe("Absolute path or path relative to the repository root"),
+			path: z
+				.string()
+				.describe("Absolute path or path relative to the repository root"),
 			content: z.string().describe("File content. Use \\n for newlines."),
 			append: z
 				.boolean()
 				.optional()
-				.describe("Append to existing file instead of overwriting (default: false)"),
+				.describe(
+					"Append to existing file instead of overwriting (default: false)",
+				),
 		}),
 	},
 );
@@ -169,7 +177,9 @@ export const listDir = tool(
 			"List the contents of a directory (files and subdirectories, non-recursive). " +
 			"Returns JSON with path and entries array (each entry has name, type, path).",
 		schema: z.object({
-			path: z.string().describe("Absolute path or path relative to the repository root"),
+			path: z
+				.string()
+				.describe("Absolute path or path relative to the repository root"),
 		}),
 	},
 );

package/harness/langchain/graph.ts

@@ -20,8 +20,6 @@ import {
 	StateGraph,
 } from "@langchain/langgraph";
 import {
-	autonomyDirective,
-	isAutonomyRequest,
 	lastMessage,
 	type ReviewVerdict,
 	ReviewVerdictSchema,
@@ -51,6 +49,11 @@ const LoopState = Annotation.Root({
 		reducer: (prev, next) => [...(prev ?? []), ...(next ?? [])],
 		default: () => [],
 	}),
+	/** File that received comments in the previous review. Used for smart-stop. */
+	lastCommentedFile: Annotation<string | undefined>({
+		reducer: (_prev, next) => next,
+		default: () => undefined,
+	}),
 });
 
 export type LoopState = typeof LoopState.State;
@@ -102,8 +105,11 @@ function reviewNode(deps: LoopDeps) {
 	return async (state: LoopState): Promise<Partial<LoopState>> => {
 		const review = await deps.review(state.plan, state.code);
 		deps.onStep?.("review", state);
+		const commentedFile =
+			review.comments.length > 0 ? (review.comments[0].file ?? "") : undefined;
 		return {
 			review,
+			lastCommentedFile: commentedFile,
 			log: [`[review:${state.iteration}] GPT verdict: ${review.verdict}`],
 		};
 	};
@@ -112,12 +118,23 @@ function reviewNode(deps: LoopDeps) {
 function finishNode() {
 	return async (state: LoopState): Promise<Partial<LoopState>> => {
 		const verdict = state.review?.verdict ?? "blocked";
-		const reason =
-			verdict === "approved"
-				? "reviewer approved"
-				: verdict === "changes_requested"
-					? `max iterations (${state.iteration}) reached with changes still requested`
-					: "reviewer blocked the task";
+		const comments = state.review?.comments ?? [];
+		let reason: string;
+		if (verdict === "approved") {
+			reason = "reviewer approved";
+		} else if (verdict === "blocked") {
+			reason = "reviewer blocked the task";
+		} else if (state.iteration >= 3) {
+			reason = `max iterations (${state.iteration}) reached with changes still requested`;
+		} else if (
+			comments.length > 0 &&
+			comments.every((c) => c.severity === "minor" || !c.severity)
+		) {
+			reason = `converged: only minor comments (${state.iteration})`;
+		} else {
+			const file = state.lastCommentedFile ?? "unknown";
+			reason = `stuck: same file (${file}) flagged for ${comments.length} comment(s) across iterations`;
+		}
 		return {
 			log: [`[finish] ${state.iteration} iteration(s) — ${reason}`],
 		};
@@ -126,13 +143,22 @@ function finishNode() {
 
 // ─── Routing ────────────────────────────────────────────────────────────────
 
+/** Smart stop: loop terminates early when quality criteria are met. */
 function routeAfterReview(
 	state: LoopState,
 	maxIterations: number,
 ): "write" | "finish" {
 	const verdict = state.review?.verdict;
+	const comments = state.review?.comments ?? [];
+	// 1. Always finish on approved or blocked
 	if (verdict === "approved" || verdict === "blocked") return "finish";
+	// 2. Hard cap
 	if (state.iteration >= maxIterations) return "finish";
+	// 3. SMART STOP: only minor comments (≤2) — converge fast
+	const hasOnlyMinor = comments.every(
+		(c) => c.severity === "minor" || !c.severity,
+	);
+	if (hasOnlyMinor && comments.length <= 2) return "finish";
 	return "write";
 }
 
@@ -229,7 +255,6 @@ export async function buildRealLoopDeps(
 	}
 
 	// Autonomy mode: computed inside plan(), stored for write()/review()
-	const _autonomous = false;
 	const _directive = "";
 
 	/** Inject scoreboard markdown into a user message. */
@@ -415,7 +440,7 @@ export function buildDryRunDeps(
 						{
 							file: "index.ts",
 							comment: "dry-run: rename the exported constant",
-							severity: "minor",
+							severity: "major",
 						},
 					],
 				};

package/harness/langchain/scoreboard.test.ts

@@ -19,7 +19,6 @@ import { buildDryRunDeps } from "./graph.js";
 import { createLoopCheckpointer } from "./checkpointer.js";
 
 describe("T15 — Scoreboard blackboard wiring", () => {
-
 	const tmpDir = `/tmp/scoreboard-test-${Date.now()}`;
 	const blackboardDir = join(tmpDir, ".write-review");
 
@@ -128,7 +127,7 @@ describe("T15 — Scoreboard blackboard wiring", () => {
 			request: "test",
 			plan: "# plan",
 			iteration: 1,
-			code: '```harness/langchain/surge.ts\nconst x = 1;\n```',
+			code: "```harness/langchain/surge.ts\nconst x = 1;\n```",
 			review: null as any,
 			log: [],
 		});
@@ -153,7 +152,10 @@ describe("T15 — Scoreboard blackboard wiring", () => {
 		);
 		assert.equal(content.phase, "changes_requested");
 		assert.equal(content.verdict, "changes_requested");
-		assert.ok(content.changesRequested?.length > 0, "changesRequested should be set");
+		assert.ok(
+			content.changesRequested?.length > 0,
+			"changesRequested should be set",
+		);
 	});
 
 	it("dry-run plan prompt includes scoreboard markdown", async () => {
@@ -218,5 +220,4 @@ describe("T15 — Scoreboard blackboard wiring", () => {
 		assert.equal(content.phase, "approved");
 		assert.equal(content.iteration, 1);
 	});
-
 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-harness-runtime",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "description": "[BETA] Codex-style /usage status + autonomous coding harness for pi. Not production ready — expect breaking changes.",
   "type": "module",
   "scripts": {

package/packages/a2a-adapter/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/a2a-adapter",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "type": "module",
   "description": "Google Agent-to-Agent (A2A) protocol adapter for pi-harness",
   "main": "dist/index.js",

package/packages/architecture-generator/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/architecture-generator",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "description": "Generate ADR and Mermaid diagrams from project analysis",
   "type": "module",
   "main": "./dist/index.js",

package/packages/auth/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/auth",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "description": "pi-harness-runtime — auth",
   "type": "module",
   "main": "./dist/index.js",

package/packages/autonomous-refactor/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/autonomous-refactor",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "type": "module",
   "description": "Autonomous refactoring engine",
   "main": "dist/index.js",

package/packages/autonomous-runtime/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/autonomous-runtime",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "description": "Autonomous Operations Runtime — task inbox, lease management, and worker lifecycle (RFC-0101)",
   "type": "module",
   "main": "./dist/index.js",

package/packages/cache-strategy/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/cache-strategy",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "type": "module",
   "description": "Smart caching with TTL and invalidation strategies (RFC-0022)",
   "main": "dist/index.js",

package/packages/capability-registry/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/capability-registry",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "description": "Capability Registry for pi-harness-runtime — tracks model capabilities and enables capability-based routing",
   "type": "module",
   "main": "./dist/index.js",

package/packages/checkpoint/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/checkpoint",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "description": "Enhanced checkpoint system with incremental snapshots, diff-based storage, and recovery strategies",
   "type": "module",
   "main": "./dist/index.js",

package/packages/cli-plugin-sdk/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/cli-plugin-sdk",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "type": "module",
   "description": "CLI Plugin SDK for pi-harness — install, list, remove, update, invoke plugins",
   "main": "dist/index.js",

package/packages/clipboard/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/clipboard",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "type": "module",
   "main": "src/index.ts",
   "exports": {

package/packages/clipboard-plugin/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/clipboard-plugin",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "type": "module",
   "main": "src/index.ts",
   "exports": {

package/packages/code-generation/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/code-generation",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "description": "Code generation pipeline for pi-harness-runtime",
   "type": "module",
   "main": "./dist/index.js",

package/packages/code-review/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/code-review",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "description": "Automated code review engine for pi-harness-runtime",
   "type": "module",
   "main": "./dist/index.js",

package/packages/codex-adapter/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/codex-adapter",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "type": "module",
   "description": "OpenAI Codex provider adapter for pi-harness",
   "main": "dist/index.js",

package/packages/config-capture/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@moocoding/config-capture",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "description": "Auto-capture API-fetched configuration values to AGENTS.md",
   "type": "module",
   "main": "src/index.ts",

package/packages/context-compiler/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/context-compiler",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "description": "Bounded context selection engine for pi-harness-runtime",
   "type": "module",
   "main": "./dist/index.js",

package/packages/context-discovery/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/context-discovery",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "type": "module",
   "main": "src/index.ts",
   "exports": {

package/packages/context-manager/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/context-manager",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "description": "Context Window Manager — prevents model context overflow via compaction and resume prompts (RFC-0010)",
   "type": "module",
   "main": "./dist/index.js",

package/packages/cookie-sanitizer/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/cookie-sanitizer",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "description": "pi-harness-runtime — cookie sanitizer (provider-quota cookie file normaliser)",
   "type": "module",
   "main": "./dist/index.js",

package/packages/cost-optimizer/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/cost-optimizer",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "description": "Cost Optimizer for pi-harness-runtime — budget tracking, forecasting, and model selection optimization",
   "type": "module",
   "main": "./dist/index.js",

package/packages/dependency-analyzer/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/dependency-analyzer",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "type": "module",
   "main": "src/index.ts",
   "exports": {

package/packages/django-plugin/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/django-plugin",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "description": "Django framework analysis — models, management commands, settings",
   "type": "module",
   "main": "./dist/index.js",

package/packages/doc-generator/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/doc-generator",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "description": "Auto-Doc Generator — generate documentation from source code (RFC-0014)",
   "type": "module",
   "main": "./dist/index.js",

package/packages/evaluation-engine/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/evaluation-engine",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "description": "Evaluation engine for pi-harness runtime (RFC-0057)",
   "type": "module",
   "main": "./dist/index.js",

package/packages/evaluation-runner/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/evaluation-runner",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "type": "module",
   "description": "Framework-agnostic evaluation framework for AI outputs (RFC-0020)",
   "main": "dist/index.js",

package/packages/event-bus/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/event-bus",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "type": "module",
   "main": "src/index.ts",
   "exports": {

package/packages/event-store/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/event-store",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "type": "module",
   "main": "src/index.ts",
   "exports": {

package/packages/experience-replay/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/experience-replay",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "description": "Experience replay for debugging and reproducibility (RFC-0059)",
   "type": "module",
   "main": "./dist/index.js",

package/packages/feedback-collector/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/feedback-collector",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "type": "module",
   "description": "Collect and aggregate human/automated feedback (RFC-0021)",
   "main": "dist/index.js",

package/packages/file-copy-helper/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@moocoding/file-copy-helper",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "description": "Injects cp/sudo cp rule when user asks to mimic/copy files",
   "type": "module",
   "main": "dist/index.js",

package/packages/framework-detector/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/framework-detector",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "description": "Framework detection system for pi-harness-runtime",
   "type": "module",
   "main": "./dist/index.js",

package/packages/framework-plugin-sdk/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/framework-plugin-sdk",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "description": "Framework plugin SDK for pi-harness-runtime",
   "type": "module",
   "main": "./dist/index.js",

package/packages/frappe-plugin/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/frappe-plugin",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "description": "Frappe/ERPNext framework analysis — DocTypes, hooks, bench sites, site config",
   "type": "module",
   "main": "./dist/index.js",

package/packages/generic-web-plugin/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/generic-web-plugin",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "description": "Generic web framework analysis — REST/GraphQL endpoints, OpenAPI, auth",
   "type": "module",
   "main": "./dist/index.js",

package/packages/health-monitor/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/health-monitor",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "type": "module",
   "description": "Health monitoring and auto-healing",
   "main": "dist/index.js",

package/packages/intent-analyzer/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/intent-analyzer",
-  "version": "1.1.49",
+  "version": "1.1.50",
   "type": "module",
   "main": "src/index.ts",
   "exports": {