@telorun/sql-sqlite 0.2.0 → 0.2.2

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"}
 ```
@@ -44,146 +44,7 @@ $ telo ./examples/hello-api.yaml
 
 ## Example manifest
 
-Here is an example Telo application that defines a simple HTTP API:
-
-```yaml
-kind: Telo.Application
-metadata:
-  name: feedback
-  version: 1.0.0
-  description: |
-    A complete feedback collection REST API — no code, pure YAML.
-    Persists entries to SQLite and serves them over HTTP.
-imports:
-  Http: std/http-server@0.12.0
-  Sql: std/sql@0.9.2
-targets:
-  - !ref Migrations
-  - !ref Server
----
-# SQLite database — swap driver/host/database for PostgreSQL with zero YAML changes
-kind: Sql.Connection
-metadata:
-  name: Db
-driver: sqlite
-file: ./tmp/feedback.db
----
-# Migrations: applied automatically before the server starts
-kind: Sql.Migrations
-metadata:
-  name: Migrations
-connection: !ref Db
----
-kind: Sql.Migration
-metadata:
-  name: Migration_20260413_182154_CreateFeedback
-version: 20260413_182154_CreateFeedback
-sql: |
-  CREATE TABLE IF NOT EXISTS feedback (
-    id        INTEGER PRIMARY KEY AUTOINCREMENT,
-    text      TEXT    NOT NULL,
-    source    TEXT,
-    score     INTEGER NOT NULL DEFAULT 0,
-    created_at DATETIME DEFAULT CURRENT_TIMESTAMP
-  )
----
-kind: Http.Server
-metadata:
-  name: Server
-baseUrl: http://localhost:8844
-port: 8844
-logger: true
-openapi:
-  info:
-    title: Feedback API
-    version: 1.0.0
-mounts:
-  - path: /v1
-    mount: !ref FeedbackRoutes
----
-kind: Http.Api
-metadata:
-  name: FeedbackRoutes
-routes:
-  # POST /v1/feedback — insert a new entry, score derived from body length heuristic
-  - request:
-      path: /feedback
-      method: POST
-      schema:
-        body:
-          type: object
-          properties:
-            text:
-              type: string
-              minLength: 1
-            source:
-              type: string
-          required: [ text ]
-    handler:
-      kind: Sql.Exec
-      connection: !ref Db
-    inputs:
-      sql: "INSERT INTO feedback (text, source, score) VALUES (?, ?, ?)"
-      bindings:
-        - "${{ request.body.text }}"
-        - "${{ request.body.source }}"
-        - "${{ size(request.body.text) }}"
-    response:
-      - status: 201
-        headers:
-          Content-Type: application/json
-        body:
-          ok: true
-          message: Feedback received
-
-  # GET /v1/feedback — list all entries, newest first
-  - request:
-      path: /feedback
-      method: GET
-    handler:
-      kind: Sql.Select
-      connection: !ref Db
-      from: feedback
-      columns: [ id, text, source, score, created_at ]
-      orderBy:
-        - { column: created_at, direction: desc }
-    response:
-      - status: 200
-        headers:
-          Content-Type: application/json
-        body: "${{ result.rows }}"
-
-  # GET /v1/feedback/{id} — fetch a single entry
-  - request:
-      path: /feedback/{id}
-      method: GET
-      schema:
-        params:
-          type: object
-          properties:
-            id:
-              type: integer
-          required: [ id ]
-    handler:
-      kind: Sql.Select
-      connection: !ref Db
-      from: feedback
-      columns: [ id, text, source, score, created_at ]
-      where:
-        - { column: id, op: "=", value: "${{ request.params.id }}" }
-    response:
-      - status: 200
-        when: "size(result.rows) > 0"
-        headers:
-          Content-Type: application/json
-        body: "${{ result.rows[0] }}"
-      - status: 404
-        headers:
-          Content-Type: application/json
-        body:
-          ok: false
-          message: Not found
-```
+See [examples/](./examples/) for a list of working applications.
 
 ## Status
 

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.2",
   "description": "Telo SqlSqlite.Connection — SQLite backend for the Sql.Connection abstract (better-sqlite3 / bun:sqlite, transactional DDL).",
   "keywords": [
     "telo",
@@ -36,14 +36,14 @@
   "dependencies": {
     "better-sqlite3": "^12.8.0",
     "kysely": "^0.28.15",
-    "@telorun/sql": "0.8.0"
+    "@telorun/sql": "0.9.0"
   },
   "devDependencies": {
     "@types/better-sqlite3": "^7.0.0",
     "@types/bun": "^1.3.10",
     "@types/node": "^20.0.0",
     "typescript": "^5.0.0",
-    "@telorun/sdk": "0.34.0"
+    "@telorun/sdk": "0.54.0"
   },
   "peerDependencies": {
     "@telorun/sdk": "*"

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[]));
         },