Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 9 additions & 0 deletions skills/pr-review/SKILL.md
Original file line number Diff line number Diff line change
Expand Up @@ -1204,6 +1204,15 @@ where `[open]` = `<` + `!--` and `[close]` = `--` + `>`.
section. If there are no findings at all, set the body to
the hidden SHA comment followed by a newline and "Looks good to me"
— omit the `## Review` header and `### Findings` section entirely.
- **No freeform verification sections.** Do not include sections
claiming to have verified properties beyond what the diff and source
files directly show (e.g., "Verified: ✅", "zero X remain",
"delivery chain verified"). The review agent performs static analysis
of the diff and source files — it cannot verify reference integrity,
credential flows, or runtime behavior. When a prior finding is no
longer present in the reviewed diff, state "not observed in current
diff" rather than "verified resolved." Never claim exhaustive
verification of any property that requires CI or runtime validation.
- **No footer.** Do not append any footer, action-hints block, or
boilerplate after findings. The post-review pipeline appends
action hints deterministically when appropriate.
Expand Down
8 changes: 8 additions & 0 deletions skills/pr-review/sub-agents/challenger.md
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,14 @@ For each finding:
higher severity and the more specific remediation.
4. **Challenge weak reasoning.** If a finding's description is vague,
speculative, or not supported by the diff, mark it for removal.
5. **Challenge verification claims.** If the aggregated output contains
claims of verification beyond the scope of static diff analysis
(e.g., "all references verified", "delivery chain confirmed",
"zero X remain"), challenge whether the agent actually performed
exhaustive checks to support that claim. The review agent can read
diffs and source files — it cannot verify runtime behavior,
credential flows, or reference integrity across the full codebase.
Remove unsubstantiated verification text.

## Output format

Expand Down
Loading