[rmarioo]

Summary

• Add rtk mvn command filter for Maven build and test output
• Achieves 60-90% token savings by extracting only errors, failures, and summaries
• Supports mvn compile, mvn test, mvn package, mvn install, mvn clean, and other Maven goals
• Adds Maven to discover rules for automatic detection

Changes

• src/mvn_cmd.rs (new): Maven command filter with build error extraction, test failure parsing, and summary condensing
• src/main.rs: Register Mvn command variant and routing
• src/discover/rules.rs: Add Maven discovery rule
• README.md: Document Maven support and token savings
• tests/fixtures/: Real Maven build and test output fixtures

Test plan

• 14 unit tests covering build success/failure, test success/failure, edge cases
• Token savings verified ≥60% on real Maven output
• cargo fmt --check passes
• cargo clippy — 0 errors (warnings are pre-existing)
• All 777 existing tests still pass (1 pre-existing failure unrelated to this PR)

🤖 Generated with Claude Code

[CLAassistant]

All committers have signed the CLA.

[pszymkowiak]

Hi! Two things needed before we can review:

1. Retarget to develop — this PR targets master, but all PRs should target develop. You can change the base branch in the PR settings (right sidebar).
2. Sign the CLA — if not already done, please sign at https://cla-assistant.io/rtk-ai/rtk

Thanks!

[aeppling]

Hey

We are cleaning up the codebase and improving the project structure for better onboarding. As part of this effort, PR #826 reorganizes src/ from a flat layout into subfolders.

No logic changes — only file moves and import path updates.

What you need to do

Rebase your branch on develop when receiving this comment:

git fetch origin && git rebase origin/develop

Git detects renames automatically. If you get import conflicts, update the paths:

use crate::git;        // now: use crate::cmds::git::git;
use crate::tracking;   // now: use crate::core::tracking;
use crate::config;     // now: use crate::core::config;
use crate::init;       // now: use crate::hooks::init;
use crate::gain;       // now: use crate::analytics::gain;

Need help rebasing? Tag @aeppling

[matheus-meneses]

Any updates, guys?

[rmarioo]

Hi @pszymkowiak @aeppling — quick update:

• ✅ Rebased on develop (force-pushed): conflicts in src/main.rs, src/cmds/jvm/mod.rs and README.md resolved. Maven filter now lives at src/cmds/jvm/mvn_cmd.rs alongside gradlew_cmd.rs, picked up via automod::dir!.
• ✅ CLA signed (mariorusso831).
• ✅ Quality gate green: cargo fmt --check, cargo clippy --all-targets (0 errors), cargo test --all (1830 passed).
• ✅ GitHub now reports the PR as MERGEABLE.

Ready for review whenever you have a moment — thanks!

[rtk-release-bot]

This pull request has been automatically marked as stale due to 90 days of inactivity. If this pull request is still relevant, please leave any comment (for example, "bump"), and we'll keep it open. Your contribution is very much appreciated — we're sorry we haven't been able to review it yet.

[KuSh]

Thanks for this. Maven support landed on develop in #1956 (merged 2026-09-05), which added src/cmds/jvm/mvn_cmd.rs along with the rtk mvn routing and the Maven discover rule that this PR introduced, and #3199 later added rtk mvnd on top. The build and test fixtures here are covered by a broader set already in tests/fixtures, including reactor, locale and no-pom cases. Closing as covered; if some part of the filtering here is still missing from the current implementation, please comment and it can be reopened.