feat(F-126): completed feature
This commit is contained in:
49
work/artifacts/F-126/architect.md
Normal file
49
work/artifacts/F-126/architect.md
Normal file
@@ -0,0 +1,49 @@
|
||||
# F-126 — Buscador /inventory por EAN o nombre
|
||||
|
||||
## Diagnóstico
|
||||
|
||||
**Este ticket está resuelto por F-131.** No requiere código nuevo.
|
||||
|
||||
El inventario busca vía `productsApi.list({ q: debouncedSearch })`:
|
||||
1. `apps/admin/src/app/(dashboard)/inventory/page.tsx:103` — pasa `q: debouncedSearch` a la API
|
||||
2. `apps/admin/src/lib/api-client.ts:66` — construye `/api/catalog/products?q=…&limit=100`
|
||||
3. **Antes de F-131**: el proxy catch-all `apps/admin/src/app/api/[...path]/route.ts` descartaba el query string → el backend recibía `/catalog/products?limit=100` (sin `q`) → devolvía todos los productos.
|
||||
4. **Después de F-131**: el proxy concatena `req.nextUrl.search` → el backend recibe `?q=…` → filtra correctamente.
|
||||
|
||||
El backend `src/modules/catalog/infrastructure/pg-product-repository.ts:188` (listAll) ya implementaba ILIKE sobre `name` y `ean` desde antes. La causa raíz del síntoma era exclusivamente el proxy.
|
||||
|
||||
## Verificación
|
||||
|
||||
### Build del admin (con F-131 aplicado)
|
||||
`apps/admin/.next/server/chunks/_1u9yuco._.js` contiene:
|
||||
```js
|
||||
async function i(e) { // DELETE handler
|
||||
return new Response(e.body, { status: e.status, ... })
|
||||
}
|
||||
async function r(e) { // GET handler
|
||||
...
|
||||
const n = e.nextUrl.search;
|
||||
...
|
||||
fetch(`${a}/${r}${n}`, { headers: { Cookie: s } });
|
||||
...
|
||||
}
|
||||
```
|
||||
|
||||
`req.nextUrl.search` está presente en el bundle compilado → F-131 efectivo.
|
||||
|
||||
### Live (admin proceso actual)
|
||||
El admin proceso (PID 97639) está sirviendo el bundle antiguo (construido antes de F-131). El operador debe reiniciar para que el nuevo bundle entre en servicio. Sin ese restart, `/inventory?q=…` sigue mostrando todos los productos aunque el código fuente ya tenga el fix.
|
||||
|
||||
## Plan
|
||||
|
||||
1. **Cerrar F-126 sin código adicional**: el fix ya está commiteado en F-131.
|
||||
2. **Verificación manual del operador tras restart**:
|
||||
- `curl /api/catalog/products?q=almendra` → 1 resultado
|
||||
- UI `/inventory`: escribir "almendra" en el buscador → muestra solo el producto coincidente.
|
||||
3. Si tras el restart la búsqueda sigue rota, abrir ticket nuevo.
|
||||
|
||||
## Notas
|
||||
|
||||
- F-126 fue un duplicado conceptual de F-131. Ambos comparten causa raíz (proxy que perdía query string).
|
||||
- Sin cambios de código en este ticket.
|
||||
- Operador: `./scripts/monolith.sh prod restart`.
|
||||
27
work/artifacts/F-126/implementer.md
Normal file
27
work/artifacts/F-126/implementer.md
Normal file
@@ -0,0 +1,27 @@
|
||||
# F-126 — Resuelto por F-131 (sin código nuevo)
|
||||
|
||||
## Resumen
|
||||
|
||||
**F-126 está resuelto por F-131.** No requiere código adicional.
|
||||
|
||||
## Causa raíz
|
||||
|
||||
Misma que F-131: el proxy catch-all del admin (`apps/admin/src/app/api/[...path]/route.ts`) descartaba `req.nextUrl.search`. La página `/inventory` envía `q=…` correctamente al proxy, pero el proxy reenviaba al backend sin la query string.
|
||||
|
||||
## Verificación
|
||||
|
||||
- Build del admin (post F-131) contiene `req.nextUrl.search` en los 5 handlers del proxy.
|
||||
- Bundle compilado `_1u9yuco._.js` actualizado a `Aug 21 18:13`.
|
||||
- Live admin proceso (PID 97639, uptime 39min) todavía sirve bundle pre-F-131 — el operador debe reiniciar para desplegar.
|
||||
|
||||
## Plan de validación tras restart
|
||||
|
||||
1. Login en `/admin/inventory`
|
||||
2. Escribir "almendra" en el buscador → debe mostrar solo "Almendras Crudas Ecologicas"
|
||||
3. Escribir parte de un EAN (p. ej. "84123") → debe filtrar por coincidencia parcial
|
||||
4. Si no funciona, abrir nuevo ticket
|
||||
|
||||
## Notas
|
||||
|
||||
- Sin cambios de código en este ticket.
|
||||
- F-126 fue un duplicado conceptual de F-131. Ambos cerrados con la misma causa raíz.
|
||||
16
work/artifacts/F-126/leader-close.json
Normal file
16
work/artifacts/F-126/leader-close.json
Normal file
@@ -0,0 +1,16 @@
|
||||
{
|
||||
"verdict": "APPROVED",
|
||||
"agent": "leader",
|
||||
"feature_id": "F-126",
|
||||
"summary": "F-126 cerrado como duplicado de F-131. Sin código nuevo.",
|
||||
"checks": [
|
||||
"reviewer.json APPROVED",
|
||||
"security.json APPROVED",
|
||||
"qa.json APPROVED",
|
||||
"implementer.md completo",
|
||||
"Causa raíz compartida con F-131, ya desplegado en bundle",
|
||||
"0 archivos modificados (resuelto por commit previo)"
|
||||
],
|
||||
"next_step": "operador: ./scripts/monolith.sh prod restart + verificación manual /inventory?q=…",
|
||||
"closed_at": "2026-08-21T16:16:00Z"
|
||||
}
|
||||
20
work/artifacts/F-126/qa.json
Normal file
20
work/artifacts/F-126/qa.json
Normal file
@@ -0,0 +1,20 @@
|
||||
{
|
||||
"verdict": "APPROVED",
|
||||
"reviewer": "qa",
|
||||
"feature_id": "F-126",
|
||||
"summary": "Verificación: bug ya arreglado por F-131, solo falta desplegar.",
|
||||
"checks": [
|
||||
"Pre-F-131: curl /api/catalog/products?q=almendra vía proxy → 13 items (no filtra)",
|
||||
"Directo backend: curl /catalog/products?q=almendra → 1 item (filtra OK)",
|
||||
"Post-F-131 build: bundle contiene nextUrl.search en los 5 handlers",
|
||||
"Live admin proceso todavía sirve bundle pre-F-131 (necesita restart)"
|
||||
],
|
||||
"verification_steps": [
|
||||
"1. Operador ejecuta ./scripts/monolith.sh prod restart",
|
||||
"2. Operador login en /admin/inventory",
|
||||
"3. Operador escribe 'almendra' en el buscador → solo 1 producto",
|
||||
"4. Operador escribe parte de un EAN → filtra correctamente"
|
||||
],
|
||||
"notes": "Tras restart, /inventory debería filtrar correctamente por nombre o EAN.",
|
||||
"reviewed_at": "2026-08-21T16:16:00Z"
|
||||
}
|
||||
15
work/artifacts/F-126/reviewer.json
Normal file
15
work/artifacts/F-126/reviewer.json
Normal file
@@ -0,0 +1,15 @@
|
||||
{
|
||||
"verdict": "APPROVED",
|
||||
"reviewer": "reviewer",
|
||||
"feature_id": "F-126",
|
||||
"summary": "F-126 resuelto por F-131 (mismo bug del proxy). Sin código nuevo.",
|
||||
"checks": [
|
||||
"Causa raíz confirmada: el proxy descartaba req.nextUrl.search (mismo bug que F-131)",
|
||||
"Frontend /inventory envía q correctamente vía productsApi.list({q: debouncedSearch})",
|
||||
"Backend /catalog/products soporta q con ILIKE sobre name y ean",
|
||||
"Build post-F-131 contiene nextUrl.search en el bundle del proxy",
|
||||
"Live admin proceso sirve bundle pre-F-131 — operador debe reiniciar para desplegar"
|
||||
],
|
||||
"notes": "Sin código nuevo. Verificación manual post-restart del operador.",
|
||||
"reviewed_at": "2026-08-21T16:16:00Z"
|
||||
}
|
||||
13
work/artifacts/F-126/security.json
Normal file
13
work/artifacts/F-126/security.json
Normal file
@@ -0,0 +1,13 @@
|
||||
{
|
||||
"verdict": "APPROVED",
|
||||
"reviewer": "security",
|
||||
"feature_id": "F-126",
|
||||
"summary": "Sin cambios. Riesgo nulo.",
|
||||
"checks": [
|
||||
"Sin cambios en endpoints ni autenticación",
|
||||
"El fix del backend (F-131) ya pasó el security gate",
|
||||
"F-126 hereda la aprobación de seguridad de F-131"
|
||||
],
|
||||
"notes": "Sin impacto.",
|
||||
"reviewed_at": "2026-08-21T16:16:00Z"
|
||||
}
|
||||
Reference in New Issue
Block a user