fix(driver-sql): don't dump a scary ERR_DLOPEN stack for the native→wasm probe (#2892)

* fix(driver-sql): don't dump a scary ERR_DLOPEN stack for the native→wasm probe

When native better-sqlite3 has an ABI mismatch (`NODE_MODULE_VERSION` after a
Node upgrade), `resolveSqliteDriver`'s probe deliberately triggers the lazy
`.node` load to detect it and steps down to wasm SQLite with a clean one-line
notice (#2229). But `SqlDriver.connect()`'s best-effort `PRAGMA auto_vacuum`
is the first query to force that load, so the failure surfaced there first and
was logged with the full multi-line `ERR_DLOPEN_FAILED` stack — twice — making
a handled, non-fatal step-down look like a fatal crash to anyone reading the
dev console.

Detect the native-addon load failure (`code === 'ERR_DLOPEN_FAILED'` or a
`NODE_MODULE_VERSION` message) in that catch and log a concise, actionable
one-liner instead of the stack, pointing at `pnpm rebuild better-sqlite3`. The
following step-down notice still explains the outcome. Any OTHER PRAGMA failure
keeps the full warning (with stack) unchanged. Purely a log-noise fix — control
flow and the fallback behavior are untouched.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01UUWGgTT5s7XgBb2eLtzhfU

* chore(changeset): patch @objectstack/driver-sql for the ERR_DLOPEN log-noise fix

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01UUWGgTT5s7XgBb2eLtzhfU

---------

Co-authored-by: Claude <noreply@anthropic.com>

Author: os-zhuang

.changeset/driver-sql-quiet-dlopen-probe.md

@@ -0,0 +1,5 @@
+---
+"@objectstack/driver-sql": patch
+---
+
+Log a concise one-liner instead of the full `ERR_DLOPEN_FAILED` stack trace when native `better-sqlite3` cannot load (an ABI / `NODE_MODULE_VERSION` mismatch after a Node upgrade, or the native addon was never built). The native → wasm SQLite step-down is unchanged — this only stops a handled, non-fatal fallback from reading like a fatal crash in the dev console, and points at `pnpm rebuild better-sqlite3` for native speed. Any other `PRAGMA` failure keeps its full warning.

packages/plugins/driver-sql/src/sql-driver.ts

@@ -580,7 +580,33 @@ export class SqlDriver implements IDataDriver {
       try {
         await this.knex.raw('PRAGMA auto_vacuum = INCREMENTAL');
       } catch (e) {
-        this.logger.warn('Failed to set PRAGMA auto_vacuum=INCREMENTAL', e);
+        // A native better-sqlite3 load failure surfaces HERE first — this PRAGMA
+        // is the first query that forces the lazy `.node` addon to load. Two
+        // real-world variants, both handled by `resolveSqliteDriver`'s probe,
+        // which catches the failure and steps down to wasm SQLite with a clean
+        // one-line notice (#2229):
+        //   • ABI mismatch  — `ERR_DLOPEN_FAILED` / "…NODE_MODULE_VERSION 127…"
+        //                      (a stale prebuilt binary after a Node upgrade)
+        //   • not built     — "Could not locate the bindings file" (native addon
+        //                      never compiled, e.g. a fresh clone / blocked build)
+        // Dumping the full multi-line stack for either looks like a fatal crash
+        // to the reader (it isn't), so log a concise, actionable one-liner and
+        // let the step-down message that follows explain the outcome. Any OTHER
+        // PRAGMA failure keeps the full warning (with stack) as before.
+        const code = (e as { code?: string } | null | undefined)?.code;
+        const msg = e instanceof Error ? e.message : String(e);
+        const isNativeLoadFailure =
+          code === 'ERR_DLOPEN_FAILED' ||
+          code === 'MODULE_NOT_FOUND' ||
+          code === 'ERR_MODULE_NOT_FOUND' ||
+          /NODE_MODULE_VERSION|could not locate the bindings|was compiled against a different/i.test(msg);
+        if (isNativeLoadFailure) {
+          this.logger.warn(
+            'native better-sqlite3 unavailable (ABI mismatch or not built) — will step down to wasm SQLite; run `pnpm rebuild better-sqlite3` for native speed',
+          );
+        } else {
+          this.logger.warn('Failed to set PRAGMA auto_vacuum=INCREMENTAL', e);
+        }
       }
     }
   }