check:affected: a changed test file must run the test-file size ratchet
メンテナーはふだん 1 日以内に返信
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 72/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- typescript
調査の方向性
Read the selection rules in scripts/check-affected/model.ts and the existing ratchet test, scripts/tests/test-file-size-ratchet.test.ts. Find where the measured file patterns and roots are defined, then check how other text-read gates declare selection reasons. Add a model.test.ts case and run it: a changed measured test file should select the ratchet with its path in the reason, while a non-test change should not.
索引モデルが issue の本文から書いたものです。
説明
Purpose
pnpm check:affected --run passed on #3732 at 63c6a66, but CI Coverage failed scripts/__tests__/test-file-size-ratchet.test.ts: packages/platform-apple/src/lifecycle.test.ts grew from 970 to 1,006 lines, past the 1,000-line tripwire. The local gate selects tests through Vitest's module graph (vitest related). The ratchet doesn't import the files it measures; it walks the tree and reads line counts. So no change to a test file ever selects it, and the failure reaches CI one review round late.
Required behavior
- When a change adds or modifies a file the ratchet measures (
*.test.ts,*.test.tsxor*.test.mjsundersrc/,packages/,test/orscripts/, excludingnode_modules,dist,dist-typesand.tmp), the affected plan selects the ratchet test, with a reason that names the triggering file. - Express it as a selection rule in
scripts/check-affected/model.ts, the same way other text-read gates are declared there. Don't special-case it inrun.ts. - Keep a single definition of which files the ratchet measures. The selection rule and the ratchet test must read the same pattern and roots, so they can't drift. Put the shared definition where both can import it.
- A change that touches no test file doesn't select the ratchet.
Completion
- A
model.test.tscase: a changed test file selects the ratchet check and gives the triggering reason, and a non-test change doesn't. It fails before the change and passes after. - Planting a 1,001-line new test file in a scratch checkout makes
pnpm check:affected --runfail locally on the ratchet. pnpm check:affected --run --head <sha>passes on the final commit.
- 主要言語
- TypeScript
- スター
- 5k
- フォーク
- 334
- 平均マージ
- 12時間 2分
- マージ済み PR（30日）
- 706
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
callstack/agent-device のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 83/100
callstack/agent-device#3792 ·
メンテナーはふだん 1 日以内に返信
-
macOS helper audio-probe always reports -90 dBFS: AVAudioPCMBuffer frameLength set after the copyオープンbug ready-for-agent
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
callstack/agent-device#3771 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
callstack/agent-device#1869 ·
メンテナーはふだん 1 日以内に返信
-
refactor(host): explicit host for configuration reads and one entered runtime store対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープンready-for-agent refactor
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
callstack/agent-device#3794 ·
メンテナーはふだん 1 日以内に返信
-
Daemon startup: wait for a live daemon instead of a fixed 15 s deadline on slow or overloaded hosts対応中かも @thymikee が 1 日前に担当しました。 オープンneeds-triage
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
callstack/agent-device#3782 ·
メンテナーはふだん 1 日以内に返信
callstack/agent-device の issue をすべて見る
似ている issue
-
freshness freshness:observed-only
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
thoroc/context-radar#280 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
-
documentation story
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
メンテナーはふだん 1 日以内に返信
-
bug category:billing priority:p3 track:fast
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
LanternOps/breeze#9223 ·
メンテナーはふだん 1 日以内に返信