github / github/github-mcp-server

pull_request_read get_files always returns full patches

Aperta
#3,236 1 commento 0 reazioni 0 assegnatari Vedi su GitHub
bug llm-context-window performance
Lingua principale
Go
Stelle
33k
Fork
5k
Merge medio
2g 1h
PR unite (30g)
52

Descrizione

### Describe the bug

`pull_request_read` with `method: get_files` always includes the full `patch` for every file, and offers no way to ask for filenames only. A 21-file pull request returned roughly 370 KB when only the paths were needed.

In the source, `convertToMinimalPRFiles` populates `MinimalPRFile.Patch` unconditionally, and `get_files` has no field-selection parameter. Several other tools do have one, for example `listPullRequestsItemFieldEnum` and `listCommitsItemFieldEnum`, so the mechanism already exists elsewhere in the codebase.

### 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

1. Call `pull_request_read` with `method: get_files` for a pull request touching a reasonable number of files.
2. Observe that every entry carries its full `patch`, with no option to omit it.

### Expected vs actual behavior

Expected: a way to request file metadata without patches, for example a `fields` parameter matching the pattern used by the list tools.

Actual: patches are always included, so a caller that only needs the list of changed paths pays the full diff in context.

Possibly related to #2122, which is closed but describes the same class of oversized `pull_request_read` payloads.

### Logs

None captured beyond the returned payload size.

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Start at the pull_request_read entry point for method get_files and inspect convertToMinimalPRFiles, which currently populates MinimalPRFile.Patch unconditionally. Compare the existing listPullRequestsItemFieldEnum and listCommitsItemFieldEnum field-selection mechanisms, then verify that callers can request file metadata without patches and that the default behavior remains clear.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
go
Ambito
api
Tipo di issue
Funzionalità
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Attiva
Chiarezza
Abbastanza chiara
Idoneità per principianti
68/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.