fix(docs): hide automated review signals - #686
Conversation
Co-Authored-By: Warp <agent@warp.dev>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
I'm starting a first review of this pull request. You can view the conversation on Warp. I completed the review and no human review was requested for this pull request. Comment Powered by Oz |
![warp-for-oss[bot]](https://avatars.githubusercontent.com/in/3450139?s=60&v=4){kind=small}
There was a problem hiding this comment.
Overview
This PR hides the automated review signal from GitHub's rendered review body while keeping the machine-readable marker in the raw body for verification and collection. The added regression coverage checks that the heading is removed, the HTML-commented signal is still present, and the existing parser can still recover the reviewer login for the matching PR and head SHA.
Concerns
- No blocking concerns found.
Verdict
Found: 0 critical, 0 important, 0 suggestions
Approve
Comment /warp-agent-review on this pull request to retrigger a review (up to 3 times on the same pull request).
Powered by Oz
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
1 participant
Summary
Hides the machine-readable review signal from GitHub's rendered automated-review body while preserving the exact record required by the review verifier and drafting-improvement collector.
Changes
[SIGNAL:pr-review]record in an HTML comment and removes its visible heading.Unverified claims
None — this change does not alter public product documentation or make product claims.
Documentation risk
Risk: low
Rationale: This internal documentation-review formatter change does not make public product claims.
Docs override: none
Validation
python3 .agents/skills/doc_quality_policy/test_publish_review_signal.py— passed (4 tests).python3 .agents/skills/doc_quality_policy/test_verify_review_signal.py— passed (14 tests).Co-Authored-By: Warp agent@warp.dev