Skip to content

Conversation

nl0
Copy link
Member

@nl0 nl0 commented Sep 23, 2025

Description

TODO

  • Unit tests
  • Security: Confirm that this change meets security best practices and does not violate the security model
  • Open and Embed: Confirm that this change doesn't break Open variant and Embed widget
  • Documentation
    • run optipng on any new PNGs
    • Python: Run build.py for new docstrings
    • JavaScript: basic explanation and screenshot of new features
    • Markdown somewhere in docs/**/*.md that explains the feature to end users (said .md files should be linked from SUMMARY.md so they appear on https://docs.quilt.bio)
    • Markdown docs for developers
  • Changelog entry (skip if change is not significant to end users, e.g. docs only)

nl0 and others added 8 commits September 23, 2025 10:47
Create new Agent component with chat interface for MCP integration. Includes routing, navigation link, and reusable chat UI components adapted from existing Assistant implementation. Dummy responses for now, MCP client implementation to follow.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
- Create dedicated Agent component separate from Qurator Assistant
- Copy and adapt Assistant model files (Bedrock, LLM, Content, Tool, Conversation)
- Implement simplified Agent API without show/hide/ask methods
- Set up AgentChat UI with real LLM conversation flow
- Add /agent route and navigation link in header
- Fix toolConfig handling for Bedrock API compatibility
- Update task tracking for completed items

This establishes the foundation for MCP integration by providing
a working LLM-powered Agent with conversation state management.
- Add MCPClient module with StreamableHTTPClientTransport
- Create MCPToolAdapter to bridge MCP tools with internal Tool system
- Implement ToolService provider in Conversation state machine
- Add ServerStatus UI component showing MCP connection state
- Auto-connect to MCP server on Agent mount
- Configure fetch server URL for testing

Note: Browser CORS limitations prevent direct MCP server connections.
Requires CORS proxy or local MCP server for full functionality.
- Use passThru pattern to ensure conversation layer sees updated tools
- Configure MCPClient to use local test server at http://127.0.0.1:8000/mcp
- Successfully connect to MCP server with proper CORS headers
- Tools are now properly loaded and available to the Agent
- Update task completion status in documentation

The Agent can now access MCP tools after they're loaded asynchronously.
Copy link

codecov bot commented Sep 23, 2025

Codecov Report

❌ Patch coverage is 0.16207% with 616 lines in your changes missing coverage. Please review.
✅ Project coverage is 38.80%. Comparing base (165c894) to head (95d88a7).

Files with missing lines Patch % Lines
catalog/app/components/Agent/Model/Conversation.ts 0.00% 147 Missing ⚠️
catalog/app/components/Agent/UI/AgentChat.tsx 0.00% 96 Missing ⚠️
catalog/app/components/Agent/Model/Agent.tsx 0.00% 75 Missing ⚠️
catalog/app/components/Agent/Model/Bedrock.ts 0.00% 71 Missing ⚠️
...talog/app/components/Agent/Model/MCPToolAdapter.ts 0.00% 50 Missing ⚠️
catalog/app/components/Agent/Model/Tool.ts 0.00% 44 Missing ⚠️
catalog/app/components/Agent/Model/MCPClient.ts 0.00% 36 Missing ⚠️
catalog/app/components/Agent/UI/ServerStatus.tsx 0.00% 31 Missing ⚠️
catalog/app/components/Agent/UI/Input.tsx 0.00% 28 Missing ⚠️
catalog/app/components/Agent/Model/LLM.ts 0.00% 13 Missing ⚠️
... and 5 more
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4556      +/-   ##
==========================================
- Coverage   39.44%   38.80%   -0.65%     
==========================================
  Files         852      866      +14     
  Lines       36847    37464     +617     
  Branches     6013     5864     -149     
==========================================
+ Hits        14536    14537       +1     
- Misses      21087    22409    +1322     
+ Partials     1224      518     -706     
Flag Coverage Δ
api-python 91.67% <ø> (ø)
catalog 19.74% <0.16%> (-0.45%) ⬇️
lambda 92.60% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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