From b13eb88ae2de699d5b036bf052d0ab3148880fcc Mon Sep 17 00:00:00 2001 From: Lex Christopherson Date: Fri, 17 Apr 2026 10:23:00 -0600 Subject: [PATCH] fix: load sketch findings into ui-phase researcher The UI researcher creates UI-SPEC.md but wasn't checking for sketch-findings skills. Validated design decisions from /gsd-sketch were being ignored, causing the researcher to re-ask questions already answered during sketching. Co-Authored-By: Claude Opus 4.6 (1M context) --- get-shit-done/workflows/ui-phase.md | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/get-shit-done/workflows/ui-phase.md b/get-shit-done/workflows/ui-phase.md index df1d38175..42c95fbd0 100644 --- a/get-shit-done/workflows/ui-phase.md +++ b/get-shit-done/workflows/ui-phase.md @@ -29,6 +29,11 @@ Parse JSON for: `phase_dir`, `phase_number`, `phase_name`, `phase_slug`, `padded **File paths:** `state_path`, `roadmap_path`, `requirements_path`, `context_path`, `research_path`. +Detect sketch findings: +```bash +SKETCH_FINDINGS_PATH=$(ls ./.claude/skills/sketch-findings-*/SKILL.md 2>/dev/null | head -1) +``` + Resolve UI agent models: ```bash @@ -77,6 +82,13 @@ Note: stack decisions (component library, styling approach) will be asked during ``` Continue (non-blocking). +**If `SKETCH_FINDINGS_PATH` is not empty:** +``` +⚡ Sketch findings detected: {SKETCH_FINDINGS_PATH} + Validated design decisions from /gsd-sketch will be loaded into the UI researcher. + Pre-validated decisions (layout, palette, typography, spacing) should be treated as locked — not re-asked. +``` + ## 4. Check Existing UI-SPEC ```bash @@ -124,6 +136,7 @@ Answer: "What visual and interaction contracts does this phase need?" - {requirements_path} (Requirements) - {context_path} (USER DECISIONS from /gsd-discuss-phase) - {research_path} (Technical Research — stack decisions) +- {SKETCH_FINDINGS_PATH} (Sketch Findings — validated design decisions, CSS patterns, visual direction from /gsd-sketch, if exists) ${AGENT_SKILLS_UI}