Skip to content

Tolerate redundant cwd prefixes in SmartTool paths#9

Merged
Zorlin merged 1 commit into
mainfrom
codex/smarttool-path-resolution
Jul 4, 2026
Merged

Tolerate redundant cwd prefixes in SmartTool paths#9
Zorlin merged 1 commit into
mainfrom
codex/smarttool-path-resolution

Conversation

@Zorlin

@Zorlin Zorlin commented Jul 4, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Add a shared SmartTool path resolver for local file arguments
  • Keep absolute paths unchanged
  • Recover when a relative path repeats the current cwd basename, e.g. cwd is .../codex-rs and the model passes codex-rs/sandboxing/mod.rs
  • Apply the resolver to both SmartRead and SmartWrite while preserving outside-project validation

Testing

  • just test -p codex-core smart_path smart_read smart_write
  • just fix -p codex-core
  • just fmt
  • cargo build --release -p codex-cli

Local install

  • Installed /home/wings/projects/rolodex/codex-rs/target/release/rolodex to /usr/local/bin/rolodex

@Zorlin Zorlin merged commit bf45ef1 into main Jul 4, 2026
7 of 30 checks passed
@Zorlin Zorlin deleted the codex/smarttool-path-resolution branch July 4, 2026 02:41
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.

1 participant