Skip to content

fix(poteto-mode): use -F for @file expansion in babysit replies - #2

Merged
adjohn merged 1 commit into
mainfrom
worktree-fix-babysit-gh-api-flag
Sep 9, 2026
Merged

fix(poteto-mode): use -F for @file expansion in babysit replies#2
adjohn merged 1 commit into
mainfrom
worktree-fix-babysit-gh-api-flag

Conversation

@adjohn

@adjohn adjohn commented Sep 9, 2026

Copy link
Copy Markdown
Owner

Why

Babysit step 8 tells the agent to post review replies with -f body=@file. That flag does not expand @file. gh api reads a file for a field only under -F, so the instruction posts the literal filename as the comment body. This happened live on cloudflare/cloudflare-docs#33091 and the comment had to be patched afterward.

Scope

One line in skills/poteto-mode/playbooks/babysit.md. The step 8 posting instruction now names -F body=@file and states why -f is wrong.

Blast Radius

Every babysit run that replies to a review thread from a file. Without the fix, each such reply posts a filename instead of the message.

Verification

Reproduced both behaviors against the GitHub API: -f body=@file posted the literal filename on cloudflare-docs#33091, and the -F form posted the file contents when patching that same comment.

🤖 Generated with Claude Code

https://claude.ai/code/session_01KybsFC28cLtu6pCdwdwBBw

gh api expands @file only under -F. The -f form posts the literal
filename as the comment body, which happened live on a PR reply.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01KybsFC28cLtu6pCdwdwBBw
@adjohn
adjohn merged commit 08c1ea8 into main Sep 9, 2026
adjohn added a commit that referenced this pull request Sep 9, 2026
Upstream's babysit step 8 now posts review replies with gh api --input
<payload.json>, which supersedes the -F @file fix from PR #2.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01X8ENwh7GfaGvuwDRykTGxx
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant