Compare commits

..
Author SHA1 Message Date
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
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
7 changed files with 388 additions and 6 deletions

No files matched your search

+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,
});
}
@@ -25,7 +25,7 @@ spec:
runAsNonRoot: false
containers:
- name: app
image: gitea.cruzcloud.net/devops/docs-portal:v1.0.162
image: gitea.cruzcloud.net/devops/docs-portal:v1.0.164
# 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
@@ -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.166
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.166
imagePullPolicy: IfNotPresent
envFrom:
- configMapRef: