@converse/skeletor 3.1.0 → 3.1.2

package/package.json

@@ -36,7 +36,7 @@
   "contributors": [
     "JC Brand"
   ],
-  "version": "3.1.0",
+  "version": "3.1.2",
   "engines": {
     "node": ">=22.0.0"
   },
@@ -46,21 +46,23 @@
     "url": "https://github.com/conversejs/skeletor.git"
   },
   "scripts": {
-    "clean": "rm -rf dist",
-    "prebuild": "npm run clean && mkdir -p dist/defs",
     "build": "npm run prebuild && npm run build:declarations && rollup -c",
-    "build:types": "tsc -p tsconfig.declarations.json",
     "build:declarations": "npm run build:types && api-extractor run --local && rm -rf dist/defs",
+    "build:types": "tsc -p tsconfig.declarations.json",
     "check": "npm run check:types && npm run test && npm run test:node && npm run lint",
     "check:types": "tsc --noEmit",
+    "clean": "rm -rf dist",
     "dev": "karma start --single-run=false",
     "lint": "eslint src/*.ts test/*.ts",
+    "prebuild": "npm run clean && mkdir -p dist/defs",
+    "prepack": "npm run build",
     "prettier": "prettier --write src/**/*.js test/**/*.js src/**/*.ts test/**/*.ts",
     "test": "karma start && karma start --mocha-only",
     "test:node": "tsx node_modules/mocha/bin/mocha.js 'test/nodeStorage.test.ts'"
   },
   "files": [
     "LICENSE",
+    "CHANGES.md",
     "dist/skeletor.js",
     "dist/skeletor.esm.js",
     "dist/skeletor.js.map",
@@ -70,7 +72,7 @@
     "dist/skeletor.node.mjs",
     "dist/skeletor.d.ts",
     "dist/skeletor.d.ts.map",
-    "src/"
+    "src/**/*.ts"
   ],
   "dependencies": {
     "@converse/localforage-getitems": "1.4.3",

package/src/index.node.ts

@@ -3,6 +3,14 @@
  * Re-exports everything from the browser entry and additionally
  * exports the NodeSQLiteStorage LocalForage implementation.
  */
+import PersistentStorage from './storage';
+import { NodeSQLiteStorage } from './drivers/nodeSQLiteStorage';
+
+// Register the SQLite driver so `new PersistentStorage(id, 'node')` resolves it.
+// The browser builds leave this unset, keeping the node-only driver out of
+// their bundles entirely.
+PersistentStorage.nodeStorage = NodeSQLiteStorage;
+
 export * from './index';
 export { default } from './index';
-export { NodeSQLiteStorage } from './drivers/nodeSQLiteStorage';
+export { NodeSQLiteStorage };

package/src/storage.ts

@@ -33,6 +33,12 @@ const weakInstances = new Set<WeakRef<PersistentStorage>>();
 // same instance twice and make flushAll() flush it more than once.
 const registered = new WeakSet<PersistentStorage>();
 
+/**
+ * Constructor for a Node.js-only storage driver. Registered by the
+ * `@converse/skeletor/node` entry point via {@link PersistentStorage.nodeStorage}.
+ */
+type NodeStorageConstructor = new (name: string) => StorageDriver;
+
 /**
  * @public
  */
@@ -49,6 +55,11 @@ class PersistentStorage {
   static sessionStorageInitialized: Promise<void>;
   static localForage: typeof localForage;
 
+  // The Node.js SQLite storage driver. Left null in browser builds so the
+  // node-only driver (and its `node:sqlite` import) never enters the browser
+  // bundle; the `@converse/skeletor/node` entry sets it.
+  static nodeStorage: NodeStorageConstructor | null = null;
+
   /** Register a storage instance with a weak reference. @public */
   static register(instance: PersistentStorage): void {
     if (registered.has(instance)) return;
@@ -110,7 +121,13 @@ class PersistentStorage {
     } else if (type === 'in_memory') {
       await localForage.config({ 'driver': IN_MEMORY });
     } else if (type === 'node') {
-      const { NodeSQLiteStorage } = await import('./drivers/nodeSQLiteStorage');
+      const NodeSQLiteStorage = PersistentStorage.nodeStorage;
+      if (!NodeSQLiteStorage) {
+        throw new Error(
+          "Skeletor.storage: the 'node' storage type is only available in Node.js builds. " +
+            "Import from '@converse/skeletor/node' instead of '@converse/skeletor'."
+        );
+      }
       this.store = new NodeSQLiteStorage(this.name);
       if (batchedWrites) {
         this.store.debouncedSetItems = mergebounce((items: Record<string, any>) => this.store.setItems!(items), 50, {