Skip to content

fix(find): reject unsafe bare native actions#2287

Open
antonio-orionus wants to merge 1 commit into
rtk-ai:developfrom
antonio-orionus:fix/find-native-fallback-test-help
Open

fix(find): reject unsafe bare native actions#2287
antonio-orionus wants to merge 1 commit into
rtk-ai:developfrom
antonio-orionus:fix/find-native-fallback-test-help

Conversation

@antonio-orionus
Copy link
Copy Markdown

Summary

  • fall back to native find for compound expressions/actions that RTK should not emulate
  • reject bare native actions like -delete/-exec unless a narrowing predicate is present
  • clarify rtk find and rtk test behavior in user docs

Why

Bare native action forms such as rtk find -delete can default to the current directory when delegated to native find, creating a destructive footgun.

Verification

  • rtk cargo fmt --all --check
  • rtk cargo test find_cmd -- --nocapture
  • rtk cargo clippy --all-targets
  • rtk cargo test --all
  • rtk git diff --check

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Jun 5, 2026

CLA assistant check
All committers have signed the CLA.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants