feat: batch 1014 — logos argentinos con control de calidad by ferminrp · Pull Request #37 · ferminrp/logos-ar

ferminrp · 2026-10-14T16:57:28Z

Resumen

Segunda pasada de calidad del batch PR #37: logos hosteados normalizados, checker visual más estricto, hojas de contacto regeneradas y captura mobile con marcas nuevas del batch.

Checker (`check:logos`)

Métricas de recorte: área mínima (~20% en marcas cuadradas), lado corto en wordmarks, lado largo ≥25%, opacidad y contraste.
Detección invisible en fondo claro: composición sobre blanco + evaluateInvisibleOnLightBackground (luminancia alta, dHash vs original).
Fixtures obligatorios: must-fail-accord-salud.png, must-fail-trapiche-white.png.
Normalización al hostear: normalize-tile-logo.ts (~68% del lado largo) vía scripts/refresh-hosted-logos.mts.

Cambios de catálogo (batch id ≥ 452)

99 entidades en el batch (se quitó un slot problemático).
Reemplazos por logos débiles: Openpay → Mobbex, Prüne → Kosiuko, La Martina → Awada, Premier → SOS Salud, Open Sports → New Balance, Accord → Prevención Salud, Natura → Natura Cosméticos, etc.
Trapiche: wordmark blanco sobre tile bordó opaco (visible en hojas light).
Fuera del batch: Stella Artois → Heineken (logo hosteado); FinanzasArgy normalizado.

Artefactos

artifacts/loguitos/contact-sheet-light-01..03.png, contact-sheet-dark-01..03.png
grid-mobile-batch-section.png (390px, anclado en PlusPagos / Mobbex)
pr37-final-brands.json

Verificación

pnpm test:logos — 12/12
LOGO_CHECK_SKIP_REDIRECT=1 pnpm check:logos — 398 entidades OK (CI corre redirect completo)

Dudas visuales

ENARGAS y RENAPER en hoja light 2: wordmarks/sellos aún pequeños o pálidos; revisar en UI si hace falta otro activo oficial.

Note

Medium Risk
User-facing logo URLs and the download API behavior change broadly; CI and check scripts depend on external fetches and sharp processing, so regressions could affect many entity pages at once.

^{Reviewed by Cursor Bugbot for commit c49b377. Bugbot is set up for automated code reviews on this repo. Configure here.}

Co-authored-by: Fermin Rodriguez Penelas <frodriguezpenelas@gmail.com>

vercel · 2026-10-14T16:57:34Z

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated
logos-ar	Ready	Preview	Oct 17, 2026 6:14am UTC

Co-authored-by: Fermin Rodriguez Penelas <frodriguezpenelas@gmail.com>

cursor

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Autofix Details

Bugbot Autofix prepared a fix for the issue found in the latest run.

✅ Fixed: Telco copy lists missing Personal
- Updated telecomunicaciones seoTopEntities and seoIntro to name Telecentro instead of Personal, matching the category entity list.

_{You can send follow-ups to the cloud agent here.}

Co-authored-by: Fermin Rodriguez Penelas <frodriguezpenelas@gmail.com>

- Checker de logos (lib/logo-quality) con script pnpm check:logos y tests - Workflow GitHub Actions logo-quality.yml en PRs que tocan datos de logos - Overrides automáticos (data/logo-overrides.json) y PNGs en public/logos/ - Limpieza de dominios/categorías: sanatorios, bancos salientes, telecom, prepagas - UI y API usan getEntityLogoUrl con fallback Google + overrides Co-authored-by: Fermin Rodriguez Penelas <frodriguezpenelas@gmail.com>

Co-authored-by: Fermin Rodriguez Penelas <frodriguezpenelas@gmail.com>

cursor

Cursor Bugbot has reviewed your changes using high effort and found 8 potential issues.

Autofix Details

Bugbot Autofix prepared fixes for all 8 issues found in the latest run.

✅ Fixed: Shared domains mix distinct brands
- Restored unique domains for AXION, Uilo Bank, Buenbit, Pampa Energía, and ANMAT so each entity resolves its own logo and download target.
✅ Fixed: Downloads fail for hosted logos
- The download route now resolves root-relative logo URLs against the request origin and sets response types from the fetched image format.
✅ Fixed: ANMAT uses generic government domain
- ANMAT’s catalog domain was changed to www.anmat.gob.ar so it no longer shares the Argentina.gob.ar override.
✅ Fixed: Changan page removed despite demand
- Re-added the Changan entity (id 358, changan.com.ar) to the automotrices category so /logo/changan is generated again.
✅ Fixed: Copied logo URLs are not usable
- Entity detail copy/Sheets actions use absolute catalog URLs and label the source via getEntityLogoSource instead of entity.logoUrl.
✅ Fixed: BYD override is not an image
- Updated the www.byd.com override to the site’s favicon.ico asset instead of an HTML landing page.
✅ Fixed: Override URLs contain encoded ampersands
- Decoded & to & in the bitkey.world and www.iveco.com override query strings.
✅ Fixed: Generic-icon check uses a dummy hash
- Replaced the placeholder digest with Google’s generic favicon SHA-256 and reject HTML/non-image logo responses in quality checks.

_{You can send follow-ups to the cloud agent here.}

Restore distinct entity domains, fix hosted logo download/copy URLs, correct override URLs, and tighten logo quality validation.

Co-authored-by: Fermin Rodriguez Penelas <frodriguezpenelas@gmail.com>

cursor

Stale Bugbot comment from a previous run.

Convert non-PNG logo sources to PNG via sharp when format=png is requested. Use embedMimeType with an image/* guard for SVG raster wrappers so application/octet-stream upstream headers do not break data URIs.

cursor

Stale Bugbot comment from a previous run.

Extract embedded images from ICO containers before PNG conversion since sharp cannot decode ICO directly, with passthrough fallback on failure. Move sharp from devDependencies to dependencies for the API route runtime.

cursor

Stale Bugbot comment from a previous run.

Co-authored-by: Fermin Rodriguez Penelas <frodriguezpenelas@gmail.com>

- Amplía el batch (id ≥ 452) a 100 marcas argentinas con checker OK - Buenbit consolidado en Nexo (aliases); Kevingston fuera; Toyota AR + hosted - Hostea logos para CONICET, Rapsodia, redirects (Grido, Noblex, TGS) - Scripts de contact sheet y captura mobile para artefactos Fermin Co-authored-by: Fermin Rodriguez Penelas <frodriguezpenelas@gmail.com>

cursor

Stale Bugbot comment from a previous run.

cursor · 2026-10-17T04:12:13Z

+      },
+      { id: 480, name: "Payway", domain: "www.payway.com.ar" },
+      { id: 481, name: "Pagos360", domain: "pagos360.com" },
+      { id: 578, name: "Cabify", domain: "cabify.com" },


Cabify listed under fintechs

Medium Severity

Cabify is added to fintechs, so it shows up beside wallets and payment apps. Cabify is a ride-hailing brand, which makes the fintech hub and related-logo list misleading.

^{Reviewed by Cursor Bugbot for commit d7591f7. Configure here.}

cursor · 2026-10-17T04:12:13Z

+      { id: 585, name: "Vital", domain: "vital.com.ar" },
+      { id: 586, name: "Cencosud", domain: "cencosud.com" },
+      { id: 596, name: "Cetrogar", domain: "www.cetrogar.com.ar" },
+      { id: 598, name: "Andreani", domain: "www.andreani.com" },


Andreani listed under retail

Low Severity

Andreani is added to retail with supermarket and electronics chains. Andreani is a courier/logistics company, so the retail hub and related logos will present it as a store brand.

^{Reviewed by Cursor Bugbot for commit d7591f7. Configure here.}

Co-authored-by: Fermin Rodriguez Penelas <frodriguezpenelas@gmail.com>

- Hostea logos oficiales/Wikimedia para ~30 marcas del batch con problemas - Checker: dHash (casi-duplicados en batch), vacíos/bajo contraste y heurística foto - Samsung pasa a Tecnologías; ANMAT usa anmat.gob.ar y logo propio - Contact sheets claro/oscuro; captura mobile en /categoria/fintechs Co-authored-by: Fermin Rodriguez Penelas <frodriguezpenelas@gmail.com>

Co-authored-by: Fermin Rodriguez Penelas <frodriguezpenelas@gmail.com>

cursor

Stale Bugbot comment from a previous run.

- Strengthen visual checks: trimmed area, min side, light-background invisibility (dHash + luminance), Accord/Trapiche fixtures - Add normalize-tile-logo (68% max dimension) and refresh-hosted-logos for batch PNGs - Swap weak batch brands (Mobbex, Kosiuko, Awada, Prevención/SOS/New Balance/Natura Cosméticos, etc.) - Trapiche burgundy tile; replace Stella with hosted Heineken; normalize FinanzasArgy - Regenerate contact sheets (99 batch brands) and mobile capture; dataset 398 entities Co-authored-by: Fermin Rodriguez Penelas <frodriguezpenelas@gmail.com>

cursor

Cursor Bugbot has reviewed your changes using default effort and found 2 potential issues.

There are 4 total unresolved issues (including 2 from previous reviews).

^{Reviewed by Cursor Bugbot for commit c49b377. Configure here.}

cursor · 2026-10-17T06:16:03Z

-      { id: 312, name: "Heineken", domain: "heineken.com.ar" },
-      { id: 313, name: "Stella Artois", domain: "stellartois.com" },
+      { id: 312, name: "Heineken", domain: "www.heineken.com" },
+      { id: 313, name: "Heineken", domain: "www.heineken.com" },


Duplicate Heineken replaces Stella Artois

High Severity

Replacing Stella Artois with a second Heineken on www.heineken.com duplicates an existing catalog entity instead of hosting the original Heineken logo. The alimentos category lists Heineken twice, /logo/heineken-313 is a twin page, and the prior Stella slug disappears without a redirect.

Additional Locations (1)

data/logos-dataset.json#L3095-L3113

^{Reviewed by Cursor Bugbot for commit c49b377. Configure here.}

cursor · 2026-10-17T06:16:03Z

+  const issues: Array<{ code: VisualIssueCode; message: string }> = []
+  if (original && hammingDistance(original.dHash, analysis.dHash) > 4) {
+    return issues
+  }


Light-background check skips white logos

Medium Severity

evaluateInvisibleOnLightBackground returns immediately when the flattened-on-white dHash differs from the original. A white-on-transparent wordmark becomes a blank white tile after flatten, so the hashes diverge and the invisibility rules never run. Production check:logos always passes the original analysis, so the new guard misses the Accord/Trapiche case it was added to catch.

Additional Locations (2)

lib/logo-quality/check.ts#L270-L274

scripts/check-logos.test.mts#L172-L188

^{Reviewed by Cursor Bugbot for commit c49b377. Configure here.}

feat: add ~100 Argentine brand logos (batch 1014)

d34754b

Co-authored-by: Fermin Rodriguez Penelas <frodriguezpenelas@gmail.com>

docs(data): map GSC top-85 to catalog; swap Personal for Telecentro

988f7c3

Co-authored-by: Fermin Rodriguez Penelas <frodriguezpenelas@gmail.com>

vercel Bot deployed to Preview October 14, 2026 17:01

View deployment

cursor Bot reviewed Oct 14, 2026

View reviewed changes

Comment thread lib/logos-data.ts Outdated

fix: replace Personal with Telecentro in telecom SEO copy

213bd41

vercel Bot deployed to Preview October 14, 2026 17:04

View deployment

cursoragent and others added 3 commits October 17, 2026 00:39

merge: sync main (SEO #38) into PR branch

920babb

Co-authored-by: Fermin Rodriguez Penelas <frodriguezpenelas@gmail.com>

chore: ignore generated logo-check-report.json

212e676

Co-authored-by: Fermin Rodriguez Penelas <frodriguezpenelas@gmail.com>

cursor Bot changed the title ~~feat: add 100 Argentine brand logos (GSC batch 1014)~~ feat: batch 1014 — logos argentinos con control de calidad Oct 17, 2026

vercel Bot deployed to Preview October 17, 2026 02:56

View deployment

cursor Bot reviewed Oct 17, 2026

View reviewed changes

fix: resolve logo domain collisions, downloads, and quality checks

a82f3be

Restore distinct entity domains, fix hosted logo download/copy URLs, correct override URLs, and tighten logo quality validation.

vercel Bot deployed to Preview October 17, 2026 03:02

View deployment

fix(logos): hosted avalian y fravega para evitar 403 en CI

b8188bc

Co-authored-by: Fermin Rodriguez Penelas <frodriguezpenelas@gmail.com>

vercel Bot deployed to Preview October 17, 2026 03:05

View deployment

cursor Bot reviewed Oct 17, 2026

View reviewed changes

Comment thread app/api/logo-download/route.ts

Comment thread app/api/logo-download/route.ts

fix: convert PNG downloads and use detected MIME in SVG embeds

da15239

Convert non-PNG logo sources to PNG via sharp when format=png is requested. Use embedMimeType with an image/* guard for SVG raster wrappers so application/octet-stream upstream headers do not break data URIs.

vercel Bot deployed to Preview October 17, 2026 03:11

View deployment

cursor Bot reviewed Oct 17, 2026

View reviewed changes

Comment thread app/api/logo-download/route.ts

Comment thread app/api/logo-download/route.ts

vercel Bot deployed to Preview October 17, 2026 03:15

View deployment

cursor Bot reviewed Oct 17, 2026

View reviewed changes

Comment thread app/api/logo-download/route.ts

fix(logos): dominios canónicos y logos hosted para CI estable

3474bf1

Co-authored-by: Fermin Rodriguez Penelas <frodriguezpenelas@gmail.com>

vercel Bot deployed to Preview October 17, 2026 03:40

View deployment

vercel Bot deployed to Preview October 17, 2026 04:10

View deployment

cursor Bot reviewed Oct 17, 2026

View reviewed changes

fix(logos): CI estable para Experta bot-wall y Bond Terminal hosted

472e761

Co-authored-by: Fermin Rodriguez Penelas <frodriguezpenelas@gmail.com>

vercel Bot deployed to Preview October 17, 2026 04:18

View deployment

cursoragent and others added 2 commits October 17, 2026 05:05

chore: remove unused import in logo check

cf21e2c

Co-authored-by: Fermin Rodriguez Penelas <frodriguezpenelas@gmail.com>

vercel Bot deployed to Preview October 17, 2026 05:06

View deployment

cursor Bot reviewed Oct 17, 2026

View reviewed changes

Comment thread lib/logo-quality/visual-analysis.ts

vercel Bot deployed to Preview October 17, 2026 06:14

View deployment

cursor Bot reviewed Oct 17, 2026

View reviewed changes

Conversation

ferminrp commented Oct 14, 2026 • edited by cursor Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Resumen

Checker (check:logos)

Cambios de catálogo (batch id ≥ 452)

Artefactos

Verificación

Dudas visuales

Uh oh!

vercel Bot commented Oct 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

cursor Bot left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

cursor Bot left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

cursor Bot left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

cursor Bot left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

cursor Bot left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

cursor Bot left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

cursor Bot Oct 17, 2026

Choose a reason for hiding this comment

Cabify listed under fintechs

Uh oh!

cursor Bot Oct 17, 2026

Choose a reason for hiding this comment

Andreani listed under retail

Uh oh!

cursor Bot left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

cursor Bot left a comment

Choose a reason for hiding this comment

Uh oh!

cursor Bot Oct 17, 2026

Choose a reason for hiding this comment

Duplicate Heineken replaces Stella Artois

Uh oh!

cursor Bot Oct 17, 2026

Choose a reason for hiding this comment

Light-background check skips white logos

Uh oh!

This branch was successfully deployed

Reviewers

Uh oh!

Assignees

ferminrp commented Oct 14, 2026 •

edited by cursor Bot

Loading

Checker (`check:logos`)

vercel Bot commented Oct 14, 2026 •

edited

Loading

cursor Bot left a comment •

edited

Loading

cursor Bot left a comment •

edited

Loading

cursor Bot left a comment •

edited

Loading

cursor Bot left a comment •

edited

Loading

cursor Bot left a comment •

edited

Loading

cursor Bot left a comment •

edited

Loading

cursor Bot left a comment •

edited

Loading