Repository navigation
Preview: una app que navega a su inicio ya no muestra la Sala adentro - #148
Merged
Merged
Conversation
El bug intermitente del chat de Multti dentro de la vista previa. La raíz "/" siempre se servía como la Sala, así que cuando la app del proyecto navegaba a su inicio (un link "Inicio", el logo, un router que redirige a "/"), el iframe pedía la raíz y recibía la Sala entera. Al recargar, el iframe volvía a /preview/<sala>/ y se veía bien, por eso parecía que pasaba a veces. Reproducido con Playwright: una app con un link a "/", clic dentro de la vista previa → el iframe mostraba Multti. Con el arreglo muestra la app. Ahora "/" y "/index.html" pedidos desde un iframe (Sec-Fetch-Dest: iframe) van al preview de la sala que dicen el Referer o la cookie. Abiertos como página siguen siendo la Sala: la Sala nunca se abre dentro de un iframe. demo:proxy-rutas suma 9 checks y entra al CI. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01HJ5oT2Xm3VZMvbPAcgKbz4
ubuntu-latest pasó a 26.04 y Playwright 1.56.1 no puede instalar Chromium
ahí ("Playwright does not support chromium on ubuntu26.04-x64"): el job se
caía en el paso de instalar el navegador, antes de la demo de ver_preview.
No es de este PR; también le pasaría a main. Se fija a 24.04 hasta subir
Playwright.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01HJ5oT2Xm3VZMvbPAcgKbz4
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Qué arregla
Es el bug intermitente de que el chat de Multti aparecía dentro de la vista previa.
La raíz
/siempre se servía como la Sala. Cuando la app del proyecto navegaba a su inicio, el iframe pedía/y recibía la Sala entera. Pasa con un link "Inicio", con el logo, o con un router que redirige a/. Al recargar, el iframe volvía a/preview/<sala>/y se veía bien. Por eso parecía que solo pasaba a veces.El cambio
parsePreviewUrl, enserver/src/engine/proxy.ts, ahora mira quién pide/o/index.html:Sec-Fetch-Dest: iframe), la petición va al preview de la sala. La sala sale del Referer o de la cookie que ya existe.Pruebas
/. Al hacer clic dentro de la vista previa:/sin cabecera sigue dando la Sala;/consec-fetch-dest: iframeda la app.demo:proxy-rutas: suma 9 checks (50/50) y ahora corre en el CI.npm run typechecky todas las demos del CI pasan.Generated by Claude Code