@vzn/vx 0.0.196 → 0.0.197

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vzn/vx",
-  "version": "0.0.196",
+  "version": "0.0.197",
   "description": "A fast, correct task runner and build cache for JavaScript monorepos.",
   "type": "module",
   "exports": {
@@ -17,10 +17,10 @@
     "node": ">=18"
   },
   "optionalDependencies": {
-    "@vzn/vx-linux-x64": "0.0.196",
-    "@vzn/vx-linux-arm64": "0.0.196",
-    "@vzn/vx-darwin-x64": "0.0.196",
-    "@vzn/vx-darwin-arm64": "0.0.196"
+    "@vzn/vx-linux-x64": "0.0.197",
+    "@vzn/vx-linux-arm64": "0.0.197",
+    "@vzn/vx-darwin-x64": "0.0.197",
+    "@vzn/vx-darwin-arm64": "0.0.197"
   },
   "dependencies": {
     "@anthropic-ai/sandbox-runtime": "^0.0.76"

package/src/cache/cache.ts

@@ -26,7 +26,16 @@
 // layer speaks is `CacheLayer` in layer.ts; `plugin-host.ts` enforces it.
 
 import { Database, type SQLQueryBindings } from 'bun:sqlite'
-import { accessSync, constants, existsSync, mkdirSync, renameSync, writeFileSync } from 'node:fs'
+import {
+  accessSync,
+  closeSync,
+  constants,
+  existsSync,
+  mkdirSync,
+  openSync,
+  renameSync,
+  writeFileSync,
+} from 'node:fs'
 import { readdir, rm, stat, unlink, writeFile } from 'node:fs/promises'
 import path from 'node:path'
 import { createTables } from './schema.js'
@@ -129,6 +138,26 @@ function unreadableIndexError(dbFile: string, err: unknown): UserError {
   )
 }
 
+/**
+ * SQLite reports a descriptor it could not get as `SQLITE_CANTOPEN`, never
+ * `EMFILE`, so an open under a low `ulimit -n` reached the user as a bare
+ * `vx: SQLiteError: unable to open database file` (D's lead, A-56). One
+ * open of a file that exists says which it was.
+ */
+function outOfFdsAtOpen(dbFile: string, err: unknown): UserError | undefined {
+  const code = (err as { code?: unknown } | null)?.code
+  if (typeof code !== 'string' || !code.startsWith('SQLITE_CANTOPEN')) return undefined
+  try {
+    closeSync(openSync(process.execPath, 'r'))
+    return undefined
+  } catch (probe) {
+    if (!isOutOfFds(probe)) return undefined
+    return new UserError(
+      `the cache index ${dbFile} could not be opened (${(err as Error).message}) — ${OUT_OF_FDS_HINT}`,
+    )
+  }
+}
+
 /** Say once, on the channel the opener has, that an upgrade emptied the index. */
 export function noteSchemaReset(cache: Cache, warn: (message: string) => void): void {
   if (cache.formatChange !== null) {
@@ -527,7 +556,11 @@ export class Cache implements CacheLayer {
     const absent = mode === 'inspect' && !existsSync(dbFile)
     this.writeBlocked = absent ? 'no index there yet' : openCacheDir(cacheDir)
     this.write = localPolicy.write && this.writeBlocked === null
-    this.db = new Database(absent ? ':memory:' : dbFile, { create: true })
+    try {
+      this.db = new Database(absent ? ':memory:' : dbFile, { create: true })
+    } catch (err) {
+      throw outOfFdsAtOpen(dbFile, err) ?? err
+    }
     // busy_timeout makes concurrent writers wait for the lock instead of
     // failing immediately with SQLITE_BUSY. Two parallel `vx run`
     // invocations in CI is a normal pattern; without this the second one
@@ -544,7 +577,16 @@ export class Cache implements CacheLayer {
       try {
         return read()
       } catch (err) {
-        if (!unreadableIndex(err)) throw err
+        if (!unreadableIndex(err)) {
+          const fds = outOfFdsAtOpen(dbFile, err)
+          if (fds === undefined) throw err
+          try {
+            closeDb(this.db)
+          } catch {
+            // The refusal is the error worth reporting.
+          }
+          throw fds
+        }
         try {
           closeDb(this.db)
         } catch {