Skip to content

fix(kb): add process_query() to KBLibrarianAgent — fixes RAG agent crash (#4531)#4542

Merged
mrveiss merged 1 commit intoDev_new_guifrom
issue-4531
Apr 14, 2026
Merged

fix(kb): add process_query() to KBLibrarianAgent — fixes RAG agent crash (#4531)#4542
mrveiss merged 1 commit intoDev_new_guifrom
issue-4531

Conversation

@mrveiss
Copy link
Copy Markdown
Owner

@mrveiss mrveiss commented Apr 14, 2026

Closes #4531

Summary

  • Added process_query() method to KBLibrarianAgent wrapping search_knowledge() and optionally answer_question(), returning a dict compatible with KBQueryResponse
  • Added four missing instance attributes (enabled, max_results, similarity_threshold, auto_summarize) that api/kb_librarian.py reads and temporarily overrides per-request
  • Also adds response key aliases used by agent_execution.py

Test Status

FAIL (pre-existing: llama_index not installed in dev env — unrelated to this fix)

🤖 Generated with Claude Code

…rs (#4531)

KBLibrarianAgent was missing process_query(), enabled, max_results,
similarity_threshold, and auto_summarize — all referenced by
api/kb_librarian.py, api/workflow.py, and agent_execution.py, causing
AttributeError on every KB/RAG query.
@mrveiss mrveiss merged commit 65abef1 into Dev_new_gui Apr 14, 2026
3 of 4 checks passed
@mrveiss mrveiss deleted the issue-4531 branch April 14, 2026 14:45
@github-actions
Copy link
Copy Markdown

✅ SSOT Configuration Compliance: Passing

🎉 No hardcoded values detected that have SSOT config equivalents!

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.

1 participant