reviewing-prs

Flujo de trabajo general para obtener, inspeccionar y revisar solicitudes de extracción de GitHub usando la CLI gh, redactar comentarios de revisión alineados con el usuario y obtener aprobación antes de...

npx skills add https://github.com/flutter/devtools --skill reviewing-prs

Reviewing Pull Requests

This skill outlines the workflow for inspecting GitHub Pull Requests using the gh CLI, drafting review feedback, and securing user approval before posting review comments.

Approval Safeguard (Strict Requirement)

[!IMPORTANT] NEVER post comments or reviews to GitHub without explicit prior user approval. Always present draft review comments to the user in natural language first. Only execute write commands (gh pr comment, gh pr review) after the user approves.

Workflow

1. Request Information via GitHub CLI

  • PR Details:
    gh pr view <pr-number> --repo <owner/repo> --json title,body,author,state,headRefName,baseRefName,comments,reviews,files
    
  • Code Diff:
    gh pr diff <pr-number> --repo <owner/repo>
    
  • Existing Inline Comments:
    gh api repos/<owner/repo>/pulls/<pr-number>/comments
    
  • CI / Status Checks:
    gh pr checks <pr-number> --repo <owner/repo>
    

2. Inspect Context & Prior Feedback

  • Read the PR description, linked issues, and full diff.
  • Verify whether existing bot or human comments have already been addressed in subsequent commits.

3. Draft Review Comments

  • Keep comments direct, concise, and focused on code quality and correctness.
  • Approvals: Keep comments concise (LGTM or A couple comments but lgtm.). Avoid fluffy praise or re-summarizing the PR.
  • Actionable Feedback: Reference specific files, line numbers, and rationale when leaving suggestions.

4. Present Draft & Post Only Upon User Approval

[!IMPORTANT] NEVER post comments or reviews to GitHub without explicit prior user approval.

  1. Present the drafted review comments to the user in your response window.
  2. Ask for confirmation: "Would you like me to submit this review to GitHub?"
  3. Once explicitly approved by the user, post the review:
    gh pr review <pr-number> --repo <owner/repo> --comment --body "<approved review text>"
    

Más skills de flutter

dart-modern-features
flutter
Para encontrar candidatos para la modernización:
flutter-fix-layout-issues
flutter
Corrige errores de diseño en Flutter (desbordamientos, restricciones ilimitadas) usando herramientas Dart y Flutter MCP. Úsalo al abordar "RenderFlex overflowed", "Vertical…
adding-release-notes
flutter
Añade descripciones de cambios visibles para el usuario a las notas de la versión de DevTools. Úsalo al documentar mejoras, correcciones o nuevas funciones en el archivo NEXT_RELEASE_NOTES.md.
reviewing-devtools-prs
flutter
Flujo de trabajo de revisión de PR específico del repositorio de DevTools que aplica las guías de estilo de DevTools y los patrones de revisión comunes. Úsalo al revisar solicitudes de extracción en el…
dart-use-primary-constructors
flutter
Ayuda a los usuarios a escribir constructores primarios sintáctica y semánticamente correctos en Dart, y a migrar/utilizar la nueva sintaxis de constructor, la sintaxis de punto y coma de cuerpo vacío,…
code-documentation
flutter
Guía para escribir documentación de código efectiva, que incluye docstrings, JSDoc, dartdoc y comentarios de implementación. Usa esta habilidad al escribir código nuevo, agregar…
api-review
flutter
Revisa el código especificado conforme a las pautas canónicas de diseño de API. Usa esta habilidad cuando el usuario solicite una revisión de API o quiera verificar el código según las pautas de diseño de API…
flutter-accessibility
flutter
Implementa los estándares de accesibilidad WCAG 2 y EN 301 549 y diseños adaptativos en aplicaciones Flutter. Aplica anotaciones semánticas, tamaños de destino táctil (mínimo 48x48 dp) y relaciones de contraste de texto (4.5:1 para texto pequeño, 3:1 para texto grande) en plataformas móviles, web y de escritorio. Proporciona lógica de decisión para la inicialización de semántica web, envoltura de widgets interactivos, cambio de diseño basado en tamaño de pantalla y manejo de entrada por teclado/ratón. Incluye gestión de enfoque de navegación mediante FocusTraversalGroup y...