Allow repackaging Service file classes with R8

[Goooler]

Follow up #2173 (comment).

---

• CHANGELOG's "Unreleased" section has been updated, if applicable.

[Copilot]

Pull request overview

This PR follows up on prior R8 relocation documentation work by adjusting Shadow’s generated R8 keep rules so that classes referenced from META-INF/services/** can still be repackaged when -repackageclasses is used, while keeping ServiceLoader wiring intact.

Changes:

• Update R8 minimizer service keep rules to permit repackaging of service interfaces/providers.
• Simplify and tighten functional tests around service file relocation and R8 repackaging behavior.
• Add a CHANGELOG entry for the new behavior.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File	Description
src/main/kotlin/com/github/jengelman/gradle/plugins/shadow/internal/R8Minimizer.kt	Adjusts generated ProGuard/R8 rules for service descriptor classes/providers during minimization.
src/functionalTest/kotlin/com/github/jengelman/gradle/plugins/shadow/transformers/ServiceFileTransformerTest.kt	Updates functional tests to validate relocated service entries and R8 repackaging results.
docs/changes/README.md	Documents the change in the Unreleased changelog section.

Suppressed comments (1)

src/main/kotlin/com/github/jengelman/gradle/plugins/shadow/internal/R8Minimizer.kt:217

• Same issue here: allowrepackage should be allowrepackaging to correctly permit R8 repackaging while still keeping the service provider class.

            .forEach { provider -> rules += "-keep,allowrepackage class $provider { *; }" }

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.