fix(types): resolve all TypeScript errors and add CI type checking

- Add missing property declarations to ClipboardManager, SelectionManager, StyleManager
- Add property declarations to NoneBordersStrategy class
- Fix FormulaMap type to match Formula interface
- Fix test files: DrawBox usage, line method call, type assertions
- Add type-check script to package.json
- Add Type Check step to CI workflow

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: sealday

.github/workflows/ci.yml

@@ -25,6 +25,9 @@ jobs:
       - name: Install dependencies
         run: pnpm install
 
+      - name: Type Check
+        run: pnpm type-check
+
       - name: Build
         run: pnpm build
 

package.json

@@ -7,6 +7,7 @@
     "build": "pnpm -r build",
     "dev": "pnpm -C packages/demo dev",
     "test": "pnpm -C packages/sheet test",
+    "type-check": "tsc --noEmit",
     "format": "biome check --write .",
     "check": "biome check .",
     "doc:build": "rspress build",

packages/sheet/src/sheet/core/cell.ts

@@ -225,9 +225,9 @@ const evalSuffixExpr = (
       stack.push(ret);
     } else if (Array.isArray(expr)) {
       const [formula, len] = expr;
-      const params = [];
+      const params: (string | number | boolean)[] = [];
       for (let j = 0; j < len; j += 1) {
-        params.push(stack.pop());
+        params.push(stack.pop() as string | number | boolean);
       }
       stack.push(formulaMap[formula].render(params.reverse()));
     } else {

packages/sheet/src/sheet/core/managers/ClipboardManager.ts

@@ -18,6 +18,7 @@ import type Selector from '../selector';
 export class ClipboardManager {
   private clipboard: Clipboard;
   private rows: Rows;
+  private merges: Merges;
   private selector: Selector;
   private getCell: (ri: number, ci: number) => Cell | null;
   private setCellText: (

packages/sheet/src/sheet/core/managers/SelectionManager.ts

@@ -19,7 +19,9 @@ import type Selector from '../selector';
 export class SelectionManager {
   private rows: Rows;
   private cols: Cols;
+  private merges: Merges;
   private selector: Selector;
+  private scroll: Scroll;
   private exceptRowSet: Set<number>;
   private unsortedRowMap: Map<number, number>;
   private freezeTotalWidth: () => number;

packages/sheet/src/sheet/core/managers/StyleManager.ts

@@ -16,8 +16,10 @@ import type Selector from '../selector';
 export class StyleManager {
   private styles: CellStyle[];
   private rows: Rows;
+  private merges: Merges;
   private selector: Selector;
   private defaultStyleGetter: () => CellStyle;
+  private onStyleChange: () => void;
 
   constructor(
     styles: CellStyle[],

packages/sheet/src/sheet/core/strategies/BorderStyleStrategy.ts

@@ -271,6 +271,9 @@ export class VerticalBordersStrategy implements BorderStyleStrategy {
  * Remove all borders
  */
 export class NoneBordersStrategy implements BorderStyleStrategy {
+  private styles: unknown[];
+  private addStyle: (style: unknown) => number;
+
   constructor(styles: unknown[], addStyle: (style: unknown) => number) {
     this.styles = styles;
     this.addStyle = addStyle;

packages/sheet/src/sheet/types.ts

@@ -287,7 +287,9 @@ export type EventListener<T = unknown> = (event: T) => void;
  */
 export interface FormulaMap {
   [key: string]: {
-    render: (params: unknown[]) => unknown;
+    render: (
+      params: (string | number | boolean)[],
+    ) => string | number | boolean;
   };
 }
 

packages/sheet/tests/canvas/draw.test.ts

@@ -74,7 +74,7 @@ describe('Canvas Draw Utils', () => {
 
     it('应该能够绘制矩形', () => {
       const draw = new Draw(canvas, 100, 100);
-      draw.rect({ x: 0, y: 0, width: 100, height: 50 }, () => {});
+      draw.rect(new DrawBox(0, 0, 100, 50), () => {});
 
       // If no exception is thrown, the test passes
       expect(draw).toBeInstanceOf(Draw);
@@ -99,10 +99,7 @@ describe('Canvas Draw Utils', () => {
     it('应该能够绘制线条', () => {
       const draw = new Draw(canvas, 100, 100);
 
-      draw.line([
-        [0, 0],
-        [100, 100],
-      ]);
+      draw.line([0, 0], [100, 100]);
 
       // If no exception is thrown, the test passes
       expect(draw).toBeInstanceOf(Draw);

packages/sheet/tests/core/helper.test.ts

@@ -22,7 +22,10 @@ describe('helper', () => {
     });
     it("should return { a: { a1: 'a2' }, b: 'b' } where the value is {a: {a1: 'a1'}, b: 'b'}, {a: {a1: 'b'}}", () => {
       const obj = { a: { a1: 'a1' }, b: 'b' };
-      const merge = helper.merge(obj, { a: { a1: 'a2' } });
+      const merge = helper.merge(obj, { a: { a1: 'a2' } }) as {
+        a: { a1: string };
+        b: string;
+      };
       assert.equal(obj.a.a1, 'a1');
       assert.equal(merge.a.a1, 'a2');
       assert.equal(merge.b, 'b');