Incomplete PR feedback API surface in github-code skill
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 72/100
- Tipo de issue
- Documentación
- Claridad
- Bien especificado
- Estado de actividad
- Tranquilo
- Stack tecnológico
- github, graphql, markdown, shell
- Área
- api, developer-experience, documentation
Línea de trabajo
Comienza con api-surface.md del skill github-code y compara sus endpoints documentados de feedback de PR con las llamadas REST y GraphQL enumeradas en el issue. Añade la sección Complete PR Feedback Retrieval, las indicaciones sobre paginación, las limitaciones de gh pr view, la nota sobre los cambios sugeridos y la cobertura de reacciones y consultas directas. El trabajo estará terminado cuando todas las fuentes de feedback enumeradas y los detalles del estado de los threads estén documentados correctamente.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- TypeScript
- Estrellas
- 367
- Forks
- 41
- Merge medio
- 6 h 56 min
- PR fusionados (30 d)
- 168
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de getsentry/junior
-
documentation
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
getsentry/junior#335 · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
Los mantenedores suelen responder en 1 día
Todos los issues de getsentry/junior
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
farbenmeer/tapi#531 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
naver/egjs-flicking#971 ·
-
Renderer treats a sub-pixel width difference as a resize, which cancels the `motion()` entranceAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
Los mantenedores suelen responder en 1 día
-
Tenant
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
MTES-MCT/Dossier-Facile-Frontend#2061 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
backnotprop/plannotator#1784 ·
Los mantenedores suelen responder en 1 día