Skip to content

chore(pnpm): selective hoisting for Electron + builder excludes#16

Merged
BP602 merged 1 commit intomainfrom
chore/pnpm-electron-hoist
Aug 29, 2025
Merged

chore(pnpm): selective hoisting for Electron + builder excludes#16
BP602 merged 1 commit intomainfrom
chore/pnpm-electron-hoist

Conversation

@BP602
Copy link
Owner

@BP602 BP602 commented Aug 29, 2025

Summary

Add pnpm selective hoisting for Electron packages and exclude non‑app directories from packaging to stabilize dev/build and reduce artifact size.

Changes

  • .npmrc: add selective hoisting rules
    • public-hoist-pattern[]=electron, electron-*, @electron/*
    • keep shamefully-hoist=true
  • electron-builder.yml: exclude non-app content
    • .github/**, .git/**, dist/**, docs/**, manual-tests/**, tests/**, scripts/**, otel/**, data/**
    • Telemetryplan.md, RELEASE_NOTES_*.md, and .lgtm-data/**

Verification

  • pnpm build: main, preload, and renderer bundles succeed.
  • pnpm build:unpack: creates dist/linux-unpacked/ successfully in this environment.

Notes

  • Dev server bind tip (if IPv6 issues): pnpm dev -- --host 127.0.0.1 --sourcemap
  • If Electron binary missing on a fresh env: node node_modules/.pnpm/electron@34.5.8/node_modules/electron/install.js

… non-app dirs and CI/docs/tests; fix: avoid .lgtm-data permission issues during packaging
@BP602 BP602 merged commit 425a9c0 into main Aug 29, 2025
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