Skip to content

Bump Gradle -> 9.8.2, roll Kotlin back to the 2.4.10 cap - #812

Merged
alexander-yevsyukov merged 3 commits into
masterfrom
bump-gradle-9.8.2
Oct 21, 2026
Merged

alexander-yevsyukov merged 3 commits into
masterfrom
bump-gradle-9.8.2

Conversation

@alexander-yevsyukov

@alexander-yevsyukov alexander-yevsyukov commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

This PR bumps Gradle to 9.8.2, rolls Kotlin back under the Kotlin version cap, and moves the .agents/shared pin to the agents tip, which brings in the new kotlin-version-cap skill.

Changes

  • Gradle 9.8.1 → 9.8.2 (5af55497). Gradle 9.8.2 still embeds Kotlin 2.4.10, per the compatibility matrix and ./gradlew --version. The cap stays at 2.4.10.
  • Roll Kotlin back to 2.4.10 (32aca21a). The SDK's Kotlin must not exceed the Kotlin embedded in the Gradle version of config's wrapper, because our Gradle plugins and the Spine Compiler run on Gradle's embedded Kotlin. Gradle pins that Kotlin with strictly constraints on buildscript classpaths. Kotlin 2.4.21 was raised in d655c277 and has been making Spine Compiler builds unresolvable in consumers.
    • Kotlin 2.4.21 → 2.4.10: Kotlin.runtimeVersion, Kotlin.embeddedVersion, and kotlinEmbeddedVersion in buildSrc/build.gradle.kts.
    • KotlinPoet 2.4.0 → 2.3.0. 2.4.0 requires Kotlin 2.4.20; 2.3.0 requires 2.3.20.
    • KotlinCompileTesting (kctfork) 0.14.0 → 0.13.0. 0.14.0 requires Kotlin 2.4.20; 0.13.0 requires 2.4.0.
  • .agents/shared f3d8ac5 → 9cb20bd (780b333a). This adds the kotlin-version-cap skill and its kotlin_cap.py script, which bump-gradle, dependency-update, and dependency-audit now use. It also brings cross-repo gating in pre-pr-gate.sh and worktree-aware hook scripts. Every hook script wired in .claude/settings.json and .codex/hooks.json still exists and is executable at 9cb20bd. None of the commits in the range asks consumers for follow-up changes.
    The submodule's content is not part of this PR's diff. Review it in the agents compare view for f3d8ac5...9cb20bd (61 commits, 33 files), where each change was reviewed in its own agents PR (Stamp copyright headers in .html files agents#44 through Cap the SDK's Kotlin at the version embedded in Gradle agents#50).

Follow-up: rebuild artifacts built with Kotlin 2.4.21

These local/ pins were published with Kotlin 2.4.21, and they keep that requirement until they are rebuilt (kotlin_cap.py requirement --cap 2.4.10):

Artifact Pinned version Kotlin requirement
io.spine:spine-base 2.0.0-SNAPSHOT.461 2.4.21
io.spine:spine-reflect 2.0.0-SNAPSHOT.202 2.4.21
io.spine.tools:base-testlib 2.0.0-SNAPSHOT.215 2.4.21
io.spine:spine-logging 2.0.0-SNAPSHOT.426 2.4.21
io.spine.tools:* (tool-base) 2.0.0-SNAPSHOT.426 2.4.21
io.spine.tools:prototap-* 0.17.3 2.4.21

After this merges, those repositories need a rebuild in dependency order (base-libraries → … → compiler → core-jvm-compiler) with this config. Their local/ pins can then move to the rebuilt versions. Every other local/ pin is within the cap.

Verification

  • ./gradlew :buildSrc:build detekt --rerun-tasks on Gradle 9.8.2 with Kotlin 2.4.10: 224 tests pass, and validatePlugins and detekt are clean.
  • kotlin_cap.py: the changed modules and kotlin-bom (checked with --platform) are all within the 2.4.10 cap.
  • /pre-pr passed: dependency-audit, spine-code-review, and kotlin-engineer all approve. The version gate does not apply, since config has no version.gradle.kts.

🤖 Generated with Claude Code

alexander-yevsyukov and others added 3 commits October 21, 2026 15:09
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The SDK's Kotlin must not exceed the Kotlin embedded in the Gradle
version of this wrapper (the `kotlin-version-cap` skill): our Gradle
plugins and the Spine Compiler run on Gradle's embedded Kotlin, and
Gradle pins it with `strictly` constraints on buildscript classpaths.
Gradle `9.8.2` embeds Kotlin `2.4.10`, so the cap is `2.4.10`.

- Kotlin `2.4.21` -> `2.4.10`: `Kotlin.runtimeVersion`,
  `Kotlin.embeddedVersion`, and `kotlinEmbeddedVersion` of the build
  script.
- KotlinPoet `2.4.0` -> `2.3.0`: `2.4.0` requires Kotlin `2.4.20`.
- KotlinCompileTesting (kctfork) `0.14.0` -> `0.13.0`: `0.14.0`
  requires Kotlin `2.4.20`.

Spine artifacts already published with Kotlin `2.4.21` keep that
requirement until rebuilt: `spine-base` `.461`, `spine-logging` `.426`,
tool-base `.426`, and ProtoTap `0.17.3`. They need a rebuild in
dependency order after this change lands.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Brings in the `kotlin-version-cap` skill and its `kotlin_cap.py`
script, which `bump-gradle`, `dependency-update`, and
`dependency-audit` now use, plus cross-repo gating in
`pre-pr-gate.sh` and worktree-aware hook scripts.

Every hook script wired in `.claude/settings.json` and
`.codex/hooks.json` is still present and executable at `9cb20bd`.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 21, 2026 14:53

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing. Review the Copilot code review billing settings.

@alexander-yevsyukov alexander-yevsyukov self-assigned this Oct 21, 2026
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-21T15:04:28.426384Z 780b333 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔵 Needs a closer look

The described shared-agent hook and gating changes were absent from the supplied diff and still require human review.

No open issues 🎉

Review effort: Balanced

@alexander-yevsyukov
alexander-yevsyukov merged commit 604420c into master Oct 21, 2026
3 of 4 checks passed
@alexander-yevsyukov
alexander-yevsyukov deleted the bump-gradle-9.8.2 branch October 21, 2026 15:19
@github-project-automation github-project-automation Bot moved this from 🏗 In progress to ✅ Done in v2.0 Oct 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants