Repository navigation
Conversation
…uisite (P4-1 to P4-6) Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…xt; add open-PR timing lesson (P4 fix round 1) Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
🔴 FAIL: OC Claude review did not completeCaution The Review, Security and Supply chain reviews did not run, so this PR could not be judged. No waiver can clear this.
Run 40905550380, attempt 1 for head 📋 Full review record: reasons, checks, signals, finding details, model summaries, token usageReviewed snapshot
Reasons the gate failed
No waiver line is shown. At least one reason cannot be waived, so a waiver would not change the result. Checks (code, no model)
Signals (context only, never block)
Dependency changes None. Finding details
Blocking is decided by code from category and severity, never by the model. A finding in a blocking category blocks only at high or critical severity. No findings from the passes that ran. The Review, Security, Supply chain pass did not run, so this is not a clean result. Model summaries
Token usage
|
|
Combined into #20. |
Docs only.
repo:OperationCode/<repo>:..., which the federation rule's subject prefix rejects. The first pilot runs failed closed as designed. The spec, the pilot steps and the phase 5 rollout steps now require checking and settinguse_immutable_subjectper repo, after checking the repo's other OIDC users and their trust policies.Unit tests: 2631 passed. Reviewed locally (one fix round); re-review clean, all dates checked against API timestamps.
🤖 Generated with Claude Code