Skip to content

Keep unmapped assignees in Slack priority digests - #692

Merged
redreceipt merged 1 commit into
mainfrom
fix/slack-assignee-fallback
Oct 23, 2026
Merged

redreceipt merged 1 commit into
mainfrom
fix/slack-assignee-fallback

Conversation

@redreceipt

@redreceipt redreceipt commented Oct 23, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Fall back to the escaped Linear name when an assignee has no configured Slack mention.
  • Preserve existing mentions and unassigned-bug classification.
  • Cover Medic, name-only identities, missing Slack IDs, and escaping.

Validation

  • Type/lint checks, all 96 tests, production build, and all 8 deterministic browser tests pass on Next.js 16.4.2.
  • npm audit --omit=dev: zero vulnerabilities after merged security patch Patch Next.js security vulnerabilities #693.

Proof

Real notify() POST to a local HTTP webhook receiver using the existing public-safe upstream fixtures and a Medic assignee. Receiver returned 200; the attached screenshot displays the captured webhook text, including medic. No real Slack post or production data change.

Rebased onto merged security patch #693. CI and Vercel Preview passed, including all 8 browser tests. One Ubuntu package-download timeout passed on retry; optional Luna suite was not scheduled. The webhook path was exercised again on the patched runtime.

Actual webhook text captured by the local HTTP receiver

Fresh webhook capture on Next.js 16.4.2 includes the medic assignee

@vercel

vercel Bot commented Oct 23, 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 23, 2026 1:15pm 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.3s
🟢 dashboard navigation and no-secrets rendering 1.6s
🟢 preset and custom date windows survive navigation 1.1s
🟢 review and DAG filters use URL state 1.9s
🟢 mobile navigation stays usable 3.1s
🟢 sign-in card has room around its action on phones and desktop 768ms
🟢 health is public, API and cron remain closed without authentication 454ms
🟢 legacy dashboard URLs still resolve 488ms
🟢 review filters stay in sync after Back and reload 802ms

e2e 0.21.2 · 12.0s · web · run artifacts

@redreceipt
redreceipt marked this pull request as ready for review October 23, 2026 13:28
@redreceipt
redreceipt merged commit c0458a3 into main Oct 23, 2026
5 of 6 checks passed
@redreceipt
redreceipt deleted the fix/slack-assignee-fallback branch October 23, 2026 13:28
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 23, 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-23T13:29:47.051711Z 0e4ede3 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.

This branch was successfully deployed

1 active deployment
Preview — 0e4ede3e Deployed Oct 23, 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