Skip to content

feat(ai): send Search the conversation context - #125

Closed
jm-rivera wants to merge 1 commit into
devfrom
feat/ai-search-conversation-context
Closed

jm-rivera wants to merge 1 commit into
devfrom
feat/ai-search-conversation-context

Conversation

@jm-rivera

@jm-rivera jm-rivera commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

Sets SEARCH_SENDS_CONVERSATION_CONTEXT on the ai service, so data-mcp sends Search the user's message and the scope entries the conversation carries (data_mcp.settings.Settings.search_sends_conversation_context). A scope choice switched on one turn, such as net disbursements or CRS 120 alone, then holds on the next.

Merge after the dfo-apis scope-choices release (DevelopmentFinanceObservatory/apis#163) and the dfo-ai one (DevelopmentFinanceObservatory/ai#88) are deployed to dev: they add the user_message and carried_entries fields this sends.

data-mcp passes the user's message and the scope entries the
conversation carries to Search, so a scope choice switched on one turn
holds on the next.
@jm-rivera

Copy link
Copy Markdown
Contributor Author

Closing: the conversation context goes to Search through a structural change in a separate pull request, not through this setting.

@jm-rivera jm-rivera closed this Oct 27, 2026
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