@telorun/sql-sqlite 0.2.0 → 0.2.1

package/README.md

@@ -23,7 +23,7 @@ Built to be language-agnostic and infinitely extensible.
 
 ```bash
 # Reconcile your manifest into a running backend
-$ telo ./examples/hello-api.yaml
+$ telo ./examples/hello-api
 
 {"level":30,"time":1771610393008,"pid":1310178,"hostname":"dev","msg":"Server listening at http://127.0.0.1:8844"}
 ```

package/dist/sqlite-driver-bun.js

@@ -5,7 +5,14 @@ export function openDatabase(file) {
         prepare(sql) {
             const stmt = db.prepare(sql);
             return {
-                reader: true,
+                // A statement is a reader iff it yields a result set. bun:sqlite has no
+                // `reader` flag (better-sqlite3 does), so derive it from the output
+                // columns: SELECT and `... RETURNING` expose column names, plain
+                // INSERT/UPDATE/DELETE expose none. Kysely routes readers through
+                // `all()` and everything else through `run()` — getting this wrong sent
+                // every mutation down the `all()` path, so `numAffectedRows` was never
+                // reported (rowCount always 0).
+                reader: stmt.columnNames.length > 0,
                 all(params) {
                     return stmt.all(...params);
                 },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@telorun/sql-sqlite",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "Telo SqlSqlite.Connection — SQLite backend for the Sql.Connection abstract (better-sqlite3 / bun:sqlite, transactional DDL).",
   "keywords": [
     "telo",

package/src/sqlite-driver-bun.ts

@@ -8,7 +8,14 @@ export function openDatabase(file: string): SqliteDb {
     prepare(sql: string) {
       const stmt = db.prepare(sql);
       return {
-        reader: true,
+        // A statement is a reader iff it yields a result set. bun:sqlite has no
+        // `reader` flag (better-sqlite3 does), so derive it from the output
+        // columns: SELECT and `... RETURNING` expose column names, plain
+        // INSERT/UPDATE/DELETE expose none. Kysely routes readers through
+        // `all()` and everything else through `run()` — getting this wrong sent
+        // every mutation down the `all()` path, so `numAffectedRows` was never
+        // reported (rowCount always 0).
+        reader: stmt.columnNames.length > 0,
         all(params: ReadonlyArray<unknown>) {
           return stmt.all(...(params as any[]));
         },