You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
list_pull_requests reports merged pull requests as merged: false #3467
list_pull_requests reports merged: false for merged pull requests. This can make an agent classify merged changes as closed without merging when it summarizes a repository's contribution history.
The REST list response includes merged_at but omits the merged boolean. convertToMinimalPullRequest currently uses pr.GetMerged(), whose default for the omitted field is false. The incorrect value appears in both text and structured output, including when selecting the merged field.
Affected version
Reproduced from source at main commit eb47a99ddb866ca2b8a162920e6bda9521f33ebb, using Go 1.27.1. This report is based on the REST response and tool-handler regression rather than a Docker release.
Steps to reproduce the behavior
Use list_pull_requests with state: "all" or state: "closed" for a repository containing merged pull requests.
The REST list returns a merged entry in this shape:
Expected: an entry with a valid merged_at and no explicit merged flag reports merged: true.
Actual: the tool reports merged: false.
Open entries and closed entries with merged_at: null should continue to report false. If the API supplies an explicit merged value, it should remain authoritative.
Logs
The regression fails on current main for both the full response and selected fields, in text and structured output:
Test_ListPullRequests_MergedState/merged_list_entry/all_fields
expected: true
actual : false
Test_ListPullRequests_MergedState/merged_list_entry/selected_fields
expected: true
actual : false
Activity