feat(gradle): add Gradle/Gradlew support with compact output (80-90% token reduction)

[sebastianbarrozo]

Summary

Adds rtk gradle command with smart filtering for Gradle/Gradlew output, achieving 80-90% token reduction.

Closes #372

Features

• Auto-detects ./gradlew wrapper, falls back to gradle global
• Trailing args: any task works (build, test, smartbuild, compile.complete, etc.)
• Discover rules: hook rewrite for gradle, gradlew, ./gradlew commands

Filtering

Removed (noise):

• Task progress lines (UP-TO-DATE, NO-SOURCE, SKIPPED, FROM-CACHE)
• > Configure project and incubating feature warnings
• --info verbose output (daemon init, file watching, evaluation, caching)
• Download progress bars and welcome banners
• * Try: suggestion blocks (all lines, not just known patterns)
• Duplicate Note: deprecation warnings
• Deep dependency tree levels (keeps top-level + 1)

Kept (signal):

• FAILED tasks + compilation errors + stack traces
• [ant:*] output (Etendo smartbuild)
• BUILD SUCCESSFUL / BUILD FAILED summaries
• Test failure details and summaries
• File:line compiler error references

Tests

• 25 tests total (19 unit + 6 insta snapshots)
• All filter scenarios covered with assert_snapshot!
• Token savings verified ≥60% for all scenarios
• cargo fmt --all --check && cargo clippy --all-targets && cargo test --all passes (634 tests)

Token savings

Scenario	Savings
Build success (UP-TO-DATE)	≥60%
--info verbose	≥80%
Dependency tree	≥60%
Build failure	Keeps errors, removes noise

Review feedback addressed

• Rebased on develop
• Fixed * Try: state machine leak — now skips all > and * lines inside block
• Fixed in_failed_test dead reset — removed unreachable empty-line check
• Fixed dep tree duplicate BUILD line
• Raised savings threshold from 40% to 60% (project minimum)
• Added 6 insta snapshot tests
• Removed accidentally committed app.log
• DCO sign-off on all commits

Signed-off-by: Sebastian Barrozo [email protected]

[sebastianbarrozo]

This PR replaces #374 (closed), which targeted master instead of develop and didn't address the review feedback.

Based on the review comments on #374:

1. Rebase on develop — clean branch from origin/develop, no merge commits
2. * Try: state machine leak — block now skips all > and * prefixed lines, not just two hardcoded patterns
3. in_failed_test dead reset — removed unreachable empty-line check (blank lines consumed earlier at line 88). Resets on > Task, BUILD, or test summary lines instead
4. Token savings threshold — raised from 40% to ≥60% (project minimum). Also fixed a bug where BUILD line was duplicated in dependency tree output
5. Snapshot tests — added 6 insta::assert_snapshot! tests covering all filter scenarios

Also removed an accidentally committed app.log and added DCO sign-off per CONTRIBUTING.md.

Aware of competing PRs (#288, #312, #368) — happy to adapt if needed.

[sishbi]

what is preventing this from being merged? I use gradle all of the time and would benefit from this greatly

[CLAassistant]

All committers have signed the CLA.

[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