pi-harness-runtime 1.1.406 → 1.1.408

package/index.ts

@@ -373,6 +373,78 @@ let _extensionApi: ExtensionAPI | null = null;
 /** How long an unanswered proposal stays quiet before it is offered again. */
 const TOOLING_PROPOSAL_COOLDOWN_MS = 30 * 24 * 60 * 60 * 1000;
 
+// --- two-phase-review: implementation first, review as a second phase -----
+
+/** The package of the two-phase workflow. */
+type TwoPhaseModule = typeof import("./packages/two-phase-review/dist/index.js");
+
+/** The todo shape the workflow needs. */
+type TwoPhaseTodoLike = import("./packages/two-phase-review/dist/index.js").TodoLike;
+
+/**
+ * Load the two-phase review package.
+ */
+async function loadTwoPhaseModule(): Promise<TwoPhaseModule | null> {
+	try {
+		return await import("./packages/two-phase-review/dist/index.js");
+	} catch (error) {
+		logToFile("[TwoPhase]", `package unavailable: ${oneLineError(error)}`);
+		return null;
+	}
+}
+
+/**
+ * Register the two-phase workflow in the session.
+ *
+ * The wiring itself lives in the package (it is tested there). The session
+ * only provides the todo list of the session and the workspace.
+ */
+async function initTwoPhaseReview(pi: ExtensionAPI): Promise<void> {
+	const mod = await loadTwoPhaseModule();
+	if (!mod) return;
+
+	/** Read the todo list of the session (the local one, never bd). */
+	const readTodos = async (ctx: unknown): Promise<{
+		available: boolean;
+		todos: TwoPhaseTodoLike[];
+		reason?: string;
+	}> => {
+		try {
+			const jev = await import("./packages/jev-judge/dist/index.js");
+			const provider = jev.createTodoProvider(ctx as never);
+			const availability = await provider.availability();
+			if (!availability.available) {
+				return {
+					available: false,
+					todos: [],
+					reason: availability.error ?? "the todo provider is not available",
+				};
+			}
+			const snapshot = await provider.getSnapshot();
+			return { available: true, todos: (snapshot.items ?? []) as TwoPhaseTodoLike[] };
+		} catch (error) {
+			return { available: false, todos: [], reason: oneLineError(error) };
+		}
+	};
+
+	try {
+		await mod.initTwoPhaseReview({
+			pi,
+			readTodos,
+			workspaceRoot: () => {
+				try {
+					return realpathSync(process.cwd());
+				} catch {
+					return process.cwd();
+				}
+			},
+			log: (message: string) => logToFile("[TwoPhase]", message),
+		});
+	} catch (error) {
+		logToFile("[TwoPhase]", `runtime not registered: ${oneLineError(error)}`);
+	}
+}
+
 /**
  * Load the tooling preference package and watch for the session start.
  *
@@ -530,6 +602,70 @@ async function startBiomeImplementation(
 	}
 }
 
+/** The last time a quality check ran - the check is not repeated in a loop. */
+let _toolingCheckAt = 0;
+/** Is a quality check running right now? */
+let _toolingCheckRunning = false;
+/** The shortest time between two quality checks. */
+const TOOLING_CHECK_INTERVAL_MS = 15_000;
+
+/**
+ * Run the post-change quality check of the workspace.
+ *
+ * The check honors the saved preference: Biome when the user asked for it,
+ * the project's own script otherwise. It never writes, it never blocks the
+ * session and it never runs without a decision of the user.
+ */
+async function maybeRunToolingCheck(ctx: ExtensionContext): Promise<void> {
+	const store = _toolingStore;
+	if (!store || _toolingCheckRunning) return;
+	const now = Date.now();
+	if (now - _toolingCheckAt < TOOLING_CHECK_INTERVAL_MS) return;
+
+	let workspaceRoot: string;
+	try {
+		workspaceRoot = realpathSync(process.cwd());
+	} catch {
+		return;
+	}
+
+	try {
+		const mod = await import("./packages/tooling-preference/dist/index.js");
+		const userKey = mod.buildToolingUserKey(readToolingIdentity());
+		const record = store.read(userKey, workspaceRoot);
+		// Without a decision of the user no tool is selected, so nothing runs.
+		if (!record?.decision) return;
+
+		const changed = mod.detectChangedJsTsFiles(workspaceRoot);
+		if (!changed.detected || changed.files.length === 0) return;
+
+		const context = mod.detectToolingContext(workspaceRoot);
+		const plan = mod.selectQualityCheck({
+			context,
+			preference: record.decision,
+			files: changed.files,
+		});
+		if (plan.tool === "none") return;
+
+		_toolingCheckRunning = true;
+		_toolingCheckAt = now;
+		try {
+			const result = mod.runQualityCheck(plan, {
+				cwd: workspaceRoot,
+				timeoutMs: 60_000,
+			});
+			const line = mod.formatQualityCheckStatus(result);
+			logToFile("[ToolingCheck]", line);
+			// The status line names the tool and says whether the check ran.
+			ctx.ui.setStatus("harness-tooling", line);
+		} finally {
+			_toolingCheckRunning = false;
+		}
+	} catch (error) {
+		logToFile("[ToolingCheck]", `check failed: ${oneLineError(error)}`);
+	}
+}
+
 async function initAutoContinue(pi: ExtensionAPI): Promise<void> {
 	addHarnessMessage("Jev", "initializing", "wait");
 	try {
@@ -3949,6 +4085,12 @@ export default async function (pi: ExtensionAPI) {
 	// The proposal is a decision of its own: it never pauses the work.
 	void initToolingPreference(pi);
 
+	// --- two-phase-review: implementation first, review as a second phase ----
+	// A review signal during the implementation is queued, never executed.
+	// The handlers are registered before the extension is ready, so a session
+	// that starts right away is already covered.
+	await initTwoPhaseReview(pi);
+
 	// --- telegram-notifications: Telegram bot for harness event notifications ----
 	// Collect promise so we can wait for initialization before showing startup notification
 	asyncInitPromises.push(
@@ -5988,6 +6130,9 @@ Run \`bd ready\` to see current bd issues.
 		void maybeAutoFetchQuota(ctx.model?.id ?? null);
 		maybeTriggerProactiveCompact(ctx);
 		maybeNotifyContextUsage(ctx);
+		// The quality check of the workspace runs after the turn: it never
+		// writes and it never steers the session.
+		void maybeRunToolingCheck(ctx);
 	});
 
 	pi.on("agent_end", () => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-harness-runtime",
-  "version": "1.1.406",
+  "version": "1.1.408",
   "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.406",
+  "version": "1.1.408",
   "type": "module",
   "description": "Google Agent-to-Agent (A2A) protocol adapter for pi-harness",
   "main": "dist/index.js",

package/packages/agent-policy/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/agent-policy",
-  "version": "1.1.406",
+  "version": "1.1.408",
   "description": "Agent policy runtime for pi-harness - enforces mandatory rule delivery and mutation gating",
   "type": "module",
   "main": "./dist/index.js",

package/packages/architecture-generator/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/architecture-generator",
-  "version": "1.1.406",
+  "version": "1.1.408",
   "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.406",
+  "version": "1.1.408",
   "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.406",
+  "version": "1.1.408",
   "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.406",
+  "version": "1.1.408",
   "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.406",
+  "version": "1.1.408",
   "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.406",
+  "version": "1.1.408",
   "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.406",
+  "version": "1.1.408",
   "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.406",
+  "version": "1.1.408",
   "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.406",
+  "version": "1.1.408",
   "type": "module",
   "main": "src/index.ts",
   "exports": {

package/packages/clipboard-plugin/package.json

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

package/packages/code-generation/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/code-generation",
-  "version": "1.1.406",
+  "version": "1.1.408",
   "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.406",
+  "version": "1.1.408",
   "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.406",
+  "version": "1.1.408",
   "type": "module",
   "description": "OpenAI Codex provider adapter for pi-harness",
   "main": "dist/index.js",

package/packages/codex-watcher/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/codex-watcher",
-  "version": "1.1.406",
+  "version": "1.1.408",
   "description": "Watches Codex CLI sessions and emits visible messages for PING-PONG monitoring",
   "type": "module",
   "main": "./dist/src/index.js",

package/packages/config-capture/package.json

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

package/packages/connection-runtime/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/connection-runtime",
-  "version": "1.1.406",
+  "version": "1.1.408",
   "type": "module",
   "description": "Connection management with jittered exponential backoff for pi-harness",
   "main": "./dist/index.js",

package/packages/context-compiler/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/context-compiler",
-  "version": "1.1.406",
+  "version": "1.1.408",
   "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.406",
+  "version": "1.1.408",
   "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.406",
+  "version": "1.1.408",
   "description": "Context Window Manager — prevents model context overflow via compaction and resume prompts (RFC-0010)",
   "type": "module",
   "main": "./dist/index.js",

package/packages/contracts/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/contracts",
-  "version": "1.1.406",
+  "version": "1.1.408",
   "type": "module",
   "description": "RPC schema definitions for pi-harness runtime",
   "main": "./dist/index.js",

package/packages/cookie-sanitizer/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/cookie-sanitizer",
-  "version": "1.1.406",
+  "version": "1.1.408",
   "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.406",
+  "version": "1.1.408",
   "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.406",
+  "version": "1.1.408",
   "type": "module",
   "main": "src/index.ts",
   "exports": {

package/packages/django-plugin/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/django-plugin",
-  "version": "1.1.406",
+  "version": "1.1.408",
   "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.406",
+  "version": "1.1.408",
   "description": "Auto-Doc Generator — generate documentation from source code (RFC-0014)",
   "type": "module",
   "main": "./dist/index.js",

package/packages/durable-review/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@moocoding/durable-review",
-  "version": "1.1.406",
+  "version": "1.1.408",
   "description": "SQLite-backed durable reviewer using @earendil-works/pi-durable",
   "type": "module",
   "main": "src/index.ts",

package/packages/e2e-tester/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/e2e-tester",
-  "version": "1.1.406",
+  "version": "1.1.408",
   "description": "WebMCP-powered E2E testing for pi-harness - natural language test scenarios",
   "type": "module",
   "main": "./dist/index.js",

package/packages/evaluation-engine/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/evaluation-engine",
-  "version": "1.1.406",
+  "version": "1.1.408",
   "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.406",
+  "version": "1.1.408",
   "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.406",
+  "version": "1.1.408",
   "type": "module",
   "main": "src/index.ts",
   "exports": {

package/packages/event-sink/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/event-sink",
-  "version": "1.1.406",
+  "version": "1.1.408",
   "type": "module",
   "description": "Atomic event sourcing with SQLite transactions for pi-harness",
   "main": "./dist/index.js",

package/packages/event-store/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/event-store",
-  "version": "1.1.406",
+  "version": "1.1.408",
   "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.406",
+  "version": "1.1.408",
   "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.406",
+  "version": "1.1.408",
   "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.406",
+  "version": "1.1.408",
   "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.406",
+  "version": "1.1.408",
   "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.406",
+  "version": "1.1.408",
   "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.406",
+  "version": "1.1.408",
   "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.406",
+  "version": "1.1.408",
   "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.406",
+  "version": "1.1.408",
   "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.406",
+  "version": "1.1.408",
   "type": "module",
   "main": "src/index.ts",
   "exports": {

package/packages/jev-judge/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/jev-judge",
-  "version": "1.1.406",
+  "version": "1.1.408",
   "description": "Jev Judge - Structured decision model for agent routing, E2E testing, and automated decisions",
   "type": "module",
   "main": "./dist/index.js",

package/packages/knowledge-graph/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/knowledge-graph",
-  "version": "1.1.406",
+  "version": "1.1.408",
   "description": "Knowledge Graph extraction and provenance linking for SKILL.md files",
   "type": "module",
   "main": "./dist/index.js",

package/packages/knowledge-retrieval/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/knowledge-retrieval",
-  "version": "1.1.406",
+  "version": "1.1.408",
   "description": "Knowledge retrieval via TencentDB-Agent-Memory MCP (RFC-0105)",
   "type": "module",
   "main": "./dist/index.js",

package/packages/laravel-plugin/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/laravel-plugin",
-  "version": "1.1.406",
+  "version": "1.1.408",
   "description": "Laravel framework analysis — artisan commands, Eloquent models, Blade",
   "type": "module",
   "main": "./dist/index.js",

package/packages/learning-engine/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/learning-engine",
-  "version": "1.1.406",
+  "version": "1.1.408",
   "description": "Learning engine for extracting lessons from runtime execution (RFC-0058)",
   "type": "module",
   "main": "./dist/index.js",

package/packages/logging/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/logging",
-  "version": "1.1.406",
+  "version": "1.1.408",
   "type": "module",
   "description": "Centralized structured application logging with buffered Cloudflare R2 and webhook transports",
   "main": "./dist/index.js",

package/packages/mcp-adapter/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/mcp-adapter",
-  "version": "1.1.406",
+  "version": "1.1.408",
   "type": "module",
   "description": "MCP server adapter for pi-harness — exposes harness capabilities as MCP tools/resources/prompts (RFC-0068)",
   "main": "dist/index.js",