Skip to content

CI/Build Fixes#80

Merged
dawiisss merged 9 commits into
devfrom
main
Jun 18, 2026
Merged

CI/Build Fixes#80
dawiisss merged 9 commits into
devfrom
main

Conversation

@dawiisss

Copy link
Copy Markdown
Owner

Description

Fixing CI for build
Fallback to using npm

dawiisss added 9 commits June 18, 2026 21:51
electron-builder needs a recognized lockfile to resolve the dep tree
for packaging. pnpm-lock.yaml isn't detected by electron-builder 26.8.1,
so we keep a dual-lockfile setup:
- pnpm-lock.yaml for pnpm installs and CI
- package-lock.json for electron-builder packaging

Generated with npm install --package-lock-only from the current deps
(dompurify 3.4.11, tldts-experimental 7.4.2, undici 7.28.0).
pnpm's module resolution caused electron-builder to miss transitive
dependencies (e.g. ms) in the asar, breaking the packaged app at runtime.
npm's flat node_modules + package-lock.json work correctly with
electron-builder 26.x.

- Revert CI workflows to npm ci / npm run
- Restore npmRebuild: true for native module rebuilding
- Remove .npmrc (shamefully-hoist / node-linker config)
- Remove pnpm-workspace.yaml
@dawiisss dawiisss merged commit 876ec32 into dev Jun 18, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant