Hacktoberfest 2026 : les issues que les mainteneurs ont marquées pour octobre, ouvertes et accessibles aux débutants. Parcourir les issues Hacktoberfest

pull_request_read get_files always returns full patches

Ouverte
#3,236 2 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

Difficulté
3/5
Temps estimé
1-2 jours
Accessibilité débutants
68/100
Type d'issue
Fonctionnalité
Clarté
Plutôt claire
Activité
Active
Stack technique
go
Domaine
api

Piste de recherche

Commencez au point d’entrée pull_request_read de la méthode get_files et examinez convertToMinimalPRFiles, qui renseigne actuellement MinimalPRFile.Patch inconditionnellement. Comparez les mécanismes existants de sélection des champs listPullRequestsItemFieldEnum et listCommitsItemFieldEnum, puis vérifiez que les appelants peuvent demander les métadonnées des fichiers sans les patches et que le comportement par défaut reste clair.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

bug llm-context-window performance
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.

Langage dominant
Go
Étoiles
33.1k
Forks
5k
Merge moyen
2 j 1 h
PR mergées (30 j)
25

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de github/github-mcp-server

Toutes les issues de github/github-mcp-server

Issues similaires

Plus d'issues Go

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.