gitnexus 1.6.4-rc.79 → 1.6.4-rc.80

package/dist/core/group/bridge-db.js

@@ -1,6 +1,6 @@
 import fsp from 'node:fs/promises';
 import path from 'node:path';
-import { createHash } from 'node:crypto';
+import { createHash, randomBytes } from 'node:crypto';
 import lbug from '@ladybugdb/core';
 import { BRIDGE_SCHEMA_QUERIES, BRIDGE_SCHEMA_VERSION } from './bridge-schema.js';
 import { closeLbugConnection, openLbugConnection, } from '../lbug/lbug-config.js';
@@ -17,7 +17,7 @@ import { dedupeContracts, dedupeCrossLinks } from './normalization.js';
  * - `.shadow` — non-blocking concurrent checkpoint sidecar (added in
  *   LadybugDB 0.15.4); same pairing constraint as `.wal`.
  *
- * `bridge-db` writes to a `bridge.lbug.tmp` file and then atomically renames
+ * `bridge-db` writes to a `bridge.lbug.tmp.<random>` file and then atomically renames
  * it into place. The rename only moves the main file; sidecars must be
  * cleaned up explicitly or the next writer trips the database-id check.
  */
@@ -33,6 +33,24 @@ async function removeLbugFile(basePath) {
         }
     }
 }
+/**
+ * Remove all stale `bridge.lbug.tmp.*` files (and their sidecars) from a
+ * group directory.  With randomBytes-based temp names, a crashed writeBridge
+ * leaves behind a uniquely-named tmp file that no future run will target by
+ * name — so we glob for the prefix and clean up everything matching.
+ */
+async function cleanStaleBridgeTmpFiles(groupDir) {
+    try {
+        const entries = await fsp.readdir(groupDir);
+        const staleBases = entries.filter((e) => e.startsWith('bridge.lbug.tmp.') && !LBUG_SIDECAR_SUFFIXES.some((s) => e.endsWith(s)));
+        for (const name of staleBases) {
+            await removeLbugFile(path.join(groupDir, name));
+        }
+    }
+    catch {
+        /* best-effort: directory may not exist yet */
+    }
+}
 export function contractNodeId(repo, contractId, role, filePath) {
     return createHash('sha256').update(`${repo}\0${contractId}\0${role}\0${filePath}`).digest('hex');
 }
@@ -211,7 +229,7 @@ export async function retryRename(src, dst, attempts = 3) {
 /* ------------------------------------------------------------------ */
 export async function writeBridgeMeta(groupDir, meta) {
     const target = path.join(groupDir, 'meta.json');
-    const tmp = `${target}.tmp.${Date.now()}`;
+    const tmp = `${target}.tmp.${randomBytes(8).toString('hex')}`;
     await fsp.writeFile(tmp, JSON.stringify(meta, null, 2), 'utf-8');
     // Use retryRename for consistency with writeBridge's atomic swap — on
     // Windows a concurrent reader can cause EBUSY/EPERM even on a tiny
@@ -244,7 +262,7 @@ export async function writeBridge(groupDir, input) {
     const contracts = dedupeContracts(input.contracts);
     const crossLinks = dedupeCrossLinks(input.crossLinks);
     const finalPath = path.join(groupDir, 'bridge.lbug');
-    const tmpPath = path.join(groupDir, 'bridge.lbug.tmp');
+    const tmpPath = path.join(groupDir, `bridge.lbug.tmp.${randomBytes(8).toString('hex')}`);
     const bakPath = path.join(groupDir, 'bridge.lbug.bak');
     const report = {
         contractsInserted: 0,
@@ -261,11 +279,12 @@ export async function writeBridge(groupDir, input) {
             report.sampleErrors.push({ kind, id, message: errMessage(err) });
         }
     };
-    // Clean up any leftover tmp main file AND its `.wal` / `.shadow` sidecars.
-    // LadybugDB 0.16.0 rejects opening a database whose sidecars belong to a
-    // different database instance (database-id check), so any stale sidecar
-    // from a crashed previous run will fail the next writeBridge.
-    await removeLbugFile(tmpPath);
+    // Clean up stale tmp files left behind by previously crashed writeBridge
+    // runs.  With randomBytes-based names each run picks a unique path, so
+    // the old fixed-name `removeLbugFile(tmpPath)` was a no-op — stale
+    // artifacts accumulated.  The glob-based helper finds *all* leftover
+    // `bridge.lbug.tmp.*` entries and removes them (including sidecars).
+    await cleanStaleBridgeTmpFiles(groupDir);
     // 1. Create temp DB, insert all data.
     //
     // Everything after `openBridgeDb` must run inside a try/finally so that

package/dist/core/group/storage.js

@@ -2,6 +2,7 @@ import * as fs from 'node:fs';
 import * as fsp from 'node:fs/promises';
 import * as path from 'node:path';
 import * as os from 'node:os';
+import { randomBytes } from 'node:crypto';
 const CONTRACTS_FILE = 'contracts.json';
 export function getDefaultGitnexusDir() {
     return process.env.GITNEXUS_HOME || path.join(os.homedir(), '.gitnexus');
@@ -21,7 +22,7 @@ export function getGroupDir(gitnexusDir, groupName) {
 }
 export async function writeContractRegistry(groupDir, registry) {
     const targetPath = path.join(groupDir, CONTRACTS_FILE);
-    const tmpPath = `${targetPath}.tmp.${Date.now()}`;
+    const tmpPath = `${targetPath}.tmp.${randomBytes(8).toString('hex')}`;
     await fsp.writeFile(tmpPath, JSON.stringify(registry, null, 2), 'utf-8');
     await fsp.rename(tmpPath, targetPath);
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gitnexus",
-  "version": "1.6.4-rc.79",
+  "version": "1.6.4-rc.80",
   "description": "Graph-powered code intelligence for AI agents. Index any codebase, query via MCP or CLI.",
   "author": "Abhigyan Patwari",
   "license": "PolyForm-Noncommercial-1.0.0",