Skip to content

fix(bundler): source macOS bundle version from Cargo - #6

Merged
unohee merged 1 commit into
masterfrom
swarm/AUD-829-cargo-bundle-version
Aug 24, 2026
Merged

fix(bundler): source macOS bundle version from Cargo#6
unohee merged 1 commit into
masterfrom
swarm/AUD-829-cargo-bundle-version

Conversation

@unohee

@unohee unohee commented Aug 24, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • resolve the bundled package version through Cargo metadata
  • write the numeric major/minor/patch tuple to macOS VST3, CLAP, AU, and standalone plist metadata
  • preserve the existing AU-only override and public helper signature
  • normalize prerelease/build suffixes because Apple requires numeric plist versions

Verification

  • cargo fmt -p nih_plug_xtask -- --check
  • cargo test -p nih_plug_xtask (7 passed)
  • prerelease/build metadata regression test
  • OpenSwarm working-tree review: Decision: APPROVE

Supports de-artifact AUD-829.

@unohee

unohee commented Aug 24, 2026

Copy link
Copy Markdown
Collaborator Author

🔍 Fresh review of 074dda0 (openswarm pr review --fresh)

✓ Decision: APPROVE
The bundle path resolves the selected workspace package through Cargo metadata and consistently propagates its normalized major.minor.patch version to macOS standalone, VST, CLAP, and AU metadata. Prerelease/build stripping is covered, and the existing public metadata helper retains its prior 1.0.0 behavior.

@unohee

unohee commented Aug 24, 2026

Copy link
Copy Markdown
Collaborator Author

🔍 Fresh review of 074dda0 (openswarm pr review --fresh)

✓ Decision: APPROVE
The bundle path resolves the selected Cargo package through cargo metadata and consistently propagates its normalized numeric version to macOS standalone, CLAP/VST, and AU metadata. AU-specific overrides remain intact and are validated through the existing integer encoding parser.

@unohee
unohee merged commit 9d1f723 into master Aug 24, 2026
8 checks passed
@unohee
unohee deleted the swarm/AUD-829-cargo-bundle-version branch August 24, 2026 18:47
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