Repository navigation
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. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3083ae181b
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| if "/src/" in stem: | ||
| modules.add(stem.split("/src/", 1)[1].replace("/", ".")) |
There was a problem hiding this comment.
For the common Python src layout, a production path such as src/pkg/tool.py produces only src.pkg.tool: the condition does not match because the stem starts with src/ rather than containing /src/. Tests normally import it as from pkg.tool import SpecificTool, so targets & modules is empty and all related assertions are silently omitted from the foundation packet. Handle a leading src/ prefix as well as embedded source directories.
Useful? React with 👍 / 👎.
Unlimited foundation packets omitted tests already present in the pinned shared index. Packets now include directly related imports and concrete assertion ranges, including nested SDK and late-file assertions. Specific capability tests rank ahead of broad entry-module matches so path order cannot exhaust their evidence budget.
Test excerpts reserve at most 20,000 bytes inside the existing 100,000-byte source budget; documentation retains 30,000 bytes. Actual numbered lines remain bound to the fixed source and native inputs. Production counters exclude tests. Finite mode, old approvals, model configuration, global version4 identity and the three-correction limit are preserved.
Validation: 75 related-test, feedback, context, native-checkpoint and publication tests passed. Read-only localization at JiuwenSwarm f0a69728 found real assertions in the session-messaging, share-image-export, trusted-search and command-sandbox tests; each packet stayed within 20,000 bytes. The initial share-image smoke omission is retained outside Git, and the corrected smoke includes its direct test file. The smoke executed no project tests or native models.