fix: rename fetch_feeds_tool to refresh_feeds_tool#3
Conversation
The old name sounded like a read operation but triggers a live network crawl of all feed sources (30-60s). Agents calling it inline during briefings hit gateway timeouts. The new name makes the cost obvious. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request renames the fetch_feeds_tool to refresh_feeds_tool in the server implementation, README, and CLAUDE.md, updating its description to clarify that it crawls feed sources over the network. The review feedback correctly identifies that the updated docstring in server.py refers to list_entries instead of the actual tool name list_entries_tool, which could cause issues or hallucinations for LLMs consuming the tool definition.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
| async def refresh_feeds_tool(feed_id: int | None = None) -> str: | ||
| """Crawl feed sources over the network for new content. Slow — takes 30-60s for all feeds. | ||
|
|
||
| Prefer the systemd timer for background updates. Use list_entries to read cached content. |
There was a problem hiding this comment.
The docstring refers to list_entries to read cached content, but the actual registered tool name is list_entries_tool. Since MCP tool docstrings are directly exposed to LLMs, using the incorrect tool name can cause the LLM to fail or hallucinate when trying to call the recommended tool. Please update this to list_entries_tool.
| Prefer the systemd timer for background updates. Use list_entries to read cached content. | |
| Prefer the systemd timer for background updates. Use list_entries_tool to read cached content. |
1 participant
Summary
fetch_feeds_tool→refresh_feeds_toolto make the network cost obviouslist_entriesfor reading cached contentTest plan
uv run ruff check src/passesrefresh_feeds_toolin MCP tool listing🤖 Generated with Claude Code