Skip to content

Conversation

Copy link

Copilot AI commented Dec 2, 2025

What changed

  • Modified ApiTracker.java to dynamically detect Java version using Runtime.version().feature() instead of hardcoding --release 24
  • Added comprehensive upstream API comparison report documenting breaking changes

Why this change is needed

The API tracker failed on Java 21 with "release version 24 not supported" errors, preventing upstream sync monitoring. The hardcoded version assumption broke the daily CI workflow and local development.

Upstream Changes Detected (7/11 classes affected):

  • Json: fromUntyped()toJson(), toUntyped()fromJson()
  • JsonArray: values()elements(), added array()
  • JsonBoolean: value()bool()
  • JsonNumber: Removed toNumber()/toBigDecimal(), added toLong()/toDouble(), changed of(BigDecimal)of(long)
  • JsonParseException: getErrorRow()getErrorLine(), getErrorColumn()getErrorPosition()
  • JsonString: value()string()
  • JsonObject: Added object()

Full analysis in updates/api-check-2025-12-02.md

How were these changes tested

Checklist

  • Code builds / passes tests
  • New tests added if needed
  • Update to use CODING_STYLE_LLM.md convensions
  • Documentation updated if needed
  • AGENTS.md updated if appropriate
Original prompt

please run the checker module to see if the upstream has got


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 2 commits December 2, 2025 15:21
Changed from hardcoded Java 24 to Runtime.version().feature() to support running on Java 21+

Co-authored-by: simbo1905 <[email protected]>
Generated detailed report showing 7 classes with breaking API changes in upstream OpenJDK sandbox

Co-authored-by: simbo1905 <[email protected]>
Copilot AI changed the title [WIP] Run checker module to verify upstream status Fix API tracker to use runtime Java version instead of hardcoded Java 24 Dec 2, 2025
Copilot AI requested a review from simbo1905 December 2, 2025 15:26
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