reviewing-prs

oleh flutter

Alur kerja umum untuk mengambil, memeriksa, meninjau GitHub Pull Requests menggunakan gh CLI, menyusun komentar tinjauan yang selaras dengan pengguna, dan mendapatkan persetujuan sebelum…

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>"
    

Lebih banyak skill dari flutter

dart-modern-features
flutter
Untuk menemukan kandidat untuk modernisasi:
flutter-fix-layout-issues
flutter
Memperbaiki kesalahan tata letak Flutter (overflow, batasan tak terbatas) menggunakan Dart dan alat Flutter MCP. Gunakan saat menangani "RenderFlex overflowed", "Vertical…
adding-release-notes
flutter
Menambahkan deskripsi perubahan yang terlihat oleh pengguna ke catatan rilis DevTools. Gunakan saat mendokumentasikan perbaikan, perbaikan bug, atau fitur baru di file NEXT_RELEASE_NOTES.md.
reviewing-devtools-prs
flutter
Alur kerja peninjauan PR khusus repositori DevTools yang menerapkan pedoman gaya DevTools dan pola peninjauan umum. Gunakan saat meninjau pull request di…
dart-use-primary-constructors
flutter
Bantu pengguna menulis konstruktor utama yang benar secara sintaksis dan semantik dalam Dart, serta migrasi/menggunakan sintaks konstruktor baru, sintaks titik koma badan kosong,…
code-documentation
flutter
Panduan untuk menulis dokumentasi kode yang efektif, termasuk docstrings, JSDoc, dartdoc, dan komentar implementasi. Gunakan keterampilan ini saat menulis kode baru, menambahkan…
api-review
flutter
Meninjau kode yang ditentukan terhadap pedoman API Design kanonik. Gunakan skill ini saat pengguna meminta tinjauan API atau untuk memeriksa kode terhadap desain API…
flutter-accessibility
flutter
Menerapkan standar aksesibilitas WCAG 2 dan EN 301 549 serta tata letak adaptif di aplikasi Flutter. Memberlakukan anotasi semantik, ukuran target ketuk (minimal 48x48 dp), dan rasio kontras teks (4,5:1 untuk teks kecil, 3:1 untuk teks besar) di platform seluler, web, dan desktop. Menyediakan logika keputusan untuk inisialisasi semantik web, pembungkusan widget interaktif, peralihan tata letak berdasarkan ukuran layar, serta penanganan input keyboard/mouse. Mencakup manajemen traversal fokus melalui FocusTraversalGroup dan...