agent: @U0AJM7X8FBR I need individual buttons to merge each pull requ…#277
agent: @U0AJM7X8FBR I need individual buttons to merge each pull requ…#277sweetmantech merged 1 commit intomainfrom
Conversation
…est in the (#276) * agent: @U0AJM7X8FBR I need individual buttons to merge each pull request in the * agent: address feedback * agent: address feedback * agent: address feedback * agent: address feedback * agent: address feedback * agent: address feedback * agent: address feedback * fix: use catch-all action handler for merge_pr prefix matching Chat SDK's onAction uses exact match, not prefix match. Changed to a catch-all handler that filters with startsWith("merge_pr:") so dynamic action IDs like "merge_pr:#42" are matched. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Recoup Agent <agent@recoupable.com> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (4)
📒 Files selected for processing (4)
📝 WalkthroughWalkthroughThis PR refactors the PR merge workflow from a single global "merge all PRs" button to individual per-PR merge buttons. It introduces per-PR action ID parsing, GitHub API merge utilities, and updates the merge handler to process individual PR merge actions with granular state management. Changes
Sequence DiagramsequenceDiagram
participant User
participant Handler as onMergeAction Handler
participant Parser as parseMergeActionId
participant API as GitHub API
participant State as Thread State
User->>Handler: Click Merge PR Button<br/>(action: merge_pr:repo#123)
Handler->>Parser: Parse action ID
Parser-->>Handler: {repo, number}
Handler->>State: Find PR in prs array
State-->>Handler: PR details
Handler->>API: PUT /repos/{owner}/{repo}/pulls/{number}/merge<br/>(merge_method: squash)
API-->>Handler: Success/Failure response
Handler->>State: Remove merged PR from prs array
Handler->>State: Recalculate merged status
State-->>Handler: Updated state
Handler-->>User: Per-PR success message
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Poem
✨ Finishing Touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
…est in the (#276)
agent: @U0AJM7X8FBR I need individual buttons to merge each pull request in the
agent: address feedback
agent: address feedback
agent: address feedback
agent: address feedback
agent: address feedback
agent: address feedback
agent: address feedback
fix: use catch-all action handler for merge_pr prefix matching
Chat SDK's onAction uses exact match, not prefix match. Changed to a catch-all handler that filters with startsWith("merge_pr:") so dynamic action IDs like "merge_pr:#42" are matched.
Summary by CodeRabbit
Release Notes
New Features
Refactor