Skip to content

Conversation

@labkey-matthewb
Copy link
Contributor

Rationale

McpService for registering MCP tools, etc.
Basic chat page for testing (test-chat.view)

Related Pull Requests

Changes

labkey-matthewb and others added 30 commits September 15, 2025 11:33
migration to spring-ai for chat client
test-chat.view (AbstractAgentAction)
@labkey-matthewb labkey-matthewb requested review from labkey-jeckels and removed request for labkey-jeckels January 14, 2026 20:11
import org.springframework.ai.chat.model.ToolContext;
import java.util.Map;

public class McpContext implements ContainerUser
Copy link
Contributor

Choose a reason for hiding this comment

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

A sentence or two of docs would be helpful here too.

// Build URL with same base as current document, endpoint /query-queryagent.api and prompt parameter
var url = new URL('./query-queryagent.api', window.location.href);
url.searchParams.set('prompt', prompt);
var req = new XMLHttpRequest();
Copy link
Contributor

Choose a reason for hiding this comment

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

Worth using LABKEY.Ajax instead?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This code is really trash. I hope it is not long lived.



@Override
public ToolCallback @NonNull [] getToolCallbacks()
Copy link
Contributor

Choose a reason for hiding this comment

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

Returning arrays always feels clunky. Consider a Collection of some sort.

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.

3 participants