Skip to content

fix(effects): Add serde defaults and aliases for robust message parsing#701

Merged
braedonsaunders merged 1 commit intoclaude/build-openstudio-app-kgia5from
claude/remove-unused-import-KfbWA-U2eAk
Jan 3, 2026
Merged

fix(effects): Add serde defaults and aliases for robust message parsing#701
braedonsaunders merged 1 commit intoclaude/build-openstudio-app-kgia5from
claude/remove-unused-import-KfbWA-U2eAk

Conversation

@braedonsaunders
Copy link
Owner

  • Add #[serde(default)] to all effect struct fields to prevent parse errors when frontend sends partial data
  • Add serde aliases for field name mismatches between TS and Rust:
    • EqBand: alias "type" for "bandType"
    • WahSettings: alias "q" for "resonance"
    • Distortion/Amp/Cabinet/Delay/Reverb: alias "type" for respective type fields
    • Bitcrusher: alias "bits" for "bitDepth"
  • Add Default derive to all effect enum types
  • Use forceSetChannelConfig on startup to ensure mono config is always sent to native bridge (fixes mono tracks being panned on load)
  • Add debug logging for channel config and pan values on startup

- Add #[serde(default)] to all effect struct fields to prevent parse
  errors when frontend sends partial data
- Add serde aliases for field name mismatches between TS and Rust:
  - EqBand: alias "type" for "bandType"
  - WahSettings: alias "q" for "resonance"
  - Distortion/Amp/Cabinet/Delay/Reverb: alias "type" for respective type fields
  - Bitcrusher: alias "bits" for "bitDepth"
- Add Default derive to all effect enum types
- Use forceSetChannelConfig on startup to ensure mono config is always
  sent to native bridge (fixes mono tracks being panned on load)
- Add debug logging for channel config and pan values on startup
@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:57pm

@braedonsaunders braedonsaunders merged commit b9688ac into claude/build-openstudio-app-kgia5 Jan 3, 2026
2 of 5 checks passed
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