Skip to content

[codex] BridgeJS: support associated-value enums in import and async paths#764

Merged
kateinoigakukun merged 3 commits into
mainfrom
codex/bridgejs-associated-value-enums
Jun 11, 2026
Merged

[codex] BridgeJS: support associated-value enums in import and async paths#764
kateinoigakukun merged 3 commits into
mainfrom
codex/bridgejs-associated-value-enums

Conversation

@kateinoigakukun

Copy link
Copy Markdown
Member

Summary

  • support associated-value enums in BridgeJS imported signatures instead of rejecting them in the import lowering/link paths
  • allow async exported associated-value enums to use the async resolvable path
  • add snapshot coverage for imported and async associated-value enum generation
  • add runtime round-trip tests for imported/exported associated-value enums, including optional cases

Why

Associated-value enums already had partial support in BridgeJS, but imported signatures were still explicitly rejected and async export resolution still treated them as unsupported. That left the feature inconsistent and blocked valid enum shapes that can be passed through the stack ABI.

Validation

  • swift test --package-path ./Plugins/BridgeJS --filter DiagnosticsTests
  • swift test --package-path ./Plugins/BridgeJS --filter BridgeJSCodegenTests
  • swift test --package-path ./Plugins/BridgeJS --filter BridgeJSLinkTests
  • mise exec -- make unittest

@kateinoigakukun kateinoigakukun marked this pull request as ready for review June 10, 2026 23:30
@kateinoigakukun kateinoigakukun requested a review from krodak June 10, 2026 23:30
@kateinoigakukun kateinoigakukun merged commit 3396b5e into main Jun 11, 2026
13 checks passed
@kateinoigakukun kateinoigakukun deleted the codex/bridgejs-associated-value-enums branch June 11, 2026 07: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