feat: Add WebFetch and WebSearch to planning tools #79
Conversation
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
- Added WebFetch and WebSearch to the default planning tools in ToolsConfig - Updated description to mention web tools for research - Updated docstring example in ClaudeTaskMasterConfig to reflect changes This enables the planner to use web search and fetch capabilities during the planning phase for research and information gathering. Co-Authored-By: Claude <noreply@anthropic.com>
…ng prompts - Updated prompts_planning.py to allow WebFetch and WebSearch tools - WebFetch enables fetching and analyzing web pages for documentation/research - WebSearch enables searching the web for current information - Updated tests in test_config.py to reflect new default planning tools Co-Authored-By: Claude <noreply@anthropic.com>
… in PLANNING phase - Updated the PLANNING phase tools list to include WebFetch and WebSearch - These tools enable web research and documentation fetching during planning - Updated Purpose description to clarify the research capability Co-Authored-By: Claude <noreply@anthropic.com>
…tools - Renamed test_webfetch_tool_forbidden() to test_webfetch_tool_allowed() - Renamed test_websearch_tool_forbidden() to test_websearch_tool_allowed() - Added assertions to verify WebFetch and WebSearch appear in ALLOWED section - Tests check tools appear before FORBIDDEN section in prompt output - All 92 tests pass successfully Co-Authored-By: Claude <noreply@anthropic.com>
This comment has been minimized.
This comment has been minimized.
- Update test_planning_phase_tools to expect WebFetch and WebSearch - Update test_working_phase_exclusive_tools to properly test that WebFetch/WebSearch are in planning but not verification phase - Add monkeypatch.chdir to temp_dir to avoid loading local config.json - Add reset_config() to ensure tests use default config values The tests were failing because they were loading the local .claude-task-master/config.json which had outdated planning tools (without WebFetch/WebSearch). Co-Authored-By: Claude <noreply@anthropic.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
This PR adds WebFetch and WebSearch tools to the planner phase, enabling Claude to research documentation and current information during the planning phase.
Changes
WebFetchandWebSearchto the default planning tools inToolsConfigWebFetchandWebSearchfrom FORBIDDEN to ALLOWED section in planning promptsCLAUDE.mdTool Configurations table to reflect new planning toolsMotivation
The planner needs to be able to:
This enables more informed and accurate plans based on up-to-date information.
Testing
Related Issues
Resolves the need for planner to access web resources for research during planning.
🤖 Generated with Claude Code
Greptile Overview
Greptile Summary
This PR adds
WebFetchandWebSearchtools to the planning phase, enabling Claude to research documentation and current information during planning. The changes are well-structured and comprehensive:ToolsConfigdefault to include web tools in planning phaseCLAUDE.mdto reflect new capabilitiesThe implementation is consistent across all components. Web tools remain forbidden in the plan update phase (via
prompts_plan_update.py), which is appropriate since plan updates should be quick modifications without external research.All 4487 tests pass, including new tests that verify web tools appear in the ALLOWED section before the FORBIDDEN section in the planning prompt.
Confidence Score: 5/5
Important Files Changed
Sequence Diagram
sequenceDiagram participant User participant Orchestrator participant Agent as Claude Agent (Planning) participant Codebase as Local Codebase participant Web as Web Resources User->>Orchestrator: claudetm start "task" Orchestrator->>Agent: build_planning_prompt(goal) Note over Agent: ALLOWED: Read, Glob, Grep,<br/>Bash, WebFetch, WebSearch Agent->>Codebase: Read files Codebase-->>Agent: File contents Agent->>Codebase: Glob patterns Codebase-->>Agent: Matching files Agent->>Codebase: Grep code Codebase-->>Agent: Search results Agent->>Web: WebFetch documentation URL Web-->>Agent: Documentation content Agent->>Web: WebSearch best practices Web-->>Agent: Search results Agent->>Agent: Analyze codebase + web info Agent->>Agent: Create task list with PRs Agent-->>Orchestrator: Plan text output Orchestrator->>Orchestrator: Extract plan from response Orchestrator->>Orchestrator: Save to plan.md Orchestrator->>Orchestrator: Begin work phase