Skip to content

Add a none provider to the ai-agent-conversation module - #14064

Merged
wu-sheng merged 1 commit into
apache:masterfrom
kezhenxu94:ai-agent-conversation-none-provider
Sep 10, 2026
Merged

wu-sheng merged 1 commit into
apache:masterfrom
kezhenxu94:ai-agent-conversation-none-provider

Conversation

@kezhenxu94

Copy link
Copy Markdown
Member

The ai-agent-conversation module, added in #14057, has no way to be turned off. The - selector cannot remove it, because GraphQLQueryProvider.requiredModules() lists AIAgentConversationModule.NAME for its two conversation resolvers, so a - makes BootstrapFlow.makeSequence() fail the boot with ModuleNotFoundException.

SW_AI_AGENT_CONVERSATION=none now selects a provider that does nothing:

  • It disables the ai_agent_session_data and ai_agent_session_flow models in prepare(), which runs for every provider before any start(), and so before the core module's @Stream scan. Neither table is created on any storage, nor the BanyanDB recordsAIAgent group, whose only members they are. A record with no worker is dropped by RecordStreamProcessor, so the bundled LAL rule stores nothing either; dropping ai-agent from SW_LOG_LAL_FILES skips the verification work as well.

  • It registers a query service that answers listConversations and getConversationRawFiles empty with an errorReason, so the GraphQL module still boots and its queries still answer.

  • It registers no HTTP handler, so the conversation view route is a 404.

  • Update the CHANGES log.

The GraphQL query module requires the module, so the '-' selector cannot
remove it. SW_AI_AGENT_CONVERSATION=none selects a provider that disables
the ai_agent_session_data and ai_agent_session_flow models before the core
module scans them, so neither table nor the BanyanDB group they are the
only members of is created, answers both conversation queries empty with
an errorReason, and registers no conversation view route.
@@ -131,6 +131,7 @@ The conversation page of the UI makes one call, this route, and nothing else.
```yaml
ai-agent-conversation:
selector: ${SW_AI_AGENT_CONVERSATION:default}

@kezhenxu94 kezhenxu94 Sep 9, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I’ll leave it to you to decide if we should disable it by default @wu-sheng

@kezhenxu94
kezhenxu94 requested a review from wu-sheng September 9, 2026 10:29
@wu-sheng wu-sheng added this to the 11.1.0 milestone Sep 10, 2026
@wu-sheng
wu-sheng merged commit adf0ecd into apache:master Sep 10, 2026
465 of 471 checks passed
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.

2 participants