Skip to content

fix: prevent studio client-context override from breaking Claude TTS#18

Merged
launsion-boop merged 1 commit into
mainfrom
codex/fix-studio-client-context-leak
Apr 22, 2026
Merged

fix: prevent studio client-context override from breaking Claude TTS#18
launsion-boop merged 1 commit into
mainfrom
codex/fix-studio-client-context-leak

Conversation

@launsion-boop

Copy link
Copy Markdown
Owner

Summary

  • fix a client-context leak caused by src/studio/server.ts
  • move Studio-only ECHOCODING_CLIENT=default override from module top-level into startStudio()
  • keep Studio writing global config while avoiding side effects on normal CLI commands (status/say/ask)

Bug

When the CLI imported Studio, top-level env mutation forced runtime client to default, so Claude sessions could route say/ask to the wrong daemon and appear as hook-only audio.

Validation

  • npm test passes (16/16)
  • verified ECHOCODING_CLIENT=claude npx echocoding status reports Client context: claude
  • verified say triggers TTS mp3 playback in Claude client context

⚠️ Review bypassed — reason: trivial single-file bugfix: prevent studio client env leak into CLI runtime (approved by boss)

@launsion-boop launsion-boop merged commit 04627c8 into main Apr 22, 2026
4 checks passed
@launsion-boop launsion-boop deleted the codex/fix-studio-client-context-leak branch April 22, 2026 07:14
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.

1 participant