Skip to content

BridgeJS: expand integer and raw-value enum support#701

Merged
krodak merged 1 commit intomainfrom
yt/more-int-width
Mar 13, 2026
Merged

BridgeJS: expand integer and raw-value enum support#701
krodak merged 1 commit intomainfrom
yt/more-int-width

Conversation

@kateinoigakukun
Copy link
Member

@kateinoigakukun kateinoigakukun commented Mar 12, 2026

Summary

  • add BridgeJS support for fixed-width integer types across code generation, link-time JS glue, and runtime intrinsics
  • simplify raw-value enum modeling by reusing BridgeIntegerType instead of maintaining a separate integer-width mapping
  • complete the missing optional and 64-bit raw-value enum paths, including correct bigint handling for JavaScript glue
  • expand snapshot, runtime, optional, and array coverage for fixed-width integers and raw-value enums, then regenerate checked-in BridgeJS outputs

Testing

  • mise exec -- ./Utilities/format.swift
  • mise exec -- swift test --package-path ./Plugins/BridgeJS --filter BridgeJSCodegenTests
  • mise exec -- swift test --package-path ./Plugins/BridgeJS --filter BridgeJSLinkTests
  • mise exec -- ./Utilities/bridge-js-generate.sh
  • mise exec -- make unittest

@kateinoigakukun kateinoigakukun changed the title Align BridgeJS raw-value handling and tests BridgeJS: expand integer and raw-value enum support Mar 12, 2026
@kateinoigakukun kateinoigakukun force-pushed the yt/more-int-width branch 2 times, most recently from 0fb3e28 to 823ee5f Compare March 12, 2026 19:10
Copy link
Member

@krodak krodak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is great 👏🏻 🙌🏻

@krodak krodak merged commit c5ae8c2 into main Mar 13, 2026
12 checks passed
@krodak krodak deleted the yt/more-int-width branch March 13, 2026 14:37
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