Next Release by aeppling · Pull Request #780 · rtk-ai/rtk

aeppling · 2026-03-22T10:20:26Z

No description provided.

* fix: remove decorative emojis from CLI output (#511) Replace decorative emojis with plain text to reduce token waste. Keep functional symbols (⚠️ ✓ ❌ ✅ ℹ️) that convey meaning in fewer tokens. Signed-off-by: Patrick Szymkowiak <patrick@rtk.ai> Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu> * fix: remove remaining decorative emojis from find_cmd and formatter Missed in initial emoji cleanup pass: 📁 in find_cmd.rs and parser/formatter.rs Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu> * fix: remove all decorative emojis from CLI output (#511) Replace emojis with plain text tokens across all production files for better LLM compatibility. Test fixtures and external tool detection patterns (e.g. Black's "All done!") are preserved. Signed-off-by: Patrick Szymkowiak <patrick@rtk-ai.app> Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu> * fix: remove last decorative emoji from next_cmd.rs Remove ⚡ from Next.js Build header, missed in previous passes. Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu> * fix: remove remaining emojis from gh_cmd.rs and init.rs Replace production emojis: - gh_cmd.rs: 🟣→[merged], ⚪→[unknown]/[pending], ⭐→removed, 🔱→removed - init.rs: ⚪→[--] for "not found" status indicators Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu> * fix: remove all checkmark emojis from CLI output Replace ✓ (U+2713) with plain text across 19 files: - "ok ✓" → "ok" (git add/commit/push/pull) - "✓ cargo test: ..." → "cargo test: ..." (all tool summaries) - Preserved ✓ in input detection patterns and test fixtures LLMs cannot interpret emoji semantics; plain text is clearer. Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu> --------- Signed-off-by: Patrick Szymkowiak <patrick@rtk.ai> Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu> Signed-off-by: Patrick Szymkowiak <patrick@rtk-ai.app>

* feat: add OpenClaw plugin for transparent exec rewriting Adds an OpenClaw plugin that intercepts exec tool calls via the before_tool_call hook and rewrites commands to their RTK equivalents. This is the OpenClaw equivalent of hooks/rtk-rewrite.sh for Claude Code. The plugin: - Registers a before_tool_call hook on the exec tool - Rewrites git, grep, find, ls, gh, docker, kubectl, and test commands - Guards against rewriting piped/compound commands and heredocs - Returns properly typed PluginHookBeforeToolCallResult - Supports enabled/verbose config options Measured savings: 48-87% token reduction on common commands. Files: - openclaw/index.ts — plugin source - openclaw/openclaw.plugin.json — plugin manifest - openclaw/README.md — installation and usage docs * refactor: delegate OpenClaw plugin to rtk rewrite Replace 60+ hardcoded regex rules with a single call to `rtk rewrite`, matching the OpenCode plugin pattern (hooks/opencode-rtk.ts). Benefits: - Zero maintenance: new RTK filters work automatically - Single source of truth: rewrite logic in Rust (src/discover/registry.rs) - 122 → 73 lines, no rule duplication Also: rebase on develop, fix homepage URL, bump version to 1.0.0. Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu> * feat: add package.json for npm publishing Enables `openclaw plugins install @rtk-ai/rtk-rewrite` for OpenClaw users. Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu> --------- Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu> Co-authored-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>

- Add `rtk hook gemini` command: native Rust hook processor for Gemini CLI BeforeTool hooks. Reads JSON from stdin, delegates to `rewrite_command()` (single source of truth), outputs Gemini-format JSON response. - Add `--gemini` flag to `rtk init`: installs hook wrapper script, GEMINI.md, and patches ~/.gemini/settings.json with BeforeTool hook entry. - Add `rtk init -g --gemini --uninstall`: clean removal of all Gemini artifacts. - 6 unit tests covering hook format, rewrite delegation, and exclusions. Replaces PR #174 which had too many conflicts after upstream restructuring. Signed-off-by: Ousama Ben Younes <benyounes.ousama@gmail.com> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

) * feat(init): add Codex CLI support via AGENTS.md + RTK.md workflow Add --codex mode to rtk init for Codex CLI integration using AGENTS.md + RTK.md, while keeping the newer develop init/opencode flow intact. Includes Codex install/show/uninstall handling, ASCII status output, stricter flag validation, and expanded tests for Codex AGENTS lifecycle and patch-mode rejection. Signed-off-by: Zacaria <havesomecode@gmail.com> * docs: fix validation metadata Signed-off-by: Zacaria <havesomecode@gmail.com> --------- Signed-off-by: Zacaria <havesomecode@gmail.com>

@jeziellopes

Add `rtk hook copilot` command that handles both VS Code Copilot Chat (updatedInput rewrite) and GitHub Copilot CLI (deny-with-suggestion). - Auto-detects format: snake_case (VS Code) vs camelCase (Copilot CLI) - Delegates to `rtk rewrite` (single source of truth) - 14 hook tests (format detection, rewrite gating, output shape) - .github/hooks/rtk-rewrite.json for repo-scoped hook config - .github/copilot-instructions.md for RTK awareness - Test script: hooks/test-copilot-rtk-rewrite.sh Rebased on develop (includes Gemini #573, Codex #377, OpenClaw #358). Original work by @jeziellopes, cleaned up and rebased by maintainer. Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu> Co-authored-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>

Add `rtk init -g --agent cursor` to install RTK hooks for Cursor Agent. Cursor's preToolUse hook supports command rewriting via updated_input, functionally identical to Claude Code's PreToolUse. Works with both the Cursor editor and cursor-cli (they share ~/.cursor/hooks.json). Changes: - New `--agent <name>` flag (claude|cursor) on `rtk init`, extensible for future agents. Default is claude (backward compatible). - Cursor hook script (hooks/cursor-rtk-rewrite.sh) outputs Cursor's JSON format: {permission, updated_input} vs Claude's hookSpecificOutput. - `rtk init --show` reports Cursor hook and hooks.json status. - `rtk init -g --uninstall` removes Cursor artifacts. - `rtk discover` notes that Cursor sessions are tracked via `rtk gain` (Cursor transcripts lack structured tool_use/tool_result blocks). - Unit tests for Cursor hooks.json patching, detection, and removal. Made-with: Cursor Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu> Co-authored-by: Moisei <1199723+moisei@users.noreply.github.com>

Install RTK rules in .windsurfrules (project-scoped) so Cascade prefixes shell commands with rtk for token savings. Windsurf hooks don't support command rewriting (only blocking), so RTK uses the rules-based approach (like Codex with AGENTS.md). Tested: Windsurf Cascade correctly uses rtk git status after install. Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>

) Install RTK rules in .clinerules (project-scoped) so Cline prefixes shell commands with rtk for token savings. Same rules-based approach as Windsurf and Codex. Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>

…how_config) Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>

… hint (#717) * fix(skill/rtk-triage): increase PR/issue limit to 200 with pagination hint Raise gh pr list limit from 60 to 200 to match gh's max per call. Add inline comment explaining how to paginate for repos with >200 open PRs. Update threshold warning from >60 to >200 PRs/issues. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Signed-off-by: Florian BRUNIAUX <florian@bruniaux.com> * docs(architecture): update module count to 67 (hook_cmd added in #573) hook_cmd.rs was added in feat: add Gemini CLI support (#573) but ARCHITECTURE.md was not updated. Fixes pre-push validation failure. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Signed-off-by: Florian BRUNIAUX <florian@bruniaux.com> --------- Signed-off-by: Florian BRUNIAUX <florian@bruniaux.com> Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

golangci-lint v2 removed --out-format=json in favour of --output.json.path stdout. Detect the installed major version at runtime via golangci-lint --version and branch on the correct flag and JSON extraction strategy. - Use --output.json.path stdout for v2, --out-format=json for v1 - Extract JSON from first line only on v2 (v2 appends trailing metadata after JSON line) - Deserialise new v2 fields: SourceLines, Severity, Offset (serde default for v1 compat) - Show first source line per linter-file group on v2 for richer context - Always forward stderr to caller (was silently dropped unless --verbose) - Falls back to v1 behaviour on any version detection failure Signed-off-by: Adam Powis <a.powis.91@gmail.com>

…724) * feat(ruby): add Ruby on Rails support (rspec, rubocop, rake, bundle) Unifies 5 competing PRs (#198, #292, #379, #534, #643) into a single coherent implementation. New commands: - rtk rspec: JSON parsing with text fallback (60%+ savings) - rtk rubocop: JSON parsing, group by cop/severity (60%+ savings) - rtk rake test: Minitest state machine parser (85-90% savings) - rtk bundle install: TOML filter, strip Using lines (90%+ savings) Shared infrastructure: ruby_exec(), fallback_tail(), exit_code_from_output(), count_tokens() in utils.rs. Discover/rewrite rules for rspec, rubocop, rake, rails, bundle including bundle exec and bin/ variants. E2E smoke tests (scripts/test-ruby.sh) covering all 4 commands. 56 new unit tests + 4 inline TOML tests. All 1035 tests passing. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Signed-off-by: Navid EMAD <navid.emad@yespark.fr> * fix(ruby): use TEST= env var for rake single-file test in smoke tests Rails' `rake test` ignores positional file args; use `TEST=path` syntax. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Signed-off-by: Navid EMAD <navid.emad@yespark.fr> * docs(ruby): add Ruby module architecture and update attribution Integrate ARCHITECTURE.md Ruby Module Architecture section and CLAUDE.md module table/fork-features from PR #643. Update PR description attribution. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Signed-off-by: Navid EMAD <navid.emad@yespark.fr> * chore: remove PULL_REQUEST_DESCRIPTION.md from repo PR description lives on GitHub, no need to track in the codebase. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Signed-off-by: Navid EMAD <navid.emad@yespark.fr> --------- Signed-off-by: Navid EMAD <navid.emad@yespark.fr> Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

) - git diff: raise max_hunk_lines from 30 to 100 (LLMs need full hunks) - git log: show 3 body lines instead of 1 (preserves BREAKING CHANGE, migration notes) - json: show values by default (LLMs need values for config debugging), add --schema for types-only Tested with phi4:14b on local LLM — all 3 fixes improve comprehension. Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>

Signed-off-by: YoubAmj <11021965+youbamj@users.noreply.github.com>

rake test ignores positional file arguments and only supports TEST=path for single-file runs. When users pass positional test files (e.g., `rtk rake test file1.rb file2.rb` or `rtk rake test file.rb:15`), select_runner() now switches to `rails test` which handles single files, multiple files, and line-number syntax natively. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Nicholas Lee <nicholas.lee@appfolio.com>

added script to act like release please (release please flag was unclear) added workflow dispatch event + dev like for prelease debug guards for workflow_dispatch (limit to push master for release events) Signed-off-by: aesoft <43991222+aeppling@users.noreply.github.com>

Signed-off-by: aesoft <43991222+aeppling@users.noreply.github.com>

fix(ruby): use rails test for positional file args in rtk rake

fix(cargo): preserve test compile diagnostics

…fixture to file Signed-off-by: Adam Powis <apowis@vulncheck.com>

fix(golangci-lint): add v2 compatibility with runtime version detection

fix(cicd): issue 668 - pre release tag

…ting Platfform modes Signed-off-by: Daniel Marbach <danielmarbach@users.noreply.github.com>

format_compact() was truncating error messages to 2 lines via .take(2). Playwright (and vitest, pytest, cargo test) errors contain the critical expected/received diff and call log starting at line 3+, so agents saw only the error type with no actionable debug information. Fix: iterate all lines of error_message for each failure instead of taking the first 2. Summary line stays compact; only the per-failure detail is preserved in full. Affects all test runners using TestResult (playwright, vitest, cargo, pytest, dotnet) — any test failure is now fully visible. Closes #690 Signed-off-by: Ousama Ben Younes <benyounes.ousama@gmail.com>

Add crate::tee::tee_and_hint call to playwright_cmd::run(), matching the pattern used by all other test runners (vitest, pytest, cargo, go, rspec, rake). When playwright fails, the full raw output is saved to ~/.local/share/rtk/tee/ and a one-line hint is appended so the LLM can read the complete output without re-running. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Signed-off-by: Ousama Ben Younes <benyounes.ousama@gmail.com>

fix(dotnet): TRX injection for Microsoft.Testing.Platform projects

License to Apache 2.0

Add --comments to the passthrough trigger list in should_passthrough_pr_view(). The --comments flag changes gh output to include comments, but the rtk filter hardcodes a JSON field list without 'comments', silently discarding all comment data. This misleads AI agents into concluding issues have no responses. Add should_passthrough_issue_view() for view_issue(), which previously had no passthrough check at all. Both functions now passthrough on --comments, --json, --jq, and --web. Fixes #720 Signed-off-by: Matt Van Horn <455140+mvanhorn@users.noreply.github.com>

…essage-truncation fix(formatter): show full error message for test failures (#690)

…passthrough fix(gh): passthrough --comments flag in issue/pr view

Was using develop ancestor -> wrong behavior because version happen on master branch

Cicd wrong prerelease tag

When `gh pr diff` is called with output-format-changing flags like --name-only, --stat, --name-status, --numstat, or --shortstat, the output is a plain filename/stat list rather than a unified diff. compact_diff() expects diff headers and hunks, so it produces empty output from these formats. Skip filtering and passthrough directly when any of these flags are present. The output is already compact and doesn't benefit from diff compaction. Fixes #730 Signed-off-by: Matt Van Horn <455140+mvanhorn@users.noreply.github.com>

CLAassistant · 2026-03-22T10:20:39Z

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
4 out of 14 committers have signed the CLA.

✅ ousamabenyounes
✅ jeziellopes
✅ mvanhorn
✅ aeppling
❌ pszymkowiak
❌ aneym
❌ FlorianBruniaux
❌ moisei
❌ youbamj
❌ Maimer
❌ navidemad
❌ danielmarbach
❌ Zacaria
❌ apowis
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

pszymkowiak · 2026-03-22T10:21:04Z

[w] wshm · Automated triage by AI

📊 Automated PR Analysis


✨ Type	`feature`
🔴 Risk	`high`

Summary

This is a release PR merging the develop branch into master, containing a wide range of changes including CI/CD pipeline improvements (conventional commit-based versioning, workflow_dispatch trigger, cross-compilation matrix), new Copilot instructions, a pre-tool-use hook configuration, updated triage skill pagination limits, and numerous new or improved command filters and modules across the Rust codebase.

Review Checklist

Tests present
Breaking change
Docs updated

Analyzed automatically by wshm · This is an automated analysis, not a human review.

fix(gh): skip compact_diff for --name-only/--stat in pr diff

aeppling · 2026-03-22T15:30:17Z

1 non passing smoke test (from scripts/test-all.sh)

Results: 104 PASS, 1 FAIL

Failure: rtk json shows schema

Test checks for "string" in output
Actual output: name: "test" — value shown, not type annotation

pszymkowiak and others added 30 commits March 18, 2026 10:41

chore: merge master into develop (resolve emoji conflicts in Cursor s…

4020aa0

…how_config) Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>

fix: preserve cargo test compile diagnostics

15d5beb

Signed-off-by: YoubAmj <11021965+youbamj@users.noreply.github.com>

fix(cicd): missing doc

8657494

Signed-off-by: aesoft <43991222+aeppling@users.noreply.github.com>

Merge pull request #741 from Maimer/pim_fix_rake_multiple_test_files

ec92c43

fix(ruby): use rails test for positional file args in rtk rake

Merge pull request #738 from youbamj/fix/cargo-test-compile-diagnostics

97b6878

fix(cargo): preserve test compile diagnostics

fix(golangci): use resolved_command for version detection, move test …

6aa5e90

…fixture to file Signed-off-by: Adam Powis <apowis@vulncheck.com>

Merge pull request #722 from apowis/fix/golangci-lint-v2-compatibility

95a4961

fix(golangci-lint): add v2 compatibility with runtime version detection

Merge pull request #736 from rtk-ai/fix/issue-668/pre-release-tag

200af43

fix(cicd): issue 668 - pre release tag

Fix trx injection for dotnet tests running with various Microsoft Tes…

cadbb23

…ting Platfform modes Signed-off-by: Daniel Marbach <danielmarbach@users.noreply.github.com>

Add files via upload

4a79e48

Update ci.yml

da999c8

Update CONTRIBUTING.md

4b2179e

Merge pull request #746 from danielmarbach/dotnet-test

8eefef1

fix(dotnet): TRX injection for Microsoft.Testing.Platform projects

Merge pull request #768 from rtk-ai/license-to-apache-2-0

dff0d5a

License to Apache 2.0

mvanhorn and others added 7 commits March 21, 2026 13:54

Merge pull request #729 from ousamabenyounes/fix/test-failure-error-m…

dc6b026

…essage-truncation fix(formatter): show full error message for test failures (#690)

Merge pull request #775 from mvanhorn/osc/720-issue-pr-view-comments-…

75cd223

…passthrough fix(gh): passthrough --comments flag in issue/pr view

fix(cicd): pre-release correct tag

1536667

Was using develop ancestor -> wrong behavior because version happen on master branch

fix(cicd): explicit fetch tag

3b94b60

Merge pull request #778 from rtk-ai/cicd-wrong-prerelease-tag

4b52bfe

Cicd wrong prerelease tag

Merge branch 'master' into develop

0bb3eaf

pszymkowiak added effort-large Plusieurs jours, nouveau module enhancement New feature or request labels Mar 22, 2026

Merge pull request #773 from mvanhorn/osc/730-pr-diff-name-only-empty

c576249

fix(gh): skip compact_diff for --name-only/--stat in pr diff

aeppling mentioned this pull request Mar 22, 2026

rtk gh pr diff --name-only returns empty output — diff filter applied to non-diff data #730

Closed

aeppling requested review from FlorianBruniaux and pszymkowiak March 22, 2026 11:55

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Next Release#780

Next Release#780
aeppling wants to merge 39 commits intomasterfrom
develop

aeppling commented Mar 22, 2026

Uh oh!

CLAassistant commented Mar 22, 2026 •

edited

Loading

Uh oh!

pszymkowiak commented Mar 22, 2026

Uh oh!

aeppling commented Mar 22, 2026 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

15 participants

Conversation

aeppling commented Mar 22, 2026

Uh oh!

CLAassistant commented Mar 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

pszymkowiak commented Mar 22, 2026

📊 Automated PR Analysis

Summary

Review Checklist

Uh oh!

aeppling commented Mar 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

1 non passing smoke test (from scripts/test-all.sh)

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

15 participants

CLAassistant commented Mar 22, 2026 •

edited

Loading

aeppling commented Mar 22, 2026 •

edited

Loading