fix(get): force literal tree lookup - #44
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Warning Review limit reachedNext included review available in 44 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Team Run ID: 📒 Files selected for processing (2)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Intent
Close the post-merge review finding on PR #43: Git pathspec magic must not affect missing-path classification.
Decision
Prefix the diagnostic
git ls-treepath with:(literal). The user path remains a literal repository path even when it begins with Git pathspec syntax.Checks
:(literal)README.mdis absent and returnsnot_foundeven thoughREADME.mdexists.not_found../scripts/check.sh: passed.Rollback
Revert this PR to restore pathspec interpretation in the diagnostic lookup.