fix: correct summarizer typos and wire dead web search fallback route - #157
fix: correct summarizer typos and wire dead web search fallback route#157bhavyakeerthi3 wants to merge 3 commits into
Conversation
Fixed multiple typos ('adn' -> 'and', 'infromation' -> 'information',
'retining' -> 'retaining', 'drived' -> 'derived') and corrected the
broken numbering sequence in the expert curator system prompt.
Clean and professional prompts improve the LLM's adherence to
instructions and overall response quality.
Typos and numbering in the summarizer prompt, from #157.
|
Thank you — both halves of this were real, and verified against current Taken: the prompt typos, merged in #175. Confirmed but not taken yet: the dead Not fixed now because it sits in the Cross-Database Prototype profile, which Closing as partially harvested rather than stale. |
🐛Summary
Surgical hardening of the
CrossDatabaseagent: fixes pre-existing typos in the summarization prompt and resolves a silent "dead route" where web search decisions were never executed.Changes
1. Prompt Improvements & Typo Fixes
src/agent/tasks/cross_database/summarize_reactome_uniprot.pyADN→and,infromation→information)2. Wiring the Web Search Fallback
src/agent/profiles/cross_database.pyThe Issue:
The
perform_web_searchroute was hardcoded to point directly togenerate_final_response— the agent would decide to search but skip execution entirely, producing incomplete responses.The Fix:
web_search_resultstoCrossDatabaseStateperform_web_searchnode using the existingsearch_workflowassess_completeness→perform_web_search→generate_final_responseImpact
Verification
CrossDatabaseStatecorrectly holdsweb_search_resultsassess_completeness→perform_web_search→generate_final_responsetests/test_config.pypassingFixes #123