Skip to content

fix(dev): rebuild audify for Electron on npm install - #617

Merged
CaYatur merged 1 commit into
mainfrom
fix/postinstall-rebuild-audify
Sep 26, 2026
Merged

CaYatur merged 1 commit into
mainfrom
fix/postinstall-rebuild-audify

Conversation

@CaYatur

@CaYatur CaYatur commented Sep 26, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • After a ZIP/npm install source checkout, audify.node was often missing → audio helper non-JSON stdout → INVALID_HELPER_OUTPUT.
  • Add @electron/rebuild, postinstall / npm run rebuild:audio (scripts/rebuild-audify.js), and README notes (TR/EN).
  • Soft-fail on rebuild errors so install still completes; point to rebuild:audio / check:runtime.

Test plan

Linux box verification (2026-09-26)

  • node --check scripts/rebuild-audify.js (exit 0)
  • npm run rebuild:audio / node scripts/rebuild-audify.js produces node_modules/audify/build/Release/audify.node (exit 0)
  • Fresh tree (/tmp/sv-fresh-test): npm install then npm run check:runtime exit 0 — native module loads; device list empty OK on this box
  • Short xvfb-run npm start (~45s): log shows [loopback] NO-DEVICE, no INVALID_HELPER_OUTPUT banner/crash (killed after timeout; no interactive click-through)

Source installs often left audify without a native binary, so
loopback-helper returned non-JSON stdout and Admin showed
INVALID_HELPER_OUTPUT. Add @electron/rebuild, postinstall +
rebuild:audio script, and short README notes.
@CaYatur
CaYatur merged commit e6bdb76 into main Sep 26, 2026
@CaYatur

CaYatur commented Sep 26, 2026

Copy link
Copy Markdown
Owner Author

Linux box verification — 2026-09-26 (Europe/Istanbul)

Commands / results:

Step Exit
node --check scripts/rebuild-audify.js 0
npm run rebuild:audio 0 → audify.node present
Fresh tree /tmp/sv-fresh-test: npm install + npm run check:runtime 0 (0 devices — OK on this box; native module loads)
timeout 45 xvfb-run -a npm start timeout kill (124); log: [loopback] NO-DEVICE; no INVALID_HELPER_OUTPUT

Full npm run smoke not used as PASS gate here (no audio hardware → frame-count fail expected). No interactive GUI click-through.

@CaYatur
CaYatur deleted the fix/postinstall-rebuild-audify branch September 30, 2026 19:03
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