pi-harness-runtime 1.1.439 → 1.1.440

package/harness/e2e/auto-test-loop.ts

@@ -100,6 +100,9 @@ export async function runAutoTestLoop(
 	);
 
 	// Handle decision
+	const runTests = () =>
+		runTestLoop(cfg, toolsConfig, retriesAttempted, startTime);
+
 	switch (decision.suggestedAction) {
 		case "skip":
 			logInfo(null, "Skipping E2E tests - not needed right now");
@@ -130,12 +133,13 @@ export async function runAutoTestLoop(
 				logInfo(null, "Generating test stubs...");
 				await generateTestStubs(cfg.projectRoot, toolsConfig.projectType);
 			}
-		// Fall through to run tests
+			// The stubs are the extra step; the test loop runs in both cases.
+			return runTests();
 
 		case "auto_run":
 		default:
 			// Run the test loop
-			return await runTestLoop(cfg, toolsConfig, retriesAttempted, startTime);
+			return runTests();
 	}
 }
 

package/harness/e2e/tools-detector.test.ts

@@ -2,7 +2,6 @@
  * E2E Tools Detector Tests
  */
 
-// @ts-expect-error - bun:test types
 import { describe, test, expect, beforeEach, afterEach } from "bun:test";
 import {
 	detectProjectType,

package/harness/index.ts

@@ -44,7 +44,8 @@ export type {
 export type { CheckpointManager } from "./job-state-machine.js";
 
 // Task Graph
-export type { TaskGraphManager } from "./task-graph.js";
+export { TaskGraphManager } from "./task-graph.js";
+export type { TaskDescription, TaskGraphOptions } from "./task-graph.js";
 
 // Loop Runtime
 export { LoopRuntime } from "./loop-runtime.js";

package/harness/langchain/scoreboard.test.ts

@@ -47,8 +47,10 @@ describe("T15 — Scoreboard blackboard wiring", () => {
 			code: "",
 			review: null as any,
 			log: [],
-			lastCommentedFile: undefined,
-			seenCommentPairs: [],
+			prevCode: undefined,
+			writtenFiles: {},
+			userInput: null,
+			userResponse: null,
 		});
 
 		const statusPath = join(blackboardDir, "status.json");
@@ -68,8 +70,10 @@ describe("T15 — Scoreboard blackboard wiring", () => {
 			code: "",
 			review: null as any,
 			log: [],
-			lastCommentedFile: undefined,
-			seenCommentPairs: [],
+			prevCode: undefined,
+			writtenFiles: {},
+			userInput: null,
+			userResponse: null,
 		});
 
 		const content = JSON.parse(
@@ -92,8 +96,10 @@ describe("T15 — Scoreboard blackboard wiring", () => {
 			code: "",
 			review: null as any,
 			log: [],
-			lastCommentedFile: undefined,
-			seenCommentPairs: [],
+			prevCode: undefined,
+			writtenFiles: {},
+			userInput: null,
+			userResponse: null,
 		});
 
 		deps.onStep?.("write", {
@@ -103,8 +109,10 @@ describe("T15 — Scoreboard blackboard wiring", () => {
 			code: "``` harness/langchain/surge.ts\nconst x = 1;\n```",
 			review: null as any,
 			log: [],
-			lastCommentedFile: undefined,
-			seenCommentPairs: [],
+			prevCode: undefined,
+			writtenFiles: {},
+			userInput: null,
+			userResponse: null,
 		});
 
 		const content = JSON.parse(
@@ -129,8 +137,10 @@ describe("T15 — Scoreboard blackboard wiring", () => {
 			code: "",
 			review: null as any,
 			log: [],
-			lastCommentedFile: undefined,
-			seenCommentPairs: [],
+			prevCode: undefined,
+			writtenFiles: {},
+			userInput: null,
+			userResponse: null,
 		});
 
 		deps.onStep?.("write", {
@@ -140,8 +150,10 @@ describe("T15 — Scoreboard blackboard wiring", () => {
 			code: "```harness/langchain/surge.ts\nconst x = 1;\n```",
 			review: null as any,
 			log: [],
-			lastCommentedFile: undefined,
-			seenCommentPairs: [],
+			prevCode: undefined,
+			writtenFiles: {},
+			userInput: null,
+			userResponse: null,
 		});
 
 		deps.onStep?.("review", {
@@ -157,8 +169,10 @@ describe("T15 — Scoreboard blackboard wiring", () => {
 				],
 			},
 			log: [],
-			lastCommentedFile: undefined,
-			seenCommentPairs: [],
+			prevCode: undefined,
+			writtenFiles: {},
+			userInput: null,
+			userResponse: null,
 		});
 
 		const content = JSON.parse(
@@ -186,8 +200,10 @@ describe("T15 — Scoreboard blackboard wiring", () => {
 			code: "",
 			review: null as any,
 			log: [],
-			lastCommentedFile: undefined,
-			seenCommentPairs: [],
+			prevCode: undefined,
+			writtenFiles: {},
+			userInput: null,
+			userResponse: null,
 		});
 
 		// Call plan() — the stub returns a string that includes scoreboard context
@@ -212,8 +228,10 @@ describe("T15 — Scoreboard blackboard wiring", () => {
 			code: "",
 			review: null as any,
 			log: [],
-			lastCommentedFile: undefined,
-			seenCommentPairs: [],
+			prevCode: undefined,
+			writtenFiles: {},
+			userInput: null,
+			userResponse: null,
 		});
 		deps.onStep?.("write", {
 			request: "test",
@@ -222,8 +240,10 @@ describe("T15 — Scoreboard blackboard wiring", () => {
 			code: "```a.ts\n1\n```",
 			review: null as any,
 			log: [],
-			lastCommentedFile: undefined,
-			seenCommentPairs: [],
+			prevCode: undefined,
+			writtenFiles: {},
+			userInput: null,
+			userResponse: null,
 		});
 		deps.onStep?.("review", {
 			request: "test",
@@ -232,8 +252,10 @@ describe("T15 — Scoreboard blackboard wiring", () => {
 			code: "```a.ts\n1\n```",
 			review: { verdict: "approved", summary: "ok", comments: [] },
 			log: [],
-			lastCommentedFile: undefined,
-			seenCommentPairs: [],
+			prevCode: undefined,
+			writtenFiles: {},
+			userInput: null,
+			userResponse: null,
 		});
 
 		const content = JSON.parse(

package/harness/master-planner.ts

@@ -6,7 +6,7 @@
  */
 
 import type { TaskGraph } from "../packages/types/dist/runtime-types.js";
-import type { TaskGraphManager } from "./task-graph.ts";
+import { TaskGraphManager } from "./task-graph.js";
 
 export interface PlanResult {
 	success: boolean;

package/harness/output-limit-handler.ts

@@ -259,7 +259,7 @@ export class OutputLimitHandler {
 		}
 
 		// Basic validation: content should be longer than partial
-		return content.length > this.getPartials()[0]?.length ?? 0;
+		return content.length > (this.getPartials()[0]?.length ?? 0);
 	}
 
 	/**

package/harness/project-detector/detector.ts

@@ -10,8 +10,8 @@ import type {
 	ProjectDetection,
 	SeedStrategy,
 	E2EStrategy,
-} from "../packages/types/dist/runtime-types.js";
-import { readJson } from "../../cli.ts";
+} from "../../packages/types/dist/runtime-types.js";
+import { readJson } from "../../cli.js";
 import { join } from "node:path";
 
 export interface DetectionSignal {
@@ -190,9 +190,7 @@ export class ProjectDetector {
 				"database/seeders",
 			];
 
-			// @ts-expect-error - Bun has built-in file system access
 			const { existsSync } = await import("node:fs");
-			// @ts-expect-error - Bun has built-in file system access
 			const { readdirSync } = await import("node:fs");
 
 			for (const check of checks) {