borinquenkid opened a new pull request, #16203: URL: https://github.com/apache/grails-core/pull/16203
## Summary Stacked on #16201. Fixes a series of IntelliJ-flagged issues in `org.grails.gorm.graphql.fetcher.*`: - `DeleteEntityDataFetcher`/`EntityDataFetcher`: simplified branch-per-response construction and added a missing `(T)` cast on `executeQuery`'s `DetachedCriteria#list` result, matching the cast convention every sibling fetcher already uses. - `CustomInterceptorInvoker`/`ProvidedInterceptorInvoker`: deduplicated an identical 9-line "resolve name → invoke → log if prevented → return" fragment into a shared `logIfPrevented` method on `InterceptorInvoker`. - `InterceptorInvoker`: replaced deprecated `DataFetchingEnvironment#getFields()` with `getMergedField()`. - `DefaultGormDataFetcher`: fixed a type mismatch (`Map<String, Association>` → `Map<String, Association<?>>`... with a documented reason it can't be fully parameterized under Groovy's static type checker), dropped an unused constructor parameter, and extracted `resolveDatastore()` to eliminate an ambiguous "might not be assigned" local variable. ## Test plan - [x] `./gradlew :grails-data-graphql-core:test :grails-data-graphql:test :grails-data-graphql-core:codeStyle :grails-data-graphql:codeStyle` passes 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
