Skip to content

Default AI view to transcription tab#2335

Merged
yujonglee merged 2 commits intomainfrom
feature/default-transcription-tab
Dec 17, 2025
Merged

Default AI view to transcription tab#2335
yujonglee merged 2 commits intomainfrom
feature/default-transcription-tab

Conversation

@ComputelessComputer
Copy link
Collaborator

Set the AI view's default active tab to "transcription" instead of "intelligence". This changes the fallback when tab.state.tab is undefined so the UI now opens the transcription panel by default.

Set the AI view's default active tab to "transcription" instead of "intelligence". This changes the fallback when tab.state.tab is undefined so the UI now opens the transcription panel by default.
@netlify
Copy link

netlify bot commented Dec 16, 2025

Deploy Preview for hyprnote ready!

Name Link
🔨 Latest commit 8295468
🔍 Latest deploy log https://app.netlify.com/projects/hyprnote/deploys/6941fce0b8734f0008c706e1
😎 Deploy Preview https://deploy-preview-2335--hyprnote.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link

netlify bot commented Dec 16, 2025

Deploy Preview for hyprnote-storybook ready!

Name Link
🔨 Latest commit 8295468
🔍 Latest deploy log https://app.netlify.com/projects/hyprnote-storybook/deploys/6941fce036d8430008ad697f
😎 Deploy Preview https://deploy-preview-2335--hyprnote-storybook.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 16, 2025

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Change switches the AI tab default from "intelligence" to "transcription" by updating the fallback tab in the desktop UI and the enum default in the Windows plugin, altering which sub-component is shown when no tab state exists.

Changes

Cohort / File(s) Change Summary
Desktop AI tab UI
apps/desktop/src/components/main/body/ai.tsx
Changed fallback/default active AI tab from "intelligence" to "transcription", so the STT/transcription view is shown by default when no tab state is set.
Windows plugin tab state
plugins/windows/src/tab/state.rs
Moved the #[default] attribute on AiTab from Intelligence to Transcription (affects the enum's default variant used by derive(Default)).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Small, localized changes across 2 files.
  • Review focus:
    • Ensure UI fallback change aligns with any telemetry/feature flags.
    • Confirm enum default change doesn't break deserialization, persisted state, or downstream pattern matches.

Possibly related PRs

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: setting the default AI view to show the transcription tab instead of intelligence.
Description check ✅ Passed The description accurately explains the change, specifying what was modified (default tab from 'intelligence' to 'transcription') and when it applies (when tab.state.tab is undefined).

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 389b2a3 and 8295468.

📒 Files selected for processing (1)
  • plugins/windows/src/tab/state.rs (1 hunks)

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.

@yujonglee yujonglee merged commit 0aca8ae into main Dec 17, 2025
12 of 15 checks passed
@yujonglee yujonglee deleted the feature/default-transcription-tab branch December 17, 2025 00:44
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