fix(ext/node): move sqlite_extension into separate module from sqlite_extension_test to permit loadable_extension feature

This allows the rusqlite `loadable_extension` feature to be used -- without it, we were building an extension that triggered a segfault on load (because it wasn't using the pointer provided during extension load to access the previously-initialized sqlite library instance). Because that feature conflicts with other features used _outside_ the `sqlite_extension_test` module, we split off to a separate workspace.

The test harness stays in `sqlite_extension_test`, as it depends on other parts of the workspace.

Author: charles-dyfis-net

Cargo.lock

@@ -51,7 +51,9 @@ members = [
   "tests/sqlite_extension_test",
   "tests/util/server",
 ]
-exclude = ["tests/util/std/hash/_wasm"]
+# tests/sqlite_extension is excluded because it requires rusqlite's "loadable_extension"
+# feature, which is incompatible with the "session" feature used by the rest of the workspace.
+exclude = ["tests/util/std/hash/_wasm", "tests/sqlite_extension"]
 
 [workspace.package]
 authors = ["the Deno authors"]

Cargo.toml

@@ -0,0 +1,23 @@
+# Copyright 2018-2025 the Deno authors. MIT license.
+
+# This crate is excluded from the workspace because it requires rusqlite's
+# "loadable_extension" feature, which is incompatible with the "session" feature
+# used by the rest of the workspace. Build with:
+#   cargo build --manifest-path tests/sqlite_extension/Cargo.toml
+
+[package]
+name = "test_sqlite_extension"
+version = "0.1.0"
+authors = ["the Deno authors"]
+edition = "2024"
+license = "MIT"
+publish = false
+repository = "https://github.com/denoland/deno"
+
+[lib]
+crate-type = ["cdylib"]
+
+[workspace]
+
+[dependencies]
+rusqlite = { version = "0.37.0", default-features = false, features = ["loadable_extension", "functions"] }

tests/sqlite_extension/Cargo.lock

@@ -0,0 +1,42 @@
+// Copyright 2018-2025 the Deno authors. MIT license.
+
+//! A simple SQLite loadable extension for testing.
+//!
+//! This extension provides a `test_func(x)` function that returns its argument unchanged.
+
+use std::os::raw::c_char;
+use std::os::raw::c_int;
+
+use rusqlite::Connection;
+use rusqlite::Result;
+use rusqlite::ffi;
+use rusqlite::functions::FunctionFlags;
+use rusqlite::types::ToSqlOutput;
+use rusqlite::types::Value;
+
+/// Entry point for SQLite to load the extension.
+#[expect(clippy::not_unsafe_ptr_arg_deref)]
+#[unsafe(no_mangle)]
+pub unsafe extern "C" fn sqlite3_extension_init(
+  db: *mut ffi::sqlite3,
+  pz_err_msg: *mut *mut c_char,
+  p_api: *mut ffi::sqlite3_api_routines,
+) -> c_int {
+  // SAFETY: This function is called by SQLite with valid pointers.
+  // extension_init2 handles the API initialization.
+  unsafe { Connection::extension_init2(db, pz_err_msg, p_api, extension_init) }
+}
+
+fn extension_init(db: Connection) -> Result<bool> {
+  db.create_scalar_function(
+    "test_func",
+    1,
+    FunctionFlags::SQLITE_DETERMINISTIC,
+    |ctx| {
+      // Return the argument value unchanged
+      let value: Value = ctx.get(0)?;
+      Ok(ToSqlOutput::Owned(value))
+    },
+  )?;
+  Ok(false)
+}