Add support for Java on darwin-x64 - #2701
Conversation
* Initial plan * Add Java darwin-x64 native support Co-authored-by: edburns <75821+edburns@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: edburns <75821+edburns@users.noreply.github.com>
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
Release workflows omit the new classifier, and attempt-scoped test artifacts break partial assembly reruns.
Get a fresh assessment by requesting another Copilot review.
Review effort: Balanced (auto)
Findings: 1
Note
Copilot is running an experiment and ran this review at Balanced.
Open (1)
What changed in this PR
Adds Java in-process runtime packaging and testing for Intel macOS (darwin-x64).
Changes:
- Adds Maven packaging and classifier attachment profiles.
- Extends host validation and publication tests.
- Adds Intel macOS CI coverage and documentation.
| File | Description |
|---|---|
.github/workflows/java-sdk-tests.yml |
Adds Intel macOS testing and publication validation. |
java/sdk/pom.xml |
Selects darwin-x64 on Intel macOS. |
java/copilot-native/pom.xml |
Packages and attaches the native classifier. |
java/copilot-native/scripts/validate-native-host.mjs |
Validates Intel macOS hosts. |
java/copilot-native/scripts/validate-native-host.test.mjs |
Tests host validation. |
java/copilot-native/scripts/validate-native-artifact.test.mjs |
Tests classifier artifacts. |
java/copilot-native/scripts/validate-local-publication.mjs |
Requires the classifier in publications. |
java/copilot-native/scripts/fetch-native.test.mjs |
Covers native fetching. |
java/README.md |
Documents Intel macOS support. |
java/docs/adr/adr-007-native-bundling-strategy.md |
Updates platform and publication scope. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 9f79eaea-d0bb-4e69-9d32-499a9c186788
SDK Consistency ReviewThis PR adds a
No public SDK API surface is touched — nothing under Conclusion: No cross-SDK consistency issues. This is an internal, Java-only build/packaging enhancement (adding a supported platform) and does not require changes in other language SDKs.
|

Fixes #2428 . See the issue for details.