feat: Add auto-copy response to clipboard toggle

[danpeg]

Summary

• Adds a "Copy responses" toggle in the menu bar panel
• When enabled, Clicky's response text is automatically copied to the macOS clipboard after each response
• Uses the clean spoken text (no [POINT:...] tags)
• Defaults to OFF, persists via UserDefaults

Changes

• CompanionManager.swift — new isAutoCopyResponseEnabled property, setter, and clipboard copy logic
• CompanionPanelView.swift — new toggle row in the panel, following the existing "Show Clicky" toggle pattern

Test plan

• Build and run in Xcode (Cmd+R)
• Complete onboarding so the toggle appears
• Toggle "Copy responses" ON
• Ask Clicky a question, then Cmd+V to verify the response was copied
• Toggle OFF, ask another question, verify clipboard was NOT overwritten
• Quit and relaunch — verify toggle state persists

🤖 Generated with Claude Code

[Qodo-Free-For-OSS]

Hi, When auto-copy is enabled, the code clears and overwrites the general pasteboard even if spokenText is empty (e.g., a response containing only [POINT:none]), unexpectedly wiping the user’s clipboard.

Severity: action required | Category: correctness

How to fix: Guard empty clipboard writes

Agent prompt to fix - you can give this to your LLM of choice:

Issue description

Auto-copy currently clears and writes the clipboard even when the "clean" spokenText is empty. This can wipe a user’s clipboard (data loss) if Claude returns only a [POINT:none] tag or otherwise yields an empty spokenText after parsing.

Issue Context

The code already guards TTS playback on spokenText emptiness; clipboard write should follow the same rule (and ideally avoid clearContents() unless you’re actually going to write).

Fix Focus Areas

• leanring-buddy/CompanionManager.swift[693-697]
• leanring-buddy/CompanionManager.swift[717-720]
• leanring-buddy/CompanionManager.swift[809-812]

Implementation notes

• Add let trimmed = spokenText.trimmingCharacters(in: .whitespacesAndNewlines) and only copy if !trimmed.isEmpty.
• Avoid calling clearContents() when not copying.

We noticed a couple of other issues in this PR as well - happy to share if helpful.

---

Found by Qodo code review