P3: visual:reference does not reproduce committed reference-canvas.png on the pinned Chromium
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 半日
- 初心者へのやさしさ
- 62/100
- issue の種類
- バグ
- 明瞭さ
- おおむね明確
- 活発さ
- 活発
- 技術スタック
- javascript, nodejs, playwright
調査の方向性
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.
索引モデルが issue の本文から書いたものです。
説明
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
--forceis passed, and document that regenerating it means re-recording every platform.
🤖 Generated with Claude Code
- 主要言語
- TypeScript
- スター
- 0
- フォーク
- 0
- 平均マージ
- 12時間 24分
- マージ済み PR（30日）
- 46
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
hyperdrive-technology/holodeck のほかの issue
-
area:ci review-finding
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
hyperdrive-technology/holodeck#261 ·
メンテナーはふだん 1 日以内に返信
-
review-finding
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
hyperdrive-technology/holodeck#259 ·
メンテナーはふだん 1 日以内に返信
-
area:holodeck review-finding
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
hyperdrive-technology/holodeck#256 ·
メンテナーはふだん 1 日以内に返信
-
area:holodeck review-finding
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
hyperdrive-technology/holodeck#254 ·
メンテナーはふだん 1 日以内に返信
-
area:holodeck bug review-finding
難易度 2/5 1〜3時間 初心者へのやさしさ 64/100
hyperdrive-technology/holodeck#252 ·
メンテナーはふだん 1 日以内に返信
hyperdrive-technology/holodeck の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 73/100
hugomrdias/playwright-test#753 ·
メンテナーはふだん 1 日以内に返信
-
Review view: Daily/Weekly Review header buttons use the app-wide toolbar control style対応中かも @colherdesopa が今日担当しました。 オープン
難易度 1/5 1〜3時間 初心者へのやさしさ 90/100
dongdongbh/Mindwtr#1515 ·
メンテナーはふだん 1 日以内に返信
-
pending-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
kirodotdev/Kiro#12301 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 83/100