Non-Blocking Review Concern: PULL_REQUEST_TEMPLATE.md in doc-only allowlist could bypass required checklist changes
Source: claude (self-review bot)
Location: .github/workflows/claude-blocking-review.yml
PR: #47 — feat: doc-only fast-skip + SHA marker + prior-comment collapse (#47)
Date: 2026-04-18
What was flagged
.github/PULL_REQUEST_TEMPLATE.md is treated as doc-only, but PR templates can embed reviewer checklists, required sign-offs, or label instructions. A template change that removes a required security checklist would sail through with no review. Worth discussing whether to treat it like CODEOWNERS (excluded) rather than prose.
Context
This issue was automatically created from a non-blocking concern identified
during pre-merge review of PR #47. It was safe to merge but worth tracking.
Created by lib-review-issues.sh
Non-Blocking Review Concern: PULL_REQUEST_TEMPLATE.md in doc-only allowlist could bypass required checklist changes
Source: claude (self-review bot)
Location:
.github/workflows/claude-blocking-review.ymlPR: #47 — feat: doc-only fast-skip + SHA marker + prior-comment collapse (#47)
Date: 2026-04-18
What was flagged
.github/PULL_REQUEST_TEMPLATE.mdis treated as doc-only, but PR templates can embed reviewer checklists, required sign-offs, or label instructions. A template change that removes a required security checklist would sail through with no review. Worth discussing whether to treat it like CODEOWNERS (excluded) rather than prose.Context
This issue was automatically created from a non-blocking concern identified
during pre-merge review of PR #47. It was safe to merge but worth tracking.
Created by lib-review-issues.sh