rtk-ai/rtk

gradle.toml matcher does not match real `rtk ./gradlew` / `rtk gradle` invocations

Open

#1177 opened on Apr 10, 2026

View on GitHub
 (1 comment) (1 reaction) (0 assignees)Rust (48,085 stars) (2,914 forks)batch import
bugeffort-smallfilter-qualitygood first issue

Description

src/filters/gradle.toml currently has a match_command that does not match the command string RTK actually looks up at runtime, so Gradle output falls back to passthrough.

MWE

# in any Gradle project with ./gradlew present
RTK_TOML_DEBUG=1 rtk ./gradlew tasks

# actual:
# [rtk:toml] looking up filter for: "gradlew tasks" (58 filters loaded)
# [rtk:toml] no filter matched — passthrough

Expected

[rtk:toml] matched filter: 'gradle'

Debug

RTK_TOML_DEBUG=1 rtk ./gradlew tasks
RTK_TOML_DEBUG=1 rtk gradle tasks # if a gradle executable is on the path

Local workaround Use a user/project filter override with:

match_command = "^(?:\\./)?gradlew?\\b"

Proposed fix in rtk

  • Update src/filters/gradle.toml matcher to match the normalized lookup string.
  • Add inline tests for gradlew ... and ./gradlew ....
  • Add an integration test covering actual fallback lookup behavior.

Context

  • Similar fix was already proposed in PR #966, but it was closed unmerged.
  • Related: #1178.

Contributor guide

Tech stack
rustshell
Domain
clitooling
Issue type
bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
under 1 hour
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
fresh
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
basic regexTOML configuration
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
80
Research direction
The fix requires updating the match command regex in `src/filters/gradle.toml` to match commands like `gradlew` or `./gradlew` with arguments. Similar work was attempted in PR #966, which was closed unmerged; reviewing that PR may provide insights. The issue also references #1178 for context. The solution should also add inline tests in the same file and possibly an integration test to verify the fallback lookup behavior. The maintainer has not yet commented on scope, but the issue is well defined.