Skip to content

Feat/direnv exec guard#985

Open
ramarnat wants to merge 2 commits intortk-ai:developfrom
locusai:feat/direnv-exec-guard
Open

Feat/direnv exec guard#985
ramarnat wants to merge 2 commits intortk-ai:developfrom
locusai:feat/direnv-exec-guard

Conversation

@ramarnat
Copy link
Copy Markdown

@ramarnat ramarnat commented Apr 2, 2026

Summary

  • add first-class rtk direnv support for literal direnv exec ...
  • rewrite only direnv exec to rtk direnv ..., while leaving non-exec direnv subcommands as passthrough
  • add built-in TOML guards that redact env dumps and auth-token reads on both stdout and stderr

Test plan

  • cargo fmt --all && cargo clippy --all-targets && cargo test
  • Manual testing: rtk <command> output inspected

Manual checks:

  • target/debug/rtk direnv exec . env
  • target/debug/rtk direnv exec . bash -lc 'printenv >&2'
  • target/debug/rtk direnv exec . gh auth token
  • target/debug/rtk direnv allow .
  • target/debug/rtk direnv exec . pnpm --version

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Apr 2, 2026

CLA assistant check
All committers have signed the CLA.

@ramarnat ramarnat changed the base branch from master to develop April 2, 2026 21:55
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