Skip to content

feat(webhooks): add suppression webhooks - #353

Open
Cisneiros wants to merge 1 commit into
mainfrom
feat/suppression-webhooks
Open

feat(webhooks): add suppression webhooks#353
Cisneiros wants to merge 1 commit into
mainfrom
feat/suppression-webhooks

Conversation

@Cisneiros

@Cisneiros Cisneiros commented Jul 28, 2026

Copy link
Copy Markdown
Member

Includes suppression.added and suppression.removed webhooks.


Summary by cubic

Adds suppression webhooks to the CLI: suppression.added and suppression.removed. The "all" shorthand now expands to 19 events, and help/listen outputs reflect the new category.

  • New Features

    • Added suppression.added and suppression.removed to ALL_WEBHOOK_EVENTS.
    • Updated webhooks create and webhooks help to list the Suppression category.
    • webhooks listen now shows the suppressed email in the event detail.
    • Tests updated to expect 19 events when using --events all.
  • Dependencies

    • Bumped resend to 6.18.1.
    • Updated workspace config to exclude resend from minimum release age.

Written for commit 5d59a60. Summary will update on new commits.

Review in cubic

@socket-security

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedresend@​6.18.198100100100100

View full report

@Cisneiros
Cisneiros marked this pull request as ready for review July 28, 2026 13:46

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

2 issues found across 9 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="src/commands/webhooks/create.ts">

<violation number="1" location="src/commands/webhooks/create.ts:37">
P3: The `--events` option description still says "all 17 events" but the PR expands the set to 19 events with the new suppression webhooks. Update it to "all 19 events".</violation>
</file>

<file name="pnpm-workspace.yaml">

<violation number="1" location="pnpm-workspace.yaml:5">
P2: Broadening `minimumReleaseAgeExclude` from `resend@6.18.0` to `resend` removes supply-chain protection for all future versions of `resend`. A scoped entry like `resend@6.18.1` would achieve the immediate need while keeping later versions subject to the release-age check.</violation>
</file>

Reply with feedback, questions, or to request a fix.

Fix all with cubic | Re-trigger cubic

@resend resend deleted a comment from cubic-dev-ai Bot Jul 28, 2026
@resend resend deleted a comment from cubic-dev-ai Bot Jul 28, 2026
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.

2 participants