Skip to content

La miga de pan salia dos veces en trece paginas - #1009

Merged
github-actions[bot] merged 1 commit into
developfrom
fix/la-miga-de-pan-sale-dos-veces
Oct 12, 2026
Merged

github-actions[bot] merged 1 commit into
developfrom
fix/la-miga-de-pan-sale-dos-veces

Conversation

@frapercan

Copy link
Copy Markdown
Owner

Lo que pasaba

app/[locale]/layout.tsx:230 pinta <Breadcrumbs /> para todas las rutas, sin
condicion, justo encima de <main>{children}</main>. Trece paginas la pintaban
ademas por su cuenta, asi que salian dos.

Por que no se veia en el HTML

Un curl de la pagina de proteina devuelve una sola aria-label="Breadcrumb".
Esas paginas son componentes de cliente ("use client"), asi que el SSR entrega el
estado de carga, que no incluye la miga de la pagina, y la segunda aparece al
hidratar. Contar sobre el HTML servido es la medida equivocada aqui.

El cambio

22 <Breadcrumbs /> fuera y 13 imports que se quedaban sin uso, en:
admin/api-keys, admin/experiment-runs, admin/users, auth/pending-approval,
instrument/datasets, instrument/datasets/[id],
instrument/functional-annotation/[id], instrument/jobs/[id],
instrument/proteins/[accession], login, logout, profile, signup.

Se queda la del layout. El componente ya devuelve null cuando la ruta tiene menos
de dos segmentos (Breadcrumbs.tsx:86), asi que las rutas que no deben llevarla
siguen sin llevarla.

Las paginas con varias apariciones (profile 4, api-keys 3, datasets/[id] 3,
users 3) las tenian en ramas distintas del render, carga y error, asi que nunca
salian mas de dos a la vez; la correccion es la misma.

Comprobado

tsc --noEmit sobre el worktree: cero errores en los trece ficheros. El unico
error que queda, tests/useUrlParam.test.tsx: Cannot find name 'beforeEach', es
identico en develop sin estos cambios, asi que es previo y ajeno.

Solo borrados: 35 lineas, ninguna anadida.

El layout pinta <Breadcrumbs /> para todas las rutas, sin condicion, justo
encima de <main>{children}</main>. Trece paginas la pintaban ademas por su
cuenta, asi que salian dos.

No se veia en el HTML servido porque esas paginas son componentes de
cliente: el SSR entrega el estado de carga, sin la miga de la pagina, y la
segunda aparece al hidratar. Un recuento sobre el HTML da una sola y es la
medida equivocada.

22 <Breadcrumbs /> fuera y 13 imports que se quedaban sin uso. Se queda la
del layout, que es la que cubre todas las rutas; el componente ya devuelve
null cuando la ruta tiene menos de dos segmentos, asi que las paginas que
no deben llevarla siguen sin llevarla.

Las paginas con varias apariciones (profile, api-keys, datasets/[id],
users) las tenian en ramas distintas del render, carga y error, asi que
nunca salian mas de dos a la vez.
@github-actions
github-actions Bot enabled auto-merge (squash) October 12, 2026 20:26
@codecov

codecov Bot commented Oct 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.03%. Comparing base (d8bc801) to head (6ec116b).
⚠️ Report is 108 commits behind head on develop.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #1009      +/-   ##
===========================================
+ Coverage    87.85%   88.03%   +0.17%     
===========================================
  Files          306      330      +24     
  Lines        23414    25590    +2176     
===========================================
+ Hits         20570    22527    +1957     
- Misses        2844     3063     +219     

see 40 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions
github-actions Bot merged commit 68133f6 into develop Oct 12, 2026
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant