Skip to content

Fix webfetch#859

Closed
LizBing wants to merge 0 commit intosipeed:mainfrom
LizBing:main
Closed

Fix webfetch#859
LizBing wants to merge 0 commit intosipeed:mainfrom
LizBing:main

Conversation

@LizBing
Copy link

@LizBing LizBing commented Feb 27, 2026

📝 Description

The return of Tool DuckDuckGo isn’t passed to LLM previously.

🗣️ Type of Change

  • 🐞 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 📖 Documentation update
  • ⚡ Code refactoring (no functional changes, no api changes)

🤖 AI Code Generation

  • 🤖 Fully AI-generated (100% AI, 0% Human)
  • 🛠️ Mostly AI-generated (AI draft, Human verified/modified)
  • 👨‍💻 Mostly Human-written (Human lead, AI assisted or none)

🔗 Related Issue

📚 Technical Context (Skip for Docs)

  • Reference URL:
  • Reasoning:

🧪 Test Environment

  • Hardware:
  • OS:
  • Model/Provider:
  • Channels:

📸 Evidence (Optional)

Click to view Logs/Screenshots

☑️ Checklist

  • My code/docs follow the style of this project.
  • I have performed a self-review of my own changes.
  • I have updated the documentation accordingly.

Copy link

@nikolasdehor nikolasdehor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Correct fix. The previous code returned the metadata summary ("Fetched N bytes from URL...") in ForLLM but omitted the actual fetched text content, which means the LLM could see that a page was fetched but could never read the content. The fix appends the actual text to the ForLLM field so the LLM can reason over the fetched content.

LGTM.

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.

2 participants