Skip to content

feat: auto-setup GitHub webhook for PR Review workflows#391

Merged
alari76 merged 2 commits intomainfrom
feat/auto-webhook-setup
Apr 12, 2026
Merged

feat: auto-setup GitHub webhook for PR Review workflows#391
alari76 merged 2 commits intomainfrom
feat/auto-webhook-setup

Conversation

@alari76
Copy link
Copy Markdown
Contributor

@alari76 alari76 commented Apr 12, 2026

Summary

  • When creating a PR Review workflow, the server now automatically configures the GitHub webhook on the repo (derives owner/repo from git remote, generates secret, creates webhook via GitHub API)
  • The Add Workflow modal shows setup results after creation — a success message if the webhook was configured, or step-by-step manual setup instructions if auto-setup failed
  • Frontend passes the webhook URL to the server; API returns webhookSetup result alongside the config

Test plan

  • Create a new PR Review workflow for a repo — verify webhook is auto-created on GitHub
  • Verify the modal shows a green success message with "Workflow created"
  • Test with a repo that has no GitHub remote — verify the modal shows a yellow warning with manual setup instructions
  • Verify existing non-PR-review workflows still create normally (immediate modal close)
  • Run npm test — all 1592 tests pass
  • Run npm run build — builds successfully

🤖 Generated with Claude Code

alari76 and others added 2 commits April 12, 2026 20:25
When a user creates a PR Review workflow, the server now automatically
attempts to configure the GitHub webhook on the repo. The modal shows
the setup result — success or manual setup instructions if it fails.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- PR Review modal now always shows result screen (fallback message if
  server returns no setup info), ensuring consistent UX
- Extract parseGitHubSlug as a testable pure function with 8 unit tests
  covering SSH, HTTPS, dots in names, non-GitHub remotes, etc.
- Fix regex to allow dots in repo names (e.g. my_repo.name)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@alari76 alari76 force-pushed the feat/auto-webhook-setup branch from 68e29c9 to 6a66774 Compare April 12, 2026 17:25
@alari76 alari76 merged commit 02b2e38 into main Apr 12, 2026
@alari76 alari76 deleted the feat/auto-webhook-setup branch April 12, 2026 17:25
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