refactor(database): replace Error with BlockSuiteError (#7732)

Author: zzj3720

packages/blocks/src/database-block/data-view/common/ast.ts

@@ -1,3 +1,5 @@
+import { BlockSuiteError, ErrorCode } from '@blocksuite/global/exceptions';
+
 import type { TType } from '../logical/typesystem.js';
 import type { UniComponent } from '../utils/uni-component/uni-component.js';
 
@@ -63,7 +65,10 @@ export const getRefType = (vars: Variable[], ref: VariableOrProperty) => {
 export const firstFilterName = (vars: Variable[], ref: VariableOrProperty) => {
   const type = getRefType(vars, ref);
   if (!type) {
-    throw new Error(`can't resolve ref type`);
+    throw new BlockSuiteError(
+      ErrorCode.DatabaseBlockError,
+      `can't resolve ref type`
+    );
   }
   return filterMatcher.match(type)?.name;
 };
@@ -87,7 +92,10 @@ export const firstFilter = (vars: Variable[]): SingleFilter => {
   };
   const filter = firstFilterName(vars, ref);
   if (!filter) {
-    throw new Error(`can't match any filter`);
+    throw new BlockSuiteError(
+      ErrorCode.DatabaseBlockError,
+      `can't match any filter`
+    );
   }
   return {
     type: 'filter',

packages/blocks/src/database-block/data-view/logical/typesystem.ts

@@ -1,3 +1,5 @@
+import { BlockSuiteError, ErrorCode } from '@blocksuite/global/exceptions';
+
 export interface TUnion {
   type: 'union';
   title: 'union';
@@ -131,7 +133,10 @@ export class DataDefine<Data extends DataTypeShape = Record<string, unknown>> {
   isSuperOf(subType: TDataType): boolean {
     const dataDefine = this.dataMap.get(subType.name);
     if (!dataDefine) {
-      throw new Error('bug');
+      throw new BlockSuiteError(
+        ErrorCode.DatabaseBlockError,
+        'data config not found'
+      );
     }
     return dataDefine.isSubOfByName(this.config.name);
   }
@@ -235,7 +240,10 @@ export class Typesystem {
     if (this.isDataType(sub)) {
       const dataDefine = this.dataMap.get(sub.name);
       if (!dataDefine) {
-        throw new Error('bug');
+        throw new BlockSuiteError(
+          ErrorCode.DatabaseBlockError,
+          'data config not found'
+        );
       }
       if (!this.isDataType(superType)) {
         return false;
@@ -265,7 +273,10 @@ export class Typesystem {
         case 'array':
           return tArray(subst(type.ele));
         case 'function':
-          throw new Error('TODO');
+          throw new BlockSuiteError(
+            ErrorCode.DatabaseBlockError,
+            'not implement yet'
+          );
       }
     };
     const result = tFunction({

packages/blocks/src/database-block/data-view/view/data-view.ts

@@ -6,14 +6,18 @@ import type {
 import type { Disposable, Slot } from '@blocksuite/global/utils';
 import type { Doc } from '@blocksuite/store';
 
+import { BlockSuiteError, ErrorCode } from '@blocksuite/global/exceptions';
+
 import type { DataSource } from '../common/data-source/base.js';
 import type { ViewSource } from '../common/index.js';
 import type { DataViewRenderer } from '../data-view.js';
 import type { DataViewSelection, InsertToPosition } from '../types.js';
 import type { UniComponent } from '../utils/uni-component/index.js';
 import type { DataViewWidget } from '../widget/types.js';
-import type { DataViewManagerBase } from './data-view-manager.js';
-import type { DataViewManager } from './data-view-manager.js';
+import type {
+  DataViewManager,
+  DataViewManagerBase,
+} from './data-view-manager.js';
 
 export interface DataViewProps<
   T extends DataViewManager = DataViewManager,
@@ -123,7 +127,10 @@ export class ViewRendererManager {
   getView(type: string): DataViewRendererConfig {
     const view = this.map.get(type);
     if (!view) {
-      throw new Error(`${type} is not exist`);
+      throw new BlockSuiteError(
+        ErrorCode.DatabaseBlockError,
+        `${type} is not exist`
+      );
     }
     return view;
   }

packages/blocks/src/database-block/data-view/view/presets/kanban/controller/selection.ts

@@ -1,5 +1,6 @@
 import type { ReactiveController } from 'lit';
 
+import { BlockSuiteError, ErrorCode } from '@blocksuite/global/exceptions';
 import { assertExists } from '@blocksuite/global/utils';
 
 import type { KanbanGroup } from '../group.js';
@@ -307,8 +308,8 @@ export class KanbanSelectionController implements ReactiveController {
         groupIndex => (groupIndex === 0 ? groups.length - 1 : groupIndex - 1)
       );
     }
-
-    throw new Error(
+    throw new BlockSuiteError(
+      ErrorCode.DatabaseBlockError,
       'Unknown arrow keys, only support: up, down, left, and right keys.'
     );
   }
@@ -393,8 +394,8 @@ export class KanbanSelectionController implements ReactiveController {
         groupIndex => (groupIndex === 0 ? groups.length - 1 : groupIndex - 1)
       );
     }
-
-    throw new Error(
+    throw new BlockSuiteError(
+      ErrorCode.DatabaseBlockError,
       'Unknown arrow keys, only support: up, down, left, and right keys.'
     );
   }

packages/blocks/src/database-block/utils.ts

@@ -1,3 +1,4 @@
+import { BlockSuiteError, ErrorCode } from '@blocksuite/global/exceptions';
 import { nanoid } from '@blocksuite/store';
 
 import type {
@@ -72,7 +73,10 @@ const initMap: Record<
     };
     const column = allowList.sort((a, b) => getWeight(b) - getWeight(a))[0];
     if (!column) {
-      throw new Error('not implement yet');
+      throw new BlockSuiteError(
+        ErrorCode.DatabaseBlockError,
+        'not implement yet'
+      );
     }
     return {
       id,

packages/blocks/src/database-block/view-source.ts

@@ -1,3 +1,4 @@
+import { BlockSuiteError, ErrorCode } from '@blocksuite/global/exceptions';
 import { Slot } from '@blocksuite/global/utils';
 
 import type { SingleViewSource, ViewSource } from './data-view/common/index.js';
@@ -61,7 +62,10 @@ export class DatabaseBlockViewSource implements ViewSource {
       };
       const view = getView();
       if (!view) {
-        throw new Error('view not found');
+        throw new BlockSuiteError(
+          ErrorCode.DatabaseBlockError,
+          'view not found'
+        );
       }
       // eslint-disable-next-line @typescript-eslint/no-this-alias
       const self = this;
@@ -81,7 +85,10 @@ export class DatabaseBlockViewSource implements ViewSource {
         get view() {
           const view = getView();
           if (!view) {
-            throw new Error('view not found');
+            throw new BlockSuiteError(
+              ErrorCode.DatabaseBlockError,
+              'view not found'
+            );
           }
           return view;
         },

packages/framework/global/src/exceptions/code.ts

@@ -17,6 +17,7 @@ export enum ErrorCode {
   SelectionError,
   GfxBlockElementError,
   MissingViewModelError,
+  DatabaseBlockError,
 
   // Fatal error should be greater than 10000
   DefaultFatalError = 10000,