Incomplete PR feedback API surface in github-code skill
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 72/100
- Tipo di issue
- Documentazione
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Tranquilla
- Stack tecnologico
- github, graphql, markdown, shell
- Ambito
- api, developer-experience, documentation
Direzione di ricerca
Inizia con api-surface.md dello skill github-code e confronta i suoi endpoint documentati per il feedback delle PR con le chiamate REST e GraphQL elencate nell'issue. Aggiungi la sezione Complete PR Feedback Retrieval, le indicazioni sulla paginazione, i limiti di gh pr view, la nota sulle modifiche suggerite e la copertura delle reazioni e delle ricerche dirette. Il lavoro è completo quando tutte le fonti di feedback elencate e i dettagli sullo stato dei thread sono documentati correttamente.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
The github-code skill's api-surface.md only documents 2 of ~8 endpoints needed for complete PR feedback retrieval. This causes the agent to miss inline review comments, review thread state, and conversation comments when inspecting PRs.
Gaps
- No
reviewThreadsGraphQL query — REST cannot exposeisResolved,isOutdated, orresolvedBy. This is GraphQL-only and completely missing from the surface. - No warning that
gh pr view --json reviewsomits inline review comments —--json commentsreturns conversation comments only,--json reviewsreturns review summaries only. Inline code review comments require/pulls/{number}/commentsor GraphQLreviewThreads/reviews.comments. - Missing
/issues/{number}/comments— conversation comments live at the issues endpoint, not/pulls/{number}/comments(which is inline review comments only). REST equivalent is undocumented. - Missing review-scoped comments endpoint —
GET /pulls/{number}/reviews/{review_id}/commentsfor walking from a review to its inline comments. - Missing direct lookup endpoints — single review comment, single review, single issue comment by ID.
- Missing pagination guidance — no
--paginatedocumentation; PRs with heavy review activity silently truncate. - Missing suggested changes note — suggested changes are embedded in review comment
bodyas markdown suggestion fences, not a separate endpoint. - Missing reactions endpoints —
/pulls/comments/{comment_id}/reactionsand/issues/comments/{comment_id}/reactions.
Recommended Fix
Add a "Complete PR Feedback Retrieval" section with:
REST minimum (3 paginated calls):
gh api --paginate repos/{owner}/{repo}/issues/{number}/comments # conversation comments
gh api --paginate repos/{owner}/{repo}/pulls/{number}/reviews # review summaries
gh api --paginate repos/{owner}/{repo}/pulls/{number}/comments # inline review comments
GraphQL for thread state (1 call):
pullRequest(number: N) {
comments(first: 100) { ... }
reviews(first: 100) { ... }
reviewThreads(first: 100) {
nodes { isResolved isOutdated path line comments(first: 100) { ... } }
}
}
Add explicit warning about gh pr view --json field limitations.
Full analysis: https://sentry.slack.com/docs/T024ZCV9U/F0B44A567QE
Action taken on behalf of David Cramer.
- Lingua principale
- TypeScript
- Stelle
- 362
- Fork
- 38
- Merge medio
- 6h 39m
- PR unite (30g)
- 175
Preparare l'ambiente
- Include un Dockerfile o un file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di getsentry/junior
-
documentation
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 64/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
getsentry/junior#335 · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno
-
task
Difficoltà 5/5 Più di una settimana Idoneità per principianti 28/100
getsentry/junior#2051 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di getsentry/junior
Issue simili
-
enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
Small-tailqwq/dsh-deep-whale#187 ·
I maintainer di solito rispondono entro 1 giorno
-
Tela Marca manda conferir o campo "Razão social", que em Portugal se chama "Denominação social"Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
melgarafael/DeskcommCRM#2503 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
0x80/isolate-package#218 ·
-
bug via-triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
pingdotgg/t3code#16859 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
I maintainer di solito rispondono entro 1 giorno