P2: CI's HOLODECK_CHROMIUM (auto-updating runner Chrome) counts as a pinned browser for the fidelity gate

オープン
#87 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

まだ誰も着手していません。

評価

難易度
3/5
見積もり時間
1〜2日
初心者へのやさしさ
45/100
issue の種類
バグ
明瞭さ
おおむね明確
活発さ
活発
技術スタック
typescript
領域
ci-cd, testing

調査の方向性

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.

索引モデルが issue の本文から書いたものです。

説明

area:ci area:holodeck priority:p2 review-finding

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

主要言語
TypeScript
スター
0
フォーク
0
平均マージ
12時間 24分
マージ済み PR(30日)
46

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

hyperdrive-technology/holodeck のほかの issue

hyperdrive-technology/holodeck の issue をすべて見る

似ている issue

TypeScript の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。