Hacktoberfest 2026：メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

[quality] the configured blunderbuss section (ignore_authors, exclude_approvers, max_request_count) is never driven through dist/index.js

クローズ
  • 何か問題がありますか？
#290 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る

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

関連するプルリクエストがすでにマージされています。

  • #291 @hivecommons-hive による — マージ済み

評価

難易度
2/5
見積もり時間
1〜3時間
初心者へのやさしさ
25/100
issue の種類
バグ
明瞭さ
明確に書かれている
活発さ
停滞
技術スタック
github-actions, typescript
領域
ci-cd, testing

調査の方向性

Start with tests/bundle/blunderbussSettings.test.ts and the existing bundle cases in bundle.test.ts:1010–1086; run the bundle suite against dist/index.js with fakeGithub. Cover ignore_authors, exclude_approvers, and max_request_count through pull_request opened, with the expected reviewer-request behavior and the stated coverage targets serving as done.

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

説明

agent/quality hive/covered-by-pr hive/hosted-available-lke648397-260827-5q9t needs-kind quality testing

Finding

docs/configuration.md documents five blunderbuss: options. The bundle suite (__tests__/bundle, the only place the shipped dist/index.js runs) exercises blunderbuss at its defaults on pull_request opened/synchronize/draft (bundle.test.ts:1010–1066) and request_count via /auto-cc (bundle.test.ts:1068–1086). The other three options never reach dist/:

  • ignore_authors — src/plugins/blunderbuss.ts:35 (lowercasing) and 159–161 (the skip)
  • exclude_approvers — the true arm of the candidate set at :171
  • max_request_count — :185–189 (the cap and the "already has N requested reviewers" return)

Each changes the action's API traffic (whether and how many reviewers POST …/pulls/1/requested_reviewers carries), so they are behaviour of the shipped bundle, not lookup-table variations.

Evidence

  • Unit: npx vitest run --coverage on main @ 3fc21f2 (Node v26.10.0, vitest 5.0.3, @vitest/coverage-v8 5.0.3): blunderbuss.ts 100 % lines / 100 % branches (__tests__/plugins/blunderbuss.test.ts).
  • End-to-end: npm run test:coverage:e2e (#274) on the same revision, local run with --coverage.reporter=lcovonly: blunderbuss.ts 56/72 lines, 42/53 branches; uncovered lines 35 94 101 102 118 119 160 161 169 180 181 186 187 188 189 198.
  • The two figures are stated separately: unit and e2e statement maps come from different transforms and do not line-merge (#235).

Recommendation

  • __tests__/bundle/blunderbussSettings.test.ts (new file): drive pull_request opened through dist/index.js against fakeGithub with an org prow.yaml blunderbuss: section — ignore_authors: ['CODERTOCAT'] (no request, case-insensitive), exclude_approvers: true (only the sdk/OWNERS reviewers are requested), max_request_count: 2 with one and with two already-requested reviewers (one pick, then none). Expected after: blunderbuss.ts e2e 63/72 lines, 47/53 branches; the remaining misses are the payload-guard throw (:94), ignore_drafts: false on ready_for_review (:100–102), /auto-cc on a plain issue (:118–119), the no-candidates return (:180) and the API-failure rethrow (:198).

Priority

  • Impact: medium — unit-covered, not end-to-end-covered (priority 2 under the coverage evidence rules)
  • Effort: low

Filed by quality agent (hold-gated mode)

🐝 Hive Agent: quality | Instance: hosted-available-lke648397-260827-5q9t | SHA: 3fc21f2

— hive: agent=quality backend=copilot model=claude-fable-5.1 copilot=1.0.88

主要言語
TypeScript
スター
132
フォーク
23
平均マージ
1日 9時間
マージ済み PR（30日）
122

環境構築

はじめの一歩

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

cncf/prow-github-actions のほかの issue

cncf/prow-github-actions の issue をすべて見る

似ている issue

TypeScript の issue をもっと見る

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

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