From 6603984233812167c5283e3d713e50034f14840d Mon Sep 17 00:00:00 2001 From: Colby McHenry Date: Wed, 20 May 2026 11:54:02 -0500 Subject: [PATCH] fix(db): require better-sqlite3 ^12.4.1 so Node 24 gets the native backend (#203) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit better-sqlite3 ^11.0.0 (latest 11.10.0) ships no prebuilt binary for Node 24's ABI (node-v137) and predates Node 24, so every Node 24 install silently fell back to the 5-10x-slower WASM backend. Bump to ^12.4.1 — the first 12.x with the Node 24 prebuild — and raise the engines floor to Node 20 (Node 18 is EOL and dropped from better-sqlite3 12.x prebuilds). Verified on macOS Node 24.15.0 (ABI 137): prebuilt binary used with no compiler (installs even with CC/CXX sabotaged), `codegraph init -i` shows no WASM banner, and `codegraph status` reports Backend: native. 639/639 tests pass on Node 22. Co-Authored-By: Claude Opus 4.7 (1M context) --- CHANGELOG.md | 20 ++++++++++++++++++++ package-lock.json | 13 ++++++++----- package.json | 4 ++-- 3 files changed, 30 insertions(+), 7 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 4f1508375..a3c76ee24 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -33,6 +33,12 @@ and adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html). setup is actually fast. `codegraph uninit` removes any hooks it installed. ### Changed +- **Minimum Node.js is now 20** (was 18). Node 18 is end-of-life and the + native SQLite binding (`better-sqlite3` 12.x) no longer ships a Node 18 + prebuilt binary. Node 22 LTS and Node 24 get the native backend out of the + box; on other Node versions CodeGraph still runs via the WASM fallback + (slower, but functional). Node 25+ remains blocked (V8 WASM JIT crash, see + [#81](https://github.com/colbymchenry/codegraph/issues/81)). - **MCP / explore**: `codegraph_explore` output is now adaptive to project size. The tool used to apply a fixed 35KB cap regardless of how large the codebase was, which on small projects (~100 files) produced bigger @@ -57,6 +63,20 @@ and adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html). Thanks to [@essopsp](https://github.com/essopsp) for the repro. ### Fixed +- **Native SQLite backend on Node 24**: indexing on Node 24 always dropped to + the 5-10x-slower WASM backend, printing a `better-sqlite3 unavailable` + warning that `npm rebuild better-sqlite3` / `xcode-select --install` could + not clear ([#203](https://github.com/colbymchenry/codegraph/issues/203)). + The bundled `better-sqlite3` was pinned to a v11 release that ships no + prebuilt binary for Node 24's ABI (`node-v137`), so every Node 24 install + silently degraded — and because CodeGraph is usually installed globally, the + `npm install` / `npm rebuild` people ran in their own project never touched + CodeGraph's copy. CodeGraph now requires `better-sqlite3` `^12.4.1`, whose + prebuilds include Node 24, so a fresh install on Node 22 or Node 24 gets the + native backend with no compiler. On an already-broken install, reinstall + CodeGraph (e.g. `npm install -g @colbymchenry/codegraph`) to pull the new + binding; `codegraph status` should then report `Backend: native`. Thanks to + [@Finndersen](https://github.com/Finndersen) for the report. - **MCP**: tools no longer fail with "CodeGraph not initialized" when the index actually exists. This hit clients that launch the MCP server from a directory other than your project and don't report a workspace root in `initialize` diff --git a/package-lock.json b/package-lock.json index 2d4e515a8..1b4ce89d6 100644 --- a/package-lock.json +++ b/package-lock.json @@ -31,10 +31,10 @@ "vitest": "^2.1.9" }, "engines": { - "node": ">=18.0.0 <25.0.0" + "node": ">=20.0.0 <25.0.0" }, "optionalDependencies": { - "better-sqlite3": "^11.0.0" + "better-sqlite3": "^12.4.1" } }, "node_modules/@clack/core": { @@ -992,15 +992,18 @@ "optional": true }, "node_modules/better-sqlite3": { - "version": "11.10.0", - "resolved": "https://registry.npmjs.org/better-sqlite3/-/better-sqlite3-11.10.0.tgz", - "integrity": "sha512-EwhOpyXiOEL/lKzHz9AW1msWFNzGc/z+LzeB3/jnFJpxu+th2yqvzsSWas1v9jgs9+xiXJcD5A8CJxAG2TaghQ==", + "version": "12.10.0", + "resolved": "https://registry.npmjs.org/better-sqlite3/-/better-sqlite3-12.10.0.tgz", + "integrity": "sha512-CyzaZRQKyHkB2ZInfTTl2nvT33EbDpjkLEbE8/Zck3Ll6O0qqvuGdrJ45HgtH+HykRg88ITY3AdreBGN70aBSQ==", "hasInstallScript": true, "license": "MIT", "optional": true, "dependencies": { "bindings": "^1.5.0", "prebuild-install": "^7.1.1" + }, + "engines": { + "node": "20.x || 22.x || 23.x || 24.x || 25.x || 26.x" } }, "node_modules/bindings": { diff --git a/package.json b/package.json index 60dc5c710..202e9a48d 100644 --- a/package.json +++ b/package.json @@ -51,9 +51,9 @@ "vitest": "^2.1.9" }, "optionalDependencies": { - "better-sqlite3": "^11.0.0" + "better-sqlite3": "^12.4.1" }, "engines": { - "node": ">=18.0.0 <25.0.0" + "node": ">=20.0.0 <25.0.0" } }