From 50a9334652793791bc23bd8ff40675798b7f956f Mon Sep 17 00:00:00 2001 From: EC2 Default User Date: Wed, 1 Apr 2026 21:57:32 +0000 Subject: [PATCH] fix(review): show LGTM when review finds no issues Add instruction to the review skill output section so the agent responds with a concise LGTM message instead of a generic reply when the review produces no findings. Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com> --- plugins/core/skills/review/SKILL.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/plugins/core/skills/review/SKILL.md b/plugins/core/skills/review/SKILL.md index c518245..cd9256c 100644 --- a/plugins/core/skills/review/SKILL.md +++ b/plugins/core/skills/review/SKILL.md @@ -248,4 +248,6 @@ Before approving a candidate: When invoked locally (TUI/CLI), analyze the changes and provide a structured summary of findings. List each finding with its priority, file, line, and description. +If the review produces **no findings**, respond with a short **LGTM** message (e.g., "LGTM — no issues found."). Do not pad it with caveats or disclaimers. + Do **not** post inline comments to the PR or submit a GitHub review unless the user explicitly asks for it.