@homeofthings/sqlite3 6.3.1 → 7.0.0

package/package.json

@@ -1,18 +1,12 @@
 {
   "name": "@homeofthings/sqlite3",
   "description": "Asynchronous, non-blocking SQLite3 bindings",
-  "version": "6.3.1",
+  "version": "7.0.0",
   "homepage": "https://github.com/gms1/node-sqlite3",
   "author": {
     "name": "Mapbox",
     "url": "https://mapbox.com/"
   },
-  "binary": {
-    "napi_versions": [
-      3,
-      6
-    ]
-  },
   "contributors": [
     "Daniel Lockyer <hi@daniellockyer.com>",
     "Konstantin Käfer <mail@kkaefer.com>",
@@ -36,7 +30,9 @@
     "binding.gyp",
     "deps/",
     "lib/**/*.js",
+    "lib/**/*.mjs",
     "lib/**/*.d.ts",
+    "prebuilds/**",
     "src/"
   ],
   "repository": {
@@ -44,18 +40,16 @@
     "url": "https://github.com/gms1/node-sqlite3.git"
   },
   "dependencies": {
-    "bindings": "^1.5.0",
     "node-addon-api": "^8.7.0",
-    "prebuild-install": "^7.1.3",
-    "tar": "^7.5.13"
+    "node-gyp-build": "^4.8.4"
   },
   "devDependencies": {
     "@eslint/js": "^10.0.1",
     "eslint": "^10.2.0",
-    "globals": "^17.4.0",
+    "globals": "^17.5.0",
     "mocha": "11.7.5",
     "nyc": "^18.0.0",
-    "prebuild": "13.0.1",
+    "prebuildify": "^6.0.1",
     "tinybench": "^6.0.0"
   },
   "peerDependencies": {
@@ -73,13 +67,13 @@
     "node": ">=20.17.0"
   },
   "scripts": {
-    "install": "prebuild-install -r napi || node-gyp rebuild",
-    "prebuild": "prebuild --runtime napi --all --verbose",
+    "install": "node-gyp-build",
+    "prebuild": "prebuildify --napi --strip --tag-libc",
     "rebuild": "node-gyp rebuild",
-    "upload": "prebuild --verbose --prerelease",
     "frozen-install": "yarn install --frozen-lockfile",
     "lint": "eslint lib/ test/ tools/",
-    "test": "node test/support/createdb.js && nyc mocha -R spec --timeout 480000"
+    "test": "node test/support/createdb.js && nyc --reporter=text --reporter=lcov  --reporter=html mocha -R spec --timeout 480000 \"test/*.test.js\" \"test/*.test.mjs\"",
+    "bear": "bear -- node-gyp rebuild"
   },
   "license": "BSD-3-Clause",
   "publishConfig": {
@@ -91,6 +85,21 @@
     "sqlite3",
     "database"
   ],
-  "main": "./lib/sqlite3",
-  "types": "./lib/sqlite3.d.ts"
+  "main": "./lib/sqlite3.js",
+  "module": "./lib/sqlite3.mjs",
+  "types": "./lib/sqlite3.d.ts",
+  "exports": {
+    ".": {
+      "types": "./lib/sqlite3.d.ts",
+      "import": "./lib/sqlite3.mjs",
+      "require": "./lib/sqlite3.js",
+      "default": "./lib/sqlite3.js"
+    },
+    "./promise": {
+      "types": "./lib/promise/index.d.ts",
+      "import": "./lib/promise/index.mjs",
+      "require": "./lib/promise/index.js",
+      "default": "./lib/promise/index.js"
+    }
+  }
 }

package/src/backup.cc

@@ -173,6 +173,14 @@ Backup::Backup(const Napi::CallbackInfo& info) : Napi::ObjectWrap<Backup>(info)
     baton->filenameIsDest = filenameIsDest.Value();
 
     this->db->Schedule(Work_BeginInitialize, baton);
+
+    // Initialize retryErrors with default values so GetRetryErrors() never
+    // crashes on an empty Napi::Reference, even if the JS wrapper that sets
+    // backup.retryErrors is bypassed (i.e. direct new sqlite3.Backup(...)).
+    Napi::Array defaultRetryErrors = Napi::Array::New(env, 2);
+    defaultRetryErrors.Set(static_cast<uint32_t>(0), Napi::Number::New(env, SQLITE_BUSY));
+    defaultRetryErrors.Set(static_cast<uint32_t>(1), Napi::Number::New(env, SQLITE_LOCKED));
+    retryErrors.Reset(defaultRetryErrors, 1);
 }
 
 void Backup::Work_BeginInitialize(Database::Baton* baton) {

package/src/macros.h

@@ -4,14 +4,10 @@
 const char* sqlite_code_string(int code);
 const char* sqlite_authorizer_string(int type);
 #include <vector>
-#include <atomic>
 
 // TODO: better way to work around StringConcat?
 #include <napi.h>
 
-// Forward declaration of shutdown flag from node_sqlite3.cc
-extern std::atomic<bool> g_env_shutting_down;
-
 inline Napi::String StringConcat(Napi::Value str1, Napi::Value str2) {
   return Napi::String::New(str1.Env(), str1.As<Napi::String>().Utf8Value() +
                     str2.As<Napi::String>().Utf8Value() );
@@ -133,21 +129,8 @@ inline bool OtherIsInt(Napi::Number source) {
     if ((argc != 0) && (passed_argv != NULL)) {\
       args.assign(passed_argv, passed_argv + argc);\
     }\
-    try {\
-        Napi::Value res = (callback).Call(Napi::Value(context), args);\
-        if (res.IsEmpty()) return __VA_ARGS__;\
-    } catch (const Napi::Error&) {\
-        /* During Node.js/Electron shutdown, when using NAPI_CPP_EXCEPTIONS,*/\
-				/* we must take care to not throw any exceptions. */\
-        /* But when napi_call_function returns napi_cannot_run_js */\
-        /* this throws a C++ exception, when NAPI_CPP_EXCEPTIONS is enabled. */\
-        if (g_env_shutting_down.load()) {\
-             /* We need to silently ignore exceptions during shutdown. */\
-            return __VA_ARGS__;\
-        }\
-        /* Real rror - re-throw to propagate it */\
-        throw;\
-    }
+    Napi::Value res = (callback).Call(Napi::Value(context), args);\
+    if (res.IsEmpty()) return __VA_ARGS__;
 
 #define WORK_DEFINITION(name)                                                  \
     Napi::Value name(const Napi::CallbackInfo& info);                          \

package/src/node_sqlite3.cc

@@ -2,7 +2,6 @@
 #include <sstream>
 #include <cstring>
 #include <string>
-#include <atomic>
 #include <sqlite3.h>
 
 #include "macros.h"
@@ -12,21 +11,11 @@
 
 using namespace node_sqlite3;
 
-// Global flag set when the environment is shutting down.
-std::atomic<bool> g_env_shutting_down{false};
-
-static void EnvCleanupHook(void* /*data*/) {
-    g_env_shutting_down.store(true);
-}
-
 namespace {
 
 Napi::Object RegisterModule(Napi::Env env, Napi::Object exports) {
     Napi::HandleScope scope(env);
 
-    // Register cleanup hook to detect shutdown
-    napi_add_env_cleanup_hook(env, EnvCleanupHook, nullptr);
-
     Database::Init(env, exports);
     Statement::Init(env, exports);
     Backup::Init(env, exports);

package/deps/extract.js

@@ -1,19 +0,0 @@
-const tar = require("tar");
-const fs = require("fs");
-const path = require("path");
-const tarball = path.resolve(process.argv[2]);
-const dirname = path.resolve(process.argv[3]);
-
-tar.extract({
-    sync: true,
-    file: tarball,
-    cwd: dirname,
-});
-
-// SQLite >= 3.49 ships a VERSION file that conflicts with the C++20 <version>
-// header on case-insensitive filesystems (macOS/Windows).
-const base = path.basename(tarball, ".tar.gz");
-const versionFile = path.join(dirname, base, "VERSION");
-if (fs.existsSync(versionFile)) {
-    fs.unlinkSync(versionFile);
-}