@bacnh85/pi-serena 0.1.0 → 0.1.2

package/README.md

@@ -6,6 +6,12 @@ Note: Serena itself is a Python package. The TypeScript worker owns lifecycle, r
 
 ## Install
 
+Install the published package from npm:
+
+```bash
+pi install npm:@bacnh85/pi-serena
+```
+
 From this repository checkout, install only this extension package:
 
 ```bash
@@ -18,6 +24,8 @@ For local development from a checkout:
 pi -e ./extensions/pi-serena
 ```
 
+The package manifest points Pi directly at `./index.ts`, so published npm installs and local installs load the same extension entrypoint.
+
 There is intentionally no repository-level Pi package. Install each extension from its own subdirectory, matching `extensions/pi-rtk` and future extensions.
 
 After install or update, restart Pi or run `/reload` in an existing Pi session.

package/index.ts

@@ -1,5 +1,7 @@
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { Type } from "typebox";
+import { withFileMutationQueue } from "@earendil-works/pi-coding-agent";
+import path from "node:path";
 import { SerenaWorkerClient, type SerenaWorkerResponse } from "./worker";
 
 const DEFAULT_CONTEXT = "ide";
@@ -214,15 +216,27 @@ export default function serenaToolsExtension(pi: ExtensionAPI) {
 		return worker;
 	};
 
-	const callSerena = async (ctx: any, tool: string, rawParams: Record<string, unknown>) => {
+	const callSerena = async (ctx: any, tool: string, rawParams: Record<string, unknown>, lockPath?: string) => {
 		const { project, context, timeoutMs, params } = stripControlParams(rawParams);
-		const response = await getWorker(ctx).request({ action: "call", project, context, tool, params }, timeoutMs);
-		return {
-			content: [{ type: "text" as const, text: resultText(response) }],
-			details: response,
+		const run = async () => {
+			const response = await getWorker(ctx).request({ action: "call", project, context, tool, params }, timeoutMs);
+			return {
+				content: [{ type: "text" as const, text: resultText(response) }],
+				details: response,
+			};
 		};
+		return lockPath ? withFileMutationQueue(lockPath, run) : run();
 	};
 
+	const lockPathForRelativeFile = (rawParams: Record<string, unknown>): string | undefined => {
+		const project = normalizeProject(rawParams.project);
+		return typeof rawParams.relative_path === "string" && rawParams.relative_path.trim()
+			? path.resolve(project, rawParams.relative_path)
+			: undefined;
+	};
+
+	const lockPathForProject = (rawParams: Record<string, unknown>): string => path.resolve(normalizeProject(rawParams.project));
+
 	pi.registerTool({
 		name: "serena_status",
 		label: "Serena Status",
@@ -299,7 +313,7 @@ export default function serenaToolsExtension(pi: ExtensionAPI) {
 		promptGuidelines: ["Use serena_replace_symbol_body only after serena_find_symbol identifies the exact symbol to replace."],
 		parameters: replaceBodySchema,
 		async execute(_id, params, _signal, _onUpdate, ctx) {
-			return callSerena(ctx, "replace_symbol_body", params);
+			return callSerena(ctx, "replace_symbol_body", params, lockPathForRelativeFile(params));
 		},
 	});
 
@@ -311,7 +325,7 @@ export default function serenaToolsExtension(pi: ExtensionAPI) {
 		promptGuidelines: ["Use serena_insert_before_symbol for symbol-adjacent code insertion after locating the target symbol."],
 		parameters: insertSchema,
 		async execute(_id, params, _signal, _onUpdate, ctx) {
-			return callSerena(ctx, "insert_before_symbol", params);
+			return callSerena(ctx, "insert_before_symbol", params, lockPathForRelativeFile(params));
 		},
 	});
 
@@ -323,7 +337,7 @@ export default function serenaToolsExtension(pi: ExtensionAPI) {
 		promptGuidelines: ["Use serena_insert_after_symbol for adding sibling/helper symbols after a located symbol."],
 		parameters: insertSchema,
 		async execute(_id, params, _signal, _onUpdate, ctx) {
-			return callSerena(ctx, "insert_after_symbol", params);
+			return callSerena(ctx, "insert_after_symbol", params, lockPathForRelativeFile(params));
 		},
 	});
 
@@ -335,7 +349,7 @@ export default function serenaToolsExtension(pi: ExtensionAPI) {
 		promptGuidelines: ["Use serena_rename_symbol for cross-file code renames after finding the exact symbol and references."],
 		parameters: renameSchema,
 		async execute(_id, params, _signal, _onUpdate, ctx) {
-			return callSerena(ctx, "rename_symbol", params);
+			return callSerena(ctx, "rename_symbol", params, lockPathForProject(params));
 		},
 	});
 
@@ -347,7 +361,7 @@ export default function serenaToolsExtension(pi: ExtensionAPI) {
 		promptGuidelines: ["Use serena_safe_delete_symbol instead of text deletion when remaining references matter."],
 		parameters: safeDeleteSchema,
 		async execute(_id, params, _signal, _onUpdate, ctx) {
-			return callSerena(ctx, "safe_delete_symbol", params);
+			return callSerena(ctx, "safe_delete_symbol", params, lockPathForRelativeFile(params));
 		},
 	});
 
@@ -371,7 +385,7 @@ export default function serenaToolsExtension(pi: ExtensionAPI) {
 		promptGuidelines: ["Prefer symbol-aware Serena edit tools for whole symbols; use serena_replace_content for non-symbol scoped replacements supported by Serena."],
 		parameters: replaceContentSchema,
 		async execute(_id, params, _signal, _onUpdate, ctx) {
-			return callSerena(ctx, "replace_content", params);
+			return callSerena(ctx, "replace_content", params, lockPathForRelativeFile(params));
 		},
 	});
 
@@ -455,7 +469,7 @@ export default function serenaToolsExtension(pi: ExtensionAPI) {
 		promptGuidelines: ["Use serena_write_memory after onboarding or when durable verified project knowledge should be available to Serena."],
 		parameters: writeMemorySchema,
 		async execute(_id, params, _signal, _onUpdate, ctx) {
-			return callSerena(ctx, "write_memory", params);
+			return callSerena(ctx, "write_memory", params, lockPathForProject(params));
 		},
 	});
 
@@ -467,7 +481,7 @@ export default function serenaToolsExtension(pi: ExtensionAPI) {
 		promptGuidelines: ["Use serena_delete_memory only when the user explicitly asks to remove a Serena memory."],
 		parameters: memoryNameSchema,
 		async execute(_id, params, _signal, _onUpdate, ctx) {
-			return callSerena(ctx, "delete_memory", params);
+			return callSerena(ctx, "delete_memory", params, lockPathForProject(params));
 		},
 	});
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bacnh85/pi-serena",
-  "version": "0.1.0",
+  "version": "0.1.2",
   "description": "Pi extension that provides Serena semantic code tools through a persistent worker.",
   "license": "MIT",
   "publishConfig": {
@@ -28,7 +28,7 @@
   ],
   "pi": {
     "extensions": [
-      "."
+      "./index.ts"
     ]
   },
   "peerDependencies": {

package/worker.ts

@@ -21,8 +21,8 @@ function piConfigDirs(): string[] {
 	return process.env.PI_CODING_AGENT_DIR ? [process.env.PI_CODING_AGENT_DIR] : [path.join(os.homedir(), ".pi", "agent"), path.join(os.homedir(), ".pi", "agents")];
 }
 
-function loadEnv(cwd = process.cwd()): void {
-	for (const file of [path.resolve(cwd, ".env.local"), path.resolve(cwd, ".env"), ...piConfigDirs().flatMap((dir) => [path.join(dir, ".env.local"), path.join(dir, ".env")])]) {
+function loadEnv(): void {
+	for (const file of piConfigDirs().flatMap((dir) => [path.join(dir, ".env.local"), path.join(dir, ".env")])) {
 		let text: string;
 		try { text = fs.readFileSync(file, "utf8"); } catch (error: any) { if (error?.code === "ENOENT") continue; throw error; }
 		for (const rawLine of text.split(/\r?\n/)) {