Skip to content

docs(#11): require Recommended on follow-up replies in SKILL.md - #12

Open
fullsend-ai-coder[bot] wants to merge 1 commit into
mainfrom
agent/11-skill-recommended-followup
Open

fullsend-ai-coder[bot] wants to merge 1 commit into
mainfrom
agent/11-skill-recommended-followup

Conversation

@fullsend-ai-coder

Copy link
Copy Markdown
Contributor

Summary

  • Aligns SKILL.md step 2 with grillme.md's "Rules for thread replies" by adding the **Recommended:** requirement for follow-up replies that are decision questions
  • PR fix(grillme): require a recommended answer on inline questions #9 added this instruction to grillme.md but left SKILL.md unchanged, creating a divergence between the skill spec and the agent definition
  • Single-line text change; no code or workflow modifications

Testing

  • Verified both SKILL.md and grillme.md now reference **Recommended:** for follow-up replies (grep -n Recommended on both files)
  • Confirmed step 2 reads naturally with the added clause and does not break the numbered-list flow
  • All existing tests pass (npm test — 6/6)

Closes #11

Post-script verification

  • Branch is not main/master (agent/11-skill-recommended-followup)
  • Secret scan passed (gitleaks — aa086868a84b8178ad86ab5d4ff9e7ec5910aede..HEAD)
  • PR body secret scan passed (gitleaks — no-git)

SKILL.md step 2 of the Turn model omitted the requirement to include
a **Recommended:** line when a follow-up reply is itself a decision
question. grillme.md already had this instruction under "Rules for
thread replies" (added in PR #9), creating a divergence between the
skill spec and the agent definition.

Update step 2 to match: when replying in-thread with a follow-up that
is a decision question, include **Recommended: <answer and why>** the
same way as new comments.

Closes #11
@fullsend-ai-review

fullsend-ai-review Bot commented Aug 21, 2026 •

Copy link
Copy Markdown

🤖 Finished Review · ✅ Success · Started 7:58 PM UTC · Completed 8:07 PM UTC

Commit: 2f64938 · View workflow run →

@fullsend-ai-review

Copy link
Copy Markdown

Looks good to me


Labels: PR is a documentation-only change to a skill definition file

@fullsend-ai-review fullsend-ai-review Bot added ready-for-merge All reviewers approved — ready to merge documentation Improvements or additions to documentation labels Aug 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation ready-for-merge All reviewers approved — ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update SKILL.md step 2 to require Recommended on follow-up replies

0 participants