fix(research): expose manage_research as a native tool - #6331
Open
asd-standard wants to merge 1 commit into
Open
asd-standard wants to merge 1 commit into
asd-standard wants to merge 1 commit into
Conversation
manage_research (list/read/delete saved deep-research reports) was fully implemented and indexed but had no entry in FUNCTION_TOOL_SCHEMAS, so native function-calling models never saw it. Register the schema and co-select it with trigger_research on generic "research" intents. Fixes odysseus-dev#2130 Co-Authored-By: DeepSeek-v4-pro
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
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
manage_research(list/read/delete saved deep-research reports) was fully implemented and indexed but missing fromFUNCTION_TOOL_SCHEMAS, so native function-calling models reported it as unavailable. This registers the native schema (actionlist/read/delete,id,search) and makes the generic "research" keyword hint co-select it alongsidetrigger_research, matching the siblingmanage_documentstool.Target branch
dev, notmain. All PRs land indev.Linked Issue
Fixes #2130
Type of Change
Checklist
dev.)devmanage_researchand returns the saved report.How to Test
manage_researchappears in the tool list and returns the report text + sources. Before this fix, the model reported it had nomanage_researchtool.python -m pytest tests/test_manage_research_tool_schema.py tests/test_tool_index_schema_parity.py tests/test_tool_rag_keyword_hints.py -qpython -m py_compile src/tool_schemas.py src/tool_index.pyVisual / UI changes — REQUIRED if you touched anything that renders
No visual/UI change — Python-only (
src/), nostatic/, CSS, or HTML files modified.Screenshots / clips
Not applicable.
Transparency: this PR was AI-assisted (DeepSeek-v4-pro). It is a single, targeted, issue-backed fix, not a bulk auto-generated PR. Commits carry a
Co-Authored-By: DeepSeek-v4-protrailer.