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

watch-pr: review threads are fetched without pagination (first 100 only)

Fermée Adaptée aux débutants
  • Quelque chose ne va pas ?
#471 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

@Yi-111-a y travaille déjà.

Depuis le 2/10/2026.

  • #484 par @Yi-111-a — ouverte

Évaluation

Difficulté
2/5
Temps estimé
Une demi-journée
Accessibilité débutants
84/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
Active
Stack technique
typescript
Domaine
api, tooling

Piste de recherche

Commencez dans pstack/skills/poteto-mode/scripts/watch-pr/github.ts en examinant la requête reviewThreads à la ligne 5, parseReviewThreads vers la ligne 360, reviewThreads() près de la ligne 571 et la boucle de pagination existante vers la ligne 616. Exécutez le watcher sur une pull request comportant plus de 100 review threads, dont un thread non résolu situé plus loin. C’est terminé lorsque les pages suivantes sont récupérées et que ce thread non résolu influence la décision de readiness au lieu d’être omis silencieusement.

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

Description

Observed at commit 2eb7ed4613cfc8f098dfe464a23680ea44d84c5e.

pstack/skills/poteto-mode/scripts/watch-pr/github.ts line 5 defines the review-thread query as:

reviewThreads(first: 100) {
  nodes { id isResolved comments(first: 10) { nodes { ... } } }
}

It has no pageInfo and no cursor. reviewThreads() at line 571 issues this single query and parseReviewThreads (line 360) reads only reviewThreads.nodes. By contrast, the check-rollup query in the same file (line 9) paginates with pageInfo { hasNextPage endCursor } and a loop (around line 616).

Impact

On a PR with more than 100 review threads, threads after the first 100 are never seen. policy.ts line 66 builds the PR snapshot from this list (line 66), so an unresolved thread beyond the first page is invisible to the readiness decision. Comments beyond the first 10 in a thread are likewise dropped, though that matters less because only thread-level resolution seems to be used.

Minimal reproduction

Run the watcher against a PR with 101+ review threads where only a thread past the first 100 is unresolved; the PR is reported as having no unresolved threads.

Suggested fix

Add pageInfo { hasNextPage endCursor } and an $after cursor to the reviewThreads query and loop until hasNextPage is false, as is already done for contexts. If a hard cap is wanted, report an explicit "truncated" state rather than silently using the first page.

Langage dominant
TypeScript
Étoiles
12.5k
Forks
1.2k
Merge moyen
11 h 5 min
PR mergées (30 j)
125

Préparer son environnement

Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.

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 cursor/plugins

Toutes les issues de cursor/plugins

Issues similaires

Plus d'issues TypeScript

Recevez les nouvelles issues par e-mail

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