Skip to content

Update whisper dependency for Windows setup fix#7786

Merged
kodjima33 merged 1 commit into
BasedHardware:mainfrom
tianmind-studio:codex/windows-whisper-pubspec-ref
Jun 11, 2026
Merged

Update whisper dependency for Windows setup fix#7786
kodjima33 merged 1 commit into
BasedHardware:mainfrom
tianmind-studio:codex/windows-whisper-pubspec-ref

Conversation

@tianmind-studio

Copy link
Copy Markdown
Contributor

Summary

Fixes #4806

Windows context

  • The current dependency pin points at 684fb43710dd4de5538ac2da7d0b1fa405363212 on whisper_flutter_new/main.
  • The new pin is the whisper_flutter_new fix commit titled Fixed the Comma error which was causing the setup to fail in OMI for Windows user.
  • A normal filtered clone of BasedHardware/whisper_flutter_new can resolve the pinned SHA with git cat-file -e af6b457cca4cdd6c97cc2dfa8557333ee09c81f1^{commit}.

Testing

  • git diff --check
  • Verified app/pubspec.yaml and app/pubspec.lock both reference af6b457cca4cdd6c97cc2dfa8557333ee09c81f1.
  • git ls-remote https://github.com/BasedHardware/whisper_flutter_new.git shows the current main SHA and the target fix SHA.

Not run: flutter pub get or a Flutter app build, because Flutter/Dart is not installed in this Windows Codex environment.

@greptile-apps

greptile-apps Bot commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR bumps the whisper_flutter_new git dependency from commit 684fb437 to af6b457c — an upstream fix titled "Fixed the Comma error which was causing the setup to fail in OMI for Windows user" — targeting the Windows build failure reported in #4806. Both pubspec.yaml and pubspec.lock are updated consistently, and no other dependencies are touched.

  • app/pubspec.yaml: ref updated to af6b457cca4cdd6c97cc2dfa8557333ee09c81f1.
  • app/pubspec.lock: ref and resolved-ref aligned to the same SHA, keeping the lock file in sync.

Confidence Score: 5/5

The only change is a targeted dependency SHA bump in two files; both are updated consistently and point to an upstream bug-fix commit.

Both pubspec.yaml and pubspec.lock reference the same new SHA with no drift between them. The upstream commit is described as a narrow comma-syntax fix for Windows, not a feature or breaking change. The sole caveat noted by the author — that flutter pub get and a full build were not run locally — is a minor gap, but the change is minimal enough that the risk of a hidden regression is low.

No files require special attention; both changed files are straightforward and consistent.

Important Files Changed

Filename Overview
app/pubspec.yaml Updates whisper_flutter_new git ref from 684fb437 to af6b457c to fix Windows setup failure; no other dependencies changed.
app/pubspec.lock Aligns ref and resolved-ref for whisper_flutter_new with the new SHA in pubspec.yaml; all other entries unchanged.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[app/pubspec.yaml] -->|ref: af6b457c| B[BasedHardware/whisper_flutter_new.git]
    C[app/pubspec.lock] -->|ref + resolved-ref: af6b457c| B
    B -->|commit: af6b457c| D["Fixed Comma error\n(Windows setup fix)"]
    E[Old pin: 684fb437] -.->|replaced by| D
Loading

Reviews (1): Last reviewed commit: "Fix Windows whisper dependency pubspec p..." | Re-trigger Greptile

@kodjima33 kodjima33 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mobile bug fix: bump whisper_flutter_new pin to BasedHardware-owned commit fixing Windows setup (#4806); conf 4/5

@kodjima33 kodjima33 merged commit e5ce41b into BasedHardware:main Jun 11, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Windows Setup Failed: Encoding error in whisper_flutter_new dependency

2 participants