fixup! feat: add Maven (mvn) command support with 7 subcommands (#338)

fixup! feat: add Maven (`mvn`) command support with 7 subcommands (#338)

Address PR #368 review comments:

- Fix multi-module test accumulation (aggregate vs per-class lines)
- Tighten test name heuristic (exclude stack traces and error messages)
- Add Failsafe warning for `integration-test` deferred reporting
- Extract `run_mvn_filtered` helper (eliminate ~280 lines of duplication)
- Fix 6-part classifier coordinate parsing in dependency:tree
- Fix over-broad `Building ` noise filter (preserve jar/war lines)
- Replace raw byte indexing with safe `strip_prefix`
- Add mvnd detection and discovery rules
- Add missing subcmd_savings (package, install, integration-test)

Co-Authored-By: Claude Opus 4.6 <[email protected]>

Author: alexvanacker

src/discover/rules.rs

@@ -45,7 +45,7 @@ pub const PATTERNS: &[&str] = &[
     r"^go\s+(test|build|vet)",
     r"^golangci-lint(\s|$)",
     // Maven
-    r"^(\.\/mvnw|mvnw|mvn)\s+",
+    r"^(\.\/mvnw|mvnw|mvnd|mvn)\s+",
     // AWS CLI
     r"^aws\s+",
     // PostgreSQL
@@ -304,13 +304,16 @@ pub const RULES: &[RtkRule] = &[
     // Maven
     RtkRule {
         rtk_cmd: "rtk mvn",
-        rewrite_prefixes: &["./mvnw", "mvnw", "mvn"],
+        rewrite_prefixes: &["./mvnw", "mvnw", "mvnd", "mvn"],
         category: "Build",
         savings_pct: 80.0,
         subcmd_savings: &[
             ("test", 90.0),
             ("clean", 90.0),
             ("compile", 75.0),
+            ("package", 80.0),
+            ("install", 80.0),
+            ("integration-test", 85.0),
             ("dependency:tree", 65.0),
         ],
         subcmd_status: &[],