P2: CI's HOLODECK_CHROMIUM (auto-updating runner Chrome) counts as a pinned browser for the fidelity gate
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 45/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Attiva
- Stack tecnologico
- typescript
Direzione di ricerca
Start with resolveChromium in packages/editor-2.5d-react/src/test-support/chromium.ts, which marks any existing HOLODECK_CHROMIUM as pinned, and with the fidelity gate's pinning assertion that relies on it. Pick one of the two fixes the issue offers (record the browser version per platform record, or pin a Chrome for Testing build in CI), and check with a maintainer first since that is a design choice. Done when a test shows that pointing HOLODECK_CHROMIUM at an auto-updating system Chrome no longer counts as pinned.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
From the GPT Sol 6.1 review of #84. This is not a regression: CI already used the runner's Chrome before #84.
resolveChromium in packages/editor-2.5d-react/src/test-support/chromium.ts marks any existing HOLODECK_CHROMIUM as pinned: true. On CI that variable points at the GitHub runner's preinstalled Google Chrome, which changes with each image update (154 and 155 so far). The fidelity gate's pinning assertion therefore passes on a browser that can move under an unchanged Linux record. Chrome 154 and 155 happened to render byte-identically; a later version might not.
Fix: record the browser version with each platform record and fail (or warn, and skip the ratchet) when the version differs. Alternatively, use a fixed Chrome for Testing build in CI once the install hang (#29) is worked around. Add a test showing that pointing HOLODECK_CHROMIUM at an auto-updating system Chrome does not by itself count as pinned.
🤖 Generated with Claude Code
- Lingua principale
- TypeScript
- Stelle
- 0
- Fork
- 0
- Merge medio
- 10h 39m
- PR unite (30g)
- 38
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di hyperdrive-technology/holodeck
-
area:holodeck review-finding
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
hyperdrive-technology/holodeck#201 ·
I maintainer di solito rispondono entro 1 giorno
-
area:holodeck review-finding
Difficoltà 2/5 1-3 ore Idoneità per principianti 67/100
hyperdrive-technology/holodeck#196 ·
I maintainer di solito rispondono entro 1 giorno
-
area:docs review-finding
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 70/100
hyperdrive-technology/holodeck#180 ·
I maintainer di solito rispondono entro 1 giorno
-
Flaky E3 test: native cut with no keydown (holodeck#123) in editing-ux-render.test.tsForse già presa @dezlitz l’ha presa 2 giorni fa. Apertaarea:holodeck review-finding
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
hyperdrive-technology/holodeck#141 ·
I maintainer di solito rispondono entro 1 giorno
-
assetUrlProblem: data:image check uses the un-normalised URL, so ' data:image/png...' is rejectedApertaarea:holodeck review-finding
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
hyperdrive-technology/holodeck#104 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di hyperdrive-technology/holodeck
Issue simili
-
area: app type: bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
ExpidetGames/code-reviewer#585 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 83/100
module-federation/core#5205 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 73/100
LanternOps/breeze#9392 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
engine-rs P3 review-deferral wasm
Difficoltà 2/5 1-3 ore Idoneità per principianti 77/100
maniator/verticopolis#1154 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno