QVAC-18995 feat[api]: bump transcription-whispercpp to 0.9.0 and surface backend/GPU stats#2488
Conversation
…/GPU stats Bump @qvac/transcription-whispercpp ^0.8.0 -> ^0.9.0 and expose the new runtimeStats backend fields (backendDevice, backendId, gpuMemTotalMb, gpuMemFreeMb) through transcribeStatsSchema and the addon stats mapping.
10ccced
QVAC E2E —
|
QVAC E2E —
|
simon-iribarren
left a comment
There was a problem hiding this comment.
CI Status
Build/validation are green, but the PR-attached QVAC Tests workflow is still not green: Android run-producer failed after the consumer registered but never bootstrapped, and macOS/Windows desktop jobs were still pending when I checked. The Android failure log shows all 95 tests marked failed after the consumer was declared unresponsive.
API Surface & Tagging
This is correctly tagged [api]: it adds optional public transcription stats fields (backendDevice, backendId, gpuMemTotalMb, gpuMemFreeMb) and the PR body includes a usage example.
Code Review
No blocking code issues found. The addon response shape, server stats mapping, public schema, and unit coverage line up with the stated API change and preserve existing CPU/parakeet behavior.
Recommendation
Codewise LGTM, but holding approval until the PR-attached tests are green or the Android consumer failure is rerun/acknowledged as infrastructure.
|
/review |
Tier-based Approval Status |
🎯 What problem does this PR solve?
@qvac/transcription-whispercpp@^0.8.0, missing the 0.9.0 release.📝 How does it solve it?
@qvac/transcription-whispercpp^0.8.0→^0.9.0.runtimeStatsfields —backendDevice,backendId,gpuMemTotalMb,gpuMemFreeMb— through all three stats layers: the addon read shape (addon-responses.ts), thetranscribeop mapping, and the publictranscribeStatsSchema.-1is the no-accounting sentinel for the GPU memory fields.🧪 How was it tested?
transcription-stats-schema.test.ts: round-trips the GPU fields, asserts they're optional (CPU/parakeet path), accepts the-1sentinel, and rejects non-numeric backend values (4/4 tests, 11/11 asserts).bun run typecheckandeslinton changed files pass.check:deps-vs-sdk) verified — whispercpp is an excluded plugin addon, no sync needed.🔌 API Changes