Skip to content

Restore small PR reminders after a week awaiting review - #696

Merged
redreceipt merged 1 commit into
mainfrom
restore-small-stale-pr-reminders
Oct 26, 2026
Merged

redreceipt merged 1 commit into
mainfrom
restore-small-stale-pr-reminders

Conversation

@redreceipt

@redreceipt redreceipt commented Oct 26, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Add Other PRs below Urgent/High review reminders: requests waiting at least seven days, fewer than 200 added lines, unlimited deletions.
  • Include lower/unset priority and unlinked PRs; retain existing readiness gates, reviewer grouping, and averages.
  • Cover boundaries, exclusions, ordering, and a public-safe fixture.

Validation

  • npm run check
  • npm test — 98 passing
  • npm run build — passes; existing dynamic config tracing warning
  • CI: validate (including 8 dashboard E2E tests), fixture exploration, and Vercel preview passed. The optional luna job was not scheduled.

Explorer findings are unrelated to this change: team sorting/person metrics and singular DAG wording use unchanged code and fixtures. The issue-title finding compares PR titles with linked Linear issue titles (app/reviews/page.tsx renders PR titles). The explorer's comment reporter also hit the unchanged workflow's read-only-token permission; artifacts and the job succeeded. These were not changed or silently treated as fixes.

Proof

Ran the real notify("stale", ...) path locally with the committed upstream fixture double and a loopback HTTP webhook receiver. Preview notifications cannot send from the hosted app, so this exercises delivery without posting to Slack.

The attached screenshot is the actual webhook receiver's response containing the received message: Urgent/High → Other PRs → stale issues. The Other PR has 199 additions, 55 deletions, and a 23-day review request. Delivery returned sent; retry returned already-claimed. Proof files are outside Git; the local receiver and browser were closed.

webhook-receipt

@vercel

vercel Bot commented Oct 26, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
bug-board Ready Ready Preview Oct 26, 2026 4:05pm UTC

Request Review

@github-actions

Copy link
Copy Markdown

🟢 e2e: 8 passed

All 8 tests in 1 file
Test Time
🟢 tests/dashboard.e2e.ts · 8 passed 10.9s
🟢 dashboard navigation and no-secrets rendering 1.6s
🟢 preset and custom date windows survive navigation 1.2s
🟢 review and DAG filters use URL state 2.0s
🟢 mobile navigation stays usable 3.3s
🟢 sign-in card has room around its action on phones and desktop 775ms
🟢 health is public, API and cron remain closed without authentication 473ms
🟢 legacy dashboard URLs still resolve 538ms
🟢 review filters stay in sync after Back and reload 971ms

e2e 0.21.2 · 12.6s · web · run artifacts

@redreceipt
redreceipt marked this pull request as ready for review October 26, 2026 16:19
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 26, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-26T16:20:40.309719Z da70864 Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@redreceipt
redreceipt merged commit 4fb8f40 into main Oct 26, 2026
5 checks passed
@redreceipt
redreceipt deleted the restore-small-stale-pr-reminders branch October 26, 2026 16:20

This branch was successfully deployed

1 active deployment
Preview — da708641 Deployed Oct 26, 2026 by vercel[bot]
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