Repository navigation
test: enforce current projection presentation isolation contract - #351
Merged
Merged
Conversation
HouMinXi
marked this pull request as ready for review
November 1, 2026 15:58
Qodo reviews are paused for this user.Troubleshooting steps vary by plan Learn more → On a Teams plan? Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center? |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Contract and provenance
Phase 4 f0951ad and DTO serializer fe4c107 legitimately superseded the unwired blanket-import assumption. Mutation-infra pins trace 7d9a476. product_round_history extraction originated ae53306; f1b4a78 restored the leaf definition and e436c7a tightened unavailable-fixpoint validation. Phase 7 relocation 53311c2 retains the leaf body. This clarifies the abbreviated provenance comment without broad golden refresh.
Final integration validation (2026-11-01)
f0d76b10987e10ecef61e4cd0aa68e602e3777e4into this branch after #332 Phase 7 wave 2a: breaker keeps preserve, CI reuse starts empty, audit-unavailable preserve #352, cli: report unsupported assertion advice on resolved outlets #349 and test: correct null baseline deadline configuration oracle #350. Final head:ea35cdf078461a337503f29b7645f103165fb7ea.c1f314c5d1d12bcbfc9241b2b8c598d1a5a6063dexactly matches the prospectively reviewed four-PR composition. This PR's delta against current main remains only the workflow and projection isolation test file. chore(lint): clear retained Ruff debt and gate the full repository #348's full Ruff gate is unchanged.Explicit verification limits
The broader real-CLI
ci_pass_stubcompatibility path was previously denied over possible Semgrep transmission. A later same-path local run completed before that restriction was re-read; it was disclosed, preserved and excluded from qualified acceptance evidence. No further rerun or alternate route is being used. The fresh checks above are pure-local tests/static checks; no broader local full-suite-green claim is made.The repository owner separately authorized merging and cleaning up this four-PR batch after composition checks. This PR description itself is not merge authorization.