bluera-knowledge 0.11.18 → 0.11.20

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "bluera-knowledge",
-  "version": "0.11.18",
+  "version": "0.11.20",
   "description": "Clone repos, crawl docs, search locally. Fast, authoritative answers for AI coding agents.",
   "mcpServers": {
     "bluera-knowledge": {

package/CHANGELOG.md

@@ -2,6 +2,61 @@
 
 All notable changes to this project will be documented in this file. See [commit-and-tag-version](https://github.com/absolute-version/commit-and-tag-version) for commit guidelines.
 
+## [0.11.20](https://github.com/blueraai/bluera-knowledge/compare/v0.11.6...v0.11.20) (2026-01-10)
+
+
+### Features
+
+* **code-unit:** add interface and type extraction support ([190dded](https://github.com/blueraai/bluera-knowledge/commit/190dded78be68985c3f94c0da6ebed03659da313))
+* **commands:** add test-plugin command for comprehensive plugin testing ([c6eb5e7](https://github.com/blueraai/bluera-knowledge/commit/c6eb5e7762376810a9ff3e79f794f05ff0c77b97))
+* **scripts:** add post-release npm validation script ([e4c29a0](https://github.com/blueraai/bluera-knowledge/commit/e4c29a0c83907de4bc293a69a58412629457fb22))
+* **scripts:** add suggest, sync, serve, mcp tests to npm validation ([49d85da](https://github.com/blueraai/bluera-knowledge/commit/49d85dad1a89691060c12f152d644844baf6e6e6))
+* **scripts:** log expected vs installed version in validation script ([c77d039](https://github.com/blueraai/bluera-knowledge/commit/c77d039b27a3ccf54d50006af161ac4dcfea7b21))
+
+
+### Bug Fixes
+
+* address code review issues - resource leak and error handling ([c14e9b2](https://github.com/blueraai/bluera-knowledge/commit/c14e9b22782f0d62c04fb312a70345d7f4248ed4))
+* **bridge:** close readline interfaces on stop to prevent resource leak ([e970bf9](https://github.com/blueraai/bluera-knowledge/commit/e970bf94f74bdba24803eed8714d47ed8e874117))
+* **cli:** plugin-api commands now respect global options ([d3cca02](https://github.com/blueraai/bluera-knowledge/commit/d3cca02ffc679ffc187b76c7682f3cc177eabdea))
+* **commands:** fix test-plugin reliability issues ([2841e9a](https://github.com/blueraai/bluera-knowledge/commit/2841e9aa6438f9fd60c70ab921575088b2921810))
+* **commands:** move test-plugin to commands/ for plugin distribution ([0a3ff5f](https://github.com/blueraai/bluera-knowledge/commit/0a3ff5f91666db7efa213d5abd4c447fb07749fc))
+* **crawl:** throw errors instead of fallback/graceful degradation ([19962c2](https://github.com/blueraai/bluera-knowledge/commit/19962c2114406f2fea22e5973e242f190ab65fb7))
+* **git-clone:** add error handler for spawn failures ([4bf50e2](https://github.com/blueraai/bluera-knowledge/commit/4bf50e2348505dccdc94522cf42f1d4d3471480c))
+* improve error handling and type safety from code review ([61710aa](https://github.com/blueraai/bluera-knowledge/commit/61710aaed48c87c457628b75e3968686377a6c92))
+* **plugin:** properly close services after command execution ([eeaf743](https://github.com/blueraai/bluera-knowledge/commit/eeaf743750be73fd9c7a9e72440b2fd0fb5a53fa))
+* **scripts:** show real-time output in validation script ([8a4bdec](https://github.com/blueraai/bluera-knowledge/commit/8a4bdec8b63c504d34ba35bfe19da795f7f7fd07))
+* **scripts:** use mktemp for temp directories in validation script ([3107861](https://github.com/blueraai/bluera-knowledge/commit/3107861bd7a966016fde2a121469dd84756f39be))
+* **search:** add defaults for env vars so CLI works standalone ([b2d2ce5](https://github.com/blueraai/bluera-knowledge/commit/b2d2ce534e8cd2ba0fc0abdac505c912a1a76035))
+* **services:** throw errors instead of graceful degradation ([ca992b5](https://github.com/blueraai/bluera-knowledge/commit/ca992b5cfa4d9ebad62ee82231cfdd2d3d64012e))
+
+## [0.11.19](https://github.com/blueraai/bluera-knowledge/compare/v0.11.6...v0.11.19) (2026-01-10)
+
+
+### Features
+
+* **code-unit:** add interface and type extraction support ([190dded](https://github.com/blueraai/bluera-knowledge/commit/190dded78be68985c3f94c0da6ebed03659da313))
+* **commands:** add test-plugin command for comprehensive plugin testing ([c6eb5e7](https://github.com/blueraai/bluera-knowledge/commit/c6eb5e7762376810a9ff3e79f794f05ff0c77b97))
+* **scripts:** add post-release npm validation script ([e4c29a0](https://github.com/blueraai/bluera-knowledge/commit/e4c29a0c83907de4bc293a69a58412629457fb22))
+* **scripts:** add suggest, sync, serve, mcp tests to npm validation ([49d85da](https://github.com/blueraai/bluera-knowledge/commit/49d85dad1a89691060c12f152d644844baf6e6e6))
+* **scripts:** log expected vs installed version in validation script ([c77d039](https://github.com/blueraai/bluera-knowledge/commit/c77d039b27a3ccf54d50006af161ac4dcfea7b21))
+
+
+### Bug Fixes
+
+* **bridge:** close readline interfaces on stop to prevent resource leak ([e970bf9](https://github.com/blueraai/bluera-knowledge/commit/e970bf94f74bdba24803eed8714d47ed8e874117))
+* **cli:** plugin-api commands now respect global options ([d3cca02](https://github.com/blueraai/bluera-knowledge/commit/d3cca02ffc679ffc187b76c7682f3cc177eabdea))
+* **commands:** fix test-plugin reliability issues ([2841e9a](https://github.com/blueraai/bluera-knowledge/commit/2841e9aa6438f9fd60c70ab921575088b2921810))
+* **commands:** move test-plugin to commands/ for plugin distribution ([0a3ff5f](https://github.com/blueraai/bluera-knowledge/commit/0a3ff5f91666db7efa213d5abd4c447fb07749fc))
+* **crawl:** throw errors instead of fallback/graceful degradation ([19962c2](https://github.com/blueraai/bluera-knowledge/commit/19962c2114406f2fea22e5973e242f190ab65fb7))
+* **git-clone:** add error handler for spawn failures ([4bf50e2](https://github.com/blueraai/bluera-knowledge/commit/4bf50e2348505dccdc94522cf42f1d4d3471480c))
+* improve error handling and type safety from code review ([61710aa](https://github.com/blueraai/bluera-knowledge/commit/61710aaed48c87c457628b75e3968686377a6c92))
+* **plugin:** properly close services after command execution ([eeaf743](https://github.com/blueraai/bluera-knowledge/commit/eeaf743750be73fd9c7a9e72440b2fd0fb5a53fa))
+* **scripts:** show real-time output in validation script ([8a4bdec](https://github.com/blueraai/bluera-knowledge/commit/8a4bdec8b63c504d34ba35bfe19da795f7f7fd07))
+* **scripts:** use mktemp for temp directories in validation script ([3107861](https://github.com/blueraai/bluera-knowledge/commit/3107861bd7a966016fde2a121469dd84756f39be))
+* **search:** add defaults for env vars so CLI works standalone ([b2d2ce5](https://github.com/blueraai/bluera-knowledge/commit/b2d2ce534e8cd2ba0fc0abdac505c912a1a76035))
+* **services:** throw errors instead of graceful degradation ([ca992b5](https://github.com/blueraai/bluera-knowledge/commit/ca992b5cfa4d9ebad62ee82231cfdd2d3d64012e))
+
 ## [0.11.18](https://github.com/blueraai/bluera-knowledge/compare/v0.11.6...v0.11.18) (2026-01-10)
 
 

package/dist/{chunk-6FHWC36B.js → chunk-HRQD3MPH.js}

@@ -9,7 +9,7 @@ var WatchService = class {
     this.indexService = indexService;
     this.lanceStore = lanceStore;
   }
-  async watch(store, debounceMs = 1e3, onReindex) {
+  async watch(store, debounceMs, onReindex, onError) {
     if (this.watchers.has(store.id)) {
       return Promise.resolve();
     }
@@ -28,16 +28,18 @@ var WatchService = class {
             await this.lanceStore.initialize(store.id);
             await this.indexService.indexStore(store);
             onReindex?.();
-          } catch (error) {
-            console.error("Error during reindexing:", error);
+          } catch (e) {
+            const error = e instanceof Error ? e : new Error(String(e));
+            onError(error);
           }
         })();
       }, debounceMs);
       this.pendingTimeouts.set(store.id, timeout);
     };
     watcher.on("all", reindexHandler);
-    watcher.on("error", (error) => {
-      console.error("Watcher error:", error);
+    watcher.on("error", (e) => {
+      const error = e instanceof Error ? e : new Error(String(e));
+      onError(error);
     });
     this.watchers.set(store.id, watcher);
     return Promise.resolve();
@@ -64,4 +66,4 @@ var WatchService = class {
 export {
   WatchService
 };
-//# sourceMappingURL=chunk-6FHWC36B.js.map
+//# sourceMappingURL=chunk-HRQD3MPH.js.map

package/dist/chunk-HRQD3MPH.js.map

@@ -0,0 +1 @@
+{"version":3,"sources":["../src/services/watch.service.ts"],"sourcesContent":["import { watch, type FSWatcher } from 'chokidar';\nimport type { IndexService } from './index.service.js';\nimport type { LanceStore } from '../db/lance.js';\nimport type { FileStore, RepoStore } from '../types/store.js';\n\nexport class WatchService {\n  private readonly watchers: Map<string, FSWatcher> = new Map();\n  private readonly pendingTimeouts: Map<string, NodeJS.Timeout> = new Map();\n  private readonly indexService: IndexService;\n  private readonly lanceStore: LanceStore;\n\n  constructor(indexService: IndexService, lanceStore: LanceStore) {\n    this.indexService = indexService;\n    this.lanceStore = lanceStore;\n  }\n\n  async watch(\n    store: FileStore | RepoStore,\n    debounceMs: number,\n    onReindex: (() => void) | undefined,\n    onError: (error: Error) => void\n  ): Promise<void> {\n    if (this.watchers.has(store.id)) {\n      return Promise.resolve(); // Already watching\n    }\n\n    let timeout: NodeJS.Timeout | null = null;\n\n    const watcher = watch(store.path, {\n      ignored: /(^|[/\\\\])\\.(git|node_modules|dist|build)/,\n      persistent: true,\n      ignoreInitial: true,\n    });\n\n    const reindexHandler = (): void => {\n      if (timeout) clearTimeout(timeout);\n      timeout = setTimeout(() => {\n        this.pendingTimeouts.delete(store.id);\n        void (async (): Promise<void> => {\n          try {\n            await this.lanceStore.initialize(store.id);\n            await this.indexService.indexStore(store);\n            onReindex?.();\n          } catch (e) {\n            const error = e instanceof Error ? e : new Error(String(e));\n            onError(error);\n          }\n        })();\n      }, debounceMs);\n      this.pendingTimeouts.set(store.id, timeout);\n    };\n\n    watcher.on('all', reindexHandler);\n\n    watcher.on('error', (e) => {\n      const error = e instanceof Error ? e : new Error(String(e));\n      onError(error);\n    });\n\n    this.watchers.set(store.id, watcher);\n    return Promise.resolve();\n  }\n\n  async unwatch(storeId: string): Promise<void> {\n    // Clear any pending timeout to prevent timer leak\n    const pendingTimeout = this.pendingTimeouts.get(storeId);\n    if (pendingTimeout) {\n      clearTimeout(pendingTimeout);\n      this.pendingTimeouts.delete(storeId);\n    }\n\n    const watcher = this.watchers.get(storeId);\n    if (watcher) {\n      await watcher.close();\n      this.watchers.delete(storeId);\n    }\n  }\n\n  async unwatchAll(): Promise<void> {\n    for (const [id] of this.watchers) {\n      await this.unwatch(id);\n    }\n  }\n}\n"],"mappings":";AAAA,SAAS,aAA6B;AAK/B,IAAM,eAAN,MAAmB;AAAA,EACP,WAAmC,oBAAI,IAAI;AAAA,EAC3C,kBAA+C,oBAAI,IAAI;AAAA,EACvD;AAAA,EACA;AAAA,EAEjB,YAAY,cAA4B,YAAwB;AAC9D,SAAK,eAAe;AACpB,SAAK,aAAa;AAAA,EACpB;AAAA,EAEA,MAAM,MACJ,OACA,YACA,WACA,SACe;AACf,QAAI,KAAK,SAAS,IAAI,MAAM,EAAE,GAAG;AAC/B,aAAO,QAAQ,QAAQ;AAAA,IACzB;AAEA,QAAI,UAAiC;AAErC,UAAM,UAAU,MAAM,MAAM,MAAM;AAAA,MAChC,SAAS;AAAA,MACT,YAAY;AAAA,MACZ,eAAe;AAAA,IACjB,CAAC;AAED,UAAM,iBAAiB,MAAY;AACjC,UAAI,QAAS,cAAa,OAAO;AACjC,gBAAU,WAAW,MAAM;AACzB,aAAK,gBAAgB,OAAO,MAAM,EAAE;AACpC,cAAM,YAA2B;AAC/B,cAAI;AACF,kBAAM,KAAK,WAAW,WAAW,MAAM,EAAE;AACzC,kBAAM,KAAK,aAAa,WAAW,KAAK;AACxC,wBAAY;AAAA,UACd,SAAS,GAAG;AACV,kBAAM,QAAQ,aAAa,QAAQ,IAAI,IAAI,MAAM,OAAO,CAAC,CAAC;AAC1D,oBAAQ,KAAK;AAAA,UACf;AAAA,QACF,GAAG;AAAA,MACL,GAAG,UAAU;AACb,WAAK,gBAAgB,IAAI,MAAM,IAAI,OAAO;AAAA,IAC5C;AAEA,YAAQ,GAAG,OAAO,cAAc;AAEhC,YAAQ,GAAG,SAAS,CAAC,MAAM;AACzB,YAAM,QAAQ,aAAa,QAAQ,IAAI,IAAI,MAAM,OAAO,CAAC,CAAC;AAC1D,cAAQ,KAAK;AAAA,IACf,CAAC;AAED,SAAK,SAAS,IAAI,MAAM,IAAI,OAAO;AACnC,WAAO,QAAQ,QAAQ;AAAA,EACzB;AAAA,EAEA,MAAM,QAAQ,SAAgC;AAE5C,UAAM,iBAAiB,KAAK,gBAAgB,IAAI,OAAO;AACvD,QAAI,gBAAgB;AAClB,mBAAa,cAAc;AAC3B,WAAK,gBAAgB,OAAO,OAAO;AAAA,IACrC;AAEA,UAAM,UAAU,KAAK,SAAS,IAAI,OAAO;AACzC,QAAI,SAAS;AACX,YAAM,QAAQ,MAAM;AACpB,WAAK,SAAS,OAAO,OAAO;AAAA,IAC9B;AAAA,EACF;AAAA,EAEA,MAAM,aAA4B;AAChC,eAAW,CAAC,EAAE,KAAK,KAAK,UAAU;AAChC,YAAM,KAAK,QAAQ,EAAE;AAAA,IACvB;AAAA,EACF;AACF;","names":[]}

package/dist/{chunk-ZZNABJMQ.js → chunk-MQGRQ2EG.js}

@@ -323,8 +323,9 @@ var JobService = class {
       const content = fs.readFileSync(jobFile, "utf-8");
       return JSON.parse(content);
     } catch (error) {
-      console.error(`Error reading job ${jobId}:`, error);
-      return null;
+      throw new Error(
+        `Failed to read job ${jobId}: ${error instanceof Error ? error.message : String(error)}`
+      );
     }
   }
   /**
@@ -352,7 +353,9 @@ var JobService = class {
           jobs.push(job);
         }
       } catch (error) {
-        console.error(`Error reading job file ${file}:`, error);
+        throw new Error(
+          `Failed to read job file ${file}: ${error instanceof Error ? error.message : String(error)}`
+        );
       }
     }
     jobs.sort((a, b) => new Date(b.updatedAt).getTime() - new Date(a.updatedAt).getTime());
@@ -413,7 +416,9 @@ var JobService = class {
           fs.unlinkSync(jobFile);
           cleaned++;
         } catch (error) {
-          console.error(`Error deleting job file ${job.id}:`, error);
+          throw new Error(
+            `Failed to delete job file ${job.id}: ${error instanceof Error ? error.message : String(error)}`
+          );
         }
       }
     }
@@ -431,8 +436,9 @@ var JobService = class {
       fs.unlinkSync(jobFile);
       return true;
     } catch (error) {
-      console.error(`Error deleting job ${jobId}:`, error);
-      return false;
+      throw new Error(
+        `Failed to delete job ${jobId}: ${error instanceof Error ? error.message : String(error)}`
+      );
     }
   }
   /**
@@ -2572,6 +2578,16 @@ var CodeUnitService = class {
         type = "class";
         break;
       }
+      if (line.match(new RegExp(`interface\\s+${symbolName}(?:\\s|{|<)`))) {
+        startLine = i + 1;
+        type = "interface";
+        break;
+      }
+      if (line.match(new RegExp(`type\\s+${symbolName}(?:\\s|=|<)`))) {
+        startLine = i + 1;
+        type = "type";
+        break;
+      }
       if (line.match(new RegExp(`(?:const|let|var)\\s+${symbolName}\\s*=`))) {
         startLine = i + 1;
         type = "const";
@@ -2582,6 +2598,23 @@ var CodeUnitService = class {
     let endLine = startLine;
     let braceCount = 0;
     let foundFirstBrace = false;
+    if (type === "type") {
+      const firstLine2 = lines[startLine - 1] ?? "";
+      if (!firstLine2.includes("{") && firstLine2.includes(";")) {
+        endLine = startLine;
+        const fullContent2 = firstLine2;
+        const signature2 = this.extractSignature(firstLine2, symbolName, type);
+        return {
+          type,
+          name: symbolName,
+          signature: signature2,
+          fullContent: fullContent2,
+          startLine,
+          endLine,
+          language
+        };
+      }
+    }
     let inSingleQuote = false;
     let inDoubleQuote = false;
     let inTemplateLiteral = false;
@@ -2672,6 +2705,16 @@ var CodeUnitService = class {
     if (type === "class") {
       return `class ${name}`;
     }
+    if (type === "interface") {
+      return `interface ${name}`;
+    }
+    if (type === "type") {
+      const typeMatch = sig.match(new RegExp(`type\\s+(${name}(?:<[^>]+>)?)\\s*=`));
+      if (typeMatch?.[1] !== void 0 && typeMatch[1].length > 0) {
+        return `type ${typeMatch[1]}`;
+      }
+      return `type ${name}`;
+    }
     if (type === "const") {
       const arrowMatch = sig.match(
         new RegExp(
@@ -3061,15 +3104,18 @@ var SearchService = class {
   async ftsSearch(query, stores, limit) {
     const results = [];
     for (const storeId of stores) {
-      const hits = await this.lanceStore.fullTextSearch(storeId, query, limit);
-      results.push(
-        ...hits.map((r) => ({
-          id: r.id,
-          score: r.score,
-          content: r.content,
-          metadata: r.metadata
-        }))
-      );
+      try {
+        const hits = await this.lanceStore.fullTextSearch(storeId, query, limit);
+        results.push(
+          ...hits.map((r) => ({
+            id: r.id,
+            score: r.score,
+            content: r.content,
+            metadata: r.metadata
+          }))
+        );
+      } catch {
+      }
     }
     return results.sort((a, b) => b.score - a.score).slice(0, limit);
   }
@@ -3640,6 +3686,9 @@ async function cloneRepository(options) {
     git.stderr.on("data", (data) => {
       stderr += data.toString();
     });
+    git.on("error", (error) => {
+      resolve3(err(error));
+    });
     git.on("close", (code) => {
       if (code === 0) {
         resolve3(ok(targetDir));
@@ -3993,6 +4042,8 @@ var PythonBridge = class {
   process = null;
   pending = /* @__PURE__ */ new Map();
   stoppingIntentionally = false;
+  stdoutReadline = null;
+  stderrReadline = null;
   start() {
     if (this.process) return Promise.resolve();
     logger3.debug("Starting Python bridge process");
@@ -4015,8 +4066,8 @@ var PythonBridge = class {
       this.stoppingIntentionally = false;
     });
     if (this.process.stderr) {
-      const stderrRl = createInterface({ input: this.process.stderr });
-      stderrRl.on("line", (line) => {
+      this.stderrReadline = createInterface({ input: this.process.stderr });
+      this.stderrReadline.on("line", (line) => {
         logger3.warn({ stderr: line }, "Python bridge stderr output");
       });
     }
@@ -4025,8 +4076,8 @@ var PythonBridge = class {
       this.process = null;
       return Promise.reject(new Error("Python bridge process stdout is null"));
     }
-    const rl = createInterface({ input: this.process.stdout });
-    rl.on("line", (line) => {
+    this.stdoutReadline = createInterface({ input: this.process.stdout });
+    this.stdoutReadline.on("line", (line) => {
       if (!line.trim().startsWith("{")) {
         return;
       }
@@ -4188,6 +4239,14 @@ var PythonBridge = class {
     return new Promise((resolve3) => {
       this.stoppingIntentionally = true;
       this.rejectAllPending(new Error("Python bridge stopped"));
+      if (this.stdoutReadline) {
+        this.stdoutReadline.close();
+        this.stdoutReadline = null;
+      }
+      if (this.stderrReadline) {
+        this.stderrReadline.close();
+        this.stderrReadline = null;
+      }
       const proc = this.process;
       if (proc === null) {
         resolve3();
@@ -4332,18 +4391,14 @@ var LanceStore = class {
   }
   async fullTextSearch(storeId, query, limit) {
     const table = await this.getTable(storeId);
-    try {
-      const results = await table.search(query, "fts").limit(limit).toArray();
-      return results.map((r) => ({
-        id: createDocumentId(r.id),
-        content: r.content,
-        score: r._score,
-        // eslint-disable-next-line @typescript-eslint/consistent-type-assertions
-        metadata: JSON.parse(r.metadata)
-      }));
-    } catch {
-      return [];
-    }
+    const results = await table.search(query, "fts").limit(limit).toArray();
+    return results.map((r) => ({
+      id: createDocumentId(r.id),
+      content: r.content,
+      score: r._score,
+      // eslint-disable-next-line @typescript-eslint/consistent-type-assertions
+      metadata: JSON.parse(r.metadata)
+    }));
   }
   async deleteStore(storeId) {
     const tableName = this.getTableName(storeId);
@@ -4419,18 +4474,28 @@ async function createServices(configPath, dataDir, projectRoot) {
 }
 async function destroyServices(services) {
   logger4.info("Shutting down services");
+  const errors = [];
   try {
     await services.lance.closeAsync();
   } catch (e) {
-    logger4.error({ error: e }, "Error closing LanceStore");
+    const error = e instanceof Error ? e : new Error(String(e));
+    logger4.error({ error }, "Error closing LanceStore");
+    errors.push(error);
   }
   try {
     await services.pythonBridge.stop();
   } catch (e) {
-    logger4.error({ error: e }, "Error stopping Python bridge");
+    const error = e instanceof Error ? e : new Error(String(e));
+    logger4.error({ error }, "Error stopping Python bridge");
+    errors.push(error);
   }
   await new Promise((resolve3) => setTimeout(resolve3, 100));
   await shutdownLogger();
+  if (errors.length === 1 && errors[0] !== void 0) {
+    throw new Error(`Service shutdown failed: ${errors[0].message}`, { cause: errors[0] });
+  } else if (errors.length > 1) {
+    throw new AggregateError(errors, "Multiple errors during service shutdown");
+  }
 }
 
 export {
@@ -4453,4 +4518,4 @@ export {
   createServices,
   destroyServices
 };
-//# sourceMappingURL=chunk-ZZNABJMQ.js.map
+//# sourceMappingURL=chunk-MQGRQ2EG.js.map