Skip to content

Add web fetch tool#36

Open
amrit110 wants to merge 2 commits intomainfrom
ak/add_web_fetch_tool
Open

Add web fetch tool#36
amrit110 wants to merge 2 commits intomainfrom
ak/add_web_fetch_tool

Conversation

@amrit110
Copy link
Member

@amrit110 amrit110 commented Feb 4, 2026

Summary

This PR adds a new tool to the tools package. The tool is for fetching web content (HTML pages and PDFs given a URL)

Type of Change

  • 🐛 Bug fix (non-breaking change that fixes an issue)
  • ✨ New feature (non-breaking change that adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📝 Documentation update
  • 🔧 Refactoring (no functional changes)
  • ⚡ Performance improvement
  • 🧪 Test improvements
  • 🔒 Security fix

Changes Made

  • Add web fetch tool (fetches HTML content and PDFs)
  • Add unit tests

Testing

  • Tests pass locally (uv run pytest tests/)
  • Type checking passes (uv run mypy <src_dir>)
  • Linting passes (uv run ruff check src_dir/)
  • Manual testing performed (describe below)

Manual testing details:
Tested with Agent that can call this tool

Screenshots/Recordings

NA

Related Issues

NA

Deployment Notes

Checklist

  • Code follows the project's style guidelines
  • Self-review of code completed
  • Documentation updated (if applicable)
  • No sensitive information (API keys, credentials) exposed

@amrit110 amrit110 marked this pull request as ready for review February 4, 2026 22:14
@amrit110 amrit110 self-assigned this Feb 4, 2026
@amrit110 amrit110 added the enhancement New feature or request label Feb 4, 2026
@amrit110 amrit110 requested review from fcogidi and lotif February 4, 2026 22:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant