Skip to content

docs(fix-review): tighten weak language and add trigger phrase #751

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
ralphbean merged 3 commits into from
Aug 14, 2026
Merged

docs(fix-review): tighten weak language and add trigger phrase #751

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
199fb15
docs(fix-review): tighten weak language and add trigger phrase
ralphbean Aug 11, 2026
5bf4f21
docs(fix-review): indent skillsaw directive to preserve list
fullsend-ai-coder[bot] Aug 12, 2026
585c707
docs(fix-review): move trigger phrase to start of description
fullsend-ai-coder[bot] Aug 13, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 6 additions & 4 deletions skills/fix-review/SKILL.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
---
name: fix-review
description: >-
Use when implementing fixes for review comments left on an open PR.
Step-by-step procedure for addressing review feedback on an existing PR.
Reads review comments, plans targeted fixes, implements, verifies with
tests and linters, commits, and produces structured output for the
Expand Down Expand Up @@ -327,6 +328,7 @@ git diff

Read every line. Check for:
- Changes that don't trace to a review comment
<!-- skillsaw-disable-next-line content-placeholder-text -->
- Debug prints, commented-out code, TODO comments
- Secret material
- Protected-path files
Expand Down Expand Up @@ -424,10 +426,10 @@ Write a JSON file to `$FULLSEND_OUTPUT_DIR/agent-result.json`:
object. Any extra fields you invent will cause validation to fail. Only use
the fields shown in this section.

**`trigger_source` field:** This must be the **normalized trigger type** you
derived in step 1 — either `"bot"` or `"human"`. Do NOT use the raw
`TRIGGER_SOURCE` environment variable value (the GitHub username). The schema
enforces an enum of `["bot", "human"]`; any other value fails validation.
**`trigger_source` field:** Use the **normalized trigger type** you derived
in step 1 — `"bot"` or `"human"` — not the raw `TRIGGER_SOURCE` environment
variable value (the GitHub username). The schema enforces an enum of
`["bot", "human"]`; any other value fails validation.

**Action types:**

Expand Down
Loading