@pi-unipi/memory 0.1.9 → 0.1.11

package/embedding.ts

@@ -79,8 +79,7 @@ export async function generateEmbedding(
     });
 
     if (!response.ok) {
-      const errText = await response.text().catch(() => "unknown");
-      console.warn(`[unipi/memory] Embedding API error ${response.status}: ${errText}`);
+      // Removed console.warn — embedding errors cause fallback to fuzzy search.
       return null;
     }
 
@@ -88,7 +87,7 @@ export async function generateEmbedding(
     const values = data?.data?.[0]?.embedding;
 
     if (!Array.isArray(values)) {
-      console.warn("[unipi/memory] Unexpected embedding response format");
+      // Removed console.warn — unexpected format causes fallback to fuzzy search.
       return null;
     }
 
@@ -102,9 +101,9 @@ export async function generateEmbedding(
     return vec;
   } catch (err: any) {
     if (err?.name === "TimeoutError") {
-      console.warn("[unipi/memory] Embedding API timeout");
+      // Removed console.warn — timeout causes fallback to fuzzy search.
     } else {
-      console.warn("[unipi/memory] Embedding error:", err?.message || err);
+      // Removed console.warn — embedding error causes fallback to fuzzy search.
     }
     return null;
   }
@@ -209,8 +208,8 @@ export async function reembedAllMemories(ctx: ExtensionCommandContext): Promise<
       }
 
       storage.close();
-    } catch (err) {
-      console.warn(`[unipi/memory] Failed to re-embed project ${projectName}:`, err);
+    } catch (_err) {
+      // Re-embedding failure — existing embeddings preserved.
     }
   }
 

package/index.ts

@@ -78,11 +78,10 @@ export default function (pi: ExtensionAPI) {
       
       // Sync any orphaned markdown files into the database
       const synced = projectStorage.syncOrphanedFiles();
-      if (synced > 0) {
-        console.warn(`[unipi/memory] Synced ${synced} orphaned memory files into database`);
-      }
-    } catch (err) {
-      console.warn("[unipi/memory] Failed to initialize storage, running without memory:", (err as any)?.message ?? err);
+      // Removed console.warn — orphaned file sync is informational only.
+      // Visible via memory tool list or info-screen memory group.
+    } catch (_err) {
+      // Memory init failure — running without memory. Silent startup.
       projectStorage = null;
     }
 
@@ -113,7 +112,6 @@ export default function (pi: ExtensionAPI) {
     // Register info group
     const registry = getInfoRegistry();
     if (registry) {
-      console.debug("[memory] Registering info group");
       registry.registerGroup({
         id: "memory",
         name: "Memory",
@@ -143,8 +141,8 @@ export default function (pi: ExtensionAPI) {
           try {
             projectMemories = projectStorage.listAll();
             allMemories = listAllProjects();
-          } catch (err) {
-            console.warn("[unipi/memory] Failed to list memories for info panel:", err);
+          } catch (_err) {
+            // Info panel data unavailable — shows empty values.
           }
           const uniqueProjects = [...new Set(allMemories.map((m) => m.project))];
 
@@ -175,8 +173,8 @@ export default function (pi: ExtensionAPI) {
       try {
         projectCount = projectStorage?.listAll()?.length ?? 0;
         projectCountAll = listAllProjects().length;
-      } catch (err) {
-        console.warn("[unipi/memory] Failed to count memories for status:", err);
+      } catch (_err) {
+        // Count unavailable — status bar shows 0.
       }
       const vecReady = isEmbeddingReady();
       const vecIcon = vecReady ? "⚡" : "📝";
@@ -196,8 +194,7 @@ export default function (pi: ExtensionAPI) {
     let projectMemories: Array<{ id: string; title: string; type: string }> = [];
     try {
       projectMemories = projectStorage.listAll();
-    } catch (err) {
-      console.warn("[unipi/memory] Failed to list memories for recall:", err);
+    } catch (_err) {
       recallDone = true; // Skip recall on error
       return;
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-unipi/memory",
-  "version": "0.1.9",
+  "version": "0.1.11",
   "description": "Persistent cross-session memory with vector search for Pi coding agent",
   "type": "module",
   "license": "MIT",

package/storage.ts

@@ -222,9 +222,8 @@ export class MemoryStorage {
           // Do NOT delete the DB: another session may have it open
           // and deleting open files on WSL/Windows is unsafe.
           const delayMs = 50 * Math.pow(2, attempt - 1); // 50, 100, 200, 400
-          console.warn(
-            `[unipi/memory] Transient error on attempt ${attempt}/${maxRetries}, retrying in ${delayMs}ms...`
-          );
+          // Removed console.warn — transient retries are normal during concurrent access.
+          // Memory availability visible via info-screen memory group.
           const end = Date.now() + delayMs;
           while (Date.now() < end) { /* busy wait */ }
           continue;
@@ -233,10 +232,7 @@ export class MemoryStorage {
         // Either non-transient error, or retries exhausted.
         // Log and throw — this session will run without memory.
         if (isTransient) {
-          console.warn(
-            "[unipi/memory] Could not open database after retries. " +
-            "Another session may have the DB locked. Memory unavailable this session."
-          );
+          // Removed console.warn — memory unavailable status visible via info-screen.
         }
         throw err;
       }
@@ -256,8 +252,8 @@ export class MemoryStorage {
     // Load sqlite-vec extension
     try {
       sqliteVec.load(this.db);
-    } catch (err) {
-      console.warn("[unipi/memory] Failed to load sqlite-vec, fuzzy-only mode:", err);
+    } catch (_err) {
+      // sqlite-vec unavailable — fuzzy-only mode. Silent startup.
     }
 
     // Create tables
@@ -308,7 +304,7 @@ export class MemoryStorage {
       try {
         if (fs.existsSync(file)) {
           fs.unlinkSync(file);
-          console.warn(`[unipi/memory] Removed corrupted file: ${file}`);
+          // Removed console.warn — corrupted file cleanup is silent.
         }
       } catch {
         // Ignore removal errors
@@ -401,8 +397,8 @@ export class MemoryStorage {
             BigInt(this.idToRowid(record.id)),
             Buffer.from(record.embedding.buffer)
           );
-        } catch (err) {
-          console.warn("[unipi/memory] Failed to insert vector:", err);
+        } catch (_err) {
+          // Vector insert failure — memory still searchable via text/FTS.
         }
       }
     });
@@ -418,10 +414,8 @@ export class MemoryStorage {
         fs.mkdirSync(dir, { recursive: true });
       }
       fs.writeFileSync(mdPath, mdContent, "utf-8");
-    } catch (err) {
-      // DB write succeeded but file write failed — log but don't throw
-      // Memory is still in DB and searchable
-      console.warn("[unipi/memory] Failed to write markdown file:", err);
+    } catch (_err) {
+      // DB write succeeded but file write failed — memory still in DB and searchable.
     }
   }
 
@@ -474,9 +468,9 @@ export class MemoryStorage {
         );
 
         synced++;
-        console.warn(`[unipi/memory] Synced orphaned file: ${file}`);
-      } catch (err) {
-        console.warn(`[unipi/memory] Failed to sync ${file}:`, err);
+        // Removed console.warn — orphaned file sync is silent.
+      } catch (_err) {
+        // Sync failure — file remains as standalone markdown.
       }
     }