Repository navigation
Conversation
|
Warning Review limit reachedThe included review limit has been reached and this organization has disabled usage-based review continuation. Wait for reviews to reset or ask a billing admin to change After included review limits.
Reviews can continue after your included limit without a manual trigger. An admin must approve usage-based billing. Next included review available in 56 minutes. View limit details
|
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: 7c0cd0e3c1
ℹ️ 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 The human's November 5 workflow requires exhaustive review of the current complete PR before merge. Please review the full current diff against main, including existing upstream/fork adaptations and the group-cache race correction, generated MEX notice, license preservation and regression tests. Report all remaining actionable correctness findings. Prior review and green checks do not qualify this successor. Extra local validation was waived; hosted CI is required. |
|
@codex security review The human requires current-head security review with no remaining findings before merge. Review the complete PR against main, including protocol/crypto/session boundaries, CAT failure handling, upstream dependency/protobuf changes and cache concurrency. This request does not authorize package publication, deployment or provider effects. |
|
Codex Review: Didn't find any major issues. Already looking forward to the next diff. 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: 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". |
🛡️ Codex Security Review · Automatically triggeredSecurity 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 Security Review · Automatically triggeredSecurity 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
Sync upstream WhatsMeow while retaining HyperMeow's module paths, Signal fork, bounded caches and lifecycle controls. Terminal CAT refresh failures stop automatic reconnect and notify generic and connection-scoped observers.
The continuation fixes the reported group-permission cache race. Group metadata and both participant/admin slices are copied under the cache lock before callers read them. Two regression tests cover snapshot isolation and concurrent notification updates.
Fork-owned notices now include Polymorfa (Softium LLC). Existing personal and upstream attribution, years, license grants and the MPL/MIT boundary are preserved. The existing MEX generator produced its updated notice.
Stack
Library: this PR → newsletter observations #54.
Consumers: runner #871 → newsletter backend and docs #874, with SDK #366 and CLI #92.
Qualify the complete family before merging, then merge bottom-up. Business #55 remains under its existing owner and is not part of this writer's changes.
Verification
No API schema, SDK/CLI surface, Admin control, QuickLink/tunnel, ZK custody, feature default or history profile changes are introduced by this continuation. No live WhatsApp, package publication, deployment or activation is claimed.
Provenance and rollback
One continuing source owner: Codex in T3 Code, OpenAI, gpt-6.1-sol, xhigh, standard service. No new writer or delegate. Registration and original checkpoint. Capture is partial/manual. No UI changed.
Rollback coordinates the library integration and consumer pin, preserving device data. Merge authorization does not authorize tags, packages, deployment or feature activation.