-
-
Notifications
You must be signed in to change notification settings - Fork 1
feat:Add Assistants, MCP discovery, vector-store endpoints; schema updates #208
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
WalkthroughAdds new public API endpoints for Assistants, MCP discovery, and vector-store retrieval; introduces and refactors numerous schemas (Assistant lifecycle requests, tool definitions/resources, enums); updates references to a unified ToolDefinition; augments run result/error handling; and renames/realigns tooling resources within a single OpenAPI file. Changes
Sequence Diagram(s)sequenceDiagram
autonumber
participant C as Client
participant API as Studio API
participant AS as AssistantsService
participant EE as ExecutionEngine
participant ST as Storage
rect rgba(200,240,255,0.4)
Note over C,API: New Assistant endpoints
C->>API: POST /studio/v1/assistants (CreateAssistantRequest)
API->>AS: createAssistant(request)
AS-->>API: Assistant
API-->>C: 200 Assistant
C->>API: PATCH /studio/v1/assistants/{id} (ModifyAssistantRequest)
API->>AS: modifyAssistant(id, request)
AS-->>API: Assistant
API-->>C: 200 Assistant
end
rect rgba(220,255,220,0.4)
C->>API: POST /studio/v1/assistants/{id}/run (RunAssistantRequest)
API->>EE: runAssistant(id, inputs, tools, resources)
EE->>ST: fetch tool resources / configs
ST-->>EE: resources
EE-->>API: MaestroRunResult | MaestroRunError
API-->>C: 200 MaestroRunResult
end
alt Validation error
API-->>C: 422 HTTPValidationError
end
sequenceDiagram
autonumber
participant C as Client
participant API as Studio API
participant MCP as MCP Registry/Discovery
rect rgba(255,245,200,0.4)
Note over C,API: New MCP discovery endpoint
C->>API: POST /studio/v1/mcp/discover (MCPDefinition)
API->>MCP: discover(definition)
MCP-->>API: MCPDiscoveryResponse (tools, resources)
API-->>C: 200 MCPDiscoveryResponse
end
alt Validation error
API-->>C: 422 HTTPValidationError
end
sequenceDiagram
autonumber
participant C as Client
participant API as Studio API
participant VS as VectorStoreService
participant DB as VectorStore DB
rect rgba(240,240,255,0.4)
C->>API: GET /studio/v1/demos/regulations/vector-store/{vector_store_id}
API->>VS: getVectorStore(id)
VS->>DB: select by id
DB-->>VS: vector store record
VS-->>API: vector store
API-->>C: 200 vector store
end
Estimated code review effort🎯 4 (Complex) | ⏱️ ~60 minutes Poem
Pre-merge checks and finishing touches❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✨ Finishing touches🧪 Generate unit tests
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro ⛔ Files ignored due to path filters (136)
📒 Files selected for processing (1)
🔇 Additional comments (29)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary by CodeRabbit
New Features
Refactor