mirror of
https://github.com/memohai/Memoh.git
synced 2026-04-27 07:16:19 +09:00
6acdd191c7
commit bcdb026ae43e4f95d0b2c4f9bd440a2df9d6b514 Author: Ran <16112591+chen-ran@users.noreply.github.com> Date: Thu Feb 12 17:10:32 2026 +0800 chore: update DEVELOPMENT.md commit30281742efMerge:ca5c6a15b05f13Author: BBQ <bbq@BBQdeMacBook-Air.local> Date: Thu Feb 12 15:49:17 2026 +0800 merge(github/main): integrate fx dependency injection framework Merge upstream fx refactor and adapt all services to use go.uber.org/fx for dependency injection. Resolve conflicts in main.go, server.go, and service constructors while preserving our domain model changes. - Fix telegram adapter panic on shutdown (double close channel) - Fix feishu adapter processing messages after stop - Increase directory lookup timeout from 2s to 5s commitca5c6a1866Author: BBQ <bbq@BBQdeMacBook-Air.local> Date: Thu Feb 12 15:33:09 2026 +0800 refactor(core): restructure conversation, channel and message domains - Rename chat module to conversation with flow-based architecture - Move channelidentities into channel/identities subpackage - Add channel/route for routing logic - Add message service with event hub - Add MCP providers: container, directory, schedule - Refactor Feishu/Telegram adapters with directory and stream support - Add platform management page and channel badges in web UI - Update database schema for conversations, messages and channel routes - Add @memoh/shared package for cross-package type definitions commit75e2ef0467Merge:d99ba3801cb6c8Author: BBQ <bbq@BBQdeMacBook-Air.local> Date: Thu Feb 12 14:45:49 2026 +0800 merge(github): merge github/main, resolve index.ts URL conflict Keep our defensive absolute-URL check in createAuthFetcher. commitd99ba38b7dMerge:860e20f35ce7d1Author: BBQ <bbq@BBQdeMacBook-Air.local> Date: Thu Feb 12 05:20:18 2026 +0800 merge(github): merge github/main, keep our code and docs/spec commit860e20fe70Author: BBQ <bbq@BBQdeMacBook-Air.local> Date: Wed Feb 11 22:13:27 2026 +0800 docs(docs): add concepts and style guides for VitePress site - Add concepts: identity-and-binding, index (en/zh) - Add style: terminology (en/zh) - Update index and zh/index - Update .vitepress/config.ts commita75fdb8040Author: BBQ <bbq@BBQdeMacBook-Air.local> Date: Wed Feb 11 17:37:16 2026 +0800 refactor(mcp): standardize unified tool gateway on go-sdk Split business executors from federation sources and migrate unified tool/federation transports to the official go-sdk for stricter MCP compliance and safer session lifecycle handling. Add targeted regression tests for accept compatibility, initialization retries, pending cleanup, and include updated swagger artifacts. commit02b33c8e85Author: BBQ <bbq@BBQdeMacBook-Air.local> Date: Wed Feb 11 15:42:21 2026 +0800 refactor(core): finalize user-centric identity and policy cleanup Unify auth and chat identity semantics around user_id, enforce personal-bot owner-only authorization, and remove legacy compatibility branches in integration tests. commit06e8619a37Author: BBQ <bbq@BBQdeMacBook-Air.local> Date: Wed Feb 11 14:47:03 2026 +0800 refactor(core): migrate channel identity and binding across app Align channel identity and bind flow across backend and app-facing layers, including generated swagger artifacts and package lock updates while excluding docs content changes.
127 lines
3.6 KiB
Go
127 lines
3.6 KiB
Go
package mcp
|
|
|
|
import (
|
|
"context"
|
|
"errors"
|
|
"log/slog"
|
|
"testing"
|
|
)
|
|
|
|
type gatewayTestProvider struct {
|
|
tools []ToolDescriptor
|
|
callResult map[string]map[string]any
|
|
callErr map[string]error
|
|
}
|
|
|
|
func (p *gatewayTestProvider) ListTools(ctx context.Context, session ToolSessionContext) ([]ToolDescriptor, error) {
|
|
return p.tools, nil
|
|
}
|
|
|
|
func (p *gatewayTestProvider) CallTool(ctx context.Context, session ToolSessionContext, toolName string, arguments map[string]any) (map[string]any, error) {
|
|
if err, ok := p.callErr[toolName]; ok {
|
|
return nil, err
|
|
}
|
|
if result, ok := p.callResult[toolName]; ok {
|
|
return result, nil
|
|
}
|
|
return nil, ErrToolNotFound
|
|
}
|
|
|
|
func TestToolGatewayServiceListTools(t *testing.T) {
|
|
providerA := &gatewayTestProvider{
|
|
tools: []ToolDescriptor{
|
|
{Name: "tool_a", InputSchema: map[string]any{"type": "object"}},
|
|
{Name: "dup_tool", InputSchema: map[string]any{"type": "object"}},
|
|
},
|
|
}
|
|
providerB := &gatewayTestProvider{
|
|
tools: []ToolDescriptor{
|
|
{Name: "tool_b", InputSchema: map[string]any{"type": "object"}},
|
|
{Name: "dup_tool", InputSchema: map[string]any{"type": "object"}},
|
|
},
|
|
}
|
|
service := NewToolGatewayService(slog.Default(), []ToolExecutor{providerA, providerB}, nil)
|
|
|
|
tools, err := service.ListTools(context.Background(), ToolSessionContext{BotID: "bot-1"})
|
|
if err != nil {
|
|
t.Fatalf("list tools failed: %v", err)
|
|
}
|
|
if len(tools) != 3 {
|
|
t.Fatalf("expected 3 tools after dedupe, got %d", len(tools))
|
|
}
|
|
}
|
|
|
|
func TestToolGatewayServiceCallToolSuccess(t *testing.T) {
|
|
provider := &gatewayTestProvider{
|
|
tools: []ToolDescriptor{
|
|
{Name: "echo_tool", InputSchema: map[string]any{"type": "object"}},
|
|
},
|
|
callResult: map[string]map[string]any{
|
|
"echo_tool": {
|
|
"content": []map[string]any{
|
|
{"type": "text", "text": "ok"},
|
|
},
|
|
},
|
|
},
|
|
callErr: map[string]error{},
|
|
}
|
|
service := NewToolGatewayService(slog.Default(), []ToolExecutor{provider}, nil)
|
|
|
|
result, err := service.CallTool(context.Background(), ToolSessionContext{BotID: "bot-1"}, ToolCallPayload{
|
|
Name: "echo_tool",
|
|
Arguments: map[string]any{"value": "hello"},
|
|
})
|
|
if err != nil {
|
|
t.Fatalf("call tool should not fail: %v", err)
|
|
}
|
|
if _, ok := result["content"]; !ok {
|
|
t.Fatalf("expected content in tool result")
|
|
}
|
|
}
|
|
|
|
func TestToolGatewayServiceCallToolNotFound(t *testing.T) {
|
|
provider := &gatewayTestProvider{
|
|
tools: []ToolDescriptor{},
|
|
callResult: map[string]map[string]any{},
|
|
callErr: map[string]error{},
|
|
}
|
|
service := NewToolGatewayService(slog.Default(), []ToolExecutor{provider}, nil)
|
|
|
|
result, err := service.CallTool(context.Background(), ToolSessionContext{BotID: "bot-1"}, ToolCallPayload{
|
|
Name: "missing_tool",
|
|
Arguments: map[string]any{},
|
|
})
|
|
if err != nil {
|
|
t.Fatalf("call should return mcp error result instead of failing: %v", err)
|
|
}
|
|
isErr, _ := result["isError"].(bool)
|
|
if !isErr {
|
|
t.Fatalf("expected isError=true for missing tool")
|
|
}
|
|
}
|
|
|
|
func TestToolGatewayServiceCallToolProviderError(t *testing.T) {
|
|
provider := &gatewayTestProvider{
|
|
tools: []ToolDescriptor{
|
|
{Name: "broken_tool", InputSchema: map[string]any{"type": "object"}},
|
|
},
|
|
callResult: map[string]map[string]any{},
|
|
callErr: map[string]error{
|
|
"broken_tool": errors.New("boom"),
|
|
},
|
|
}
|
|
service := NewToolGatewayService(slog.Default(), []ToolExecutor{provider}, nil)
|
|
|
|
result, err := service.CallTool(context.Background(), ToolSessionContext{BotID: "bot-1"}, ToolCallPayload{
|
|
Name: "broken_tool",
|
|
Arguments: map[string]any{},
|
|
})
|
|
if err != nil {
|
|
t.Fatalf("call should not return hard error: %v", err)
|
|
}
|
|
isErr, _ := result["isError"].(bool)
|
|
if !isErr {
|
|
t.Fatalf("expected isError=true for provider failure")
|
|
}
|
|
}
|