Skip to content

fix(wrap): prepend rtk bin dir to PATH for wrapped tools#1752

Open
ketpatil77 wants to merge 3 commits into
headroomlabs-ai:mainfrom
ketpatil77:codex/headroom-rtk-path-launch
Open

fix(wrap): prepend rtk bin dir to PATH for wrapped tools#1752
ketpatil77 wants to merge 3 commits into
headroomlabs-ai:mainfrom
ketpatil77:codex/headroom-rtk-path-launch

Conversation

@ketpatil77

@ketpatil77 ketpatil77 commented Jul 3, 2026

Copy link
Copy Markdown

Description

Make wrapped tools able to resolve Headroom-managed rtk without requiring users to manually add ~/.headroom/bin to PATH.

headroom wrap now prepends the managed rtk bin directory to the child process PATH before launching the tool.

Changes Made

  • headroom/cli/wrap.py: added _prepend_rtk_bin_to_path() and call it before launching wrapped tools
  • tests/test_cli/test_wrap_helpers.py: regression test for PATH prepending

Testing

  • pytest tests/test_cli/test_wrap_helpers.py -k "prepend_rtk_bin_to_path_injects_managed_bin_dir"
  • ruff check headroom/cli/wrap.py tests/test_cli/test_wrap_helpers.py

Impact

Bare rtk ... rewrite output now works inside wrapped shells.

@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

PR governance

This PR does not yet satisfy the required template fields:

  • Missing required section Type of Change.
  • Missing required section Real Behavior Proof.
  • Missing required section Review Readiness.
  • Paste real command output or artifact links in TestingTest Output.
  • Check I have performed a self-review before requesting human review.
  • Check This PR is ready for human review or convert the PR back to draft.

Please update the PR body, or move the PR back to draft while it is still in progress.

@github-actions github-actions Bot added the status: needs author action Pull request body or readiness checklist still needs author updates label Jul 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: needs author action Pull request body or readiness checklist still needs author updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant