Skip to content

Merge dev to main#1174

Merged
zbigniewsobiecki merged 1 commit intomainfrom
dev
Apr 23, 2026
Merged

Merge dev to main#1174
zbigniewsobiecki merged 1 commit intomainfrom
dev

Conversation

@zbigniewsobiecki
Copy link
Copy Markdown
Member

Summary

Test plan

  • CI green on dev

…1171)

* fix(github): add pull_request_review_comment to webhook event lists

* fix(github): import GITHUB_WEBHOOK_EVENTS in setup-webhooks to prevent drift

Remove the duplicated GITHUB_WEBHOOK_EVENTS array from tools/setup-webhooks.ts
and import it from src/api/routers/webhooks/github.ts instead, so both the
API webhook creator and the CLI setup tool always stay in sync with a single
source of truth.

Also update the guard test's error message to no longer recommend editing
tools/setup-webhooks.ts, since it now imports the shared array automatically.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Cascade Bot <bot@cascade.dev>
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
@zbigniewsobiecki zbigniewsobiecki merged commit 7bce8b6 into main Apr 23, 2026
15 checks passed
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 23, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

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