ultimate-pi 0.20.0 → 0.22.0

package/.pi/auto-commit.json

@@ -2,7 +2,8 @@
 	"dryRun": false,
 	"coAuthor": {
 		"login": "pi-mono",
-		"email": "261679550+pi-mono@users.noreply.github.com"
+		"email": "261679550+pi-mono@users.noreply.github.com",
+		"required": true
 	},
 	"branch": {
 		"strategy": "auto-feature-branch",
@@ -15,6 +16,12 @@
 		"ignore": true
 	},
 	"message": {
-		"scopeDefault": "harness"
+		"template": "{type}({scope}): {subject}",
+		"templateNoScope": "{type}: {subject}",
+		"typeDefault": "chore",
+		"scopeDefault": "harness",
+		"bodySeparator": "\n\n",
+		"coAuthorTrailer": "Co-authored-by: {login} <{email}>",
+		"maxSubjectLength": 72
 	}
 }

package/.pi/extensions/debate-orchestrator.ts

@@ -18,6 +18,7 @@ import {
 } from "../lib/debate-bus-state.js";
 import { isHarnessProjectEnabled } from "../lib/harness-project-config.js";
 import { getRunIdFromSession } from "../lib/harness-run-context.js";
+import { completeDebateOpen } from "../lib/harness-slash-completions.js";
 import { normalizePlanDebateId } from "../lib/plan-debate-id.js";
 import { initPlanMessenger } from "../lib/plan-messenger.js";
 
@@ -55,6 +56,8 @@ export default function debateOrchestrator(pi: ExtensionAPI) {
 
 	pi.registerCommand("harness-debate-open", {
 		description: "Open a headless debate session",
+		getArgumentCompletions: (prefix) =>
+			completeDebateOpen(prefix, process.cwd()),
 		handler: async (args, ctx) => {
 			const runId = getRunId(ctx);
 			const trimmed = args.trim();

package/.pi/extensions/harness-anchored-edit.ts

@@ -85,14 +85,10 @@ export default function harnessAnchoredEdit(pi: ExtensionAPI): void {
 		parameters: readSchema,
 		async execute(toolCallId, params, signal, onUpdate, ctx) {
 			const base = getReadTool(ctx.cwd);
-			const result = await base.execute(
-				toolCallId,
-				params,
-				signal,
-				onUpdate,
-				ctx,
-			);
-			const taskId = anchoredEditTaskId(ctx);
+			const result = await base.execute(toolCallId, params, signal, onUpdate);
+			const taskId = anchoredEditTaskId({
+				sessionId: (ctx as { sessionId?: string }).sessionId,
+			});
 			const absolutePath = resolve(ctx.cwd, params.path);
 			for (const block of result.content) {
 				if (block.type !== "text") continue;
@@ -116,7 +112,9 @@ export default function harnessAnchoredEdit(pi: ExtensionAPI): void {
 		],
 		async execute(_toolCallId, params, _signal, _onUpdate, ctx) {
 			const absolutePath = resolve(ctx.cwd, params.path);
-			const taskId = anchoredEditTaskId(ctx);
+			const taskId = anchoredEditTaskId({
+				sessionId: (ctx as { sessionId?: string }).sessionId,
+			});
 			const edits = params.edits as AnchoredEdit[];
 
 			const result = await applyAnchoredEditsToFile(

package/.pi/extensions/harness-ask-user.ts

@@ -1,23 +1,16 @@
 /**
  * harness-ask-user — structured user decisions for harness planning and setup.
- * Design references: pi-ask-user, @pi-unipi/ask-user, rpiv-ask-user-question (not vendored).
  */
 
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
-import { runAskDialog } from "../lib/ask-user/dialog.js";
-import { runAskFallback } from "../lib/ask-user/fallback.js";
+import { runAskUser } from "../lib/ask-user/index.js";
 import { renderAskCall, renderAskResult } from "../lib/ask-user/render.js";
 import {
 	AskUserParamsSchema,
 	PROMPT_GUIDELINES,
 	PROMPT_SNIPPET,
 } from "../lib/ask-user/schema.js";
-import type { AskUserParams, DialogResult } from "../lib/ask-user/types.js";
-import {
-	formatResultText,
-	toToolDetails,
-	validateAskParams,
-} from "../lib/ask-user/validate.js";
+import type { AskUserParams } from "../lib/ask-user/types.js";
 import { claimHarnessGovernanceLoad } from "../lib/extension-load-guard.js";
 
 // @ts-expect-error pi extensions run as ESM
@@ -35,36 +28,22 @@ export default function harnessAskUser(pi: ExtensionAPI) {
 		parameters: AskUserParamsSchema,
 
 		async execute(_toolCallId, params, _signal, _onUpdate, ctx) {
-			const validated = validateAskParams(params as AskUserParams);
-			if (typeof validated === "string") {
+			const result = await runAskUser(params as AskUserParams, {
+				ui: ctx.ui,
+				hasUI: ctx.hasUI,
+				sessionName: undefined,
+			});
+
+			if ("error" in result) {
 				return {
-					content: [{ type: "text", text: validated }],
-					details: {
-						question: params.question ?? "",
-						options: [],
-						response: null,
-						cancelled: true,
-					},
+					content: [{ type: "text", text: result.error }],
+					details: result.details,
 				};
 			}
 
-			let outcome: DialogResult;
-			if (ctx.hasUI) {
-				outcome = await runAskDialog(ctx.ui, validated);
-			} else {
-				outcome = await runAskFallback(ctx.ui, validated);
-			}
-
-			const details = toToolDetails(
-				validated,
-				outcome.response,
-				outcome.cancelled,
-			);
-			const text = formatResultText(outcome.response, outcome.cancelled);
-
 			return {
-				content: [{ type: "text", text }],
-				details,
+				content: result.content,
+				details: result.details,
 			};
 		},
 

package/.pi/extensions/harness-debate-tools.ts

@@ -51,7 +51,6 @@ import {
 	withReviewRoundYamlWrite,
 } from "../lib/harness-debate-workflow-deps.js";
 
-// @ts-expect-error pi extensions run as ESM
 const MODULE_URL = import.meta.url;
 
 function getRunId(ctx: {
@@ -98,9 +97,7 @@ function subagentResults(
 
 const USE_SUBMIT_TOOLS = process.env.HARNESS_SUBMIT_TOOLS !== "0";
 
-export default function harnessDebateTools(pi: ExtensionAPI) {
-	if (!claimHarnessGovernanceLoad("harness-debate-tools", MODULE_URL)) return;
-
+function registerHarnessDebateHandler1(pi: ExtensionAPI) {
 	pi.on("tool_result", async (event, ctx) => {
 		if (event.isError || event.toolName !== "subagent") return;
 		const runId = getRunId(ctx);
@@ -171,7 +168,9 @@ export default function harnessDebateTools(pi: ExtensionAPI) {
 			details: { applied, status },
 		});
 	});
+}
 
+function registerHarnessDebateHandler2(pi: ExtensionAPI) {
 	pi.registerTool({
 		name: "harness_plan_debate_eligibility",
 		label: "Plan Debate Eligibility",
@@ -247,7 +246,9 @@ export default function harnessDebateTools(pi: ExtensionAPI) {
 			};
 		},
 	});
+}
 
+function registerHarnessDebateHandler3(pi: ExtensionAPI) {
 	pi.registerTool({
 		name: "harness_debate_open",
 		label: "Open Plan Debate",
@@ -332,7 +333,9 @@ export default function harnessDebateTools(pi: ExtensionAPI) {
 			};
 		},
 	});
+}
 
+function registerHarnessDebateHandler4(pi: ExtensionAPI) {
 	pi.registerTool({
 		name: "harness_messenger_post",
 		label: "Post Debate Messenger Message",
@@ -397,7 +400,9 @@ export default function harnessDebateTools(pi: ExtensionAPI) {
 			};
 		},
 	});
+}
 
+function registerHarnessDebateHandler5(pi: ExtensionAPI) {
 	pi.registerTool({
 		name: "harness_messenger_read_round",
 		label: "Read Debate Round Transcript",
@@ -422,7 +427,9 @@ export default function harnessDebateTools(pi: ExtensionAPI) {
 			};
 		},
 	});
+}
 
+function registerHarnessDebateHandler6(pi: ExtensionAPI) {
 	pi.registerTool({
 		name: "harness_debate_submit_round",
 		label: "Submit Plan Review Round",
@@ -551,7 +558,9 @@ export default function harnessDebateTools(pi: ExtensionAPI) {
 			};
 		},
 	});
+}
 
+function registerHarnessDebateHandler7(pi: ExtensionAPI) {
 	pi.registerTool({
 		name: "harness_debate_consensus",
 		label: "Finalize Plan Debate Consensus",
@@ -590,7 +599,9 @@ export default function harnessDebateTools(pi: ExtensionAPI) {
 			};
 		},
 	});
+}
 
+function registerHarnessDebateHandler8(pi: ExtensionAPI) {
 	pi.registerTool({
 		name: "harness_debate_apply_lane",
 		label: "Apply Debate Lane YAML + Messenger",
@@ -624,7 +635,9 @@ export default function harnessDebateTools(pi: ExtensionAPI) {
 			};
 		},
 	});
+}
 
+function registerHarnessDebateHandler9(pi: ExtensionAPI) {
 	pi.registerTool({
 		name: "harness_debate_round_status",
 		label: "Plan Debate Round Status",
@@ -669,7 +682,9 @@ export default function harnessDebateTools(pi: ExtensionAPI) {
 			};
 		},
 	});
+}
 
+function registerHarnessDebateHandler10(pi: ExtensionAPI) {
 	pi.registerTool({
 		name: "harness_debate_focus_coverage",
 		label: "Plan Debate Focus Coverage",
@@ -712,7 +727,9 @@ export default function harnessDebateTools(pi: ExtensionAPI) {
 			};
 		},
 	});
+}
 
+function registerHarnessDebateHandler11(pi: ExtensionAPI) {
 	pi.registerTool({
 		name: "harness_debate_advance_thread",
 		label: "Advance Plan Debate Thread",
@@ -746,7 +763,9 @@ export default function harnessDebateTools(pi: ExtensionAPI) {
 			};
 		},
 	});
+}
 
+function registerHarnessDebateHandler12(pi: ExtensionAPI) {
 	pi.registerTool({
 		name: "harness_plan_scope_check",
 		label: "Plan Scope Drift Check",
@@ -771,3 +790,23 @@ export default function harnessDebateTools(pi: ExtensionAPI) {
 		},
 	});
 }
+
+function registerHarnessDebateToolHandlers(pi: ExtensionAPI) {
+	registerHarnessDebateHandler1(pi);
+	registerHarnessDebateHandler2(pi);
+	registerHarnessDebateHandler3(pi);
+	registerHarnessDebateHandler4(pi);
+	registerHarnessDebateHandler5(pi);
+	registerHarnessDebateHandler6(pi);
+	registerHarnessDebateHandler7(pi);
+	registerHarnessDebateHandler8(pi);
+	registerHarnessDebateHandler9(pi);
+	registerHarnessDebateHandler10(pi);
+	registerHarnessDebateHandler11(pi);
+	registerHarnessDebateHandler12(pi);
+}
+
+export default function harnessDebateTools(pi: ExtensionAPI) {
+	if (!claimHarnessGovernanceLoad("harness-debate-tools", MODULE_URL)) return;
+	registerHarnessDebateToolHandlers(pi);
+}

package/.pi/extensions/harness-live-widget.ts

@@ -84,28 +84,37 @@ function visibleWidth(input: string): number {
 	return width;
 }
 
+const WIDE_SINGLE_CODE_POINTS = new Set([0x2329, 0x232a]);
+const WIDE_RANGES: ReadonlyArray<readonly [number, number]> = [
+	[0x1100, 0x115f],
+	[0x2e80, 0xa4cf],
+	[0xac00, 0xd7a3],
+	[0xf900, 0xfaff],
+	[0xfe10, 0xfe19],
+	[0xfe30, 0xfe6f],
+	[0xff00, 0xff60],
+	[0xffe0, 0xffe6],
+	[0x1f300, 0x1faff],
+	[0x20000, 0x3fffd],
+];
+
+function inRange(codePoint: number, start: number, end: number): boolean {
+	return codePoint >= start && codePoint <= end;
+}
+
+function isWideCodePoint(codePoint: number): boolean {
+	if (WIDE_SINGLE_CODE_POINTS.has(codePoint)) return true;
+	if (inRange(codePoint, 0x2e80, 0xa4cf) && codePoint === 0x303f) return false;
+	return WIDE_RANGES.some(([start, end]) => inRange(codePoint, start, end));
+}
+
 function charDisplayWidth(char: string): number {
 	const codePoint = char.codePointAt(0);
 	if (codePoint == null) return 0;
-	if (codePoint <= 0x1f || (codePoint >= 0x7f && codePoint <= 0x9f)) return 0;
-	if (codePoint >= 0x300 && codePoint <= 0x36f) return 0;
-	if (
-		codePoint >= 0x1100 &&
-		(codePoint <= 0x115f ||
-			codePoint === 0x2329 ||
-			codePoint === 0x232a ||
-			(codePoint >= 0x2e80 && codePoint <= 0xa4cf && codePoint !== 0x303f) ||
-			(codePoint >= 0xac00 && codePoint <= 0xd7a3) ||
-			(codePoint >= 0xf900 && codePoint <= 0xfaff) ||
-			(codePoint >= 0xfe10 && codePoint <= 0xfe19) ||
-			(codePoint >= 0xfe30 && codePoint <= 0xfe6f) ||
-			(codePoint >= 0xff00 && codePoint <= 0xff60) ||
-			(codePoint >= 0xffe0 && codePoint <= 0xffe6) ||
-			(codePoint >= 0x1f300 && codePoint <= 0x1faff) ||
-			(codePoint >= 0x20000 && codePoint <= 0x3fffd))
-	) {
-		return 2;
-	}
+	if (inRange(codePoint, 0x00, 0x1f) || inRange(codePoint, 0x7f, 0x9f))
+		return 0;
+	if (inRange(codePoint, 0x300, 0x36f)) return 0;
+	if (isWideCodePoint(codePoint)) return 2;
 	return 1;
 }