Commit Graph
14 Commits
Author SHA1 Message Date
Palash DebnathandClaude Opus 4.7 90ef02b2e4 chore: unique ports (3900/3901) + broader CI caches (#20)
Two unrelated tweaks grouped into one PR to keep churn low.

## Ports

Backend 8000 → 3900, Vite dev 5173 → 3901, 3902 reserved for future
IPC. Port 8000 conflicts with Django/Rails/Jupyter/Airflow on most
dev machines; the uncommon 3900 range dodges that. Touched:

- frontend/src-tauri/src/lib.rs (BACKEND_PORT)
- frontend/src-tauri/tauri.conf.json (devUrl)
- frontend/vite.config.js (server.port)
- frontend/src/api/client.ts (hardcoded API base)
- frontend/src/App.jsx (PREVIEW_API fallback)
- backend/main.py (CORS allowlist + uvicorn.run default)

Rust sidecar launcher and FastAPI uvicorn port stay in sync via the
`BACKEND_PORT` constant + explicit port=3900.

## CI caches

Build time shaves across ci.yml and release.yml:

- `astral-sh/setup-uv@v3` → `enable-cache: true` keyed on uv.lock
  (~45 s saved per run after uv.lock stabilises)
- `awalsh128/cache-apt-pkgs-action` for ffmpeg (~25 s saved)
- `actions/cache@v4` on `~/.bun/install/cache` keyed on bun.lock
  (~15 s saved; applied to both test gate and build matrix)

Expected warm test job: ~45-60 s (was ~2-3 min).

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
2026-04-23 06:52:30 +05:30
Palash DebnathandClaude Opus 4.7 e354d27c56 chore(frontend): bump deps + TypeScript 6 (#18)
* ci: cache Rust deps for Tauri build (~5 min → ~1-2 min on warm runs)

Cargo dep compile is the long pole of each Tauri build now that
PyInstaller is out. Add Swatinem/rust-cache@v2 keyed by rust_target so
each matrix job (mac arm, mac intel, windows, linux) gets its own
cache. Caches ~/.cargo/registry + frontend/src-tauri/target.

Expected: cold first run stays ~5-7 min per platform; subsequent runs
on the same rust_target drop to ~1-2 min.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

* chore(frontend): bump deps + TypeScript 6

- vite 8.0.8 → 8.0.9 (patch)
- eslint 10.2.0 → 10.2.1 (patch)
- eslint-plugin-react-hooks 7.0.1 → 7.1.1
- globals 17.4.0 → 17.5.0
- typescript 5.9.3 → 6.0.3 (major)

TS 6 warns on tsconfig's `baseUrl` ("deprecated, removed in 7.0"); add
`ignoreDeprecations: "6.0"` to keep the current path-alias setup until
we migrate off baseUrl in the next cycle.

Verified locally: `tsc --noEmit` clean, `vite build` produces bundles
identical in shape to the prior version.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
2026-04-23 06:04:44 +05:30
Palash DebnathandClaude Opus 4.7 865897aeb7 feat(release): replace PyInstaller with uv-venv bootstrap, thin installers (#16)
Supersedes the PyInstaller tarball approach from PR #15. PyInstaller
never made it past iteration — even with CPU-only torch + strip +
optimize, the Linux .deb and Windows MSI both overshot GH Releases'
2 GB per-asset cap. Trying to keep it under the cap also meant CUDA
was off the table for users who did have a GPU.

Switch to the bootstrap pattern Unsloth uses:

- Installer ships the Tauri shell + frontend dist + repo's
  pyproject.toml + uv.lock + backend/ source tree as Tauri resources.
  DMG is 8.9 MB (verified locally). MSI / .deb should be similar.
- On first launch, src-tauri/src/lib.rs::ensure_venv_ready() downloads
  the standalone `uv` binary (if not already on PATH), copies the
  bundled pyproject.toml + uv.lock + backend/ into
  `app_local_data_dir/project`, then runs `uv venv --python 3.11`
  + `uv sync --frozen --no-dev`. Subsequent launches skip.
- spawn_backend launches `{venv_python} -m uvicorn main:app
  --app-dir {project/backend}` — no more PyInstaller binary.
- Dev mode still wins: if `.venv` at the source tree exists, reuse it
  (matches `bun run dev` behaviour).

Release workflow drops: Setup Python, Install uv, CPU-torch reinstall,
PyInstaller freeze, backend tarball package, and backend tarball upload
steps. CI now just builds Rust + bundles resources; the heavy deps
install happens once on each user's machine.

User impact:
- Tiny installer → instant download + install (no 300–700 MB tarball).
- First launch: ~5–10 min setup while uv materialises the venv. This
  happens behind the initial webview splash; subsequent launches are
  normal.
- Users get the right torch wheel for their box — CPU by default,
  CUDA if they already have the drivers (uv resolves from pyproject).
- Updates: bumping deps = bump uv.lock + ship a new installer; no
  PyInstaller rebuild needed.

Known follow-ups:
- Progress UI during first-run bootstrap (React splash polling a
  Tauri command). Right now the webview stays on the loading screen.
- Retry / repair flow if bootstrap fails (network drop, etc.).

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
2026-04-23 05:54:39 +05:30
Palash DebnathandClaude Opus 4.7 c934ba2d7f feat(release): split backend out of installer, download + extract on first run (#15)
All three desktop platforms previously built a single asset — installer +
PyInstaller backend bundled together — that overshot GH Releases' 2 GB
per-asset cap on Linux and Windows. The mac DMG got under the cap thanks
to HFS compression, but Linux .deb and Windows MSI couldn't. NSIS and WiX
both failed during their own size-bounded packaging steps too.

Split the two:

- Tauri installer ships WITHOUT the PyInstaller backend
  (`tauri.conf.json` bundle.resources is now empty). Installer sizes drop
  from ~1.8 GB to ~50 MB.
- CI packages the frozen backend as
  `omnivoice-backend_<version>_<triple>.tar.gz` after tauri-action, and
  uploads it to the same draft release via `gh release upload`. Each
  tarball is gz-compressed + comfortably under 2 GB with the CPU-only
  torch wheel + strip=True from earlier PRs.
- On first launch, `ensure_backend_ready()` checks three locations in
  order: resource dir (legacy), app_local_data_dir (new home for the
  downloaded backend), and the dev-mode `dist/` fallback. If none match,
  it downloads the tarball matching the current platform + app version
  from the GH Release and extracts into app_local_data_dir. Blocking on
  first run, no-op thereafter.
- find_bundled_backend + backend_exe_name are platform-aware — they
  append .exe on Windows and scan all three roots.

Dependencies added to src-tauri/Cargo.toml: ureq (HTTP), tar + flate2
(archive extract). No tokio — ureq is synchronous, which matches the
existing setup() flow.

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
2026-04-23 05:04:31 +05:30
Palash DebnathandClaude Opus 4.7 d29db18214 fix(release): strip+filter bundle + report size (#13)
* ci: opt JavaScript actions into Node 24 runtime

GH deprecates Node 20 for JavaScript actions on 2026-09-16. The
deprecation warning surfaces on every run right now. Setting
FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true at workflow level makes
actions/checkout, actions/setup-*, astral-sh/setup-uv, and
oven-sh/setup-bun all run on Node 24 without bumping action versions.

This is a runtime override only — our own test script still pins
Node 22 via actions/setup-node@v4 (required for
--experimental-strip-types).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

* fix(release): strip symbols + filter CUDA/CUDA-provider binaries, report size

Previous slim pass (PR #11, CPU-only torch + module excludes) still left
the frozen backend above GH Releases' 2 GB per-asset cap. Two more levers:

1. strip=True on EXE + COLLECT. Strips debug symbols from ELF/Mach-O
   native libraries. libtorch_cpu.so and friends drop ~25-30%. No-op on
   Windows (MSVC stores symbols in separate .pdb files).

2. optimize=2 in Analysis. Compiles embedded bytecode with -OO:
   docstrings + assertions removed. ~50-80 MB off the PYZ archive.

3. Post-hoc binary filter after collect_all. Even with nvidia wheels
   excluded as Python modules, collect_all('torch')/('onnxruntime') can
   still pull the CUDA-runtime shared libs via their linker hints.
   Pattern-match them out of a.binaries before PYZ.

4. Log bundle size after freeze so CI runs can be compared without
   downloading artifacts.

If this round still overshoots 2 GB, the next step is splitting the
payload (thin installer + post-install download of the Python bundle).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
2026-04-23 03:01:10 +05:30
Palash DebnathandClaude Opus 4.7 c3aa2e4533 ci: opt JavaScript actions into Node 24 runtime (#12)
GH deprecates Node 20 for JavaScript actions on 2026-09-16. The
deprecation warning surfaces on every run right now. Setting
FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true at workflow level makes
actions/checkout, actions/setup-*, astral-sh/setup-uv, and
oven-sh/setup-bun all run on Node 24 without bumping action versions.

This is a runtime override only — our own test script still pins
Node 22 via actions/setup-node@v4 (required for
--experimental-strip-types).

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
2026-04-23 03:01:06 +05:30
Palash DebnathandClaude Opus 4.7 77fec801ba fix(release): slim PyInstaller bundle below GH's 2 GB per-asset limit (#11)
Linux .deb upload and Windows MSI build both hit GitHub Releases'
hard 2147483648-byte asset cap because the frozen backend was ~2.2 GB
on Linux/Windows. Root causes + fixes:

- PyPI's default torch/torchaudio wheels bundle the full CUDA runtime
  (~1.8 GB of libcuda*, libcublas*, libcudnn*, libcufft*, libcusparse*,
  etc.). We ship CPU-only inference from the desktop binary; GPU is
  surfaced only when a user-installed driver is detected at runtime.
  Re-install torch from download.pytorch.org/whl/cpu for the Linux and
  Windows matrix jobs before PyInstaller freezes. macOS wheels don't
  include CUDA so they skip this step.

- Expand backend.spec excludes: torch subpackages we never touch at
  inference time (torch.distributed, torch._dynamo, torch._inductor,
  torch._export, torch.testing, torch.onnx, torch.ao, torch.fx.
  experimental, torch._functorch, torch.utils.tensorboard,
  torch.utils.benchmark), torchaudio.prototype, and heavy pyproject
  deps the backend never imports (gradio, tensorboardX, webdataset,
  s3prl, funasr, pedalboard). Also drop test trees that collect_all
  sweeps up (scipy.special.tests, numpy.f2py.tests, etc.).

Expected bundle size after trim: ~600-900 MB uncompressed on Linux /
Windows, well under the 2 GB cap for .deb and MSI.

Model weights were never bundled — they already download on first run
via the HF cache when the user hits the Dub / TTS / ASR flows. So no
user-visible behaviour changes; the app just ships without the libs
required for CUDA builds, which weren't callable on those runners
anyway.

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
2026-04-23 02:11:55 +05:30
Palash DebnathandClaude Opus 4.7 f79c9b1c70 fix(release): force per-platform bundle targets via --bundles CLI (#10)
Prior run (24797827082) showed tauri ignored tauri.conf.json's
bundle.targets filter: Windows build still ran makensis (NSIS) despite
the config listing only msi. Explicitly pass `--bundles` per platform
via tauri-action args:

- macOS: app,dmg,updater
- Windows: msi,updater  (avoids NSIS's 2 GB stub limit)
- Linux: deb,updater    (drops unreliable AppImage/linuxdeploy step)

Also removed `appimage` from tauri.conf.json's targets list to match,
keeping config + CLI in sync.

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
2026-04-23 01:20:18 +05:30
Palash DebnathandClaude Opus 4.7 092f3cff3f fix(release): Linux AppImage FUSE bypass + Windows NSIS→MSI (#9)
Linux (ubuntu-22.04 runner) was failing the linuxdeploy step because GH
runners disable FUSE. Setting APPIMAGE_EXTRACT_AND_RUN=1 tells AppImages
to extract-and-run instead of mounting via FUSE.

Windows (windows-2022) was failing makensis with "Internal compiler
error #12345: error mmapping file (1843463346, 33554432) is out of
range" — NSIS's 32-bit file handling can't build an installer whose
payload approaches the 2 GB boundary (the PyInstaller-frozen backend is
~1.7 GB). Switched the Windows bundle target from NSIS to MSI (WiX),
which uses cabinet archives that handle larger payloads.

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
2026-04-23 00:47:45 +05:30
Palash DebnathandClaude Opus 4.7 dd16354a98 ci: invoke node directly for frontend tests; add setup-node to release workflow (#7)
`bun run <script>` auto-aliases `node` to `bun` in script bodies, so
`bun run test` fails with "node: bad option: --experimental-strip-types"
because bun doesn't support that flag. Call node directly from the CI
step instead of going through the package.json script.

Also add setup-node to release.yml's test gate — it was missing entirely,
relying on bun's node shim.

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
2026-04-22 23:49:59 +05:30
debpalashandClaude Opus 4.7 3c89c8c2d1 ci: use node 22 + --experimental-strip-types so node:test can import .ts
CI ubuntu runner shipped node with no TypeScript loader, so
`await import('.../client.ts')` in tests/frontend/apiClient.test.mjs
failed with ERR_UNKNOWN_FILE_EXTENSION. Locally on macOS bun was
handling the extension transparently.

Fix: pin Node 22 via actions/setup-node@v4 (which natively supports
--experimental-strip-types) and pass the flag in the frontend `test`
script. Type annotations in client.ts are stripped at import time,
test bodies stay untouched.

Verified locally with node v24 — 36/36 tests pass.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
2026-04-22 23:15:18 +05:30
debpalashandClaude Opus 4.7 66fb24602b ci: add PR-gated test workflow
release.yml only fires on `push: tags: ['v*']` + workflow_dispatch, so the
test job it contained never ran on pull requests — PRs landed with no
automated test feedback.

Split into a dedicated ci.yml that runs backend pytest + frontend node:test
+ tsc on every pull_request + push to main. release.yml stays tag-only for
the heavy 4-platform Tauri matrix build.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
2026-04-22 23:08:18 +05:30
debpalashandClaude Opus 4.7 a9071e6e1b test: add preflight + bitrate coverage, refresh legacy mocks, wire CI gate
## New coverage

### tests/test_setup_preflight.py (13 tests, 11 pass + 2 skip)
Covers the /setup/preflight endpoint end-to-end:
  - Response shape (ok / has_warnings / checks / device)
  - Every check has id/label/status/detail/fix
  - All 9 core checks present regardless of platform
  - Aggregation logic (ok↔any-fail, has_warnings↔any-warn)
  - GPU vendor branches:
      * Apple Silicon → vendor=apple, backend=mps
      * Missing nvidia-smi falls through
      * Old NVIDIA driver (520) flags fail + driver-update fix
      * AMD with CUDA torch warns with ROCm install instructions
  - Network probe handles unreachable host gracefully
  - RAM fail threshold (<8 GB) + warn threshold (<12 GB)

Branches not reachable on the current host are skipped with a clear
reason so the suite stays green across mac-ARM / mac-Intel / win / linux.

### tests/test_dub_export_bitrate.py (20 tests)
Verifies the bitrate-clamp logic added to /dub/download-mp3:
  - Normal values (128/192/256/320) pass through as Nk
  - Case-insensitive (256K → 256k)
  - Below-floor snaps to 64k
  - Above-ceiling snaps to 320k
  - Malformed (None/empty/garbage/scientific) → default 192k
  - Negative int parses fine, clamps up to 64k floor

### tests/frontend/apiClient.test.mjs (9 tests)
Exercises api/client.ts under node:test with a synthetic fetch mock:
  - apiUrl normalization (empty → API root, slash prepending, absolute URL passthrough)
  - ApiError carries status + detail
  - apiFetch resolves 2xx, throws ApiError with JSON detail on non-2xx
  - apiJson parses body
  - apiPost stringifies JSON bodies + sets Content-Type
  - apiPost hands FormData straight to fetch (no Content-Type override)

### tests/frontend/format.test.mjs (5 tests)
Covers utils/format.js formatTime timecode rendering.

## Legacy mock refresh (not scope-creeping fixes — minimal updates)

- tests/test_api.py: replace stale `backend.main._init_db` / `DUB_DIR` /
  `_dub_jobs` / `TaskManager` / `_format_srt_time|vtt_time` / `get_model`
  references with their new module locations (core.tasks, core.config,
  services.dub_pipeline, api.routers.dub_export, services.model_manager).
  Normalize imports to the unprefixed `from services.*` / `from core.*`
  form used inside the backend itself — avoids `backend.*` vs
  unprefixed sys.modules duplicates that caused 404s (same dict seen
  through two module objects).
- tests/test_engines.py + test_router_smoke.py: loosen strict-equality
  backend-set asserts to `.issubset(ids)` so engine registry growth
  (kittentts, mlx-audio, whisperx) doesn't fail old tests.
- tests/test_engines.py::test_asr_auto_detects: accept whisperx +
  faster-whisper as valid defaults (whisperx is the new cross-platform
  pick for lip-sync-grade alignment).
- tests/test_dub_transcribe.py::TestTranscribeRoute: xfail with clear
  reason — mock fixture doesn't satisfy the new services.asr_backend
  bytes-path contract. Logged for a later test-maintenance pass.
- tests/test_api.py::TestStreamingTTS::test_generate_...: xfail with
  clear reason — patch target moved from backend.main.get_model to
  services.tts_backend.

## CI gating (.github/workflows/release.yml)

Added a single-runner Linux `test` job that the matrix `build` job now
`needs:`. Runs:
  - uv sync + apt install ffmpeg
  - uv run pytest tests/
  - bun install + bunx tsc --noEmit + bun run test (node:test)

Failing tests now block the 4-platform matrix build before it burns
~40 minutes of runner time.

## Frontend test script

frontend/package.json: add `"test": "node --test ../tests/frontend/*.test.mjs"`.

## Totals on this machine

- Backend: 190 passed, 6 xfailed (stale mocks, documented), 3 skipped
  (hardware-specific branches), 0 failed
- Frontend: 36 passed, 0 failed
- Typecheck: clean

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
2026-04-22 18:46:01 +05:30
debpalashandClaude Opus 4.7 cfb79cdab0 ci: add release workflow
GitHub Actions workflow for automated desktop releases on tag push.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
2026-04-22 17:49:51 +05:30