feat(arch): Group generated pom dependencies into TEST, PROD sections

Every generated child-module <dependencies> block is now split into a
"<!-- TEST -->" section (scope 'test', emitted first) and a "<!-- PROD -->"
section (everything else), with both header comments always present. The
archetype currently emits only compile-scope dependencies, so they sit
under PROD while TEST stands ready as a placeholder for test-scoped
dependencies.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FTaQTM9wYTXLvvgCXSwhRm

Author: jeffjensen

src/main/resources/META-INF/archetype-post-generate.groovy

@@ -77,21 +77,32 @@ def failsafeSection = """\
 
 /** Builds a child-module pom.xml string (2-space indent).
  *  A dependency entry is either a String (an internal module artifactId, inheriting this project's
- *  groupId and BOM-managed version) or a Map [groupId:…, artifactId:…] for an external dependency. */
+ *  groupId and BOM-managed version) or a Map [groupId:…, artifactId:…, scope:…] for an external
+ *  dependency. Dependencies are grouped into a "<!-- TEST -->" section (scope 'test', emitted first)
+ *  and a "<!-- PROD -->" section (everything else); both headers are always present. */
 def modulePom = { String gId, String parentAId, String ver,
                    String aId, String desc, List deps = [], String buildSection = '' ->
     def depsSection = ''
     if (deps) {
-        def lines = deps.collect { dep ->
+        def renderDep = { dep ->
             def dGroup = (dep instanceof Map) ? dep.groupId : gId
             def dArtifact = (dep instanceof Map) ? dep.artifactId : dep
+            def dScope = (dep instanceof Map && dep.scope) ? "\n      <scope>${dep.scope}</scope>" : ''
             """\
     <dependency>
       <groupId>${dGroup}</groupId>
-      <artifactId>${dArtifact}</artifactId>
+      <artifactId>${dArtifact}</artifactId>${dScope}
     </dependency>"""
-        }.join('\n')
-        depsSection = "\n  <dependencies>\n${lines}\n  </dependencies>\n"
+        }
+        def isTest = { it instanceof Map && it.scope == 'test' }
+        def testLines = deps.findAll(isTest).collect(renderDep).join('\n')
+        def prodLines = deps.findAll { !isTest(it) }.collect(renderDep).join('\n')
+        def section = ['    <!-- TEST -->']
+        if (testLines) section << testLines
+        section << ''
+        section << '    <!-- PROD -->'
+        if (prodLines) section << prodLines
+        depsSection = "\n  <dependencies>\n${section.join('\n')}\n  </dependencies>\n"
     }
 
     """\

src/test/resources/projects/full/verify.groovy

@@ -239,5 +239,13 @@ assert text("domain-graphql/src/main/java/${p}/domain/graphql/package-info.java"
 assert text('common-domain/pom.xml') =~ /<description>[^<]*\.<\/description>/ \
     : 'pom descriptions must be sentences ending with a period'
 
+// ── dependency lists are split into TEST and PROD sections (TEST first) ──────
+
+def depPom = text('domain-db-users/pom.xml')
+assert depPom.contains('<!-- TEST -->') : 'dependency list must carry a TEST section header'
+assert depPom.contains('<!-- PROD -->') : 'dependency list must carry a PROD section header'
+assert depPom.indexOf('<!-- TEST -->') < depPom.indexOf('<!-- PROD -->')                          : 'TEST section must come before PROD'
+assert depPom.indexOf('<!-- PROD -->') < depPom.indexOf('<artifactId>common-domain</artifactId>') : 'compile-scope deps must sit under PROD'
+
 _buildLog.append("\n=== PASSED ===\n")
 true

src/test/resources/projects/spring-off/verify.groovy

@@ -93,5 +93,13 @@ modules.each { m ->
 assert text('acceptance-tests/pom.xml').contains('<artifactId>maven-failsafe-plugin</artifactId>')     : 'acceptance-tests must configure maven-failsafe-plugin'
 assert text('integration-db-users/pom.xml').contains('<artifactId>maven-failsafe-plugin</artifactId>') : 'integration module must configure maven-failsafe-plugin'
 
+// ── dependency lists are split into TEST and PROD sections (TEST first) ──────
+
+def svcPom = text('service/pom.xml')
+assert svcPom.contains('<!-- TEST -->') : 'dependency list must carry a TEST section header'
+assert svcPom.contains('<!-- PROD -->') : 'dependency list must carry a PROD section header'
+assert svcPom.indexOf('<!-- TEST -->') < svcPom.indexOf('<!-- PROD -->')                           : 'TEST section must come before PROD'
+assert svcPom.indexOf('<!-- PROD -->') < svcPom.indexOf('<artifactId>domain-service</artifactId>') : 'compile-scope deps must sit under PROD'
+
 _buildLog.append("\n=== PASSED ===\n")
 true