Skip to content

test(hook): cover option-bearing uv runs in Claude chains#3021

Open
GratefulDave wants to merge 1 commit into
rtk-ai:developfrom
GratefulDave:fix/uv-run-options-hook-rewrite
Open

test(hook): cover option-bearing uv runs in Claude chains#3021
GratefulDave wants to merge 1 commit into
rtk-ai:developfrom
GratefulDave:fix/uv-run-options-hook-rewrite

Conversation

@GratefulDave

Copy link
Copy Markdown

Summary

  • add registry coverage for uv run --extra dev pytest, the option-bearing form skipped by RTK 0.43.0
  • add an end-to-end Claude PreToolUse regression for the frequent compound shape cd ... && uv run --extra dev pytest ... 2>&1 | tail ...
  • lock in develop's dedicated rtk uv routing so this high-frequency Claude Code command keeps receiving an updatedInput rewrite

Related to #294. The production routing landed in #1405; this PR covers the stable-version regression at the Claude hook boundary.

Test plan

  • cargo fmt --all --check && cargo clippy --all-targets && cargo test
  • cargo test uv_run
  • Manual testing: exact Claude Bash payload rewrites uv run --extra dev pytest ... to rtk uv run --extra dev pytest ... inside the compound chain

Important: Targets develop as required by the contribution guide.

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@GratefulDave

Copy link
Copy Markdown
Author

@CLAassistant I signed this. Why does it not show as signed? I signed multiple times.

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.

3 participants