Skip to content

ADFA-3906: upgrade to latest kotlin-android#1307

Open
itsaky-adfa wants to merge 1 commit into
stagefrom
fix/ADFA-3906
Open

ADFA-3906: upgrade to latest kotlin-android#1307
itsaky-adfa wants to merge 1 commit into
stagefrom
fix/ADFA-3906

Conversation

@itsaky-adfa
Copy link
Copy Markdown
Contributor

This upgrade resolves a missing class issue in the Kotlin analysis API. See appdevforall/kotlin-android@853aa4b

Fixes ADFA-3906

Signed-off-by: Akash Yadav <akashyadav@appdevforall.org>
@itsaky-adfa itsaky-adfa requested a review from a team May 14, 2026 11:29
@itsaky-adfa itsaky-adfa self-assigned this May 14, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 14, 2026

Review Change Stack

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: 31e81424-4463-46fc-90cd-b3e22fc01211

📥 Commits

Reviewing files that changed from the base of the PR and between b85bbc2 and e060395.

📒 Files selected for processing (1)
  • subprojects/kotlin-analysis-api/build.gradle.kts

📝 Walkthrough

Release Notes: Upgrade to Latest kotlin-android

Changes

  • Updated kotlin-android dependency to commit 853aa4b (from ac7aa4b)
  • Updated external JAR dependency tag from v2.3.255-ac7aa4b to v2.3.255-853aa4b
  • Updated SHA256 checksum for the Kotlin analysis API standalone embeddable JAR artifact

Bug Fix

  • Resolves a missing class issue in the Kotlin analysis API

Impact

  • Affects the Kotlin Analysis API module used for IDE code analysis functionality
  • No changes to public APIs or exported entities
  • Build-only changes with no impact on runtime behavior other than the bug fix

Risk Considerations

  • Dependency Update Risk: This updates an external dependency (kotlin-android) to a specific commit. Ensure the upstream repository and commit are thoroughly vetted before merging
  • Limited Testing Scope: Only two build configuration lines changed; ensure integration testing validates the missing class fix works as expected
  • Artifact Verification: The SHA256 checksum change must be verified to ensure the correct artifact is being downloaded and there are no supply chain concerns

Walkthrough

This PR updates the Kotlin Android analysis API external jar dependency in the Gradle build script by bumping the release tag identifier and updating the corresponding SHA-256 checksum used to verify the downloaded artifact.

Changes

Kotlin Analysis API Dependency Update

Layer / File(s) Summary
External Kotlin Android dependency version and integrity
subprojects/kotlin-analysis-api/build.gradle.kts
The ktAndroidTag release identifier used to construct the jar download URL and the matching sha256Checksum for artifact verification are both updated to acquire a newer version of the analysis API standalone jar.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • appdevforall/CodeOnTheGo#1286: Both PRs modify the same external ktAndroidTag and sha256Checksum for the analysis API jar in the same build script.
  • appdevforall/CodeOnTheGo#1112: Both PRs update the Gradle dependency resolution for the Kotlin analysis API external jar by changing its release tag and checksum.

Suggested reviewers

  • Daniel-ADFA

Poem

🐰 A rabbit hops through build scripts bright,
Two checksums dance in gradle's light,
The Android tag gets its refresh,
With hashes verified, clean and fresh! 🥕

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title 'ADFA-3906: upgrade to latest kotlin-android' directly matches the main change: updating the Kotlin Android dependency to a newer version.
Description check ✅ Passed The description is related to the changeset, explaining that the upgrade resolves a missing class issue and references a specific commit.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/ADFA-3906

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.

2 participants