Compare commits

..
Author SHA1 Message Date
devops 949788cda7 Merge fix/docs-portal-fase2-contenido-narrativo into main
Build and Push Docs Portal / Escaneo de Secretos (Gitleaks) (push) Successful in 21s
Build and Push Docs Portal / Construir y publicar Docs Portal (push) Successful in 5m22s
2026-08-26 23:51:18 -05:00
devops 056a7fb76a docs(docs-portal): Fase 2 - contenido narrativo real de arquitectura/decisiones/aprendizajes
Reemplaza los placeholders "TODO (fase 2+)" por narrativa completa,
grounded en el codigo/manifiestos/incidentes reales del lab (no
generico):

- ADR 0001 (por que k3d) y ADR 0002 (por que Argo CD): pasan de
  template vacio a Contexto/Opciones/Decision/Consecuencias reales,
  apoyados en known-issues.md (blackhole MTU) y en la config real de
  ecommerce-app.yaml.
- vision-general.md: el "por que" de cada frontera de confianza del
  diagrama.
- gitops-flujo.md: ejemplo real con hashes de commit (Fase 3b de
  ADR 0006, v1.0.171 -> hotfix -> v1.0.172), selfHeal explicado con el
  syncPolicy real del repo.
- red-y-exposicion.md: diagrama de terminacion TLS real, por que todo
  va "proxied", patron de diagnostico "interno sano, publico roto".
- glosario.md: 9 terminos nuevos, cada uno con ejemplo real y
  verificado del repo (Namespace, ResourceQuota, LimitRange,
  NetworkPolicy, Deployment, Service, Ingress, StatefulSet, Init
  container).
- notas-sueltas.md: MinIO/S3, limites reales del subscriber
  revalidate-storefront.ts, que se automatizaria desde cero.

Validado con `docker build --target build` (mkdocs build --strict,
mismo paso que el pipeline real) -- cero links rotos, cero warnings.
2026-08-26 23:51:18 -05:00
gitea-actions b7443a034f chore(gitops): deploy Docs Portal v1.0.173 [skip ci] 2026-08-27 04:46:06 +00:00
devops 4ee892a43a Merge fix/adr0005-cerrar-pendiente-demo-audit into main
Build and Push Docs Portal / Escaneo de Secretos (Gitleaks) (push) Successful in 22s
Build and Push Docs Portal / Construir y publicar Docs Portal (push) Successful in 5m1s
2026-08-26 23:40:53 -05:00
devops c6c66b76a3 docs(adr0005): cerrar pendiente de usuario admin temporal
[email protected] (creado para la auditoria de Etapa 1) fue
borrado 2026-08-27 via Medusa Admin UI -- ya no queda pendiente
abierto en el ADR.
2026-08-26 23:40:53 -05:00
devops 7406aff34f chore: release v1.0.172 [skip ci] 2026-08-27 04:29:21 +00:00
devops ba3e6c7322 Merge fix/adr0006-etapa2-fase3b-count-limit0-bug into main
Build and Push Frontend / Escaneo de Secretos (Gitleaks) (push) Successful in 17s
Build and Push Frontend / Construir y Subir Imagen (push) Successful in 8m22s
2026-08-26 23:20:40 -05:00
devops 884dbd1d93 fix(ecommerce): total de facetas no depende de count con limit=0
Verificado en vivo contra shop.cruzcloud.net: /store/search reporta
estimatedTotalHits (campo `count`) en 0 cuando limit=0, aunque
facetDistribution si trae los valores correctos (limit>=1 lo reporta
bien). El bucket "Todas"/"Todos" del sidebar de facetas (Fase 3b)
dependia de ese campo -- ahora suma los valores de la propia
facetDistribution, que da el total real sin ese problema.
2026-08-26 23:20:39 -05:00
devops 951682a4d4 chore: release v1.0.171 [skip ci] 2026-08-27 04:06:42 +00:00
devops 6b5450f16f Merge fix/adr0006-etapa2-fase3b-facetas-sidebar into main
Build and Push Frontend / Escaneo de Secretos (Gitleaks) (push) Successful in 37s
Build and Push Frontend / Construir y Subir Imagen (push) Successful in 10m53s
2026-08-26 22:55:13 -05:00
devops 83d448b282 feat(ecommerce): facetas reales de Meilisearch en el sidebar del catalogo
Fase 3b (ultima) de ADR 0006 Etapa 2: los conteos de categoria/marca/edad
del sidebar de /catalog dejan de ser un snapshot client-side fijo. Ahora
se refetchean en vivo contra /api/search en cada cambio de filtro (query
debounced 300ms), tres llamadas paralelas -- una por dimension, cada una
omitiendo su propio filtro para preservar el patron Amazon de "cuantos
resultados habria si eligiera este otro valor".

- catalog-client.tsx: useMemo client-side originales pasan a ser fallback
  (mock/shopify sin searchWithFacets, o cuando la categoria activa es un
  grupo raiz sintetico que Meilisearch no puede filtrar).
- ProductFilters gana `limit` opcional; /api/search y el provider medusa
  lo reenvian -- las llamadas de solo-conteo piden limit=0 para no
  rehidratar precios de productos que no se van a mostrar.
2026-08-26 22:53:49 -05:00
devops 1ecd2015a4 chore: release v1.0.170 [skip ci] 2026-08-26 22:38:31 +00:00
devops 609fc19252 Merge fix/adr0006-etapa2-fase3a-buscador-navbar into main
Build and Push Frontend / Escaneo de Secretos (Gitleaks) (push) Successful in 22s
Build and Push Frontend / Construir y Subir Imagen (push) Successful in 7m16s
2026-08-26 17:30:54 -05:00
devops 902350b58b Fase 3a ADR 0006 Etapa 2: buscador del navbar via Meilisearch
PredictiveSearch pasa de filtrar en memoria el arreglo completo de
productos (prop products, substring match) a un fetch debounced
(250ms, con AbortController) contra la nueva ruta /api/search --
mismo componente visual, solo cambia de donde salen los datos, tal
como pide el ADR 0006 seccion 5.

/api/search (nueva, server-side) es el puente same-origin: el
componente es cliente y no puede pegarle directo al provider
server-side (MEDUSA_BACKEND_URL es DNS interno del cluster, no
resuelve desde el navegador).

CatalogProvider gana un metodo OPCIONAL searchWithFacets? (no cambia
la firma de getProducts/searchProducts) -- mock.ts y shopify.ts
compilan sin tocarlos, siguen sin implementarlo. medusa.ts lo
implementa contra /store/search (Fase 2), reutilizando mapProduct()
ya que la ruta devuelve el mismo shape de producto que /store/products.
searchProducts tambien se repunta a /store/search, dejando de usar el
?q= + re-filtro client-side de fetchProducts/matchesFilters para busqueda
(que sigue sirviendo para navegacion sin busqueda, sin cambios).

Navbar deja de pasarle products a PredictiveSearch (ya no los necesita)
pero los sigue recibiendo el mismo -- collections todavia se deriva de
ahi.

Limpieza: borrado patches/frontend-medusa-env.yaml, confirmado
huerfano (ningun kustomization lo referencia; el patch que si esta
vivo es frontend/patches/deployment.yaml).

Validado: build completo de Docker (target builder) -- npm ci,
next build con typecheck, /api/search aparece en la tabla de rutas
generada. Cero errores nuevos (los warnings de ESLint son
preexistentes en catalog-client.tsx, no tocado en esta fase).
2026-08-26 17:30:47 -05:00
gitea-actions 7d2c03f10f chore(gitops): deploy Docs Portal v1.0.169 [skip ci] 2026-08-26 22:16:37 +00:00
devops 72ae63f2e3 Merge fix/adr0006-nota-implementacion-2-gate-fase2 into main
Build and Push Docs Portal / Escaneo de Secretos (Gitleaks) (push) Successful in 20s
Build and Push Docs Portal / Construir y publicar Docs Portal (push) Successful in 4m2s
2026-08-26 17:12:13 -05:00
devops 4afc63bdc6 docs(adr): nota de implementacion 2 -- hallazgos del gate en vivo de Fase 2 2026-08-26 17:12:10 -05:00
gitea-actions ed9891577f chore(gitops): deploy Medusa v1.0.168 [skip ci] 2026-08-26 22:03:37 +00:00
devops 41c7ad25e7 Merge fix/adr0006-etapa2-fase2-currency-code-search into main
Build and Push Medusa / Escaneo de Secretos (Gitleaks) (push) Successful in 18s
Build and Push Medusa / Construir y publicar Medusa (push) Successful in 11m17s
2026-08-26 16:52:06 -05:00
devops 2056e6c766 Fase 2 fix: /store/search resuelve currency_code, no solo region_id
Confirmado en vivo: pasar solo region_id a QueryContext no alcanza --
"Method calculatePrices requires currency_code in the pricing
context" persistia. A diferencia de /store/products nativo (resuelto
por el middleware setPricingContext), esta ruta propia no pasa por esa
cadena, asi que resuelve la moneda real de la region con un
query.graph adicional antes de pedir calculated_price -- no asume COP
fijo por si el catalogo suma otra region/moneda despues.
2026-08-26 16:52:04 -05:00
gitea-actions f2377be69c chore(gitops): deploy Medusa v1.0.167 [skip ci] 2026-08-26 21:44:33 +00:00
devops d6b84e4480 Merge fix/adr0006-etapa2-fase2-region-id-search into main
Build and Push Medusa / Escaneo de Secretos (Gitleaks) (push) Successful in 24s
Build and Push Medusa / Construir y publicar Medusa (push) Successful in 12m49s
2026-08-26 16:31:24 -05:00
devops f8f540af6a Fase 2 fix: /store/search acepta region_id para calculated_price
Mismo error encontrado en el gate de verificacion, ahora en la ruta
propia en vez del job de sync: "Method calculatePrices requires
currency_code in the pricing context" -- query.graph necesita
region_id/currency_code en el contexto para resolver
variants.calculated_price. El storefront ya manda region_id contra
/store/products (medusa.ts, regionId()); esta ruta ahora acepta el
mismo query param y arma el contexto de pricing con QueryContext(),
igual que la ruta oficial /store/meilisearch/products del plugin.
2026-08-26 16:31:21 -05:00
gitea-actions 51eadd98dc chore(gitops): deploy Medusa v1.0.166 [skip ci] 2026-08-26 21:23:08 +00:00
devops fbf0aec892 Merge fix/adr0006-etapa2-fase2-precio-crudo-indexado into main
Build and Push Medusa / Escaneo de Secretos (Gitleaks) (push) Successful in 29s
Build and Push Medusa / Construir y publicar Medusa (push) Successful in 17m43s
2026-08-26 16:05:03 -05:00
devops e3eca46e0e Fase 2 fix: indexar precio crudo (variants.prices), no calculated_price
Encontrado en vivo (gate de verificacion de Fase 2): el job de sync
inicial del plugin fallaba en TODOS los productos con "Method
calculatePrices requires currency_code in the pricing context" --
variants.calculated_price necesita region_id/currency_code en el
contexto de query.graph, que el workflow interno del plugin
(sync-products.js) no provee. Las categorias si indexaron bien (21),
solo productos fallaba, por eso numberOfDocuments quedaba en 0 pese a
que el indice y sus settings se crearon correctamente.

variants.prices (lista cruda, sin resolver promociones/reglas de
precio) no requiere ese contexto y alcanza para el filtro/orden por
precio de la busqueda -- no necesitamos el precio final promocional
ahi, esa resolucion ya la hace la ruta /store/search via query.graph
con el contexto real al re-hidratar los productos.
2026-08-26 16:04:59 -05:00
gitea-actions 586313d6d7 chore(gitops): deploy Medusa v1.0.165 [skip ci] 2026-08-26 20:56:17 +00:00
devops 77a15c26b1 Merge fix/adr0006-etapa2-fase2-plugin-medusa into main
Build and Push Medusa / Escaneo de Secretos (Gitleaks) (push) Successful in 19s
Build and Push Medusa / Construir y publicar Medusa (push) Successful in 16m14s
2026-08-26 15:39:41 -05:00
devops 6be8dd409c Fase 2 ADR 0006 Etapa 2: plugin de Meilisearch + ruta de facetas
Instala @rokmohar/medusa-plugin-meilisearch@^1.4.1 (linea v1, la unica
compatible con nuestro Medusa 2.18.0 exacto -- confirmado contra
peerDependencies reales de npm: @medusajs/* ^2.15.2, satisfecho por
2.18.0). Conectado via un array `plugins:` nuevo en medusa-config.ts
(primer uso en este repo, no `modules:`), env vars leidos con `?? ""`
en vez de required() -- si Meilisearch esta caido o mal configurado,
el plugin se degrada solo en vez de tumbar el arranque de Medusa
entero.

No se escribe un subscriber propio: se verifico contra el codigo
fuente real del paquete (descargado el tarball exacto de npm,
inspeccionado en /app/node_modules dentro de una imagen Docker de
verdad) que trae sus propios subscribers de product/category/
collection/variant/etc, descubiertos por Medusa en runtime desde
dentro del propio paquete -- confirmado que sobreviven a
`npm prune --omit=dev` y llegan intactos a la imagen final. Tambien
trae un job de indexado inicial de una sola corrida al boot
(numberOfExecutions: 1, no un cron recurrente -- no viola la
restriccion del ADR de "nunca cron").

Nueva ruta Store API propia (src/api/store/search/route.ts): las
rutas /store/meilisearch/* que trae el plugin no devuelven
facetDistribution (verificado leyendo su codigo fuente), asi que esta
ruta pide facets explicitamente via additionalOptions.facets, y
despues re-hidrata los productos via query.graph (mismos campos que
ya usa el storefront) en vez de confiar en precios/inventario
indexados -- mismo patron "safe path" que ya usa la ruta oficial del
plugin para /store/meilisearch/products.

Transformer custom en medusa-config.ts que espeja mapProduct() del
storefront (category/collection/brand/ageRange desde metadata,
disponibilidad desde la primera variante) para que el documento
indexado sea exactamente lo que el storefront ya muestra.

Validado en vivo: build completo de Docker (target builder) corre
limpio -- npm ci, medusa build (typecheck + compile backend/frontend),
npm prune -- cero errores. package-lock.json regenerado con un
contenedor node:22.18.0-bookworm-slim real (mismo base image que el
Dockerfile), no a mano.

Dockerfile: dummy MEILISEARCH_HOST/MEILISEARCH_API_KEY en el stage de
build (insurance barata, no son required() asi que no eran
estrictamente necesarios para que el build pasara, y de hecho paso
sin ellos en la prueba -- se dejan de todas formas por consistencia
con el resto de dummies de esa capa).
2026-08-26 15:39:30 -05:00
gitea-actions 8cd43a2584 chore(gitops): deploy Docs Portal v1.0.164 [skip ci] 2026-08-26 20:06:30 +00:00
devops 688a7d8e97 Merge fix/adr0006-etapa2-fase1-meilisearch-infra into main 2026-08-26 15:01:20 -05:00
devops 751817ed61 Fase 1 ADR 0006 Etapa 2: infra de Meilisearch (StatefulSet+Service)
Motor de busqueda self-hosted, sin que Medusa lo conozca todavia
(eso es Fase 2). Mismo patron que redis/postgres/minio en este mismo
namespace: StatefulSet + PVC via volumeClaimTemplates en local-path,
nunca Deployment ni emptyDir -- el indice de Meilisearch necesita
sobrevivir a un reinicio del pod para no reindexar todo desde cero.

Imagen getmeili/meilisearch:v1.53.1 (ultimo release estable no-RC,
verificado contra la API de GitHub, no memoria de entrenamiento).
Recursos exactos del ADR (request 100m/256Mi, limit 500m/768Mi).
Sin securityContext explicito -- verificado en vivo que la imagen no
trae un usuario no-root (Alpine minimo, /etc/passwd sin usuario
dedicado), misma convencion que redis.yaml/minio.yaml en este repo,
que tampoco fuerzan runAsNonRoot.

MEILISEARCH_HOST (no secreto) va al ConfigMap commerce-config.
MEILI_MASTER_KEY/MEILISEARCH_API_KEY (mismo valor, dos nombres de env
var -- uno para el servidor, otro para el futuro plugin de Medusa en
Fase 2) documentados en secrets.template.yaml; el valor real ya se
aplico al Secret commerce-secrets en el cluster antes de este merge
(patch puntual, no via create-commerce-secrets.sh completo -- ese
script regenera TODAS las credenciales en cada corrida y hubiera
rotado postgres/redis/minio ya en uso).
2026-08-26 15:01:04 -05:00
devops ebe143107d Merge fix/adr0006-motor-busqueda-facetas into main
Build and Push Docs Portal / Escaneo de Secretos (Gitleaks) (push) Successful in 21s
Build and Push Docs Portal / Construir y publicar Docs Portal (push) Successful in 8m52s
2026-08-26 14:56:25 -05:00
devops c243f02ec4 docs(adr): marcar ADR 0006 como aceptada, nota de implementación
Build and Push Docs Portal / Escaneo de Secretos (Gitleaks) (pull_request) Successful in 51s
Build and Push Docs Portal / Construir y publicar Docs Portal (pull_request) Skipped
Aprobada tras el diagnóstico de memoria del host (swap saturado
resuelto, ~500-700MiB liberados al desactivar monitoring). Agrega
nota de implementación con 3 desviaciones reales encontradas al
verificar el código fuente del plugin en el tag v1.4.1 antes de
escribir código: no hace falta subscriber propio (el plugin trae los
suyos), las rutas del plugin no devuelven facetDistribution (se
agrega una ruta Store API propia), y se conecta vía plugins: no
modules:. Ninguna cambia la decisión, solo el cómo.
2026-08-26 14:56:19 -05:00
devops 31782a2838 Merge main into fix/adr0006-motor-busqueda-facetas 2026-08-26 14:55:47 -05:00
devops 9069f165c0 Merge fix/disable-monitoring-stack into main 2026-08-26 14:08:28 -05:00
devops 60766ae98e Desactivar temporalmente el stack de monitoring (Grafana+Prometheus)
Sin uso actual, y el host está evaluando capacidad de memoria para la
Etapa 2 del ADR 0006 (Meilisearch) con swap bajo presión. Se quita del
app-of-apps para liberar los recursos del namespace `monitoring`
(Grafana, Prometheus, Alertmanager, kube-state-metrics,
prometheus-operator -- estos dos últimos venían en CrashLoopBackOff
desde hace 37 días, ~8000 reinicios cada uno, conexión rechazada
contra el apiserver in-cluster -- y los 3 node-exporter).

Reversible: reagregar ambos archivos re-despliega el stack completo
vía Helm/kustomize sin pasos manuales. Sin pérdida de datos -- el
namespace monitoring no tenía ningún PVC, confirmado antes de borrar
(kubectl -n monitoring get pvc -> vacío).

Se agregó el finalizer resources-finalizer.argocd.argoproj.io a ambas
Applications (monitoring-app, monitoring-governance) antes de este
commit para que Argo CD borre en cascada los recursos desplegados al
podarlas, no solo el objeto Application -- sin eso quedarían pods
huérfanos corriendo sin liberar memoria.
2026-08-26 14:08:12 -05:00
devops 69467d5f4d Merge fix/commerce-root-redirect-app into main 2026-08-26 13:33:19 -05:00
devops 0189f2369e commerce-ingress: redirect root to /app instead of 404ing
commerce.cruzcloud.net (medusa-svc) has no route at "/" — Medusa only
serves /health, /app (admin), /store, etc. lab-agent's generic public
domain check flagged the resulting 404 as an anomaly every cycle even
though the service is healthy.

Adds a Traefik Middleware (redirectRegex) scoped to the commerce host
that 301s "/" to "/app", so the DNS/routing layer owns the entry point
instead of every client (including monitoring) having to know to
append /app manually. Regex includes the host explicitly because the
same Ingress also routes media.cruzcloud.net and shares
router.middlewares at the Ingress-annotation level.
2026-08-26 13:29:41 -05:00
gitea-actions f5d09059f0 chore(gitops): deploy Docs Portal v1.0.162 [skip ci] 2026-08-26 05:38:36 +00:00
devops 56ab9d1862 Merge branch 'fix/lab-agent-telegram-horario' into main
Build and Push Docs Portal / Escaneo de Secretos (Gitleaks) (push) Successful in 40s
Build and Push Docs Portal / Construir y publicar Docs Portal (push) Successful in 8m43s
Documenta el cambio a Telegram (silencioso para info, hilos al
recuperarse, log adjunto), la pausa nocturna 06:00-23:00 con su
tensión sin resolver con la ventana de medusa-deploy, la activación
del timer de systemd (60 min, bajado de 15 para reducir costo) con el
gotcha de Environment=HOME= para systemd --user, y el costo mensual
estimado actualizado.
2026-08-26 00:29:13 -05:00
devops ce8c96d3ee Actualizar costo estimado tras bajar el timer a 60 min
17 corridas/día en vez de 68. Nota importante: el costo por corrida
sube al perder la ventaja del cache tibio de Claude Code (~1h TTL,
corridas espaciadas 60 min lo pierden casi siempre) -- el ahorro real
es ~1.5-2x, no 4x. También documenta el trade-off de latencia de
detección (~70 min en vez de ~25 min hasta detectar un problema).
2026-08-26 00:28:01 -05:00
devops 10e2db5643 Documentar activación del timer y el gotcha de HOME= en systemd --user
Timer activo desde 2026-08-26. Documenta el hallazgo en vivo: el
manager de systemd --user de devops arranca con HOME=/DATA (el de
/etc/passwd), no con el override de /etc/profile.d/zz-devops-home.sh
que solo aplica a shells interactivas -- sin Environment=HOME=... en
la unidad, claude -p habría fallado la autenticación en cada corrida
del timer, silenciosamente. Se detectó y corrigió antes de activar,
corriendo el servicio manualmente vía systemctl --user start con el
entorno real como último paso de validación.
2026-08-26 00:25:53 -05:00
devops 7c6f3d9c27 Documentar notificación silenciosa, hilos y adjunto de log en Telegram
Actualiza la sección de Notificaciones: formato con íconos por tipo de
evento, log completo adjunto como documento, nivel "info" silencioso
(disable_notification), y respuestas en hilo cuando un recurso
accionado se recupera solo -- con la limitación conocida de que un
desbloqueo manual (borrando la entrada de state.json) pierde el
message_id y no queda threadeado. Validado en vivo con el canario.
2026-08-26 00:21:28 -05:00
devops e493f7d39e Documentar cambio a Telegram, pausa nocturna y costo estimado
Actualiza docs/arquitectura/agente-monitoreo.md: canal de notificación
cambiado de ntfy a Telegram Bot API (con la evidencia de la prueba de
entrega), horario activo 06:00-23:00 con su justificación de costo, la
tensión sin resolver con la ventana de medusa-deploy (00:00-06:00, no
se solapa con el horario activo del agente), y la estimación de costo
mensual (~US$60-65, basada en las corridas reales medidas).
2026-08-26 00:01:08 -05:00
gitea-actions 680f3ba068 chore(gitops): deploy Docs Portal v1.0.161 [skip ci] 2026-08-26 04:29:35 +00:00
devops 5014903c51 Merge branch 'fix/lab-agent-docs-arquitectura' into main
Build and Push Docs Portal / Escaneo de Secretos (Gitleaks) (push) Successful in 3m25s
Build and Push Docs Portal / Construir y publicar Docs Portal (push) Successful in 9m42s
Documentación de gobernanza y arquitectura del agente lab-agent:
lista blanca aprobada, diseño de seguridad (--allowedTools "" en vez
de un Bash tool con patrón, por la vulnerabilidad de coincidencia por
prefijo encontrada en vivo), autenticación por login de suscripción, y
evidencia completa de la prueba end-to-end del 2026-08-26.
2026-08-25 23:16:25 -05:00
devops 0052dfa1da Documentar lab-agent: lista blanca, diseño de seguridad y prueba e2e
Agrega docs/arquitectura/agente-monitoreo.md como referencia técnica y
documento de gobernanza del wrapper headless de diagnóstico/corrección
(scripts/lab-agent). Cubre la lista blanca de acciones automáticas
aprobada, por qué Claude corre con --allowedTools "" en vez de un Bash
tool con prefijo (coincidencia por prefijo permite inyección de
comandos, confirmado en vivo), el rediseño de autenticación a login de
suscripción en vez de API key, y la evidencia completa de la prueba
end-to-end del 2026-08-26 (DRY_RUN, ejecución real, rate limit/bloqueo,
2 bugs encontrados y corregidos).
2026-08-25 23:11:10 -05:00
devops 5ce7030c4d docs(adr): propuesta de motor de busqueda y facetas (Fase 5 Etapa 3)
Build and Push Docs Portal / Escaneo de Secretos (Gitleaks) (pull_request) Successful in 32s
Build and Push Docs Portal / Construir y publicar Docs Portal (pull_request) Skipped
Etapa 1 (auditoria/propuesta, sin codigo) de la Etapa 3 de Fase 5:
motor de busqueda y filtros facetados reales, reemplazando el
filtrado/conteo 100% client-side que dejo la sub-fase 3 de Etapa 2.

Verificado contra documentacion oficial vigente (no memoria de
entrenamiento):
- Medusa no tiene modulo de busqueda propio integrado con
  Meilisearch; el Search Module nativo llego en Medusa 2.19, y el
  plugin comunitario @rokmohar/medusa-plugin-meilisearch se conecta a
  el solo desde su version ^2.0.0 (requiere Medusa >=2.19). Nuestro
  backend corre Medusa 2.18.0 -- la version compatible es la linea
  ^1.4.1 del plugin (Medusa >=2.15 <2.19), documentada como deuda
  tecnica explicita si se sube Medusa mas adelante.
- Impacto de recursos evaluado contra el estado real del host (swap
  al 100%, mismo tipo de contencion que el incidente de Gitea):
  Meilisearch recomendado sobre Typesense por menor huella de RAM y
  por ser el unico con integracion Medusa mantenida para nuestra
  version -- comparacion mantenida breve porque la diferencia es
  clara, sin profundizar mas de lo necesario.
- Diseno de indexacion (campos buscables/filtrables/ordenables),
  sincronizacion (mismo patron de subscriber por eventos que
  revalidate-storefront.ts, sumando product-category.* al set de
  eventos) y UI de facetas/autocompletado (reutiliza catalog-client.tsx
  y CatalogProvider.searchProducts ya existentes, sin componentes
  nuevos).

Cross-referencia agregada en ADR 0004 (que ya habia identificado
Meilisearch como paso natural) y entrada nueva en mkdocs.yml.

Etapa 2 (implementacion) NO arranca hasta aprobacion explicita de esta
propuesta.
2026-08-25 18:10:28 -05:00
gitea-actions 3b2f99c7a0 chore(gitops): deploy Docs Portal v1.0.159 [skip ci] 2026-08-25 21:08:06 +00:00
devops 00d7f58c9a Merge pull request 'docs(adr): marcar Etapa 2 del ADR 0005 como completa' (#43) from fix/adr0005-marcar-etapa2-completa into main
Build and Push Docs Portal / Escaneo de Secretos (Gitleaks) (push) Successful in 21s
Build and Push Docs Portal / Construir y publicar Docs Portal (push) Successful in 6m17s
2026-08-25 21:01:32 +00:00
devops 6f24109d91 docs(adr): marcar Etapa 2 del ADR 0005 como completa
Build and Push Docs Portal / Escaneo de Secretos (Gitleaks) (pull_request) Successful in 21s
Build and Push Docs Portal / Construir y publicar Docs Portal (pull_request) Skipped
Estado seguia diciendo "propuesta -- pendiente de aprobacion antes de
Etapa 2" aunque las 3 sub-fases (datos/categorias, navegacion,
componentes visuales -- PRs #29-#42) ya estan implementadas y en
produccion desde el 2026-08-25. Actualiza el header de estado y
reemplaza la seccion 6 (antes "Proximos pasos" pendiente de
aprobacion) por el resumen real de lo ejecutado, dejando explicito que
sigue pendiente borrar el usuario admin temporal de la auditoria y que
no hay Etapa 3 definida todavia.
2026-08-25 16:00:40 -05:00
devops edd14ca790 chore: release v1.0.157 [skip ci] 2026-08-25 20:29:09 +00:00
devops b6e3bddbc6 Merge pull request 'fix(ecommerce): corregir error de tipos en indice de tile promocional' (#42) from fix/etapa2-subfase3-fix-promo-index-type into main
Build and Push Frontend / Escaneo de Secretos (Gitleaks) (push) Successful in 20s
Build and Push Frontend / Construir y Subir Imagen (push) Successful in 9m29s
2026-08-25 20:19:26 +00:00
devops 3c0ce03e09 fix(ecommerce): corregir error de tipos en indice de tile promocional
Build and Push Frontend / Escaneo de Secretos (Gitleaks) (pull_request) Successful in 25s
Build and Push Frontend / Construir y Subir Imagen (pull_request) Skipped
tsconfig tiene noUncheckedIndexedAccess -- promoRoots[promoIndex %
promoRoots.length] tipa como Category | undefined aunque en runtime
siempre es valido (promoRoots.length > 0 ya se verifico antes de este
punto). Extraido a una constante con guard explicito para que TS
angoste el tipo. Sin cambio de comportamiento.

Detectado por el build real en main tras el merge de #41 -- no se
puede correr tsc localmente en esta maquina (ver notas de sesiones
anteriores), asi que esto solo lo valida el CI post-merge.
2026-08-25 15:18:12 -05:00
devops e7699befc5 Merge pull request 'fix(ecommerce): tiles promocionales intercalados en el grid del catalogo' (#41) from fix/etapa2-subfase3-tiles-promocionales into main
Build and Push Frontend / Escaneo de Secretos (Gitleaks) (push) Successful in 21s
Build and Push Frontend / Construir y Subir Imagen (push) Failing after 5m10s
2026-08-25 20:10:32 +00:00
devops 44d6e10ea2 chore: release v1.0.154 [skip ci] 2026-08-25 20:09:39 +00:00
devops f5de074ea1 Merge pull request 'fix(ecommerce): breadcrumb real + conteos por filtro en el catalogo' (#40) from fix/etapa2-subfase3-facetas-breadcrumb into main
Build and Push Frontend / Escaneo de Secretos (Gitleaks) (push) Successful in 26s
Build and Push Frontend / Construir y Subir Imagen (push) Successful in 13m4s
2026-08-25 19:56:04 +00:00
devops 8e74d0b516 chore: release v1.0.153 [skip ci] 2026-08-25 19:55:06 +00:00
devops e95ac086f0 Merge pull request 'fix(ecommerce): mensaje de urgencia por inventario real' (#39) from fix/etapa2-subfase3-urgencia-inventario into main
Build and Push Frontend / Escaneo de Secretos (Gitleaks) (push) Successful in 21s
Build and Push Frontend / Construir y Subir Imagen (push) Successful in 9m4s
2026-08-25 19:43:26 +00:00
devops 6a78257d71 chore: release v1.0.149 [skip ci] 2026-08-25 19:42:14 +00:00
devops 7a67ba0743 feat(ecommerce): tiles promocionales intercalados en el grid del catalogo
Build and Push Frontend / Escaneo de Secretos (Gitleaks) (pull_request) Successful in 23s
Build and Push Frontend / Construir y Subir Imagen (pull_request) Skipped
ADR 0005 seccion 3 (patron Walmart), ultimo item de la sub-fase de
componentes visuales: "grid de catalogo con tiles promocionales
intercalados, no solo tarjetas de producto homogeneas".

`category-tiles.tsx` ya era ese mismo primitivo visual (usado en el
home) -- se extrajo el tile individual a un componente exportado
`PromoTile` (mismo look, sin cambio visual en el home) para
reutilizarlo en `catalog-client.tsx`: un tile cada
PROMO_TILE_INTERVAL=8 productos, ciclando por las categorias raiz
(excluye la raiz que el usuario ya esta viendo via el breadcrumb de
#40). Si no hay jerarquia de categorias disponible, la grilla queda
igual que antes (solo productos) -- degradacion sin romper nada.

Depende de #40 (breadcrumb + categoryNodes en CatalogClient) -- mismo
archivo, se apila sobre esa rama en vez de main.
2026-08-25 14:40:52 -05:00
devops 8fc9df3b50 feat(ecommerce): mensaje de urgencia por inventario real
Build and Push Frontend / Escaneo de Secretos (Gitleaks) (pull_request) Successful in 26s
Build and Push Frontend / Construir y Subir Imagen (pull_request) Skipped
Expone ProductVariant.inventoryQuantity (ya calculado desde
variant.inventory_quantity en Medusa, hoy colapsado al booleano
available) y lo usa para mostrar "Quedan N" cuando el stock real es
bajo (<=5), tanto en la tarjeta de producto como en la ficha. null
indica inventario no gestionado (backorder / manage_inventory=false),
donde nunca se muestra un numero. ADR 0005 seccion 3 (patron eBay).

Catalogo mock actualizado (data/products.json) con
inventoryQuantity=null en todas las variantes para seguir tipando
contra Product[] via `satisfies`.
2026-08-25 14:33:14 -05:00
41 changed files with 2098 additions and 251 deletions

No files matched your search

-40
View File
@@ -1,40 +0,0 @@
apiVersion: argoproj.io/v1alpha1
kind: Application
metadata:
name: monitoring-app
namespace: argocd
spec:
project: default
source:
repoURL: https://prometheus-community.github.io/helm-charts
chart: kube-prometheus-stack
targetRevision: 58.3.3
helm:
parameters:
- name: prometheusOperator.admissionWebhooks.patch.ttlSecondsAfterFinished
value: "60"
valuesObject:
grafana:
ingress:
enabled: true
ingressClassName: traefik
hosts:
- grafana.cruzcloud.net
destination:
server: https://kubernetes.default.svc
namespace: monitoring
syncPolicy:
automated:
prune: true
selfHeal: true
syncOptions:
- CreateNamespace=true
- ServerSideApply=true
-20
View File
@@ -1,20 +0,0 @@
apiVersion: argoproj.io/v1alpha1
kind: Application
metadata:
name: monitoring-governance
namespace: argocd
spec:
project: default
source:
repoURL: https://gitea.cruzcloud.net/devops/platform-infra.git
path: monitoring-governance
targetRevision: main
destination:
server: https://kubernetes.default.svc
namespace: monitoring
syncPolicy:
automated:
prune: true
selfHeal: true
syncOptions:
- CreateNamespace=true
+3 -1
View File
@@ -21,7 +21,9 @@ ENV CI=true \
S3_ACCESS_KEY_ID=build-access-key \
S3_SECRET_ACCESS_KEY=build-secret-key \
S3_BUCKET=ari-shopping \
S3_ENDPOINT=http://127.0.0.1:9000
S3_ENDPOINT=http://127.0.0.1:9000 \
MEILISEARCH_HOST=http://127.0.0.1:7700 \
MEILISEARCH_API_KEY=build-only-meili-key
COPY package.json package-lock.json .npmrc ./
+153
View File
@@ -10,6 +10,91 @@ const required = (name: string): string => {
return value
}
// Mismos campos derivados de metadata/categories/collection/variants que
// mapProduct() en workloads/ecommerce/lib/headless/providers/medusa.ts --
// el documento indexado debe reflejar exactamente lo que el storefront
// ya muestra, no una vista distinta del catálogo.
function stringMetadata(
metadata: Record<string, unknown> | null | undefined,
key: string,
): string {
const value = metadata?.[key]
return typeof value === "string" ? value : ""
}
function booleanMetadata(
metadata: Record<string, unknown> | null | undefined,
key: string,
): boolean {
const value = metadata?.[key]
return value === true || value === "true"
}
async function meilisearchProductTransformer(
document: Record<string, unknown>,
) {
const metadata = (document.metadata ?? null) as Record<
string,
unknown
> | null
const categories = document.categories as
| Array<{ name?: string | null }>
| null
| undefined
const collection = document.collection as { title?: string | null } | null
const variants = document.variants as
| Array<{
inventory_quantity?: number | null
manage_inventory?: boolean | null
allow_backorder?: boolean | null
prices?: Array<{
amount?: number | null
currency_code?: string | null
}> | null
}>
| null
| undefined
const variant = variants?.[0]
const tags = document.tags as Array<{ value?: string | null }> | null
const category =
categories?.[0]?.name || stringMetadata(metadata, "category") || "Variedades"
const available =
!variant ||
variant.allow_backorder === true ||
variant.manage_inventory === false ||
(variant.inventory_quantity ?? 0) > 0
// variants.calculated_price requiere currency_code/region_id en el
// contexto de pricing, que el workflow de sync interno del plugin no
// provee -- hace fallar TODO el job de indexado (confirmado en vivo:
// "Method calculatePrices requires currency_code in the pricing
// context"). variants.prices es la lista cruda de precios (sin
// resolver promociones/reglas), no necesita contexto -- alcanza para
// el filtro/orden por precio en la busqueda, que no necesita el
// precio final promocional.
const copPrice = variant?.prices?.find(
(price) => price.currency_code?.toLowerCase() === "cop",
)
return {
id: document.id,
name: document.title || "Producto ARI Shopping",
shortDescription: document.subtitle || document.title || "",
description: document.description || "",
category,
collection: collection?.title || stringMetadata(metadata, "collection") || "",
brand: stringMetadata(metadata, "brand") || "ARI Shopping",
ageRange: stringMetadata(metadata, "ageRange") || "Todas",
tags: (tags || []).map((tag) => tag.value).filter(Boolean),
price: typeof copPrice?.amount === "number" ? copPrice.amount : null,
available,
isNew: booleanMetadata(metadata, "isNew"),
featured: booleanMetadata(metadata, "featured"),
}
}
module.exports = defineConfig({
projectConfig: {
databaseUrl: required("DATABASE_URL"),
@@ -73,4 +158,72 @@ module.exports = defineConfig({
},
},
],
// ADR 0006 Etapa 2. Sin required(): a diferencia de S3/DB, Meilisearch
// es un servicio nuevo y todavia no probado en produccion en un host
// con historial de swap saturado -- si esta caido o mal configurado,
// el plugin se degrada solo (soporta enabled:false, "silently skipped"
// segun su propio README) en vez de tumbar el arranque de Medusa
// entero, que es lo que si haria required().
plugins: [
{
resolve: "@rokmohar/medusa-plugin-meilisearch",
options: {
config: {
host: process.env.MEILISEARCH_HOST ?? "",
apiKey: process.env.MEILISEARCH_API_KEY ?? "",
},
settings: {
products: {
type: "products",
enabled: Boolean(process.env.MEILISEARCH_HOST),
primaryKey: "id",
// Campos que el plugin pide via query.graph antes de pasarlos
// al transformer de arriba -- si falta alguno acá, llega
// undefined al transformer sin importar lo que este haga.
fields: [
"id",
"title",
"subtitle",
"description",
"handle",
"status",
"thumbnail",
"metadata",
"tags.value",
"categories.name",
"collection.title",
"variants.inventory_quantity",
"variants.manage_inventory",
"variants.allow_backorder",
"variants.prices.amount",
"variants.prices.currency_code",
],
indexSettings: {
// Tabla de ADR 0006 §3: Buscable / Filtrable-faceta / Ordenable.
searchableAttributes: [
"name",
"shortDescription",
"description",
"category",
"collection",
"brand",
"tags",
],
filterableAttributes: [
"category",
"collection",
"brand",
"ageRange",
"price",
"available",
],
sortableAttributes: ["price", "isNew", "featured"],
},
transformer: meilisearchProductTransformer,
},
},
},
},
],
})
+30 -1
View File
@@ -12,7 +12,8 @@
"@medusajs/admin-sdk": "2.18.0",
"@medusajs/cli": "2.18.0",
"@medusajs/framework": "2.18.0",
"@medusajs/medusa": "2.18.0"
"@medusajs/medusa": "2.18.0",
"@rokmohar/medusa-plugin-meilisearch": "^1.4.1"
},
"devDependencies": {
"@swc/core": "^1.7.28",
@@ -6561,6 +6562,28 @@
"node": ">=14.0.0"
}
},
"node_modules/@rokmohar/medusa-plugin-meilisearch": {
"version": "1.4.3",
"resolved": "https://registry.npmjs.org/@rokmohar/medusa-plugin-meilisearch/-/medusa-plugin-meilisearch-1.4.3.tgz",
"integrity": "sha512-+iStv5sc2vuLk+O7OcQLlNR99fMBhGdBOJMCmDJYoTFGhzBA4ZEV18IIr2xqxUDgdQo1haF0TtYv3JmRQUuY1w==",
"license": "MIT",
"dependencies": {
"meilisearch": "^0.56.0"
},
"engines": {
"node": ">=22"
},
"peerDependencies": {
"@medusajs/admin-sdk": "^2.15.2",
"@medusajs/framework": "^2.15.2",
"@medusajs/medusa": "^2.15.2",
"@medusajs/ui": "^4.1.12",
"@tanstack/react-query": "5.64.2",
"awilix": "^8.0.1",
"react": "^18",
"react-dom": "^18"
}
},
"node_modules/@rolldown/pluginutils": {
"version": "1.0.0-beta.27",
"resolved": "https://registry.npmjs.org/@rolldown/pluginutils/-/pluginutils-1.0.0-beta.27.tgz",
@@ -11774,6 +11797,12 @@
"node": ">= 0.6"
}
},
"node_modules/meilisearch": {
"version": "0.56.0",
"resolved": "https://registry.npmjs.org/meilisearch/-/meilisearch-0.56.0.tgz",
"integrity": "sha512-kBXba8DcSrLWHYqopCm2JL90oBy97VdIfHkP1ii7/eHufeEEEk9Zu5Vv/IFQpNzmhEpWszVFVlGvZoI6QJeGzA==",
"license": "MIT"
},
"node_modules/merge-descriptors": {
"version": "1.0.3",
"resolved": "https://registry.npmjs.org/merge-descriptors/-/merge-descriptors-1.0.3.tgz",
+2 -1
View File
@@ -14,7 +14,8 @@
"@medusajs/admin-sdk": "2.18.0",
"@medusajs/cli": "2.18.0",
"@medusajs/framework": "2.18.0",
"@medusajs/medusa": "2.18.0"
"@medusajs/medusa": "2.18.0",
"@rokmohar/medusa-plugin-meilisearch": "^1.4.1"
},
"devDependencies": {
"@swc/core": "^1.7.28",
@@ -0,0 +1,197 @@
import type { MedusaRequest, MedusaResponse } from "@medusajs/framework";
import {
ContainerRegistrationKeys,
QueryContext,
} from "@medusajs/framework/utils";
import {
MEILISEARCH_MODULE,
type MeiliSearchService,
} from "@rokmohar/medusa-plugin-meilisearch";
// Las rutas /store/meilisearch/* que trae el plugin (ver ADR 0006 nota de
// implementacion) no devuelven facetDistribution -- necesitamos esta
// ruta propia para pedirlo explicitamente via additionalOptions.facets,
// que el servicio del plugin reenvia tal cual al cliente Meilisearch.
const FACETS = ["category", "collection", "brand", "ageRange"];
// Mismos campos que ya usa el storefront contra /store/products
// (workloads/ecommerce/lib/headless/providers/medusa.ts) -- una vez que
// Meilisearch nos da los ids candidatos, volvemos a Medusa via
// query.graph para precio/inventario reales, nunca confiamos en el
// valor indexado (puede tener unos segundos de retraso respecto al
// evento de sincronizacion).
const PRODUCT_FIELDS = [
"id",
"handle",
"title",
"subtitle",
"description",
"thumbnail",
"external_id",
"metadata",
"images.url",
"tags.value",
"categories.name",
"collection.title",
"variants.id",
"variants.title",
"variants.sku",
"variants.manage_inventory",
"variants.allow_backorder",
"variants.inventory_quantity",
"variants.calculated_price.calculated_amount",
"variants.calculated_price.original_amount",
"variants.calculated_price.currency_code",
"variants.options.value",
"variants.options.option.title",
];
function stringParam(
query: MedusaRequest["query"],
name: string,
): string | undefined {
const value = query[name];
return typeof value === "string" && value.length > 0 ? value : undefined;
}
function numberParam(
query: MedusaRequest["query"],
name: string,
): number | undefined {
const value = stringParam(query, name);
if (!value) return undefined;
const parsed = Number(value);
return Number.isFinite(parsed) ? parsed : undefined;
}
// Meilisearch usa comillas dobles para valores de filtro con espacios;
// escapamos las que vengan en el propio valor para no romper la sintaxis.
function escapeFilterValue(value: string): string {
return value.replace(/\\/g, "\\\\").replace(/"/g, '\\"');
}
export async function GET(req: MedusaRequest, res: MedusaResponse) {
const meilisearchService = req.scope.resolve<MeiliSearchService>(
MEILISEARCH_MODULE,
);
const query = req.scope.resolve(ContainerRegistrationKeys.QUERY);
const q = stringParam(req.query, "q") ?? "";
const limit = numberParam(req.query, "limit") ?? 24;
const offset = numberParam(req.query, "offset") ?? 0;
// calculated_price exige region_id/currency_code en el contexto de
// pricing o query.graph tira "Method calculatePrices requires
// currency_code in the pricing context" (mismo error que ya rompio el
// job de sync -- ver medusa-config.ts). El storefront ya manda
// region_id contra /store/products (medusa.ts, regionId()); esta ruta
// espera el mismo query param.
const regionId = stringParam(req.query, "region_id");
const filterParts: string[] = [];
for (const facet of ["category", "collection", "brand", "ageRange"]) {
const value = stringParam(req.query, facet);
if (value) {
filterParts.push(`${facet} = "${escapeFilterValue(value)}"`);
}
}
const minPrice = numberParam(req.query, "minPrice");
const maxPrice = numberParam(req.query, "maxPrice");
if (typeof minPrice === "number") filterParts.push(`price >= ${minPrice}`);
if (typeof maxPrice === "number") filterParts.push(`price <= ${maxPrice}`);
const indexes = await meilisearchService.getIndexesByType("products");
if (indexes.length === 0) {
// Meilisearch no configurado o el indice "products" esta
// deshabilitado -- degradar a "sin resultados" en vez de 500, el
// frontend cae de nuevo a su propio filtrado si esta ruta falla.
res.json({ products: [], facetDistribution: {}, count: 0, limit, offset });
return;
}
const results = await Promise.all(
indexes.map((indexKey) =>
meilisearchService.search(indexKey, q, {
filter: filterParts.length ? filterParts.join(" AND ") : undefined,
paginationOptions: { limit, offset },
additionalOptions: { facets: FACETS },
}),
),
);
const hits = results.flatMap((result) => result.hits);
const totalCount = results.reduce(
(sum, result) => sum + result.estimatedTotalHits,
0,
);
const facetDistribution: Record<string, Record<string, number>> = {};
for (const result of results) {
const resultFacets = (
result as unknown as {
facetDistribution?: Record<string, Record<string, number>>;
}
).facetDistribution;
if (!resultFacets) continue;
for (const [facet, values] of Object.entries(resultFacets)) {
facetDistribution[facet] ??= {};
for (const [value, count] of Object.entries(values)) {
facetDistribution[facet][value] =
(facetDistribution[facet][value] ?? 0) + count;
}
}
}
const productIds = hits.map((hit) => (hit as { id: string }).id);
if (productIds.length === 0) {
res.json({ products: [], facetDistribution, count: 0, limit, offset });
return;
}
// calculatePrices exige currency_code en el contexto -- no alcanza con
// region_id solo (a diferencia de /store/products nativo, que lo
// resuelve el middleware setPricingContext; esta ruta no pasa por esa
// cadena). Resolvemos la moneda real de la region en vez de asumir COP
// fijo, por si el catalogo suma otra region/moneda mas adelante.
let currencyCode: string | undefined;
if (regionId) {
const { data: regions = [] } = await query.graph({
entity: "region",
fields: ["id", "currency_code"],
filters: { id: regionId },
});
currencyCode = regions[0]?.currency_code;
}
const { data: products = [] } = await query.graph({
entity: "product",
fields: PRODUCT_FIELDS,
filters: { id: productIds },
context:
regionId && currencyCode
? {
variants: {
calculated_price: QueryContext({
region_id: regionId,
currency_code: currencyCode,
}),
},
}
: {},
});
// Preserva el ranking de relevancia de Meilisearch -- query.graph no
// garantiza el mismo orden que la busqueda.
const orderedProducts = [...products].sort(
(a, b) => productIds.indexOf(a.id) - productIds.indexOf(b.id),
);
res.json({
products: orderedProducts,
facetDistribution,
count: totalCount,
limit,
offset,
});
}
@@ -1,20 +1,9 @@
# Notas sueltas / aprendizajes
> **TODO (fase 2+):** esto es un scratchpad estructurado, no un
> playbook — va creciendo con notas cortas a medida que aparecen. Las
> entradas de abajo son semillas reales extraídas de incidentes ya
> documentados; expandir cada una con más contexto en fases siguientes.
## Qué haría distinto
- **Fijar `TUNNEL_TRANSPORT_PROTOCOL=http2` desde el día uno**, no
después de un incidente — QUIC/UDP sobre NAT doméstico fue un problema
predecible en retrospectiva. Ver [ADR 0003](../decisiones/0003-por-que-http2-sobre-quic-tunnel.md).
- **Separar siempre "chequeo de disponibilidad" de "migraciones"** en
init containers distintos, y que el chequeo use el mínimo de
parámetros posible (host/puerto/usuario/db) en vez de reusar una URI
de conexión pensada para la app. Confirmado como decisión correcta en
el [incidente de crashloop de Medusa](../playbooks/incidente-crashloop-medusa.md).
> Esto es un scratchpad estructurado, no un playbook — sigue creciendo
> con notas cortas a medida que aparecen. Cada entrada está extraída de
> un incidente o hallazgo real ya documentado en otra parte del sitio,
> no es especulación.
## Gotchas de Medusa v2
@@ -26,8 +15,70 @@
del array `images` — hay que setearlo explícito desde el Admin si el
frontend lo usa como imagen hero en listados.
## TODO
## MinIO / S3 provider
- Notas sobre MinIO / S3 provider.
- Notas sobre el subscriber `revalidate-storefront.ts` y sus límites.
- Qué automatizaría si volviera a montar este lab desde cero.
- Medusa sube imágenes de producto a MinIO (`minio-svc:9000`, bucket
`ari-products`) usando el provider S3-compatible estándar — MinIO no
sabe que no es AWS real, y Medusa tampoco necesita saberlo.
- La URL pública que termina viendo el navegador **no** es la interna
(`minio-svc:9000`): es `S3_FILE_URL=https://media.cruzcloud.net/ari-products`,
un dominio propio proxied por NPM hacia el mismo MinIO. Esto tiene
una consecuencia directa que ya rompió producción una vez: si
`media.cruzcloud.net` deja de rutear bien (mismo patrón de
`forward_host` roto que el resto del sitio), **todas** las imágenes
de producto del storefront se caen aunque MinIO y Medusa estén
perfectamente sanos — porque el navegador nunca habla con MinIO
directo, solo con esa URL pública. Ver [Red y exposición](../arquitectura/red-y-exposicion.md).
## Subscriber `revalidate-storefront.ts` y sus límites
- Escucha 6 eventos de Medusa (`product.created/updated/deleted` +
los 3 equivalentes de `product-variant`) y por cada uno hace un
único `POST` a `${STOREFRONT_URL}/api/revalidate` con un header
secreto (`x-revalidate-secret`) — esa ruta del frontend llama
`revalidateTag("medusa-products")` de Next.js.
- **Se degrada solo, no tira el evento entero**: si
`STOREFRONT_URL`/`REVALIDATE_SECRET` no están configurados, o el
`fetch` falla, solo loguea un `warn` y sigue — un fallo de red hacia
el frontend nunca revierte ni bloquea la operación real en Medusa
(crear/editar un producto). Mismo criterio de "degradar, no romper"
que se repite en el resto del storefront (`searchWithFacets`
opcional, `/api/search` cayendo a `{}` si Meilisearch no responde).
- **Límite real, todavía no un problema**: no hay debounce ni
deduplicación — si una operación dispara varios eventos juntos (ej.
editar un producto con 3 variantes a la vez), son varios `POST`
independientes al mismo endpoint. `revalidateTag` de Next.js es
barato e idempotente, así que hoy no se nota, pero si el catálogo
creciera mucho y las ediciones masivas se volvieran frecuentes, sería
el primer lugar donde agregar algún tipo de agrupación.
## Qué haría distinto si volviera a montar este lab desde cero
- **Fijar `TUNNEL_TRANSPORT_PROTOCOL=http2` desde el día uno** (ver
[ADR 0003](../decisiones/0003-por-que-http2-sobre-quic-tunnel.md)) y
**ajustar el MTU de `flannel.1` desde el primer `deploy-lab.sh`** —
ambos terminaron siendo incidentes reales
(504 intermitente, blackhole cross-node) que en retrospectiva eran
predecibles: red doméstica con NAT agresivo + Docker-in-Docker
multi-nodo son una combinación conocida para tener problemas de MTU
y de transporte UDP. Hoy los dos ya están automatizados
(`fix_flannel_mtu` en `deploy-lab.sh`, la variable de entorno del
túnel), pero se aprendieron reactivamente, no se anticiparon.
- **Usar `git merge-base --is-ancestor` para la promoción segura del
CI desde el primer pipeline**, no después de ver la condición de
carrera en vivo (dos pipelines corriendo en serie sobre el mismo
commit, el segundo pisando o saltándose la promoción del primero).
El patrón correcto no es complicado una vez que se conoce — el costo
fue no anticiparlo.
- **Separar siempre "chequeo de disponibilidad" de "migraciones"** en
init containers distintos, y que el chequeo use el mínimo de
parámetros posible (host/puerto/usuario/db) en vez de reusar una URI
de conexión pensada para la app. Confirmado como decisión correcta en
el [incidente de crashloop de Medusa](../playbooks/incidente-crashloop-medusa.md).
- **No escribir un script que regenere credenciales completas en cada
corrida** (`create-commerce-secrets.sh` rota JWT/COOKIE/passwords de
Postgres/Redis/MinIO enteros cada vez que se ejecuta, incluso si solo
hacía falta agregar una clave nueva) — un script de setup inicial y
uno de "agregar una clave a un secret ya vivo" deberían haber sido
dos herramientas distintas desde el principio, no una sola que hay
que saber cuándo NO correr completa.
@@ -0,0 +1,307 @@
# Agente de diagnóstico/corrección (lab-agent)
Wrapper headless (`scripts/lab-agent/lab-agent.sh`) que corre `claude -p`
periódicamente para diagnosticar el estado del lab y, solo para un
conjunto explícito y aprobado de recursos, ejecutar una acción
correctiva de bajo riesgo. Este documento es tanto referencia técnica
como decisión de gobernanza — la lista blanca de abajo es lo que
autoriza al agente a actuar sin intervención humana, y debe poder
auditarse.
Estado: **timer de systemd ACTIVO desde 2026-08-26** (`lab-agent.timer`,
usuario `devops`, cada 60 min entre 06:00–23:00 hora local — bajado de
15 a 60 min el mismo día para reducir costo). Script,
Telegram y las unidades systemd, todo validado en vivo antes de
activar.
## Lista blanca de acciones correctivas automáticas
Aprobada 2026-08-25, antes de escribir una sola línea de código.
### 1. `kubectl rollout restart deployment/<nombre>`
| Condición | Valor |
|---|---|
| Trigger | Deployment con `readyReplicas < replicas` o condición `Available=False` durante **≥10 minutos consecutivos** |
| Namespace elegible | `ecommerce` (todos los deployments detectados ahí en el momento de la corrida) |
| Excepción | `medusa-deploy` — el deployment real de "commerce-backend" en la terminología de `CLAUDE.md` (confirmado en vivo 2026-08-25: no existe un deployment llamado literalmente `commerce-backend`) — solo elegible si además la hora **local** del host está entre **00:00–06:00** |
| Namespaces excluidos siempre | `argocd`, `kube-system`, cualquier otro namespace de plataforma |
### 2. `docker restart <contenedor>`
| Condición | Valor |
|---|---|
| Contenedores elegibles | **Únicamente `gitea-runner`** |
| Trigger | Contenedor no está en estado `running` durante ≥10 min, **o** la API de Gitea Actions reporta un run en estado `running` atascado por más de 10 min sin terminar (señal del incidente de runner zombie del 2026-08-16) |
| Degradación segura | Si no hay `GITEA_TOKEN_FILE` configurado o la API falla, esa señal se omite — nunca dispara una acción sobre una lectura no verificada, solo queda la señal de "contenedor caído" |
| Excluidos siempre | `gitea`, `nginxproxymanager`, `cloudflared` |
### 3. Fuera de alcance automático — SIEMPRE (solo diagnóstico + notificación)
- Borrar cualquier recurso, escalar réplicas, tocar `ConfigMap`/`Secret`/configuración.
- Reiniciar `gitea`, `nginxproxymanager`, `cloudflared`.
- Cualquier acción sobre `medusa-deploy` fuera de la ventana 00:00–06:00.
- Cualquier namespace/deployment no detectado explícitamente en `ecommerce`.
### 4. Límites obligatorios
- **Rate limit:** máx. 1 acción correctiva por recurso cada 30 min.
- **Sin reintento silencioso:** tras un primer intento cuyo problema no
se resuelve, el recurso queda `blocked` — solo notificación, hasta
que un humano limpie la entrada en `state.json` o el recurso se
recupere solo.
- **Logging completo:** cada decisión (actúe o no) se registra en
`actions.jsonl` con timestamp, recurso, decisión, resultado y motivo.
- **Notificación siempre:** toda acción correctiva o anomalía dispara
notificación — nunca es silenciosa.
## Diseño de seguridad: por qué Claude nunca ejecuta comandos
`claude -p` se invoca con `--allowedTools ""` (cero herramientas) y
`--json-schema`, así que el modelo **nunca** ejecuta un comando —
solo devuelve una decisión estructurada
(`{"actions":[{"resource","decision","reason"}], "summary"}`). El
script bash es el único que ejecuta `kubectl`/`docker`, y solo después
de revalidar la decisión contra su propio cálculo de elegibilidad —
nunca confía en el JSON de Claude por sí solo.
Se descartó dar a Claude un `Bash` tool con un patrón tipo
`Bash(kubectl rollout restart deployment/medusa-deploy -n ecommerce)`
porque se comprobó en vivo (2026-08-25) que `--allowedTools` hace
**coincidencia por prefijo, no exacta**: un patrón `Bash(echo pintest)`
también permite ejecutar `echo pintest && echo INJECTED` o
`echo pintest; rm -rf /`. Para un comando root-capaz eso no es un
límite seguro. Con `--allowedTools ""`, el peor caso posible es que
Claude proponga una decisión incorrecta en el JSON — y el wrapper la
rechaza si el recurso no está en su propia lista de elegibles.
El prompt también advierte contra prompt injection: los datos de
diagnóstico (estado de pods, logs) podrían contener texto arbitrario
si algo en el cluster está comprometido, e instruye a Claude a ignorar
cualquier instrucción incrustada ahí.
## Autenticación: login de suscripción, no API key
`ANTHROPIC_API_KEY_FILE` queda vacío por defecto — `claude -p` usa el
login de suscripción ya activo del usuario que corre el script
(`~/.claude/.credentials.json`), el mismo que usa una sesión
interactiva normal. Esta decisión se tomó en vivo (2026-08-26) al
confirmar que no había una API key real disponible; el diseño original
pedía un secreto root-only para la API key, pero se simplificó porque:
1. No hacía falta una API key aparte — la suscripción ya cubre esto.
2. Elimina la necesidad de crear secretos root-only vía `sudo` (que
además está bloqueado para una sesión de Claude Code interactiva
por diseño — ver `feedback_scoped_root_access_pattern` en memoria
del proyecto).
**Consecuencia directa:** el servicio systemd (cuando se active) debe
correr como usuario `devops`, no como `root` — root no tiene acceso a
ese login. `CLAUDE_BIN`/`KUBECTL_BIN` apuntan por ruta absoluta a
`/DATA/AppData/home-devops/.local/bin/` porque ese directorio no está
en el `PATH` de root ni de systemd (confirmado en vivo: `sudo which
claude` no lo encuentra).
El único secreto que queda es `GITEA_TOKEN_FILE` (token de Gitea
Actions, opcional, scope lectura), dueño `devops`, `600` — sin sudo.
## Prueba end-to-end (2026-08-26)
Ejecutada contra el cluster real, rama `fix/lab-agent-diagnostico-headless`.
Umbral bajado a 2 minutos solo para esta prueba (`UNHEALTHY_THRESHOLD_MIN`,
valor real en producción: 10). Canario desechable: `deployment/test-agent-canary`
en `ecommerce`, imagen `busybox:stable` con `command: ["sh","-c","exit 1"]`
— garantiza `CrashLoopBackOff` predecible, sin relación con
`catalog-api-deploy`/`frontend-deploy`/`medusa-deploy`.
### Bugs encontrados y corregidos durante la prueba
1. **Subshell perdía el resultado de la acción.** `execute_corrective_action`
se invocaba como `result="$(execute_corrective_action "$resource")"` —
al correr en un subshell, las variables `LAST_ACTION_CMD`/`STDOUT`/`STDERR`
nunca volvían al script principal → crash con `unbound variable`.
Corregido: la función ahora devuelve éxito/fallo por exit status, invocada
directamente (sin `$(...)`) para que sus variables globales persistan.
2. **DRY_RUN contaminaba el estado real.** El wrapper marcaba `blocked=true`
en `state.json` incluso en modo simulación, lo que habría bloqueado
incorrectamente una corrida real posterior sobre el mismo recurso.
Corregido: la actualización de `state.json` (bloqueo/rate-limit) solo
ocurre fuera de `DRY_RUN`.
3. **`claude`/`kubectl` fuera del PATH de root/systemd** — ver sección de
autenticación arriba. Corregido con `CLAUDE_BIN`/`KUBECTL_BIN` por ruta
absoluta.
### Evidencia por paso
**DRY_RUN (elegibilidad + decisión, sin tocar el cluster):**
- 1ª corrida (recién detectado): `elegibilidad=not_eligible:bajo_umbral(0m<2m)`
→ Claude devolvió `notify_only` para los 5 recursos.
- 2ª corrida (&gt;2 min después): `elegibilidad=eligible` para
`test-agent-canary` → Claude devolvió `decision:"restart"` solo para
ese recurso, `notify_only` para los otros 4 (sanos).
- Log: `ACCIÓN EJECUTADA: kubectl -n ecommerce rollout restart
deployment/test-agent-canary -> OK`, pero verificado en el cluster real
que **no hubo restart** (sin anotación `restartedAt`, mismo pod, misma
edad). `actions.jsonl` registra `"dry_run": true`.
**Ejecución real (`--live`):**
- Misma decisión de Claude (`restart` para el canario, `notify_only` para
el resto).
- Verificado en el cluster: nueva anotación
`kubectl.kubernetes.io/restartedAt` en el deployment, evento
`ScalingReplicaSet` de un ReplicaSet **nuevo**, pod nuevo en otro nodo.
- `actions.jsonl`: `"result": "success", "dry_run": false`.
- `state.json`: `last_action_at` seteado, `blocked: true`.
**Rate limit / sin reintento silencioso:**
- El canario (siempre crashea por diseño) volvió a estar unhealthy sin
intervención extra.
- Corrida siguiente: `elegibilidad=not_eligible:blocked_tras_intento_previo_sin_recuperacion`.
- Claude devolvió `notify_only` para **todos** los recursos, incluido el
canario — ningún segundo restart propuesto ni ejecutado.
- `state.json` sin cambios (mismo `last_action_at` que la corrida real
anterior) — confirma que no hubo reintento.
- Notificación real entregada vía ntfy.sh, confirmada leyendo el propio
historial del topic (no solo el log del script).
**Limpieza:** `deployment/test-agent-canary` eliminado, entrada
correspondiente en `state.json` borrada. Los logs de la prueba
(`actions.jsonl`, `run-*.log`) se conservan como evidencia auditable —
no se purgaron.
## Notificaciones
**Canal: Telegram Bot API** (reemplaza a ntfy, cambiado 2026-08-26 por
preferencia — no por costo, ver sección de costo abajo). Bot creado
manualmente vía [@BotFather](https://t.me/BotFather); el token vive en
`TELEGRAM_BOT_TOKEN_FILE` (dueño `devops`, `600` — **no** root-only,
mismo criterio que `GITEA_TOKEN_FILE`: el servicio corre como `devops`
por el login de suscripción, un archivo `root:root` le sería ilegible).
`TELEGRAM_CHAT_ID` no es secreto, va directo en `lab-agent.env`.
El mensaje se manda como texto plano, sin `parse_mode`: el motivo que
devuelve Claude es texto libre y puede traer `_`, `*`, `[`, `]` — con
Markdown/MarkdownV2 esos caracteres rompen el parser de Telegram (`400
Bad Request`) si no se escapan con cuidado. Se prioriza entrega
confiable sobre formato.
**Validado en vivo 2026-08-26** con el mismo código que usa
`notify_send()` (lectura del token, `sendMessage` vía `curl
--data-urlencode`, formato `[title]\nmessage`): entrega confirmada
(`"ok":true`) y contenido recibido tal cual en Telegram. Gotcha
encontrado durante la prueba: el primer intento falló con `400 chat not
found` porque el `chat_id` pegado correspondía a otro bot — el `chat_id`
de un usuario es el mismo across bots, pero Telegram igual devuelve
`chat not found` si *ese bot puntual* nunca recibió un mensaje de esa
persona. Se resolvió confirmando el `chat_id` real vía `getUpdates` del
bot correcto.
**Formato:** cada notificación lleva un ícono según su naturaleza
(✅ acción ok, 🛑 acción falló/rechazada, 🚨 alerta de seguridad,
❌ error operativo, 👀 solo diagnóstico) y el log completo de esa
corrida va adjunto como documento (`sendDocument`), no solo como ruta —
útil para revisar desde el celular sin acceso a shell.
**Notificación silenciosa y respuestas en hilo (2026-08-26):** el nivel
`info` (nada elegible, todo diagnóstico rutinario) se manda con
`disable_notification=true` — no interrumpe; acciones, fallas y alertas
de seguridad sí suenan. Cuando se ejecuta una acción correctiva, el
`message_id` de Telegram queda guardado por recurso en `state.json`
(`telegram_message_id`); si ese recurso se recupera solo en una corrida
posterior, se manda un aviso silencioso como **respuesta** (hilo) al
mensaje original antes de limpiar el estado — agrupa visualmente todo
un incidente. Limitación conocida: si un humano desbloquea el recurso a
mano borrando su entrada de `state.json` (ver más abajo), el
`message_id` se pierde con esa entrada, así que ese caso puntual no
queda threadeado — solo el "se recuperó solo" está cubierto hoy.
Validado en vivo con el canario: acción real → `message_id` capturado →
canario parcheado a una imagen sana → reply en hilo confirmado,
silencioso, al recuperarse.
## Horario activo y costo
**Pausa nocturna (`ACTIVE_HOURS_START`/`END`, hora local del host):**
activo `06:00`–`23:00`, pausado `23:00`–`06:00`. El script chequea esto
como lo primero que hace tras inicializar logging — si está fuera de
horario, sale **antes** de tocar `kubectl`/`docker`/`curl` o invocar
`claude -p`, así que la pausa ahorra el costo completo de la corrida,
no solo la notificación. El timer de systemd además solo dispara dentro
de esa ventana (`OnCalendar=*-*-* 06..22:00:00`) — la doble
verificación es deliberada: si alguien edita la unidad de systemd sin
tocar `lab-agent.env` (o corre el script a mano), el chequeo interno
sigue protegiendo.
**Tensión sin resolver con la ventana de `medusa-deploy`:** la ventana
de auto-restart para `medusa-deploy` (`COMMERCE_BACKEND_WINDOW_*`,
`00:00`–`06:00`) **no se solapa** con el horario activo del agente
(`06:00`–`23:00`). Esto significa que, en la práctica, `medusa-deploy`
nunca es candidato real a auto-restart: la única ventana en la que la
política lo permitiría es exactamente cuando el agente está pausado y
ni siquiera corre. La regla sigue viva en el código (por si el horario
activo cambia más adelante) pero hoy es efectivamente inalcanzable.
Pendiente de decisión: mover la ventana de `medusa-deploy` a que caiga
dentro del horario activo, o aceptar que ese recurso queda cubierto
solo por diagnóstico/notificación manual.
**Estimación de costo mensual — actualizada 2026-08-26, frecuencia
bajada de 15 a 60 min explícitamente para reducir costo:**
| | |
|---|---|
| Corridas/día (17h activas ÷ 60 min) | 17 |
| Costo por corrida | ~US$0.04–0.12 — **más alto por corrida que a 15 min**: el cache de prompt de Claude Code dura ~1h, y con corridas espaciadas 60 min casi siempre expira entre una y la siguiente, así que la mayoría de las corridas pagan cache "fría" en vez de "tibia" |
| **Estimado mensual (× 30 días)** | **~US$20–60** (punto medio ~US$35) |
Ojo con la intuición: bajar la frecuencia 4× (68→17 corridas/día) NO
baja el costo 4× — el ahorro real es más cercano a 1.5–2× frente a la
estimación anterior (~US$60–65/mes), porque cada corrida individual
sale más cara al perder la ventaja del cache tibio. Sigue siendo menos
costo total, solo que no linealmente. Basado en pocas corridas reales
medidas — revisar el costo real después de la primera semana en
producción (Telegram/Gitea Actions no cambian este número, el costo es
100% de la API de Claude).
**Trade-off de latencia de detección:** con corridas cada 60 min en vez
de 15, un problema que recién cruza el umbral de 10 minutos puede
tardar hasta ~70 min en ser detectado y potencialmente corregido (antes
era hasta ~25 min). Aceptado a cambio del ahorro de costo — si en la
práctica resulta muy lento para algún incidente real, la frecuencia se
puede volver a subir sin tocar nada más (`OnCalendar` en
`lab-agent.timer`).
## Activación del timer (2026-08-26)
Habilitado tras confirmación explícita del propietario del lab, viendo
antes el costo mensual estimado. Unidad de usuario (`systemctl --user`,
corre como `devops`, no `root`), con límites de CPU/memoria (`CPUQuota`,
`MemoryMax`) — instalada en `/DATA/.config/systemd/user/`, no en el
`~/.config/...` de una shell interactiva.
**Gotcha encontrado y corregido en vivo:** el manager de `systemd
--user` de `devops` arranca con `HOME=/DATA` (el valor de
`/etc/passwd`), no `/DATA/AppData/home-devops` — ese override solo se
aplica al abrir una shell interactiva vía
`/etc/profile.d/zz-devops-home.sh`, que el manager de systemd nunca
carga. Sin corregirlo, `claude -p` habría buscado
`~/.claude/.credentials.json` en `/DATA/.claude/` (no existe) y la
autenticación habría fallado en cada corrida del timer, silenciosamente,
hasta la primera vez que alguien mirara el log. `lab-agent.service`
fija `Environment=HOME=/DATA/AppData/home-devops` explícitamente para
evitarlo. Se detectó y corrigió *antes* de habilitar el timer, corriendo
el servicio manualmente vía `systemctl --user start` con el entorno
real (no desde una shell interactiva) como último paso de validación.
También requirió `sudo loginctl enable-linger devops` (una vez) — sin
eso, `systemctl --user` ni siquiera conecta (no hay `/run/user/<uid>`
sin una sesión de login o linger activo).
**Pendiente, no bloqueante:**
- Opcional: `GITEA_TOKEN_FILE` para activar la señal de "runner
atascado" vía API de Gitea Actions (hoy se omite, degradación segura).
- Decidir la tensión con la ventana de `medusa-deploy` (ver arriba) —
sigue sin resolver, el timer se activó igual porque no bloquea el
resto de la lista blanca.
@@ -1,9 +1,9 @@
# Del commit al despliegue: flujo GitOps
> **TODO (fase 2+):** narrar cada paso con ejemplos reales tomados de
> `.gitea/workflows/build-medusa.yaml` y `build.yaml`. El diagrama de
> secuencia de abajo refleja el pipeline real (build → push imagen →
> promoción segura → commit del tag → sync de Argo CD).
El diagrama de abajo refleja el pipeline real (build → push imagen →
promoción segura → commit del tag → sync de Argo CD). Después del
diagrama hay un ejemplo con hashes de commit reales, tomado de un
cambio real hecho sobre este mismo repo.
## Secuencia: un commit hasta quedar corriendo en el cluster
@@ -42,9 +42,53 @@ sequenceDiagram
evita que un pipeline viejo sobreescriba el tag con una versión
anterior si hubo un push más reciente mientras corría.
## TODO
## Ejemplo real: un commit de principio a fin
- Ejemplo real con hashes de commit (tomar del historial de
`apps-registry`).
- Explicar `syncPolicy.automated.selfHeal` con un caso concreto (qué
pasa si alguien hace `kubectl edit` a mano).
Fase 3b del ADR 0006 (facetas del sidebar del catálogo, ver
`decisiones/0006-motor-busqueda-facetas.md`) recorrió el flujo
completo dos veces seguidas el mismo día:
| Paso | Commit / evento real |
|---|---|
| Feature | `83d448b` — `feat(ecommerce): facetas reales de Meilisearch en el sidebar del catalogo`, en `fix/adr0006-etapa2-fase3b-facetas-sidebar` |
| Merge a `main` | `6b5450f` |
| CI build+push | Gitea Actions task 258, imagen `gitea.cruzcloud.net/devops/ecommerce-frontend` |
| Release commit (CD) | `951682a` — `chore: release v1.0.171 [skip ci]`, bump automático del tag en `frontend/patches/deployment.yaml` |
| Sync de Argo CD | dos ciclos distintos: primero sincronizó contra `6b5450f` (el merge, todavía con el tag viejo), y en el ciclo siguiente detectó `951682a` y aplicó el tag nuevo |
| Rollout | `frontend-deploy` con 2/2 réplicas en `v1.0.171` |
El gate en vivo contra ese mismo cambio encontró un bug real
(`estimatedTotalHits` en 0 con `limit=0` en Meilisearch) — el fix
repitió el mismo ciclo completo: `884dbd1` → merge `ba3e6c7` → release
`v1.0.172` → sync → rollout. Dos vueltas completas del diagrama de
arriba en la misma sesión, cada una verificada contra el endpoint real
antes de darla por cerrada.
**Detalle no obvio:** Argo CD sincroniza el *merge commit* primero
(todavía con la imagen vieja, porque el bump de tag es un commit
*posterior* que hace el propio pipeline) y recién en un segundo ciclo
de poll detecta el commit de release. Son dos sincronizaciones
distintas, no una sola — hay que esperar ambas antes de asumir que el
pod nuevo está corriendo.
## `selfHeal`, con la configuración real
Todas las Applications de este repo comparten el mismo `syncPolicy`
(`apps/ecommerce-app.yaml`):
```yaml
syncPolicy:
automated:
prune: true
selfHeal: true
```
Con `selfHeal: true`, si alguien corre `kubectl scale deployment
frontend-deploy --replicas=1` directo contra el cluster (saltándose
Git por completo), Argo CD no se queda en mostrar `OutOfSync` — en el
siguiente ciclo de reconciliación vuelve a aplicar el manifiesto tal
como está en `main` y el Deployment queda otra vez en las réplicas
declaradas ahí. El cambio manual no se pierde silenciosamente: queda
un instante como drift visible en la UI de Argo CD, pero nunca se
vuelve el estado final. La única forma de que ese cambio persista es
que exista en Git — lo mismo que describe [ADR 0002](../decisiones/0002-por-que-argocd-vs-manual.md).
@@ -65,9 +65,142 @@ plantillas (a diferencia de Helm).
incluir y qué parches aplicarles, sin necesidad de un lenguaje de
templating separado.
## TODO
## Namespace
- Namespace, ResourceQuota, LimitRange, NetworkPolicy
- Ingress vs. Service vs. Deployment
- StatefulSet (por qué Postgres lo usa y el frontend no)
- Init container
**En una frase:** una partición lógica dentro del mismo cluster físico
— agrupa recursos relacionados y les pone un límite alrededor.
**Detalle:** este lab tiene un namespace por dominio de negocio
(`ecommerce`, `docs-portal`) más `argocd` para el propio Argo CD. Todo
lo que gobierna a un namespace (cuotas, políticas de red) vive en
`platform-infra`, separado de los manifiestos de la app — así una app
nueva no puede "olvidarse" de traer su propia gobernanza.
## ResourceQuota
**En una frase:** el techo total de recursos que un namespace entero
puede consumir, sumando todos sus pods.
**Detalle:** la cuota real de `ecommerce`
(`platform-infra/ecommerce-governance/resourcequota.yaml`):
```yaml
hard:
requests.cpu: "3"
requests.memory: 4Gi
limits.cpu: "8"
limits.memory: 8Gi
pods: "12"
persistentvolumeclaims: "6"
requests.storage: 80Gi
```
Si un Deployment nuevo pidiera más `requests.memory` del que queda
disponible bajo ese techo, Kubernetes rechaza el pod directamente — no
lo deja arrancar y luego fallar.
## LimitRange
**En una frase:** los valores por defecto (y los mínimos/máximos) que
aplican a un contenedor *individual* dentro de un namespace, para
cuando el manifiesto no los declara explícitos.
**Detalle:** complementa a `ResourceQuota` (que mira el namespace
entero) a nivel de un solo contenedor. La de `ecommerce`
(`limitrange.yaml`) pone `defaultRequest: 100m/128Mi`,
`default: 500m/512Mi` (límites si el contenedor no los declara), y un
rango permitido de `10m/16Mi` a `2 CPU/1Gi` por contenedor — evita
tanto el contenedor que no pide nada y termina compitiendo mal por
recursos, como el que pide de más y se come la cuota del namespace
entero él solo.
## NetworkPolicy
**En una frase:** un firewall a nivel de Kubernetes — qué tráfico de
red se permite entrar (o salir) de un pod.
**Detalle:** sin ninguna `NetworkPolicy`, todos los pods de un cluster
pueden hablarse entre sí libremente. La de `ecommerce`
(`networkpolicy.yaml`) usa `podSelector: {}` (aplica a todos los pods
del namespace) y permite ingress solo desde `namespaceSelector: {}`
(cualquier namespace del cluster, típicamente Traefik llegando desde
`kube-system` o el namespace de ingress) — bloquea tráfico externo al
cluster que intentara saltarse Traefik y pegarle directo a un Service
interno.
## Deployment
**En una frase:** el controlador para cargas de trabajo **sin
estado** — réplicas intercambiables entre sí, ninguna con identidad
propia.
**Detalle:** `frontend-deploy` es un Deployment de 2 réplicas: no
importa cuál de las dos atienda una petición, ambas sirven exactamente
lo mismo y ninguna guarda datos propios en disco. Si un pod muere,
Kubernetes crea otro cualquiera para reemplazarlo — no hay "el pod
original" que recuperar. Contrastar con **StatefulSet** abajo.
## Service
**En una frase:** un nombre de red estable que apunta a un grupo de
pods, aunque esos pods cambien.
**Detalle:** los pods son efímeros (mueren y se recrean con IP nueva
cada vez); un Service (ej. `frontend-svc`, `postgres-svc`) les da un
nombre DNS interno fijo (`frontend-svc.ecommerce.svc.cluster.local`,
o simplemente `frontend-svc` desde dentro del mismo namespace) que no
cambia aunque los pods de atrás sí. `postgres-svc` en este lab es
además un Service `clusterIP: None` ("headless") — necesario porque
`commerce-postgres` es un StatefulSet y cada pod necesita su propia
identidad de red estable, no una IP compartida balanceada entre
réplicas.
## Ingress
**En una frase:** la regla que dice "las peticiones a este dominio y
esta ruta van a este Service", para tráfico HTTP que entra al cluster.
**Detalle:** `ecommerce-ingress` enruta por *path*, no solo por
dominio: `shop.cruzcloud.net/api/catalog` va a `catalog-api-svc`, y
todo lo demás (`/`) va a `frontend-svc` — un solo objeto Ingress con
dos backends distintos según el prefijo de la URL. Es Traefik (el
Ingress Controller del cluster) quien lee estos objetos y hace el
enrutamiento real; el Ingress en sí es solo la declaración de la
regla, no el componente que la ejecuta.
## StatefulSet
**En una frase:** el controlador para cargas de trabajo **con
estado** — cada réplica tiene identidad propia y almacenamiento propio
que sobrevive a que el pod se reinicie.
**Detalle:** `commerce-postgres` es un StatefulSet, no un Deployment,
porque el frontend puede tener 2 réplicas idénticas sin datos propios,
pero Postgres no — sus datos tienen que vivir en un disco específico
que sobreviva un reinicio del pod y sepa exactamente cuál pod es
"el suyo". Cada pod de un StatefulSet obtiene su propio
`PersistentVolumeClaim` vía `volumeClaimTemplates` (acá,
`postgres-data`, 10Gi en `local-path`) y un nombre de red estable y
numerado (`commerce-postgres-0`), a diferencia de los nombres
aleatorios de un Deployment. Meilisearch, Redis y MinIO en este lab
siguen el mismo patrón por la misma razón — todos guardan datos en
disco que tienen que sobrevivir al pod.
## Init container
**En una frase:** un contenedor que corre **antes** que los
contenedores principales de un pod, y tiene que terminar bien (exit
0) para que el resto arranque.
**Detalle:** `medusa-deploy` usa dos, y a propósito están separados en
vez de ser uno solo: `wait-for-postgres` (`pg_isready` en loop, solo
espera a que Postgres acepte conexiones) y `migrations`
(`db:migrate`, corre las migraciones de verdad). Un incidente real
mostró por qué separarlos importa: cuando el chequeo de disponibilidad
reusaba la misma `DATABASE_URL` completa de la app (con
`?sslmode=disable` incluido), el parser de `pg_isready` no la entendía
y el pod quedaba atascado en `Init` — reportado como si fuera
`CrashLoopBackOff`, sin serlo. El fix fue que el chequeo de
disponibilidad use el mínimo de parámetros posible
(host/puerto/usuario/db), nunca la URI completa pensada para la app.
Ver el [playbook del crashloop de Medusa](../playbooks/incidente-crashloop-medusa.md).
@@ -1,8 +1,7 @@
# Red y exposición
> **TODO (fase 2+):** explicar cada dominio, por qué está proxied vs
> DNS-only en Cloudflare, y el manejo de TLS extremo a extremo. Tabla
> de dominios abajo es real, pendiente de expandir con capturas/ejemplos.
Los 5 dominios de abajo son reales y comparten exactamente el mismo
camino de red — ninguno tiene un caso especial.
## Dominios activos
@@ -14,12 +13,63 @@
| `gitea.cruzcloud.net` | Gitea | — | Cloudflare (edge) |
| `docs.cruzcloud.net` | Este portal | `docs` | Cloudflare (edge) |
## TODO
## Por qué todos van "proxied" (naranja), ninguno "DNS-only"
- Diagrama de TLS termination (dónde termina TLS realmente: en
Cloudflare, no en Traefik — el tráfico interno NPM→k3d es HTTP plano).
- Explicar por qué el transporte del túnel se fijó a HTTP/2 en vez de
QUIC (ver [ADR 0003](../decisiones/0003-por-que-http2-sobre-quic-tunnel.md)
y el [playbook del 504](../playbooks/incidente-504-gitea-tunnel.md)).
- Patrón de diagnóstico "interno sano, público roto" (ver
[playbook de imágenes NPM](../playbooks/incidente-imagenes-npm.md)).
Un registro DNS-only en Cloudflare expondría directamente la IP real
del túnel — pero acá no hay IP pública fija que exponer: el NAS está
detrás de NAT residencial. El modo proxied es lo que hace que
Cloudflare Tunnel tenga sentido en primer lugar (el tráfico entra por
el edge de Cloudflare, nunca directo al origen), así que no hay ningún
dominio de este lab para el que DNS-only sea una opción real.
## Dónde termina TLS de verdad
```mermaid
flowchart LR
User(["Usuario"]) -->|HTTPS| CF["Cloudflare edge<br/>🔒 TLS termina acá"]
CF -->|túnel cifrado, HTTP/2| CFD["cloudflared"]
CFD -->|HTTP plano| NPM["Nginx Proxy Manager"]
NPM -->|HTTP plano| LB["k3d-proxy :90"]
LB -->|HTTP plano| TR["Traefik"]
TR -->|HTTP plano| SVC["Service interno"]
```
TLS termina en el **edge de Cloudflare**, no en Traefik ni en NPM. De
ahí hacia adentro (`cloudflared` → NPM → `k3d-proxy` → Traefik →
Service) todo el tráfico va en HTTP plano — seguro en este contexto
porque ese tramo entero vive dentro de la red local del NAS, nunca
sale a internet. Ningún componente interno necesita certificado propio
ni gestionar renovación de TLS.
## Por qué HTTP/2 y no QUIC en el túnel
`cloudflared` puede usar QUIC (UDP) o HTTP/2 (TCP) para sus conexiones
salientes hacia el edge. Este lab fuerza HTTP/2 explícitamente — no es
el default. Un incidente real de `504 Gateway Timeout` intermitente en
**todos** los subdominios (no solo uno) se rastreó hasta fallos
recurrentes del "control stream" de QUIC sobre la red doméstica del
NAS. Ver [ADR 0003](../decisiones/0003-por-que-http2-sobre-quic-tunnel.md)
para la decisión completa y el [playbook del 504](../playbooks/incidente-504-gitea-tunnel.md)
para el diagnóstico paso a paso.
## Patrón de diagnóstico: "interno sano, público roto"
Cuando un dominio público falla pero todo dentro del cluster responde
bien (pods `Running`, probes verdes, `kubectl exec` con `curl` al
Service funciona), la causa casi nunca está en Kubernetes — está en la
capa NPM. La forma rápida de confirmarlo es saltarse NPM a propósito:
`curl -H "Host: <dominio>" http://192.168.68.61:90/` habla directo con
`k3d-proxy`/Traefik sin pasar por NPM. Si eso funciona pero el dominio
público no, el problema está específicamente en el `forward_host`/
`forward_port` del proxy host en NPM (ej.: un scheme o un path colado
en `forward_host`, que rompe todo el enrutamiento).
Este patrón se repitió dos veces en este lab con la misma causa raíz
exacta: el [incidente de imágenes en `media.cruzcloud.net`](../playbooks/incidente-imagenes-npm.md)
(playbook completo) y un incidente posterior de `commerce.cruzcloud.net`
devolviendo 400 por el mismo tipo de `forward_host` roto (sin playbook
propio todavía). Una lección adicional de ese segundo incidente:
editar el `proxy_host` a mano en el sqlite de NPM **no** regenera el
`.conf` de nginx automáticamente — el cambio tiene que pasar por la
propia UI/API de NPM, o el `.conf` hay que parchearlo a mano y
recargar nginx.
@@ -1,10 +1,10 @@
# Visión general
> **TODO (fase 2+):** narrativa completa explicando cada capa del
> diagrama, con foco en el "por qué" de cada frontera de confianza. El
> diagrama de abajo es real (validado contra `docs/known-issues.md` y
> los playbooks de incidentes de `apps-registry`), pero el texto que lo
> acompaña es solo el esqueleto.
Este lab corre sobre un NAS ZimaOS doméstico, sin IP pública ni
presupuesto de nube — cada capa del diagrama de abajo existe por una
razón concreta de ese contexto, no por moda arquitectónica. Esta
página explica el "por qué" de cada frontera; el "cómo" vive en las
páginas enlazadas al final de cada sección.
## Flujo de una petición pública
@@ -40,4 +40,52 @@ flowchart LR
| Argo CD | Sincroniza el estado del cluster con Git (patrón App of Apps) | Namespace `argocd` |
| Gitea | Origen de verdad de los manifiestos y del código de las apps | Contenedor propio, expuesto vía túnel |
## Por qué cada frontera está donde está
**Cloudflare Tunnel, no port-forwarding.** El NAS vive detrás de NAT
residencial, sin IP pública fija. En vez de abrir puertos en el router
(superficie de ataque directa contra el hogar), `cloudflared` abre una
conexión **saliente** hacia Cloudflare y todo el tráfico público viaja
hacia adentro por esa misma conexión — el origen nunca expone un
puerto escuchando en internet. El transporte de esa conexión se fijó a
HTTP/2 en vez del QUIC por defecto tras un incidente real de
inestabilidad; ver [ADR 0003](../decisiones/0003-por-que-http2-sobre-quic-tunnel.md).
**Nginx Proxy Manager, fuera del cluster.** No todo lo que sirve
`*.cruzcloud.net` vive dentro de k3d — Gitea, por ejemplo, corre como
contenedor Docker aparte. NPM es el único punto de enrutamiento por
dominio que necesita conocer *ambos* mundos (cluster y no-cluster), así
que vive fuera de Kubernetes a propósito, no por descuido.
**`k3d-lab-cluster-serverlb`, la frontera de entrada al cluster.** Es
el load balancer que el propio k3d monta automáticamente para publicar
puertos del host hacia los nodos internos — no es un componente que
este lab configuró a mano, viene con la topología multi-nodo elegida
en [ADR 0001](../decisiones/0001-por-que-k3d.md).
**Traefik, enrutamiento por `Host` dentro del cluster.** Una vez el
tráfico entra al cluster por un único puerto (90/9443), Traefik es
quien decide a qué Service mandarlo según el header `Host` de cada
petición — así NPM solo necesita saber "todo `*.cruzcloud.net` va al
puerto 90 del cluster", sin tener que aprender un forward_host nuevo
cada vez que se agrega un servicio interno. Cuando esta capa falla en
silencio (`forward_host`/`forward_port` mal apuntado en NPM), el
síntoma es siempre el mismo: "internamente sano, públicamente roto" —
ver [Red y exposición](red-y-exposicion.md).
**Argo CD, el único escritor del cluster.** Ningún pipeline de CI ni
ninguna persona corre `kubectl apply` contra producción — Argo CD es
la única pieza con permisos de escritura, y reconstruye
automáticamente cualquier diferencia entre Git y el cluster real
(`selfHeal: true`). El porqué completo de esa elección está en
[ADR 0002](../decisiones/0002-por-que-argocd-vs-manual.md); el paso a
paso de un commit real hasta quedar corriendo, en
[Flujo GitOps](gitops-flujo.md).
**Gitea, autoalojado.** Además de alojar el código, Gitea corre el CI
(Gitea Actions) y el registry de imágenes en el mismo lugar — un solo
componente autoalojado en vez de depender de la disponibilidad y los
límites de un servicio externo gratuito para un lab que ya de por sí
no tiene presupuesto de nube.
Ver también: [Red y exposición](red-y-exposicion.md) · [Flujo GitOps](gitops-flujo.md) · [Glosario](glosario.md)
@@ -1,33 +1,67 @@
# ADR 0001 — Por qué k3d (y no k3s directo, minikube, o un cluster cloud)
> **TODO (fase 2+):** completar contexto/opciones/consecuencias con
> criterio real (recursos del NAS ZimaOS, necesidad de multi-nodo para
> reproducir el blackhole de MTU documentado en `known-issues.md`,
> costo cero vs. cluster gestionado).
**Estado:** aceptada (placeholder — pendiente de redactar)
**Fecha:** TODO
**Estado:** aceptada
**Fecha:** 2026-08 (retroactiva — decisión tomada al arrancar el lab, documentada en Fase 2 del portal)
## Contexto
TODO
El lab corre sobre un NAS ZimaOS doméstico, sin presupuesto para un
cluster gestionado y con recursos modestos y compartidos con otras
apps del NAS (Gitea, Nginx Proxy Manager, este mismo portal). El
propósito del lab es doble: servir de portafolio técnico real y de
material de estudio para el hijo del usuario (7mo semestre de
Ingeniería de Sistemas) — ver [guía del estudiante](../guia-estudiante/README.md).
Para que ambos objetivos tengan sentido, el cluster necesitaba
comportarse como un Kubernetes real y multi-nodo, no como una caja de
arena de un solo proceso: patrones como afinidad de pods, StatefulSets
con almacenamiento persistente por nodo, y tráfico cross-node debían
poder reproducirse y depurarse de verdad, no simularse.
También pesó la restricción del propio sistema operativo: `/` en
ZimaOS es un `squashfs` de solo lectura y `/usr` es un overlay
igualmente inmutable — instalar un binario nativo como `k3s` que
espera escribir en `/usr/local/bin`, `/etc/rancher`, etc. de forma
persistente no encaja bien con ese filesystem sin trabajo extra de
por medio.
## Opciones consideradas
| Opción | A favor | En contra |
|---|---|---|
| k3d | | |
| k3s directo (sin Docker) | | |
| minikube | | |
| Cluster gestionado (cloud) | | |
| **k3d** | Multi-nodo real corriendo como contenedores Docker (server + agent), se crea/destruye en segundos, no toca el filesystem inmutable de ZimaOS más allá de `/DATA` | Al ser Docker-in-Docker, hereda las capas de red de Docker — un blackhole de MTU/PMTUD real apareció por esto (ver Consecuencias) |
| k3s directo (sin Docker) | Sin la capa extra de red de Docker-in-Docker, más cercano a un nodo "real" | Necesita instalar servicios systemd y escribir en rutas de sistema (`/usr/local/bin`, `/etc/rancher`) que en ZimaOS son de solo lectura o requieren workarounds — solo `/DATA` y `/etc` quedan escribibles |
| minikube | Instalación simple, muy documentado, ideal para aprender lo básico | Single-node por defecto — no permite reproducir tráfico cross-node, afinidad de pods, ni el tipo de incidentes de infraestructura real que este lab busca documentar como aprendizaje |
| Cluster gestionado (cloud) | Cero mantenimiento de infraestructura, SLA real | Costo recurrente que este lab no tiene presupuestado; además reduce el valor de aprendizaje — gran parte de lo documentado en `playbooks/` (MTU, DNS, límites de recursos) es justamente lo que un proveedor gestionado abstrae |
## Decisión
TODO
k3d, multi-nodo (hoy: 1 server + 1 agent, `k3d-lab-cluster-server-0` /
`k3d-lab-cluster-agent-0`), corriendo como contenedores Docker sobre
el propio NAS.
## Consecuencias
TODO — mencionar explícitamente el blackhole de red cross-node
(MTU/PMTUD) como consecuencia real y documentada de correr multi-nodo
sobre Docker-in-Docker sin ajuste de MTU (ver
[playbook del crashloop de Medusa](../playbooks/incidente-crashloop-medusa.md)).
- **Se ganó** un cluster real y gratuito, reproducible con un solo
script (`scripts/deploy-lab.sh`), y suficientemente parecido a un
Kubernetes de producción como para que los incidentes documentados
en `playbooks/` sean transferibles a un entorno cloud real.
- **Se pagó** con un blackhole de red cross-node real: cualquier pod
agendado en un nodo distinto al de `commerce-postgres-0`
(StatefulSet, fijo a un nodo) fallaba de forma consistente al hablar
con Postgres — diagnosticado hasta el bit exacto (payload interno
1420 bytes OK, 1450 bytes blackhole total, sin ICMP "frag needed")
y resuelto ajustando el MTU real de la interfaz `flannel.1` a 1400
en cada nodo. Ver [known-issues.md](https://gitea.cruzcloud.net/devops/apps-registry/src/branch/main/docs/known-issues.md)
y el [playbook del crashloop de Medusa](../playbooks/incidente-crashloop-medusa.md)
para el diagnóstico completo. El fix no persiste solo — está
automatizado en `scripts/deploy-lab.sh` (función `fix_flannel_mtu`),
se reaplica cada vez que el cluster arranca o se recrea.
- **Deuda aceptada a propósito**: el `podAffinity` que fuerza a
`medusa-deploy` a compartir nodo con Postgres se dejó como red de
contención extra después del fix de fondo — no es estrictamente
necesario, pero es barato mantenerlo.
- **Para revertir esta decisión** haría falta migrar a un cluster
gestionado o a `k3s` nativo, lo que implicaría asumir el costo
recurrente (cloud) o resolver las restricciones de filesystem
inmutable de ZimaOS (k3s nativo) — ninguna de las dos se justifica
hoy con el tamaño real del lab.
@@ -1,30 +1,80 @@
# ADR 0002 — Por qué Argo CD (y no `kubectl apply` manual o un script de CI)
> **TODO (fase 2+):** completar con el criterio real detrás de elegir
> reconciliación continua (`selfHeal`) sobre despliegue imperativo desde
> CI.
**Estado:** aceptada (placeholder — pendiente de redactar)
**Fecha:** TODO
**Estado:** aceptada
**Fecha:** 2026-08 (retroactiva — decisión tomada al arrancar el lab, documentada en Fase 2 del portal)
## Contexto
TODO
Desde el principio el lab iba a tener más de un repo Git (hoy:
`apps-registry` para las Applications y los manifiestos reales de los
workloads, `platform-infra` para la gobernanza de cada namespace,
`scripts` para bootstrap/operación) y más de una app corriendo en el
cluster (ecommerce, docs-portal, nginx, cada una con su propia
gobernanza). Se necesitaba una forma de mantener el estado del
cluster sincronizado con lo declarado en Git que escalara a "agregar
una app nueva" sin volverse un proceso manual que dependiera de que
una sola persona recordara qué `kubectl apply` correr y en qué orden.
También importaba la trazabilidad: en un lab que sirve de portafolio,
poder señalar "este commit es exactamente lo que está corriendo en
producción" (y por qué) tiene valor por sí mismo — no solo operativo.
## Opciones consideradas
| Opción | A favor | En contra |
|---|---|---|
| Argo CD (GitOps, pull-based) | | |
| `kubectl apply` manual | | |
| `kubectl apply` desde el pipeline de CI (push-based) | | |
| **Argo CD (GitOps, pull-based)** | Reconciliación continua: un proceso dentro del cluster compara el estado real contra Git y corrige la diferencia solo, sin que nadie tenga que acordarse de aplicar nada. Patrón App of Apps nativo — agregar una app nueva es un archivo YAML más en `apps/`, no un paso manual nuevo | Componente extra que mantener sano (su propio namespace, su propio consumo de recursos) |
| `kubectl apply` manual | Cero infraestructura extra, control total en cada comando | Sin detección de drift — si alguien edita algo a mano y lo olvida, nadie se entera hasta que rompe; no escala más allá de un solo operador que recuerde el estado completo de memoria |
| `kubectl apply` desde el pipeline de CI (push-based) | No agrega un componente nuevo al cluster, reutiliza el CI que ya existe para construir imágenes | El pipeline de CI necesitaría credenciales de escritura amplias sobre el cluster — más superficie de riesgo si esas credenciales se filtran. Push-based además significa que nada vuelve a comparar el estado después de aplicado: un drift posterior (alguien con `kubectl` directo) queda invisible hasta el siguiente deploy |
## Decisión
TODO
Argo CD, patrón App of Apps: una Application raíz
(`apps-registry/application.yaml`) vigila la carpeta `apps/`, y cada
archivo ahí dentro es una Application hija que apunta a un workload
real. Toda Application se declara con el mismo `syncPolicy`:
```yaml
syncPolicy:
automated:
prune: true
selfHeal: true
syncOptions:
- CreateNamespace=true
```
(configuración real de `apps/ecommerce-app.yaml`, repetida en cada
Application del repo)
## Consecuencias
TODO — mencionar el patrón App of Apps (`application.yaml` +
`apps/*.yaml`) y cómo `selfHeal:true` cambia el modelo mental de
"quién puede tocar el cluster".
- **`selfHeal: true` cambia el modelo mental de quién puede tocar el
cluster de verdad.** No es solo que Argo CD detecte diferencias y
las muestre como `OutOfSync` — las revierte activamente. Si alguien
corre `kubectl scale` o `kubectl edit` directo contra un recurso
gestionado, en el siguiente ciclo de reconciliación (poll periódico,
o inmediato si el commit llegó por webhook) Argo CD lo vuelve a
dejar exactamente como dice Git. Git deja de ser "la intención" y
pasa a ser literalmente la única fuente de la que el cluster puede
aprender su propio estado — ver [flujo GitOps](../arquitectura/gitops-flujo.md).
- **`prune: true`** significa que borrar un recurso del repo (por
ejemplo, quitar `meilisearch.yaml` de una `kustomization.yaml`) lo
borra también del cluster en el siguiente sync, sin un paso manual
de limpieza — confirmado en la práctica en varias fases de este lab
(ver los ADRs de rediseño del storefront y de Meilisearch).
- **El rol de CI se redujo a "construir, publicar imagen, y dejar un
commit en Git"** — nunca aplica nada directo al cluster. Esto evitó
tener que darle a CI credenciales de cluster, y dejó un problema real
documentado (la condición de carrera de promoción cuando dos
pipelines corren en serie sobre el mismo commit) resoluble
comparando contra Git (`git merge-base --is-ancestor`) en vez de
contra el estado del cluster.
- **Se paga con un componente más que mantener sano**: Argo CD corre
en su propio namespace, con su propio consumo de recursos y su
propia superficie de fallo — un Argo CD caído no rompe lo que ya
está corriendo (el cluster sigue con el último estado aplicado),
pero sí detiene la sincronización de cambios nuevos.
- **Para revertir esta decisión** haría falta volver a un modelo
push-based (CI aplicando directo) o puramente manual, perdiendo la
reconciliación continua y aceptando que un drift manual pueda pasar
desapercibido.
@@ -109,7 +109,10 @@ Un motor de búsqueda dedicado (Meilisearch, autohospedado) queda
identificado como el siguiente paso natural una vez el catálogo
crezca más allá de ~100 productos activos, pero no es parte del
alcance inmediato de Etapa 2 — se revisará cuando ese techo esté
cerca.
cerca. **Actualización 2026-08-25:** terminada la Etapa 2 (ver
[ADR 0005](0005-rediseno-storefront.md)), el disparador real resultó
ser la complejidad de facetas de la taxonomía, no el volumen de
productos — ver [ADR 0006](0006-motor-busqueda-facetas.md).
**Compatibilidad con Medusa Admin confirmada:** ninguna de las tres
opciones evaluadas, incluida la elegida, toca
@@ -1,13 +1,14 @@
# ADR 0005 — Rediseño del storefront para catálogo multi-categoría
**Estado:** propuesta — pendiente de aprobación antes de Etapa 2
**Fecha:** 2026-08-23
**Estado:** aceptada — Etapa 2 implementada y en producción
**Fecha:** 2026-08-23 (Etapa 1) — actualizado 2026-08-25 (Etapa 2 completa)
> Este documento es la propuesta de Etapa 1 del rediseño de ARI
> Shopping. No incluye código de producción — solo auditoría,
> decisión (ver también [ADR 0004](0004-evaluacion-stack-multi-categoria.md)),
> patrones de UX y datos demo ya creados y visibles en producción
> como línea base.
> Este documento arrancó como la propuesta de Etapa 1 del rediseño de
> ARI Shopping (auditoría, decisión — ver también [ADR 0004](0004-evaluacion-stack-multi-categoria.md) —
> patrones de UX y datos demo). Etapa 2 fue aprobada y se implementó
> completa en tres sub-fases (datos/categorías, navegación,
> componentes visuales) — ver la sección 6 para el detalle de cada
> sub-fase y sus PRs.
## 1. Resumen de la auditoría
@@ -255,12 +256,33 @@ objetivo es validar la tubería Admin → MinIO → NPM → frontend, no el
contenido visual final.
Usuario admin temporal `[email protected]` usado para esta
auditoría — pendiente de borrar una vez aprobada esta propuesta.
auditoría — pendiente de borrar (ver sección 6).
## 6. Próximos pasos
## 6. Etapa 2 — implementada (2026-08-23 a 2026-08-25)
Pendiente de tu aprobación explícita antes de iniciar Etapa 2. Cuando
apruebes, Etapa 2 se dividirá en sub-fases (datos/categorías primero,
luego navegación, luego componentes visuales), validando en cada paso
contra estos 9 productos demo que precio/descripción/imágenes siguen
actualizándose correctamente desde Admin.
Aprobada y ejecutada en tres sub-fases, cada una validada en vivo
contra estos 9 productos demo:
1. **Datos/categorías** (2026-08-23/24): fix del bug de Store API que
impedía ver la categoría real de un producto (causa raíz del ADR
0004); 21 categorías/9 productos demo creados; `legacyProductMeta`
eliminado (PR #29) — 17 productos legacy con categoría/colección/
marca/rango de edad reales.
2. **Navegación** (2026-08-25): navbar y tiles del home pasaron de
arrays hardcodeados a data-driven contra el árbol real de
categorías (`getCategories()`, PR #32); guard de CI actualizado
(PR #33); fix de tipos (PR #34); home/layout forzados a render
dinámico para evitar que quedaran cacheados con datos mock (PR #35).
3. **Componentes visuales** (2026-08-25), los 4 patrones de la
sección 3 más el breadcrumb/facetas de Amazon: mega-menú
"Departamentos" multi-columna (PR #36), badges antes/ahora
(PR #37), swatches de variante inline (PR #38), mensaje de
urgencia por inventario real (PR #39), breadcrumb real + conteos
por filtro en el catálogo (PR #40), tiles promocionales
intercalados en la grilla (PR #41, con fix-forward de tipos en
PR #42).
El usuario admin temporal `[email protected]` (creado para la
auditoría de la sección 5) fue borrado el 2026-08-27 vía Medusa Admin.
Sin próxima etapa definida todavía — este ADR no especifica una
Etapa 3.
@@ -0,0 +1,330 @@
# ADR 0006 — Motor de búsqueda y filtros facetados (Fase 5, Etapa 3)
**Estado:** aceptada — Etapa 2 en implementación (aprobada 2026-08-26 tras diagnóstico de memoria del host)
**Fecha:** 2026-08-25
## Contexto
[ADR 0004](0004-evaluacion-stack-multi-categoria.md) ya identificó un
motor de búsqueda dedicado (Meilisearch, autohospedado) como "el
siguiente paso natural" para ARI Shopping, pero lo dejó fuera del
alcance de Etapa 2 explícitamente hasta que el catálogo se acercara al
techo real de ~100 productos activos que impone
`fetchProducts(filters, limit = 100)` en `medusa.ts`.
**Ese techo todavía no llegó** — el catálogo real hoy son 26 productos
(17 legacy + 9 demo, ver [ADR 0005](0005-rediseno-storefront.md)
sección 5), lejos de 100. El disparador de esta propuesta no es el
techo de volumen sino la **complejidad de facetas**: la Etapa 2 ya
construyó una taxonomía real de 6 categorías raíz / 21 nodos
(navegación, mega-menú, breadcrumb — PRs #36-42) y un filtrado con
conteo por valor (`categoryCounts`/`brandCounts`/`ageCounts` en
`catalog-client.tsx`, PR #40) que hoy se calcula **100% client-side**
sobre los productos ya traídos — el mismo patrón de "deuda de
alcance" que el hallazgo #4 de ADR 0004 ya señaló, ahora con más
dimensiones de filtro que multiplicar. Sigue siendo tratable a 26
productos; deja de serlo mucho antes de llegar a 100 si el catálogo
crece a la vez en número de productos y en dimensiones filtrables.
Restricción no negociable, igual que en toda la Fase 5: el flujo
**Admin → evento `product.updated` → `revalidate-storefront.ts` →
`revalidateTag("medusa-products")`**, que hoy mantiene el storefront
sincronizado sin intervención manual, no se toca. Cualquier índice de
búsqueda debe sincronizarse por el mismo mecanismo de eventos, nunca
por un cron ni un paso manual.
## 1. Estado real de la integración Medusa v2 + Meilisearch
Verificado contra la documentación oficial vigente el 2026-08-25, no
contra memoria de entrenamiento:
- **Medusa no tiene un módulo de búsqueda propio integrado con
Meilisearch hoy.** La guía oficial
(`docs.medusajs.com/resources/integrations/guides/meilisearch`) es
un tutorial de módulo custom desde cero (`npm install meilisearch` +
workflows/subscribers propios) — no un paquete instalable.
- **Medusa sí introdujo un "Search Module" propio en la versión
2.19.0.** El paquete comunitario `@rokmohar/medusa-plugin-meilisearch`
(el más usado y mantenido para Meilisearch+Medusa) se conecta a ese
Search Module desde su versión `^2.0.0`, pero esa versión del plugin
**requiere Medusa `^2.19.0`**.
- **Nuestro backend corre Medusa `2.18.0`** (`commerce-backend/package.json`)
— justo por debajo de ese corte. La tabla de compatibilidad del
plugin (`github.com/rokmohar/medusa-plugin-meilisearch`) resuelve
esto: la línea `^1.4.1` soporta `Medusa >= 2.15 < 2.19`, servidor
Meilisearch `>= 1.5`, Node `>= 22`. El propio repo lo explica: *"The
2.19 release removed the search interface the v1 line was written
against, so the two lines do not overlap: v1 stops at Medusa 2.18,
v2 starts at 2.19."*
- Node de `commerce-backend` ya es `22.18.0` (Dockerfile) — cumple el
mínimo del plugin sin cambios.
**Implicación concreta:** para adoptar Meilisearch sin subir de
versión Medusa (fuera de alcance de este ADR — sería su propia
migración), la pieza a instalar es `@rokmohar/medusa-plugin-meilisearch@^1.4.1`,
no la última versión del plugin. Esto es deuda técnica documentada,
no un obstáculo: si en el futuro se sube Medusa a `2.19+`, migrar a la
línea `^2.0.0` del plugin (y al Search Module nativo) es el camino
recto.
## 2. Impacto de recursos
Estado real del host ZimaOS al momento de esta auditoría (`free -h`,
`docker stats`): 15 GiB RAM totales, **swap al 100% (5.2/5.2 GiB
usados)**, solo 351 MiB realmente libres (6.5 GiB "available" vía
cache reclamable), 4 vCPU — la misma clase de contención que ya forzó
el fix de `cpu_shares` de Gitea. Cualquier servicio nuevo en este host
es una decisión de capacidad real, no teórica.
- **Meilisearch no publica un mínimo de RAM oficial.** Su propio FAQ
(`meilisearch.com/docs/resources/help/faq`) solo dice que "más RAM
implica búsquedas más rápidas" y recomienda ~10x el tamaño del
dataset crudo en disco — sin piso duro. La cifra de "~1 GiB
cómodo" que circula en blogs de la comunidad es una estimación, no
una garantía del proveedor — se cita aquí como referencia, no como
hecho verificado.
- Meilisearch usa almacenamiento memory-mapped: el índice puede vivir
parcialmente en disco cuando no entra en RAM, degradando latencia en
vez de fallar. A la escala actual (26 productos, dataset de KB) el
uso real de RAM del índice en sí es insignificante — el costo es el
proceso base de Meilisearch, no el dataset.
- **Comparación con Typesense** (pedida explícitamente, mantenida
breve porque la diferencia relevante para este caso es clara y no
requiere más profundidad): Typesense mantiene el índice completo en
RAM (~30% más RAM que Meilisearch para el mismo dataset, según
benchmarks públicos) y, a diferencia de Meilisearch, **no tiene
ningún plugin ni integración con Medusa** — habría que escribir la
sincronización completa desde cero igual que con la guía "custom"
de Meilisearch, pero sin ninguna base comunitaria mantenida
compatible con nuestra versión de Medusa. Con Meilisearch ya
teniendo un plugin activo para `Medusa 2.18` y menor huella de RAM,
no hay motivo para evaluar Typesense más a fondo en este caso.
- **Recomendación:** desplegar Meilisearch en k3d con `requests`/`limits`
explícitos y conservadores desde el día uno (ej. `request: 256Mi/0.1
CPU`, `limit: 768Mi/0.5 CPU` — a ajustar con datos reales de uso, no
supuestos), y tratarlo como el mismo tipo de riesgo operativo que ya
se vivió con Gitea: monitorear consumo real post-despliegue antes de
darlo por estable. Dado el swap ya saturado, este es el punto de
mayor incertidumbre de toda la propuesta — más que la elección de
motor de búsqueda en sí.
## 3. Modelo de indexación
Basado en la taxonomía real ya viva en Medusa (Etapa 2) y en
`Product`/`Category` de `lib/headless/types.ts`:
| Campo | Buscable (texto, tolerante a typos) | Filtrable / faceta | Ordenable |
|---|---|---|---|
| `name` | Sí | No | No |
| `shortDescription` / `description` | Sí | No | No |
| `category` (nombre de la hoja real) | Sí | Sí (con conteo) | No |
| `collection` | Sí | Sí (con conteo) | No |
| `brand` | Sí | Sí (con conteo) | No |
| `ageRange` | No | Sí (con conteo) | No |
| `tags` | Sí | No | No |
| `price` | No | Sí (rango) | Sí |
| `compareAtPrice` | No | No | No (solo se usa para el badge, no como filtro) |
| `isNew` / `featured` | No | No | Sí (orden "Novedades"/"Destacados", igual que hoy) |
| `available` | No | Sí (booleano) | No |
| `slug`, `thumbnail`, `id` | No | No | No (solo se devuelven, no se indexan como facetas) |
La jerarquía de categorías (raíz/hija) no se indexa como un campo
aparte: el índice guarda el nombre de la hoja real tal como hoy lo
hace `product.category`, y el breadcrumb/mega-menú siguen resolviendo
la jerarquía contra `/store/product-categories` (sin cambios, ya
funciona desde la sub-fase 2). El motor de búsqueda resuelve
**búsqueda de texto + conteo de facetas**, no reemplaza la fuente de
la jerarquía.
## 4. Diseño de sincronización
Mismo patrón que `revalidate-storefront.ts`, no uno nuevo: un
subscriber adicional en `commerce-backend/src/subscribers/` (ej.
`sync-search-index.ts`) que reacciona a eventos y le pide al plugin
que reindexe el documento afectado — nunca Medusa escribiendo
directamente en Meilisearch fuera de este canal.
Eventos a escuchar, confirmados en la documentación oficial de
eventos de Medusa (`docs.medusajs.com/resources/events-reference`):
- `product.created` / `product.updated` / `product.deleted`
- `product-variant.created` / `product-variant.updated` / `product-variant.deleted`
(mismo set que ya usa `revalidate-storefront.ts`, hoy validado en
producción)
- **Nuevo respecto al subscriber actual:** `product-category.created`
/ `product-category.updated` / `product-category.deleted` — un
cambio de nombre o jerarquía de categoría debe refrescar las
facetas del índice, y hoy `revalidate-storefront.ts` no escucha
estos eventos porque el storefront los resuelve en cada request
contra `/store/product-categories` directamente, sin necesidad de
revalidación. El índice de búsqueda sí necesita enterarse.
**Punto a verificar en vivo en Etapa 2, no asumido aquí:** existe al
menos un reporte público de un evento `product-collection.updated`
que no se disparaba de forma confiable en ciertas versiones de Medusa
(`github.com/medusajs/medusa/issues/11197`). Antes de dar por cerrado
el diseño de sincronización hay que repetir exactamente la validación
que ya se hizo para `revalidate-storefront.ts` en la auditoría de
Etapa 1 de Fase 5: crear/editar una categoría real vía Admin y
confirmar en vivo que el evento llega al subscriber, no confiar en que
la documentación coincide con el comportamiento real de `2.18.0`
(el mismo tipo de brecha que ya se encontró una vez con la relación
`+categories` de `/store/products`).
## 5. Diseño de UI — filtros facetados y búsqueda
**Filtros facetados (sidebar de `/catalog`):** mismo layout que ya
construyó la sub-fase 3 (`catalog-client.tsx`, PR #40) — sidebar con
"Categoría"/"Marca"/"Edad" como pills con conteo entre paréntesis y
slider de precio máximo. Lo único que cambia es **de dónde sale el
conteo**: hoy `categoryCounts`/`brandCounts`/`ageCounts` se calculan
recorriendo el arreglo de productos ya traído en el cliente; con el
motor de búsqueda, esos conteos vienen del propio Meilisearch
(`facetDistribution`) en la misma respuesta que trae los productos
filtrados, sin recorrer nada en el navegador. El breadcrumb no cambia
— sigue resolviéndose contra la jerarquía real de categorías, no
contra el índice de búsqueda.
```
Inicio > Mochilas y morrales > Marvel breadcrumb real (sin cambios)
┌────────────┬──────────────────────────────────────┐
│ Categoría │ [buscador con autocompletado] │ ← nuevo
│ ☐ Marvel(6)│ [card][card][card] │
│ ☐ Frozen(4)│ [card][card][card] │
│ Marca │ │
│ ☐ Disney(2)│ conteos ahora vienen de Meilisearch, │
│ Precio │ no de recorrer el arreglo en cliente │
│ [====○===] │ │
└────────────┴──────────────────────────────────────┘
```
**Búsqueda con autocompletado (barra del navbar):** hoy
`PredictiveSearch` (`components/search/predictive-search.tsx`) filtra
por substring sobre los productos ya cargados en memoria del cliente —
funciona a 26 productos, no escala y no tolera errores de tipeo (ADR
0004, hallazgo #5). Se reemplaza la fuente de datos del mismo
componente visual por `searchProducts(query)` del `CatalogProvider`
(interfaz que **ya existe** en `lib/headless/types.ts`, sin necesidad
de un tipo nuevo) apuntando a Meilisearch en vez de al `?q=` de
Medusa: tolerancia a typos y ranking por relevancia nativos del motor,
mismo contrato de datos que ya consume el dropdown de resultados —
cero cambio de componente React, solo de qué hay detrás de
`CatalogProvider.searchProducts`.
## Opciones consideradas
| Opción | A favor | En contra |
|---|---|---|
| **Meilisearch autohospedado** (`@rokmohar/medusa-plugin-meilisearch@^1.4.1`, sincronizado por eventos) | Plugin comunitario activo y compatible con nuestra versión exacta de Medusa (2.18); menor huella de RAM que la alternativa evaluada; almacenamiento memory-mapped se degrada en vez de fallar bajo presión de memoria; mismo patrón de subscriber que `revalidate-storefront.ts`, cero riesgo para ese flujo | Ningún paquete "oficial" de Medusa (el plugin es comunitario, no del core); atado a la línea `v1` del plugin mientras no se suba Medusa a 2.19+; un servicio más para operar en un host ya con swap saturado |
| **Typesense autohospedado** | Autocompletado ~2x más rápido según benchmarks públicos | Cero integración con Medusa (habría que escribir la sincronización completa desde cero, igual esfuerzo que la guía "custom" de Meilisearch pero sin base comunitaria); ~30% más RAM que Meilisearch para el mismo dataset, el peor trade-off posible dado el estado actual del host |
| **Módulo custom desde cero** (seguir la guía oficial de Medusa, sin el plugin de rokmohar) | Sin dependencia de un paquete de terceros; control total | Reimplementa lo que el plugin comunitario ya resuelve (declaración de índices, reindexado atómico, tracking de escrituras); más superficie propia para mantener sin ganancia clara sobre usar el plugin ya compatible con 2.18 |
| **No adoptar motor de búsqueda todavía** (seguir con filtrado 100% client-side) | Cero riesgo, cero componente nuevo | No resuelve la deuda de facetas ya identificada en ADR 0004 y agravada por la sub-fase 3; el techo de complejidad de filtros llega antes que el de 100 productos si el catálogo sigue creciendo en categorías |
## Decisión
**Meilisearch autohospedado**, vía `@rokmohar/medusa-plugin-meilisearch@^1.4.1`
(no la última versión del plugin — esa exige Medusa 2.19+, fuera de
alcance), sincronizado por el mismo patrón de subscriber por eventos
que ya usa `revalidate-storefront.ts`, con `product-category.*` sumado
al set de eventos existente. Se despliega en k3d con límites de
recursos conservadores desde el primer día y se trata como
experimento a monitorear, no como capacidad garantizada, dado el
estado actual de memoria del host.
**No se sube la versión de Medusa como parte de esta decisión** —
queda como deuda técnica documentada para cuando se justifique
migrar a la línea `v2` del plugin y al Search Module nativo de
Medusa.
## Consecuencias
- Se gana: conteo de facetas real (no client-side), búsqueda tolerante
a errores de tipeo con ranking por relevancia, y una arquitectura
que ya no depende del techo de 100 productos de `fetchProducts`.
- Se acepta como deuda técnica: quedar atado a la línea `v1` del
plugin de Meilisearch mientras Medusa siga en `2.18.x`; migrar a
Medusa `2.19+` es un proyecto aparte, no parte de esta Etapa 3.
- Riesgo operativo real y no resuelto todavía por este ADR: el host ya
corre con el swap saturado — el despliegue de Etapa 2 debe empezar
con límites de recursos conservadores y validación en vivo de
consumo real antes de confiar en el servicio, siguiendo la misma
lección que dejó el incidente de recursos de Gitea.
- Revertir esta decisión implica desinstalar el plugin y el
subscriber nuevo, y volver el buscador/filtrado a 100% client-side
— no hay migración de datos que deshacer, porque Meilisearch nunca
es la fuente de verdad, solo una copia de solo lectura indexada
desde Medusa.
## Nota de implementación (2026-08-26, antes de escribir código de Etapa 2)
Antes de tocar código se verificó el código fuente real de
`@rokmohar/medusa-plugin-meilisearch` en el tag exacto `v1.4.1`
(`github.com/rokmohar/medusa-plugin-meilisearch`), no su README
genérico ni memoria de entrenamiento — misma disciplina que ya destapó
el bug de `+categories` en `/store/products`. Aparecieron tres
desviaciones reales respecto al diseño de §4/§5 de este ADR:
1. **No se escribe un subscriber nuevo.** El plugin trae sus propios
subscribers (`meilisearch-product-upsert.ts`,
`meilisearch-category-upsert.ts`, variantes/colección/inventario),
auto-registrados al instalar el paquete — el propio README dice
*"You don't need custom subscribers anymore, you can remove them."*
Escribir `sync-search-index.ts` como describe §4 causaría doble
indexado. La escucha de `product-category.*` que pedía §4 ya viene
incluida en los subscribers del plugin.
2. **Las rutas Store API del plugin no devuelven `facetDistribution`**,
pese a que §5 asume que sí. El servicio interno
(`MeiliSearchService.search()`) sí puede pedirlo vía
`additionalOptions: { facets: [...] }`, pero las rutas
empaquetadas nunca lo piden ni lo reenvían. Se agrega una ruta
Store API propia (`src/api/store/search/route.ts`) que resuelve el
servicio del plugin desde el container y arma
`{ products, facetDistribution }`. Consecuencia positiva no
buscada: Meilisearch queda 100% `ClusterIP`-only, sin Ingress, sin
key pública ni ruta de red navegador→Meilisearch — todo pasa por
Medusa.
3. **El plugin se conecta vía un array `plugins:` nuevo en
`medusa-config.ts`**, no otra entrada en `modules:` (el único
precedente en este repo, el file-provider S3, usa `modules:`), y
usa nombres de env var propios (`MEILISEARCH_HOST`/
`MEILISEARCH_API_KEY`, leídos por el plugin) distintos del env var
del servidor Meilisearch en sí (`MEILI_MASTER_KEY`, leído por el
contenedor `getmeili/meilisearch`) — mismo valor, dos nombres de
secret.
Ninguna de las tres cambia la decisión ni sus consecuencias — cambian
el cómo, no el qué. Plan de implementación completo, fase por fase,
en la sesión que aprobó esta Etapa 2 (2026-08-26).
## Nota de implementación 2 (2026-08-26, gate en vivo de Fase 2)
Dos bugs reales encontrados y corregidos durante el gate de
verificación en vivo (crear/editar productos, categoría y colección
reales y confirmar que llegan a Meilisearch), ambos con la misma causa
raíz: `variants.calculated_price` de Medusa exige `region_id` **y**
`currency_code` resueltos en el contexto de pricing de la consulta, y
ninguno de los dos caminos nuevos de este ADR pasa por el middleware
nativo (`setPricingContext`) que se lo da gratis a `/store/products`:
1. El job de sync inicial del plugin (`sync-products.js`, corre una
vez al boot) hacía fallar el indexado de **todos** los productos
con `Method calculatePrices requires currency_code in the pricing
context` — las categorías sí indexaban bien (no llevan precio).
Arreglado indexando el precio crudo (`variants.prices`, sin
resolver promociones/reglas) en vez de `calculated_price` — alcanza
para el filtro/orden por precio de la búsqueda.
2. La ruta propia `/store/search` pegaba el mismo error al re-hidratar
productos vía `query.graph`. Pasar `region_id` solo no alcanzó;
hubo que resolver el `currency_code` real de la región con una
consulta adicional antes de armar el `QueryContext`.
Confirmado en vivo, con el fix aplicado: búsqueda por texto, facetas
sobre las 27 unidades del catálogo real, y precio calculado correcto
(`calculated_amount`/`currency_code`) en la respuesta de
`/store/search`. También confirmado en vivo que los tres eventos que
el ADR original marcaba como "verificar, no asumir" — `product.*`,
`product-category.*` y **`product-collection.updated`** (el que tenía
un reporte público de no disparar siempre) — sí sincronizan de forma
confiable en esta combinación exacta (Medusa 2.18.0 + plugin 1.4.1),
propagando un rename de categoría/colección al documento indexado en
menos de 10 segundos.
+2
View File
@@ -84,6 +84,7 @@ nav:
- Visión general: arquitectura/vision-general.md
- Red y exposición: arquitectura/red-y-exposicion.md
- Flujo GitOps: arquitectura/gitops-flujo.md
- Agente de monitoreo: arquitectura/agente-monitoreo.md
- Glosario: arquitectura/glosario.md
- Decisiones (ADRs):
- "0001 · Por qué k3d": decisiones/0001-por-que-k3d.md
@@ -91,6 +92,7 @@ nav:
- "0003 · HTTP/2 sobre QUIC": decisiones/0003-por-que-http2-sobre-quic-tunnel.md
- "0004 · Stack multi-categoría": decisiones/0004-evaluacion-stack-multi-categoria.md
- "0005 · Rediseño storefront": decisiones/0005-rediseno-storefront.md
- "0006 · Motor de búsqueda y facetas": decisiones/0006-motor-busqueda-facetas.md
- Plantilla ADR: decisiones/plantilla-adr.md
- Playbooks:
- Crashloop Medusa: playbooks/incidente-crashloop-medusa.md
@@ -25,7 +25,7 @@ spec:
runAsNonRoot: false
containers:
- name: app
image: gitea.cruzcloud.net/devops/docs-portal:v1.0.125
image: gitea.cruzcloud.net/devops/docs-portal:v1.0.173
# El base saca TODAS las capabilities. Probado en vivo y
# revertido: con drop: ALL (aunque se agregue NET_BIND_SERVICE
# de vuelta) el propio entrypoint de nginx:1.27-alpine falla al
@@ -0,0 +1,45 @@
import { searchWithFacets } from "@/lib/headless/client";
import type { ProductFilters } from "@/lib/headless/types";
// PredictiveSearch es un componente cliente y no puede pegarle directo
// al provider server-side (MEDUSA_BACKEND_URL es un nombre DNS interno
// del cluster, no resuelve desde el navegador) -- esta ruta es el
// puente same-origin.
export async function GET(request: Request) {
const { searchParams } = new URL(request.url);
const filters: ProductFilters = {
query: searchParams.get("q") ?? undefined,
category: searchParams.get("category") ?? undefined,
collection: searchParams.get("collection") ?? undefined,
brand: searchParams.get("brand") ?? undefined,
ageRange: searchParams.get("ageRange") ?? undefined,
};
const minPrice = searchParams.get("minPrice");
if (minPrice) filters.minPrice = Number(minPrice);
const maxPrice = searchParams.get("maxPrice");
if (maxPrice) filters.maxPrice = Number(maxPrice);
const limit = searchParams.get("limit");
if (limit) filters.limit = Number(limit);
try {
const result = await searchWithFacets(filters);
// provider actual no implementa searchWithFacets (mock/shopify) --
// degradar a vacío, no a 500; el componente cliente cae de nuevo a
// su propio filtrado si esta ruta no trae nada útil.
return Response.json(
result ?? { products: [], facetDistribution: {}, count: 0 },
);
} catch (error) {
return Response.json(
{
products: [],
facetDistribution: {},
count: 0,
error: (error as Error).message,
},
{ status: 502 },
);
}
}
@@ -19,3 +19,8 @@ data:
S3_ENDPOINT: http://minio-svc:9000
S3_REGION: us-east-1
S3_BUCKET: ari-products
# Leido por el plugin @rokmohar/medusa-plugin-meilisearch (nombre de
# env var del plugin, no del servidor Meilisearch en si -- ver
# MEILI_MASTER_KEY en secrets.template.yaml).
MEILISEARCH_HOST: http://meilisearch-svc:7700
@@ -4,6 +4,7 @@ metadata:
name: commerce-ingress
annotations:
traefik.ingress.kubernetes.io/router.entrypoints: web
traefik.ingress.kubernetes.io/router.middlewares: ecommerce-commerce-redirect-app@kubernetescrd
spec:
ingressClassName: traefik
rules:
@@ -7,5 +7,7 @@ resources:
- redis.yaml
- minio.yaml
- minio-bootstrap-job.yaml
- meilisearch.yaml
- medusa/
- ingress-commerce.yaml
- middleware-commerce-redirect.yaml
@@ -65,7 +65,7 @@ spec:
memory: 64Mi
- name: migrations
image: gitea.cruzcloud.net/devops/ecommerce-medusa:v1.0.127
image: gitea.cruzcloud.net/devops/ecommerce-medusa:v1.0.168
imagePullPolicy: IfNotPresent
command:
- node
@@ -86,7 +86,7 @@ spec:
containers:
- name: app
image: gitea.cruzcloud.net/devops/ecommerce-medusa:v1.0.127
image: gitea.cruzcloud.net/devops/ecommerce-medusa:v1.0.168
imagePullPolicy: IfNotPresent
envFrom:
- configMapRef:
@@ -0,0 +1,86 @@
apiVersion: v1
kind: Service
metadata:
name: meilisearch-svc
spec:
selector:
app: commerce-meilisearch
ports:
- name: http
port: 7700
targetPort: 7700
---
apiVersion: apps/v1
kind: StatefulSet
metadata:
name: commerce-meilisearch
spec:
serviceName: meilisearch-svc
replicas: 1
selector:
matchLabels:
app: commerce-meilisearch
template:
metadata:
labels:
app: commerce-meilisearch
spec:
terminationGracePeriodSeconds: 30
containers:
- name: meilisearch
image: getmeili/meilisearch:v1.53.1
imagePullPolicy: IfNotPresent
env:
- name: MEILI_MASTER_KEY
valueFrom:
secretKeyRef:
name: commerce-secrets
key: MEILI_MASTER_KEY
- name: MEILI_ENV
value: production
# Sin telemetria a un tercero por defecto (misma postura del
# resto del namespace: nada de egress no explicado).
- name: MEILI_NO_ANALYTICS
value: "true"
ports:
- name: http
containerPort: 7700
# /health no requiere auth (confirmado en vivo) -- sirve tal
# cual para las dos probes, igual que minio usa sus propios
# endpoints /minio/health/*.
readinessProbe:
httpGet:
path: /health
port: http
initialDelaySeconds: 5
periodSeconds: 10
timeoutSeconds: 5
failureThreshold: 6
livenessProbe:
httpGet:
path: /health
port: http
initialDelaySeconds: 15
periodSeconds: 20
timeoutSeconds: 5
failureThreshold: 6
resources:
requests:
cpu: 100m
memory: 256Mi
limits:
cpu: 500m
memory: 768Mi
volumeMounts:
- name: meilisearch-data
mountPath: /meili_data
volumeClaimTemplates:
- metadata:
name: meilisearch-data
spec:
accessModes:
- ReadWriteOnce
storageClassName: local-path
resources:
requests:
storage: 5Gi
@@ -0,0 +1,16 @@
apiVersion: traefik.io/v1alpha1
kind: Middleware
metadata:
name: commerce-redirect-app
spec:
# Redirige solo la raíz exacta de commerce.cruzcloud.net a /app (admin de
# Medusa) — el regex incluye el host a propósito: esta Middleware se
# referencia desde el mismo Ingress que también enruta media.cruzcloud.net,
# y sin el host en el regex un request a la raíz de ese otro dominio
# también matchearía (ambas rutas comparten Ingress -> mismas
# router.middlewares). Si el regex no matchea, Traefik deja pasar el
# request sin modificarlo.
redirectRegex:
regex: "^https?://commerce\\.cruzcloud\\.net/$"
replacement: "https://commerce.cruzcloud.net/app"
permanent: true
@@ -23,6 +23,14 @@ stringData:
JWT_SECRET: CAMBIAR
COOKIE_SECRET: CAMBIAR
# Master key del servidor Meilisearch (StatefulSet commerce-meilisearch,
# env MEILI_MASTER_KEY del propio contenedor getmeili/meilisearch) Y
# credencial que usa el plugin de Medusa para autenticarse contra el
# -- MISMO valor, dos nombres de secret porque cada lado (servidor vs.
# plugin cliente) usa su propia convencion de nombre de env var.
MEILI_MASTER_KEY: CAMBIAR
MEILISEARCH_API_KEY: CAMBIAR
# Debe ser el MISMO valor que REVALIDATE_SECRET en el Secret
# commerce-storefront: el subscriber de Medusa lo envía como header
# x-revalidate-secret al invocar POST /api/revalidate en el storefront.
@@ -3,10 +3,18 @@
import Link from "next/link";
import { ChevronRight, SlidersHorizontal, X } from "lucide-react";
import { useEffect, useMemo, useState } from "react";
import { PromoTile } from "@/components/home/category-tiles";
import { ProductCard } from "@/components/product/product-card";
import { Button } from "@/components/ui/button";
import { Input } from "@/components/ui/input";
import type { Category, Product } from "@/lib/headless/types";
import type { Category, FacetDistribution, Product } from "@/lib/headless/types";
// Cada cuantas tarjetas de producto se intercala un tile promocional en
// la grilla (patron Walmart, ADR 0005 seccion 3): grid con tiles
// promocionales intercalados, no solo tarjetas homogeneas.
const PROMO_TILE_INTERVAL = 8;
type GridItem = { type: "product"; product: Product } | { type: "promo"; category: Category };
type CatalogClientProps = {
products: Product[];
@@ -96,6 +104,11 @@ export function CatalogClient({
// trata como el nombre de categoria exacto de siempre.
const allowedCategories = categoryGroups[category] ?? [category];
const breadcrumb = useMemo(() => (category === "Todos" ? [] : buildBreadcrumb(categoryNodes, category)), [categoryNodes, category]);
// No promociona la misma categoria raiz que el usuario ya esta viendo.
const promoRoots = useMemo(() => {
const excludeRootId = breadcrumb[0]?.id;
return categoryNodes.filter((node) => !node.parentId && node.id !== excludeRootId);
}, [categoryNodes, breadcrumb]);
const filtered = useMemo(
() =>
@@ -119,11 +132,34 @@ export function CatalogClient({
[age, allowedCategories, brand, category, collection, maxPrice, products, query, sort],
);
// Intercala un tile promocional cada PROMO_TILE_INTERVAL productos,
// ciclando por las categorias raiz -- si no hay jerarquia de
// categorias disponible, la grilla queda igual que antes (solo
// productos).
const gridItems = useMemo<GridItem[]>(() => {
if (promoRoots.length === 0) return filtered.map((product) => ({ type: "product", product }));
const items: GridItem[] = [];
let promoIndex = 0;
filtered.forEach((product, index) => {
items.push({ type: "product", product });
const isLast = index === filtered.length - 1;
const promoCategory = promoRoots[promoIndex % promoRoots.length];
if (!isLast && (index + 1) % PROMO_TILE_INTERVAL === 0 && promoCategory) {
items.push({ type: "promo", category: promoCategory });
promoIndex += 1;
}
});
return items;
}, [filtered, promoRoots]);
// Conteos por valor de filtro (patron Amazon: "Marca (12)"), calculados
// sobre los productos que ya pasan el resto de filtros activos -- cada
// dimension se cuenta excluyendose a si misma para reflejar cuantos
// resultados habria si se eligiera ese valor.
const categoryCounts = useMemo(() => {
// resultados habria si se eligiera ese valor. Sirven de fallback cuando
// el provider activo no soporta facetas reales (mock/shopify) o cuando
// la busqueda en vivo todavia no respondio.
const fallbackCategoryCounts = useMemo(() => {
const base = products.filter(
(product) => matchesQuery(product, query) && matchesCollection(product, collection) && matchesBrand(product, brand) && matchesAge(product, age) && matchesPrice(product, maxPrice),
);
@@ -136,7 +172,7 @@ export function CatalogClient({
return counts;
}, [age, brand, categories, categoryGroups, collection, maxPrice, products, query]);
const brandCounts = useMemo(() => {
const fallbackBrandCounts = useMemo(() => {
const base = products.filter(
(product) =>
matchesQuery(product, query) &&
@@ -153,7 +189,7 @@ export function CatalogClient({
return counts;
}, [age, allowedCategories, brands, category, collection, maxPrice, products, query]);
const ageCounts = useMemo(() => {
const fallbackAgeCounts = useMemo(() => {
const base = products.filter(
(product) =>
matchesQuery(product, query) &&
@@ -170,6 +206,108 @@ export function CatalogClient({
return counts;
}, [ages, allowedCategories, brand, category, collection, maxPrice, products, query]);
// "Mochilas y morrales" etc.: nombre sintetico de grupo que no existe
// como valor indexado en Meilisearch (ningun producto lo tiene como
// category real, solo sus hijos -- ver categoryGroups arriba). Mientras
// esa sea la categoria activa, las facetas en vivo de marca/edad no
// pueden filtrar por ella y se degrada al conteo client-side, que si
// sabe expandir el grupo via matchesCategory/allowedCategories.
const categoryIsLeafOrNone = category === "Todos" || !categoryGroups[category];
// Conteos reales via Meilisearch (ADR 0006 Etapa 2 Fase 3b): un fetch
// por dimension contra /api/search, cada uno omitiendo su propio filtro
// (mismo criterio de auto-exclusion que el fallback de arriba) para que
// los conteos reflejen el subconjunto ya filtrado por las OTRAS
// dimensiones y se puedan seguir eligiendo otros valores. `limit=0` evita
// rehidratar precios de productos que no vamos a mostrar -- solo importan
// `facetDistribution` y `count`.
const [liveFacets, setLiveFacets] = useState<{
category: Record<string, number> | null;
categoryTotal: number | null;
brand: Record<string, number> | null;
brandTotal: number | null;
ageRange: Record<string, number> | null;
ageTotal: number | null;
}>({ category: null, categoryTotal: null, brand: null, brandTotal: null, ageRange: null, ageTotal: null });
useEffect(() => {
const controller = new AbortController();
function buildParams(omit: "category" | "brand" | "ageRange"): URLSearchParams {
const params = new URLSearchParams({ limit: "0" });
if (query) params.set("q", query);
if (collection) params.set("collection", collection);
if (omit !== "category" && category !== "Todos" && categoryIsLeafOrNone) {
params.set("category", category);
}
if (omit !== "brand" && brand !== "Todas") params.set("brand", brand);
if (omit !== "ageRange" && age !== "Todas") params.set("ageRange", age);
if (maxPrice > 0) params.set("maxPrice", String(maxPrice));
return params;
}
async function fetchFacet(omit: "category" | "brand" | "ageRange") {
const response = await fetch(`/api/search?${buildParams(omit).toString()}`, {
signal: controller.signal,
});
return (await response.json()) as { facetDistribution?: FacetDistribution };
}
// El "count"/estimatedTotalHits que devuelve /api/search viene en 0 con
// limit=0 (confirmado en vivo contra Meilisearch real -- limit>=1 lo
// reporta bien, limit=0 no). facetDistribution si es correcto con
// limit=0, y cada producto tiene siempre un valor para cada dimension
// (mapProduct usa defaults como "ARI Shopping"/"Todas", nunca null), asi
// que sumar sus valores da el total real sin depender de ese campo.
function sumFacetValues(facet: Record<string, number> | null): number {
return facet ? Object.values(facet).reduce((sum, value) => sum + value, 0) : 0;
}
const timer = setTimeout(() => {
Promise.all([fetchFacet("category"), fetchFacet("brand"), fetchFacet("ageRange")])
.then(([categoryRes, brandRes, ageRes]) => {
// Un facetDistribution vacio ({}) es la senal de degrade explicita
// de /api/search (provider sin searchWithFacets, o Meilisearch
// sin indices todavia) -- distinto de una faceta real con 0
// resultados, que igual trae la clave presente.
const categoryFacet = categoryRes.facetDistribution && "category" in categoryRes.facetDistribution ? categoryRes.facetDistribution.category : null;
const brandFacet = brandRes.facetDistribution && "brand" in brandRes.facetDistribution ? brandRes.facetDistribution.brand : null;
const ageFacet = ageRes.facetDistribution && "ageRange" in ageRes.facetDistribution ? ageRes.facetDistribution.ageRange : null;
setLiveFacets({
category: categoryFacet,
categoryTotal: categoryFacet ? sumFacetValues(categoryFacet) : null,
brand: categoryIsLeafOrNone ? brandFacet : null,
brandTotal: categoryIsLeafOrNone && brandFacet ? sumFacetValues(brandFacet) : null,
ageRange: categoryIsLeafOrNone ? ageFacet : null,
ageTotal: categoryIsLeafOrNone && ageFacet ? sumFacetValues(ageFacet) : null,
});
})
.catch((error) => {
if ((error as Error).name !== "AbortError") {
setLiveFacets({ category: null, categoryTotal: null, brand: null, brandTotal: null, ageRange: null, ageTotal: null });
}
});
}, 300);
return () => {
clearTimeout(timer);
controller.abort();
};
}, [age, brand, category, categoryIsLeafOrNone, collection, maxPrice, query]);
const categoryCounts = useMemo(
() => (liveFacets.category ? { Todos: liveFacets.categoryTotal ?? 0, ...liveFacets.category } : fallbackCategoryCounts),
[fallbackCategoryCounts, liveFacets.category, liveFacets.categoryTotal],
);
const brandCounts = useMemo(
() => (liveFacets.brand ? { Todas: liveFacets.brandTotal ?? 0, ...liveFacets.brand } : fallbackBrandCounts),
[fallbackBrandCounts, liveFacets.brand, liveFacets.brandTotal],
);
const ageCounts = useMemo(
() => (liveFacets.ageRange ? { Todas: liveFacets.ageTotal ?? 0, ...liveFacets.ageRange } : fallbackAgeCounts),
[fallbackAgeCounts, liveFacets.ageRange, liveFacets.ageTotal],
);
const reset = () => {
setQuery("");
setCategory("Todos");
@@ -284,9 +422,13 @@ export function CatalogClient({
{filtered.length ? (
<div className="grid gap-5 sm:grid-cols-2 xl:grid-cols-3">
{filtered.map((product) => (
<ProductCard key={product.id} product={product} />
))}
{gridItems.map((item, index) =>
item.type === "product" ? (
<ProductCard key={item.product.id} product={item.product} />
) : (
<PromoTile key={`promo-${index}`} category={item.category} className="min-h-[22rem]" />
),
)}
</div>
) : (
<div className="grid min-h-80 place-items-center rounded-3xl border border-dashed border-slate-300 text-center">
@@ -2,6 +2,7 @@ import Image from "next/image";
import Link from "next/link";
import { ArrowUpRight } from "lucide-react";
import type { Category } from "@/lib/headless/types";
import { cn } from "@/lib/utils";
type TileContent = { subtitle: string; image?: string; color: string };
@@ -20,6 +21,37 @@ const tileContent: Record<string, TileContent> = {
const defaultTile: TileContent = { subtitle: "Explora esta categoría", color: "from-slate-400 to-slate-600" };
// Tile individual reutilizado tanto por la seccion de mundos del home
// como por el grid del catalogo (tiles promocionales intercalados,
// ADR 0005 seccion 3, patron Walmart) -- mismo primitivo visual, no un
// componente nuevo.
export function PromoTile({ category, className = "" }: { category: Category; className?: string }) {
const content = tileContent[category.handle] ?? defaultTile;
return (
<Link
href={`/catalog?category=${encodeURIComponent(category.name)}`}
className={cn(`group relative min-h-64 overflow-hidden rounded-[2rem] bg-gradient-to-br ${content.color} shadow-xl`, className)}
>
{content.image && (
<>
<Image src={content.image} alt={category.name} fill sizes="(max-width: 640px) 100vw, (max-width: 1024px) 33vw, 16vw" className="object-cover transition duration-500 group-hover:scale-105" />
<div className="absolute inset-0 bg-gradient-to-t from-slate-950/80 via-transparent to-white/5" />
</>
)}
<div className="absolute inset-x-0 bottom-0 flex items-end justify-between p-6 text-white">
<div>
<h3 className="text-xl font-black">{category.name}</h3>
<p className="mt-1 text-sm font-semibold text-white/80">{content.subtitle}</p>
</div>
<span className="grid size-11 shrink-0 place-items-center rounded-full bg-white text-slate-950 transition group-hover:rotate-45">
<ArrowUpRight />
</span>
</div>
</Link>
);
}
export function CategoryTiles({ categories }: { categories: Category[] }) {
const roots = categories.filter((category) => !category.parentId);
@@ -35,33 +67,9 @@ export function CategoryTiles({ categories }: { categories: Category[] }) {
<Link href="/catalog" className="hidden font-black text-cyan-700 sm:block">Ver todo →</Link>
</div>
<div className="grid gap-5 sm:grid-cols-2 lg:grid-cols-3 xl:grid-cols-6">
{roots.map((category) => {
const content = tileContent[category.handle] ?? defaultTile;
return (
<Link
key={category.id}
href={`/catalog?category=${encodeURIComponent(category.name)}`}
className={`group relative min-h-64 overflow-hidden rounded-[2rem] bg-gradient-to-br ${content.color} shadow-xl`}
>
{content.image && (
<>
<Image src={content.image} alt={category.name} fill sizes="(max-width: 640px) 100vw, (max-width: 1024px) 33vw, 16vw" className="object-cover transition duration-500 group-hover:scale-105" />
<div className="absolute inset-0 bg-gradient-to-t from-slate-950/80 via-transparent to-white/5" />
</>
)}
<div className="absolute inset-x-0 bottom-0 flex items-end justify-between p-6 text-white">
<div>
<h3 className="text-xl font-black">{category.name}</h3>
<p className="mt-1 text-sm font-semibold text-white/80">{content.subtitle}</p>
</div>
<span className="grid size-11 shrink-0 place-items-center rounded-full bg-white text-slate-950 transition group-hover:rotate-45">
<ArrowUpRight />
</span>
</div>
</Link>
);
})}
{roots.map((category) => (
<PromoTile key={category.id} category={category} />
))}
</div>
</section>
);
@@ -56,7 +56,7 @@ export function Navbar({ products, categories }: { products: Product[]; categori
<Link href="/" className="relative size-16 overflow-hidden rounded-full border-2 border-slate-900 bg-black shadow-lg lg:size-20">
<Image src="/images/brand/logo-ari.png" alt="ARI Shopping" fill priority sizes="80px" className="object-cover" />
</Link>
<PredictiveSearch products={products} />
<PredictiveSearch />
<div className="flex items-center gap-2">
<Button variant="outline" size="icon" aria-label="Mi cuenta" className="hidden border-slate-300 bg-white text-slate-900 hover:border-violet-400 hover:bg-violet-50 sm:inline-flex" onClick={() => setAccountOpen(true)}><UserRound /></Button>
<Button variant="outline" size="icon" aria-label={`Favoritos: ${wishlistCount}`} className="relative hidden border-slate-300 bg-white text-slate-900 hover:border-fuchsia-400 hover:bg-fuchsia-50 sm:inline-flex" onClick={openWishlist}>
@@ -2,12 +2,12 @@
import Image from "next/image";
import Link from "next/link";
import { Heart, ShoppingBag, Sparkles } from "lucide-react";
import { Heart, ShoppingBag, Sparkles, TriangleAlert } from "lucide-react";
import { useState } from "react";
import { Badge } from "@/components/ui/badge";
import { Button } from "@/components/ui/button";
import type { Product } from "@/lib/headless/types";
import { formatMoney } from "@/lib/utils";
import { formatMoney, LOW_STOCK_THRESHOLD } from "@/lib/utils";
import { useCartStore } from "@/store/cart-store";
import { useWishlistStore } from "@/store/wishlist-store";
@@ -23,6 +23,14 @@ export function ProductCard({ product, priority = false }: { product: Product; p
: null;
const [selectedVariantId, setSelectedVariantId] = useState(product.variants[0]?.id);
const selectedVariant = product.variants.find((variant) => variant.id === selectedVariantId) ?? product.variants[0];
const lowStock =
selectedVariant?.inventoryQuantity !== null &&
selectedVariant?.inventoryQuantity !== undefined &&
selectedVariant.inventoryQuantity > 0 &&
selectedVariant.inventoryQuantity <= LOW_STOCK_THRESHOLD
? selectedVariant.inventoryQuantity
: null;
return (
<article className="group flex h-full flex-col overflow-hidden rounded-3xl border border-slate-100 bg-white shadow-lg shadow-slate-900/5 transition duration-300 hover:-translate-y-1.5 hover:shadow-2xl hover:shadow-violet-500/10">
@@ -56,6 +64,9 @@ export function ProductCard({ product, priority = false }: { product: Product; p
))}
</div>
)}
{lowStock !== null && (
<span className="flex items-center gap-1.5 text-xs font-bold text-amber-600"><TriangleAlert className="size-3.5" />Quedan {lowStock}</span>
)}
<div className="mt-auto flex items-center justify-between gap-3 pt-1">
<div className="flex flex-col">
{product.compareAtPrice && <span className="text-xs font-bold text-slate-400 line-through">{formatMoney(product.compareAtPrice, product.currency)}</span>}
@@ -1,14 +1,22 @@
"use client";
import { CheckCircle2, ShoppingBag, Truck } from "lucide-react";
import { CheckCircle2, ShoppingBag, TriangleAlert, Truck } from "lucide-react";
import { useState } from "react";
import { Button } from "@/components/ui/button";
import type { Product } from "@/lib/headless/types";
import { formatMoney } from "@/lib/utils";
import { formatMoney, LOW_STOCK_THRESHOLD } from "@/lib/utils";
import { useCartStore } from "@/store/cart-store";
export function ProductPurchase({ product }: { product: Product }) {
const [variantId, setVariantId] = useState(product.variants[0]?.id);
const addItem = useCartStore((state) => state.addItem);
return <div><div className="mb-5 flex flex-wrap gap-2">{product.variants.map((variant) => <button key={variant.id} onClick={() => setVariantId(variant.id)} className={`rounded-full border px-4 py-2 text-sm font-bold ${variantId === variant.id ? "border-fuchsia-500 bg-fuchsia-50 text-fuchsia-700" : "border-slate-200"}`}>{variant.value}</button>)}</div><div className="mb-6 flex items-end gap-3"><strong className="text-3xl font-black">{formatMoney(product.price, product.currency)}</strong>{product.compareAtPrice && <del className="text-slate-400">{formatMoney(product.compareAtPrice, product.currency)}</del>}</div><Button variant="magenta" size="lg" className="w-full" onClick={() => addItem(product, variantId)}><ShoppingBag />Añadir al carrito</Button><div className="mt-5 grid gap-3 rounded-2xl bg-slate-50 p-4 text-sm"><span className="flex items-center gap-2 font-bold"><CheckCircle2 className="size-4 text-emerald-500" />Disponibilidad por confirmar</span><span className="flex items-center gap-2 font-bold"><Truck className="size-4 text-cyan-600" />Envíos nacionales</span></div></div>;
const selectedVariant = product.variants.find((variant) => variant.id === variantId) ?? product.variants[0];
const lowStock =
selectedVariant?.inventoryQuantity !== null &&
selectedVariant?.inventoryQuantity !== undefined &&
selectedVariant.inventoryQuantity > 0 &&
selectedVariant.inventoryQuantity <= LOW_STOCK_THRESHOLD
? selectedVariant.inventoryQuantity
: null;
return <div><div className="mb-5 flex flex-wrap gap-2">{product.variants.map((variant) => <button key={variant.id} onClick={() => setVariantId(variant.id)} className={`rounded-full border px-4 py-2 text-sm font-bold ${variantId === variant.id ? "border-fuchsia-500 bg-fuchsia-50 text-fuchsia-700" : "border-slate-200"}`}>{variant.value}</button>)}</div><div className="mb-6 flex items-end gap-3"><strong className="text-3xl font-black">{formatMoney(product.price, product.currency)}</strong>{product.compareAtPrice && <del className="text-slate-400">{formatMoney(product.compareAtPrice, product.currency)}</del>}</div><Button variant="magenta" size="lg" className="w-full" onClick={() => addItem(product, variantId)}><ShoppingBag />Añadir al carrito</Button><div className="mt-5 grid gap-3 rounded-2xl bg-slate-50 p-4 text-sm">{lowStock !== null ? <span className="flex items-center gap-2 font-bold text-amber-600"><TriangleAlert className="size-4" />Quedan {lowStock} unidades</span> : <span className="flex items-center gap-2 font-bold"><CheckCircle2 className="size-4 text-emerald-500" />Disponibilidad por confirmar</span>}<span className="flex items-center gap-2 font-bold"><Truck className="size-4 text-cyan-600" />Envíos nacionales</span></div></div>;
}
@@ -4,20 +4,45 @@ import Image from "next/image";
import Link from "next/link";
import { useRouter } from "next/navigation";
import { Search, X } from "lucide-react";
import { FormEvent, useMemo, useState } from "react";
import { FormEvent, useEffect, useState } from "react";
import { Input } from "@/components/ui/input";
import type { Product } from "@/lib/headless/types";
export function PredictiveSearch({ products }: { products: Product[] }) {
export function PredictiveSearch() {
const router = useRouter();
const [query, setQuery] = useState("");
const normalized = query.trim().toLocaleLowerCase("es");
const matches = useMemo(
() => normalized.length < 2
? []
: products.filter((product) => [product.name, product.category, product.collection, product.brand].join(" ").toLocaleLowerCase("es").includes(normalized)).slice(0, 6),
[normalized, products],
);
const [matches, setMatches] = useState<Product[]>([]);
const [loading, setLoading] = useState(false);
const normalized = query.trim();
useEffect(() => {
if (normalized.length < 2) {
setMatches([]);
setLoading(false);
return;
}
const controller = new AbortController();
setLoading(true);
const timer = setTimeout(() => {
fetch(`/api/search?q=${encodeURIComponent(normalized)}`, {
signal: controller.signal,
})
.then((response) => response.json())
.then((data: { products?: Product[] }) => {
setMatches((data.products ?? []).slice(0, 6));
})
.catch((error) => {
if ((error as Error).name !== "AbortError") setMatches([]);
})
.finally(() => setLoading(false));
}, 250);
return () => {
clearTimeout(timer);
controller.abort();
};
}, [normalized]);
const submit = (event: FormEvent<HTMLFormElement>) => {
event.preventDefault();
@@ -36,6 +61,11 @@ export function PredictiveSearch({ products }: { products: Product[] }) {
{query && <button type="button" aria-label="Limpiar búsqueda" className="absolute right-4 top-1/2 -translate-y-1/2 rounded-full p-1 text-slate-400 hover:bg-slate-100" onClick={() => setQuery("")}><X className="size-4" /></button>}
</div>
</div>
{loading && normalized.length >= 2 && matches.length === 0 && (
<div className="absolute left-0 right-0 top-[calc(100%+10px)] z-50 rounded-2xl border border-slate-200 bg-white p-4 text-center text-sm text-slate-500 shadow-2xl">
Buscando…
</div>
)}
{matches.length > 0 && (
<div className="absolute left-0 right-0 top-[calc(100%+10px)] z-50 overflow-hidden rounded-2xl border border-slate-200 bg-white p-2 shadow-2xl">
{matches.map((product) => (
+34 -17
View File
@@ -30,7 +30,8 @@
"id": "ari-001-toy-story-5-rex-default",
"name": "Presentación",
"value": "Edición original",
"available": true
"available": true,
"inventoryQuantity": null
}
],
"thumbnail": "/images/thumbnails/ari-001-toy-story-5-rex.webp"
@@ -66,7 +67,8 @@
"id": "ari-002-toy-story-5-buzz-lightyear-default",
"name": "Presentación",
"value": "Edición original",
"available": true
"available": true,
"inventoryQuantity": null
}
],
"thumbnail": "/images/thumbnails/ari-002-toy-story-5-buzz-lightyear.webp"
@@ -102,7 +104,8 @@
"id": "ari-003-toy-story-5-jessie-default",
"name": "Presentación",
"value": "Edición original",
"available": true
"available": true,
"inventoryQuantity": null
}
],
"thumbnail": "/images/thumbnails/ari-003-toy-story-5-jessie.webp"
@@ -138,7 +141,8 @@
"id": "ari-004-toy-story-5-woody-default",
"name": "Presentación",
"value": "Edición original",
"available": true
"available": true,
"inventoryQuantity": null
}
],
"thumbnail": "/images/thumbnails/ari-004-toy-story-5-woody.webp"
@@ -174,7 +178,8 @@
"id": "ari-005-intensamente-furia-default",
"name": "Presentación",
"value": "Edición original",
"available": true
"available": true,
"inventoryQuantity": null
}
],
"thumbnail": "/images/thumbnails/ari-005-intensamente-furia.webp"
@@ -210,7 +215,8 @@
"id": "ari-006-intensamente-tristeza-default",
"name": "Presentación",
"value": "Edición original",
"available": true
"available": true,
"inventoryQuantity": null
}
],
"thumbnail": "/images/thumbnails/ari-006-intensamente-tristeza.webp"
@@ -246,7 +252,8 @@
"id": "ari-007-minnie-y-mickey-mouse-default",
"name": "Presentación",
"value": "Edición original",
"available": true
"available": true,
"inventoryQuantity": null
}
],
"thumbnail": "/images/thumbnails/ari-007-minnie-y-mickey-mouse.webp"
@@ -282,7 +289,8 @@
"id": "ari-008-angel-lilo-y-stitch-default",
"name": "Presentación",
"value": "Edición original",
"available": true
"available": true,
"inventoryQuantity": null
}
],
"thumbnail": "/images/thumbnails/ari-008-angel-lilo-y-stitch.webp"
@@ -318,7 +326,8 @@
"id": "ari-009-mochilas-disney-coleccion-marvel-default",
"name": "Presentación",
"value": "Rosa",
"available": true
"available": true,
"inventoryQuantity": null
}
],
"thumbnail": "/images/thumbnails/ari-009-mochilas-disney-coleccion-marvel.webp"
@@ -354,7 +363,8 @@
"id": "ari-010-mochila-ligera-y-resistente-al-agua-de-d-default",
"name": "Presentación",
"value": "Rosa",
"available": true
"available": true,
"inventoryQuantity": null
}
],
"thumbnail": "/images/thumbnails/ari-010-mochila-ligera-y-resistente-al-agua-de-d.webp"
@@ -390,7 +400,8 @@
"id": "ari-011-mochila-escolar-de-spider-man-de-disney--default",
"name": "Presentación",
"value": "Rosa",
"available": true
"available": true,
"inventoryQuantity": null
}
],
"thumbnail": "/images/thumbnails/ari-011-mochila-escolar-de-spider-man-de-disney-.webp"
@@ -426,7 +437,8 @@
"id": "ari-012-disney-mochila-de-poliester-ultraligera--default",
"name": "Presentación",
"value": "Rosa",
"available": true
"available": true,
"inventoryQuantity": null
}
],
"thumbnail": "/images/thumbnails/ari-012-disney-mochila-de-poliester-ultraligera-.webp"
@@ -462,7 +474,8 @@
"id": "ari-013-mochila-pequena-bob-esponja-default",
"name": "Presentación",
"value": "Rosa",
"available": true
"available": true,
"inventoryQuantity": null
}
],
"thumbnail": "/images/thumbnails/ari-013-mochila-pequena-bob-esponja.webp"
@@ -498,7 +511,8 @@
"id": "ari-014-mochila-casual-de-gran-capacidad-con-est-default",
"name": "Presentación",
"value": "Rosa",
"available": true
"available": true,
"inventoryQuantity": null
}
],
"thumbnail": "/images/thumbnails/ari-014-mochila-casual-de-gran-capacidad-con-est.webp"
@@ -534,7 +548,8 @@
"id": "ari-015-mochila-escolar-estilo-britanico-de-disn-default",
"name": "Presentación",
"value": "Rosa",
"available": true
"available": true,
"inventoryQuantity": null
}
],
"thumbnail": "/images/thumbnails/ari-015-mochila-escolar-estilo-britanico-de-disn.webp"
@@ -570,7 +585,8 @@
"id": "ari-016-disney-mochila-iron-man-de-poliester-ult-default",
"name": "Presentación",
"value": "Rosa",
"available": true
"available": true,
"inventoryQuantity": null
}
],
"thumbnail": "/images/thumbnails/ari-016-disney-mochila-iron-man-de-poliester-ult.webp"
@@ -606,7 +622,8 @@
"id": "ari-017-mochila-de-disney-marvel-capitan-america-default",
"name": "Presentación",
"value": "Rosa",
"available": true
"available": true,
"inventoryQuantity": null
}
],
"thumbnail": "/images/thumbnails/ari-017-mochila-de-disney-marvel-capitan-america.webp"
@@ -25,7 +25,7 @@ spec:
runAsUser: 1001
containers:
- name: app
image: gitea.cruzcloud.net/devops/ecommerce-frontend:v1.0.147
image: gitea.cruzcloud.net/devops/ecommerce-frontend:v1.0.172
env:
- name: HEADLESS_PROVIDER
value: medusa
@@ -26,3 +26,7 @@ export async function searchProducts(query: string, limit?: number) {
export async function getCategories() {
return provider().getCategories();
}
export async function searchWithFacets(filters?: ProductFilters) {
return provider().searchWithFacets?.(filters);
}
@@ -1,6 +1,8 @@
import type {
CatalogProvider,
Category,
FacetDistribution,
FacetedSearchResult,
Product,
ProductFilters,
ProductVariant,
@@ -157,11 +159,19 @@ function mapVariant(variant: MedusaVariant): ProductVariant {
variant.manage_inventory === false ||
(variant.inventory_quantity ?? 0) > 0;
// null = inventario no gestionado (backorder o manage_inventory=false),
// no hay un numero real que mostrar como "quedan N".
const inventoryQuantity =
variant.allow_backorder === true || variant.manage_inventory === false
? null
: (variant.inventory_quantity ?? 0);
return {
id: variant.id,
name: option?.option?.title || "Presentación",
value: option?.value || variant.title || "Estándar",
available,
inventoryQuantity,
};
}
@@ -377,6 +387,56 @@ async function fetchCategories(): Promise<Category[]> {
}));
}
type MedusaSearchResponse = {
products: MedusaProduct[];
facetDistribution?: FacetDistribution;
count?: number;
};
// Pega contra /store/search (ADR 0006 Etapa 2) en vez de /store/products
// -- Meilisearch resuelve texto tolerante a errores de tipeo y facetas
// reales, la ruta del backend re-hidrata precio/inventario vía
// query.graph antes de responder, así que el shape de MedusaProduct es
// exactamente el mismo que ya mapea mapProduct().
async function fetchWithFacets(
filters?: ProductFilters,
limit = 24,
): Promise<FacetedSearchResult> {
const params = new URLSearchParams({
limit: String(limit),
region_id: regionId(),
});
if (filters?.query) params.set("q", filters.query);
if (filters?.category && filters.category !== "Todos") {
params.set("category", filters.category);
}
if (filters?.collection) params.set("collection", filters.collection);
if (filters?.brand && filters.brand !== "Todas") {
params.set("brand", filters.brand);
}
if (filters?.ageRange && filters.ageRange !== "Todas") {
params.set("ageRange", filters.ageRange);
}
if (typeof filters?.minPrice === "number") {
params.set("minPrice", String(filters.minPrice));
}
if (typeof filters?.maxPrice === "number") {
params.set("maxPrice", String(filters.maxPrice));
}
const response = await medusaFetch<MedusaSearchResponse>(
"/store/search",
params,
);
return {
products: response.products.map(mapProduct),
facetDistribution: response.facetDistribution ?? {},
count: response.count ?? response.products.length,
};
}
export const medusaCatalogProvider: CatalogProvider = {
async getProducts(filters) {
return fetchProducts(filters);
@@ -408,6 +468,11 @@ export const medusaCatalogProvider: CatalogProvider = {
},
async searchProducts(query, limit = 8) {
return fetchProducts({ query }, limit);
const { products } = await fetchWithFacets({ query }, limit);
return products;
},
async searchWithFacets(filters) {
return fetchWithFacets(filters, filters?.limit ?? 24);
},
};
+19
View File
@@ -3,6 +3,7 @@ export interface ProductVariant {
name: string;
value: string;
available: boolean;
inventoryQuantity: number | null;
}
export interface Product {
@@ -37,6 +38,10 @@ export interface ProductFilters {
ageRange?: string;
minPrice?: number;
maxPrice?: number;
// Solo lo usa searchWithFacets -- limit=0 pide facetDistribution/count
// sin rehidratar precios de productos que no se van a mostrar (ver
// sidebar de facetas en catalog-client.tsx).
limit?: number;
}
export interface Category {
@@ -46,9 +51,23 @@ export interface Category {
parentId: string | null;
}
// facet -> valor -> cuantos productos del resultado filtrado tienen ese
// valor (ej. { brand: { Marvel: 5, Disney: 10 } }).
export type FacetDistribution = Record<string, Record<string, number>>;
export interface FacetedSearchResult {
products: Product[];
facetDistribution: FacetDistribution;
count: number;
}
export interface CatalogProvider {
getProducts(filters?: ProductFilters): Promise<Product[]>;
getProductBySlug(slug: string): Promise<Product | null>;
searchProducts(query: string, limit?: number): Promise<Product[]>;
getCategories(): Promise<Category[]>;
// Opcional a propósito: mock.ts y shopify.ts no lo implementan y
// siguen compilando igual -- los llamadores caen de nuevo al
// filtrado/conteo 100% client-side que ya existe cuando no está.
searchWithFacets?(filters?: ProductFilters): Promise<FacetedSearchResult>;
}
+5
View File
@@ -13,3 +13,8 @@ export function formatMoney(value: number | null, currency = "COP"): string {
maximumFractionDigits: 0,
}).format(value);
}
// Umbral desde el cual mostramos "Quedan N" en vez de solo "Disponible".
// inventoryQuantity=null significa inventario no gestionado (backorder /
// manage_inventory=false en Medusa) -- ahi nunca mostramos el mensaje.
export const LOW_STOCK_THRESHOLD = 5;
@@ -1,21 +0,0 @@
apiVersion: apps/v1
kind: Deployment
metadata:
name: frontend-deploy
spec:
template:
spec:
containers:
- name: web
env:
- name: HEADLESS_PROVIDER
value: medusa
- name: MEDUSA_BACKEND_URL
value: http://medusa-svc:9000
- name: MEDUSA_PUBLISHABLE_KEY
valueFrom:
secretKeyRef:
name: commerce-storefront
key: MEDUSA_PUBLISHABLE_KEY