@cosmicdrift/kumiko-framework 0.383.0 → 0.383.1

package/dist/db/connection.d.ts

@@ -22,7 +22,11 @@ export type DbConnectionHandle = {
     close: () => Promise<void>;
 };
 export declare const DEFAULT_DB_CLOSE_TIMEOUT_SECONDS = 5;
-export declare function openUnboundedSiblingPool(db: unknown): PgClient | undefined;
+export type UnboundedSiblingPool = {
+    readonly client: PgClient;
+    readonly close: (timeoutSeconds: number) => Promise<void>;
+};
+export declare function openUnboundedSiblingPool(db: unknown): UnboundedSiblingPool | undefined;
 export declare function createDbConnection({ url, ...options }: CreateDbConnectionOptions): DbConnectionHandle;
 export declare function dbConnectionOptionsFromEnv(env?: Readonly<Record<string, string | undefined>>): DbConnectionOptions;
 export declare const DEFAULT_SERVER_STATEMENT_TIMEOUT_MS = 60000;

package/dist/db/connection.js

@@ -38,18 +38,38 @@ export function createDbConnection({ url, ...options }) {
     if (Object.keys(serverParameters).length > 0)
         pgOptions.connection = serverParameters;
     const client = postgres(url, pgOptions);
-    // Two connections are enough for lock-in-tx plus DDL; the cap keeps parallel rebuilds from exhausting max_connections.
-    unboundedSiblingFactories.set(client, () => postgres(url, {
-        ...pgOptions,
-        max: 4,
-        connection: { ...pgOptions.connection, statement_timeout: 0 },
-    }));
+    // A run that never settles (a caller gave up on it via its own timeout) never reaches the
+    // sibling's own close; close() ends it so shutdown and DROP DATABASE are not blocked by it.
+    const openSiblings = new Set();
+    unboundedSiblingFactories.set(client, () => {
+        // Two connections are enough for lock-in-tx plus DDL; the cap keeps parallel rebuilds from exhausting max_connections.
+        const sibling = postgres(url, {
+            ...pgOptions,
+            max: 4,
+            connection: { ...pgOptions.connection, statement_timeout: 0 },
+        });
+        openSiblings.add(sibling);
+        return {
+            client: sibling,
+            close: async (timeoutSeconds) => {
+                openSiblings.delete(sibling);
+                await sibling.end({ timeout: timeoutSeconds });
+            },
+        };
+    });
     return {
         db: client,
         client,
-        close: () => client.end({
-            timeout: options.closeTimeoutSeconds ?? DEFAULT_DB_CLOSE_TIMEOUT_SECONDS,
-        }),
+        close: async () => {
+            const timeout = options.closeTimeoutSeconds ?? DEFAULT_DB_CLOSE_TIMEOUT_SECONDS;
+            unboundedSiblingFactories.delete(client);
+            const pools = [client, ...openSiblings];
+            openSiblings.clear();
+            const results = await Promise.allSettled(pools.map((pool) => pool.end({ timeout })));
+            const failure = results.find((result) => result.status === "rejected");
+            if (failure !== undefined)
+                throw failure.reason;
+        },
     };
 }
 // DB_PROVIDER=bun selected the removed Bun.SQL driver; ignoring it would boot

package/dist/db/statement-timeout.js

@@ -10,9 +10,9 @@ export async function withoutStatementTimeout(db, run) {
     if (unbounded === undefined)
         return run(db);
     try {
-        return await run(unbounded);
+        return await run(unbounded.client);
     }
     finally {
-        await unbounded.end({ timeout: UNBOUNDED_POOL_CLOSE_TIMEOUT_SECONDS });
+        await unbounded.close(UNBOUNDED_POOL_CLOSE_TIMEOUT_SECONDS);
     }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cosmicdrift/kumiko-framework",
-  "version": "0.383.0",
+  "version": "0.383.1",
   "description": "Framework core — engine, pipeline, API, DB, and every other bit that makes Kumiko go.",
   "license": "BUSL-1.1",
   "author": "Marc Frost <marc@cosmicdriftgamestudio.com>",
@@ -199,7 +199,7 @@
     "./package.json": "./package.json"
   },
   "dependencies": {
-    "@cosmicdrift/kumiko-types": "0.383.0",
+    "@cosmicdrift/kumiko-types": "0.383.1",
     "bullmq": "^5.76.7",
     "bun-types": "^1.3.13",
     "hono": "^4.13.1",
@@ -215,7 +215,7 @@
     "zod": "^4.4.3"
   },
   "devDependencies": {
-    "@cosmicdrift/kumiko-testing": "0.383.0",
+    "@cosmicdrift/kumiko-testing": "0.383.1",
     "bun-types": "^1.3.13",
     "pino-pretty": "^13.1.3"
   },

package/src/changes.json

@@ -1,4 +1,10 @@
 [
+  {
+    "version": "0.383.1",
+    "type": "fix",
+    "title": "close() ends unbounded pools of abandoned long runs",
+    "detail": "withoutStatementTimeout (reindexEntity, PII backfill, projection and multi-stream rebuilds, event-store index builds) runs on its own pool without statement_timeout and closes it when the run settles. A run that never settles, such as a reindex a boot rebuild gave up on via its own timeout, kept that pool connected, so DROP DATABASE in test cleanup failed with \"is being accessed by other users\". The connection handle's close() now ends these pools too. A rebuild that is still running when close() is called is ended after the same close timeout instead of keeping shutdown waiting."
+  },
   {
     "version": "0.383.0",
     "type": "breaking",