GeezerVison

Walkthrough

This PR introduces font customization, note styling options, and Groq Cloud speech recognition support. It redesigns the Settings UI panel with cascade buttons for better organization, updates theme/font application mechanisms across the app, adds localization entries for new settings, and integrates Groq API support for cloud-based speech transcription with improved OAuth error handling.

Changes

Sequence Diagram(s)

sequenceDiagram
    participant User
    participant NoteWindow
    participant SpeechRecognizerFactory
    participant GroqWhisperRecognizer
    participant GroqAPI["Groq Cloud API"]

    User->>NoteWindow: Start voice dictation (Groq model)
    NoteWindow->>NoteWindow: Fetch Groq API key from AiManager
    NoteWindow->>SpeechRecognizerFactory: Create(model, groqApiKey)
    SpeechRecognizerFactory->>GroqWhisperRecognizer: new GroqWhisperRecognizer(apiKey, modelId)
    GroqWhisperRecognizer->>GroqWhisperRecognizer: Start recording PCM16 audio
    GroqWhisperRecognizer->>GroqWhisperRecognizer: Buffer into 5-sec chunks
    GroqWhisperRecognizer->>GroqAPI: POST /openai/v1/audio/transcriptions<br/>(WAV chunk + API key)
    GroqAPI-->>GroqWhisperRecognizer: JSON {text: "transcribed..."}
    GroqWhisperRecognizer-->>NoteWindow: Emit final text result
    NoteWindow->>NoteWindow: Insert text into editor
    User->>NoteWindow: Stop recording
    GroqWhisperRecognizer->>GroqWhisperRecognizer: Cleanup recording + cancel loop

sequenceDiagram
    participant User
    participant MainWindow
    participant ActionPanel as ActionPanel.ShowVoiceModelsPanel
    participant AiManager
    participant SettingsUI as Settings UI Panel

    User->>MainWindow: Open voice models settings
    MainWindow->>MainWindow: Ensure AiManager loaded
    MainWindow->>ActionPanel: ShowVoiceModelsPanel(aiManager)
    ActionPanel->>AiManager: HasApiKey("groq")
    AiManager-->>ActionPanel: true/false
    alt Groq key exists
        ActionPanel->>ActionPanel: Add Groq Cloud section<br/>with API key input
        ActionPanel->>SettingsUI: Render Groq model buttons
        User->>SettingsUI: Select Groq model
        ActionPanel->>AiManager: Update selection
        ActionPanel->>User: Hide flyout
    else No Groq key
        ActionPanel->>SettingsUI: Show Groq models disabled
    end

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~75 minutes

Possibly related PRs

• PR #2: Modifies ActionPanel.CreateSettings() with AI callbacks, introduces AiManager integration in MainWindow/VoiceNoteWindow, and changes voice-model panel setup—overlapping feature and architecture.
• PR #3: Updates backdrop/acrylic theming codepath by changing how AppSettings.ApplyToWindow() and ApplyThemeToWindow() handle config sources and acrylic controller state management.

Poem

🐰 Fonts now dance in Geist and Inter cheer,
Notes wear styles—full, titled, or clear!
Groq clouds whisper transcriptions so true,
Settings cascade with buttons anew! 🎨
Settings fly, styles bloom, voices soar! 🚀

• X
• Mastodon
• Reddit
• LinkedIn

• Added custom font support (Geist, Inter, JetBrains Mono, Segoe UI) with runtime switching
• Implemented note color styling with title bar and full coloring modes
• Integrated Groq Cloud Whisper v3/v3 Turbo STT with DPAPI-secured API key storage
• Refactored settings panel with cascading Raycast-style submenus for cleaner UX
• Fixed Firebase OAuth flow by adding Google client secret support
• Fixed theme/backdrop synchronization across all window types

Refactor settings with cascading submenus

ActionPanel.cs

Add font and note style persistence

AppSettings.cs

Apply font resource on app launch

App.xaml.cs

Description

Inter/JetBrains Mono are referenced as runtime font assets, but the project only copies *.otf fonts
to the output directory, so those fonts may be missing at runtime. Selecting “Inter” or “JetBrains
Mono” can silently fall back or fail to load the font resources.

Code

NoteUI.csproj[R25-27]

+    <Content Include="Assets\Fonts\*.otf">
+      <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
+    </Content>

Evidence

The app references multiple .ttf font assets in XAML and in GetFontFamily(), but the .csproj change
only copies .otf files, leaving .ttf fonts not guaranteed to be present in the output/publish
directory.

NoteUI.csproj[18-28]
App.xaml[12-21]
AppSettings.cs[82-91]

Agent prompt

The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
Inter and JetBrains Mono are referenced as Assets/Fonts/*.ttf, but the project only copies Assets/Fonts/*.otf to the output directory. This can make Inter/JetBrains unavailable at runtime.

## Issue Context
- App.xaml defines FontFamily resources for Inter/JetBrains using .ttf paths.
- AppSettings.GetFontFamily("inter"/"jetbrains") also returns .ttf-based FontFamily values.

## Fix Focus Areas
- NoteUI.csproj[18-28]

## Suggested fix
- Add an additional Content include for `Assets\Fonts\*.ttf` (or a single `Assets\Fonts\*.*` include) with `CopyToOutputDirectory=PreserveNewest`.
- Ensure publish output includes these files as well (same mechanism as the .otf entry).

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

Description

ApplyNoteColor() permanently rewrites the RichEditBox document character formatting to black in
"full" mode, but when leaving "full" it only clears the control Foreground property, leaving the
document’s RTF content black. This can make notes unreadable in dark theme after toggling note
style/color mode.

Code

NoteWindow.xaml.cs[R362-374]

+        // Editor text color: force black in full mode (pastel backgrounds)
+        if (isFull)
+        {
+            NoteEditor.Foreground = new SolidColorBrush(Microsoft.UI.Colors.Black);
+            TaskNoteEditor.Foreground = new SolidColorBrush(Microsoft.UI.Colors.Black);
+            SetEditorTextColor(NoteEditor, Microsoft.UI.Colors.Black);
+            SetEditorTextColor(TaskNoteEditor, Microsoft.UI.Colors.Black);
+        }
+        else
+        {
+            NoteEditor.ClearValue(RichEditBox.ForegroundProperty);
+            TaskNoteEditor.ClearValue(RichEditBox.ForegroundProperty);
+        }

Evidence

In full mode the code calls SetEditorTextColor() which sets range.CharacterFormat.ForegroundColor
(document-level formatting). When not full, it only clears RichEditBox.ForegroundProperty and does
not restore document formatting; the app also autosaves the document as RTF on TextChanged, so the
black formatting can persist into stored note content.

NoteWindow.xaml.cs[302-395]
NoteWindow.xaml.cs[417-423]
NoteWindow.xaml.cs[1019-1023]

Agent prompt

The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
`ApplyNoteColor()` sets document-wide character formatting to black in full-color mode via `SetEditorTextColor()`. When switching back to non-full mode, it only clears `RichEditBox.ForegroundProperty`, leaving the document's RTF character colors black.

## Issue Context
- Notes are persisted as RTF (`TextGetOptions.FormatRtf`) and saved on `NoteEditor_TextChanged`, so document formatting changes can become durable.

## Fix Focus Areas
- NoteWindow.xaml.cs[302-395]
- NoteWindow.xaml.cs[417-423]
- NoteWindow.xaml.cs[1019-1023]

## Suggested fix options
1) **Preferred (non-destructive):** Stop modifying document character formatting.
  - Remove/avoid `SetEditorTextColor(...)` calls.
  - Rely on `RichEditBox.Foreground` (and/or default character format for new text) to control readability without rewriting existing RTF.

2) **If you must rewrite the document:** Explicitly restore a theme-appropriate document color when leaving full mode.
  - In the `else` branch (non-full), compute the theme text color (e.g., from `TextFillColorPrimaryBrush`) and call `SetEditorTextColor(editor, restoredColor)`.
  - Consider temporarily suppressing autosave while applying formatting (so a style toggle doesn’t unintentionally persist formatting changes).

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

Description

In the new storage submenu, clicking “Local” does not disconnect WebDAV when WebDAV is currently
connected. This leaves the app still configured for WebDAV even though the UI action implies
switching back to local storage.

Code

ActionPanel.cs[R400-405]

+        bool isLocal = !isFirebaseConnected && !isWebDavConnected && !isCustomFolder;
+        panel.Children.Add(CreateCheckItem(Lang.T("local"), isLocal, () =>
+        {
+            if (isCustomFolder) onResetFolder(); else if (isFirebaseConnected) onDisconnectFirebase();
+            onDone();
+        }));

Evidence

CreateStorageSubMenu() receives isWebDavConnected and an onDisconnectWebDav action, but the
“Local” handler only resets a custom folder or disconnects Firebase; it never calls
onDisconnectWebDav when WebDAV is connected.

ActionPanel.cs[384-406]
ActionPanel.cs[426-435]

Agent prompt

The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
Selecting “Local” in the storage submenu does not disconnect WebDAV when WebDAV is connected, so the app may remain in WebDAV mode.

## Issue Context
`CreateStorageSubMenu()` already receives both `isWebDavConnected` and `onDisconnectWebDav`, but the Local handler doesn’t use them.

## Fix Focus Areas
- ActionPanel.cs[384-406]

## Suggested fix
- Update the Local handler to also handle WebDAV:
 - If `isWebDavConnected`, call `onDisconnectWebDav()` before `onDone()`.
- (Optional) Consider making “Local”, “Cloud (Firebase)”, “WebDAV”, and “Custom folder” mutually exclusive transitions (disconnect the other modes when switching).

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

Description

GroqWhisperRecognizer ignores HTTP status codes and swallows JSON parse errors by returning an empty
string, so transcription failures can look like “no speech” with no error surfaced. This makes cloud
STT issues hard to diagnose and can break dictation silently.

Code

SpeechRecognitionService.cs[R625-635]

+        var response = await http.PostAsync(
+            "https://api.groq.com/openai/v1/audio/transcriptions", content, ct).ConfigureAwait(false);
+        var json = await response.Content.ReadAsStringAsync(ct).ConfigureAwait(false);
+
+        try
+        {
+            using var doc = JsonDocument.Parse(json);
+            return doc.RootElement.GetProperty("text").GetString() ?? "";
+        }
+        catch { return ""; }
+    }

Evidence

SendToGroqAsync() reads the response body but never checks response.IsSuccessStatusCode; it then
attempts to parse JSON and returns "" on any exception. Callers only emit a final result when text
is non-empty, so errors are silently dropped.

SpeechRecognitionService.cs[592-635]

Agent prompt

The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
Groq cloud transcription failures are silently ignored: non-2xx responses aren’t checked, and JSON parse errors return an empty string.

## Issue Context
The UI only appends text on non-empty final results, so returning "" effectively hides errors.

## Fix Focus Areas
- SpeechRecognitionService.cs[592-635]

## Suggested fix
- Check `response.IsSuccessStatusCode` and, on failure, parse and propagate an error message (e.g., throw an exception and catch it in `ProcessLoopAsync`, or invoke a dedicated error event).
- Include a short error string in the UI (or log) so users understand why transcription stopped working.
- Consider adding a small backoff/retry for transient failures.

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools