Skip to content

fix: display "Local" instead of "OpenAI" for Whisper models in AI settings#109

Merged
nchopra merged 1 commit intomainfrom
whisper-local-label
Feb 27, 2026
Merged

fix: display "Local" instead of "OpenAI" for Whisper models in AI settings#109
nchopra merged 1 commit intomainfrom
whisper-local-label

Conversation

@nchopra
Copy link
Contributor

@nchopra nchopra commented Feb 26, 2026

Summary\n- label local Whisper models as "Local" in AI settings\n- add local model icon assets\n\n## Notes\n- Split out from toast-dismiss work so both changes ship independently

Summary by CodeRabbit

  • Chores
    • Updated Whisper model provider information and icons.
    • Updated Amical Cloud model icon.

@coderabbitai
Copy link

coderabbitai bot commented Feb 26, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 0a2d11d and 6409c2d.

⛔ Files ignored due to path filters (3)
  • apps/desktop/public/assets/icon_logo.svg is excluded by !**/*.svg
  • apps/desktop/public/assets/icon_logo@3x.png is excluded by !**/*.png
  • apps/desktop/public/icons/models/local.svg is excluded by !**/*.svg
📒 Files selected for processing (1)
  • apps/desktop/src/constants/models.ts

📝 Walkthrough

Walkthrough

The change updates provider labels and icon paths for AI model entries in the constants configuration, reassigning several Whisper models from OpenAI to Local provider and updating the Amical Cloud icon reference.

Changes

Cohort / File(s) Summary
Model Provider Configuration
apps/desktop/src/constants/models.ts
Updated provider labels and icon paths for Whisper model entries (7 variants) from "OpenAI" to "Local" provider with corresponding icon changes, and updated Amical Cloud icon path from /assets/logo.svg to /assets/icon_logo.svg.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Whispers shift from cloud to home,
Local models learn to roam,
Icons dance in paths anew,
Constants fresh with morning dew!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: updating Whisper model provider labels from 'OpenAI' to 'Local' and updating associated icons in the AI settings.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch whisper-local-label

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@nchopra nchopra merged commit 62e7c1b into main Feb 27, 2026
4 checks passed
@nchopra nchopra deleted the whisper-local-label branch February 27, 2026 02:39
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.

2 participants