fix(dependencies): vendored better-sqlite3 copies ship x86_64-only prebuilds — breaks ARM64/Termux

[diegosouzapw]

Bug Report

Source: Discussion #6573, reported by @sonnysin805-bit

Environment

• Platform: Termux (Android), aarch64 (ARM64)
• Node.js: v24.17.0
• Install method: pnpm install -g omniroute
• OmniRoute version: 3.8.48 (also reproduced on 3.8.44/3.0.0)

Problem

The app bundles multiple internal copies of better-sqlite3 under different nested paths inside node_modules/omniroute/app/node_modules/better-sqlite3 (observed versions 12.6.2 and 12.8.0, differing from the top-level dependency's 12.11.1). These vendored copies contain only a prebuilt .node binary compiled for EM_X86_64, with no binding.gyp, src/, or build/ scaffolding — so they cannot be rebuilt for EM_AARCH64.

Error

[proxyLogger] Failed to load from DB: dlopen failed: ".../better-sqlite3/build/Release/better_sqlite3.node" is for EM_X86_64 (62) instead of EM_AARCH64 (183)
[Middleware] settings_error: Settings read failed: dlopen failed: ...same...
[AUTH] Login failed: TypeError: Cannot read properties of undefined (reading 'safeParse')

Proposed Fix

The packaging/build process should either:

1. (a) Dedupe to a single better-sqlite3 instance across the app bundle (eliminate vendored copies)
2. (b) Include full source (or platform-appropriate prebuilds for arm64/android) in all vendored copies

Additional Context

• Discussion: Omniroute Internal Server Error #6573 (22-comment mega-thread tracing multiple Termux issues)
• npm install OOMs on Termux; pnpm install works (suggests install graph is heavier than necessary)
• Top-level better-sqlite3@12.11.1 builds correctly for aarch64 — only the vendored copies are broken

[diegosouzapw]

This report is covered by the existing contributor PR #8922, which addresses the Android/Termux packaging path for playwright-core and better-sqlite3, including the ARM64-specific native dependency problem described here. I am leaving the issue open and redirecting implementation and review to that PR: #8922

[diegosouzapw]

This issue is already fixed by PR #8922 (fix: add Termux/Android support for playwright-core and better-sqlite3), which was merged into release/v3.8.50 on 2026-08-06. The postinstall step (scripts/build/postinstall.mjs) now handles ARM64/Termux by (1) copying the correctly-built better-sqlite3.node from the top-level node_modules into the standalone bundle, (2) patching node-gyp's common.gypi to include android_ndk_path so that a source rebuild works on Termux, and (3) patching playwright-core to accept process.platform === "android". The fix was validated on Termux (Android aarch64, Node.js v26.4.0, OmniRoute v3.8.48). Closing as completed.