Repository navigation
Summary comment format fixes, unsigned-commit row, phase 5 records - #26
Conversation
…im pilot detail Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…unsigned commits row
- Dedupe: the dependency-review job result and step outcome show as one
row (table and record); passes that failed for one reason share a row.
- Repo setting: when dependency review failed and GitHub answers the
dependency comparison with a non-rate-limit 403 (Dependency graph off),
the row, callout, checks strip and record name it a repo setting that
only a repo admin can fix. The author is not given that step.
- Paths show as plain text in code spans (same _code_span rules), and
reason details too, instead of backslash-escaped prose.
- Token usage: one line instead of an all "not recorded" table.
- Informational row: count of PR commits that are not verified-signed,
with GitHub's signing guide link. Read by the gate job from
GET /pulls/{n}/commits; never a gate input, never in the heading count.
Display only: the gate result, reasons, exit code and waiver line do not
change. Unit tests: 2873 -> 2896.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…rix (P5-10), phase 5 exit (P5-11) Spec "Summary comment": one row per cause, repo setting rows (the Dependency graph probe), plain paths in code spans, the one-line token usage, and the informational unsigned-commit row, all display only. Decision log: P5-9 acceptance re-run (8 of 8 pass, unsigned commits block finding), P5-10 bypass matrix and switch rehearsal (pass, stage 4 return test), P5-11 phase 5 exit, P6-1. Build plan: phase 5 items and exit criteria checked; phase 6 release item for the comment format. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…eeds evidence, safe pipes - Unsigned-commit row: the summary is rendered without it first; it is kept only when the body with it differs by the row alone (no new cut, no overflow). Any error or exception in the commit read gives no row. - Repo setting label: a 403 alone is no longer enough. The label needs GitHub's JSON error message to say the Dependency graph is disabled, with no rate limit; otherwise the neutral dependency row stays. The transport keeps an error body up to the byte cap for this. - Code spans in tables: a backslash run before a pipe is split from it with a ZWSP, so each pipe has exactly one escaping backslash. Tests: near-limit boundary (signed vs unsigned, same exit and body), read errors, 14 probe answers that are not evidence, hostile file names under two table-splitting rules. Unit tests 2896 -> 2928. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
🔴 Hold deployment: Claude of Duty identified 1 item requiring attentionMission: Review the code. Protect the branch. Deploy the future. Caution Out of range. This change is too large for the model review. File checks ran, but model checks did not. A maintainer can waive this.
Run 41784305401, attempt 1 for head Claude of Duty never approves a PR. A human reviewer gives the final approval. 🔑 Waiver: how a maintainer clears thisThis PR fails for waivable reasons only. A person with write or admin access, who is not the PR author, posts this exact line as a new comment and replaces A waiver covers the code at this exact head and merge base. Any new push needs a new waiver. 📋 Full review record: reasons, checks, signals, finding details, model summaries, token usageReviewed snapshot
Reasons the gate failed
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
|
Summary
Summary comment format fixes from the pilot, plus phase 5 records.
Test plan
cd review && uv run pytest tests/unit: 2928 passed (was 2873 on main)make release-plan/make releaseafter merge (comment code changed)🤖 Generated with Claude Code