fix: kysely closing db on execute (#155)

Co-authored-by: DominicGBauer <[email protected]>

Author: DominicGBauer

.changeset/metal-cheetahs-try.md

@@ -0,0 +1,5 @@
+---
+"@powersync/kysely-driver": minor
+---
+
+Made `destroy` and `releaseConnection` no-op functions. If you relied on `destroy` you will need to use `disconnectAndClear` on the PowerSync DB directly.

packages/kysely-driver/src/sqlite/sqlite-connection.ts

@@ -105,9 +105,7 @@ export class PowerSyncConnection implements DatabaseConnection {
     this.releaseTransaction();
   }
 
-  async releaseConnection(): Promise<void> {
-    this.#db.close();
-  }
+  async releaseConnection(): Promise<void> {}
 
   private releaseTransaction() {
     if (!this.#completeTransaction) {

packages/kysely-driver/src/sqlite/sqlite-driver.ts

@@ -35,7 +35,8 @@ export class PowerSyncDriver implements Driver {
     await connection.releaseConnection();
   }
 
-  async destroy(): Promise<void> {
-    this.#db.disconnectAndClear();
-  }
+  /**
+    This will do nothing. Instead use PowerSync `disconnectAndClear` function.
+   */
+  async destroy(): Promise<void> {}
 }

packages/kysely-driver/tests/sqlite/db.test.ts

@@ -15,7 +15,7 @@ describe('CRUD operations', () => {
   });
 
   afterEach(async () => {
-    await db.destroy();
+    await powerSyncDb.disconnectAndClear();
   });
 
   it('should insert a user and select that user', async () => {