Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

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

Abierto
  • ¿Algo no está bien?
#290 1 comentario 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

@hivecommons-hive ya está trabajando en esto.

Desde el 4/10/2026.

  • #291 de @hivecommons-hive — abierto

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
25/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Estancado
Stack tecnológico
github-actions, typescript
Área
ci-cd, testing

Línea de trabajo

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.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

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

Lenguaje dominante
TypeScript
Estrellas
132
Forks
23
Merge medio
1 d 2 h
PR fusionados (30 d)
98

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de cncf/prow-github-actions

Todos los issues de cncf/prow-github-actions

Issues similares

Más issues de TypeScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.