Skip to main content

review-pr-thread-verify

Called by review-pr for each unresolved review thread it authored: judges from the thread, the author's replies, and the current code whether the original concern is addressed, stale, or still standing, and returns exactly one disposition with a drafted reply where one is needed. Not for direct use.

Datos de origen

Repositorio
doodledood/manifest-dev
Última actividad en el origen
3 de septiembre de 2026 a las 13:23
Idioma detectado de SKILL.md
inglés
Estrellas
73
Forks
10

Opciones de instalación

De forma predeterminada está seleccionado el prompt que primero revisa el origen. Puedes cambiar a un comando directo o descargar una copia local.

Revisa los archivos de origen

Lee SKILL.md y los archivos complementarios que muestra SkillsMP antes de decidir si quieres instalarlo.

Mostrando SKILL.md

SKILL.md
Instrucciones de origen · Vista previa de solo lectura
name
review-pr-thread-verify
description
Called by review-pr for each unresolved review thread it authored: judges from the thread, the author's replies, and the current code whether the original concern is addressed, stale, or still standing, and returns exactly one disposition with a drafted reply where one is needed. Not for direct use.
user-invocable
false
# review-pr-thread-verify Decide what has happened to one review finding since we posted it. The caller hands over everything; this skill judges and returns, and never posts, resolves, or edits anything itself. **Receives.** The original finding (its anchor, body, the commit or head SHA it was reviewed at, and the concrete concern it raised); every author reply on the thread since our last message, plus every prior reply we posted there; the current code at the anchor with nearby context, and any commits since the original review that touched the relevant range; the full PR history (all comments, review threads, commit messages, the current PR description, our prior review and comment bodies); bundle context for each linked PR (diff, description, top-level conversation); and the caller's voice profile — its rules for how a posted comment reads — to apply to any reply drafted here. **Judges.** Read the concrete concern first and hold it fixed: the question is whether *that* failure mode is gone, conceded, disproven, or still live — not whether the thread has been discussed. A code change counts only when it touches the relevant range and removes the failure mode; a reply counts only when a fair-minded human reviewer would concede it. Push back only on new signal — a new author reply or a code change — and stay on the specific point under contention; never repeat an argument already made on the thread. **Returns exactly one disposition:** - `addressed-by-fix` — A commit after our review modified the relevant code and removes the concrete failure mode. Straightforward code fixes need no reply unless there was active discussion to answer; draft one only then. - `addressed-by-valid-reply` — The author replied on this or another thread with an argument a fair-minded human reviewer would concede: correct factual context, a deliberate owner trade-off, a valid out-of-scope boundary, or code elsewhere already covering the concern. Draft a short concession or acknowledgment. - `false-positive-or-stale` — Our comment was wrong, stale, or is disproven by current PR history. Draft a brief correction that owns the miss. - `needs-our-pushback` — The author replied or changed code, but the concrete concern remains. Draft a short reply for the existing thread on that specific point. - `still-pending` — No relevant new author reply and no relevant code change since our last look. Draft nothing. **Return shape.** The disposition name, one short reason (the evidence: the commit, the reply, or the absence of either), and the drafted reply text when the disposition calls for one, written in the supplied voice profile.
Ver en GitHub