@duro-app/cli 5.16.0 → 5.18.0

package/skills/ui/SKILL.md

@@ -0,0 +1,139 @@
+---
+name: ui
+description: Polishes the surfaces, icons and motion in your project with exact values for border radius, optical alignment, shadows, icon states and animation. Use when adding or tuning a UI animation or transition, such as a menu or dialog enter and exit, staggered list items, hover and press feedback or an icon swap.
+---
+
+# UI polish
+
+This skill holds the visual polish for surfaces, icons and motion, with the exact value each detail takes. It applies once the underlying interaction is sound, and a polish finding never outranks a broken interaction.
+
+A decision of this repository (resolved at its applicable scope with `aval resolve`) outranks a value in this skill. Apply token requirements and gap routing only where the repository's decisions require them, keeping their named exemptions. On `duro-stack`, a non-exempt measure without a token is a design-system gap under ADR-0006/0027, never a local literal.
+
+Where Duro defines the value, these win: the `spacing.css.ts` radii, `shadows.css.ts` and `motion.css.ts`.
+
+The ui values with no token (`300ms` enter, `100ms` stagger, `cubic-bezier(0.2, 0, 0, 1)`) are known design-system gaps tracked in one issue (https://github.com/fredericrous/duro-design-system/issues/120), so a review reports them once against that issue and not per use.
+
+## Exact values, optional polish
+
+The values below are exact, not ranges to approximate. `cubic-bezier(0.2, 0, 0, 1)` is not `cubic-bezier(0.4, 0, 0.2, 1)`, and `0.96` is not `0.95`. The optical nudges and the concentric padding cutoff are the exceptions. They are starting points, judged by eye.
+
+Keep the project's component library, tokens and density. A project token always wins over a value here, and this skill's value applies only where the project has no token for it. Match the project's motion language wherever no rule here gives a value. A deliberate and consistent project convention, such as a style with no shadows, is a preference and not a finding. The same detail done two ways within the project is a finding.
+
+Text wrapping, font rendering, tabular numbers and text spacing belong to `typography`. Hit areas, keyboard support, ARIA and the reduced-motion requirement belong to `accessibility`. Grouping, section spacing, breakpoints and spatial RTL belong to `layout`, except directional icon mirroring. Color tokens and contrast measurement belong to `colors`.
+
+## Outer radius equals inner radius plus padding
+
+Where nested surfaces share a visible, even inset, the outer radius is the inner radius plus the padding plus any border width. Past `24px` of padding, or where the padding is deliberately asymmetric, treat the layers as separate surfaces and keep each one's radius token. Recipes are in [surfaces.md](surfaces.md).
+
+## Align optically where geometry looks off
+
+Where geometric centering looks off, nudge by eye. Give a button `2px` less padding on its icon side, shift a play triangle toward its point and fix asymmetric glyphs in the SVG itself. Recipes are in [surfaces.md](surfaces.md#optical-alignment).
+
+## Shadows for elevation, borders for structure
+
+Where a border exists only to create depth, replace it with layered transparent `box-shadow` values. Keep borders on dividers, separators, table cells and selected states. Keep them on form inputs too, whose boundary needs 3:1 non-text contrast under `accessibility`. Focus rings belong to `accessibility` as well.
+
+Forced-colors mode removes every `box-shadow`. Keep `border: 1px solid transparent` under a shadow ring so that mode still draws an edge. Recipes are in [surfaces.md](surfaces.md#shadow-recipes).
+
+## Outline images in pure black or white
+
+Give content images a `1px` outline inset by `1px`. Use `oklch(0 0 0 / 0.1)` in light mode and `oklch(1 0 0 / 0.1)` in dark. Never use a palette near-black, a tinted neutral or the accent color, because the tint shows as a colored fringe on the image edge. Skip transparent artwork such as logos and illustrations. The recipe is in [surfaces.md](surfaces.md#image-outlines).
+
+## Transitions, not keyframes, for interactive state
+
+Drive interactive state changes with CSS transitions or a motion library's springs, which retarget when the user reverses mid-flight. Keyframes run a fixed timeline and cannot reverse, so reserve them for sequences that run once. See [animations.md](animations.md).
+
+## Press scales to 0.96
+
+A pressed button scales to `0.96` over `150ms` with `ease-out`, and a disabled one never scales. Give the button an opt-out where the motion would distract, through the component's existing variant API or a `static` prop. See [recipes for CSS, Tailwind and Motion](animations.md#scale-on-press).
+
+## High-frequency interactions get no animation
+
+Keystrokes, row hovers and tab switches get instant feedback, or a transition of `150ms` or less on `opacity` or `background-color`. Reserve expressive motion for infrequent moments such as a view's first load, a success state or an empty state.
+
+Every animated state change also leaves a static cue in the form of a color, icon or label. Motion is never the only feedback channel.
+
+## Gate motion behind the reduced-motion preference
+
+`accessibility` owns the requirement, and no recipe here ships without it. Run movement, scale and blur only under `prefers-reduced-motion: no-preference`. Under reduced motion, replace them with an opacity cross-fade rather than removing the element instantly. Recipes for CSS, Tailwind and Motion are in [animations.md](animations.md#reduced-motion-fallback).
+
+## Stagger infrequent entrances by 100ms
+
+Where sequence communicates hierarchy in a staged entrance, split the content into semantic chunks such as title, description and actions. Stagger them `100ms` apart, each entering with opacity, `4px` of blur and `12px` of `translateY` over `300ms` with `ease-out`. Never stagger routine interactions. See [enter-exit.md](enter-exit.md).
+
+## Exits are shorter and smaller than enters
+
+Exit with opacity, `4px` of blur and a fixed `-12px` `translateY` over `150ms` with `ease-out`, never the full container height. Slide fully out only where the destination carries meaning, such as a drawer closing. Remove the element instantly where motion adds no information. See [enter-exit.md](enter-exit.md#exit-animations).
+
+## Skip state animations on first render
+
+Set `initial={false}` on an `AnimatePresence` that wraps a state swap, so the default state does not animate in on mount. Never set it around an intentional entrance such as a staggered hero. See [animations.md](animations.md#skip-animation-on-page-load).
+
+## Cross-fade contextual icons with exact values
+
+Where an icon swaps on an infrequent state change, such as play to pause or copy to copied, cross-fade it. Scale runs `0.25` to `1`, opacity `0` to `1` and blur `4px` to `0px`. With a motion library, use `{ type: "spring", duration: 0.3, bounce: 0 }`, and bounce is always `0`. Without one, keep both icons in the DOM and cross-fade over `300ms` with `cubic-bezier(0.2, 0, 0, 1)`.
+
+A tab's icon swap and the actions a row reveals on hover fall under **High-frequency interactions get no animation**. Both recipes are in [icon-transitions.md](icon-transitions.md).
+
+## Suppress transitions on theme switch
+
+Disable every transition for the theme swap, force a style flush and restore transitions after the next frame. Otherwise every color transition fires at once and the switch smears. See the [recipe](animations.md#suppress-transitions-on-theme-switch).
+
+## Transition only what changes
+
+Name the exact properties, as in `transition-property: scale, opacity`, and never `all`. See [performance.md](performance.md).
+
+## Name the animated property in will-change
+
+Add `will-change` only after you see first-frame stutter, and name the property you animate: `scale` for `scale`, `transform` for `transform`. Limit it to transform properties, `opacity` and `filter`, and never use `all`. See [performance.md](performance.md#use-will-change-sparingly).
+
+## Hover effects only on hover-capable pointers
+
+Put hover-only styling behind `@media (hover: hover)`. On touch, `:hover` latches after a tap and reads as a stuck selected state. Tailwind 4's `hover:` variant already compiles under that query, and Tailwind 3's does not. Where a control draws its own pressed state, set `-webkit-tap-highlight-color: transparent` so the default gray flash does not double it.
+
+## Contain scroll inside overlays
+
+Give scrollable dialogs, drawers, menus and side panels `overscroll-behavior: contain`, so scrolling past their end never scrolls the page behind.
+
+## Match icon stroke to text weight
+
+An icon's rendered stroke tracks the weight of the text beside it, from `1.5px` at 400 to `2.5px` at 700. Use one icon library per surface. The table, sizing and grid rules are in [icons.md](icons.md).
+
+## One SVG, recolored per state
+
+Icons use `currentColor` and take hover, selected and disabled states from CSS color and opacity, never from separate assets. Outline is the default variant, and fill marks the active state. Under `dir="rtl"`, mirror only icons whose meaning follows reading direction. See [icons.md](icons.md#icons-in-rtl).
+
+## Before you finish
+
+| Detection | Fix |
+| --- | --- |
+| Padded parent and child with the same `rounded-*` or `border-radius` | Add the padding to the outer radius |
+| `box-shadow: 0 0 0 1px` ring with no `border` beside it | `border: 1px solid transparent` |
+| `outline-slate-*`, `outline-zinc-*` or a hex outline color on `img` | `outline-black/10` and `dark:outline-white/10` |
+| `transition-all` or `transition: all` | Name the changing properties |
+| `animation:` set on `:hover`, `.open` or another toggled class | A transition on the same properties |
+| `active:scale-95`, `scale-90` or `whileTap` below `0.96` | `0.96` |
+| `:active` scale with no `:disabled` exclusion | `enabled:active:` or `:not(:disabled):active` |
+| `bounce:` set to anything but `0` | `bounce: 0` |
+| `opacity: 0` in a base rule that only an `animation` reveals | Hidden state in the `from` keyframe, `animation-fill-mode: both` |
+| `translate`, `scale`, `filter` or `motion.*` animation with no `prefers-reduced-motion`, `motion-safe:` or `MotionConfig` guard | Gate it and cross-fade opacity under reduced motion |
+| Color transitions plus a theme toggle with no `disableTransitionOnChange` or override | The theme-switch suppression recipe |
+| `will-change: all`, or `will-change` on an element that never animates | Name the animated property or delete it |
+| `fill="#..."` or `stroke="#..."` inside an icon SVG | `currentColor` |
+| Plain-CSS `:hover` or Tailwind 3 `hover:` outside `@media (hover: hover)` | Wrap it in the query |
+| `overflow: auto` or `overflow-y-auto` on a dialog, drawer or menu with no `overscroll-behavior` | `overscroll-behavior: contain` |
+
+## Reporting
+
+**Severity.** `HIGH` breaks an interaction, as a keyframe toggle that cannot reverse or a hover state stuck on touch does. Two of `interface`'s escalation triggers land here and are `HIGH` on sight. One is motion that ignores `prefers-reduced-motion`, and the other is a state change carried by motion alone. `MEDIUM` is a visible inconsistency in surfaces, icons or motion. `LOW` is isolated polish.
+
+**Verification.** Without a browser, read every state the component defines from the code, such as hover, pressed, selected, loading and empty, with its durations and easings. With one, walk each state and replay motion at 10% speed in the browser's Animations panel. Report every check you could not run as `Not verified`.
+
+**Format.** Group findings under the principle each violates, ordered by severity, one row per root cause listing every location it appears in:
+
+| Severity | Location | Before | After | Why |
+| --- | --- | --- | --- | --- |
+
+`Location` is `path/to/file:line`. `Why` names the principle and the user impact.
+
+End with `Block` when any `HIGH` remains, `Approve` otherwise, leaving the rest in the table as work to do. Never `Approve` coverage you did not inspect. With nothing to report, state "No actionable UI-polish findings" and report verification.

package/skills/ui/animations.md

@@ -0,0 +1,170 @@
+# Animations
+
+Recipes for interruptible transitions, press feedback, first-render behavior, theme switches and the reduced-motion fallback. Staged entrances and exits live in [enter-exit.md](enter-exit.md), and icon swaps in [icon-transitions.md](icon-transitions.md).
+
+## Interruptible animations
+
+| | CSS transitions | CSS keyframe animations |
+| --- | --- | --- |
+| **Behavior** | Interpolate toward the latest state | Run on a fixed timeline |
+| **Interrupted** | Retarget mid-flight and reverse | Snap or restart from the beginning |
+| **Use for** | Hover, toggle, open and close | Sequences that run once, such as enters and loading |
+
+```css
+/* Good: clicking again mid-animation reverses smoothly */
+.drawer {
+  transform: translateX(-100%);
+  transition: transform 200ms ease-out;
+}
+.drawer.open {
+  transform: translateX(0);
+}
+
+/* Bad: closing mid-animation snaps */
+.drawer.open {
+  animation: slideIn 200ms ease-out forwards;
+}
+```
+
+## Scale on press
+
+```css
+.button {
+  transition-property: scale;
+  transition-duration: 150ms;
+  transition-timing-function: ease-out;
+}
+
+.button:not(:disabled):active {
+  scale: 0.96;
+}
+```
+
+```tsx
+// Tailwind
+<button className="transition-transform duration-150 ease-out enabled:active:scale-[0.96]">
+  Click me
+</button>
+
+// Motion
+<motion.button whileTap={disabled ? undefined : { scale: 0.96 }} disabled={disabled}>
+  Click me
+</motion.button>
+```
+
+### Static prop pattern
+
+Where the button has no variant API to carry the opt-out, apply the scale class conditionally on a `static` prop:
+
+```tsx
+const tapScale = "enabled:active:scale-[0.96]";
+
+function Button({ static: isStatic, className, children, ...props }) {
+  return (
+    <button
+      className={cn(
+        "transition-transform duration-150 ease-out",
+        !isStatic && tapScale,
+        className,
+      )}
+      {...props}
+    >
+      {children}
+    </button>
+  );
+}
+
+// Usage
+<Button>Click me</Button>           {/* scales on press */}
+<Button static>Submit</Button>       {/* no scale */}
+```
+
+## Skip animation on page load
+
+The icon recipe in [icon-transitions.md](icon-transitions.md#motion) shows `initial={false}` used correctly. It suits icon swaps, toggles, tabs and segmented controls, anything with a default state on page load.
+
+Setting it around a first-time entrance skips that entrance entirely. Check the component on a full page refresh before adding it.
+
+```tsx
+// Bad: initial={false} skips the staggered page enter
+<AnimatePresence initial={false}>
+  <motion.div initial="hidden" animate="visible" variants={...}>
+    ...
+  </motion.div>
+</AnimatePresence>
+```
+
+## Suppress transitions on theme switch
+
+Inject a stylesheet that turns off every transition, force a style flush so the new colors commit while it applies, then remove it after the next frame:
+
+```tsx
+"use client";
+
+import { useEffect } from "react";
+
+export function DisableThemeTransitions() {
+  useEffect(() => {
+    const mql = window.matchMedia("(prefers-color-scheme: dark)");
+
+    const handleChange = () => {
+      const style = document.createElement("style");
+      style.append(
+        document.createTextNode(
+          "*,*::before,*::after{transition:none !important}"
+        )
+      );
+      document.head.append(style);
+
+      void document.body.offsetHeight; // forces a synchronous style flush
+
+      requestAnimationFrame(() => {
+        requestAnimationFrame(() => style.remove());
+      });
+    };
+
+    mql.addEventListener("change", handleChange);
+    return () => mql.removeEventListener("change", handleChange);
+  }, []);
+
+  return null;
+}
+```
+
+The new theme resolves while the override is still in the document, so no transition starts. The nested `requestAnimationFrame` removes the override only after that paint.
+
+That covers the OS-level change. An in-app toggle needs the same steps around its own flip: apply the override, change the theme, flush and remove. `next-themes` ships this as `disableTransitionOnChange`.
+
+## High-frequency motion
+
+```css
+/* Good: high-frequency hover gets a minimal transition */
+.row:hover {
+  background-color: var(--surface-hover);
+  transition: background-color 100ms ease-out;
+}
+
+/* Bad: every hover replays a full entrance */
+.row:hover .row-icon {
+  animation: bounceIn 500ms;
+}
+```
+
+## Reduced-motion fallback
+
+The opacity fade runs for everyone, and movement, scale and blur join it only without the preference:
+
+```css
+.panel {
+  transition: opacity 150ms ease-out;
+}
+
+@media (prefers-reduced-motion: no-preference) {
+  .panel {
+    transition-property: opacity, translate, scale, filter;
+    transition-duration: 300ms;
+  }
+}
+```
+
+In Tailwind, prefix movement, scale and blur utilities with `motion-safe:`. In Motion, wrap the app in `<MotionConfig reducedMotion="user">`. It turns off transform and layout animations under the preference and keeps opacity. Blur still runs there, so branch on `useReducedMotion()` where a recipe animates `filter`.

package/skills/ui/enter-exit.md

@@ -0,0 +1,157 @@
+# Enter and exit animations
+
+Recipes for staged entrances and the exits that follow them. For interactive state feedback see [animations.md](animations.md), and for icon swaps see [icon-transitions.md](icon-transitions.md).
+
+## Enter animations
+
+1. **Split** into semantic groups such as title, description and actions.
+2. **Stagger** the groups `100ms` apart.
+3. **Split a short display headline** into words, staggered `80ms` apart, where it should read word by word.
+4. **Combine** `opacity`, `blur` and `translateY` for each group's enter.
+
+### Motion
+
+```tsx
+const item = {
+  hidden: { opacity: 0, y: 12, filter: "blur(4px)" },
+  visible: {
+    opacity: 1,
+    y: 0,
+    filter: "blur(0px)",
+    transition: { duration: 0.3, ease: "easeOut" },
+  },
+};
+
+function PageHeader() {
+  return (
+    <motion.div
+      initial="hidden"
+      animate="visible"
+      variants={{ visible: { transition: { staggerChildren: 0.1 } } }}
+    >
+      <motion.h1 variants={item}>Welcome</motion.h1>
+      <motion.p variants={item}>A description of the page.</motion.p>
+      <motion.div variants={item}>
+        <Button>Get started</Button>
+      </motion.div>
+    </motion.div>
+  );
+}
+```
+
+### CSS
+
+The hidden state lives in the `from` keyframe, so an item whose animation never runs stays visible. Reduced motion keeps only the opacity keyframe.
+
+```css
+.stagger-item {
+  animation: fade-in 300ms ease-out both;
+}
+
+@media (prefers-reduced-motion: no-preference) {
+  .stagger-item {
+    animation-name: fade-in-up;
+  }
+}
+
+.stagger-item:nth-child(2) { animation-delay: 100ms; }
+.stagger-item:nth-child(3) { animation-delay: 200ms; }
+
+@keyframes fade-in {
+  from { opacity: 0; }
+}
+
+@keyframes fade-in-up {
+  from {
+    opacity: 0;
+    transform: translateY(12px);
+    filter: blur(4px);
+  }
+}
+```
+
+### Entering from display: none
+
+A popover, dialog or element toggled from `display: none` has no previous style to transition from. `@starting-style` supplies one, and `transition-behavior: allow-discrete` keeps it rendered through the exit. The open state's duration applies on enter and the closed state's on exit.
+
+```css
+[popover] {
+  opacity: 0;
+  translate: 0 12px;
+  transition-property: opacity, translate, display, overlay;
+  transition-duration: 150ms;
+  transition-timing-function: ease-out;
+  transition-behavior: allow-discrete;
+}
+
+[popover]:popover-open {
+  opacity: 1;
+  translate: 0 0;
+  transition-duration: 300ms;
+}
+
+@starting-style {
+  [popover]:popover-open {
+    opacity: 0;
+    translate: 0 12px;
+  }
+}
+```
+
+`@starting-style` and discrete `display` transitions are Baseline 2024. `overlay` is Chromium only, and browsers without support show and hide instantly.
+
+## Exit animations
+
+### Subtle exit
+
+```tsx
+<motion.div
+  exit={{
+    opacity: 0,
+    y: -12,
+    filter: "blur(4px)",
+    transition: { duration: 0.15, ease: "easeOut" },
+  }}
+>
+  {content}
+</motion.div>
+```
+
+### Full exit
+
+Slide fully out only where the destination carries meaning, such as a card returning to a list or a drawer closing:
+
+```tsx
+<motion.div
+  exit={{
+    opacity: 0,
+    x: "-100%",
+    transition: { duration: 0.2, ease: "easeOut" },
+  }}
+>
+  {content}
+</motion.div>
+```
+
+### Good and bad
+
+```css
+/* Good: subtle exit */
+.item-exit {
+  opacity: 0;
+  transform: translateY(-12px);
+  transition: opacity 150ms ease-out, transform 150ms ease-out;
+}
+
+/* Bad: dramatic exit that steals focus */
+.item-exit {
+  opacity: 0;
+  transform: translateY(-100%) scale(0.5);
+  transition: all 400ms ease-out;
+}
+
+/* Sometimes correct: remove at once when motion adds no information */
+.item-exit {
+  display: none;
+}
+```

package/skills/ui/icon-transitions.md

@@ -0,0 +1,72 @@
+# Icon transitions
+
+Cross-fade recipes for an icon that swaps on a state change, with and without a motion library. Icon weight, color and direction live in [icons.md](icons.md).
+
+## Which library
+
+Check the project's `package.json`. Import from `"motion/react"` when `motion` is installed, or `"framer-motion"` when that is. Where both exist, follow the imports the component or its nearest peers already use. Where neither is present, use the CSS cross-fade and never add a dependency just for icon transitions.
+
+## Motion
+
+```tsx
+import { AnimatePresence, motion } from "motion/react";
+
+function IconButton({ isActive, icon: Icon }) {
+  return (
+    <button>
+      <AnimatePresence initial={false} mode="popLayout">
+        <motion.span
+          key={isActive ? "active" : "inactive"}
+          initial={{ opacity: 0, scale: 0.25, filter: "blur(4px)" }}
+          animate={{ opacity: 1, scale: 1, filter: "blur(0px)" }}
+          exit={{ opacity: 0, scale: 0.25, filter: "blur(4px)" }}
+          transition={{ type: "spring", duration: 0.3, bounce: 0 }}
+        >
+          <Icon />
+        </motion.span>
+      </AnimatePresence>
+    </button>
+  );
+}
+```
+
+`initial={false}` keeps the default icon from animating in on mount.
+
+## CSS, both icons in the DOM
+
+Both icons stay mounted in one grid cell, so enter and exit both animate and the larger icon sets the size. The `motion-reduce:` classes leave only the opacity fade under reduced motion.
+
+```tsx
+const layer = cn(
+  "[grid-area:1/1] transition-[opacity,filter,scale] duration-300",
+  "ease-[cubic-bezier(0.2,0,0,1)]",
+  "motion-reduce:scale-100 motion-reduce:blur-[0px]"
+);
+const shown = "scale-100 opacity-100 blur-[0px]";
+const hidden = "scale-[0.25] opacity-0 blur-[4px]";
+
+function IconButton({ isActive, ActiveIcon, InactiveIcon }) {
+  return (
+    <button>
+      <span className="grid place-items-center">
+        <span className={cn(layer, isActive ? shown : hidden)}>
+          <ActiveIcon />
+        </span>
+        <span className={cn(layer, isActive ? hidden : shown)}>
+          <InactiveIcon />
+        </span>
+      </span>
+    </button>
+  );
+}
+```
+
+Tailwind 4's `scale-*` utilities write the `scale` property, so `transition-[...scale]` animates them. Tailwind 3's write `transform`, so name `transform` in the transition instead.
+
+## Which icons animate
+
+| Animate | Don't animate |
+| --- | --- |
+| Infrequent state swaps such as play to pause, like to liked or copy to copied | Hover-revealed actions in list rows |
+| Icons in a toolbar that appears on selection | Tab and navigation icon swaps |
+| Loading and success indicators | Decorative or always-visible icons |

package/skills/ui/icons.md

@@ -0,0 +1,105 @@
+# Icons
+
+Icon stroke weight, sizing, state colors, variants and RTL mirroring.
+
+## Match icon stroke to text weight
+
+| Adjacent text | Rendered stroke width |
+| --- | --- |
+| Regular (400), 14–16px | `1.5px` |
+| Medium or semibold (500–600) | `2px` |
+| Bold (700), or an emphasized standalone icon | `2.5px` |
+
+The widths are rendered pixels. A 24px-grid icon drawn at 16px scales its stroke by 16/24, so `stroke-width="2"` paints about `1.33px`. Use the library's absolute-stroke option, such as Lucide's `absoluteStrokeWidth`, or set `vector-effect: non-scaling-stroke` on the SVG's shapes.
+
+```tsx
+// Good: stroke tuned to the semibold label
+<button className="flex items-center gap-2 font-semibold">
+  <Plus size={16} strokeWidth={2} absoluteStrokeWidth />
+  New project
+</button>
+
+// Bad: 1.5px stroke against a bold label
+<button className="flex items-center gap-2 font-bold">
+  <Plus size={16} strokeWidth={1.5} absoluteStrokeWidth />
+  New project
+</button>
+```
+
+- **One library per surface.** Never mix icon libraries on one toolbar. Where the library has no stroke variants, keep its native stroke and use size or color for emphasis.
+- **Size inline icons at `1em`–`1.25em`** of the adjacent font size, so the pair scales together.
+
+## One SVG, recolored per state
+
+Draw every icon with `currentColor` and let CSS state drive the color. Strip hardcoded fills such as `fill="#666"` to `currentColor` when importing icons.
+
+```html
+<svg fill="none" stroke="currentColor" stroke-width="2">…</svg>
+```
+
+```css
+/* Use the project's color tokens */
+.icon-button { color: var(--icon-muted); }
+.icon-button:hover { color: var(--icon-strong); }
+.icon-button[aria-pressed="true"] { color: var(--icon-active); }
+.icon-button:disabled { opacity: 0.4; }
+```
+
+```html
+<!-- Tailwind, with the project's token names -->
+<button class="text-icon-muted hover:text-icon-strong aria-pressed:text-icon-active disabled:opacity-40">
+  <BookmarkIcon />
+</button>
+```
+
+## Outline default, fill active
+
+Where an icon set offers outline and filled variants, use them as a state pair, never interchangeably:
+
+| Variant | Use for |
+| --- | --- |
+| Outline | Default state in toolbars, list rows and inline text |
+| Fill | Selected or active state, such as the active tab, a toggled bookmark or a liked heart |
+
+```tsx
+// Good: variant communicates state
+<TabIcon variant={isActive ? "solid" : "outline"} />
+
+// Bad: filled everywhere, so the active tab has no state signal
+<TabIcon variant="solid" />
+```
+
+Cross-fade the swap with the values in [icon-transitions.md](icon-transitions.md) only where the change is infrequent, such as a bookmark or a like. An active tab swaps instantly.
+
+## Design at render size
+
+- Test every icon at the smallest size it renders, often `16px`. It must stay recognizable there.
+- Prefer a simplified glyph for small contexts over scaling down detailed artwork.
+- Where the set ships per-size grids such as `16`, `20` and `24`, use the grid that matches the render size. A 24px drawing scaled to 16px lands off the pixel grid and renders soft.
+- Always SVG, never raster.
+
+## Icons in RTL
+
+Under `dir="rtl"`, flip icons whose meaning is tied to reading direction, and leave the rest alone:
+
+| Flip | Don't flip |
+| --- | --- |
+| Back and forward arrows, navigation chevrons | Logos and brand marks |
+| Text-block glyphs for alignment, lists and indent | Checkmarks |
+| Speaker and volume waves | Physical objects such as clocks, cups and pencils |
+| Directional glyphs such as "send" | Media playback, which stays LTR by convention |
+
+Use one mechanism per element. Two flips on one icon can compound back to unflipped.
+
+```css
+[dir="rtl"] .icon-directional {
+  scale: -1 1;
+}
+```
+
+```tsx
+// Tailwind
+<ChevronRightIcon className="rtl:-scale-x-100" />
+```
+
+Analyze composite icons part by part. A badge or slash overlay may keep its position while the base glyph flips. Accessible names for icon-only buttons belong to `accessibility`.