Skip to content

Claude/remove unused import kfb wa#698

Merged
braedonsaunders merged 2 commits intoclaude/build-openstudio-app-kgia5from
claude/remove-unused-import-KfbWA
Jan 3, 2026
Merged

Claude/remove unused import kfb wa#698
braedonsaunders merged 2 commits intoclaude/build-openstudio-app-kgia5from
claude/remove-unused-import-KfbWA

Conversation

@braedonsaunders
Copy link
Owner

No description provided.

claude added 2 commits January 3, 2026 21:54
…startup

1. Join as Performer button:
   - Now only requires bridge connection, not audio running
   - Audio starts AFTER joining, not before
   - Removed confusing auto-proceed behavior

2. Effects fixes:
   - Added #[serde(default)] to ALL effects in EffectsSettings
   - Fixes deserialization when frontend sends partial effects
   - Added effects chain processing to monitoring path (was DRY, now WET)
   - Added logging for updateEffects to help debug

3. Startup sync:
   - Send initial effects to native bridge on audio start
   - Send pan value with track state on startup
   - Log which effects are enabled for debugging
Empty buffer on reconnect is normal - only warn about actual buffer
overflow (samples being dropped), not empty buffer conditions.
@vercel
Copy link

vercel bot commented Jan 3, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
openstudio Ready Ready Preview, Comment Jan 3, 2026 10:11pm

@braedonsaunders braedonsaunders merged commit bdaf573 into claude/build-openstudio-app-kgia5 Jan 3, 2026
5 checks passed
@braedonsaunders braedonsaunders deleted the claude/remove-unused-import-KfbWA branch January 4, 2026 01:05
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