@sqb/sqlite 6.0.13 → 6.1.1

package/README.md

@@ -42,7 +42,7 @@ import { SqbClient } from '@sqb/connect';
 
 const client = new SqbClient({
   driver: 'sqlite',
-  database: '/path/to/database.db', // or ':memory:'
+  database: '/path/to/database.db', // or ':memory:' (optionally suffixed, e.g. ':memory:test1')
 });
 ```
 

package/constants.d.ts

@@ -1,3 +1,3 @@
 /** This package's own version, kept in step with its `package.json` by `rman version`
  *  (see `.rmanrc` `version.stamp`) - never edit it by hand. */
-export declare const version = "6.0.13";
+export declare const version = "6.1.1";

package/constants.js

@@ -1,3 +1,3 @@
 /** This package's own version, kept in step with its `package.json` by `rman version`
  *  (see `.rmanrc` `version.stamp`) - never edit it by hand. */
-export const version = '6.0.13';
+export const version = '6.1.1';

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@sqb/sqlite",
   "description": "SQB adapter for Node.js and Bun's native SQLite drivers",
-  "version": "6.0.13",
+  "version": "6.1.1",
   "author": "Panates",
   "license": "BSD-3-Clause",
   "homepage": "https://www.sqbjs.org",
@@ -9,9 +9,9 @@
     "tslib": "^2.8.1"
   },
   "peerDependencies": {
-    "@sqb/builder": "^6.0.13",
-    "@sqb/connect": "^6.0.13",
-    "@sqb/sqlite-dialect": "^6.0.13"
+    "@sqb/builder": "^6.1.1",
+    "@sqb/connect": "^6.1.1",
+    "@sqb/sqlite-dialect": "^6.1.1"
   },
   "type": "module",
   "module": "./index.js",

package/sqlite-adapter.js

@@ -45,7 +45,10 @@ export class SqliteAdapter {
             intlDb._refCount++;
         }
         else {
-            intlDb = (await openDatabase(dbName));
+            // SQLite only knows the bare `:memory:`; a `:memory:name` passed
+            // through as-is would open a file of that name. Every name gets its
+            // own `:memory:` handle, kept apart by the cache key.
+            intlDb = (await openDatabase(isMemory ? ':memory:' : dbName));
             intlDb._refCount = isMemory ? 0 : 1;
             dbCache.set(dbName, intlDb);
         }

package/sqlite-connection.d.ts

@@ -33,10 +33,11 @@ export declare class SqliteConnection implements Adapter.Connection {
      * runs the statement then a synthesized follow-up `SELECT` to emulate
      * `RETURNING` (an `INSERT` is matched back by `ROWID`, an `UPDATE` by
      * reusing the original `WHERE` clause) - `DELETE` isn't emulated here
-     * since there's nothing left to re-select afterward. Otherwise prepares
-     * and iterates the statement, wrapping it in a {@link SqliteCursor}
-     * (returned directly in cursor mode, or eagerly drained up to
-     * `fetchRows` rows otherwise).
+     * since there's nothing left to re-select afterward. A statement that
+     * produces no result set (DDL, `PRAGMA`, ...) is simply run. Otherwise
+     * prepares and iterates the statement, wrapping it in a {@link
+     * SqliteCursor} (returned directly in cursor mode, or eagerly drained up
+     * to `fetchRows` rows otherwise).
      */
     execute(query: QueryRequest): Promise<Adapter.Response>;
     /** Copies a synthesized `SELECT` statement's rows/fields onto `out` - used to emulate `RETURNING` on `INSERT`/`UPDATE`. */

package/sqlite-connection.js

@@ -79,10 +79,11 @@ export class SqliteConnection {
      * runs the statement then a synthesized follow-up `SELECT` to emulate
      * `RETURNING` (an `INSERT` is matched back by `ROWID`, an `UPDATE` by
      * reusing the original `WHERE` clause) - `DELETE` isn't emulated here
-     * since there's nothing left to re-select afterward. Otherwise prepares
-     * and iterates the statement, wrapping it in a {@link SqliteCursor}
-     * (returned directly in cursor mode, or eagerly drained up to
-     * `fetchRows` rows otherwise).
+     * since there's nothing left to re-select afterward. A statement that
+     * produces no result set (DDL, `PRAGMA`, ...) is simply run. Otherwise
+     * prepares and iterates the statement, wrapping it in a {@link
+     * SqliteCursor} (returned directly in cursor mode, or eagerly drained up
+     * to `fetchRows` rows otherwise).
      */
     async execute(query) {
         assertDefined(this.intlcon);
@@ -119,11 +120,20 @@ export class SqliteConnection {
             return out;
         }
         const stmt = intlcon.prepare(query.sql);
+        if (!stmt.columns().length) {
+            // A statement with no result set (DDL, PRAGMA, `INSERT OR REPLACE`,
+            // ...). `iterate()` is lazy - it would not run the statement until
+            // its first `next()` - so run it outright.
+            const info = stmt.run(params);
+            if (/^\s*(insert|replace|update|delete)\b/i.test(query.sql))
+                out.rowsAffected = info.changes;
+            if (query.autoCommit)
+                await this.commit();
+            return out;
+        }
         const rowType = query.objectRows ? 'object' : 'array';
         const iterator = stmt.iterate(params);
         const columns = stmt.columns();
-        if (!columns.length)
-            return out;
         out.fields = this._convertFields(columns);
         out.rowType = rowType;
         const cursor = new SqliteCursor(iterator, { rowType, columns });