Hacktoberfest 2026: as issues que os mantenedores marcaram para outubro, abertas e boas para iniciantes. Ver issues do Hacktoberfest

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

Fechada Para iniciantes
  • Algo errado?
#471 1 comentário 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 1 dia

@Yi-111-a já está trabalhando nisso.

Desde 2/10/2026.

  • #484 de @Yi-111-a — aberto

Avaliação

Dificuldade
2/5
Tempo estimado
Meio dia
Facilidade para iniciantes
84/100
Tipo de issue
Bug
Clareza
Claramente especificada
Status de atividade
Ativa
Stack de tecnologia
typescript
Domínio
api, tooling

Direção de pesquisa

Comece em pstack/skills/poteto-mode/scripts/watch-pr/github.ts, revisando a consulta reviewThreads na linha 5, parseReviewThreads por volta da linha 360, reviewThreads() próximo à linha 571 e o loop de paginação existente por volta da linha 616. Execute o watcher em um pull request com mais de 100 review threads, incluindo um thread posterior não resolvido. Está concluído quando as páginas posteriores são buscadas e esse thread não resolvido afeta a decisão de readiness, em vez de ser omitido silenciosamente.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

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.

Linguagem predominante
TypeScript
Estrelas
13.9k
Forks
1.4k
Merge médio
6h 55min
PRs com merge (30d)
158

Preparar o ambiente

Este projeto não oferece contêiner de desenvolvimento, Dockerfile nem guia de contribuição, então a configuração fica por sua conta: comece pelo README e veja nosso guia da primeira contribuição para os passos gerais.

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de cursor/plugins

Todas as issues de cursor/plugins

Issues semelhantes

Mais issues de TypeScript

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.