Skip to content

Mix enhance update#3

Merged
daveronan merged 3 commits into
masterfrom
mix-enhance-update
Apr 21, 2026
Merged

Mix enhance update#3
daveronan merged 3 commits into
masterfrom
mix-enhance-update

Conversation

@daveronan
Copy link
Copy Markdown
Collaborator

@daveronan daveronan commented Apr 21, 2026

Note

Medium Risk
Medium risk because it changes public enums/models and request payload shapes/defaults (e.g., removed fix_drc_issues, new enhancement flags), which can break existing client integrations and alter server behavior if consumers relied on old defaults.

Overview
Bumps the SDK to 1.3.1 and realigns client models/enums with the server: EnhanceMusicalStyle is replaced with the server’s 18 values, AnalysisMusicalStyle drops DANCE and adds 9 mood-based styles, SoundSource renames BACKING_VOCALS_GROUP to BACKING_VOX_GROUP and adds BRASS_GROUP, and LoudnessPreference adds NO_CHANGE (now the default for MixEnhanceRequest).

Updates request payload construction to match server expectations: removes unsupported fixDRCIssues, adds applyDrumEnhancement/applyVocalEnhancement and getProcessedStems, and omits webhookURL entirely unless explicitly provided (also applied to mixing and mastering preview payloads). Adds missing dependency tenacity, updates docs/examples, and expands tests with a new live-API tests/smoke_test.py plus unit test adjustments for retry behavior and payload changes.

Reviewed by Cursor Bugbot for commit d67b824. Bugbot is set up for automated code reviews on this repo. Configure here.

Dave Ronan added 3 commits April 21, 2026 20:01
- Align EnhanceMusicalStyle enum with server (18 correct values)
- Align AnalysisMusicalStyle enum (remove DANCE, add 9 mood styles)
- Rename SoundSource.BACKING_VOCALS_GROUP to BACKING_VOX_GROUP, add BRASS_GROUP
- Add LoudnessPreference.NO_CHANGE and set as default for MixEnhanceRequest
- Add MixEnhanceRequest fields: get_processed_stems, apply_drum_enhancement, apply_vocal_enhancement
- Remove MixEnhanceRequest.fix_drc_issues (not supported by server)
- Fix webhookURL sent as null — now always a string in enhance and mix payloads
- Add tenacity to install_requires (was imported but missing from package metadata)
- Update examples, docstrings, README, and tests to reflect all changes

Made-with: Cursor
Only include webhookURL in API payloads when explicitly provided.
Previously sent as empty string or null, which could cause server errors.

Made-with: Cursor
@daveronan daveronan merged commit 056bd38 into master Apr 21, 2026
1 check 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.

1 participant