fix: verify fresh dist output

Author: TurboTheTurtle

dist/src/retriever.js

@@ -876,8 +876,9 @@ export class MemoryRetriever {
                 const model = this.config.rerankModel || "jina-reranker-v3";
                 const endpoint = this.config.rerankEndpoint || "https://api.jina.ai/v1/rerank";
                 const documents = results.map((r) => r.entry.text);
+                const rerankTopN = Math.min(results.length, Math.max(1, this.config.candidatePoolSize));
                 // Build provider-specific request
-                const { headers, body } = buildRerankRequest(provider, this.config.rerankApiKey || "", model, query, documents, results.length);
+                const { headers, body } = buildRerankRequest(provider, this.config.rerankApiKey || "", model, query, documents, rerankTopN);
                 // Timeout: configurable via rerankTimeoutMs (default: 5000ms)
                 const controller = new AbortController();
                 const timeout = setTimeout(() => controller.abort(), this.config.rerankTimeoutMs ?? 5000);

dist/src/store.js

@@ -29,6 +29,21 @@ export function __setLockfileModuleForTests(module) {
 }
 export const loadLanceDB = async () => {
     if (!lancedbImportPromise) {
+        // @lancedb/lancedb's napi-rs loader (dist/native.js) detects musl via
+        // process.report.getReport(). The report's network section performs
+        // reverse-DNS lookups for every network interface, synchronously on the
+        // calling thread; on hosts with several interfaces and a slow or flaky
+        // DNS path (measured on WSL2 + Tailscale) this blocks the event loop for
+        // 110-250s on the FIRST LanceDB load — the host app's HTTP server and
+        // pollers freeze with CPU at 0% (main thread stuck in poll()). Excluding
+        // the network section turns the measured 235s hang into ~3ms and loses
+        // nothing: isMusl() only reads report.header.glibcVersionRuntime.
+        try {
+            process.report.excludeNetwork = true;
+        }
+        catch {
+            /* Node < 22 without the flag — keep the previous behavior */
+        }
         // Use a createRequire-built require() so LanceDB's CommonJS native bindings
         // keep Windows-safe CJS semantics while still working in pure ESM runtimes.
         // Do not name this binding "require": bundlers may rewrite bare require()
@@ -50,7 +65,11 @@ function clampInt(value, min, max) {
         return min;
     return Math.min(max, Math.max(min, Math.floor(value)));
 }
+const LEGACY_STABLE_MEMORY_ID_REGEX = /^[A-Za-z0-9][A-Za-z0-9._:-]{2,127}$/;
 const LEGACY_SECONDS_TIMESTAMP_MAX = 1_000_000_000_000;
+function isLegacyStableMemoryId(id) {
+    return LEGACY_STABLE_MEMORY_ID_REGEX.test(id);
+}
 export function normalizeMemoryTimestamp(value, fallback = Date.now()) {
     const raw = value instanceof Date
         ? value.getTime()
@@ -1704,20 +1723,19 @@ export class MemoryStore {
             throw new Error(`Memory ${id} is outside accessible scopes`);
         }
         return this.runWithFileLock(() => this.runSerializedUpdate(async () => {
-            // Support both full UUID and short prefix (8+ hex chars), same as delete()
-            // Also support legacy mem-md-N format from older memory-lancedb-pro versions
+            // Support full UUID, short hex prefixes, and constrained exact legacy IDs imported
+            // from older stores (for example "mem-md-..." or "data-pointer-...").
             const uuidRegex = /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i;
             const prefixRegex = /^[0-9a-f]{8,}$/i;
-            const legacyRegex = /^mem-md-\d+$/i;
             const isFullId = uuidRegex.test(id);
             const isPrefix = !isFullId && prefixRegex.test(id);
-            const isLegacy = !isFullId && !isPrefix && legacyRegex.test(id);
-            if (!isFullId && !isPrefix && !isLegacy) {
+            const isLegacyStableId = !isFullId && !isPrefix && isLegacyStableMemoryId(id);
+            if (!isFullId && !isPrefix && !isLegacyStableId) {
                 throw new Error(`Invalid memory ID format: ${id}`);
             }
             let rows;
-            if (isFullId || isLegacy) {
-                // Legacy IDs use exact string match like full UUIDs
+            if (isFullId || isLegacyStableId) {
+                // Legacy IDs use exact string match like full UUIDs.
                 const safeId = escapeSqlLiteral(id);
                 rows = await this.table.query()
                     .where(`id = '${safeId}'`)

scripts/verify-package-runtime.mjs

@@ -1,4 +1,5 @@
 import { existsSync, readFileSync, statSync } from "node:fs";
+import { spawnSync } from "node:child_process";
 import path from "node:path";
 import { fileURLToPath } from "node:url";
 
@@ -14,6 +15,18 @@ function readJson(relativePath) {
   return JSON.parse(readFileSync(path.join(repoRoot, relativePath), "utf8"));
 }
 
+function verifyDistFreshness() {
+  const result = spawnSync("git", ["diff", "--quiet", "--", "dist"], {
+    cwd: repoRoot,
+    encoding: "utf8",
+  });
+  if (result.status === 0) return;
+  if (result.status === 1) {
+    fail("dist/ is stale after build; run npm run build and commit the generated dist output");
+  }
+  fail(`could not verify dist freshness with git diff: ${result.stderr || result.stdout || `exit ${result.status}`}`);
+}
+
 function normalizeRuntimePath(value) {
   if (typeof value !== "string" || value.trim().length === 0) {
     fail(`invalid runtime path ${JSON.stringify(value)}`);
@@ -54,4 +67,6 @@ if (!files.includes("dist/**/*")) {
   fail('package.json files must include "dist/**/*" so compiled runtime output is published');
 }
 
-console.log("Package runtime entries point to compiled dist output");
+verifyDistFreshness();
+
+console.log("Package runtime entries point to compiled fresh dist output");

test/package-runtime.test.mjs

@@ -25,5 +25,5 @@ const result = spawnSync(process.execPath, ["scripts/verify-package-runtime.mjs"
 assert.equal(
   result.status,
   0,
-  result.stderr || result.stdout || "verify-package-runtime.mjs should pass",
+  result.stderr || result.stdout || "verify-package-runtime.mjs should pass and dist should be fresh",
 );