From f423f524ea77d53b220aba3ca5688dfd02f3a9c5 Mon Sep 17 00:00:00 2001 From: Trevin Chow Date: Sun, 29 Mar 2026 14:12:32 -0700 Subject: [PATCH] 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) --- plugins/compound-engineering/skills/git-commit-push-pr/SKILL.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/plugins/compound-engineering/skills/git-commit-push-pr/SKILL.md b/plugins/compound-engineering/skills/git-commit-push-pr/SKILL.md index 1ed5a769..e8831214 100644 --- a/plugins/compound-engineering/skills/git-commit-push-pr/SKILL.md +++ b/plugins/compound-engineering/skills/git-commit-push-pr/SKILL.md @@ -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)