Skip to content

fix: fix: voxtream backend should log model download progress to stderr (#33) [wshm]#34

Merged
pszymkowiak merged 1 commit intomainfrom
wshm/fix-33
Mar 18, 2026
Merged

fix: fix: voxtream backend should log model download progress to stderr (#33) [wshm]#34
pszymkowiak merged 1 commit intomainfrom
wshm/fix-33

Conversation

@pszymkowiak
Copy link
Contributor

🧞 wshm · Automated triage by AI

Fixes #33

🤖 Auto-generated fix

This PR was automatically generated by wshm using claude-code.

⚠️ This is a draft PR — a human must review and approve before merging.

Issue

fix: voxtream backend should log model download progress to stderr

When using vox -b voxtream "text" for the first time, the model download (~500MB) happens silently — no progress indication.

The voxtream backend in src/backend/voxtream.rs should print a message to stderr before calling the voxtream CLI:

eprintln!("Loading VoXtream2 model (first run may download ~500MB)...");

Similar to how qwen-native does it in src/backend/qwen_native.rs:35.

Small fix, one line.

Security

  • Author verified as collaborator
  • Generated diff scanned for suspicious patterns
  • Human review required

🤖 Generated automatically by wshm · This is an automated analysis, not a human review.

@pszymkowiak
Copy link
Contributor Author

pszymkowiak commented Mar 18, 2026

🧞 wshm · Automated triage by AI

📊 Automated PR Analysis

🐛 Type bug-fix
🟢 Risk low

Summary

Adds a stderr log message to the voxtream backend to inform users about model download progress on first run. This is a one-line fix adding an eprintln! call before the voxtream CLI invocation, matching the existing pattern used in the qwen-native backend.

Review Checklist

  • Tests present
  • Breaking change
  • Docs updated

Linked issues: #33


🤖 Analyzed automatically by wshm · This is an automated analysis, not a human review.

@pszymkowiak pszymkowiak marked this pull request as ready for review March 18, 2026 11:34
@pszymkowiak pszymkowiak merged commit bcf0e25 into main Mar 18, 2026
0 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug-fix good first issue Good for newcomers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: voxtream backend should log model download progress to stderr

1 participant