Test dependency behavior and preserve usage during review refresh - #161
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: 2edb487b1a
ℹ️ 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".
|
@codex review Please review the updated head |
|
Codex Review: Didn't find any major issues. Swish! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
|
@codex review Please review final head |
|
Codex Review: Didn't find any major issues. What shall we delve into next? Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
1 participant
Dependency updates failed CI because workflow tests required specific action versions and the image test loaded a private Blume dependency. The tests now check setup order, caching, release safeguards, and Blume's public image audit. The audit checks valid image dimensions and bounds malformed-image processing with a timeout.
The full suite also exposed a Review refresh bug. A raw Snapshot could reset Agent usage after a call finished. Refresh now preserves session usage and the newer Agent notes while still accepting fresh repository metadata. Note timestamps and completion state prevent an older raw Snapshot from replacing work that finished during the refresh. Regression checks force a stale raw refresh to finish after an Agent note, cover remote outages, and check usage reset when a new Review reuses cached Agent notes.
Validation:
pnpm test: all 411 tests pass, including coverage, browser, and platform checks.pnpm run lintandpnpm run docs:check: pass.CI=true corepack pnpm run setupandCI=true corepack pnpm run checkpass.Related dependency PRs: #157, #158, #159, #160.