Non-Blocking Review Concern: Doc-only allowlist misses common non-code meta files
Source: pre-push whole-codebase review
Location: .github/workflows/claude-blocking-review.yml:148-158
Date: 2026-04-18
What was flagged
The doc-only allowlist covers .md/.rst/.txt/LICENSE/CHANGELOG/gitignore/editorconfig/mailmap/ISSUE_TEMPLATE/FUNDING/docs. It does not cover image assets (*.png, *.jpg, *.svg, *.gif) often committed alongside docs, nor docs/**/* non-markdown assets. A PR that only updates a README screenshot would still trigger a paid review. This is conservative-safe (no false-skip risk) but leaves money on the table. Consider adding common image extensions in a follow-up if the 2026-04-17 AAR data warrants it.
Context
This issue was automatically created from a non-blocking concern identified
during pre-push whole-codebase review. It was flagged for tracking.
Created by lib-review-issues.sh
Non-Blocking Review Concern: Doc-only allowlist misses common non-code meta files
Source: pre-push whole-codebase review
Location:
.github/workflows/claude-blocking-review.yml:148-158Date: 2026-04-18
What was flagged
The doc-only allowlist covers .md/.rst/.txt/LICENSE/CHANGELOG/gitignore/editorconfig/mailmap/ISSUE_TEMPLATE/FUNDING/docs. It does not cover image assets (
*.png,*.jpg,*.svg,*.gif) often committed alongside docs, nordocs/**/*non-markdown assets. A PR that only updates a README screenshot would still trigger a paid review. This is conservative-safe (no false-skip risk) but leaves money on the table. Consider adding common image extensions in a follow-up if the 2026-04-17 AAR data warrants it.Context
This issue was automatically created from a non-blocking concern identified
during pre-push whole-codebase review. It was flagged for tracking.
Created by lib-review-issues.sh