fix: add missing tools declarations to agent-sdk-verifier agents#196
Open
xiaolai wants to merge 1 commit intoccplugins:mainfrom
Open
fix: add missing tools declarations to agent-sdk-verifier agents#196xiaolai wants to merge 1 commit intoccplugins:mainfrom
xiaolai wants to merge 1 commit intoccplugins:mainfrom
Conversation
Both agent-sdk-verifier-ts.md and agent-sdk-verifier-py.md had no tools declared, but their bodies explicitly require Bash (for running tsc/pip) and WebFetch (for fetching SDK docs). Without the tools declaration, Claude Code cannot grant the agent access to these tools and verification steps silently fail. Co-Authored-By: Claude Code <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bug
Both
plugins/agent-sdk-dev/agents/agent-sdk-verifier-ts.mdandagent-sdk-verifier-py.mdhave notools:declaration in their frontmatter, but their bodies explicitly require tools:agent-sdk-verifier-ts.mdstep 3 instructs: "Runnpx tsc --noEmitto check for type errors" → requires Bash. Step 2 instructs: "Use WebFetch to reference the official TypeScript SDK docs" → requires WebFetch. Reading project files requires Read.agent-sdk-verifier-py.mdsteps instruct readingrequirements.txt,pyproject.toml, etc. → requires Read. Checking SDK docs → requires WebFetch. Runningpip/pythonvalidation commands → requires Bash.Impact
Without a
tools:declaration, Claude Code does not grant the agent access to any tools. The verification steps that call Bash or WebFetch silently produce no output or an error, making the agent appear to complete but actually skip its most critical checks (type-checking, dependency validation, SDK doc comparison).Fix
Add
tools: Bash, Read, WebFetchto the frontmatter of both verifier agents — the minimal set required by their documented verification procedures.