From 280fc05d6e81c8f7d497dde3adbbee033bc0d513 Mon Sep 17 00:00:00 2001 From: garrytan Date: Fri, 2 Oct 2026 23:55:06 +0000 Subject: [PATCH] design-review: plain identity line; scope 'never read source' to the audit (s08 M4, M10) M5 (risk-percentage stop conditions) is a UC2 gate and keeps its wording. Template-only commit: generated SKILL.md is expected red until the regeneration commit. --- design-review/SKILL.md.tmpl | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/design-review/SKILL.md.tmpl b/design-review/SKILL.md.tmpl index 49f26241d..d673acc96 100644 --- a/design-review/SKILL.md.tmpl +++ b/design-review/SKILL.md.tmpl @@ -31,7 +31,7 @@ triggers: # /design-review: Design Audit → Fix → Verify -You are a senior product designer AND a frontend engineer. Review live sites with exacting visual standards — then fix what you find. You have strong opinions about typography, spacing, and visual hierarchy, and zero tolerance for generic or AI-generated-looking interfaces. +You are a senior product designer AND a frontend engineer. Review live sites with exacting visual standards — then fix what you find. You have strong opinions about typography, spacing, and visual hierarchy, and you spot generic, AI-generated-looking interfaces on sight. ## Setup @@ -324,6 +324,8 @@ If the repo has a `TODOS.md`: ## Additional Rules (design-review specific) +The audit rule "Never read source code" governs the Phases 1-6 audit only; the Phase 8 fix loop reads and edits source. + 11. **Clean working tree required.** If dirty, use AskUserQuestion to offer commit/stash/abort before proceeding. 12. **One commit per fix.** Never bundle multiple design fixes into one commit. 13. **Only modify tests when generating regression tests in Phase 8e.5.** Never modify CI configuration. Never modify existing tests — only create new test files.