Skip to content

Fix find native expression passthrough#2000

Open
serrebidev wants to merge 1 commit into
rtk-ai:developfrom
serrebidev:codex/find-native-passthrough-master
Open

Fix find native expression passthrough#2000
serrebidev wants to merge 1 commit into
rtk-ai:developfrom
serrebidev:codex/find-native-passthrough-master

Conversation

@serrebidev
Copy link
Copy Markdown

Summary

  • pass native find expressions/actions through to real find instead of rejecting or partially compacting them
  • preserve shared AI hook rewrites for Claude, Cursor, Gemini, VS Code Copilot, and Copilot CLI while keeping instruction-only agents on the fixed binary path
  • avoid rtk find broken-pipe panics when compact output is piped to commands like head

Verification

  • cargo fmt --check
  • cargo test find_cmd
  • cargo test test_hook_formats_rewrite_native_find_passthrough_command
  • cargo test test_rewrite_find_native_expression_passthrough_safe
  • cargo test
  • live: rtk find . \( -name '*.rs' -o -name '*.toml' \) -printf '%p\\n' | head -n 5
  • live: rtk find . -m 10000 | head -n 1

@rtk-release-bot rtk-release-bot Bot added the wrong-base PR targets master instead of develop label May 20, 2026
@rtk-release-bot
Copy link
Copy Markdown
Contributor

Automatic message from CI checks : It seems like this branch is targeting the wrong branch, any contribution should target develop branch.

See CONTRIBUTING.md for details.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 20, 2026

CLA assistant check
All committers have signed the CLA.

@serrebidev serrebidev marked this pull request as ready for review May 20, 2026 18:17
@serrebidev serrebidev force-pushed the codex/find-native-passthrough-master branch from 238173c to 159b9df Compare May 20, 2026 18:41
@serrebidev serrebidev changed the base branch from master to develop May 20, 2026 18:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

wrong-base PR targets master instead of develop

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants