Skip to content

docs: add Gradle 9 developer skill for monorepo builds - #16083

Merged
jamesfredley merged 3 commits into
8.0.xfrom
docs/gradle-developer-skill
Aug 5, 2026
Merged

docs: add Gradle 9 developer skill for monorepo builds#16083
jamesfredley merged 3 commits into
8.0.xfrom
docs/gradle-developer-skill

Conversation

@jamesfredley

@jamesfredley jamesfredley commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds a Gradle 9 developer skill so agents write Gradle the way this monorepo already does on 8.0.x, instead of inventing generic Gradle patterns that burn review cycles.

The skill is intentionally repo-first: when official Gradle docs and this monorepo disagree, this repo wins (configuration cache off, no version catalogs, native platform() instead of Spring DM for real apps, heavy projectDir remapping, presence-based -P flags, custom BOM validator).

Changes

  1. .agents/skills/gradle-developer/SKILL.md - monorepo Gradle guide covering:

  2. AGENTS.md

    • Mandatory skill bullet + skills table entry for gradle-developer
    • mono-repo-integration table entry
    • Spring Boot BOM pin corrected to 4.1.x (matches Technology Stack)
    • build-logic uses its own wrapper (cd build-logic && ./gradlew build)

Research basis

Polished against:

Source What we took
#15483 (user-linked) Cacheability: no absolute base.dir in Test props; SBOM normalization; doFirst vs cache eligibility
#15532 / #15537 Overlapping outputs kill caching; bad upToDateWhen blocks cache load; per-task paths
#16076 / #16009 / #16006 No config-time classpath resolve; afterEvaluate races; ReplaceTokens as task inputs
#15467 / #15730 Platforms replace Spring DM; prefer Boot BOM inheritance
#15763 / gradle-bootstrap Multi-location wrapper discipline (root moved by bootstrap)
#16069 / #16078 Cheap functional substitution; CLI/test tier off production classpath
High-quality 8.0.x AI PRs (#15798 skill, #15797 style) Problem/Fix/Why/Verification + scoped evidence
Live tree fact-check Plugin IDs, end-to-end README, SbomPlugin normalization, BaseDirArgumentProvider

Also skimmed recent merged 8.0.x and 7.0.x Gradle/build/agent PRs so the skill encodes durable patterns, not one-off version bumps.

Notes

  • Docs/skills only; no build or runtime code changes.
  • Based on current 8.0.x Gradle 9.6.0 layout.
  • Goal: AI can write monorepo-acceptable Gradle on the first try.

Test plan

  • Spot-check skill claims against settings.gradle, dependencies.gradle, end-to-end/README.md, gradle-bootstrap/build.gradle, SbomPlugin, BaseDirArgumentProvider, convention plugins
  • Dual review-gate (oracle + codex) on polish commit
  • git diff --check
  • Human skim of skill for anything still wrong or missing for day-to-day Gradle work

Copilot AI review requested due to automatic review settings August 2, 2026 16:04

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.

Pull request overview

Adds repository-specific Gradle 9 build guidance as an agent “skill”, and wires it into AGENTS.md so contributors/agents are prompted to load the Gradle guidance before touching build logic. This helps keep Gradle changes aligned with established 8.0.x monorepo conventions (composite builds, BOM/platform usage, wrapper sync, and Gradle 9 configuration-time pitfalls).

Changes:

  • Introduces .agents/skills/gradle-developer/SKILL.md with monorepo-first Gradle 9.6.x guidance (topology, BOM rules, wrapper sync, and configuration hygiene).
  • Updates AGENTS.md to explicitly include Gradle build changes in the “mandatory skill” list and adds the new skill to the skills table.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
AGENTS.md Adds Gradle-specific skill guidance to the mandatory skills list and table so build changes are routed to the new skill.
.agents/skills/gradle-developer/SKILL.md New Gradle 9 monorepo build skill documenting established repo patterns (composite builds, BOM/platform rules, wrapper sync, and Gradle 9 configuration hygiene).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@codecov

codecov Bot commented Aug 2, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 52.3462%. Comparing base (2e2ce06) to head (e4d26c5).
⚠️ Report is 174 commits behind head on 8.0.x.

Additional details and impacted files

Impacted file tree graph

@@                Coverage Diff                 @@
##                8.0.x     #16083        +/-   ##
==================================================
+ Coverage     51.8811%   52.3462%   +0.4651%     
- Complexity      18118      18298       +180     
==================================================
  Files            2046       2036        -10     
  Lines           96274      96347        +73     
  Branches        16727      16829       +102     
==================================================
+ Hits            49948      50434       +486     
+ Misses          38954      38489       -465     
- Partials         7372       7424        +52     

see 61 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@testlens-app

This comment has been minimized.

Capture 8.0.x Gradle conventions so agents write build scripts the way
this repository already does: composite builds, BOM platforms, wrapper
sync via gradle-bootstrap, end-to-end TestCaseMavenRepo flow, and
Gradle 9 configuration hygiene from recent PRs.

Assisted-by: Sisyphus:xai/grok-4.5
Make the Gradle 9 monorepo skill mandatory for build-script work and list
it alongside the other agent skills.

Assisted-by: Sisyphus:xai/grok-4.5
…ssons

Encode durable Gradle lessons from merged 8.0.x/7.x PRs so agents write
acceptable monorepo build changes on the first try.

- Build cacheability rules from #15483, #15532, #15537 (portable test props,
  no overlapping outputs, SBOM classpath normalization, upToDateWhen traps)
- base.dir dual rule: required @internal compile provider, forbidden absolute
  Test.systemProperties
- Fact fixes: Spring DM regression fixture exception, bootstrap wrapper scope,
  grails-publish plugin id, skipDependencyValidation presence semantics
- Agent quality bar, change workflow, expanded shared scripts and PR lessons
- AGENTS.md: Boot BOM pin 4.1.x, mono-repo-integration table entry, build-logic
  local wrapper command

Assisted-by: Sisyphus:xai/grok-4.5
@jamesfredley
jamesfredley force-pushed the docs/gradle-developer-skill branch from 3caaa7a to e4d26c5 Compare August 5, 2026 20:55
@jamesfredley
jamesfredley merged commit 3de06ee into 8.0.x Aug 5, 2026
52 checks passed
@jamesfredley
jamesfredley deleted the docs/gradle-developer-skill branch August 5, 2026 20:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants