Skip to content

fix(graalvm): include community innovation releases - #584

Merged
roele merged 2 commits into
jdx:mainfrom
thomaswue:fix/graalvm-community-innovation
Oct 2, 2026
Merged

roele merged 2 commits into
jdx:mainfrom
thomaswue:fix/graalvm-community-innovation

Conversation

@thomaswue

@thomaswue thomaswue commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

Accept Community archive names with innovation prefixes and additional Java version components. Match Oracle GraalVM by tagging these releases with the innovation feature and keeping version and java_version at the underlying JDK version. For example, 25i4-25.0.4.1.1 becomes graalvm-community-innovation-25.0.4.1.1 in mise.

Filename parsing tests cover releases 25.1 through 25.4 across Linux, macOS, and Windows. Offline mapped-record tests verify that innovation archives receive the feature tag and standard Community archives do not, while preserving the underlying JDK versions, download URLs, and checksum metadata.

Validation: all four focused GraalVM tests, formatting, and Clippy with warnings denied passed. A live GitHub check also verified all 16 innovation platform records, checksum downloads, and archive URL availability; full JDK installation was not tested.

AI-assisted — Tool: Codex; model: OpenAI/gpt-6.1-sol; version: 0.160.0.

Accept Community archive names with innovation prefixes and additional Java
version components. Match Oracle GraalVM by tagging these releases with the
innovation feature and keeping version and java_version at the underlying
JDK version. For example, 25i4-25.0.4.1.1 becomes
graalvm-community-innovation-25.0.4.1.1 in mise.

Extend the existing filename parsing cases to cover releases 25.1 through
25.4 across Linux, macOS, and Windows.

*AI-assisted — Tool: Codex; model: OpenAI/gpt-6.1-sol; version: 0.160.0.*
@coderabbitai

coderabbitai Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Central YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Advanced

Run ID: 750c3250-d5c7-4c65-b733-1d39bad2eee0

  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@greptile-apps

greptile-apps Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

[Medium risk] Updates GraalVM community release parsing and metadata.

The PR appears safe to merge based on the reviewed changes.

Summary

The PR adds parsing and metadata mapping for GraalVM Community innovation archives, including the innovation feature and underlying JDK version. The latest changes extract metadata mapping for direct tests and preserve checksum URL construction.

Reviews (2) · Last reviewed commit: "test(graalvm): cover community innovatio..."

Comment thread src/jvm/vendor/graalvm.rs
Separate checksum downloading from Community metadata construction so the
mapped records can be tested without network access. Assert the innovation
feature and underlying JDK versions for an innovation archive, and verify
that a standard Community archive has no innovation feature. Also check
that download URLs and checksum metadata are preserved.

*AI-assisted — Tool: Codex; model: OpenAI/gpt-6.1-sol; version: 0.160.0.*
@roele

roele commented Oct 2, 2026

Copy link
Copy Markdown
Collaborator

Thanks a lot for your contribution @thomaswue, much appreciated!

@roele
roele merged commit d97a1cf into jdx:main Oct 2, 2026
4 checks passed
@thomaswue

Copy link
Copy Markdown
Contributor Author

Thank you for the very quick reaction!

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