more ci fixes

Author: kostyafarber

.github/workflows/ci.yml

@@ -14,6 +14,8 @@ jobs:
     steps:
       - name: Checkout repository
         uses: actions/checkout@v4
+        with:
+          submodules: true
 
       - name: Setup pnpm
         uses: pnpm/action-setup@v4
@@ -39,6 +41,8 @@ jobs:
     steps:
       - name: Checkout repository
         uses: actions/checkout@v4
+        with:
+          submodules: true
 
       - name: Setup pnpm
         uses: pnpm/action-setup@v4
@@ -64,6 +68,8 @@ jobs:
     steps:
       - name: Checkout repository
         uses: actions/checkout@v4
+        with:
+          submodules: true
 
       - name: Setup pnpm
         uses: pnpm/action-setup@v4
@@ -80,6 +86,9 @@ jobs:
       - name: Install dependencies
         run: pnpm install --frozen-lockfile
 
+      - name: Build native module
+        run: pnpm build:native
+
       - name: Run integration tests
         run: pnpm test:integration
 
@@ -90,6 +99,8 @@ jobs:
     steps:
       - name: Checkout repository
         uses: actions/checkout@v4
+        with:
+          submodules: true
 
       - name: Setup pnpm
         uses: pnpm/action-setup@v4

.github/workflows/deadcode.yml

@@ -12,6 +12,8 @@ jobs:
     steps:
       - name: Checkout repository
         uses: actions/checkout@v4
+        with:
+          submodules: true
 
       - name: Setup pnpm
         uses: pnpm/action-setup@v4

apps/desktop/package.json

@@ -34,6 +34,7 @@
     "@electron/fuses": "^1.8.0",
     "@tailwindcss/vite": "^4.1.11",
     "@types/electron-squirrel-startup": "^1.0.2",
+    "@types/node": "^25.3.0",
     "@types/throttle-debounce": "^5.0.2",
     "@vitejs/plugin-react": "^4.6.0",
     "electron": "40.1.0",

apps/desktop/src/renderer/src/engine/editing.ts

@@ -173,6 +173,7 @@ export class EditingManager {
     this.#dispatchVoid(this.#engine.raw.removePoints(pointIds));
   }
 
+  /** @knipclassignore used via Pick<EditingManager, CommandEditingMethods> */
   insertPointBefore(beforePointId: PointId, edit: PointEdit): PointId {
     const ids = this.#dispatch(
       this.#engine.raw.insertPointBefore(
@@ -214,6 +215,7 @@ export class EditingManager {
     this.#dispatchVoid(this.#engine.raw.setXAdvance(width));
   }
 
+  /** @knipclassignore used via Pick<EditingManager, CommandEditingMethods> */
   translateLayer(dx: number, dy: number): void {
     this.#dispatchVoid(this.#engine.raw.translateLayer(dx, dy));
   }
@@ -231,10 +233,12 @@ export class EditingManager {
     this.#dispatchVoid(this.#engine.raw.reverseContour(contourId));
   }
 
+  /** @knipclassignore used via Pick<EditingManager, CommandEditingMethods> */
   removeContour(contourId: ContourId): void {
     this.#dispatchVoid(this.#engine.raw.removeContour(contourId));
   }
 
+  /** @knipclassignore used via Pick<EditingManager, CommandEditingMethods> */
   openContour(contourId: ContourId): void {
     this.#dispatchVoid(this.#engine.raw.openContour(contourId));
   }

package.json

@@ -4,6 +4,7 @@
   "private": true,
   "description": "Shift font editor monorepo",
   "scripts": {
+    "prepare": "git submodule update --init --recursive && pnpm generate:glyph-info",
     "dev": "pnpm run build:native:debug && pnpm --filter @shift/desktop dev",
     "dev:watch": "scripts/watch.sh",
     "build": "turbo run build",