test: bind restored URL and independence contract - #14
Conversation
The merged restore harness already asserts service URL survival and a collecting-without-retention revision. The TypeScript contract now requires those exact markers so a later edit cannot drop them unnoticed.
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. |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![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: 878d0ce5ed
ℹ️ 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".
Addresses PR #14 P2 review: the restored-URL and collection_without_retention_count markers alone could not detect a coupled seed+query change of revision ...0003 from retention none to applies. The contract now requires the ...0003 none seed tuple, its support_email item seed, and the revision-specific query predicates together. Negative control: old markers pass the coupled mutation, new predicates fail it; all 63 tests pass on the correct script.
|
P2 addressed at d0da193: the contract now binds the ...0003 none seed tuple, its support_email item seed, and the revision-specific query predicates together. Negative control verified locally: old markers pass the coupled none→applies mutation, new predicates fail it; 63/63 tests pass on the correct script. Exact-head CI re-running on d0da193. |
df9eef5
into
codex/policyweave-concurrent-writer-evidence
1 participant
Successor to PR #8 head 5309a61. Binds restored service URL survival and collecting-without-retention cross-state markers in the TypeScript restore contract so later edits cannot drop them unnoticed. Local exact-head gates GREEN: lint, 63 tests, build. CI-only Postgres evidence; no hosted adapter enablement. Merges via ordinary merge commit without force-push.