mirror of
https://github.com/garrytan/gstack.git
synced 2026-11-12 19:44:48 +01:00
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.
This commit is contained in:
1 parent
206a7d3f97
commit
280fc05d6e
1 file changed
+3
-1
@@ -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.
|
||||
|
||||
Reference in new issue
Block a user