fix(desktop): include bindings package for better-sqlite3 native module

[saddlepaddle]

Summary

• Fixes crash on startup in production build caused by missing bindings package
• The better-sqlite3 native module uses bindings to locate its .node file at runtime
• Added bindings and file-uri-to-path to electron-builder configuration
• Updated copy-native-modules.ts to handle these dependencies

Test plan

• Build production release
• Verify app launches without "Cannot find module 'bindings'" error
• Verify SQLite database operations work correctly

🤖 Generated with Claude Code

Summary by CodeRabbit

• Chores
  • Improved native-module packaging and install process for the desktop app, including added support for required native runtimes and build steps to ensure modules are included at release.
• Bug Fixes
  • Resolved issues with native bindings and path handling to reduce runtime errors and improve stability on desktop installations.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Warning

Rate limit exceeded

@saddlepaddle has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 3 minutes and 4 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between 0028a26 and dd4246a.

📒 Files selected for processing (1)

• apps/desktop/electron-builder.ts

Walkthrough

Adds unpack/copy rules and dependencies for native-related modules in the Electron desktop build and refactors the native-module copy script to distinguish required vs optional native modules and to replace workspace symlinks with real files before packaging.

Changes

Cohort / File(s)	Summary
Build configuration apps/desktop/electron-builder.ts	Adds bindings and file-uri-to-path to asarUnpack and files packaging rules, enables npmRebuild: true, and adds comments about Bun workspace symlink handling.
Package dependencies apps/desktop/package.json	Adds runtime dependencies: bindings@^1.5.0, file-uri-to-path@^1.0.0, and node-addon-api@^7.1.0.
Native module copy script apps/desktop/scripts/copy-native-modules.ts	Adds copyModuleIfSymlink() helper, introduces NATIVE_MODULE_DEPS (optional deps) and refactors prepareNativeModules() to copy required and optional native modules with explicit logging and symlink replacement behavior.

Sequence Diagram(s)

(omitted)

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• Adjust build script to work with isolated node modules (bun 1.3) #172: Earlier changes to desktop build tooling to handle Bun workspace symlinked native modules; related groundwork for native-module copying and unpacking.

Pre-merge checks and finishing touches

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.
Description check	❓ Inconclusive	The description covers the main issue, root cause, solution, and test plan. However, it deviates from the required template structure by using a custom Summary/Test Plan format instead of the standard Description/Related Issues/Type of Change/Testing sections.	Restructure the description to follow the template: add Related Issues section, clearly mark Type of Change (Bug fix), and organize Testing as a separate section with detailed verification steps.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly summarizes the main change: including the bindings package for better-sqlite3 native module support in the desktop build, which directly addresses the startup crash issue in production.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[github-actions]

🧹 Preview Cleanup Complete

The following preview resources have been cleaned up:

Service	Status
Database (Neon)	✅

Thank you for your contribution! 🎉

---

Preview resources have been processed for cleanup