pi-interview 0.5.4 → 0.5.5

package/README.md

@@ -63,7 +63,7 @@ Restart pi to load the extension.
 
 **Timeout behavior:** The countdown (visible in corner) resets on any activity - typing, clicking, or mouse movement. When it expires, an overlay appears giving the user a chance to continue. Progress is never lost thanks to localStorage auto-save.
 
-**Multi-agent behavior:** When multiple agents run interviews simultaneously, only the first auto-opens the window. Subsequent interviews are queued and shown as a URL in the tool output, preventing focus stealing. Active interviews also surface a top-right toast with a dropdown to open queued sessions. A session status bar at the top of each form shows the project path, git branch, and session ID for easy identification.
+**Multi-agent behavior:** When multiple agents run interviews simultaneously, only the first auto-opens the window. Subsequent interviews are queued and shown as a URL in the tool output, preventing focus stealing. When you submit the active interview, the window automatically redirects to the next queued interview. Active interviews also surface a top-right toast with a dropdown to open queued sessions. A session status bar at the top of each form shows the project path, git branch, and session ID for easy identification.
 
 ## Usage
 

package/form/script.js

@@ -2405,13 +2405,21 @@
         body: JSON.stringify({ token: sessionToken, ...payload }),
       });
 
-      const result = await response.json().catch(() => ({ ok: false, error: "Invalid server response" }));
+      let submitResult;
+      try {
+        submitResult = await response.json();
+      } catch (err) {
+        const message = err instanceof Error ? err.message : String(err);
+        showGlobalError(`Invalid server response: ${message}`);
+        submitBtn.disabled = false;
+        return;
+      }
 
-      if (!response.ok || !result.ok) {
-        if (result.field) {
-          setFieldError(result.field, result.error || "Invalid input");
+      if (!response.ok || !submitResult.ok) {
+        if (submitResult.field) {
+          setFieldError(submitResult.field, submitResult.error || "Invalid input");
         } else {
-          showGlobalError(result.error || "Submission failed.");
+          showGlobalError(submitResult.error || "Submission failed.");
         }
         submitBtn.disabled = false;
         return;
@@ -2427,6 +2435,12 @@
       stopHeartbeat();
       stopQueuePolling();
       session.ended = true;
+
+      if (submitResult.nextUrl) {
+        window.location.href = submitResult.nextUrl;
+        return;
+      }
+
       successOverlay.classList.remove("hidden");
       setTimeout(() => {
         closeWindow();

package/index.ts

@@ -242,8 +242,9 @@ function loadSavedInterview(html: string, filePath: string): SavedQuestionsFile
 	let data: unknown;
 	try {
 		data = JSON.parse(match[1]);
-	} catch {
-		throw new Error("Invalid saved interview: malformed JSON");
+	} catch (err) {
+		const message = err instanceof Error ? err.message : String(err);
+		throw new Error(`Invalid saved interview: malformed JSON (${message})`);
 	}
 
 	const raw = data as Record<string, unknown>;
@@ -412,7 +413,6 @@ export default function (pi: ExtensionAPI) {
 			let glimpseWin: GlimpseWindow | null = null;
 			let resolved = false;
 			let url = "";
-
 			const cleanup = () => {
 				if (server) {
 					server.close();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-interview",
-  "version": "0.5.4",
+  "version": "0.5.5",
   "description": "Interactive interview form extension for pi coding agent",
   "author": "Nico Bailon",
   "license": "MIT",

package/server.ts

@@ -303,8 +303,9 @@ async function parseJSONBody(req: IncomingMessage): Promise<unknown> {
 		req.on("end", () => {
 			try {
 				resolve(JSON.parse(body));
-			} catch {
-				reject(new Error("Invalid JSON"));
+			} catch (err) {
+				const message = err instanceof Error ? err.message : String(err);
+				reject(new Error(`Invalid JSON: ${message}`));
 			}
 		});
 
@@ -807,6 +808,7 @@ export async function startInterviewServer(
 	let browserConnected = false;
 	let lastHeartbeatAt = Date.now();
 	let watchdog: NodeJS.Timeout | null = null;
+	let sessionKeepAlive: NodeJS.Timeout | null = null;
 	let completed = false;
 
 	const stopWatchdog = () => {
@@ -816,10 +818,18 @@ export async function startInterviewServer(
 		}
 	};
 
+	const stopSessionKeepAlive = () => {
+		if (sessionKeepAlive) {
+			clearInterval(sessionKeepAlive);
+			sessionKeepAlive = null;
+		}
+	};
+
 	const markCompleted = () => {
 		if (completed) return false;
 		completed = true;
 		stopWatchdog();
+		stopSessionKeepAlive();
 		return true;
 	};
 
@@ -839,6 +849,20 @@ export async function startInterviewServer(
 			const url = new URL(req.url || "/", `http://${req.headers.host || "127.0.0.1"}`);
 			log(verbose, `${method} ${url.pathname}`);
 
+			const parseBodyOrRespond = async (): Promise<unknown | null> => {
+				try {
+					return await parseJSONBody(req);
+				} catch (err) {
+					if (err instanceof BodyTooLargeError) {
+						sendJson(res, err.statusCode, { ok: false, error: err.message });
+						return null;
+					}
+					const message = err instanceof Error ? err.message : String(err);
+					sendJson(res, 400, { ok: false, error: message });
+					return null;
+				}
+			};
+
 			if (method === "GET" && url.pathname === "/") {
 				if (!validateTokenQuery(url, sessionToken, res)) return;
 				touchHeartbeat();
@@ -978,11 +1002,8 @@ export async function startInterviewServer(
 			}
 
 			if (method === "POST" && url.pathname === "/heartbeat") {
-				const body = await parseJSONBody(req).catch(() => null);
-				if (!body) {
-					sendJson(res, 400, { ok: false, error: "Invalid body" });
-					return;
-				}
+				const body = await parseBodyOrRespond();
+				if (!body) return;
 				if (!validateTokenBody(body, sessionToken, res)) return;
 				touchHeartbeat();
 				sendJson(res, 200, { ok: true });
@@ -990,14 +1011,7 @@ export async function startInterviewServer(
 			}
 
 			if (method === "POST" && url.pathname === "/cancel") {
-				const body = await parseJSONBody(req).catch((err) => {
-					if (err instanceof BodyTooLargeError) {
-						sendJson(res, err.statusCode, { ok: false, error: err.message });
-						return null;
-					}
-					sendJson(res, 400, { ok: false, error: err.message });
-					return null;
-				});
+				const body = await parseBodyOrRespond();
 				if (!body) return;
 				if (!validateTokenBody(body, sessionToken, res)) return;
 				if (completed) {
@@ -1020,14 +1034,7 @@ export async function startInterviewServer(
 			}
 
 			if (method === "POST" && url.pathname === "/submit") {
-				const body = await parseJSONBody(req).catch((err) => {
-					if (err instanceof BodyTooLargeError) {
-						sendJson(res, err.statusCode, { ok: false, error: err.message });
-						return null;
-					}
-					sendJson(res, 400, { ok: false, error: err.message });
-					return null;
-				});
+				const body = await parseBodyOrRespond();
 				if (!body) return;
 				if (!validateTokenBody(body, sessionToken, res)) return;
 				if (completed) {
@@ -1143,20 +1150,22 @@ export async function startInterviewServer(
 
 				markCompleted();
 				unregisterSession(sessionId);
-				sendJson(res, 200, { ok: true });
+				const nextSession = getActiveSessions()
+					.filter((s) => s.id !== sessionId)
+					.sort((a, b) => {
+						if (a.startedAt !== b.startedAt) {
+							return a.startedAt - b.startedAt;
+						}
+						return a.id.localeCompare(b.id);
+					})[0];
+				const nextUrl = nextSession ? nextSession.url : null;
+				sendJson(res, 200, { ok: true, nextUrl });
 				setImmediate(() => callbacks.onSubmit(responses));
 				return;
 			}
 
 			if (method === "POST" && url.pathname === "/save") {
-				const body = await parseJSONBody(req).catch((err) => {
-					if (err instanceof BodyTooLargeError) {
-						sendJson(res, err.statusCode, { ok: false, error: err.message });
-						return null;
-					}
-					sendJson(res, 400, { ok: false, error: err.message });
-					return null;
-				});
+				const body = await parseBodyOrRespond();
 				if (!body) return;
 				if (!validateTokenBody(body, sessionToken, res)) return;
 				// Note: don't check `completed` - allow save after submit
@@ -1307,6 +1316,11 @@ export async function startInterviewServer(
 				lastSeen: now,
 			};
 			registerSession(sessionEntry);
+			const keepAliveEntry = sessionEntry;
+			sessionKeepAlive = setInterval(() => {
+				if (completed) return;
+				touchSession(keepAliveEntry);
+			}, 10000);
 			if (!watchdog) {
 				watchdog = setInterval(() => {
 					if (completed || !browserConnected) return;

package/settings.ts

@@ -1,4 +1,4 @@
-import { readFileSync } from "node:fs";
+import { existsSync, readFileSync } from "node:fs";
 import { join } from "node:path";
 import { homedir } from "node:os";
 
@@ -22,10 +22,19 @@ export interface InterviewSettings {
 }
 
 export function loadSettings(): InterviewSettings {
-	try {
-		const data = JSON.parse(readFileSync(SETTINGS_PATH, "utf-8"));
-		return (data.interview as InterviewSettings) ?? {};
-	} catch {
+	if (!existsSync(SETTINGS_PATH)) {
 		return {};
 	}
+
+	const parsed = JSON.parse(readFileSync(SETTINGS_PATH, "utf-8"));
+	if (typeof parsed !== "object" || parsed === null) {
+		return {};
+	}
+
+	const interview = (parsed as Record<string, unknown>).interview;
+	if (typeof interview !== "object" || interview === null) {
+		return {};
+	}
+
+	return interview as InterviewSettings;
 }