diff --git a/.macroscope/check-run-agents/ui-consistency.md b/.macroscope/check-run-agents/ui-consistency.md index c2f2c57c1cf2..7005b21ac587 100644 --- a/.macroscope/check-run-agents/ui-consistency.md +++ b/.macroscope/check-run-agents/ui-consistency.md @@ -1,92 +1,56 @@ --- title: UI Consistency model: claude-opus-5 -effort: high +effort: medium input: full_diff tools: - browse_code - - git_tools - - github_api_read_only - modify_pr include: - - "apps/web/src/**/*.ts" - "apps/web/src/**/*.tsx" - "apps/web/src/**/*.css" +exclude: + - "apps/web/src/**/*.test.tsx" conclusion: failure -showToolCalls: true +maxBudgetPerRun: 10 --- # UI consistency review -Review changed web UI code and directly affected call sites for consistency with the shared component system, Tailwind ownership, and the behavioral constraints below. Apply these rules when a pull request creates, moves, or modifies controls or styling. Do not demand unrelated repository-wide cleanup. +This is a styling guard for `apps/web`, not a general review. Review only the changed lines in the diff and answer three questions. Do not build the project, inspect emitted CSS, trace selector consumers across the codebase, or ask for screenshots. If the diff does not make a violation obvious, there is no finding. -The goal is not to minimize CSS or class counts at any cost. The goal is to put each behavior in the smallest correct owner while preserving interaction, theming, accessibility, layout, and browser behavior. +## 1. Shared primitives over custom controls -## Shared controls and variants +Product UI must use the primitives in `apps/web/src/components/ui` (`Button`, `Input`, `InputGroup`, `Select`, `Toggle`, `Menu`, `Dialog`, `Tooltip`, `ScrollArea`, and the rest of that directory) instead of rebuilding them. -- Prefer the core UI primitives in `apps/web/src/components/ui` over native controls or locally reconstructed primitives. In ordinary product UI, a raw `