Claude-Code-Workflow team-ui-polish
Unified team skill for UI polish. Auto-discover and fix UI design issues using Impeccable design standards. Anti-AI-slop detection, color/typography/spacing quality, motion, interaction states, visual hierarchy. Uses team-worker agent architecture. Triggers on "team ui polish", "ui polish", "design polish".
git clone https://github.com/catlog22/Claude-Code-Workflow
T=$(mktemp -d) && git clone --depth=1 https://github.com/catlog22/Claude-Code-Workflow "$T" && mkdir -p ~/.claude/skills && cp -r "$T/.codex/skills/team-ui-polish" ~/.claude/skills/catlog22-claude-code-workflow-team-ui-polish-cf9802 && rm -rf "$T"
.codex/skills/team-ui-polish/SKILL.mdTeam UI Polish
Automatic UI quality improvement pipeline: scan -> diagnose -> optimize -> verify. Built on team-worker agent architecture -- all worker roles share a single agent definition with role-specific Phase 2-4 loaded from
roles/<role>/role.md.
Architecture
Skill(skill="team-ui-polish", args="task description") | SKILL.md (this file) = Router | +--------------+--------------+ | | no --role flag --role <name> | | Coordinator Worker roles/coordinator/role.md roles/<name>/role.md | +-- analyze -> dispatch -> spawn workers -> STOP | +-------+-------+-------+ v v v v [team-worker agents, each loads roles/<role>/role.md] scanner diagnostician optimizer <-> verifier (GC loop)
Role Registry
| Role | Path | Prefix | Inner Loop |
|---|---|---|---|
| coordinator | roles/coordinator/role.md | -- | -- |
| scanner | roles/scanner/role.md | SCAN-* | false |
| diagnostician | roles/diagnostician/role.md | DIAG-* | false |
| optimizer | roles/optimizer/role.md | OPT-* | true |
| verifier | roles/verifier/role.md | VERIFY-* | false |
Role Router
Parse
$ARGUMENTS:
- Has
-> Read--role <name>
, execute Phase 2-4roles/<name>/role.md - No
->--role
, execute entry routerroles/coordinator/role.md
Delegation Lock
Coordinator is a PURE ORCHESTRATOR. It coordinates, it does NOT do.
Before calling ANY tool, apply this check:
| Tool Call | Verdict | Reason |
|---|---|---|
, , , , | ALLOWED | Orchestration |
| ALLOWED | Agent health check |
| ALLOWED | User interaction |
| ALLOWED | Message bus |
on files | ALLOWED | Session state |
on , , | ALLOWED | Loading own instructions |
on project source code | BLOCKED | Delegate to worker |
on any file outside | BLOCKED | Delegate to worker |
| BLOCKED | Only workers call CLI |
running build/test/lint commands | BLOCKED | Delegate to worker |
If a tool call is BLOCKED: STOP. Create a task, spawn a worker.
No exceptions for "simple" tasks. Even a single-file read-and-report MUST go through spawn_agent.
Shared Constants
- Session prefix:
UIP - Session path:
.workflow/.team/UIP-<date>-<slug>/ - CLI tools:
(read-only),ccw cli --mode analysis
(modifications)ccw cli --mode write - Message bus:
mcp__ccw-tools__team_msg(session_id=<session-id>, ...) - Max GC rounds: 2
Worker Spawn Template
Coordinator spawns workers using this template:
spawn_agent({ agent_type: "team_worker", task_name: "<task-id>", fork_turns: "none", message: `## Role Assignment role: <role> role_spec: <skill_root>/roles/<role>/role.md session: <session-folder> session_id: <session-id> requirement: <task-description> inner_loop: <true|false> Read role_spec file (<skill_root>/roles/<role>/role.md) to load Phase 2-4 domain instructions. ## Task Context task_id: <task-id> title: <task-title> description: <task-description> pipeline_phase: <pipeline-phase> ## Upstream Context <prev_context>` })
After spawning, use
wait_agent({ timeout_ms: 1800000 }) to collect results. If result.timed_out, send STATUS_CHECK via followup_task (wait 3 min), then FINALIZE with interrupt (wait 3 min), then mark timed_out and close agents. Use close_agent({ target }) each worker.
Model Selection Guide
UI polish is a sequential pipeline where scan findings drive all downstream work. Scanner and verifier need thorough analysis, optimizer needs creative problem-solving.
| Role | reasoning_effort | Rationale |
|---|---|---|
| scanner | high | Deep 8-dimension audit must catch every issue |
| diagnostician | high | Root cause analysis requires careful classification |
| optimizer | high | Creative fix application following design standards |
| verifier | high | Regression detection must be thorough |
Scan-to-Diagnosis Context Flow
Scanner findings must reach diagnostician via coordinator's upstream context:
// After SCAN-001 completes, coordinator sends findings to diagnostician spawn_agent({ agent_type: "team_worker", task_name: "DIAG-001", fork_turns: "none", message: `## Upstream Context Scan report: <session>/scan/scan-report.md` })
User Commands
| Command | Action |
|---|---|
/ | View execution status graph |
/ | Advance to next step |
Specs Reference
- specs/pipelines.md -- Pipeline definitions and task registry
- specs/team-config.json -- Team configuration
- specs/anti-patterns.md -- AI slop and design anti-pattern catalog
- specs/design-standards.md -- Impeccable positive design standards
- specs/fix-strategies.md -- Issue-to-fix mapping
- specs/scoring-guide.md -- Scoring rubric
Session Directory
.workflow/.team/UIP-<date>-<slug>/ +-- .msg/ | +-- messages.jsonl # Team message bus | +-- meta.json # Pipeline config + GC state +-- scan/ # Scanner output | +-- scan-report.md +-- diagnosis/ # Diagnostician output | +-- diagnosis-report.md +-- optimization/ # Optimizer output | +-- fix-log.md +-- verification/ # Verifier output | +-- verify-report.md +-- evidence/ # Screenshots (before/after) | +-- *.png +-- wisdom/ # Cross-task knowledge
v4 Agent Coordination
Message Semantics
| Intent | API | Example |
|---|---|---|
| Queue supplementary info (don't interrupt) | | Send scan findings to running diagnostician |
| Assign fix from verification feedback | | Assign OPT-fix task after verify fails |
| Check running agents | | Verify agent health during resume |
Agent Health Check
Use
list_agents({}) in handleResume and handleComplete:
// Reconcile session state with actual running agents const running = list_agents({}) // Compare with tasks.json active tasks // Reset orphaned tasks (in_progress but agent gone) to pending
Named Agent Targeting
Workers are spawned with
task_name: "<task-id>" enabling direct addressing:
-- send additional scan findings to diagnosticiansend_message({ target: "DIAG-001", message: "..." })
-- assign optimization from diagnosis reportfollowup_task({ target: "OPT-001", message: "..." })
-- cleanup after verificationclose_agent({ target: "VERIFY-001" })
Error Handling
| Scenario | Resolution |
|---|---|
| Unknown command | Error with available command list |
| Role not found | Error with role registry |
| Session corruption | Attempt recovery, fallback to manual |
| Fast-advance conflict | Coordinator reconciles on next callback |
| Completion action fails | Default to Keep Active |
| GC loop stuck > 2 rounds | Escalate to user: accept / retry / terminate |