Skip to content

Add reaction-based PR merge flow#17

Merged
rossnelson merged 1 commit intomainfrom
claude/1774913464-766059
Mar 31, 2026
Merged

Add reaction-based PR merge flow#17
rossnelson merged 1 commit intomainfrom
claude/1774913464-766059

Conversation

@rossnelson
Copy link
Copy Markdown
Member

Summary

  • New reaction event handler: when a user adds :approved: to a root thread message, the bot merges the associated PR and adds :merged: reaction
  • Deploy command (/sb-deploy) now only creates the PR — workspace stays alive until merge
  • Cleanup (session, checkpoint, workspace) happens after successful merge via the reaction flow
  • Added TypeScript to the runtime Docker image

Test plan

  • Deploy creates PR without cleaning up workspace
  • Adding :approved: reaction to root thread message merges the PR
  • Bot adds :merged: reaction after successful merge
  • Thread resources are cleaned up after merge
  • Error cases: no PR, closed PR, already merged PR

Replace the deploy command's auto-cleanup with a reaction-driven workflow:
- New reaction handler listens for :approved: on root thread messages
- Merges the associated PR, adds :merged: reaction, then cleans up
- Deploy command now only creates the PR (workspace stays alive for merge)
- Add TypeScript to runtime Docker image
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