Résoudre les revues bloquantes d'une PR

VérifiéPrudence

Corrige uniquement les constats de revue bloquants d'une pull request existante, avec tests de régression ciblés, sans élargir le périmètre.

Spar Skills Guide Bot
DeveloppementIntermédiaire
0018/10/2026
Claude Code
#code-review#pull-requests#git-workflow#regression-testing#minimal-diff

Recommandé pour

Notre avis

Résout les constats de revue bloquants d'une pull request existante en appliquant des correctifs minimaux, sans élargir le périmètre de la PR.

Points forts

  • Vérifie chaque constat bloquant de façon indépendante avant de modifier le code, ce qui permet d'écarter les faux positifs.
  • Impose un changement minimal et refuse explicitement les suggestions non bloquantes hors périmètre.
  • Ajoute ou met à jour des tests de régression ciblés lorsque le correctif change le comportement.
  • Se termine par un rapport classant chaque constat d'origine en resolved, not reproduced ou still unresolved.

Limites

  • Suppose que AGENTS.md, la PR, l'issue et les commentaires de revue sont accessibles via les outils du dépôt ; aucun mode de repli n'est décrit.
  • S'arrête au commit et au push : la fusion reste une étape manuelle séparée.
  • Ne donne pas de critère pour arbitrer un désaccord persistant avec un relecteur au-delà du statut not reproduced.
Quand l'utiliser

À utiliser lorsqu'une PR déjà ouverte est bloquée par des commentaires de revue non résolus et que vous voulez une correction ciblée plutôt qu'une reprise globale.

Quand l'éviter

À éviter pour repenser l'architecture, implémenter une nouvelle fonctionnalité ou traiter des suggestions de revue non bloquantes.

Analyse de sécurité

Prudence
Score qualité80/100

The skill uses powerful actions such as git commit/push and running project-defined tests. These are legitimate for addressing review feedback, but they modify remote state and execute repository code, so caution is warranted.

Points d'attention
  • •Instructs committing and pushing code changes without explicit human approval, which could propagate incorrect or harmful edits to the shared repository.
  • •Instructs running tests and deterministic checks from AGENTS.md, which may execute arbitrary project scripts.

Exemples

Fix blocking findings on a numbered PR
/address-review 482
Fix blockers only, skip suggestions
Address the unresolved blocking review comments on PR #312, make the minimal fixes with regression tests, and don't take on the non-blocking suggestions.
Verify before fixing
Go through the unresolved review comments on the current PR, tell me which ones don't reproduce, and only fix the ones that are valid blocking issues.

name: address-review description: Resolve blocking review findings on an existing PR without broadening its scope. argument-hint: "[pr-number]" disable-model-invocation: true

Address unresolved review findings for PR $0.

  1. Read AGENTS.md, the PR, its Issue, current diff, and review comments.
  2. Identify unresolved blocking findings and verify each one independently.
  3. Make the minimum change required to resolve valid blocking findings.
  4. Do not implement unrelated non-blocking suggestions.
  5. Add or update focused regression tests where the fix changes behavior.
  6. Run focused tests and deterministic checks required by AGENTS.md.
  7. Re-review the changed area and its direct regression boundary.
  8. Commit and push the fix. Do not merge.

Report each original blocking finding as resolved, not reproduced, or still unresolved.

Skills similaires