The GitHub Actions job "CI" on grails-core.git/grails8-groovy6-canary has failed. Run started by GitHub user jamesfredley (triggered by jamesfredley).
Head commit for run: 3cbd88b15a2f19b0e331f6abc57bea80166c5664 / James Fredley <[email protected]> Drop GORM generic-getter workaround - GROOVY-11986 fixed upstream Apache Groovy fixed the genericGetMethod over-permissive registration in: apache/groovy 999f6dcd "GROOVY-11986: genericGetMethod registration too permissive: matches any get(X) where X is a supertype of String" apache/groovy a4caaa4b "GROOVY-11986: ... (test)" Both committed shortly after build #571 (the audit baseline at canary commit a69a157b). Latest 6.0.0-SNAPSHOT publication on Apache snapshots is build #609 (timestamp 20260508.194756) which includes the fix. Removed: - GormEntityTransformation: per-entity AST INSTANCE Object get(String) shim (lines around 295-320). The shim was added to give Groovy 6's instance MOP a more specific candidate than the inherited GormEntity.get(Serializable) so dynamic property reads on @Entity instances would fall through to propertyMissing(String) instead of being hijacked by the generic-getter. With GROOVY-11986 in, the generic-getter is no longer registered for the supertype Serializable signature, so the dispatch routes correctly without the shim. - GormEntity: stale doc comment block on get(Serializable) describing the now-resolved Groovy 6 dispatch hijack and the AST workaround that replaced an earlier trait-static guard. - GormEntityTransformSpec: 'test Groovy 6 generic-getter instance- dispatch guard' regression test. It only verified the AST shim was added (Book.getDeclaredMethod('get', String) != null), so it has no meaning once the shim is gone. The actual dispatch behaviour is exercised by the Hibernate5 / Functional / Mongodb integration suites (DataServiceConnectionRoutingSpec, CrossLayerMultiDataSourceSpec) which originally surfaced the regression and will continue to gate the canary CI matrix. Verified locally on JDK 21 against the latest 6.0.0-SNAPSHOT cached from Apache snapshots (publication 20260508.194756, build #609): ./gradlew :grails-datamapping-core:compileGroovy BUILD SUCCESSFUL ./gradlew :grails-datamapping-core:test BUILD SUCCESSFUL Full integration validation (Hibernate5, Functional, Mongodb under both -PgrailsIndy=false and -PgrailsIndy=true) is deferred to the canary CI matrix on this PR. Report URL: https://github.com/apache/grails-core/actions/runs/25581023695 With regards, GitHub Actions via GitBox
