Skip to content

liberica*: Fix checkver JSONPath#590

Open
B67687 wants to merge 1 commit intoScoopInstaller:masterfrom
B67687:liberica-checkver-jsonpath
Open

liberica*: Fix checkver JSONPath#590
B67687 wants to merge 1 commit intoScoopInstaller:masterfrom
B67687:liberica-checkver-jsonpath

Conversation

@B67687
Copy link
Copy Markdown
Contributor

@B67687 B67687 commented Apr 23, 2026

Closes #547

Summary

BellSoft's Liberica API returns an array of release objects for these checkver endpoints, but the manifests currently use $.version.

That still lets checkver find a version, but it also emits:

Property 'version' not valid on JArray.

This updates the affected Liberica manifests to read versions from the array with $[*].version.

No package URLs, hashes, versions, architecture entries, or install logic are changed.

Verification

Ran checkver against all changed Liberica manifests. The manifests resolve to their current versions without the JArray warning.

Also spot-checked URLs for representative variants:

.\bin\checkurls.ps1 -App liberica-jdk -Dir .\bucket
.\bin\checkurls.ps1 -App liberica-lts-jdk -Dir .\bucket
.\bin\checkurls.ps1 -App liberica8-jre -Dir .\bucket

Results:

[2][2][0] liberica-jdk
[2][2][0] liberica-lts-jdk
[2][2][0] liberica8-jre

Summary by CodeRabbit

Chores

  • Updated version detection configuration across 48 Liberica JDK and JRE package manifests to improve how available versions are discovered from the release API, enabling proper parsing of array-based API responses for autoupdate functionality.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 23, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 04046a62-8c41-43cc-a507-9789c20f88f2

📥 Commits

Reviewing files that changed from the base of the PR and between aded4b6 and ce6eb1f.

📒 Files selected for processing (49)
  • bucket/liberica-full-jdk.json
  • bucket/liberica-full-jre.json
  • bucket/liberica-full-lts-jdk.json
  • bucket/liberica-full-lts-jre.json
  • bucket/liberica-jdk.json
  • bucket/liberica-jre.json
  • bucket/liberica-lite-jdk.json
  • bucket/liberica-lite-lts-jdk.json
  • bucket/liberica-lts-jdk.json
  • bucket/liberica-lts-jre.json
  • bucket/liberica11-full-jdk.json
  • bucket/liberica11-full-jre.json
  • bucket/liberica11-jdk.json
  • bucket/liberica11-jre.json
  • bucket/liberica11-lite-jdk.json
  • bucket/liberica16-full-jdk.json
  • bucket/liberica16-full-jre.json
  • bucket/liberica16-jdk.json
  • bucket/liberica16-jre.json
  • bucket/liberica16-lite-jdk.json
  • bucket/liberica17-full-jdk.json
  • bucket/liberica17-full-jre.json
  • bucket/liberica17-jdk.json
  • bucket/liberica17-jre.json
  • bucket/liberica17-lite-jdk.json
  • bucket/liberica21-full-jdk.json
  • bucket/liberica21-full-jre.json
  • bucket/liberica21-full-lts-jdk.json
  • bucket/liberica21-full-lts-jre.json
  • bucket/liberica21-jdk.json
  • bucket/liberica21-jre.json
  • bucket/liberica21-lite-jdk.json
  • bucket/liberica21-lite-lts-jdk.json
  • bucket/liberica21-lts-jdk.json
  • bucket/liberica21-lts-jre.json
  • bucket/liberica25-full-jdk.json
  • bucket/liberica25-full-jre.json
  • bucket/liberica25-full-lts-jdk.json
  • bucket/liberica25-full-lts-jre.json
  • bucket/liberica25-jdk.json
  • bucket/liberica25-jre.json
  • bucket/liberica25-lite-jdk.json
  • bucket/liberica25-lite-lts-jdk.json
  • bucket/liberica25-lts-jdk.json
  • bucket/liberica25-lts-jre.json
  • bucket/liberica8-full-jdk.json
  • bucket/liberica8-full-jre.json
  • bucket/liberica8-jdk.json
  • bucket/liberica8-jre.json

Walkthrough

Updated checkver.jsonpath configuration across 49 Liberica bucket manifest files from $.version (single field) to $[*].version (array iteration) to match the API response structure which returns an array of version objects rather than a single root version field.

Changes

Cohort / File(s) Summary
Liberica Full JDK/JRE
bucket/liberica-full-jdk.json, bucket/liberica-full-jre.json, bucket/liberica8-full-jdk.json, bucket/liberica8-full-jre.json, bucket/liberica11-full-jdk.json, bucket/liberica11-full-jre.json, bucket/liberica16-full-jdk.json, bucket/liberica16-full-jre.json, bucket/liberica17-full-jdk.json, bucket/liberica17-full-jre.json, bucket/liberica21-full-jdk.json, bucket/liberica21-full-jre.json, bucket/liberica25-full-jdk.json, bucket/liberica25-full-jre.json
Updated checkver.jsonpath from $.version to $[*].version to extract version from all array elements instead of a single root field.
Liberica Full LTS JDK/JRE
bucket/liberica-full-lts-jdk.json, bucket/liberica-full-lts-jre.json, bucket/liberica21-full-lts-jdk.json, bucket/liberica21-full-lts-jre.json, bucket/liberica25-full-lts-jdk.json, bucket/liberica25-full-lts-jre.json
Updated checkver.jsonpath from $.version to $[*].version to parse array-structured API responses.
Liberica Standard JDK/JRE
bucket/liberica-jdk.json, bucket/liberica-jre.json, bucket/liberica8-jdk.json, bucket/liberica8-jre.json, bucket/liberica11-jdk.json, bucket/liberica11-jre.json, bucket/liberica16-jdk.json, bucket/liberica16-jre.json, bucket/liberica17-jdk.json, bucket/liberica17-jre.json, bucket/liberica21-jdk.json, bucket/liberica21-jre.json, bucket/liberica25-jdk.json, bucket/liberica25-jre.json
Updated checkver.jsonpath from $.version to $[*].version to handle array-based release data structure.
Liberica Lite JDK
bucket/liberica-lite-jdk.json, bucket/liberica11-lite-jdk.json, bucket/liberica16-lite-jdk.json, bucket/liberica17-lite-jdk.json, bucket/liberica21-lite-jdk.json, bucket/liberica25-lite-jdk.json
Updated checkver.jsonpath from $.version to $[*].version to accommodate array-formatted API response.
Liberica Lite LTS JDK
bucket/liberica-lite-lts-jdk.json, bucket/liberica21-lite-lts-jdk.json, bucket/liberica25-lite-lts-jdk.json
Updated checkver.jsonpath from $.version to $[*].version to extract versions from array elements.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested labels

review-needed

Suggested reviewers

  • se35710
  • chawyehsu

Poem

🐰 Hops through the array with glee,
No more single version, now we see!
$[*].version brings them all to light,
API responses parsed just right!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: updating JSONPath in Liberica manifests from $.version to $[*].version to fix checkver compatibility with the API response format.
Description check ✅ Passed The description clearly explains the issue (API returns array, causing JArray warnings), the solution (update JSONPath), scope (checkver configuration only), and includes verification steps with confirmed results.
Linked Issues check ✅ Passed The PR comprehensively addresses issue #547 by updating all affected Liberica manifests' checkver JSONPath from $.version to $[*].version, eliminating JArray warnings as required.
Out of Scope Changes check ✅ Passed All changes are scope-limited to JSONPath updates in checkver configurations; no URLs, hashes, versions, architecture entries, or install logic are modified as confirmed in the PR description.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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 and usage tips.

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.

Liberica checkver.jsonpath error

1 participant