Skip to content

fix(cursor-review): correct Matt's handle in slack-notify DM map#2

Merged
mattmillerai merged 1 commit into
mainfrom
fix/slack-notify-handle
Jun 2, 2026
Merged

fix(cursor-review): correct Matt's handle in slack-notify DM map#2
mattmillerai merged 1 commit into
mainfrom
fix/slack-notify-handle

Conversation

@mattmillerai

Copy link
Copy Markdown
Contributor

The centralized slack-notify.sh (sourced from comfy-cloud-mcp-server in #1) still maps the retired millermedia GitHub handle. That handle now belongs to a different account, so cursor-review DMs to Matt resolve the wrong email (or none). cloud's copy already uses mattmillerai; this matches it.

  • One-key change: GITHUB_EMAIL_MAP[millermedia]GITHUB_EMAIL_MAP[mattmillerai]. Rest of the map is byte-identical to cloud's.
  • Fixes mcp-server's cursor-review DM routing (currently misrouted via the reusable workflow), and is a prerequisite for migrating cloud — without it, cloud would regress from its already-correct handle.

🤖 Generated with Claude Code

The centralized slack-notify.sh (sourced from comfy-cloud-mcp-server)
still mapped the retired `millermedia` GitHub handle, which now belongs
to a different account — so cursor-review DMs to Matt resolve the wrong
email (or none). cloud's copy already uses `mattmillerai`; match it.

Single-key change; rest of GITHUB_EMAIL_MAP is identical to cloud's.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@mattmillerai mattmillerai merged commit c1771f2 into main Jun 2, 2026
2 checks passed
@mattmillerai mattmillerai deleted the fix/slack-notify-handle branch June 2, 2026 08:35
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