feat(audit): component re-audit + fixes P0–P5 (P4 complete, P5 in progress)
Re-audit of the whole component catalog at pilot depth (91 fichas + the
checkpoint verdicts in docs/audit/components/) and the executed fix packages.
- P0–P3: doctrine (sema participation, CANON §3 intent, component-guide §5 +
API naming, §32 outline), guards (morfo-vocabulary-check S10/S11d,
component-audit A-3.1/A-1.4/S9, eidos-lint classHooks), morfo contracts
(role=application removed ×4, aria-selected off the Day, drp translationRef,
field data-state prune, pin-input commit-set, media-player renames), 13 new
sema packs + 12 morfos family-default → pack.
- P4 (N1–N10) COMPLETE: selectionMode (select/combobox/toggle-group/calendar);
onValueCommit terminal-callback norm (pin-input/search/password/textarea +
date/time/color-field add); typed validation reason + onInvalid (tags-input,
css-field); index/onIndexChange (carousel); deselectable; openDelay/
groupSkipDelay; allowCustomValue; defaultValue prune.
- P5 (in progress): S1 class-hooks codemod (22 hooks → data-attrs across
cropper/button/image-picker/toggle/image-adjustments); S5 outline remnant in
field.css; depth pass (data-depth=overlay on tooltip/toast/float-panel);
touch-rows (--touch-target token, decoupled ::before hit-slop, area-not-visual,
44 AAA). S6 (Field composition) + S8 (calendar-surface) pending.
Verification: errors-outside-(alpha|words|palabras|chronos) == 59 (baseline);
morfo-vocabulary-check exit 0; eidos-lint class-hooks/invalid 0 on touched
components; per-component vitest suites green.
Handoff + remaining plan: docs/process/continue-audit-fixes-2026-07.md
Excluded (broken by the N1 rename, left broken per user decision, not staged):
words/**, palabras/**, chronos, web/routes/alpha/**.
Reconciliation pending: the touch-rows ::before for checkbox/switch reverses
changelog §37's earlier pseudo-element rejection (WCAG-overlap) that had routed
markers to a labeled-row/Field task — flagged for the user in the handoff.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
3 months ago
|
|
|
|
# gradient-builder — ficha de auditoría (RE-AUDITADA 2026-07-07, profundidad piloto)
|
|
|
|
|
|
|
|
|
|
|
|
- **Familia**: pickers/herramientas (iniciativa gradient 2026-06-27) · **Método**: solo análisis · demos fuera
|
|
|
|
|
|
- **Capas leídas**: [morfo](../../../src/uix/morfo/components/gradient-builder.ts) (263 líneas; eventos :26-51+; **sin `expression`**) · pack ✓ existe
|
|
|
|
|
|
- **Máquina — VEREDICTO RECOMPUTADO**: el "BROKEN — único del catálogo" venía de `D-1.2` (demo fuera de plantilla) — **regla de demo, fuera del alcance de esta auditoría** → el estado real de las CAPAS del componente es **NEEDS-WORK** (`F-1.1`–`F-1.4` + `A-1.4` + tests 0), con R-rules de theming LIMPIAS. El componente está más sano de lo que decía su etiqueta.
|
|
|
|
|
|
|
|
|
|
|
|
## Dimensiones (evidencia leída)
|
|
|
|
|
|
|
|
|
|
|
|
| # | Dimensión | Estado | Evidencia |
|
|
|
|
|
|
|---|---|---|---|
|
|
|
|
|
|
| 3 | Coherencia semántica | ✓ con infra-declaración | 4 eventos propios: `handle-pick`/`handle-drag` (:26-38 — familia handle, con color-picker) + `commit-set` neutral (:46-51) + 1 más; pack propio ✓; **F-1**: `expression` AUSENTE teniendo pack — legal para el guard actual (OK-si-pack-existe) pero infra-declarado: debería decir `'pack'` (misma clase que los field-ranges, resuelto con SYS-11 d) |
|
|
|
|
|
|
| 5/10 | Theming | ✓ | R-rules limpias (el CSS está sano) |
|
|
|
|
|
|
| 9 | A11y | gap | Sin APG → con gradient-picker |
|
|
|
|
|
|
| 14 | Docs+tests | gap doble | Expediente pre-flight ausente + cero tests → SYS-9/SYS-2 |
|
|
|
|
|
|
| resto | | ✓ | README ×2 ✓ |
|
|
|
|
|
|
|
|
|
|
|
|
## Hallazgos y propuestas
|
|
|
|
|
|
|
|
|
|
|
|
| ID | Hallazgo | Propuesta |
|
|
|
|
|
|
|---|---|---|
|
|
|
|
|
|
| **F-1** | `expression` sin declarar (pack existe) | Añadir `expression: 'pack'` — SYS-11 (d) |
|
|
|
|
|
|
| **F-2** | Expediente pre-flight ausente | Con gradient-picker (una sesión ambos) |
|
|
|
|
|
|
| **F-3** | Sin APG · cero tests | Declarar patrón + suite — con gradient-picker |
|
|
|
|
|
|
|
|
|
|
|
|
## Escalan al sistema
|
|
|
|
|
|
|
|
|
|
|
|
SYS-2 · SYS-9 (cerrar gradient-* como paquete) · SYS-11 (d) · **nota de proceso**: la etiqueta BROKEN del catálogo era un artefacto de demo — al excluir demos, ningún componente del catálogo está BROKEN a nivel de capas.
|
|
|
|
|
|
|
|
|
|
|
|
## Veredictos
|
|
|
|
|
|
|
|
|
|
|
|
*(pendiente — checkpoint post-barrido)*
|