fix(workbench): parse UTC timestamps on Python 3.10 - #462
Conversation
|
@codex review |
|
Codex Review: Didn't find any major issues. 🎉 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". |
|
Security review completed. No security issues were found in this pull request. Reviewed commit: Only the user who started this review can view the report in Codex. ℹ️ About Codex security reviews in GitHubThis is an experimental Codex feature. Security reviews are triggered when:
Once complete, Codex will leave suggestions, or a comment if no findings are found. |
|
@codex review Please review the current 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". |
|
Security review completed. No security issues were found in this pull request. Reviewed commit: Only the user who started this review can view the report in Codex. ℹ️ About Codex security reviews in GitHubThis is an experimental Codex feature. Security reviews are triggered when:
Once complete, Codex will leave suggestions, or a comment if no findings are found. |
zcrab-oai
left a comment
There was a problem hiding this comment.
Reviewed Python 3.10 timestamp compatibility, lease handling, and the synchronized bundled-plugin version.
|
@codex review |
|
Codex Review: Didn't find any major issues. Delightful! 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". |
|
Security review completed. No security issues were found in this pull request. Reviewed commit: Only the user who started this review can view the report in Codex. ℹ️ About Codex security reviews in GitHubThis is an experimental Codex feature. Security reviews are triggered when:
Once complete, Codex will leave suggestions, or a comment if no findings are found. |
Summary
Restore Deep Scan deadlines and remediation-claim expiration on supported Python 3.10 installations.
Fixes #35. Fixes #450.
Changes
Testing
ceca62b5: 20 passed with Python 3.10.19 andtomli==2.3.0; the same 20 passed with Python 3.14.5.pnpm run types,pnpm run format, andgit diff --check: passed.check:package, and separate installed-package smoke: passed. The archive contains 222 entries and 106 bundled plugin files; the installed smoke exercised the nested worker without a global Codex installation.21c1a776passed 1,313 tests with 11 skips and zero failures. That result is historical; a new full suite was not run for this unchanged main integration.Risk and rollout
No persisted data is rewritten, and deadline and lease semantics are unchanged. The npm package version is unchanged. Main currently uses plugin 0.1.20; if another plugin-changing PR lands first, merge that main additively, advance both plugin version fields monotonically, and rerun the current-head cache/version and review checks.
Public disclosure review
Newly authored material was reviewed for public disclosure. Existing automated review comments contain access-restricted report references; maintainer cleanup remains outstanding.