Make on-demand Whisper installer reachable on first-run (#36) - #37
Merged
Merged
Conversation
…#36) Picking a non-English language (e.g. German) and transcribing for the first time returned a dead-end error -- the backend sent the needs_whisper_install flag but the initial-transcribe handler ignored it and just toasted the raw error text, leaving the user with no way to install the engine. The on-demand installer only existed behind the Retranscribe modal, which doesn't appear until after a successful transcription. - project.html: the first-run transcribe error now detects needs_whisper_install and renders an 'Install non-English support' button in the transcribe card, then auto-retries once the engine is ready. - Refactor the modal's startWhisperInstall() into a shared runWhisperInstall(container, onDone) used by both the first-run error and the Retranscribe modal -- one install/polling code path. - updateWhisperBanner now assumes Whisper is missing if the engine probe fails, so it shows the (idempotent) install prompt instead of letting a non-English transcription fail later with no recourse. - dashboard.html: hint under the language picker that non-English downloads the Whisper engine (~200MB) on first use. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ❌ Deployment failed View logs |
doza-assist | 042136c | Jun 05 2026, 01:34 PM |
DozaVisuals
added a commit
that referenced
this pull request
Jun 5, 2026
Surface the on-demand Whisper installer from the initial transcribe handler, not just the Retranscribe modal: when /transcribe returns needs_whisper_install, render an "Install non-English support" button that installs Whisper once and auto-retries. Refactor startWhisperInstall -> runWhisperInstall(container, onDone) so both entry points share it. Add a language hint on the dashboard. Frontend-only; backend endpoints already present. Ports OSS doza-assist v3.5.8 (PR #37) to the direct release core. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
Issue #36: a user picks a non-English language (German), transcribes for the first time, and gets a dead-end message saying it "needs Whisper which is not installed" — with no way to install it.
The backend already had a complete on-demand installer (`/api/install-whisper` + polling) and the `/transcribe` route already returns a `needs_whisper_install` flag. But the only UI that surfaced the installer lived in the Retranscribe modal, which doesn't exist until after a successful transcription. The initial transcribe handler ignored the flag and just toasted the raw error text — a dead end for a first-time non-English user.
Fix (frontend only — backend was already correct)
Verification
Closes #36