checkPage ignores a screen's declared readiness signal
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 74/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- playwright, typescript
- 領域
- accessibility, testing
調査の方向性
src/surfaces/playwright-helper.tsから始め、設定済み画面に対するCLIの準備状態チェックとcheckPageを比較します。screenIdが画面にどのように解決されるか、またreachedWhenがどのように評価されるかを追います。準備状態セレクターがない場合に、それを示すカバレッジギャップが発生し、providerが実行されない状態になれば完了です。関連するPlaywright helperのテストを実行してください。
索引モデルが issue の本文から書いたものです。
説明
What happens
checkPage (the Playwright helper, src/surfaces/playwright-helper.ts) runs the providers right away on the page it is given. It uses its own PAGE_CHECK_CONFIG, which holds no surfaces, so a screen's declared reachedWhen selector is never checked.
Why it matters
The CLI path makes a screen a not-covered gap when its declared selector does not appear. checkPage does not. If a test calls checkPage with a configured screenId while the page shows only its empty shell, the result can be verified. That result is advisory and does not reach the merge gate, but it still reads as a pass for content that was never on the page.
Expected
When the screenId names a configured screen that declares reachedWhen, checkPage applies the same readiness rule as the CLI. If the signal does not hold, the result is a coverage gap that names the selector, and no providers run.
Status
Found in review by reading the code. This is old behavior, not added by a recent change. Related: #304 (other checkPage input checks).
- 主要言語
- TypeScript
- スター
- 0
- フォーク
- 0
- 平均マージ
- 2時間 17分
- マージ済み PR（30日）
- 10
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドなし
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
usabl-dev/usabl のほかの issue
-
area: engine bug
難易度 2/5 1〜3時間 初心者へのやさしさ 86/100
メンテナーはふだん 1 日以内に返信
-
Stop hook allows a stop on an unknown verdict, and the self-check prints IDLE for a null verdictオープンarea: assistants bug
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
メンテナーはふだん 1 日以内に返信
-
gate() does not check draft enums, so a misspelled evidence class never gates対応中かも @eparenti が 1 日前に担当しました。 オープンarea: engine bug security
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信
-
Overlay: browser context failures are counted per plugin, not per browser, and close is not sharedオープンarea: overlay bug
難易度 2/5 1〜3時間 初心者へのやさしさ 64/100
メンテナーはふだん 1 日以内に返信
-
area: engine enhancement performance
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
メンテナーはふだん 1 日以内に返信
usabl-dev/usabl の issue をすべて見る
似ている issue
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
outline/outline#14348 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
bug needs-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
メンテナーはふだん 1 日以内に返信
-
[Bug]: Agent import retains quotes in .env API keys followed by inline comments対応中かも @yxr-2025 が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
elie222/rakazo#2322 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
awslabs/visual-asset-management-system#804 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
dsh-market/dsh-market#1027 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信