Merge pull request #106 from danribes/claude/follow-next-task-prompt-01Prwo5GytFEBAvsrRHz4RBR

Fix build: remove only canvas, keep rollup native deps

Author: danribes

.github/workflows/build-installer.yml

@@ -74,7 +74,16 @@ jobs:
       # -----------------------------------------------------------------------
       - name: Install desktop app dependencies
         working-directory: desktop-app
-        run: npm ci --omit=optional
+        run: npm ci
+
+      - name: Remove canvas (optional dep that fails to build on Windows CI)
+        working-directory: desktop-app
+        run: |
+          if (Test-Path "node_modules/canvas") {
+            Remove-Item -Recurse -Force "node_modules/canvas"
+            Write-Host "Removed canvas package"
+          }
+        shell: pwsh
 
       - name: Build desktop app
         working-directory: desktop-app

desktop-app/.npmrc

@@ -1,4 +1,6 @@
-# Skip optional dependencies that require native compilation
-# This prevents canvas (optional dep of pdfjs-dist) from being installed
-# canvas requires GTK/Cairo which isn't available on Windows CI runners
-omit=optional
+# Note: canvas is an optional dependency of pdfjs-dist that requires GTK/Cairo
+# to compile. It's removed in the CI workflow after npm install to avoid
+# build failures on Windows runners.
+#
+# Do NOT use omit=optional here as it breaks @rollup/rollup-win32-x64-msvc
+# which is needed for Vite to build on Windows.