Skip to content

fix(skills): drop $ARGUMENTS from detail-bugs and detail-create-rules#284

Merged
sachiniyer merged 1 commit into
mainfrom
siyer/skills-drop-arguments
May 8, 2026
Merged

fix(skills): drop $ARGUMENTS from detail-bugs and detail-create-rules#284
sachiniyer merged 1 commit into
mainfrom
siyer/skills-drop-arguments

Conversation

@sachiniyer
Copy link
Copy Markdown
Contributor

Summary

  • Both skills currently template $ARGUMENTS into commands like detail bugs list $ARGUMENTS. When the skill is invoked with prose ("triage today's bugs"), Claude Code forwards the full sentence as $ARGUMENTS, producing detail bugs list "Check for detail bugs..." — the CLI treats that as a literal owner/repo and every command in the skill fails.
  • The CLI already infers the repo from the git remote, and Claude can pass an explicit flag if a different repo is needed, so the templating doesn't carry its weight.
  • This drop-the-arg approach is the cleaner alternative to the shape-checking half-fix in usedetail/detail#11222.

Changes

  • Removed argument-hint: <owner/repo> from both skill frontmatters.
  • Removed every $ARGUMENTS interpolation from command examples.
  • Replaced the "Determining the Repository" section in each skill with a single sentence pointing at git-remote inference and explicit flags.

Test plan

  • grep -r '\$ARGUMENTS' .claude/skills/ returns nothing
  • grep -r 'argument-hint' .claude/skills/ returns nothing
  • Invoke detail-bugs skill with a prose request and confirm the detail bugs list command runs cleanly
  • Invoke detail-create-rules skill and confirm the detail rules create / detail rules propose commands run cleanly

🤖 Generated with Claude Code

…reate-rules

When invoked with prose, Claude Code interpolates the full sentence into
$ARGUMENTS and the CLI rejects it as a malformed owner/repo. The CLI
already infers the repo from the git remote; if a different repo is
needed, Claude can pass an explicit flag instead of relying on skill
templating.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@sachiniyer sachiniyer temporarily deployed to integration-tests May 8, 2026 01:16 — with GitHub Actions Inactive
@sachiniyer sachiniyer merged commit 639059c into main May 8, 2026
13 checks passed
@sachiniyer sachiniyer deleted the siyer/skills-drop-arguments branch May 8, 2026 05:34
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