From c19626bea31fb5b1a69677c6359cdadeef871d57 Mon Sep 17 00:00:00 2001 From: Deploy Date: Wed, 26 Aug 2026 18:07:49 +0200 Subject: [PATCH] fix(inventory-opt): inventory optimization: pagination, queries, filters for 10k+ products --- backlog/features.json | 14 +- .../src/app/(dashboard)/inventory/page.tsx | 571 ++++++++++-------- project/apps/admin/src/lib/api-client.ts | 18 + project/apps/admin/src/types/index.ts | 19 + .../tests/inventory-admin-overview.itest.ts | 186 ++++++ .../modules/inventory/api/inventory.routes.ts | 242 ++++++++ work/artifacts/INVENTORY-OPT/architect.md | 63 ++ work/artifacts/INVENTORY-OPT/implementer.md | 79 +++ .../artifacts/INVENTORY-OPT/leader-close.json | 21 + work/artifacts/INVENTORY-OPT/qa.json | 30 + work/artifacts/INVENTORY-OPT/reviewer.json | 29 + work/artifacts/INVENTORY-OPT/security.json | 19 + work/runtime-status.json | 184 +++--- 13 files changed, 1114 insertions(+), 361 deletions(-) create mode 100644 project/src/app/tests/inventory-admin-overview.itest.ts create mode 100644 work/artifacts/INVENTORY-OPT/architect.md create mode 100644 work/artifacts/INVENTORY-OPT/implementer.md create mode 100644 work/artifacts/INVENTORY-OPT/leader-close.json create mode 100644 work/artifacts/INVENTORY-OPT/qa.json create mode 100644 work/artifacts/INVENTORY-OPT/reviewer.json create mode 100644 work/artifacts/INVENTORY-OPT/security.json diff --git a/backlog/features.json b/backlog/features.json index f32cbc3..2fcb3a1 100644 --- a/backlog/features.json +++ b/backlog/features.json @@ -7824,14 +7824,16 @@ "description": "Need change", "priority": "med", "risk": "low", - "status": "pending", + "status": "done", "created_at": "2026-08-24", "gates": { - "reviewer": false, - "security": false, - "qa": false + "reviewer": true, + "security": true, + "qa": true, + "close": true }, - "phase": "backend" + "phase": "backend", + "completed_at": "2026-08-26T16:07:48Z" }, { "id": "TPV-FIXES", @@ -8006,7 +8008,7 @@ "description": "Phase 1. Add club members, device tokens, recovery-ready identity model, transaction ledger as source of truth, cashback config, basic backend endpoints, migrations and tests.", "priority": "high", "risk": "high", - "status": "in_progress", + "status": "blocked", "created_at": "2026-08-26", "gates": { "reviewer": false, diff --git a/project/apps/admin/src/app/(dashboard)/inventory/page.tsx b/project/apps/admin/src/app/(dashboard)/inventory/page.tsx index ea14313..3286138 100644 --- a/project/apps/admin/src/app/(dashboard)/inventory/page.tsx +++ b/project/apps/admin/src/app/(dashboard)/inventory/page.tsx @@ -1,26 +1,16 @@ 'use client'; -import { useState, useEffect, useCallback } from 'react'; -import { productsApi, inventoryApi, pricingApi } from '@/lib/api-client'; -import type { ProductVariant, StockAvailability, VariantPrice } from '@/types'; -interface ProductRow { - productId: string; - productName: string; - productSlug: string; - productActive: boolean; - variant: ProductVariant; - stock: StockAvailability | null; - price: VariantPrice | null; - expirationDate: string | null; - loading: boolean; +import { useCallback, useEffect, useState } from 'react'; +import { inventoryApi, productsApi } from '@/lib/api-client'; +import type { InventoryOverviewItem } from '@/types'; + +interface ProductRow extends InventoryOverviewItem { editing: boolean; editValue: string; saving: boolean; - msg: string; editingEan: boolean; editEanValue: string; savingEan: boolean; - // F-130: caducidad editable inline editingExp: boolean; editExpValue: string; savingExp: boolean; @@ -37,19 +27,30 @@ const FILTER_LABELS: Record = { low_margin: '📉 Margen <30%', }; -const VAT_MULTIPLIER: Record = { - general: 1.21, - reduced: 1.1, - 'super-reduced': 1.04, -}; +const PAGE_SIZE = 50; function StockBadge({ qty }: { qty: number }) { - if (qty === 0) return Sin stock; - if (qty < 5) return Stock bajo ({qty}); - return En stock ({qty}); + if (qty === 0) { + return ( + + Sin stock + + ); + } + if (qty < 5) { + return ( + + Stock bajo ({qty}) + + ); + } + return ( + + En stock ({qty}) + + ); } -/** Días hasta la fecha de caducidad (negativo si ya caducó). */ function daysUntil(dateIso: string): number { const target = new Date(`${dateIso.slice(0, 10)}T00:00:00`); const today = new Date(); @@ -57,26 +58,27 @@ function daysUntil(dateIso: string): number { return Math.round((target.getTime() - today.getTime()) / 86_400_000); } -/** Margen bruto % = (PVP − coste) / PVP × 100. Null si falta coste o precio. */ -function marginPercent(price: VariantPrice | null): number | null { - if (!price || price.costCents === null) return null; - const grossCents = Math.round(price.netUnitAmountCents * (VAT_MULTIPLIER[price.vatRate] ?? 1.21)); - if (grossCents <= 0) return null; - return Math.round(((grossCents - price.costCents) / grossCents) * 100); -} - function ExpirationCell({ dateIso }: { dateIso: string | null }) { if (!dateIso) return —; const days = daysUntil(dateIso); const label = new Date(`${dateIso.slice(0, 10)}T00:00:00`).toLocaleDateString('es-ES', { - day: '2-digit', month: '2-digit', year: 'numeric', + day: '2-digit', + month: '2-digit', + year: 'numeric', }); if (days < 0) { - return Caducado ({label}); + return ( + + Caducado ({label}) + + ); } if (days <= 30) { return ( - + Caduca en {days} día{days === 1 ? '' : 's'} ); @@ -84,6 +86,21 @@ function ExpirationCell({ dateIso }: { dateIso: string | null }) { return {label}; } +function toRow(item: InventoryOverviewItem): ProductRow { + return { + ...item, + editing: false, + editValue: String(item.stock.availableQuantity), + saving: false, + editingEan: false, + editEanValue: item.variant.ean ?? '', + savingEan: false, + editingExp: false, + editExpValue: item.expirationDate?.slice(0, 10) ?? '', + savingExp: false, + }; +} + export default function InventoryPage() { const [rows, setRows] = useState([]); const [loading, setLoading] = useState(true); @@ -91,103 +108,65 @@ export default function InventoryPage() { const [filter, setFilter] = useState('all'); const [search, setSearch] = useState(''); const [debouncedSearch, setDebouncedSearch] = useState(''); + const [offset, setOffset] = useState(0); + const [total, setTotal] = useState(0); + const [stats, setStats] = useState({ total: 0, inStock: 0, lowStock: 0, outOfStock: 0 }); useEffect(() => { const t = setTimeout(() => setDebouncedSearch(search), 400); return () => clearTimeout(t); }, [search]); - // Load products + single internal variant + stock + price (F-108/F-110) + useEffect(() => { + setOffset(0); + }, [debouncedSearch, filter]); + const load = useCallback(async () => { setLoading(true); setError(''); try { - const { items: products } = await productsApi.list({ - limit: 100, + const result = await inventoryApi.listOverview({ q: debouncedSearch || undefined, + filter, + limit: PAGE_SIZE, + offset, }); - - const productRows: ProductRow[] = []; - - for (const product of products ?? []) { - const { items: variants } = await productsApi.getVariants(product.id); - const variant = variants?.[0]; - if (!variant) continue; - productRows.push({ - productId: product.id, - productName: product.name, - productSlug: product.slug, - productActive: product.state === 'active', - variant, - stock: null, - price: null, - expirationDate: product.expirationDate ?? null, - loading: true, - editing: false, - editValue: '', - saving: false, - msg: '', - editingEan: false, - editEanValue: '', - savingEan: false, - editingExp: false, - editExpValue: product.expirationDate?.slice(0, 10) ?? '', - savingExp: false, - }); - } - - setRows(productRows); - - for (const row of productRows) { - inventoryApi.getAvailability(row.variant.id) - .then((stock) => { - setRows((prev) => - prev.map((r) => - r.variant.id === row.variant.id - ? { ...r, stock, loading: false, editValue: String(stock.availableQuantity) } - : r, - ), - ); - }) - .catch(() => { - setRows((prev) => - prev.map((r) => - r.variant.id === row.variant.id ? { ...r, loading: false, editValue: '0' } : r, - ), - ); - }); - - pricingApi.getVariantPrice(row.variant.id) - .then((price) => { - setRows((prev) => - prev.map((r) => (r.variant.id === row.variant.id ? { ...r, price } : r)), - ); - }) - .catch(() => { - // Sin precio: margen no disponible. - }); - } + setRows(result.items.map(toRow)); + setTotal(result.total); + setStats(result.stats); } catch (e) { setError(e instanceof Error ? e.message : 'Error al cargar inventario'); } finally { setLoading(false); } - }, [debouncedSearch]); + }, [debouncedSearch, filter, offset]); - useEffect(() => { load(); }, [load]); + useEffect(() => { + void load(); + }, [load]); - // Save EAN inline const handleSaveEan = async (variantId: string, productId: string, newEan: string) => { - setRows(prev => prev.map(r => r.variant.id === variantId ? { ...r, savingEan: true } : r)); + setRows((prev) => prev.map((r) => (r.variant.id === variantId ? { ...r, savingEan: true } : r))); try { const updated = await productsApi.updateVariant(productId, variantId, { ean: newEan || null }); - setRows(prev => prev.map(r => r.variant.id === variantId ? { ...r, variant: { ...r.variant, ean: updated.ean }, editingEan: false, savingEan: false } : r)); + setRows((prev) => + prev.map((r) => + r.variant.id === variantId + ? { + ...r, + variant: { ...r.variant, ean: updated.ean }, + editingEan: false, + editEanValue: updated.ean ?? '', + savingEan: false, + } + : r, + ), + ); } catch { - setRows(prev => prev.map(r => r.variant.id === variantId ? { ...r, savingEan: false } : r)); + setRows((prev) => prev.map((r) => (r.variant.id === variantId ? { ...r, savingEan: false } : r))); } }; - // Save expiration date inline (F-130) const handleSaveExp = async (variantId: string, productId: string, value: string) => { setRows((prev) => prev.map((r) => (r.variant.id === variantId ? { ...r, savingExp: true } : r))); try { @@ -195,7 +174,7 @@ export default function InventoryPage() { const newIso = updated.expirationDate ?? null; setRows((prev) => prev.map((r) => - r.variant.id === variantId + r.productId === productId ? { ...r, expirationDate: newIso, @@ -211,14 +190,13 @@ export default function InventoryPage() { } }; - // Save Stock inline (used by Stock cell on blur/Enter) const saveStockInline = async (variantId: string, value: string) => { const qty = parseInt(value, 10); if (isNaN(qty) || qty < 0) { setRows((prev) => prev.map((r) => r.variant.id === variantId - ? { ...r, editing: false, editValue: String(r.stock?.availableQuantity ?? 0), msg: 'Error' } + ? { ...r, editing: false, editValue: String(r.stock.availableQuantity) } : r, ), ); @@ -236,14 +214,11 @@ export default function InventoryPage() { editing: false, saving: false, editValue: String(result.available), - msg: '✓ Guardado', } : r, ), ); - setTimeout(() => { - setRows((prev) => prev.map((r) => (r.variant.id === variantId ? { ...r, msg: '' } : r))); - }, 3000); + await load(); } catch { setRows((prev) => prev.map((r) => @@ -252,8 +227,7 @@ export default function InventoryPage() { ...r, saving: false, editing: false, - editValue: String(r.stock?.availableQuantity ?? 0), - msg: 'Error', + editValue: String(r.stock.availableQuantity), } : r, ), @@ -261,57 +235,37 @@ export default function InventoryPage() { } }; - // Filter + sort rows - const filtered = (() => { - let result = rows.filter((r) => { - const qty = r.stock?.availableQuantity ?? 0; - if (filter === 'in_stock') return qty >= 5; - if (filter === 'low_stock') return qty > 0 && qty < 5; - if (filter === 'out_of_stock') return qty === 0; - if (filter === 'expiring') { - return r.expirationDate !== null && daysUntil(r.expirationDate) <= 30; - } - if (filter === 'low_margin') { - const margin = marginPercent(r.price); - return margin !== null && margin < 30; - } - return true; - }); - if (filter === 'expiring') { - result = [...result].sort( - (a, b) => daysUntil(a.expirationDate ?? '') - daysUntil(b.expirationDate ?? ''), - ); - } else if (filter === 'low_margin') { - result = [...result].sort( - (a, b) => (marginPercent(a.price) ?? 0) - (marginPercent(b.price) ?? 0), - ); - } - return result; - })(); - - const inStockCount = rows.filter((r) => (r.stock?.availableQuantity ?? 0) >= 5).length; - const lowStockCount = rows.filter((r) => { - const q = r.stock?.availableQuantity ?? 0; - return q > 0 && q < 5; - }).length; - const outOfStockCount = rows.filter((r) => (r.stock?.availableQuantity ?? 0) === 0).length; + const page = Math.floor(offset / PAGE_SIZE) + 1; + const totalPages = Math.max(1, Math.ceil(total / PAGE_SIZE)); + const from = total === 0 ? 0 : offset + 1; + const to = Math.min(offset + PAGE_SIZE, total); return (
- {/* Header */}

Inventario

-

{rows.length} productos

+

{total} variantes

- {/* Stats */}
{[ - { label: 'En stock', count: inStockCount, cls: 'bg-green-50 border-green-100 text-green-700' }, - { label: 'Stock bajo', count: lowStockCount, cls: 'bg-amber-50 border-amber-100 text-amber-700' }, - { label: 'Sin stock', count: outOfStockCount, cls: 'bg-red-50 border-red-100 text-red-700' }, + { + label: 'En stock', + count: stats.inStock, + cls: 'bg-green-50 border-green-100 text-green-700', + }, + { + label: 'Stock bajo', + count: stats.lowStock, + cls: 'bg-amber-50 border-amber-100 text-amber-700', + }, + { + label: 'Sin stock', + count: stats.outOfStock, + cls: 'bg-red-50 border-red-100 text-red-700', + }, ].map(({ label, count, cls }) => (

{count}

@@ -320,48 +274,50 @@ export default function InventoryPage() { ))}
- {/* Search + filters */}
setSearch(e.target.value)} className="w-full pl-10 pr-4 py-2.5 border border-gray-300 rounded-xl text-sm focus:ring-2 focus:ring-[#2D6A4F] focus:border-transparent outline-none" /> - - + + +
- {(Object.keys(FILTER_LABELS) as RowFilter[]).map((f) => ( + {(Object.keys(FILTER_LABELS) as RowFilter[]).map((currentFilter) => ( ))}
-
- {/* Table */}
{loading ? (
@@ -369,58 +325,104 @@ export default function InventoryPage() { Cargando inventario...
) : error ? ( -
+

{error}

- +
- ) : filtered.length === 0 ? ( + ) : rows.length === 0 ? (

📦

No hay productos para este filtro

) : ( -
- - - - - - - - - - - - - - {filtered.map((row) => { - const margin = marginPercent(row.price); - return ( + <> +
+
ProductoEANCaducidadMargenStockEstadoTienda
+ + + + + + + + + + + + + {rows.map((row) => ( - ); - })} - -
+ Producto + + SKU / EAN + + Caducidad + + Margen + + Stock + + Estado + + Tienda +

{row.productName}

+

{row.variant.sku}

 - {row.editingEan ? ( - setRows(prev => prev.map(r => r.variant.id === row.variant.id ? { ...r, editEanValue: e.target.value } : r))} - onBlur={() => handleSaveEan(row.variant.id, row.productId, row.editEanValue)} - onKeyDown={e => { if (e.key === 'Enter') handleSaveEan(row.variant.id, row.productId, row.editEanValue); if (e.key === 'Escape') setRows(prev => prev.map(r => r.variant.id === row.variant.id ? { ...r, editingEan: false } : r)); }} - disabled={row.savingEan} - placeholder="—" - className="w-full px-2 py-1 border border-[#2D6A4F] rounded text-xs focus:ring-1 focus:ring-[#2D6A4F] outline-none disabled:opacity-50" - /> - ) : ( - - )} +
+

SKU: {row.variant.sku}

+ {row.editingEan ? ( + + setRows((prev) => + prev.map((r) => + r.variant.id === row.variant.id + ? { ...r, editEanValue: e.target.value } + : r, + ), + ) + } + onBlur={() => void handleSaveEan(row.variant.id, row.productId, row.editEanValue)} + onKeyDown={(e) => { + if (e.key === 'Enter') { + void handleSaveEan(row.variant.id, row.productId, row.editEanValue); + } + if (e.key === 'Escape') { + setRows((prev) => + prev.map((r) => + r.variant.id === row.variant.id ? { ...r, editingEan: false } : r, + ), + ); + } + }} + disabled={row.savingEan} + placeholder="EAN" + className="w-full px-2 py-1 border border-[#2D6A4F] rounded text-xs focus:ring-1 focus:ring-[#2D6A4F] outline-none disabled:opacity-50" + /> + ) : ( + + )} +
{row.editingExp ? ( @@ -437,15 +439,18 @@ export default function InventoryPage() { ), ) } - onBlur={() => handleSaveExp(row.variant.id, row.productId, row.editExpValue)} + onBlur={() => void handleSaveExp(row.variant.id, row.productId, row.editExpValue)} onKeyDown={(e) => { - if (e.key === 'Enter') handleSaveExp(row.variant.id, row.productId, row.editExpValue); - if (e.key === 'Escape') + if (e.key === 'Enter') { + void handleSaveExp(row.variant.id, row.productId, row.editExpValue); + } + if (e.key === 'Escape') { setRows((prev) => prev.map((r) => r.variant.id === row.variant.id ? { ...r, editingExp: false } : r, ), ); + } }} disabled={row.savingExp} className="px-2 py-1 border border-[#2D6A4F] rounded text-xs focus:ring-1 focus:ring-[#2D6A4F] outline-none disabled:opacity-50" @@ -467,78 +472,80 @@ export default function InventoryPage() { )} - {margin === null ? ( + {row.marginPercent === null ? ( — ) : ( - = 30 ? 'text-green-600' : margin >= 10 ? 'text-amber-600' : 'text-red-600'}`}> - {margin}% + = 30 + ? 'text-green-600' + : row.marginPercent >= 10 + ? 'text-amber-600' + : 'text-red-600' + }`} + > + {row.marginPercent}% )} - {row.loading ? ( - — - ) : row.editing ? ( -
- + {row.editing ? ( + + setRows((prev) => + prev.map((r) => + r.variant.id === row.variant.id + ? { ...r, editValue: e.target.value } + : r, + ), + ) + } + onBlur={() => void saveStockInline(row.variant.id, row.editValue)} + onKeyDown={(e) => { + if (e.key === 'Enter') { + void saveStockInline(row.variant.id, row.editValue); + } + if (e.key === 'Escape') { setRows((prev) => prev.map((r) => r.variant.id === row.variant.id - ? { ...r, editValue: e.target.value } + ? { + ...r, + editing: false, + editValue: String(r.stock.availableQuantity), + } : r, ), - ) + ); } - onBlur={() => saveStockInline(row.variant.id, row.editValue)} - onKeyDown={(e) => { - if (e.key === 'Enter') saveStockInline(row.variant.id, row.editValue); - if (e.key === 'Escape') - setRows((prev) => - prev.map((r) => - r.variant.id === row.variant.id - ? { - ...r, - editing: false, - editValue: String(r.stock?.availableQuantity ?? 0), - } - : r, - ), - ); - }} - disabled={row.saving} - className="w-20 px-2 py-1 border border-gray-300 rounded-lg text-sm focus:ring-1 focus:ring-[#2D6A4F] outline-none" - /> - {row.msg && ( - - {row.msg} - - )} -
+ }} + disabled={row.saving} + className="w-20 px-2 py-1 border border-gray-300 rounded-lg text-sm focus:ring-1 focus:ring-[#2D6A4F] outline-none" + /> ) : ( )} 		- + {row.productActive ? ( @@ -550,8 +557,19 @@ export default function InventoryPage() { title="Ver en la tienda" className="inline-flex h-8 w-8 items-center justify-center rounded-lg bg-green-50 text-green-600 transition-colors hover:bg-green-100 focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-[#2D6A4F]" > -
-
+ ))} + + +
+ +
+

+ Mostrando {from}-{to} de {total} +

+
+ + + Página {page} / {totalPages} + + +
+
+ )}
diff --git a/project/apps/admin/src/lib/api-client.ts b/project/apps/admin/src/lib/api-client.ts index 5cdcfe5..1b5e8f6 100644 --- a/project/apps/admin/src/lib/api-client.ts +++ b/project/apps/admin/src/lib/api-client.ts @@ -202,6 +202,24 @@ export const categoriesApi = { // ── Inventory ───────────────────────────────────────────────────────────────── export const inventoryApi = { + listOverview: (params?: { + q?: string; + filter?: 'all' | 'in_stock' | 'low_stock' | 'out_of_stock' | 'expiring' | 'low_margin'; + limit?: number; + offset?: number; + }) => { + const sp = new URLSearchParams(); + if (params?.q) sp.set('q', params.q); + if (params?.filter) sp.set('filter', params.filter); + if (params?.limit) sp.set('limit', String(params.limit)); + if (params?.offset) sp.set('offset', String(params.offset)); + const qs = sp.toString(); + return api.get<{ + items: import('@/types').InventoryOverviewItem[]; + total: number; + stats: import('@/types').InventoryOverviewStats; + }>(`/api/inventory/admin/overview${qs ? `?${qs}` : ''}`); + }, getAvailability: (variantId: string) => api.get(`/api/inventory/${variantId}/availability`), setStock: (id: string, quantity: number) => diff --git a/project/apps/admin/src/types/index.ts b/project/apps/admin/src/types/index.ts index 7d21d25..6b5804d 100644 --- a/project/apps/admin/src/types/index.ts +++ b/project/apps/admin/src/types/index.ts @@ -72,6 +72,25 @@ export interface StockItem { updatedAt: string; } +export interface InventoryOverviewItem { + productId: string; + productName: string; + productSlug: string; + productActive: boolean; + expirationDate: string | null; + variant: ProductVariant; + stock: StockAvailability; + price: VariantPrice | null; + marginPercent: number | null; +} + +export interface InventoryOverviewStats { + total: number; + inStock: number; + lowStock: number; + outOfStock: number; +} + // ── Orders ──────────────────────────────────────────────────────────────────── export type OrderState = diff --git a/project/src/app/tests/inventory-admin-overview.itest.ts b/project/src/app/tests/inventory-admin-overview.itest.ts new file mode 100644 index 0000000..f70abff --- /dev/null +++ b/project/src/app/tests/inventory-admin-overview.itest.ts @@ -0,0 +1,186 @@ +import { randomUUID } from 'node:crypto'; +import type { DestinationStream } from 'pino'; +import type pg from 'pg'; +import { afterAll, beforeAll, describe, expect, it } from 'vitest'; +import { buildApp } from '../build-app.js'; +import { createPool } from '../../infrastructure/db/pool.js'; +import { createLogger } from '../../infrastructure/logging/logger.js'; +import { + getTestDbUrl, + recreateDatabase, + runMigrations, +} from '../../infrastructure/db/tests/db-test-support.js'; +import { SESSION_COOKIE_NAME } from '../../modules/identity/index.js'; + +const hasDb = Boolean(process.env.TEST_DATABASE_URL); + +function silentLogger() { + const destination: DestinationStream = { write: () => undefined }; + return createLogger({ level: 'info', destination }); +} + +function cookieValue(setCookieHeader: string | string[] | undefined): string { + const raw = Array.isArray(setCookieHeader) ? setCookieHeader[0] : setCookieHeader; + expect(raw).toBeDefined(); + const pair = (raw as string).split(';')[0] as string; + return pair.slice(pair.indexOf('=') + 1); +} + +describe.skipIf(!hasDb)('inventory admin overview (real PostgreSQL)', () => { + const url = hasDb ? getTestDbUrl() : ''; + let pool: pg.Pool; + let app: Awaited>; + let adminCookie = ''; + + beforeAll(async () => { + await recreateDatabase(url); + await runMigrations(url, 'up'); + pool = createPool(url); + app = await buildApp({ logger: silentLogger(), pool, cookieSecure: true }); + + const user = { email: 'inventory-admin@example.com', password: 'correct horse battery staple' }; + const registered = await app.inject({ + method: 'POST', + url: '/auth/register', + headers: { 'content-type': 'application/json' }, + payload: user, + }); + const id = (registered.json() as { id: string }).id; + await pool.query( + `UPDATE identity_users + SET role = $1, + email_confirmed = true, + confirmed_at = now(), + confirmation_token = null + WHERE id = $2`, + ['admin', id], + ); + const login = await app.inject({ + method: 'POST', + url: '/auth/login', + headers: { 'content-type': 'application/json' }, + payload: user, + }); + adminCookie = cookieValue(login.headers['set-cookie']); + }); + + afterAll(async () => { + await app?.close(); + await pool?.end(); + }); + + it('returns paginated inventory rows with server-side search, filters and stats', async () => { + const createProduct = async (name: string, slug: string, expirationDate?: string | null) => { + const product = await app.inject({ + method: 'POST', + url: '/products', + headers: { 'content-type': 'application/json' }, + cookies: { [SESSION_COOKIE_NAME]: adminCookie }, + payload: { name, slug, state: 'active', expirationDate: expirationDate ?? null }, + }); + expect(product.statusCode).toBe(201); + return product.json() as { id: string }; + }; + + const productA = await createProduct('Avena bio', `avena-${randomUUID().slice(0, 8)}`); + const productB = await createProduct( + 'Semillas lino', + `lino-${randomUUID().slice(0, 8)}`, + new Date(Date.now() + 10 * 86_400_000).toISOString().slice(0, 10), + ); + + const createVariant = async (productId: string, sku: string, ean: string) => { + const response = await app.inject({ + method: 'POST', + url: `/products/${productId}/variants`, + headers: { 'content-type': 'application/json' }, + cookies: { [SESSION_COOKIE_NAME]: adminCookie }, + payload: { sku, ean }, + }); + expect(response.statusCode).toBe(201); + return response.json() as { id: string }; + }; + + const variantA = await createVariant(productA.id, `SKU-${randomUUID().slice(0, 8)}`, '111'); + const variantB = await createVariant(productB.id, `SKU-${randomUUID().slice(0, 8)}`, '222'); + + const setPrice = async ( + variantId: string, + netUnitAmountCents: number, + costCents: number, + ) => { + const response = await app.inject({ + method: 'PUT', + url: `/pricing/variants/${variantId}`, + headers: { 'content-type': 'application/json' }, + cookies: { [SESSION_COOKIE_NAME]: adminCookie }, + payload: { netUnitAmountCents, costCents, vatRate: 'general' }, + }); + expect(response.statusCode).toBe(200); + }; + + const setStock = async (variantId: string, quantity: number) => { + const response = await app.inject({ + method: 'PUT', + url: `/inventory/${variantId}/stock`, + headers: { 'content-type': 'application/json' }, + cookies: { [SESSION_COOKIE_NAME]: adminCookie }, + payload: { quantity }, + }); + expect(response.statusCode).toBe(200); + }; + + await setPrice(variantA.id, 1000, 600); + await setPrice(variantB.id, 1000, 950); + await setStock(variantA.id, 8); + await setStock(variantB.id, 2); + + const search = await app.inject({ + method: 'GET', + url: '/inventory/admin/overview?q=111&limit=10&offset=0', + cookies: { [SESSION_COOKIE_NAME]: adminCookie }, + }); + expect(search.statusCode).toBe(200); + expect(search.json()).toMatchObject({ + total: 1, + stats: { inStock: 1, lowStock: 0, outOfStock: 0 }, + items: [ + { + productName: 'Avena bio', + variant: { ean: '111' }, + stock: { availableQuantity: 8 }, + }, + ], + }); + + const lowMargin = await app.inject({ + method: 'GET', + url: '/inventory/admin/overview?filter=low_margin&limit=10&offset=0', + cookies: { [SESSION_COOKIE_NAME]: adminCookie }, + }); + expect(lowMargin.statusCode).toBe(200); + expect(lowMargin.json()).toMatchObject({ + total: 1, + items: [ + { + productName: 'Semillas lino', + stock: { availableQuantity: 2 }, + marginPercent: 21, + }, + ], + }); + + const expiring = await app.inject({ + method: 'GET', + url: '/inventory/admin/overview?filter=expiring&limit=10&offset=0', + cookies: { [SESSION_COOKIE_NAME]: adminCookie }, + }); + expect(expiring.statusCode).toBe(200); + const expiringBody = expiring.json() as { + total: number; + items: Array<{ productName: string }>; + }; + expect(expiringBody.total).toBe(2); + expect(expiringBody.items.every((item) => item.productName === 'Semillas lino')).toBe(true); + }); +}); diff --git a/project/src/modules/inventory/api/inventory.routes.ts b/project/src/modules/inventory/api/inventory.routes.ts index 69b58f8..05ca289 100644 --- a/project/src/modules/inventory/api/inventory.routes.ts +++ b/project/src/modules/inventory/api/inventory.routes.ts @@ -21,6 +21,32 @@ export interface InventoryRoutesDeps { authenticate: Authenticate; } +interface InventoryOverviewRow { + product_id: string; + product_name: string; + product_slug: string; + product_active: boolean; + product_created_at: Date; + expiration_date: string | null; + variant_id: string; + sku: string; + ean: string | null; + available_quantity: number; + net_unit_amount_cents: number | null; + offer_cents: number | null; + cost_cents: number | null; + vat_rate: 'general' | 'reduced' | 'super-reduced' | null; + gross_unit_amount_cents: number | null; + margin_percent: number | null; +} + +interface InventoryOverviewStatsRow { + total: number; + in_stock: number; + low_stock: number; + out_of_stock: number; +} + const variantParamSchema = z.object({ variantId: z.uuid() }); const productParamSchema = z.object({ productId: z.uuid() }); const availabilityQuerySchema = z.object({ @@ -44,12 +70,164 @@ const bulkAdjustBodySchema = z.object({ items: z.array(bulkAdjustItemSchema).min(1).max(100), }); +const inventoryOverviewFilterSchema = z.enum([ + 'all', + 'in_stock', + 'low_stock', + 'out_of_stock', + 'expiring', + 'low_margin', +]); +const inventoryOverviewQuerySchema = z.object({ + q: z.string().trim().max(120).optional(), + filter: inventoryOverviewFilterSchema.default('all'), + limit: z.coerce.number().int().min(1).max(200).default(50), + offset: z.coerce.number().int().min(0).max(100_000).default(0), +}); + export async function registerInventoryRoutes( app: FastifyInstance, deps: InventoryRoutesDeps, ): Promise { const inventory = new InventoryService(new PgInventoryRepository(deps.pool)); + const inventoryOverviewSchema: FastifySchema = { + tags: ['Inventory'], + summary: 'Inventory overview (admin)', + querystring: { + type: 'object', + properties: { + q: { type: 'string', maxLength: 120 }, + filter: { + type: 'string', + enum: ['all', 'in_stock', 'low_stock', 'out_of_stock', 'expiring', 'low_margin'], + default: 'all', + }, + limit: { type: 'integer', minimum: 1, maximum: 200, default: 50 }, + offset: { type: 'integer', minimum: 0, default: 0 }, + }, + }, + response: { 401: errorSchema, 403: errorSchema }, + }; + app.get('/inventory/admin/overview', { schema: inventoryOverviewSchema }, async (request, reply) => { + const user = await deps.authenticate(request); + requireRole(user, 'admin'); + const { q, filter, limit, offset } = parseJson(inventoryOverviewQuerySchema, request.query ?? {}); + + const params: unknown[] = [DEFAULT_STORE_ID]; + let nextParam = 2; + const searchConditions: string[] = []; + if (q) { + const placeholder = `$${nextParam++}`; + params.push(`%${q}%`); + searchConditions.push( + `(product_name ILIKE ${placeholder} OR sku ILIKE ${placeholder} OR COALESCE(ean, '') ILIKE ${placeholder})`, + ); + } + const filterCondition = inventoryOverviewFilterClause(filter); + const whereSearch = searchConditions.length > 0 ? `WHERE ${searchConditions.join(' AND ')}` : ''; + const whereItems = [ + ...searchConditions, + ...(filterCondition ? [filterCondition] : []), + ]; + const whereItemsSql = whereItems.length > 0 ? `WHERE ${whereItems.join(' AND ')}` : ''; + const orderBy = inventoryOverviewOrderBy(filter); + + const baseSql = ` + WITH raw AS ( + SELECT + p.id AS product_id, + p.name AS product_name, + p.slug AS product_slug, + p.state = 'active' AS product_active, + p.created_at AS product_created_at, + p.expiration_date, + v.id AS variant_id, + v.sku AS variant_sku, + v.ean, + COALESCE(stock.available, 0) AS available_quantity, + price.net_unit_amount_cents, + price.offer_cents, + price.cost_cents, + price.vat_rate, + COALESCE( + price.offer_cents, + ROUND( + price.net_unit_amount_cents * CASE price.vat_rate + WHEN 'general' THEN 1.21 + WHEN 'reduced' THEN 1.10 + WHEN 'super-reduced' THEN 1.04 + ELSE 1.21 + END + )::int + ) AS gross_unit_amount_cents + FROM catalog_product_variants v + JOIN catalog_products p ON p.id = v.product_id + LEFT JOIN inventory_stock stock ON stock.variant_id = v.id AND stock.store_id = $1 + LEFT JOIN pricing_variant_prices price ON price.variant_id = v.id AND price.currency = 'EUR' + ), + base AS ( + SELECT + product_id, + product_name, + product_slug, + product_active, + product_created_at, + expiration_date, + variant_id, + variant_sku AS sku, + ean, + available_quantity, + net_unit_amount_cents, + offer_cents, + cost_cents, + vat_rate, + gross_unit_amount_cents, + CASE + WHEN cost_cents IS NULL OR gross_unit_amount_cents IS NULL OR gross_unit_amount_cents <= 0 THEN NULL + ELSE ROUND((((gross_unit_amount_cents - cost_cents)::numeric / gross_unit_amount_cents::numeric) * 100))::int + END AS margin_percent + FROM raw + ) + `; + + const statsResult = await deps.pool.query( + `${baseSql} + SELECT + COUNT(*)::int AS total, + COUNT(*) FILTER (WHERE available_quantity >= 5)::int AS in_stock, + COUNT(*) FILTER (WHERE available_quantity > 0 AND available_quantity < 5)::int AS low_stock, + COUNT(*) FILTER (WHERE available_quantity = 0)::int AS out_of_stock + FROM base + ${whereSearch}`, + params, + ); + + const countParams = [...params, limit, offset]; + const countResult = await deps.pool.query<{ total: string }>( + `${baseSql} + SELECT COUNT(*)::int AS total + FROM base + ${whereItemsSql}`, + params, + ); + const rows = await deps.pool.query( + `${baseSql} + SELECT * + FROM base + ${whereItemsSql} + ORDER BY ${orderBy} + LIMIT $${nextParam++} OFFSET $${nextParam++}`, + countParams, + ); + + return reply.send({ + items: rows.rows.map(serializeInventoryOverviewRow), + total: Number(countResult.rows[0]?.total ?? 0), + stats: serializeInventoryOverviewStats(statsResult.rows[0]), + }); + }); + const availabilitySchema: FastifySchema = { tags: ['Inventory'], summary: 'Check availability (público)', @@ -385,3 +563,67 @@ function serializeStockItem(item: StockItem) { updatedAt: item.updatedAt.toISOString(), }; } + +function inventoryOverviewFilterClause( + filter: z.infer, +): string { + if (filter === 'in_stock') return 'available_quantity >= 5'; + if (filter === 'low_stock') return 'available_quantity > 0 AND available_quantity < 5'; + if (filter === 'out_of_stock') return 'available_quantity = 0'; + if (filter === 'expiring') { + return "expiration_date IS NOT NULL AND expiration_date <= current_date + 30"; + } + if (filter === 'low_margin') return 'margin_percent IS NOT NULL AND margin_percent < 30'; + return ''; +} + +function inventoryOverviewOrderBy(filter: z.infer): string { + if (filter === 'expiring') { + return 'expiration_date ASC NULLS LAST, product_name ASC, sku ASC'; + } + if (filter === 'low_margin') { + return 'margin_percent ASC NULLS LAST, product_name ASC, sku ASC'; + } + return 'product_created_at DESC, product_name ASC, sku ASC'; +} + +function serializeInventoryOverviewRow(row: InventoryOverviewRow) { + return { + productId: row.product_id, + productName: row.product_name, + productSlug: row.product_slug, + productActive: row.product_active, + expirationDate: row.expiration_date, + variant: { + id: row.variant_id, + productId: row.product_id, + sku: row.sku, + ean: row.ean, + attributes: {}, + }, + stock: { + available: row.available_quantity > 0, + availableQuantity: Number(row.available_quantity), + }, + price: row.net_unit_amount_cents === null || row.vat_rate === null + ? null + : { + variantId: row.variant_id, + netUnitAmountCents: Number(row.net_unit_amount_cents), + offerCents: row.offer_cents === null ? null : Number(row.offer_cents), + costCents: row.cost_cents === null ? null : Number(row.cost_cents), + vatRate: row.vat_rate, + currency: 'EUR', + }, + marginPercent: row.margin_percent === null ? null : Number(row.margin_percent), + }; +} + +function serializeInventoryOverviewStats(row: InventoryOverviewStatsRow | undefined) { + return { + total: Number(row?.total ?? 0), + inStock: Number(row?.in_stock ?? 0), + lowStock: Number(row?.low_stock ?? 0), + outOfStock: Number(row?.out_of_stock ?? 0), + }; +} diff --git a/work/artifacts/INVENTORY-OPT/architect.md b/work/artifacts/INVENTORY-OPT/architect.md new file mode 100644 index 0000000..ebd2011 --- /dev/null +++ b/work/artifacts/INVENTORY-OPT/architect.md @@ -0,0 +1,63 @@ +# Arquitectura — INVENTORY-OPT + +## Problema +La pantalla admin de inventario no escala a 10k+ productos porque hoy: +- pide `productsApi.list(limit=100)` +- por cada producto hace `getVariants` +- por cada fila hace `getAvailability` +- por cada fila hace `getVariantPrice` + +Eso introduce N+1 HTTP + N+1 SQL y además no tiene paginación real del inventario. + +## Diseño propuesto + +### 1) Nuevo endpoint backend optimizado +Crear `GET /inventory/admin/overview` en el módulo `inventory`. + +Query params: +- `q` +- `filter` +- `limit` +- `offset` + +Respuesta: +- `items[]` con fila ya enriquecida (producto + variante + stock + precio + caducidad) +- `total` +- `stats` con conteos globales (`inStock`, `lowStock`, `outOfStock`) + +### 2) Query única paginada +La query leerá directamente: +- `catalog_products` +- `catalog_product_variants` +- `inventory_stock` +- `pricing_variant_prices` + +Con esto evitamos las múltiples rondas actuales desde el admin. + +### 3) Filtros server-side +Mover al backend los filtros ya visibles en UI: +- `all` +- `in_stock` +- `low_stock` +- `out_of_stock` +- `expiring` +- `low_margin` + +### 4) UI admin +Actualizar `project/apps/admin/src/app/(dashboard)/inventory/page.tsx` para: +- consumir el nuevo endpoint +- usar paginación real +- mantener búsqueda debounced +- mantener filtros existentes +- evitar llamadas por fila + +## No entra +- Refactor de módulos no relacionados +- Reescritura de pricing o inventory core +- Cambios en TPV/storefront + +## Validación +- typecheck backend/admin +- build backend/admin +- verify.sh +- prueba funcional manual de búsqueda/filtros/paginación diff --git a/work/artifacts/INVENTORY-OPT/implementer.md b/work/artifacts/INVENTORY-OPT/implementer.md new file mode 100644 index 0000000..807227e --- /dev/null +++ b/work/artifacts/INVENTORY-OPT/implementer.md @@ -0,0 +1,79 @@ +# Implementer evidence — INVENTORY-OPT + +## Cambios realizados + +### 1) Nuevo endpoint backend optimizado +Añadí `GET /inventory/admin/overview` en: +- `project/src/modules/inventory/api/inventory.routes.ts` + +El endpoint: +- pagina server-side (`limit`, `offset`) +- busca por `nombre`, `SKU` o `EAN` +- aplica filtros server-side: + - `all` + - `in_stock` + - `low_stock` + - `out_of_stock` + - `expiring` + - `low_margin` +- devuelve filas ya enriquecidas con: + - producto + - variante + - stock + - precio + - margen + - caducidad +- devuelve además `stats` agregados para cards del dashboard de inventario + +### 2) Eliminación del N+1 en admin inventory +Antes la página hacía: +- `productsApi.list(limit=100)` +- `getVariants()` por producto +- `getAvailability()` por fila +- `getVariantPrice()` por fila + +Ahora la página consume una única llamada paginada: +- `inventoryApi.listOverview(...)` + +### 3) UI admin paginada +Actualicé: +- `project/apps/admin/src/app/(dashboard)/inventory/page.tsx` +- `project/apps/admin/src/lib/api-client.ts` +- `project/apps/admin/src/types/index.ts` + +La página ahora: +- usa búsqueda debounced server-side +- usa filtros server-side +- muestra paginación real +- sigue permitiendo editar inline: + - stock + - EAN + - caducidad + +### 4) Test de integración real PostgreSQL +Añadí: +- `project/src/app/tests/inventory-admin-overview.itest.ts` + +Valida: +- paginación +- búsqueda por EAN +- filtro `low_margin` +- filtro `expiring` +- stats agregados + +## Validación +- `cd project && npm run typecheck` ✅ +- `cd project && npm run build` ✅ +- `cd project/apps/admin && npm run build` ✅ +- `cd project && TEST_DATABASE_URL=postgres://mdv:mdv_dev_only@localhost:5432/mercadodevida_test npx vitest run src/app/tests/inventory-admin-overview.itest.ts --no-file-parallelism` ✅ +- `./scripts/verify.sh` ✅ +- `git diff --check` ✅ + +## Decisiones técnicas +- No introduje un módulo nuevo: reutilicé `inventory.routes.ts` porque el problema principal era de lectura agregada para admin. +- El inventario se optimiza a nivel de query y tráfico HTTP, que era el cuello de botella real para 10k+ productos. +- Mantuve edición inline en la UI para no romper el flujo actual del admin. + +## Deuda / siguiente paso +- Si el volumen sigue creciendo, el siguiente paso natural sería añadir índices específicos para búsquedas admin por nombre/SKU/EAN o una vista materializada/consulta especializada. +- La build del admin sigue mostrando warnings preexistentes de Turbopack sobre acceso dinámico a ficheros en uploads; no están relacionados con INVENTORY-OPT. diff --git a/work/artifacts/INVENTORY-OPT/leader-close.json b/work/artifacts/INVENTORY-OPT/leader-close.json new file mode 100644 index 0000000..768cf88 --- /dev/null +++ b/work/artifacts/INVENTORY-OPT/leader-close.json @@ -0,0 +1,21 @@ +{ + "feature_id": "INVENTORY-OPT", + "agent": "leader", + "stage": "close", + "verdict": "APPROVED", + "leader": "leader", + "summary": "INVENTORY-OPT cerrada: inventario admin con paginación real, filtros server-side y eliminación del N+1.", + "gates_summary": { + "reviewer": "APPROVED", + "security": "APPROVED", + "qa": "APPROVED" + }, + "artifacts": [ + "architect.md", + "implementer.md", + "reviewer.json", + "security.json", + "qa.json", + "leader-close.json" + ] +} diff --git a/work/artifacts/INVENTORY-OPT/qa.json b/work/artifacts/INVENTORY-OPT/qa.json new file mode 100644 index 0000000..e611568 --- /dev/null +++ b/work/artifacts/INVENTORY-OPT/qa.json @@ -0,0 +1,30 @@ +{ + "feature_id": "INVENTORY-OPT", + "agent": "qa", + "stage": "qa_gate", + "verdict": "APPROVED", + "qa_check": "qa", + "summary": "QA aprobado: inventario admin ahora escala con paginación server-side, búsqueda/filtros server-side y sin N+1 por fila.", + "test_results": { + "automated": [ + "./scripts/verify.sh ✅", + "cd project && npm run typecheck ✅", + "cd project && npm run build ✅", + "cd project/apps/admin && npm run build ✅", + "cd project && TEST_DATABASE_URL=postgres://mdv:mdv_dev_only@localhost:5432/mercadodevida_test npx vitest run src/app/tests/inventory-admin-overview.itest.ts --no-file-parallelism ✅" + ], + "coverage": [ + "búsqueda admin por nombre/SKU/EAN", + "filtro low_margin", + "filtro expiring", + "paginación y stats agregados" + ], + "manual_smoke_recommended": [ + "Comprobar en /inventory que cambiar stock inline refresca correctamente la fila al editar dentro de páginas distintas.", + "Comprobar UX con búsquedas y filtros sobre un dataset grande real." + ] + }, + "notes": [ + "El warning de Turbopack sobre uploads en admin build es preexistente y no bloquea esta feature." + ] +} diff --git a/work/artifacts/INVENTORY-OPT/reviewer.json b/work/artifacts/INVENTORY-OPT/reviewer.json new file mode 100644 index 0000000..3e55a77 --- /dev/null +++ b/work/artifacts/INVENTORY-OPT/reviewer.json @@ -0,0 +1,29 @@ +{ + "feature_id": "INVENTORY-OPT", + "agent": "reviewer", + "stage": "review_gate", + "verdict": "APPROVED", + "checks": [ + { + "item": "Admin inventory no longer performs N+1 product/variant/stock/price fetches and now reads a single paginated backend overview endpoint", + "ok": true + }, + { + "item": "Server-side query supports search by product name, SKU and EAN plus the existing stock/expiry/low-margin filters", + "ok": true + }, + { + "item": "Admin UI preserves inline stock/EAN/expiry editing while moving pagination and filtering to the server", + "ok": true + }, + { + "item": "Changed files validated by backend typecheck/build, admin build, verify.sh and a real PostgreSQL integration test for the overview endpoint", + "ok": true + } + ], + "issues": [], + "notes": [ + "The endpoint is variant-based, which matches inventory reality better than the old first-variant-per-product approximation.", + "Admin build still emits unrelated existing Turbopack warnings about dynamic uploads filesystem access; not introduced by this feature." + ] +} diff --git a/work/artifacts/INVENTORY-OPT/security.json b/work/artifacts/INVENTORY-OPT/security.json new file mode 100644 index 0000000..ee7de36 --- /dev/null +++ b/work/artifacts/INVENTORY-OPT/security.json @@ -0,0 +1,19 @@ +{ + "feature_id": "INVENTORY-OPT", + "agent": "security", + "stage": "security_gate", + "verdict": "APPROVED", + "security_check": "security", + "summary": "Aprobado: la optimización de inventario no introduce nuevas superficies sensibles y mantiene autenticación/validación correctas.", + "checks": { + "auth": "OK: el nuevo endpoint /inventory/admin/overview sigue protegido con authenticate + requireRole(admin).", + "injection": "OK: búsqueda y paginación siguen parametrizadas; el ORDER BY y filtros son ramas estáticas controladas por enum de zod, sin interpolación arbitraria del usuario.", + "data_exposure": "OK: solo devuelve información de inventario/precio ya disponible para el admin; no añade datos personales ni secretos.", + "xss": "OK: la UI sigue renderizando texto plano y no introduce HTML crudo ni APIs inseguras del navegador.", + "dependencies": "OK: no se añadieron dependencias nuevas." + }, + "notes": [ + "La búsqueda administrativa por nombre/SKU/EAN es de bajo riesgo porque usa parámetros SQL y la ruta está restringida a admin.", + "No se detectan regresiones de permisos ni de almacenamiento de credenciales." + ] +} diff --git a/work/runtime-status.json b/work/runtime-status.json index ef74522..ca83fea 100644 --- a/work/runtime-status.json +++ b/work/runtime-status.json @@ -1,97 +1,13 @@ { - "feature_id": "CLUB-001", - "stage": "build", - "agent": "implementer", - "action": "Club fase 1 backend implementado y validado", - "state": "done", - "next_agent": "reviewer", - "waiting_for": "review_gate", - "updated_at": "2026-08-26T15:52:21Z", + "feature_id": "INVENTORY-OPT", + "stage": "close", + "agent": "leader", + "action": "Cerrar INVENTORY-OPT y commit/push automático", + "state": "running", + "next_agent": "leader", + "waiting_for": "close", + "updated_at": "2026-08-26T16:07:40Z", "timeline": [ - { - "ts": "2026-08-25T20:50:29Z", - "agent": "implementer", - "stage": "build", - "state": "running", - "message": "Fix storefront card bg, stock cap, return human readable, PATCH 502" - }, - { - "ts": "2026-08-25T20:54:10Z", - "agent": "implementer", - "stage": "build", - "state": "running", - "message": "Fix storefront card bg, stock cap, return human readable, PATCH 502, selfpay button rename" - }, - { - "ts": "2026-08-25T21:03:29Z", - "agent": "implementer", - "stage": "build", - "state": "running", - "message": "Fix receipt-settings 400 and quick products save" - }, - { - "ts": "2026-08-26T05:34:46Z", - "agent": "implementer", - "stage": "build", - "state": "running", - "message": "Implementando buscador + lista de hasta 8 productos rápidos" - }, - { - "ts": "2026-08-26T05:43:09Z", - "agent": "implementer", - "stage": "build", - "state": "done", - "message": "Buscador + lista de 8 + guardado validado vía API" - }, - { - "ts": "2026-08-26T08:48:04Z", - "agent": "implementer", - "stage": "build", - "state": "running", - "message": "Fix ticket IVA=0 y ajustar mensaje de stock insuficiente en frontend" - }, - { - "ts": "2026-08-26T09:06:24Z", - "agent": "implementer", - "stage": "build", - "state": "done", - "message": "Fix ticket IVA=0 y sincronización checkout/cart duplicada" - }, - { - "ts": "2026-08-26T10:40:21Z", - "agent": "reviewer", - "stage": "review_gate", - "state": "running", - "message": "Revisión técnica de IVA POS y sync checkout" - }, - { - "ts": "2026-08-26T10:40:45Z", - "agent": "reviewer", - "stage": "review_gate", - "state": "done", - "message": "Revisión técnica aprobada para IVA POS y sync checkout" - }, - { - "ts": "2026-08-26T10:40:48Z", - "agent": "security", - "stage": "security_gate", - "state": "running", - "message": "Revisión de seguridad de POS pricing y checkout sync" - }, - { - "ts": "2026-08-26T10:41:15Z", - "agent": "security", - "stage": "security_gate", - "state": "done", - "message": "Revisión de seguridad aprobada" - }, - { - "ts": "2026-08-26T10:41:25Z", - "agent": "qa", - "stage": "qa_gate", - "state": "running", - "message": "QA de receipts POS y checkout sync" - }, { "ts": "2026-08-26T10:41:58Z", "agent": "qa", @@ -147,6 +63,90 @@ "stage": "build", "state": "done", "message": "Club fase 1 backend implementado y validado" + }, + { + "ts": "2026-08-26T15:59:00Z", + "agent": "implementer", + "stage": "build", + "state": "blocked", + "message": "CLUB-001 pausada: priorizar INVENTORY-OPT, SHIPPING-ZONES y batch de fixes solicitado por usuario" + }, + { + "ts": "2026-08-26T15:59:14Z", + "agent": "architect", + "stage": "design", + "state": "running", + "message": "Analizar inventario para optimización con 10k+ productos" + }, + { + "ts": "2026-08-26T16:00:10Z", + "agent": "architect", + "stage": "design", + "state": "done", + "message": "Diseño completado para inventario paginado y sin N+1" + }, + { + "ts": "2026-08-26T16:00:10Z", + "agent": "implementer", + "stage": "build", + "state": "running", + "message": "Implementar endpoint optimizado y UI paginada de inventario" + }, + { + "ts": "2026-08-26T16:06:51Z", + "agent": "implementer", + "stage": "build", + "state": "done", + "message": "Inventario optimizado con paginación y sin N+1" + }, + { + "ts": "2026-08-26T16:06:54Z", + "agent": "reviewer", + "stage": "review_gate", + "state": "running", + "message": "Revisión técnica de optimización de inventario" + }, + { + "ts": "2026-08-26T16:07:05Z", + "agent": "reviewer", + "stage": "review_gate", + "state": "done", + "message": "Revisión técnica aprobada para inventario" + }, + { + "ts": "2026-08-26T16:07:08Z", + "agent": "security", + "stage": "security_gate", + "state": "running", + "message": "Revisión de seguridad de inventory admin overview" + }, + { + "ts": "2026-08-26T16:07:19Z", + "agent": "security", + "stage": "security_gate", + "state": "done", + "message": "Revisión de seguridad aprobada para inventario" + }, + { + "ts": "2026-08-26T16:07:22Z", + "agent": "qa", + "stage": "qa_gate", + "state": "running", + "message": "QA de inventario paginado" + }, + { + "ts": "2026-08-26T16:07:34Z", + "agent": "qa", + "stage": "qa_gate", + "state": "done", + "message": "QA aprobada para inventario optimizado" + }, + { + "ts": "2026-08-26T16:07:40Z", + "agent": "leader", + "stage": "close", + "state": "running", + "message": "Cerrar INVENTORY-OPT y commit/push automático" } ] }