From 59dc69ebb3aa2778914b0a1b11b51f04674246fc Mon Sep 17 00:00:00 2001 From: Edgar Durand Date: Mon, 2 Nov 2026 22:14:20 -0300 Subject: [PATCH] docs: attention to detail and good taste is non-negotiable Owner rule 2026-11-02: everything a user sees (and everything the product generates) is designed, never filled in. Reuse the component that already does the job before building a look-alike, and look at the result on phone and desktop with real data before handing it over. Co-Authored-By: Claude Opus 5.5 --- AGENTS.md | 4 ++++ CLAUDE.md | 26 ++++++++++++++++++++++++++ 2 files changed, 30 insertions(+) diff --git a/AGENTS.md b/AGENTS.md index c123c20..399ad50 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -4,6 +4,10 @@ Repository-wide guidance for coding agents. Read this file before changing code. detailed design context lives in `docs/ARCHITECTURE.md`; package-specific API notes live in each package's `README.md`. +## Atención al detalle y buen gusto + +- **Atención al detalle y buen gusto:** todo lo que ve un usuario (y todo lo que genera el producto) se diseña, nunca se rellena. Reutiliza el componente que ya existe antes de crear uno "parecido", y mira el resultado en móvil y escritorio con datos reales antes de entregarlo. Sección completa en `CLAUDE.md` → "Atención al detalle y buen gusto". + ## Project overview This is an npm-workspaces monorepo for `@codeam/ide`, an embeddable VS Code-style IDE component diff --git a/CLAUDE.md b/CLAUDE.md index 0fbe68d..dd32136 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -2,6 +2,30 @@ Guidance for Claude Code (claude.ai/code) when working inside this repository. +## Atención al detalle y buen gusto — no negociable (owner, 2026-11-02) + +**Todo lo que ve un usuario se diseña; nunca se rellena.** Una pantalla fea, tosca o a medio +pensar es un bug aunque funcione, y lo es igual en lo que genera el producto (juegos, apps de +Remix): un resultado que no parece profesional no está terminado. + +- **Reutiliza antes de construir.** Antes de crear una UI, busca el componente que ya hace ese + trabajo (buscador, tarjeta de app, sheet, lista, estados vacío/carga/error) y úsalo. Nunca una + versión casera "parecida": dos componentes que se parecen son una deuda y una incoherencia visual. +- **Míralo antes de entregarlo.** Captura en móvil y escritorio, con datos reales y casos feos: + títulos largos, dos elementos con el mismo nombre, lista vacía, cargando, error, teclado. Un test + verde no significa "se ve bien". Si no pudiste verlo, dilo; no lo des por bueno. +- **Lista mínima:** nada recortado ni solapado; cada elemento identificable (dos apps con el mismo + título se distinguen por su descripción, autor o tipo); la jerarquía tipográfica, los colores, el + radio y la curva de animación de la casa; estados de carga, vacío y error diseñados; foco y + teclado; `prefers-reduced-motion`. +- **La prisa no justifica la fealdad.** Si no hay tiempo para hacerlo bien, dilo y pregunta; no + entregues algo feo para "cerrar" la tarea. + +Ejemplo real (2026-11-02): el "Enter an app you made" del reto semanal se entregó como una lista de +botones con solo el título (título del panel recortado, dos "Pesca Tranquila" idénticas, sin +contexto), cuando el producto ya tenía la búsqueda con tarjetas de app. Se rehízo reutilizando ese +componente (`SearchFrame` + `AppResultCard`, `apps/landing/src/components/remix/SearchFrame.tsx`). + ## Overview This repo is **`@codeam/ide`** — a VS Code-style IDE component library for React Native and React (DOM), designed to be embedded into any app. Three packages publish under one fixed-version line: @@ -211,6 +235,7 @@ This protocol applies when ending a Beads implementation workflow. It is subordi 2. **Run quality gates** (if code changed) - Tests, linters, builds 3. **Update issue status** - Close finished work, update in-progress items 4. **Handle git/sync by active profile**: + ```bash # Conservative/minimal/default: report status and proposed commands; wait for approval. git status @@ -221,6 +246,7 @@ This protocol applies when ending a Beads implementation workflow. It is subordi git push git status ``` + 5. **Hand off** - Summarize changes, validation, issue status, and any blocked sync/commit/push step **Critical rules:**