/review at times looks at older, stale (already committed) changes
Resolved 💬 3 comments Opened Oct 24, 2025 by guidedways Closed Jan 10, 2026
💡 Likely answer: A maintainer (github-actions[bot], contributor)
responded on this thread — see the highlighted reply below.
What version of Codex is running?
0.48.0-alpha.4
What subscription do you have?
Pro
Which model were you using?
gpt-5-codex-high
What platform is your computer?
macOS
What issue are you seeing?
Our project's root is a git repo with no code, and then several git submodules, each representing separate apps / extensions / dependencies.
Sometimes this confuses codex, it ends up looking at the wrong changes.
What steps can reproduce the bug?
Uploaded thread: 019a140b-0ccd-7b01-ac57-b8d1df9e866f
- Codex made some changes
- I run
/reviewwithReview uncommitted changes - It correctly performs a review on the changed files within a sub-module and points out an issue
- Codex then fixes the issue, I notice there are some other issues with the changes, ask a few questions and make it perform further changes
- Run
/reviewwithReview uncommitted changesagain - only this time it picks some completely random 'changes' which don't even exist, those were committed and pushed hours ago. Completely misses the only two uncommitted files in a sub-module out of the dozens of sub-modules we have
What is the expected behavior?
It should be able to figure out what's uncommitted.
I end up always performing a 'custom instructions' review, drag and drop the file paths and ask it to review these only. Then 'Review uncommitted changes' does the wrong thing majority of the times I've tried it, finally decided to report this as a bug.
Additional information
_No response_
3 Comments
Potential duplicates detected. Please review them and close your issue if it is a duplicate.
Powered by Codex Action
Looking at this carefully it turns out this is happening because codex doesn't in fact cater for submodules and so it's looking at an old submodule 'pointer' from the base repo, not actual _uncommitted_ changes as the tool promises to review.
Closing old bugs.