Hibernate 7 - Step 1

.agents/skills/grails-developer/SKILL.md

@@ -1,28 +1,13 @@
-<!--
-SPDX-License-Identifier: Apache-2.0
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    https://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
--->
 ---
 name: grails-developer
 description: Comprehensive guide for Grails 7 development, covering web applications, REST APIs, GORM, controllers, services, views, plugins, and testing with Spock and Geb
 license: Apache-2.0
-compatibility: opencode, claude, grok, gemini, copilot, cursor, windsurf
-metadata:
-  audience: developers
-  frameworks: grails
-  versions: 7
 ---
+<!--
+SPDX-License-Identifier: Apache-2.0
+
+Licensed to the Apache Software Foundation (ASF) under one or more contributor license agreements; and to You under the Apache License, Version 2.0. 
+-->
 
 ## What I Do
 

.agents/skills/groovy-developer/SKILL.md

@@ -1,29 +1,13 @@
-<!--
-SPDX-License-Identifier: Apache-2.0
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    https://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
--->
 ---
 name: groovy-developer
 description: Expert guide for Groovy 4 development, covering concise syntax, closures, DSLs, metaprogramming, static compilation, and integration with Java 17 and Grails
 license: Apache-2.0
-compatibility: opencode, claude, grok, gemini, copilot, cursor, windsurf
-metadata:
-  audience: developers
-  languages: groovy, java
-  versions: 4.0
-  paradigms: dynamic, static, functional, object-oriented, DSL-oriented
 ---
+<!--
+SPDX-License-Identifier: Apache-2.0
+
+Licensed to the Apache Software Foundation (ASF) under one or more contributor license agreements; and to You under the Apache License, Version 2.0. 
+-->
 
 ## What I Do
 

.agents/skills/java-developer/SKILL.md

@@ -1,28 +1,13 @@
-<!--
-SPDX-License-Identifier: Apache-2.0
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    https://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
--->
 ---
 name: java-developer
 description: Guide for developing in Java 17 LTS, including modern features, best practices, and integration with Groovy/Grails projects
 license: Apache-2.0
-compatibility: opencode, claude, grok, gemini, copilot, cursor, windsurf
-metadata:
-  audience: developers
-  languages: java
-  versions: 17
 ---
+<!--
+SPDX-License-Identifier: Apache-2.0
+
+Licensed to the Apache Software Foundation (ASF) under one or more contributor license agreements; and to You under the Apache License, Version 2.0. 
+-->
 
 ## What I Do
 

.agents/skills/test-fixer/SKILL.md

@@ -0,0 +1,142 @@
+---
+name: test-fixer
+description: Guide for running, reviewing, and fixing test failures across grails-core modules using Gradle test reports
+license: Apache-2.0
+---
+<!--
+SPDX-License-Identifier: Apache-2.0
+
+Licensed to the Apache Software Foundation (ASF) under one or more contributor license agreements; and to You under the Apache License, Version 2.0.
+-->
+
+## What I Do
+
+- Guide targeted and full test runs across grails-core modules.
+- Help interpret Gradle HTML and XML test reports.
+- Help triage failing specs, static state pollution, and module-specific regressions.
+
+## When to Use Me
+
+Activate this skill when:
+
+- Running a failing module or specific spec.
+- Preparing a commit and verifying affected module tests.
+- Triaging regressions after a dependency upgrade or refactor.
+- Reviewing aggregate test results from `:grails-test-report`.
+
+---
+
+## Key Tasks
+
+### Targeted Testing Recommended
+
+To save time, run only related tests:
+
+```bash
+./gradlew :grails-data-hibernate7-core:test --tests "grails.gorm.tests.BasicCollection*"
+```
+
+### Full Run
+
+```bash
+./gradlew test --continue
+```
+
+Use `--continue` when you need later modules to run after an earlier module fails.
+
+---
+
+## Aggregate Test Reports
+
+The `grails-test-report` project owns repository test aggregation.
+
+```bash
+./gradlew :grails-test-report:check --continue
+```
+
+This generates:
+
+- Unit HTML: `grails-test-report/build/reports/tests/test/index.html`
+- Unit Markdown: `grails-test-report/build/reports/tests/test.md`
+- Integration HTML: `grails-test-report/build/reports/tests/integrationTest/index.html`
+- Integration Markdown: `grails-test-report/build/reports/tests/integrationTest.md`
+- Combined HTML: `grails-test-report/build/reports/tests/combined/index.html`
+- Combined Markdown: `grails-test-report/build/reports/tests/combined.md`
+
+The aggregate report includes root subprojects that define matching `test` or `integrationTest` tasks.
+
+The collected unit and integration `Test` tasks are finalized by their matching aggregate report tasks. Reports are attempted after failures and after targeted module test runs, but they summarize the XML results currently available on disk. Use `--continue` for full-suite runs so Gradle keeps scheduling later test tasks after an earlier failure; without it, reports can be partial. Run `clean` first when stale XML results should be excluded.
+
+---
+
+## Running Tests for a Specific Module
+
+```bash
+# All tests in a module
+./gradlew :grails-data-hibernate7-dbmigration:test
+
+# A specific spec
+./gradlew :grails-data-hibernate7-dbmigration:test \
+  --tests "org.grails.plugins.databasemigration.command.DbmGenerateGormChangelogCommandSpec"
+
+# A specific feature method
+./gradlew :grails-data-hibernate7-core:test \
+  --tests "org.grails.orm.hibernate.FooSpec.my feature name"
+
+# Force rerun even if Gradle thinks it is up-to-date
+./gradlew :grails-data-hibernate7-core:test --rerun-tasks
+```
+
+---
+
+## Test Source Layout
+
+| Source set | Directory | Task |
+|------------|-----------|------|
+| Unit tests | `src/test/groovy/` | `test` |
+| Integration tests | `src/integration-test/groovy/` | `integrationTest` |
+
+Most specs in grails-core are unit tests (`src/test/groovy/`) run by the `test` task. A small number of modules use `integrationTest` for full Spring context tests.
+
+---
+
+## Parallel Test Execution
+
+Tests run in parallel (`maxParallelForks` defaults to `4` on CI and to `availableProcessors * 3/4` otherwise; override with `-PmaxTestParallel`). This can cause:
+
+- Static state pollution: one test mutating a static field that another test reads.
+- Port conflicts: multiple test JVMs binding the same port.
+- `@Shared` field contamination: shared state not properly cleaned up between feature methods.
+
+To diagnose flaky failures, rerun with forced serial execution:
+
+```bash
+./gradlew :module:test -PmaxTestParallel=1 --rerun-tasks
+```
+
+---
+
+## Test Configuration Flags
+
+| Property | Effect |
+|----------|--------|
+| `skipTests` | Skips all `Test` tasks (build only) |
+| `maxTestParallel=N` | Override parallel fork count |
+| `onlyFunctionalTests` | Run only functional test suites |
+| `skipHibernate7Tests` | Skip H7-specific test suites |
+| `onlyMongodbTests` | Run only MongoDB test suites |
+| `onlyCoreTests` | Run only core module tests |
+
+---
+
+## XML Report Location
+
+Gradle writes JUnit XML test results to:
+
+```text
+<module>/build/test-results/
+├── test/
+│   └── TEST-org.grails.SomeSpec.xml
+└── integrationTest/
+    └── TEST-org.grails.SomeIntegrationSpec.xml
+```