P3: visual:reference does not reproduce committed reference-canvas.png on the pinned Chromium

Abierto
#86 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
3/5
Tiempo estimado
Medio día
Aptitud para principiantes
62/100
Tipo de issue
Error
Claridad
Bastante claro
Estado de actividad
Activo
Stack tecnológico
javascript, nodejs, playwright
Área
testing, tooling

Línea de trabajo

Start with packages/editor-2.5d-react/scripts/make-reference-canvas.mjs, which decodes reference.jpg through the browser and writes tests/snapshots/reference-canvas.png. Compare its output with the committed snapshot to confirm the byte difference the issue describes. Choose between decoding in Node with a pinned decoder or guarding the overwrite behind --force and documenting it. Done means rerunning the script on the pinned Playwright build no longer changes the committed file, or the guard blocks the overwrite and the docs say so.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

area:holodeck review-finding

Found while reviewing #84 (pre-existing: main's script does the same).

pnpm visual:reference on the pinned Playwright build (chromium-1217) writes a reference-canvas.png that differs from the committed one. 46,763 channel bytes differ, by up to 36/255. The committed file was decoded from reference.jpg by chromium-1193, and the JPEG decode changed between those builds. The gate compares against the committed PNG, so the ratchet itself is unaffected. But anyone who reruns the script silently changes the reference, and every recorded score with it.

Where: packages/editor-2.5d-react/scripts/make-reference-canvas.mjs and tests/snapshots/reference-canvas.png.

Fix (pick one):

  • Decode the JPEG in Node, with a pinned decoder such as jpeg-js, so the output doesn't depend on the browser.
  • Or make the script refuse to overwrite the committed file unless --force is passed, and document that regenerating it means re-recording every platform.

🤖 Generated with Claude Code

Lenguaje dominante
TypeScript
Estrellas
0
Forks
0
Merge medio
12 h 24 min
PR fusionados (30 d)
46

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de hyperdrive-technology/holodeck

Todos los issues de hyperdrive-technology/holodeck

Issues similares

Más issues de TypeScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.