Skip to content

fix: add allowed-tools: Task to 14 agent-dispatch commands#10

Open
xiaolai wants to merge 1 commit intowebdevtodayjason:mainfrom
xiaolai:fix/nlpm-add-allowed-tools
Open

fix: add allowed-tools: Task to 14 agent-dispatch commands#10
xiaolai wants to merge 1 commit intowebdevtodayjason:mainfrom
xiaolai:fix/nlpm-add-allowed-tools

Conversation

@xiaolai
Copy link
Copy Markdown

@xiaolai xiaolai commented Apr 25, 2026

Automated audit: This PR was generated by NLPM, a natural language programming linter, running via claude-code-action. Please evaluate the diff on its merits.

What and why

14 of the command files in commands/ dispatch to a sub-agent (they have an agent: key in frontmatter) but omit allowed-tools. The other 6 commands in the same directory — debug.md, document.md, refactor.md, review.md, security-scan.md, and test.md — already correctly declare allowed-tools: Task.

When allowed-tools is absent, Claude Code has no declared guidance on which tools the command may invoke. Adding allowed-tools: Task explicitly permits the Task tool (used to spawn sub-agents), matching the pattern already established by the non-agent commands in this repo.

Fix

Added allowed-tools: Task to the frontmatter of all 14 affected agent-dispatch commands:
api-docs.md, api.md, content.md, deploy.md, devops.md, frontend.md, marketing.md, plan.md, product.md, requirements.md, shadcn.md, tdd.md, test-first.md, ui.md

No body text was changed.

Agent-dispatch commands that omit allowed-tools give Claude no
guidance on which tools are permitted. The 6 non-agent commands in
this repo already use allowed-tools: Task as the correct pattern;
this PR brings the 14 agent-dispatch commands into line.

Co-Authored-By: Claude Code <noreply@anthropic.com>
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