@bacnh85/pi-serena 0.3.0 → 0.3.2

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) {

package/lib/detect.ts

@@ -0,0 +1,56 @@
+/**
+ * Pure detection logic for code-vs-non-code and semantic-search-vs-text-search decisions.
+ * Extracted from index.ts so it can be tested without importing pi-coding-agent.
+ */
+
+export const CODE_FILE_EXTENSIONS = new Set([
+	".c",
+	".cc",
+	".cpp",
+	".cs",
+	".css",
+	".go",
+	".java",
+	".js",
+	".jsx",
+	".kt",
+	".lua",
+	".mjs",
+	".php",
+	".py",
+	".rb",
+	".rs",
+	".scala",
+	".sh",
+	".swift",
+	".ts",
+	".tsx",
+	".vue",
+]);
+
+const NON_SEMANTIC_FILE_EXTENSIONS = new Set([".json", ".jsonl", ".lock", ".md", ".txt", ".yaml", ".yml"]);
+
+export const SEMANTIC_MISS_THRESHOLD = 2;
+
+export function pathLooksLikeCode(value: unknown): boolean {
+	if (typeof value !== "string" || value.trim() === "") return false;
+	const cleanPath = value.split(/[?#]/, 1)[0].toLowerCase();
+	const dotIndex = cleanPath.lastIndexOf(".");
+	if (dotIndex < 0) return false;
+	return CODE_FILE_EXTENSIONS.has(cleanPath.slice(dotIndex));
+}
+
+export function pathLooksNonSemantic(value: unknown): boolean {
+	if (typeof value !== "string" || value.trim() === "") return false;
+	const cleanPath = value.split(/[?#]/, 1)[0].toLowerCase();
+	const dotIndex = cleanPath.lastIndexOf(".");
+	if (dotIndex < 0) return false;
+	return NON_SEMANTIC_FILE_EXTENSIONS.has(cleanPath.slice(dotIndex));
+}
+
+export function commandLooksLikeSemanticCodeSearch(command: string): boolean {
+	if (!/\b(rg|grep|fd|find)\b/.test(command)) return false;
+	if (/\b(SKILL\.md|README\.md|AGENTS\.md|package\.json|skill-registry\.json|skill-history\.jsonl)\b/i.test(command)) return false;
+	if (/\b(symbol|class|method|function|def|interface|references?|implementation|declaration|rename|refactor)\b/i.test(command)) return true;
+	return /\.(c|cc|cpp|cs|go|java|js|jsx|kt|lua|mjs|php|py|rb|rs|scala|sh|swift|ts|tsx|vue)\b/i.test(command);
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bacnh85/pi-serena",
-  "version": "0.3.0",
+  "version": "0.3.2",
   "description": "Pi extension that provides Serena semantic code tools through a persistent worker.",
   "license": "MIT",
   "publishConfig": {
@@ -24,7 +24,8 @@
   "files": [
     "README.md",
     "index.ts",
-    "worker.ts"
+    "worker.ts",
+    "lib/"
   ],
   "pi": {
     "extensions": [