[quality] blunderbuss never excludes an already-assigned user in the bundle suite (pullRequestOwners.ts:96 has 0 e2e hits)
メンテナーはふだん 2 日以内に返信
関連するプルリクエストがすでにマージされています。
- #403 @hivecommons-hive による — マージ済み
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 20/100
- issue の種類
- 機能追加
- 明瞭さ
- 明確に書かれている
- 活発さ
- 停滞
- 技術スタック
- typescript
- 領域
- testing
調査の方向性
The mapping is at src/utils/pullRequestOwners.ts:96 and its consumer is the excluded set in src/plugins/blunderbuss.ts:165. Read the existing unit case in __tests__/plugins/blunderbuss.test.ts and the bundle harness in __tests__/bundle/blunderbussSettings.test.ts before adding an assignees case. Linked pull request #403 is already open against this issue, so check its status before starting. Done means the bundle test passes with one mixed-case assignee excluded, and the both-assigned case logs no reviewer candidates.
索引モデルが issue の本文から書いたものです。
説明
Finding
loadPullRequestOwners lowercases a pull request's assignees (src/utils/pullRequestOwners.ts:96) and blunderbuss adds them to its exclusion set so an assignee is never asked to review their own assignment (src/plugins/blunderbuss.ts:165). No bundle test ever gives the fake GitHub a pull request with a non-empty assignees array, so the mapping callback and the exclusion it feeds have 0 hits through dist/index.js.
Evidence (main @ 20d49e7, Node v26.10.0):
- Unit:
npx vitest run --coverage→ 100 % lines;__tests__/plugins/blunderbuss.test.ts:176("skips already requested reviewers and assignees") reaches it with an in-process MSW handler. - End-to-end:
npm run test:coverage:e2eon a scratch branch =origin/main+ the heads of the 12 open hold-gated bundle PRs (#298, #314, #333, #345, #347, #378, #381, #383, #385, #395, #397, #401; 57 files / 414 tests pass) → 98.05 % lines ofsrc/.pullRequestOwners.ts:96has 0 hits (coverage-final.json:fnMapentry at line 96,f= 0). Lines 55-56 (resetPullRequestOwnersCache, test-only export) are the only other residual of that file; the reported line 74 is a source-map remap artifact — the rename ternary at:73records both arms taken bypullRequestOwnersArms.test.ts. - Consumer:
grep -n assignees src/plugins→ onlyblunderbuss.ts:165(excludedset).
Separate reporting: the unit (vite) and e2e (tsc+ncc source-map) coverage maps come from different transforms and are not combined (see vitest.e2e-coverage.config.mjs).
Recommendation
- add an
assigneescase to__tests__/bundle/blunderbussSettings.test.ts(pull_request openedwithexclude_approvers: true, one reviewer assigned under a mixed-case login → only the other is requested; both assigned →blunderbuss: no reviewer candidates for #1and no request)
Priority
- Impact: medium (covered by unit tests but not end-to-end)
- Effort: low
Filed by quality agent (hold-gated mode)
🐝 Hive Agent: quality | Instance: hosted-available-lke648397-260827-5q9t | SHA: 20d49e7
— 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 が今日担当しました。 オープン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 が 1 日前に担当しました。 オープン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 が 1 日前に担当しました。 オープン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
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
scaleway/ultraviolet#7089 ·
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 67/100
tencent-connect/dsh-qqbot#81 ·
メンテナーはふだん 1 日以内に返信
-
[Bug]: Pen taps on sidebar rows start a drag and lose their click対応中かも @makhnatkin が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
clangd/vscode-clangd#1009 ·