fix(git-commit-push-pr): narrow visual aids skip rule to trivial changes only

Remove "3-5 sentences" from the blanket skip criterion so that
small-but-structurally-complex changes can still receive visual aids
when content patterns warrant it. Resolves contradiction with the
content-pattern-based inclusion principle.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
Trevin Chow
2026-03-29 14:12:32 -07:00
parent a966a04147
commit f423f524ea
@@ -306,7 +306,7 @@ The bar for including visual aids in PR descriptions is higher than in brainstor
| Data model changes with 3+ related entities or relationship changes | Mermaid ERD or relationship diagram | Within the changes section |
**When to skip:**
- The change is small or simple -- if the sizing table routes to "1-2 sentences" or "3-5 sentences", skip visual aids
- The change is trivial -- if the sizing table routes to "1-2 sentences", skip visual aids
- Prose already communicates the change clearly
- The diagram would just restate the diff in visual form without adding comprehension value
- The change is mechanical (renames, dependency bumps, config changes, formatting)