refactor transactions:

- fix the typings
- add rollback and commit to the callback params

packages/core/database/lib/index.d.ts

@@ -164,9 +164,13 @@ export interface Database {
 
   query<T extends keyof AllTypes>(uid: T): QueryFromContentType<T>;
   transaction(
-    cb?: (trx: Knex.Transaction) => Promise<any>
+    cb?: (params: {
+      trx: Knex.Transaction;
+      rollback: () => Promise<void>;
+      commit: () => Promise<void>;
+    }) => Promise<void>
   ):
-    | Promise<void>
+    | Promise<unknown>
     | { get: () => Knex.Transaction; rollback: () => Promise<void>; commit: () => Promise<void> };
 }
 export class Database implements Database {

packages/core/database/lib/index.js

@@ -54,35 +54,35 @@ class Database {
   async transaction(cb) {
     const notNestedTransaction = !transactionCtx.get();
     const trx = notNestedTransaction ? await this.connection.transaction() : transactionCtx.get();
+
+    async function commit() {
+      if (notNestedTransaction) {
+        await trx.commit();
+      }
+    }
+
+    async function rollback() {
+      if (notNestedTransaction) {
+        await trx.rollback();
+      }
+    }
     if (!cb) {
       return {
-        async commit() {
+        commit,
-          if (notNestedTransaction) {
+        rollback,
-            await trx.commit();
-          }
-        },
-        async rollback() {
-          if (notNestedTransaction) {
-            await trx.rollback();
-          }
-        },
         get() {
           return trx;
         },
       };
     }
 
-    return transactionCtx.run(trx, async () => {
+    return transactionCtx.run({ trx, commit, rollback }, async () => {
       try {
         const res = await cb(trx);
-        if (notNestedTransaction) {
+        await commit();
-          await trx.commit();
-        }
         return res;
       } catch (error) {
-        if (notNestedTransaction) {
+        await rollback();
-          await trx.rollback();
-        }
         throw error;
       }
     });