Skip to content

fix(analytics): align PR basis, add prsOpened, and remove synthetic fallbacks - #1593

Merged
castrojo merged 1 commit into
mainfrom
plan/org-contributor-analytics
Oct 24, 2026
Merged

castrojo merged 1 commit into
mainfrom
plan/org-contributor-analytics

Conversation

@castrojo

Copy link
Copy Markdown
Contributor

Summary

  • Derives human and automation PR counts using is:merged and isBot across all 4 months.
  • Re-derives July–October with uniform org-wide query basis (eliminating artificial scope cliffs).
  • Tracks prsOpened and prsMerged explicitly in stream payload.
  • Computes PR Merge Rate as prsMerged / prsOpened without synthetic 100% fallback.
  • Replaces unbacked Median Response Time card with Human Merged PRs card.

Verification

  • node --test scripts/fetch-contributor-metrics.test.js (9/9 pass)
  • node --test scripts/contributor-charts.test.js scripts/panel-unavailability.test.js (8/8 pass)
  • just check (typecheck + lint + 1,321 tests pass)
  • npm run build:ci clean

…allbacks

- derive human and automation PR counts using is:merged and isBot across all 4 months
- backfill July–October with uniform org-wide query basis
- track prsOpened and prsMerged explicitly in stream payload
- compute PR Merge Rate as prsMerged / prsOpened without synthetic 100% fallback
- replace unbacked Median Response Time card with Human Merged PRs card

Assisted-by: Gemini 3.8 Flash via GitHub Copilot
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@castrojo
castrojo added this pull request to the merge queue Oct 24, 2026
Merged via the queue into main with commit 304c18e Oct 24, 2026
3 checks passed
@castrojo
castrojo deleted the plan/org-contributor-analytics branch October 24, 2026 19:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant