Skip to content

chore(app): bump whisper_flutter_new for context cache#9439

Draft
andreayres wants to merge 1 commit into
BasedHardware:mainfrom
andreayres:chore/bump-whisper-flutter-cache
Draft

chore(app): bump whisper_flutter_new for context cache#9439
andreayres wants to merge 1 commit into
BasedHardware:mainfrom
andreayres:chore/bump-whisper-flutter-cache

Conversation

@andreayres

@andreayres andreayres commented Jul 11, 2026

Copy link
Copy Markdown

Summary

Bumps whisper_flutter_new from af6b457…00bc626… so the app picks up:

  • process-global native context cache (no full model reload every buffer flush)
  • Whisper.releaseModel() API

Blocked on

Do not merge until BasedHardware/whisper_flutter_new#3 is merged into BasedHardware/whisper_flutter_new and commit 00bc62694782075b7f9982a4c497e7145ba7bde0 exists on that repo.

If #3 is squash-merged, update app/pubspec.yaml + lock ref / resolved-ref to the new main SHA and re-run flutter pub get.

Related

Test plan

Review in cubic

Points whisper_flutter_new at 00bc626 (cache native context between
transcribe calls + Whisper.releaseModel()). Depends on
BasedHardware/whisper_flutter_new#3 landing that SHA on the upstream
repo (re-pin if the PR is squash-merged).

Co-Authored-By: Grok <noreply@x.ai>
@Git-on-my-level Git-on-my-level added dependency-review Touches dependencies or lockfiles; needs dependency review needs-maintainer-review Needs human maintainer review before merge labels Jul 11, 2026
@Git-on-my-level

Copy link
Copy Markdown
Collaborator

Thanks for keeping this small and clearly tied to the upstream whisper_flutter_new cache work.

Maintainer note: this should stay blocked until BasedHardware/whisper_flutter_new#3 is merged (or this app PR is updated to the final merged SHA if that PR is squash-merged), then cd app && flutter pub get should be re-run so pubspec.yaml and pubspec.lock agree on a reachable commit from the BasedHardware dependency repo.

Because this is a draft dependency/lockfile bump with no CI signal yet, I’m marking it for maintainer/dependency review rather than approving. Before merge, please also include the on-device validation from the PR description: repeated short on-device Whisper utterances should avoid full model reloads after the first buffer, and stopping recording should not regress the socket-close behavior when paired with #9438.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependency-review Touches dependencies or lockfiles; needs dependency review needs-maintainer-review Needs human maintainer review before merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants