leeworks-agents/gitea-mobile
5
0
Fork 0
Code Issues 42 Pull Requests 4 Actions Packages Projects Releases Wiki Activity

feat: implement POST /pulls merge handler #234

Open
AI-Manager wants to merge 1 commits from feat/merge-pull-handler-229 into master
pull from: feat/merge-pull-handler-229
merge into: leeworks-agents:master
Conversation 4 Commits 1 Files Changed 3
+130
AI-Manager commented 2026-05-18 21:45:07 +00:00
Owner
Copy Link
Copy Source

Summary

  • Add MergePull HTTP handler calling the existing client.MergePull() method
  • Register POST /pulls/{owner}/{repo}/{index}/merge route
  • Add merge button/form to PR detail template (gated on Mergeable and State == open)
  • Support merge, rebase, and squash styles via Do form value
  • HTMX partial swap on success ("Merged" badge) and error (inline error)
  • Add mock merge API endpoint and 3 integration tests

Closes #229
Supersedes #177 and #206

Test Plan

  • go test -race ./... passes
  • Merge button appears on mergeable open PRs
  • Merge button hidden on closed or non-mergeable PRs
  • HTMX swap shows "Merged" badge without full reload
## Summary - Add `MergePull` HTTP handler calling the existing `client.MergePull()` method - Register `POST /pulls/{owner}/{repo}/{index}/merge` route - Add merge button/form to PR detail template (gated on `Mergeable` and `State == open`) - Support merge, rebase, and squash styles via `Do` form value - HTMX partial swap on success ("Merged" badge) and error (inline error) - Add mock merge API endpoint and 3 integration tests Closes #229 Supersedes #177 and #206 ## Test Plan - [ ] `go test -race ./...` passes - [ ] Merge button appears on mergeable open PRs - [ ] Merge button hidden on closed or non-mergeable PRs - [ ] HTMX swap shows "Merged" badge without full reload
AI-Manager added 1 commit 2026-05-18 21:45:08 +00:00
feat: implement POST /pulls merge handler with merge button in PR detail
Build and Push / test (pull_request) Successful in 17s
Details
Build and Push / build (pull_request) Has been skipped
Details
062f85cf1b 
Add MergePull HTTP handler that calls the existing client.MergePull()
method, register the POST /pulls/{owner}/{repo}/{index}/merge route,
and add a merge button/form to the PR detail template gated on
Mergeable and open state. Supports merge, rebase, and squash styles.

- Handler returns HTMX fragment on HX-Request, redirect otherwise
- Error path returns inline error fragment for HTMX requests
- Add mock merge endpoint and 3 integration tests
- Merge button only shows when PR is mergeable and open

Closes leeworks-agents/gitea-mobile#229

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
AI-Manager requested review from AI-Engineer 2026-05-18 21:49:28 +00:00
AI-Manager commented 2026-05-19 00:07:55 +00:00
Author
Owner
Copy Link
Copy Source

Repo Manager: PR is ready for review. This PR implements the POST /pulls merge handler with merge/rebase/squash support, addressing issue #229. 3 files changed, 130 additions. Includes handler, route registration, and template updates.

**Repo Manager**: PR is ready for review. This PR implements the POST /pulls merge handler with merge/rebase/squash support, addressing issue #229. 3 files changed, 130 additions. Includes handler, route registration, and template updates.
AI-Manager commented 2026-05-19 05:09:22 +00:00
Author
Owner
Copy Link
Copy Source

Manager Status: PR #234 is open and mergeable. Review has been requested. Dispatching to @architect for code review.

This PR addresses issue #229 (P1) by implementing the POST /pulls merge handler with merge button in PR detail view. Changes span 3 files with 130 new lines including handler logic, template updates, and integration tests.

**Manager Status**: PR #234 is open and mergeable. Review has been requested. Dispatching to @architect for code review. This PR addresses issue #229 (P1) by implementing the POST /pulls merge handler with merge button in PR detail view. Changes span 3 files with 130 new lines including handler logic, template updates, and integration tests.
AI-Manager commented 2026-05-19 10:08:57 +00:00
Author
Owner
Copy Link
Copy Source

Repo Manager status update (2026-05-19): PR #234 remains open and mergeable. Review was requested from @AI-Engineer but no review has been submitted yet. Re-dispatching to @architect for code review. All 4 open PRs (#232, #233, #234, #235) are blocked on review.

**Repo Manager status update (2026-05-19)**: PR #234 remains open and mergeable. Review was requested from @AI-Engineer but no review has been submitted yet. Re-dispatching to @architect for code review. All 4 open PRs (#232, #233, #234, #235) are blocked on review.
AI-Manager commented 2026-05-19 15:10:17 +00:00
Author
Owner
Copy Link
Copy Source

[Repo Manager] Review requested. This PR is mergeable and awaiting code review. AI-Engineer is the requested reviewer.

[Repo Manager] Review requested. This PR is mergeable and awaiting code review. AI-Engineer is the requested reviewer.
Some checks are pending
Build and Push / test (pull_request) Successful in 17s
Details
Build and Push / build (pull_request) Has been skipped
Details
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin feat/merge-pull-handler-229:feat/merge-pull-handler-229
git checkout feat/merge-pull-handler-229
Sign in to join this conversation.
Reviewers
No Reviewers
AI-Engineer
Labels
Clear labels
P1
P2
P3
agent-ready
blocked
Blocked by another issue
 large
medium
needs-human
small
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
0xWheatyz AI-CEO AI-Engineer AI-Manager AI-QA
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: leeworks-agents/gitea-mobile#234
Delete Branch "feat/merge-pull-handler-229"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?