Herramientas de usuario

Herramientas del sitio


clase:daw:daw-proyecto:tema05

Diferencias

Muestra las diferencias entre dos versiones de la página.

Enlace a la vista de comparación

Ambos lados, revisión anteriorRevisión previa
Próxima revisión
Revisión previa
clase:daw:daw-proyecto:tema05 [2026/10/14 00:18] – Nuevas secciones de tests unitarios (JUnit) y de API (REST Assured), skills /crear-test-* claudeclase:daw:daw-proyecto:tema05 [2026/10/14 14:08] (actual) – Tema nuevo «2. IA y agentes»: los temas 2 a 7 pasan a ser 3 a 8, con sus referencias cruzadas renumeradas claude
Línea 1: Línea 1:
-====== 5. Automatizando el proyecto con IA ======+====== 5. IA en el proyecto ======
  
-Objetivo del tema: que la calidad la vigilen **tests que genera la IA** en los cuatro niveles (arquitectura, unitarios, API y E2E) y que ningún issue llegue a ''develop'' sin pasar una revisión. Al acabar, cerrar un issue es ''/git-flow finalizar'' y todo lo demás ocurre solo, y un CRUD nuevo es una sola orden.+Objetivo del tema: usar bien al agente para programar. No se enseña a programar; se enseña a **pedir** el trabajo como una especificación y a comprobar que ha salido bien. Al acabar, el proyecto tiene su primer CRUD completo hecho así y la arquitectura del proyecto escrita donde el agente la lee antes de diseñar nada.
  
-===== Tests de arquitectura con ArchUnit =====+Lo que ya está contado en el [[cursos:sdd:start|curso de programación con IA]] y aquí no se repite: qué es el [[cursos:sdd:10-spec-driven-development|Spec-driven development]] y por qué encaja con los agentes, [[cursos:sdd:11-kiro|cc-sdd (Kiro)]] con sus fases y su memoria del proyecto, y la [[cursos:sdd:04-ingenieria#ingenieria_de_contexto|ingeniería de contexto]].
  
-Las reglas de arquitectura (las de Programación y DWS: capas, nombres, sin ciclos, ''domain'' sin Spring) las hace cumplir ArchUnit con tests que fallan cuando una clase está donde no debe. Lo nuevo es que **los escribe la IA** a partir de las reglas en castellano:+===== Dos caminos: spec para los CRUD, plan para el resto ===== 
 + 
 +En este proyecto hay dos clases de issue y cada una tiene su camino: 
 + 
 +  * **Un CRUD** (una entidad con su listado, alta, edición y borrado) va por **especificación**: requisitos, diseño y tareas en ficheros, generados por cc-sdd y revisados por el alumno. Un CRUD tiene una forma conocida y la spec, con el steering, es lo que garantiza que sale igual que los demás. 
 +  * **Todo lo demás** (un filtro, un informe, una regla de negocio, un cambio en una pantalla) va por **plan**: en modo plan se le dice al agente lo que se quiere y él escribe un fichero ''plan_<algo>.md'' con las tareas; el alumno lo revisa y, en otra conversación, se implementa con ''/code impl''. Es trabajo pequeño y variado: una spec entera sería más papel que código. 
 + 
 +En los dos, la regla es la misma: primero un fichero que el alumno lee y corrige, después la implementación con el contexto limpio. 
 + 
 +La spec la genera **cc-sdd** (la versión libre del flujo de Kiro), pero el alumno no escribe sus comandos: los envuelve una skill del proyecto, ''/sdd'', con dos subcomandos. Así el vocabulario es el del tema 1 y no el de la herramienta. cc-sdd se instala una vez en la raíz del repositorio:
  
 <code> <code>
-/crear-test-arquitectura +npx cc-sdd@latest --claude-skills --lang es      (OpenCode: --opencode-skills)
-- Las clases de controller solo dependen de domain. +
-- Las de persistence solo dependen de domain. +
-- Nada en domain importa org.springframework ni jakarta.persistence. +
-- Los controladores acaban en Controller y están en ..controller.. +
-- Los repositorios de persistence acaban en RepositoryImpl e implementan una interfaz de domain. +
-- Sin ciclos entre paquetes.+
 </code> </code>
  
-Cada test lleva en su ''@DisplayName'' y en un comentario **qué comprueba, por qué y las excepciones**:+Ese es el flujo de cada issue:
  
-<sxh java> +<uml> 
-@Test +start 
-@DisplayName("domain no depende de Spring: el dominio tiene que compilar y probarse sin el framework") +:/git-flow start N 
-void domainSinSpring() { +y ./app.sh start; 
-    noClasses().that().resideInAPackage("..domain..") +if (¿es un CRUD?) then (sí) 
-        .should().dependOnClassesThat().resideInAPackage("org.springframework..") +  :/sdd spec create 
-        .check(clases); +  requisitos, diseño y tareas 
-} +  en .kiro/specs/N-slug/; 
-</sxh>+  :Leer y corregir los tres ficheros; 
 +  :/clear y /git-flow update; 
 +  :/sdd spec impl 
 +  una tarea tras otra; 
 +else (no) 
 +  :Modo plan: el agente lee el código 
 +  y escribe plan_<algo>.md; 
 +  :Leer y corregir el plan; 
 +  :/clear y /git-flow update; 
 +  :/code impl plan_<algo>.md 
 +  una tarea tras otra; 
 +endif 
 +:Tests unitarios y de arquitectura 
 +después de cada tarea; 
 +:Probar en el navegador; 
 +stop 
 +</uml>
  
-**No hay un documento aparte con la arquitectura.** Mantener un Markdown sincronizado con los tests es un trabajo que nadie hace, así que la fuente de verdad son los tests con su descripción. Para cambiar una regla se le pide a la skill que cambie el test. Cuando se añade una regla que el código actual aún no cumple, ''FreezingArchRule.freeze(regla)'' congela las violaciones existentes y solo falla con las nuevas.+==== /sdd spec create ====
  
-===== Tests unitarios con JUnit =====+Se lanza en el worktree del issue, sin argumentos: el número del issue está en el nombre de la rama. Lo que hace, por dentro, son las cuatro fases de cc-sdd encadenadas:
  
-Los tests unitarios prueban la lógica del dominio **sin Spring y sin base de datos**: un servicio con sus repositorios sustituidos por mocks de Mockito. Son los más rápidos y los que mejor localizan un error. ''/crear-test-unitarios'' los genera para una clase o un paquete del dominio:+  - ''/kiro-spec-init'' con el título y el cuerpo del issue (''gh issue view N''). 
 +  - ''/kiro-spec-requirements'': los requisitos y los **criterios de aceptación**. 
 +  - ''/kiro-spec-design -y'': el diseño, leyendo antes el //steering// con la arquitectura del proyecto (abajo). 
 +  - ''/kiro-spec-tasks -y'': la lista de tareas, en orden y pequeñas. 
 + 
 +Y se para. El resultado son tres ficheros en ''.kiro/specs/<N>-<slug>/'': ''requirements.md'', ''design.md'' y ''tasks.md''. Antes de generar nada, si el issue no deja claro algo (¿el NIF es único?), pregunta. 
 + 
 +**Revisar esos ficheros es el trabajo humano del flujo.** Es mucho más barato corregir una frase de la spec que corregir el código que sale de ella. Qué mirar en cada uno: 
 + 
 +^ Fichero ^ Qué tiene que tener ^ 
 +| ''requirements.md'' | Los criterios de aceptación en frases **comprobables** («al guardar un cliente sin NIF, el formulario muestra el error y no se crea»), y entre ellos **los casos de error**: campo vacío, valor repetido, id que no existe. En el tema 6 cada frase se convierte en un test | 
 +| ''design.md'' | Que es **igual** que el CRUD de referencia del proyecto: las mismas capas, los mismos nombres, migración y datos de prueba. Si no lo es, lo que falla no es el diseño sino el steering (abajo): se corrige el steering y se vuelve a generar | 
 +| ''tasks.md'' | Tareas pequeñas y en orden: migración y datos, backend, frontend. Cada una se puede hacer y probar sola | 
 + 
 +Si se cambia algo en ''requirements.md'', se vuelve a lanzar ''/sdd spec create'': rehace el diseño y las tareas a partir de los requisitos corregidos. Se repite hasta que los tres ficheros digan lo que se quiere. 
 + 
 +==== /sdd spec impl ==== 
 + 
 +Conversación nueva (''/clear'') y una sola orden:
  
 <code> <code>
-/crear-test-unitarios PedidoService+/sdd spec impl
 </code> </code>
  
-Lo que hace: lee la clase, identifica cada comportamiento (lo que devuelve, lo que valida, lo que lanza) y escribe un test por comportamiento en ''backend/src/test/java'', con el mismo paquete, ''@DisplayName'' en castellano que diga el caso y Mockito para las dependencias. Los ejecuta con ''mvn test'' y se para si alguno falla por un error del código, en vez de retocar el test para que pase.+Por dentro es ''/kiro-impl'' sobre las tareas pendientes de la spec del worktree, una detrás de otra. Después de cada tarea pasa los tests unitarios y de arquitectura (''/test unit run'', tema 6) y la marca como hecha en ''tasks.md''. Si un test falla por un error del código, se para en esa tarea y lo dice, en vez de retocar el test para que pase. Cuando termina, el alumno prueba la aplicación en la URL del worktree y, si algo no está bien, se le pide en la misma conversación.
  
-Qué no prueban: controladores ni repositorios. Eso es de los tests de API.+Solo implementa la spec. Los planes (abajo) y las correcciones de una revisión (tema 6) se hacen con ''/code impl''.
  
-===== Tests de API con REST Assured =====+Por qué el ''/clear'' entre los dos: la conversación de especificar arrastra todo lo que el agente ha leído y descartado mientras pensaba. Ese ruido empeora el resultado y encarece cada petición. Con la spec en ficheros, esa conversación es desechable.
  
-Los tests de API prueban cada endpoint **de verdad**: arrancan la aplicación (''@SpringBootTest'' en un puerto aleatorio, contra la base de datos del worktree con sus datos de prueba) y hacen peticiones HTTP con REST Assured comprobando el código de estado y el JSON. ''/crear-test-api'' los genera para una entidad:+==== El modo plan y /code impl: lo que no es un CRUD ==== 
 + 
 +Claude Code y OpenCode tienen un **modo plan** en el que el agente solo lee: explora el código, piensa y propone, pero no toca nada. En Claude Code se entra con ''Shift+Tab'' hasta que la barra dice //plan mode//; en OpenCode, con la tecla ''Tab'' ([[cursos:sdd:03-opencode#plan_y_buildla_tecla_tab|Plan y Build]]). Es el camino de los issues que no son un CRUD: 
 + 
 +  - En el worktree del issue, modo plan, y se le dice lo que se quiere (lo que pone el issue) y que escriba el plan como **lista de tareas** (''- [ ] ...'') en ''plan_<algo>.md'', por ejemplo ''plan_filtro-provincia.md''. El agente lee el código, pregunta lo que le falte y escribe el fichero: es lo único que escribe en modo plan. 
 +  - Se sale del modo plan. El fichero queda en el worktree. 
 +  - Se lee y se corrige a mano, igual que una spec: tareas pequeñas, en orden, y que no falte ninguna pantalla ni ningún caso de error. 
 +  - **Conversación nueva** (''/clear'') y se implementa:
  
 <code> <code>
-/crear-test-api Pedido+/code impl plan_filtro-provincia.md
 </code> </code>
  
-Por cada endpoint del controlador, un test del caso bueno y uno por cada error que el plan o las validaciones prevén: ''400'' con un campo obligatorio vacío, ''404'' con un id que no existe, ''409'' con un valor único repetido. Los datos de partida son los del ''%%R__%%'' de la entidad; los tests que escriben se limpian a sí mismos (borran lo que crean) para que se puedan pasar dos veces seguidas.+Por qué en dos conversaciones y no seguidas: al planificar, el agente ha leído medio proyecto y ha descartado alternativas, y todo eso sigue en el contexto. Si implementa en la misma conversación, programa con ese ruido. Con el plan en un fichero, la conversación de planificar se tira y la de implementar empieza limpia, solo con el plan. Es la misma razón del ''/clear'' entre ''spec create'' y ''spec impl''.
  
-<sxh java> +''/code impl <plan>'' hace las tareas pendientes del fichero, una detrás de otra, y las va marcando; después de cada una pasa los tests unitarios y de arquitectura y, si uno falla por el código, se para y lo dice. Es lo que hace ''/sdd spec impl'' con ''tasks.md'', pero para cualquier lista de tareas en markdown: en el tema 6 se usa también con la revisión.
-@Test +
-@DisplayName("POST /api/pedidos sin número devuelve 400 y el nombre del campo") +
-void crearSinNumero() { +
-    given().contentType(JSON).body(Map.of("fecha", "2026-10-14", "clienteId", 1)) +
-    .when().post("/api/pedidos") +
-    .then().statusCode(400).body("errores.numero", notNullValue()); +
-} +
-</sxh>+
  
-Unitarios, de API y de arquitectura corren todos con ''mvn test'': es un solo comando para el alumno, para el agente y para CI.+La skill ''/code'' nace en este tema con el verbo ''impl''; en el tema 6 se le añade ''review''.
  
-===== Tests E2E con Playwright =====+==== La carpeta de la spec ====
  
-Los tests E2E prueban la aplicación completa desde el navegador, como lo haría el usuario. Los criterios de aceptación del plan son frases comprobables. ''/crear-test-e2e'' las convierte en tests de Playwright en TypeScript, ''frontend/e2e/<entidad>.spec.ts'', y los deja pasando:+''.kiro/specs/'' se sube a git: la spec viaja con la rama, entra en ''develop'' en el mismo commit que el código, y las skills del tema 6 la usan para generar los tests y para revisar que el código hace lo que se pidió. 
 + 
 +Es un **registro** de lo que se pidió, no documentación: cuando el código cambie después, la spec no se actualiza. Para que el agente no se confunda con specs viejas, ''AGENTS.md'' lleva esta línea:
  
 <code> <code>
-/crear-test-e2e planes/12-alta-clientes.md+.kiro/specs/ es histórico: la fuente de verdad es el código. No leas specs de otros issues salvo que te lo pidan.
 </code> </code>
  
-Cómo trabaja: lee los criterios, abre la aplicación del worktree con el Playwright MCP para ver qué hay en pantalla, escribe el test y lo ejecuta; si falla por un selector o un texto, lo arregla y lo vuelve a pasar. La IA interviene **al generar**, no al ejecutar: lo que queda en el repositorio es un ''.spec.ts'' normal, determinista, que CI ejecuta sin ninguna IA.+==== La skill /sdd ====
  
-Reglas que la skill tiene que respetar (están en ''convenciones-frontend''):+Es corta, porque el trabajo lo hacen los comandos de cc-sdd. Recibe el subcomando en ''$ARGUMENTS'' y hace una cosa u otra:
  
-  * Selectores por ''data-testid'' y por rol (''getByRole'', ''getByLabel''), nunca por clase CSS. Un test que depende del CSS se rompe al primer cambio de estilo. +<sxh yaml;title:.claude/skills/sdd/SKILL.md (cabecera)> 
-  * Un test por criterio de aceptación, con el criterio como nombre del test. +--- 
-  * Los tests cuentan con los datos de prueba del tema 3 y **se pasan sobre la base de datos recién reseteada** (''docker compose down -v && docker compose up -d'').+name: sdd 
 +description: Especificar e implementar el issue del worktree actual con cc-sdd. Subcomandos: "spec create" (requisitos, diseño y tareas en .kiro/specs/N-slug/) y "spec impl" (implementa las tareas pendientes). 
 +allowed-tools: Bash(gh issue view:*), Bash(git branch:*), Bash(mvn:*), Read, Write, Edit, Skill 
 +--- 
 +</sxh>
  
-===== Revisión por IA antes de cerrar =====+Todas las skills de tarea del proyecto son así: ''/<familia> <sustantivo> <verbo>'', con el subcomando en los argumentos. Es lo que hace que ''/test e2e run'' y ''/test e2e create'' sean la misma skill con dos subcomandos.
  
-Sin pull request nadie mira el código antes de que llegue a ''develop''. Lo mira la IA. ''/revisar'' compara el diff de la rama contra ''develop'' con dos cosas:+===== El primer CRUD =====
  
-  * **El plan del issue**: ¿hace lo que se pidió, y como se pidió? ¿Falta algún criterio de aceptación? ¿Ha hecho cosas que no estaban en el plan? +La práctica central del tema es el primer CRUD del proyecto, hecho con ''/sdd''. Pero hay un problema de huevo y gallina: el diseño tiene que salir como la arquitectura del proyecto, y la arquitectura todavía no está escrita en ningún sitio que el agente lea.
-  * **Las skills de convenciones**: ¿está en las capas correctas, con los nombres correctos, con migración y datos de prueba, con ''data-testid''?+
  
-Y clasifica cada cosa que encuentra en **grave** (no cumple el plan, rompe una convención, falta un test) o **mejorable**. ''/git-flow finalizar'' lanza ''/revisar'' como primer paso y **no fusiona si hay algo grave**: enseña la revisión y se para. El alumno corrige (o le pide al agente que corrija) y vuelve a finalizar.+Se resuelve con el primer CRUD y solo con él: se genera la spec, y en ''design.md'' el alumno **corrige a mano** todo lo que no sea la arquitectura que ya conocéis de Programación y DWS (capas ''controller'' / ''domain'' / ''persistence'', ''Repository'' definido en ''domain'' e implementado en ''persistence'', DTO ''XxxRequest'' / ''XxxResponse'', ''XxxJpaEntity'', MapStruct, ''ApiExceptionHandler''). Aquí no se inventa una arquitectura nueva. Después de implementarlo se lee **clase por clase** y se corrige hasta que esté como tiene que estar: es el código de referencia de todo lo que viene después, y de él sale el steering.
  
-<note important>El alumno lee la revisión. El código lo ha escrito la IA, pero el commit lo firma él y es responsable de lo que hay en ''develop''. La revisión es una ayuda para leer, no una excusa para no leer.</note>+===== El steering: las dos arquitecturas =====
  
-===== Un CRUD con una skill =====+Para que el segundo CRUD salga igual que el primero **de verdad**, con las mismas capas, los mismos nombres y los mismos tests, no basta con pedirlo bien: el agente tiene que tener la arquitectura escrita y leerla antes de diseñar. Eso es el //steering// de cc-sdd, y ''/sdd spec create'' lo carga en la fase de diseño. Cómo funciona está en [[cursos:sdd:11-kiro#la_memoria_del_proyecto|la memoria del proyecto]]; aquí solo lo que es de este proyecto: que tiene **dos arquitecturas**, la del cliente y la del servidor, y cómo se le cuentan.
  
-Los CRUD de este proyecto son siempre iguales: solo cambian la entidad y sus campos. Cuando el «cómo» no cambia, no hace falta un plan cada vez; hace falta una skill que tenga el «cómo» escrito:+==== Cuándo se genera ==== 
 + 
 +''/kiro-steering'' describe lo que ve en el código. Por eso se lanza **después del primer CRUD**, ya revisado, y no antes: con el esqueleto vacío describiría nada, y con un CRUD mal hecho convertiría el error en norma. Se lanza una vez, a mano, en la raíz del repositorio:
  
 <code> <code>
-/crud Pedido +/kiro-steering
-Campos: numero (texto, único, obligatorio), fecha (fecha, obligatoria), +
-cliente (relación con Cliente, obligatoria), total (decimal, >= 0).+
 </code> </code>
  
-Y genera lo mismo que tiene el CRUD de referencia: migración ''%%V<n>__pedidos.sql%%'', entidad, repositorio, servicio, controlador y DTO en sus capas, listado y formulario en Angular con sus rutas y validaciones, ''%%R__pedidos.sql%%'' con datos de prueba y los tests unitarios, de API y E2E, llamando a las skills de este tema.+Los tres ficheros que genera, ''product.md'', ''tech.md'' y ''structure.md'', entran en el contexto **siempre**, en todas las fases de todas las specs. Así que se revisan y se dejan cortos, con solo lo común: qué es el proyecto, que es Spring Boot más Angular más MariaDB, el reparto ''backend/'' y ''frontend/'', los comandos de arrancar y probar y las dos ramas. 
 + 
 +==== Dos arquitecturas, dos ficheros ==== 
 + 
 +Lo específico de cada parte no va en esos tres, sino en dos ficheros de steering **condicionales**, que solo entran en el contexto cuando la spec toca esa carpeta. Se crean con ''/kiro-steering-custom'' y llevan en la cabecera el patrón de ficheros que los activa: 
 + 
 +<sxh plain;title:.kiro/steering/backend.md> 
 +--- 
 +inclusion: fileMatch 
 +fileMatchPattern: "backend/**" 
 +--- 
 +# Arquitectura del servidor 
 +... 
 +</sxh> 
 + 
 +Y ''frontend.md'' igual, con ''%%frontend/**%%''. **Concretar las dos arquitecturas es trabajo del alumno**: ''/kiro-steering-custom'' pone la cabecera y un borrador; el contenido se escribe mirando el CRUD de referencia, y tiene que responder a esto: 
 + 
 +^ ''backend.md'' ^ ''frontend.md'' ^ 
 +| Las capas ''controller'' / ''domain'' / ''persistence'' y qué va en cada paquete | Carpeta por entidad y qué componentes tiene (listado y formulario) | 
 +| Nombres: ''XxxController'', ''XxxService'', ''Xxx'' (dominio), ''XxxRepository'' en ''domain'' y ''XxxRepositoryImpl'' en ''persistence'', ''XxxJpaEntity'', ''XxxRequest'' / ''XxxResponse'', el mapper de MapStruct | Nombres de ficheros, de componentes y de rutas (''/clientes'', ''/clientes/nuevo'', ''/clientes/:id'') | 
 +| Dónde se valida (Bean Validation en el ''Request'') y cómo se devuelven los errores: ''ApiExceptionHandler'', códigos 400 / 404 / 409 y el formato del JSON de error | El servicio HTTP de cada entidad y cómo se llama a la API | 
 +| Migraciones ''%%V<n>__%%'' y datos de prueba ''%%R__%%'', con un ejemplo | Formularios reactivos y cómo se muestran las validaciones | 
 +| Cómo se prueban los unitarios: JUnit 5 y Mockito, sin Spring | ''data-testid'' en todo lo que un test toca, nunca selectores por clase CSS | 
 + 
 +La lección: **lo que el steering no dice, el agente lo inventa distinto cada vez.** Si el segundo CRUD sale distinto del primero, falta una línea en el steering, no un prompt mejor. 
 + 
 +==== Las skills de convenciones ==== 
 + 
 +El steering solo lo leen los comandos de cc-sdd. Las demás skills del proyecto (''/test'', ''/code'') también necesitan saber cómo se escribe el código aquí. Para eso hay dos **skills de conocimiento** (no se invocan: el agente las carga solo cuando va a tocar backend o frontend) que no repiten nada: remiten al steering. 
 + 
 +<sxh yaml;title:.claude/skills/convenciones-backend/SKILL.md> 
 +--- 
 +name: convenciones-backend 
 +description: Cómo se escribe el código del backend en este proyecto (capas, nombres, DTO, migraciones, tests). Cargar siempre antes de crear, modificar o revisar cualquier clase Java, test de Java o fichero SQL. 
 +--- 
 +Lee .kiro/steering/backend.md y aplícalo. Es la única fuente de la arquitectura del servidor. 
 +</sxh> 
 + 
 +Y ''convenciones-frontend'' igual, hacia ''frontend.md''. Una sola fuente: cuando cambia una convención, cambia en un sitio. 
 + 
 +===== El contexto del agente ===== 
 + 
 +El contexto es todo lo que el modelo tiene delante en una conversación: ''AGENTS.md'', las skills cargadas, el steering, los ficheros que ha leído, la salida de los comandos que ha ejecutado y todo lo que se ha hablado. Cada mensaje lo reenvía entero. Cuanto más lleno, peor trabaja y más cuesta. Reglas: 
 + 
 +  * **Una conversación por tarea.** ''/clear'' entre tareas. Lo que tenga que sobrevivir, a un fichero. 
 +  * **El modelo bueno para especificar, el barato para implementar** cosas claras. ''/model'' en Claude Code, ''/models'' en OpenCode. 
 +  * **No pegar logs enteros.** Solo el error y las líneas de alrededor. 
 +  * **Mirar el contador.** ''/context'' y ''/cost'' en Claude Code; OpenCode lo muestra junto a la caja de entrada. Cuando el contexto pasa de la mitad, o se cierra la tarea o se hace ''/compact''. 
 + 
 +===== MCP ===== 
 + 
 +Dos servidores MCP y ninguno más (qué es un MCP: [[cursos:sdd:02-skills-agentes-mcp#mcp|MCP]]): 
 + 
 +  * **Context7**: documentación actualizada de Spring Boot y Angular. Sin él, el agente escribe código de la versión que recuerda, que no es la que usáis. 
 +  * **Playwright MCP**: el agente abre la aplicación en un navegador y la ve. Sirve para que compruebe por sí mismo que una pantalla funciona, y en el tema 6 para generar los tests E2E. 
 + 
 +Se configuran con ''claude mcp add'' en Claude Code y en ''opencode.json'' en OpenCode, siguiendo la página de cada servidor. 
 + 
 +===== Permisos del agente ===== 
 + 
 +En el tema 3 se permitió lo básico. Para programar, el agente tiene que poder compilar, pasar tests y levantar la aplicación sin preguntar en cada paso. La lista de ''.claude/settings.json'' queda así:
  
-La skill se construye a partir del CRUD del tema 4: se le pide al agente que lo lea y escriba ''/crud'' con el procedimiento (qué ficheros crea, en qué orden, cómo se nombran) y con ese CRUD como ejemplo de referencia dentro de la skill, en un fichero aparte que carga cuando la ejecuta.+  * **Permitido**: ''mvn'', ''npm'', ''npx'' (incluido ''npx playwright''), ''docker compose'', ''./app.sh'', ''gh issue'', ''gh run'' y las lecturas de git (''status'', ''diff'', ''log'', ''worktree list''). 
 +  * **Pregunta**: escribir ficheros fuera del worktree, ''docker'' a secas, cualquier ''curl'' a un sitio que no sea ''localhost''. 
 +  * **Denegado**: ''git push --force'', ''rm -rf'', y cualquier ''git'' que cambie el repositorio (''commit'', ''merge'', ''push''), que solo puede ejecutar ''/git-flow'' por su ''allowed-tools''.
  
-La lección va más allá del CRUD: **lo que se repite, se convierte en una skill**. Si la tercera vez que se pide algo sale distinto de las dos anteriores, es que faltaba la skill.+En OpenCode, lo mismo en la clave ''permission'' de ''opencode.json''.
  
 ===== Ejercicios ===== ===== Ejercicios =====
  
 ==== Ejercicio 1 ==== ==== Ejercicio 1 ====
-  * Crea un issue «Tests de arquitectura» e inícialo. +  * Configura Context7 y el MCP de Playwright en Claude Code (''claude mcp add'') y en OpenCode (''opencode.json''). 
-  * Escribe la skill ''/crear-test-arquitectura'': recibe reglas en castellano y genera los tests ArchUnit en ''backend/src/test'', cada uno con ''@DisplayName'' y comentario con qué comprueba, por qué y las excepciones. +  * Comprueba que funcionan: pide al agente la forma actual de declarar una ruta en Angular y que abra la aplicación del worktree y te diga qué ve. 
-  * Lánzala con las seis reglas del tema. ''mvn test'' tiene que pasar. +  * Completa ''.claude/settings.json'' con los permisos de este tema y súbelo.
-  * Mueve a mano el servicio del CRUD al paquete ''controller'' y comprueba que ''mvn test'' falla con el mensaje del test. Deshazlo. +
-  * Finaliza el issue.+
  
 ==== Ejercicio 2 ==== ==== Ejercicio 2 ====
-  * Crea un issue «Tests unitarios y de API del CRUD» e inícialo. +  * Crea un issue «Skill /sdd» e inícialo con ''/git-flow start''. 
-  * Añade al ''pom.xml'' Mockito (viene con ''spring-boot-starter-test'') y ''rest-assured''. +  * Instala cc-sdd en la raíz del repositorio y comprueba que ''/kiro-spec-init'' aparece como skill. 
-  * Escribe la skill ''/crear-test-unitarios'': recibe una clase o un paquete del dominio y genera un test JUnit 5 por comportamiento, con Mockito y sin Spring. +  * Escribe la skill ''/sdd'' con los dos subcomandos: ''spec create'' (lee el issue del nombre de la rama, pregunta lo que no esté claro y encadena las cuatro fases de cc-sdd, con ''-y'' en diseño y tareas) y ''spec impl'' (''/kiro-impl'' de las tareas pendientes, tests después de cada una, y parar si un test falla por el código). 
-  * Lánzala con el servicio del CRUD del tema 4. ''mvn test'' tiene que pasar. +  * Añade a ''AGENTS.md'' la línea sobre ''.kiro/specs/''. Finaliza el issue.
-  * Escribe la skill ''/crear-test-api'': recibe una entidad y genera con REST Assured un test por endpoint del caso bueno y uno por cada error previsto, sobre ''@SpringBootTest'' con la base de datos del worktree. +
-  * Lánzala con la entidad del CRUD. ''mvn test'' tiene que pasar dos veces seguidas. +
-  * Introduce un error a mano en una validación del servicio y comprueba qué tests lo detectan: tiene que caer al menos un unitario y uno de API. Deshazlo. +
-  * Finaliza el issue.+
  
 ==== Ejercicio 3 ==== ==== Ejercicio 3 ====
-  * Crea un issue «Tests E2E del CRUD» e inícialo. +  * Crea un issue para la primera entidad de tu proyecto con un CRUD completo (por ejemplo «CRUD de productos») e inícialo. 
-  * Instala Playwright en ''frontend/'' (''npm init playwright@latest'') configurado contra ''%%http://localhost:$FRONTEND_PORT%%''. +  * ''/sdd spec create''. Lee ''requirements.md'': comprueba que los criterios de aceptación son comprobables y que están los casos de error. Corrige lo que falte y vuelve a generar. 
-  * Escribe la skill ''/crear-test-e2e'': recibe un fichero de plan, resetea la base de datos, abre la aplicación con el MCP de Playwright, escribe un ''.spec.ts'' por criterio de aceptación y lo ejecuta hasta que pase. +  * Lee ''design.md'' y corrígelo a mano hasta que sea la arquitectura de Programación y DWS, clase por clase, con migración y datos de prueba. Lee ''tasks.md'' y comprueba que las tareas son pequeñas y van en orden. 
-  * Lánzala con el plan del CRUD del tema 4. ''npx playwright test'' tiene que pasar dos veces seguidas con la base de datos reseteada entre medias. +  * ''/clear'' y ''/sdd spec impl''. Comprueba que marca las tareas hechas y que pasa los tests después de cada una. 
-  * Finaliza el issue.+  * Prueba la aplicación en la URL del worktree: listado, alta, edición, borrado y las validaciones. Lo que no esté bien se pide en la misma conversación. 
 +  * Lee el código generado clase por clase y corrígelo: es la referencia de todos los CRUD que vienen después.
  
 ==== Ejercicio 4 ==== ==== Ejercicio 4 ====
-  * Crea un issue «Revisión por IA» e inícialo. +  * En el mismo worktree, lanza ''/kiro-steering''. Lee los tres ficheros y déjalos cortos, con solo lo común. 
-  * Escribe la skill ''/revisar'': diff de la rama contra ''develop'', comparación con el plan del issue y con las skills de convenciones, salida clasificada en grave / mejorable. +  * Con ''/kiro-steering-custom'' crea ''backend.md'' y ''frontend.md'' condicionales y rellénalos con la tabla del tema, mirando el CRUD que acabas de hacer. 
-  * Amplía ''/git-flow finalizar'' para que lance ''/revisar'' primero y se pare si hay algo grave. +  * Escribe ''convenciones-backend'' y ''convenciones-frontend'' apuntando a ellos. Comprueba que la ''description'' dice cuándo cargarlas. 
-  * Finaliza el issue.+  * ''/clear'' y pide un cambio pequeño en el backend. Comprueba que el agente carga ''convenciones-backend'' sin que se lo digas. 
 +  * ''/git-flow commit'' y ''/git-flow finish''. Comprueba que la spec ha entrado en ''develop'' en el mismo commit que el código. 
 +  * Mira ''/context'' al final de la conversación de implementación y anota cuánto ha ocupado.
  
 ==== Ejercicio 5 ==== ==== Ejercicio 5 ====
-  * Crea un issue para la segunda entidad de tu proyecto e inícialo. +  * Crea un issue «Skill /code: impl» e inícialo. Escribe la skill ''/code'' con el verbo ''impl <plan>'': tareas pendientes del fichero, marcarlas, tests unitarios y de arquitectura tras cada una, y parar si un test falla por el código. Finaliza el issue. 
-  * Escribe la skill ''/crud'' a partir del CRUD del tema 4: procedimiento con los ficheros que crea y en qué orden, y el CRUD de referencia como ejemplo en un fichero aparte dentro de la skill. +  * Crea un issue que no sea un CRUD (un filtro en el listado, un campo calculado, un informe) e inícialo. 
-  * Lánzala: ''/crud <Entidad>'' con sus campos. Comprueba que genera migración, capas, frontend, datos de prueba y los tests unitarios, de API y E2E, y que ''mvn test'' y ''npx playwright test'' pasan. +  * En modo plan, dile lo que quieres y que escriba el plan en ''plan_<algo>.md''. Sal del modo plan y léelo: si falta una tarea o una está mal, corrígela a mano. 
-  * Quita a mano del plan un criterio de aceptación sin quitar su código y lanza ''/git-flow finalizar''. ''/revisar'' tiene que marcarlo como grave y la fusión no debe hacerse. Restaura el plan y finaliza.+  * ''/clear'' y ''/code impl'' con el plan. Comprueba que marca las tareas y pasa los tests tras cada una. Prueba el cambio en el navegador y finaliza el issue.
  
clase/daw/daw-proyecto/tema05.1791929902.txt.gz · Última modificación: por claude