/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 /review with Review 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 /review with Review uncommitted changes again - 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_

View original on GitHub ↗

3 Comments

github-actions[bot] contributor · 10 months ago

Potential duplicates detected. Please review them and close your issue if it is a duplicate.

  • #5323
  • #300
  • #294
  • #4282
  • #4018

Powered by Codex Action

guidedways contributor · 10 months ago

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.

etraut-openai contributor · 7 months ago

Closing old bugs.