style: add @typescript-eslint/return-await rule (#9612)

eslint.config.mjs

@@ -159,6 +159,10 @@ export default tseslint.config(
       '@typescript-eslint/no-unsafe-function-type': 'error',
       '@typescript-eslint/no-wrapper-object-types': 'error',
       '@typescript-eslint/unified-signatures': 'error',
+      '@typescript-eslint/return-await': [
+        'error',
+        'error-handling-correctness-only',
+      ],
       '@typescript-eslint/no-restricted-imports': [
         'error',
         {
@@ -269,7 +273,11 @@ export default tseslint.config(
     },
   },
   {
-    files: ['packages/frontend/apps/electron/scripts/**/*'],
+    files: [
+      'packages/frontend/apps/electron/scripts/**/*',
+      'blocksuite/tests-legacy/**/*.{ts,tsx}',
+      'blocksuite/**/__tests__/**/*.{ts,tsx}',
+    ],
     rules: {
       'import-x/no-extraneous-dependencies': 'off',
     },
@@ -280,14 +288,5 @@ export default tseslint.config(
       'rxjs/finnish': 'off',
     },
   },
-  {
-    files: [
-      'blocksuite/tests-legacy/**/*.{ts,tsx}',
-      'blocksuite/**/__tests__/**/*.{ts,tsx}',
-    ],
-    rules: {
-      'import-x/no-extraneous-dependencies': 'off',
-    },
-  },
   eslintConfigPrettier
 );

packages/common/infra/src/sync/indexer/impl/indexeddb/index.ts

@@ -70,7 +70,7 @@ export class IndexedDBIndex<S extends Schema> implements Index<S> {
         return fromPromise(async () => {
           try {
             const trx = await this.data.readonly();
-            return this.data.search(trx, query, options);
+            return await this.data.search(trx, query, options);
           } catch (error) {
             logger.error('search error', error);
             throw error;
@@ -100,7 +100,7 @@ export class IndexedDBIndex<S extends Schema> implements Index<S> {
         return fromPromise(async () => {
           try {
             const trx = await this.data.readonly();
-            return this.data.aggregate(trx, query, field, options);
+            return await this.data.aggregate(trx, query, field, options);
           } catch (error) {
             logger.error('aggregate error', error);
             throw error;

packages/frontend/apps/electron/src/helper/dialog/dialog.ts

@@ -195,7 +195,7 @@ export async function loadDBFile(): Promise<LoadDBFileResult> {
     }
 
     const workspaceId = nanoid(10);
-    return loadV1DBFile(originalPath, workspaceId);
+    return await loadV1DBFile(originalPath, workspaceId);
 
     // TODO(forehalo): use `nbstore` when it is ready
     // let storage = new DocStorage(originalPath);

packages/frontend/core/src/components/blocksuite/block-suite-editor/ai/request.ts

@@ -91,7 +91,7 @@ async function resizeImage(blob: Blob | File): Promise<Blob | null> {
     if (ctx) {
       ctx.imageSmoothingQuality = 'high';
       ctx.drawImage(img, 0, 0, canvas.width, canvas.height);
-      return new Promise(resolve =>
+      return await new Promise(resolve =>
         canvas.toBlob(blob => resolve(blob), 'image/jpeg', 0.8)
       );
     }

packages/frontend/core/src/modules/cloud/services/auth.ts

@@ -183,7 +183,7 @@ export class AuthService extends Service {
       this.session.revalidate();
 
       track.$.$.auth.signedIn({ method: 'oauth', provider });
-      return res.json();
+      return await res.json();
     } catch (e) {
       track.$.$.auth.signInFail({
         method: 'oauth',