feat(recepcion): verificar el albaran con el papel delante, y retener lo que la IA marca

ALB-00119 (20/08, Alcala): el papel dice 56,57 kg de milanesas a 7,50 EUR/kg =
424,27 EUR; la pantalla dice "56,57 uds"; lo guardado son 38 paquetes, 228 uds y
unit_cost NULL. La sesion de IA traia needs_review=true con tres motivos y la
recepcion se creo con needs_review=false y la mercancia posteada.

La bandera de la IA no llegaba por dos sitios: createGoodsReceipt se llamaba sin
needsReview, y receive_goods_receipt la PISABA con needs_review=(skipped>0).

- migracion 20260820T1700 (PROPUESTA, sin aplicar): _post_goods_receipt_lines
  gana p_only_unposted; receive_goods_receipt gana p_hold (no postea nada y deja
  el albaran en 'recibido' para que oficina lo abra) y deja de pisar la bandera;
  confirm_goods_receipt postea al cerrar lo que nunca posteo y exige proveedor y
  n de albaran para cerrar un 'recibido'.
- ReceiptScanPanel/ReceiptWizard: la validacion de la IA viaja en OcrPrefill; el
  asistente ensena los motivos, cambia el boton a "Guardar y mandar a oficina" y
  no aprende de un albaran retenido. El importe de linea se prerrellena tambien
  desde cantidad x precio cuando el papel no trae el importe (causa del coste
  nulo).
- ReceiptOfficeReview: el albaran se ve AL LADO de las lineas (sticky en ancho,
  boton grande en movil) reusando ReceiptPhotoViewer, que ya existia y no se
  usaba aqui. Editor de linea con FORMATO ademas de cantidad y coste, con vista
  previa de lo que entra al almacen y motivo obligatorio; disponible tambien
  desde una linea dudosa, que antes solo ofrecia si/no. Proveedor y n de albaran
  editables y obligatorios para cerrar.
- El aviso amarillo decia siempre "lo emparejo por parecido de nombre": de las
  408 marcadas en Foodint, 400 son map_source='unmapped' y en toda la base solo
  hay 11 fuzzy. Ahora cada linea dice su motivo (unverifiedReason, 8 pruebas).
- UnverifiedLinesPage: las lineas que entraron sin confirmar, por importe, con
  su albaran a un clic. 341 lineas, 16.108,21 EUR (las 410 del encargo incluian
  albaranes anulados).

El frontend funciona antes y despues de la migracion: hold=false llama con un
solo argumento, y hold=true sin migracion deja la recepcion en borrador con un
error legible en vez de romper el recepcionar.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Vy4yW6xBAtXcaSGgKybquB

Author: claude

docs/recepcion_verificar_a_ciegas_20260820.md

@@ -0,0 +1,200 @@
+# ENCARGO «Verificar un albarán a ciegas» — resultado
+
+Fecha: 2026-08-20 · Rama: `main` y `claude/redesign-pricing-modal-amxuzy`
+
+> **Hay una migración SIN APLICAR**: `supabase/migrations/20260820T1700_recepcion_verificacion_a_ciegas.sql`.
+> El frontend está blindado para funcionar antes y después de aplicarla (ver §Orden de despliegue).
+
+---
+
+## 0 · Qué pasó en ALB-00119, con los datos delante
+
+Sesión de IA `9bae61b1-fdba-4fd0-a16e-c8450aec72d1`, leída de producción:
+
+```json
+"document":   { "handwritten": true, "supplier_name": null, "doc_number": null,
+                "bill_to_name": "LLORENTE", "tax_base_total": 424.27,
+                "tax_total": 42.27, "grand_total": 466.54 }
+"lines":      [ { "raw_text": "Milanesas pollo", "quantity": 56.57, "unit": "kg",
+                  "unit_price_net": 7.5, "line_amount": null } ]
+"validation": { "needs_review": true,
+                "reasons": ["No se pudo validar por base imponible (faltan importes)",
+                            "Documento manuscrito", "Confianza de lectura baja"] }
+```
+
+**La IA lo leyó bien.** Los tres números distintos salen de tres fallos encadenados:
+
+1. **`unit_price_net` sí estaba, `line_amount` no.** El asistente solo prerrellenaba
+   el importe desde `line_amount`, así que la casilla salió vacía → `unit_cost = NULL`.
+   424 € de pollo a coste cero.
+
+2. **El artículo se mide en unidades, no en kilos.** `Milanesa de Pollo Rebozado`
+   tiene unidad base `ud` (250 g cada una) y cuatro formatos, ninguno en kg. El papel
+   dice 56,57 kg; el asistente lo convirtió a Paquetes (6 uds = 1,5 kg):
+   56,57 ÷ 1,5 = 37,71 → **38 paquetes = 228 uds = 57 kg**. Sobran 1,72 uds (0,43 kg).
+   La pantalla enseña «56,57 uds» porque pinta `doc_qty` con la etiqueta de la unidad
+   base del artículo, sin decir que el papel hablaba de kilos.
+
+3. **La bandera de la IA no llegaba a la recepción**, y por dos sitios a la vez:
+   `createGoodsReceipt` se llamaba sin `needsReview`, y aunque se hubiera pasado,
+   `receive_goods_receipt` la PISABA con `needs_review = (v_skipped > 0)` — un albarán
+   con todas las líneas casadas sale `false` por definición, diga lo que diga la IA.
+
+---
+
+## 1 · El tamaño real, con el filtro que faltaba
+
+Las 410 del encargo incluyen líneas de albaranes **anulados o archivados**. Sin ellos:
+
+| | Del encargo | Medido hoy (albarán vivo, no anulado) |
+|---|---:|---:|
+| Líneas de recepción | 831 (dos cuentas) | 744 en Foodint |
+| Marcadas sin confirmar | 410 | **341** |
+| Importe en juego | — | **16.108,21 €** |
+| Cantidad distinta a la del papel | 53 | 53 |
+| Sin coste unitario | 8 | 5 en Foodint |
+
+**Y el aviso amarillo describe mal el problema.** Decía siempre *«Lo emparejó el sistema
+por parecido de nombre, no por código»*. De las 408 marcadas en Foodint, **400 son
+`map_source='unmapped'`** — el sistema no las casó en absoluto — y en toda la base solo
+hay **11 líneas `fuzzy`**. El aviso mandaba a mirar donde no era. Ahora cada línea dice
+lo que le pasa a ella (`unverifiedReason`, con 8 pruebas).
+
+---
+
+## 2 · Lo que se ha hecho
+
+### 2.1 · Ver el albarán — `ReceiptOfficeReview.tsx`
+
+El componente ya existía (`ReceiptPhotoViewer`, con zoom, lightbox y visor de PDF) y
+esta pantalla no lo usaba. Ahora:
+
+- **Pantalla ancha**: dos columnas, el papel a la derecha y `sticky` — sigue visible al
+  bajar por las líneas.
+- **Móvil**: botón grande *Ver el albarán* (no un icono de 9 px).
+
+Sirve para los dos formatos porque el visor ya los distinguía: ALB-00119 es `00-image.jpg`,
+ALB-00117 es un PDF de Bidfood.
+
+### 2.2 · Corregir cantidad, formato y coste
+
+`adjust_goods_receipt_line` estaba desplegada desde el 13/08 y en 24 h se llamó 2 veces
+frente a 357 peticiones de la pantalla. **No faltaba construirla: no se llegaba a ella.**
+Dos huecos, los dos tapados:
+
+- El editor **solo dejaba cambiar cantidad y coste, no el formato**. Un albarán en kilos
+  casado a un formato de 6 unidades no tenía arreglo desde la pantalla — el caso ALB-00119.
+  Ahora hay desplegable de formato con la medida en la etiqueta («Paquete (1,5 kg)»).
+- El botón **solo estaba en las líneas resueltas**. Las 341 marcadas son *dudosas*, y una
+  dudosa solo ofrecía *Sí, es esta* / *No, es otro artículo*: si el artículo era el
+  correcto pero la cantidad estaba mal, no había por dónde. Ahora hay un tercer botón,
+  **Corregir cantidad, formato o precio**.
+
+El editor enseña, mientras escribes, **lo que va a entrar al almacén** y el total.
+Exige motivo (`discrepancy_reason`) siempre que se mueva cantidad o formato — corregir
+solo el coste no mueve stock. Escribe **siempre** por `adjust_goods_receipt_line`
+(reverso + reposteo, quién y por qué), nunca por un `UPDATE` (§9.8).
+
+### 2.3 · Proveedor y nº de albarán
+
+Editables en la cabecera, y **obligatorios para cerrar** — en la pantalla y también en
+el servidor (`confirm_goods_receipt`), para que no dependa de que la pantalla se acuerde.
+La exigencia solo aplica al cierre de OFICINA (`recibido` → `confirmado`); un borrador
+manual sigue igual, para no levantar un muro sin puerta donde no se puede arreglar.
+
+Impacto medido: de 114 recepciones confirmadas, **0 sin proveedor y 4 sin nº**.
+
+### 3 · Que no se repita — la migración
+
+`20260820T1700_recepcion_verificacion_a_ciegas.sql`, tres piezas:
+
+| | Qué |
+|---|---|
+| **A** | `_post_goods_receipt_lines(p_receipt_id, p_only_unposted default false)` — postear solo las líneas que nunca han posteado. Sin el parámetro, idéntica a hoy. |
+| **B** | `receive_goods_receipt(p_receipt_id, p_hold default false)` — `p_hold` deja el albarán en `'recibido'` (para que la oficina lo pueda abrir) pero **no postea nada** y marca `needs_review`. Y deja de PISAR una bandera que venga puesta. |
+| **C** | `confirm_goods_receipt` — al cerrar un `'recibido'`, entra lo que aún no había entrado. Para un albarán normal no cambia nada: sus líneas posteables ya tienen movimiento. |
+
+**Por qué se retiene el posteo y no el estado**: un borrador va a `GoodsReceiptForm`, no
+a la pantalla de oficina, y `adjust_goods_receipt_line` exige `status='recibido'`.
+Retener en borrador dejaría el albarán fuera del único sitio donde se puede corregir.
+
+Verificado que nada más en la base llama a esas funciones: solo se mencionan en
+comentarios de `adjust_goods_receipt_line` y `post_pending_receipt_line`.
+
+En el asistente, un albarán retenido:
+- enseña al trabajador los motivos de la IA antes de guardar,
+- cambia el botón a **«Guardar y mandar a oficina»**,
+- y no aprende nada (`learnFromReceipt`, `last_price`, cascada de coste): aprender de un
+  albarán que la IA marcó dudoso es exactamente cómo se contamina el catálogo.
+
+### 6 · La lista de las líneas sin confirmar
+
+`UnverifiedLinesPage.tsx`, nueva. Todas las líneas marcadas de la cuenta, **ordenadas por
+importe**, cada una con su albarán detrás a un clic. Filtro por local. Distingue las que
+están en el almacén de las que no entraron. Entrada desde el aviso de la lista de
+recepciones, que ahora es un botón, no un cartel.
+
+---
+
+## Orden de despliegue
+
+El frontend **no depende** de que la migración esté puesta:
+
+- `hold=false` (todo el tráfico de hoy) llama a `receive_goods_receipt` con **un**
+  argumento → funciona antes y después.
+- `hold=true` solo puede ocurrir con la migración puesta; si no lo está, el error se
+  explica en castellano y **la recepción se queda en borrador**: con todo lo que escribió
+  el trabajador guardado y sin nada posteado. Recepcionar es lo que estuvo 6 días roto;
+  no se vuelve a romper por un orden de despliegue.
+
+Después de aplicar la migración, `needs_review` de la IA empieza a retener de verdad.
+
+---
+
+## Criterio 1 · La corrección de ALB-00119
+
+**No la he ejecutado yo, y es deliberado.** `adjust_goods_receipt_line` escribe
+`created_by = auth.uid()`; llamándola desde MCP (rol de servicio) el movimiento de 424 €
+quedaría firmado por nadie, que es justo lo contrario de lo que pide el criterio (§9.8,
+rastro). Se hace desde la pantalla, con el botón **Corregir** nuevo — y así el criterio 3
+se verifica con el mismo gesto.
+
+**Los números, sacados del papel:**
+
+| Campo | Valor | De dónde sale |
+|---|---|---|
+| Proveedor | **NOBLEZA VACUNA SL** (a confirmar con el papel) | es el proveedor **preferido** de este artículo y su ficha está en formato *Paquete*, el mismo de la línea. El albarán es manuscrito: la IA no leyó emisor. |
+| Nº de albarán | el que venga en el papel | la IA lo leyó como `null` |
+| Formato | **Paquete** (6 uds = 1,5 kg) | el que ya tiene; no hay ningún formato en kg |
+| Cantidad | **37,713333** | 56,57 kg ÷ 1,5 kg/paquete |
+| Coste de cada uno | **11,25 €** | 7,50 €/kg × 1,5 kg/paquete |
+| Motivo | *lo dice el albarán* | |
+
+Comprobación: 37,713333 × 11,25 = **424,27 €** ✓ (base imponible del papel)
+y 37,713333 × 6 = **226,28 uds** al almacén, frente a las 228 que hay ahora.
+
+**Aviso de precio**: `article_supplier.last_price` de este artículo con NOBLEZA está en
+1,25 €/ud = 5,00 €/kg. El papel dice 7,50 €/kg, **un 50 % más**. Puede ser real o puede
+ser un 5 leído como 7 en letra manuscrita. Con el papel delante se resuelve en un
+segundo; ése es exactamente el punto del encargo.
+
+---
+
+## Criterios
+
+| # | Criterio | Estado |
+|---|---|---|
+| 1 | ALB-00119 corregido con la función y motivo | Números listos arriba · **lo ejecuta Julio desde el botón Corregir** (razón explicada) |
+| 2 | El albarán se ve sin salir de la pantalla (foto y PDF) | ✅ implementado · falta tu captura |
+| 3 | Cambiar cantidad, formato y coste con rastro | ✅ implementado · falta tu captura |
+| 4 | Proveedor y nº se pueden poner y sin ellos no se cierra | ✅ pantalla + servidor (tras la migración) |
+| 5 | Una recepción con `needs_review` no postea stock | ✅ código · **efectivo al aplicar la migración** |
+| 6 | Lista de las líneas sin confirmar por importe | ✅ `UnverifiedLinesPage` · **341 líneas, 16.108,21 €** |
+
+## Comprobaciones
+
+`tsc -b` limpio · `npm run build` limpio · **31 pruebas, 3 ficheros, todas pasan**
+(8 nuevas de `unverifiedReason`).
+`eslint`: los mismos errores que ya había antes del encargo, verificado con `git stash`
+(2 en `GoodsReceiptsPage`, 2 en `ReceiptScanPanel`/`ReceiptWizard`, todos
+`set-state-in-effect` preexistentes). `UnverifiedLinesPage.tsx` sale limpia.

src/modules/supply/pages/GoodsReceiptForm.tsx

@@ -161,6 +161,14 @@ export interface OcrPrefill {
   rawDocumentUrl: string | null
   unmatchedSupplier: boolean
   unmatchedLocation: boolean
+  // ENCARGO CODE (20/08) «Verificar un albarán a ciegas» §3 — la IA YA sabe
+  // decir "esto hay que mirarlo" (documento manuscrito, no cuadra la base
+  // imponible, confianza de lectura baja) y ese aviso se quedaba en el panel
+  // de escaneo: nunca llegaba a la recepción. Caso real ALB-00119 (20/08):
+  // tres motivos puestos y la mercancía posteada igual, con coste nulo.
+  // Ahora viaja, y ReceiptWizard retiene el posteo.
+  aiNeedsReview: boolean
+  aiReviewReasons: string[]
   lines: OcrPrefillLine[]
 }
 export interface OcrPrefillLine {

src/modules/supply/pages/GoodsReceiptsPage.tsx

@@ -35,6 +35,7 @@ import {
   getReceiptDocTotal,
   getReceiptCorrectionStreak,
   getLineCounts,
+  countUnverifiedLines,
   type GoodsReceipt,
   type GoodsReceiptStatus,
   type CorrectionStreak,
@@ -51,6 +52,8 @@ import ReceiptScanPanel from '@/modules/supply/pages/ReceiptScanPanel'
 import ReceiptWizard from '@/modules/supply/pages/ReceiptWizard'
 import OrderReceiveFlow from '@/modules/supply/components/OrderReceiveFlow'
 import ReceiptOfficeReview from '@/modules/supply/pages/ReceiptOfficeReview'
+// ENCARGO CODE (20/08) «Verificar un albarán a ciegas» §6.
+import UnverifiedLinesPage from '@/modules/supply/pages/UnverifiedLinesPage'
 
 const STATUS_LABEL: Record<GoodsReceiptStatus, string> = {
   borrador: 'Borrador',
@@ -72,7 +75,7 @@ function formatDate(value: string | null): string {
     .format(new Date(value))
 }
 
-type View = 'list' | 'form' | 'scan' | 'receive-order' | 'wizard' | 'office-review'
+type View = 'list' | 'form' | 'scan' | 'receive-order' | 'wizard' | 'office-review' | 'unverified'
 
 export default function GoodsReceiptsPage() {
   const { activeAccountId, accountsLoading } = useActiveAccount()
@@ -103,6 +106,9 @@ export default function GoodsReceiptsPage() {
   // ENCARGO CODE (14/08) feat/recepcion-oficina-cierre, B.1 — la recepción
   // 'recibido' abre ReceiptOfficeReview (pantalla propia), no GoodsReceiptForm.
   const [officeReviewReceiptId, setOfficeReviewReceiptId] = useState<string | null>(null)
+  // ENCARGO CODE (20/08) «Verificar un albarán a ciegas» §6 — cuántas líneas
+  // entraron al almacén sin que nadie las confirmara.
+  const [unverifiedCount, setUnverifiedCount] = useState(0)
 
   // Arranque rápido desde el vigía de stock negativo (Almacén → Teórico vs
   // Real → Stock negativo): llega por navigate(state), no por props (esta
@@ -177,10 +183,13 @@ export default function GoodsReceiptsPage() {
       listGoodsReceipts({ accountId: activeAccountId, locationId: resolvedLocationId ?? undefined }),
       listSuppliers(activeAccountId),
       listSupplyLocations(activeAccountId),
+      // ENCARGO CODE (20/08) §6 — solo el número; la lista se carga al entrar.
+      // Si falla no rompe la pantalla: el aviso simplemente no sale.
+      countUnverifiedLines(activeAccountId).catch(() => 0),
     ])
-      .then(([rows, sups, locs]) => {
+      .then(([rows, sups, locs, unver]) => {
         if (cancelled) return
-        setReceipts(rows); setSuppliers(sups); setLocations(locs)
+        setReceipts(rows); setSuppliers(sups); setLocations(locs); setUnverifiedCount(unver)
       })
       .catch((err: unknown) => {
         if (cancelled) return
@@ -513,6 +522,18 @@ export default function GoodsReceiptsPage() {
     )
   }
 
+  // ── Vista SIN-VERIFICAR: las líneas que entraron sin confirmar (§6) ──
+  if (view === 'unverified' && activeAccountId) {
+    return (
+      <UnverifiedLinesPage
+        accountId={activeAccountId}
+        locationId={resolvedLocationId}
+        onBack={() => { setView('list'); setReloadTick(t => t + 1) }}
+        onOpenReceipt={(id) => { setOfficeReviewReceiptId(id); setView('office-review') }}
+      />
+    )
+  }
+
   // ── Vista OFFICE-REVIEW: oficina verifica una recepción 'recibido' ──
   if (view === 'office-review' && activeAccountId && officeReviewReceiptId) {
     return (
@@ -603,6 +624,21 @@ export default function GoodsReceiptsPage() {
         </div>
       )}
 
+      {/* ENCARGO CODE (20/08) «Verificar un albarán a ciegas» §6 — el aviso de
+          la pantalla de oficina avisaba de algo que ya había pasado y no daba
+          con qué arreglarlo: una alarma sin puerta. Ésta es la puerta. */}
+      {!loading && !error && unverifiedCount > 0 && (
+        <button type="button" onClick={() => setView('unverified')}
+          className="w-full text-left p-3 rounded-md border border-warning/30 bg-warning-bg flex items-center gap-2 hover:opacity-90 transition-base">
+          <AlertTriangle size={16} className="text-warning shrink-0" />
+          <p className="text-sm font-medium text-text-primary flex-1">
+            {unverifiedCount} línea{unverifiedCount === 1 ? '' : 's'} entr{unverifiedCount === 1 ? 'ó' : 'aron'} al
+            almacén sin que nadie {unverifiedCount === 1 ? 'la' : 'las'} confirmara
+          </p>
+          <span className="shrink-0 text-sm font-semibold text-text-primary">Verlas por importe →</span>
+        </button>
+      )}
+
       {/* ENCARGO CODE (13/08) fix/recepcion-p2-oficina, §5 — solo lectura + una
           propuesta, nada automático. Sin botón que escriba receipt_approval=
           'directo': ese valor no lo consume ninguna pantalla todavía (modo