pi-harness-runtime 1.1.404 → 1.1.405

package/packages/tooling-preference/tests/registry.test.ts

@@ -0,0 +1,51 @@
+import { describe, expect, it } from "bun:test";
+import { ToolingRequestRegistry } from "../src/registry.ts";
+
+const request = {
+	requestId: "biome-abc123",
+	workspaceRoot: "/work/web",
+	userKey: "user-1",
+	frameworks: ["next"],
+};
+
+describe("ToolingRequestRegistry", () => {
+	it("registers a proposal once", () => {
+		const registry = new ToolingRequestRegistry();
+		expect(registry.register(request)).toBe(true);
+		expect(registry.register(request)).toBe(false);
+		expect(registry.has(request.requestId)).toBe(true);
+	});
+
+	it("lets the first answer win", () => {
+		const registry = new ToolingRequestRegistry();
+		registry.register(request);
+
+		const first = registry.claim(request.requestId, "yes", "telegram");
+		expect(first?.resolution.preference).toBe("prefer_biome");
+		expect(first?.request.workspaceRoot).toBe("/work/web");
+
+		// The second click finds nothing - the first answer stays.
+		expect(registry.claim(request.requestId, "no", "line")).toBeNull();
+		expect(registry.resolutionOf(request.requestId)?.preference).toBe("prefer_biome");
+	});
+
+	it("maps a no to the current tools", () => {
+		const registry = new ToolingRequestRegistry();
+		registry.register(request);
+		const claimed = registry.claim(request.requestId, "no", "line");
+		expect(claimed?.resolution.preference).toBe("use_project_tools");
+		expect(claimed?.resolution.channel).toBe("line");
+	});
+
+	it("refuses a claim for an unknown request", () => {
+		const registry = new ToolingRequestRegistry();
+		expect(registry.claim("biome-unknown", "yes")).toBeNull();
+	});
+
+	it("does not register a request that was already answered", () => {
+		const registry = new ToolingRequestRegistry();
+		registry.register(request);
+		registry.claim(request.requestId, "no");
+		expect(registry.register(request)).toBe(false);
+	});
+});

package/packages/tooling-preference/tests/store.test.ts

@@ -0,0 +1,99 @@
+import { describe, expect, it } from "bun:test";
+import { mkdtempSync, rmSync } from "node:fs";
+import { tmpdir } from "node:os";
+import { join } from "node:path";
+import {
+	shouldAskAgain,
+	ToolingPreferenceStore,
+	toolingPreferenceKey,
+} from "../src/store.ts";
+
+function storePath(): string {
+	return join(mkdtempSync(join(tmpdir(), "tooling-store-")), "tooling-preference.json");
+}
+
+describe("ToolingPreferenceStore", () => {
+	it("has no record for an unknown user and workspace", () => {
+		const path = storePath();
+		const store = new ToolingPreferenceStore(path);
+		expect(store.read("user-1", "/work/web")).toBeUndefined();
+	});
+
+	it("keeps the record of a user in a workspace apart", () => {
+		const path = storePath();
+		const store = new ToolingPreferenceStore(path);
+		store.recordDecision("user-1", "/work/web", "prefer_biome", "biome-1");
+		store.recordDecision("user-2", "/work/web", "use_project_tools", "biome-2");
+		store.recordDecision("user-1", "/work/other", "use_project_tools", "biome-3");
+
+		expect(store.read("user-1", "/work/web")?.decision).toBe("prefer_biome");
+		expect(store.read("user-2", "/work/web")?.decision).toBe("use_project_tools");
+		expect(store.read("user-1", "/work/other")?.decision).toBe("use_project_tools");
+	});
+
+	it("remembers that the proposal was sent", () => {
+		const path = storePath();
+		const store = new ToolingPreferenceStore(path);
+		store.markAsked("user-1", "/work/web", "biome-1");
+		const record = store.read("user-1", "/work/web");
+		expect(record?.askedAt).toBeTruthy();
+		expect(record?.requestId).toBe("biome-1");
+		expect(record?.decision).toBeUndefined();
+	});
+
+	it("keeps a decision across a new store instance", () => {
+		const path = storePath();
+		new ToolingPreferenceStore(path).recordDecision(
+			"user-1",
+			"/work/web",
+			"prefer_biome",
+			"biome-1",
+			"telegram",
+		);
+		const record = new ToolingPreferenceStore(path).read("user-1", "/work/web");
+		expect(record?.decision).toBe("prefer_biome");
+		expect(record?.channel).toBe("telegram");
+	});
+
+	it("survives a broken store file", () => {
+		const path = storePath();
+		const store = new ToolingPreferenceStore(path);
+		store.markAsked("user-1", "/work/web", "biome-1");
+		// A store that cannot be read must not stop the harness.
+		Bun.write(path, "not json");
+		expect(store.read("user-1", "/work/web")).toBeUndefined();
+	});
+
+	it("keys a record by the user and the workspace", () => {
+		expect(toolingPreferenceKey("user-1", "/work/web")).not.toBe(
+			toolingPreferenceKey("user-2", "/work/web"),
+		);
+		expect(toolingPreferenceKey("user-1", "/work/web")).toBe(
+			toolingPreferenceKey("user-1", "/work/web"),
+		);
+	});
+});
+
+describe("shouldAskAgain", () => {
+	const day = 24 * 60 * 60 * 1000;
+	const now = new Date("2026-10-24T12:00:00.000Z");
+
+	it("asks a workspace that was never asked", () => {
+		expect(shouldAskAgain(undefined, 30 * day, now)).toBe(true);
+	});
+
+	it("never asks a workspace with a decision", () => {
+		const record = { decision: "prefer_biome" as const, askedAt: "2026-01-01T00:00:00.000Z" };
+		expect(shouldAskAgain(record, 30 * day, now)).toBe(false);
+	});
+
+	it("does not ask again inside the cooldown", () => {
+		const record = { askedAt: new Date(now.getTime() - day).toISOString() };
+		expect(shouldAskAgain(record, 30 * day, now)).toBe(false);
+	});
+
+	it("asks again after the cooldown", () => {
+		const record = { askedAt: new Date(now.getTime() - 31 * day).toISOString() };
+		expect(shouldAskAgain(record, 30 * day, now)).toBe(true);
+	});
+});

package/packages/trajectory/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/trajectory",
-  "version": "1.1.404",
+  "version": "1.1.405",
   "description": "Trajectory capture for the write-review loop — persists cycle records for learning",
   "type": "module",
   "main": "./dist/src/index.js",

package/packages/tui/package.json

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

package/packages/types/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/types",
-  "version": "1.1.404",
+  "version": "1.1.405",
   "description": "Shared TypeScript types for pi-harness-runtime packages",
   "type": "module",
   "main": "./dist/index.js",

package/packages/voice-api/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/voice-api",
-  "version": "1.1.404",
+  "version": "1.1.405",
   "type": "module",
   "description": "Voice-to-text API using Whisper for Telegram and LINE integration",
   "main": "./dist/index.js",

package/packages/webmcp-tester/package.json

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

package/packages/wiki/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/wiki",
-  "version": "1.1.404",
+  "version": "1.1.405",
   "description": "LLM Wiki integration for pi-harness-runtime",
   "type": "module",
   "main": "./dist/index.js",

package/packages/workflow-events/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/workflow-events",
-  "version": "1.1.404",
+  "version": "1.1.405",
   "type": "module",
   "main": "src/index.ts",
   "exports": {

package/packages/workspace-scanner/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/workspace-scanner",
-  "version": "1.1.404",
+  "version": "1.1.405",
   "type": "module",
   "main": "src/index.ts",
   "exports": {

package/packages/worktree/package.json

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

package/packages/write-review/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@moocoding/write-review",
-  "version": "1.1.404",
+  "version": "1.1.405",
   "description": "Two-agent write-review loop with blackboard coordination",
   "type": "module",
   "main": "src/index.ts",