Skip to content

refactor: replace PR assignee workflow with Slack channel notification #2328

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
tomassrnka wants to merge 8 commits into
base: main
Choose a base branch
from
Draft

refactor: replace PR assignee workflow with Slack channel notification #2328

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
83b12e0
refactor: replace PR assignee workflow with Slack channel notification
tomassrnka Apr 8, 2026
71b52d9
fix: commit package-lock.json and use npm ci
tomassrnka Apr 8, 2026
46f4eb0
fix: escape Slack mrkdwn in PR title to prevent injection
tomassrnka Apr 8, 2026
6ac8860
Merge branch 'main' into pr-slack-notify
tomassrnka Apr 8, 2026
d6de69c
Merge branch 'main' into pr-slack-notify
tomassrnka Apr 9, 2026
95d87f5
Merge branch 'main' into pr-slack-notify
tomassrnka Apr 9, 2026
2cdb2cb
refactor: use shared reusable workflow from e2b-dev/.github
tomassrnka Apr 13, 2026
0383df4
feat: add workflow_dispatch for manual testing
tomassrnka Apr 13, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 0 additions & 11 deletions .github/actions/auto-request-same-site/package.json
View file
Open in desktop

This file was deleted.

289 changes: 0 additions & 289 deletions .github/actions/auto-request-same-site/script.js
View file
Open in desktop

This file was deleted.

55 changes: 0 additions & 55 deletions .github/workflows/auto-request-same-site.yml
View file
Open in desktop

This file was deleted.

41 changes: 41 additions & 0 deletions .github/workflows/pr-slack-notify.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,41 @@
name: Post PR to Slack

on:
pull_request_target:
types:
- opened
- ready_for_review
Copy link
Copy Markdown

@cursor cursor bot Apr 13, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Draft PRs trigger premature Slack notifications

Low Severity

The opened event fires for all PRs including drafts. A draft PR will post to #code-review-requests tagging @pr-infra immediately on creation, and then again when marked ready for review via ready_for_review. This results in either a noisy premature notification for drafts, or a duplicate notification for PRs that transition from draft to ready.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 2cdb2cb. Configure here.

workflow_dispatch:
inputs:
pr_number:
description: 'PR number'
required: true
type: string
pr_title:
description: 'PR title'
required: true
type: string
pr_url:
description: 'PR URL'
required: true
type: string
pr_author:
description: 'PR author'
required: true
type: string

permissions:
contents: read

jobs:
notify:
uses: e2b-dev/.github/.github/workflows/pr-slack-notify.yml@main
secrets:
SLACK_BOT_TOKEN: ${{ secrets.SLACK_BOT_TOKEN }}
with:
channel_id: ${{ vars.SLACK_CHANNEL_ID }}
review_group: ${{ vars.SLACK_REVIEW_GROUP }}
pr_number: ${{ inputs.pr_number || '' }}
pr_title: ${{ inputs.pr_title || '' }}
pr_url: ${{ inputs.pr_url || '' }}
pr_author: ${{ inputs.pr_author || '' }}
Loading