[quality] handlePullReq's jobs-input error arms are never exercised end-to-end through dist/index.js
メンテナーはふだん 2 日以内に返信
関連するプルリクエストがすでにマージされています。
- #310 @hivecommons-hive による — マージ済み
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 25/100
- issue の種類
- 機能追加
- 明瞭さ
- 明確に書かれている
- 活発さ
- 停滞
- 技術スタック
- typescript
- 領域
- testing
調査の方向性
Start with __tests__/bundle and read src/pullReq/handlePullReq.ts and src/pullReq/onPrLgtm.ts to understand the existing end-to-end test setup and the uncovered branches. Run npm run test:coverage:e2e to establish the baseline. Done when bundle-level tests exercise the listed jobs-input error cases and the coverage increases as described.
索引モデルが issue の本文から書いたものです。
説明
Finding
src/pullReq/handlePullReq.ts runs the jobs input after the registered pull-request handlers. Three of its arms are reached by no __tests__/bundle suite, so the shipped bundle's behaviour on them is unmeasured:
- lines 62-66: the
lgtmjob'scatch(anonPrLgtmfailure is returned as a value) and thedefaultarm (could not execute <job>. May not be supported) - lines 75-80: the
TypeError: error handling pull request: …rethrow and thecore.setFailedthat ends the run src/pullReq/onPrLgtm.tsline 30:could not get labels from issuewhen the labels read fails
Evidence
- Unit:
npx vitest run --coverageonmain@ 3fc21f2 —handlePullReq.ts100 / 100 / 100 / 100,onPrLgtm.ts100 / 100 / 100 / 100 (__tests__/pullReqTest/handlePullReq.test.ts,onPrLgtm.test.ts). - End-to-end:
npm run test:coverage:e2eonmain@ 3fc21f2 (Node v26.10.0, vitest 5.0.3, local run 2026-10-04) —handlePullReq.ts80.76 % lines, uncovered46,62-66,75-80;onPrLgtm.ts83.33 % lines, uncovered19,30.
Covered by unit tests but not end-to-end → medium.
Line 46 (jobs empty and no registered handlers) and onPrLgtm.ts line 19 (a pull_request payload without pull_request.number) are not reachable from a real pull_request event and are out of scope.
Recommendation
-
__tests__/bundle/pullReqJobsArms.test.ts: apull_requestsynchronizerun with an unsupportedjobsentry fails naming the job after the handlers have run; one whoselgtmjob'sGET /issues/1answers 500 fails withcould not get labels from issueand deletes nothing;jobs: lgtm <unsupported>removes the label and still fails the run
Measured with that file: handlePullReq.ts 80.76 → 96.15 % e2e lines (only 46 left), onPrLgtm.ts 83.33 → 91.66 % (only 19 left).
Priority
- Impact: medium
- Effort: low
Filed by quality agent (hold-gated mode)
🐝 Hive Agent: quality | Instance: hosted-available-lke648397-260827-5q9t | SHA: 709cb6c
— hive: agent=quality backend=copilot model=claude-fable-5.1 copilot=1.0.88
- 主要言語
- TypeScript
- スター
- 132
- フォーク
- 24
- 平均マージ
- 1日 18時間
- マージ済み PR（30日）
- 211
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
cncf/prow-github-actions のほかの issue
-
[quality] Nothing verifies the unit test each v8-ignore hint in src/ says drives its arm; add a static guard over the 25 hints対応中かも @hivecommons-hive が 1 日前に担当しました。 オープンagent/quality needs-kind stage/040-covered-by-pr testing
難易度 3/5 1〜2日 初心者へのやさしさ 35/100
cncf/prow-github-actions#600 · コメント 1 件 ·
メンテナーはふだん 2 日以内に返信
-
agent/quality needs-kind stage/045-needs-decision testing
難易度 4/5 3〜5日 初心者へのやさしさ 30/100
cncf/prow-github-actions#594 · コメント 3 件 ·
メンテナーはふだん 2 日以内に返信
-
[quality] Pin the ten behavioural Stryker survivors in sweep's delayed-reviewer guards, meow's image validation, the tilde-fence opener and /approve cancel対応中かも @hivecommons-hive が 2 日前に担当しました。 オープンagent/quality needs-kind stage/040-covered-by-pr testing
難易度 3/5 1〜2日 初心者へのやさしさ 25/100
cncf/prow-github-actions#592 · コメント 2 件 ·
メンテナーはふだん 2 日以内に返信
-
[quality] Pin the three behavioural Stryker survivors in blunderbuss, requireMatchingLabel and approveReview対応中かも @hivecommons-hive が 2 日前に担当しました。 オープンagent/quality needs-kind stage/040-covered-by-pr testing
難易度 3/5 1〜2日 初心者へのやさしさ 25/100
cncf/prow-github-actions#590 · コメント 3 件 ·
メンテナーはふだん 2 日以内に返信
-
hive/advisory kind/cleanup
難易度 5/5 1週間以上 初心者へのやさしさ 15/100
cncf/prow-github-actions#170 · コメント 3 件 ·
メンテナーはふだん 2 日以内に返信
cncf/prow-github-actions の issue をすべて見る
似ている issue
-
bug ui-dashboard
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 91/100
caelestia-dots/caelestia#534 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
area: scripts bug
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
tdk-landscape/tdk-cli-core#2706 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
bobadilla-tech/biasmarket.com#402 ·
メンテナーはふだん 1 日以内に返信
-
bug good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
javvadivijayprasad/sel2pw#28 ·