Merge pull request #3189 from SeedCompany/tweaks

Author: CarsonF

.github/actions/edgedb-setup/action.yml

@@ -6,12 +6,6 @@ runs:
     - name: Setup EdgeDB
       uses: edgedb/setup-edgedb@v1
 
-    - run: edgedb configure set session_idle_transaction_timeout 5min
-      shell: bash
-
-    - run: edgedb migrate --dev-mode
-      shell: bash
-
     - name: Generate EdgeDB TS files
       shell: bash
       run: yarn edgedb:gen

src/core/edgedb/edgedb.module.ts

@@ -1,6 +1,6 @@
 import { Module, OnModuleDestroy } from '@nestjs/common';
 import { APP_INTERCEPTOR } from '@nestjs/core';
-import { ConnectOptions, createClient, Duration } from 'edgedb';
+import { ConnectOptions, createClient } from 'edgedb';
 import { IdResolver } from '~/common/validators/short-id.validator';
 import type { ConfigService } from '~/core';
 import { splitDb } from '../database/split-db.provider';
@@ -19,15 +19,7 @@ import { TransactionContext } from './transaction.context';
   providers: [
     {
       provide: Options,
-      useValue: Options.defaults().withConfig({
-        // Bump from 1 min, as needed by test suite.
-        // It's probably because we open & do more with in the transaction
-        // than is expected by the library.
-        // I'm not worried about this, and it's possible this can be removed
-        // after migration if app overall is faster without Neo4j.
-        // eslint-disable-next-line @typescript-eslint/naming-convention
-        session_idle_transaction_timeout: Duration.from({ minutes: 5 }),
-      }),
+      useValue: Options.defaults(),
     },
     OptionsContext,
     {