pull_request_read drops merge_commit_sha

未关闭适合新手

#3,235 创建于 2026年9月7日 0 条评论 0 个 reaction 已指派 0 人在 GitHub 查看

还没有人认领这个 Issue。

评估

难度: 2/5
预计耗时: 1-3 小时
新手友好度: 84/100

Issue 类型: 缺陷
描述清晰度: 描述清楚
活跃度: 活跃
技术栈: github, go
领域: api, backend

调研方向

从 pkg/github/pullrequests.go 开始，阅读 GetPullRequest、convertToMinimalPullRequest 和 MinimalPullRequest。添加保留合并提交 SHA 的支持，然后验证对于已合并的 pull request，使用 method get 的 pull_request_read 会公开 merge_commit_sha。

由索引模型根据 Issue 内容生成。

描述

bug

Describe the bug

pull_request_read with method: get does not return the pull request's merge_commit_sha, so there is no way to get from a merged pull request to the commit the merge produced without a second call.

In the source, GetPullRequest fetches the full pull request and then returns convertToMinimalPullRequest(pr). MinimalPullRequest carries merged, merged_at, merged_by, head and base, but has no merge_commit_sha field, so the value is dropped during conversion. merge_commit does not appear anywhere in pkg/github/pullrequests.go.

Affected version

Unknown. Observed through a hosted GitHub MCP integration on 2026-09-06, where the server version is not exposed to me. The code references above are from the repository's default branch on that date and may not match the build that was serving the request.

Steps to reproduce the behavior

Call pull_request_read with method: get for a merged pull request.
Inspect the returned object.

Expected vs actual behavior

Expected: merge_commit_sha is present for a merged pull request, as the REST API returns it.

Actual: the field is absent. Callers fall back to listing commits and matching on merge time or on the pull request number in the commit message, both of which are unreliable on an active branch.

Logs

None captured beyond the returned object.

主要语言: Go
星标: 33.1k
派生: 5k
平均合并: 2 天 1 小时
30 天内合并 PR: 25

贡献指南

打开贡献指南

从这里开始

先读完整个 Issue，再读项目的贡献指南。
在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
Fork 仓库，在一个分支上完成修改。
提交 Pull Request，并在描述里引用这个 Issue 编号。

github/github-mcp-server 的其他 Issue

Add guidance on GitHub autolinked reference formatting for AI agents 未关闭

enhancement

难度 1/5 1 小时以内新手友好度 88/100

github/github-mcp-server#3042 · 2 条评论 · 2026年8月8日创建
Incorrect install documentation leads to error "error: unknown option '-e'" 未关闭

bug

难度 2/5 1-3 小时新手友好度 72/100

github/github-mcp-server#3032 · 1 个 reaction · 2026年8月6日创建
Document Kubernetes installation with the HelmForge Helm chart 未关闭

难度 2/5 1-3 小时新手友好度 74/100

github/github-mcp-server#2803 · 1 条评论 · 2026年7月1日创建
get_discussion and get_discussion_comments accept calls with missing required parameters instead of returning a validation error 未关闭

难度 2/5 1-3 小时新手友好度 76/100

github/github-mcp-server#2740 · 2026年6月21日创建
list_discussions and get_discussion should include upvoteCount 未关闭

enhancement

难度 2/5 1-3 小时新手友好度 76/100

github/github-mcp-server#2661 · 1 条评论 · 2026年6月10日创建

查看 github/github-mcp-server 的全部 Issue

相似的 Issue

[GEP-28] `gardenadm restore`: Add developer docs for the local setup 未关闭

area/dev-productivity area/disaster-recovery area/ipcei kind/enhancement

难度 2/5 1-3 小时新手友好度 70/100

gardener/gardener#15788 · 2026年9月23日创建
Codex skills: `git rev-parse --verify --` rejects every explicit ref 未关闭

难度 1/5 1 小时以内新手友好度 85/100

kenn-io/roborev#1229 · 2026年9月23日创建
download_media returns a relative path that breaks when the bridge and MCP server run from different working directories 未关闭

难度 2/5 1-3 小时新手友好度 75/100

FelixIsaac/whatsapp-mcp-extended#85 · 2026年9月23日创建
`docker logout docker.io` does not remove Docker Hub credentials 未关闭

kind/bug status/0-triage

难度 2/5 1-3 小时新手友好度 75/100

docker/cli#7326 · 2026年9月23日创建
Truncation of tekton task names causes conflict even though names are different 未关闭

🤔 refinement needed

难度 2/5 1-3 小时新手友好度 75/100

equinor/radix-operator#1979 · 2026年9月23日创建

更多 Go Issue