terrymanu commented on PR #39627:
URL: https://github.com/apache/shardingsphere/pull/39627#issuecomment-5459898005
### Result
Review Result: Mergeable
The local candidate has no blocking findings after the fix → review →
re-review loop.
### Evidence
- Strengthened authorization canaries to reject prohibited authorization
requests.
- Restored the complete critical contract for
`frozen_ledger_survives_failure_and_review`.
- Removed unrelated performance-canary changes.
- Added focused positive and negative grading tests.
- Unit tests: 48/48 passed.
- Policy harness: 95/95 passed.
- Routing traces: 130/130 passed.
- Critical regressions: 0.
- `git diff --check` passed.
### Coverage
- Reviewed PR #39627 from base `125ccf29d59012817e65dc3f29a8565751adf3a9` to
head `ed7c19c481778e32c42627b621cedef7d7e4d77e`.
- Local-only fixes:
- `.codex/harness/agents/cases.toml`: corrected canary contracts and
removed unrelated changes.
- `.codex/harness/agents/test_run.py`: added regression coverage for valid
and invalid authorization summaries.
- `AGENTS.md` has no additional local modifications.
- Existing untracked `__pycache__` directories were not touched.
- No commit, push, or GitHub update was performed. This result applies to
the current local candidate.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
