@telorun/sql 0.22.1 → 0.23.0

package/dist/sql-connection-base.d.ts

@@ -6,8 +6,8 @@ import type { SqlConnection, SqlDialect } from "./sql-connection.js";
  * scoping, template binding and row-count normalization over a kysely instance.
  *
  * Backends extend it, supply their {@link SqlDialect}, and override only what is
- * genuinely theirs — `teardown` for resources kysely does not own, `executeScript`
- * where the driver has a native multi-statement path.
+ * genuinely theirs — extending `init`'s effect chain for resources kysely does
+ * not own, `executeScript` where the driver has a native multi-statement path.
  */
 export declare abstract class SqlConnectionBase implements SqlConnection {
     #private;
@@ -16,8 +16,10 @@ export declare abstract class SqlConnectionBase implements SqlConnection {
     protected readonly db: Kysely<any>;
     constructor(db: Kysely<any>, dialect: SqlDialect, ctx: ResourceContext);
     get kysely(): Kysely<any>;
-    init(): Promise<void>;
-    teardown(): Promise<void>;
+    /** The pool, and what destroys it. A backend that allocates more extends this
+     *  chain rather than overriding a separate teardown — so the two halves cannot
+     *  drift out of order. */
+    init(ctx: ResourceContext): import("@telorun/sdk").EffectChain<undefined>;
     runInTransaction<T>(body: (bind: (entry: ZoneEntry) => void) => Promise<T>): Promise<T>;
     hasOpenTransaction(ctx?: InvokeContext): boolean;
     bindsZone(zone: ZoneEntry): boolean;

package/dist/sql-connection-base.js

@@ -5,8 +5,8 @@ import { CompiledQuery } from "kysely";
  * scoping, template binding and row-count normalization over a kysely instance.
  *
  * Backends extend it, supply their {@link SqlDialect}, and override only what is
- * genuinely theirs — `teardown` for resources kysely does not own, `executeScript`
- * where the driver has a native multi-statement path.
+ * genuinely theirs — extending `init`'s effect chain for resources kysely does
+ * not own, `executeScript` where the driver has a native multi-statement path.
  */
 export class SqlConnectionBase {
     dialect;
@@ -26,14 +26,17 @@ export class SqlConnectionBase {
     get kysely() {
         return this.db;
     }
-    async init() {
-        await this.db.connection().execute(async () => {
-            // just checking
+    /** The pool, and what destroys it. A backend that allocates more extends this
+     *  chain rather than overriding a separate teardown — so the two halves cannot
+     *  drift out of order. */
+    init(ctx) {
+        return ctx.effect("connection pool", async () => {
+            await this.db.connection().execute(async () => {
+                // just checking
+            });
+            return { result: undefined, inverse: () => this.db.destroy() };
         });
     }
-    async teardown() {
-        await this.db.destroy();
-    }
     async runInTransaction(body) {
         this.ctx.log.debug("Transaction started");
         try {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@telorun/sql",
-  "version": "0.22.1",
+  "version": "0.23.0",
   "description": "Telo SQL module - SQL database resource kinds for Telo manifests.",
   "keywords": [
     "telo",
@@ -44,7 +44,7 @@
     "esbuild": "^0.25.12",
     "typescript": "^5.0.0",
     "vitest": "^2.1.8",
-    "@telorun/sdk": "0.80.0"
+    "@telorun/sdk": "0.82.0"
   },
   "peerDependencies": {
     "@telorun/sdk": "*"

package/src/sql-connection-base.ts

@@ -13,8 +13,8 @@ import type { SqlConnection, SqlDialect } from "./sql-connection.js";
  * scoping, template binding and row-count normalization over a kysely instance.
  *
  * Backends extend it, supply their {@link SqlDialect}, and override only what is
- * genuinely theirs — `teardown` for resources kysely does not own, `executeScript`
- * where the driver has a native multi-statement path.
+ * genuinely theirs — extending `init`'s effect chain for resources kysely does
+ * not own, `executeScript` where the driver has a native multi-statement path.
  */
 export abstract class SqlConnectionBase implements SqlConnection {
   protected readonly db: Kysely<any>;
@@ -38,16 +38,18 @@ export abstract class SqlConnectionBase implements SqlConnection {
     return this.db;
   }
 
-  async init(): Promise<void> {
-    await this.db.connection().execute(async () => {
-      // just checking
+  /** The pool, and what destroys it. A backend that allocates more extends this
+   *  chain rather than overriding a separate teardown — so the two halves cannot
+   *  drift out of order. */
+  init(ctx: ResourceContext) {
+    return ctx.effect("connection pool", async () => {
+      await this.db.connection().execute(async () => {
+        // just checking
+      });
+      return { result: undefined, inverse: () => this.db.destroy() };
     });
   }
 
-  async teardown(): Promise<void> {
-    await this.db.destroy();
-  }
-
   async runInTransaction<T>(
     body: (bind: (entry: ZoneEntry) => void) => Promise<T>,
   ): Promise<T> {