@bacnh85/pi-serena 0.3.1 → 0.4.0

package/README.md

@@ -46,7 +46,6 @@ After install or update, restart Pi or run `/reload` in an existing Pi session.
 - `serena_replace_content`
 - `serena_restart_language_server`
 - `serena_get_current_config`
-- `serena_check_onboarding_performed`
 - `serena_onboarding`
 - `serena_list_memories`
 - `serena_read_memory`
@@ -55,6 +54,8 @@ After install or update, restart Pi or run `/reload` in an existing Pi session.
 
 All tool outputs are truncated to 50KB / 2000 lines to match Pi-friendly output limits. Most tools accept optional `timeout_ms`.
 
+When a worker request exceeds the configured timeout, the Python bridge process is automatically killed and a fresh worker is started for the next call. This prevents cascading timeouts from blocking future requests.
+
 ### Pattern search
 
 Use the Pi-facing `pattern` field with `serena_search_for_pattern`:

package/index.ts

@@ -150,6 +150,9 @@ export function normalizeSearchPatternParams(params: Record<string, unknown>): R
 		normalized.substring_pattern = normalized.pattern;
 	}
 	delete normalized.pattern;
+	// The Python SearchForPatternTool.apply() does not accept a multiline parameter.
+	// Strip it here to avoid TypeError when the parameter description says "when supported by Serena".
+	delete normalized.multiline;
 	return normalized;
 }
 
@@ -206,8 +209,16 @@ function truncateText(text: string): string {
 }
 
 function resultText(response: SerenaWorkerResponse): string {
-	const text = !response.ok ? `Error: ${response.error ?? "Unknown Serena error"}` : typeof response.result === "string" ? response.result : JSON.stringify(response, null, 2);
-	return truncateText(text);
+	if (!response.ok) {
+		return `Error: ${response.error ?? "Unknown Serena error"}`;
+	}
+	// For search results, show a friendly empty-state instead of raw "{}".
+	if (response.tool === "search_for_pattern") {
+		if (response.result == null || (typeof response.result === "object" && Object.keys(response.result as object).length === 0)) {
+			return "No results found.";
+		}
+	}
+	return typeof response.result === "string" ? response.result : JSON.stringify(response, null, 2);
 }
 
 export default function serenaToolsExtension(pi: ExtensionAPI) {
@@ -423,18 +434,6 @@ export default function serenaToolsExtension(pi: ExtensionAPI) {
 		},
 	});
 
-	pi.registerTool({
-		name: "serena_check_onboarding_performed",
-		label: "Serena Check Onboarding",
-		description: "Check whether Serena onboarding memories exist for this project.",
-		promptSnippet: "Check whether project onboarding was already performed",
-		promptGuidelines: ["Use serena_check_onboarding_performed before relying on Serena project memories."],
-		parameters: emptyToolSchema,
-		async execute(_id, params, _signal, _onUpdate, ctx) {
-			return callSerena(ctx, "check_onboarding_performed", params);
-		},
-	});
-
 	pi.registerTool({
 		name: "serena_onboarding",
 		label: "Serena Onboarding",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bacnh85/pi-serena",
-  "version": "0.3.1",
+  "version": "0.4.0",
   "description": "Pi extension that provides Serena semantic code tools through a persistent worker.",
   "license": "MIT",
   "publishConfig": {

package/worker.ts

@@ -272,7 +272,13 @@ export class SerenaWorkerClient {
 		return new Promise((resolve, reject) => {
 			const timer = setTimeout(() => {
 				this.pending.delete(id);
-				reject(new Error(`Serena worker request timed out: ${payload.action ?? "unknown"}`));
+				// Kill and reset worker so subsequent requests don't pile up.
+				// The next request() call triggers ensureStarted() which spawns a fresh worker.
+				this.killAndReset();
+				reject(new Error(
+					`Serena worker request timed out: ${payload.action ?? "unknown"}. ` +
+					`Worker has been restarted; retry if needed.`
+				));
 			}, timeoutMs);
 			this.pending.set(id, { resolve, reject, timer });
 			this.process!.stdin.write(`${JSON.stringify(request)}\n`);
@@ -353,6 +359,16 @@ export class SerenaWorkerClient {
 		}
 	}
 
+	private killAndReset(): void {
+		if (this.process) {
+			this.process.kill();
+			this.process = undefined;
+			this.onStatus?.(undefined);
+		}
+		this.failAll(new Error("Serena worker killed due to timeout, restarted"));
+		this.buffer = "";
+	}
+
 	private failAll(error: Error): void {
 		for (const [id, pending] of this.pending) {
 			clearTimeout(pending.timer);