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.
242 lines
6.7 KiB
Go
242 lines
6.7 KiB
Go
package channel
|
|
|
|
import (
|
|
"context"
|
|
"fmt"
|
|
"io"
|
|
"log/slog"
|
|
"strings"
|
|
"sync"
|
|
"testing"
|
|
"time"
|
|
)
|
|
|
|
type fakeConfigStore struct {
|
|
effectiveConfig ChannelConfig
|
|
channelIdentityConfig ChannelIdentityBinding
|
|
configsByType map[ChannelType][]ChannelConfig
|
|
boundChannelIdentityID string
|
|
}
|
|
|
|
func (f *fakeConfigStore) ResolveEffectiveConfig(ctx context.Context, botID string, channelType ChannelType) (ChannelConfig, error) {
|
|
return f.effectiveConfig, nil
|
|
}
|
|
|
|
func (f *fakeConfigStore) GetChannelIdentityConfig(ctx context.Context, channelIdentityID string, channelType ChannelType) (ChannelIdentityBinding, error) {
|
|
if f.channelIdentityConfig.ID == "" && len(f.channelIdentityConfig.Config) == 0 {
|
|
return ChannelIdentityBinding{}, fmt.Errorf("channel user config not found")
|
|
}
|
|
return f.channelIdentityConfig, nil
|
|
}
|
|
|
|
func (f *fakeConfigStore) UpsertChannelIdentityConfig(ctx context.Context, channelIdentityID string, channelType ChannelType, req UpsertChannelIdentityConfigRequest) (ChannelIdentityBinding, error) {
|
|
return f.channelIdentityConfig, nil
|
|
}
|
|
|
|
func (f *fakeConfigStore) ListConfigsByType(ctx context.Context, channelType ChannelType) ([]ChannelConfig, error) {
|
|
if f.configsByType == nil {
|
|
return nil, nil
|
|
}
|
|
return f.configsByType[channelType], nil
|
|
}
|
|
|
|
func (f *fakeConfigStore) ResolveChannelIdentityBinding(ctx context.Context, channelType ChannelType, criteria BindingCriteria) (string, error) {
|
|
if f.boundChannelIdentityID == "" {
|
|
return "", fmt.Errorf("channel user binding not found")
|
|
}
|
|
return f.boundChannelIdentityID, nil
|
|
}
|
|
|
|
type fakeInboundProcessorIntegration struct {
|
|
resp *OutboundMessage
|
|
err error
|
|
gotCfg ChannelConfig
|
|
gotMsg InboundMessage
|
|
}
|
|
|
|
func (f *fakeInboundProcessorIntegration) HandleInbound(ctx context.Context, cfg ChannelConfig, msg InboundMessage, sender StreamReplySender) error {
|
|
f.gotCfg = cfg
|
|
f.gotMsg = msg
|
|
if f.err != nil {
|
|
return f.err
|
|
}
|
|
if f.resp == nil {
|
|
return nil
|
|
}
|
|
if sender == nil {
|
|
return fmt.Errorf("sender missing")
|
|
}
|
|
return sender.Send(ctx, *f.resp)
|
|
}
|
|
|
|
type fakeAdapter struct {
|
|
channelType ChannelType
|
|
mu sync.Mutex
|
|
started []ChannelConfig
|
|
sent []OutboundMessage
|
|
stops int
|
|
}
|
|
|
|
func (f *fakeAdapter) Type() ChannelType {
|
|
return f.channelType
|
|
}
|
|
|
|
func (f *fakeAdapter) Descriptor() Descriptor {
|
|
return Descriptor{Type: f.channelType, DisplayName: "Fake", Capabilities: ChannelCapabilities{Text: true}}
|
|
}
|
|
|
|
func (f *fakeAdapter) ResolveTarget(channelIdentityConfig map[string]any) (string, error) {
|
|
value := strings.TrimSpace(ReadString(channelIdentityConfig, "target"))
|
|
if value == "" {
|
|
return "", fmt.Errorf("missing target")
|
|
}
|
|
return "resolved:" + value, nil
|
|
}
|
|
|
|
func (f *fakeAdapter) NormalizeTarget(raw string) string { return strings.TrimSpace(raw) }
|
|
|
|
func (f *fakeAdapter) Connect(ctx context.Context, cfg ChannelConfig, handler InboundHandler) (Connection, error) {
|
|
f.mu.Lock()
|
|
f.started = append(f.started, cfg)
|
|
f.mu.Unlock()
|
|
stop := func(context.Context) error {
|
|
f.mu.Lock()
|
|
f.stops++
|
|
f.mu.Unlock()
|
|
return nil
|
|
}
|
|
return NewConnection(cfg, stop), nil
|
|
}
|
|
|
|
func (f *fakeAdapter) Send(ctx context.Context, cfg ChannelConfig, msg OutboundMessage) error {
|
|
f.mu.Lock()
|
|
f.sent = append(f.sent, msg)
|
|
f.mu.Unlock()
|
|
return nil
|
|
}
|
|
|
|
func TestManagerHandleInboundIntegratesAdapter(t *testing.T) {
|
|
t.Parallel()
|
|
|
|
log := slog.New(slog.NewTextHandler(io.Discard, &slog.HandlerOptions{}))
|
|
store := &fakeConfigStore{}
|
|
processor := &fakeInboundProcessorIntegration{
|
|
resp: &OutboundMessage{
|
|
Target: "123",
|
|
Message: Message{
|
|
Text: "ok",
|
|
},
|
|
},
|
|
}
|
|
reg := NewRegistry()
|
|
adapter := &fakeAdapter{channelType: ChannelType("test")}
|
|
manager := NewManager(log, reg, store, processor)
|
|
manager.RegisterAdapter(adapter)
|
|
|
|
cfg := ChannelConfig{
|
|
ID: "cfg-1",
|
|
BotID: "bot-1",
|
|
ChannelType: ChannelType("test"),
|
|
Credentials: map[string]any{"botToken": "token"},
|
|
UpdatedAt: time.Now(),
|
|
}
|
|
err := manager.handleInbound(context.Background(), cfg, InboundMessage{
|
|
Channel: ChannelType("test"),
|
|
Message: Message{Text: "hi"},
|
|
BotID: "bot-1",
|
|
ReplyTarget: "123",
|
|
Conversation: Conversation{
|
|
ID: "chat-1",
|
|
Type: "p2p",
|
|
},
|
|
})
|
|
if err != nil {
|
|
t.Fatalf("expected no error, got %v", err)
|
|
}
|
|
|
|
if processor.gotMsg.Conversation.ID != "chat-1" || processor.gotMsg.Message.PlainText() != "hi" || processor.gotMsg.BotID != "bot-1" {
|
|
t.Fatalf("unexpected inbound message: %+v", processor.gotMsg)
|
|
}
|
|
|
|
adapter.mu.Lock()
|
|
defer adapter.mu.Unlock()
|
|
if len(adapter.sent) != 1 {
|
|
t.Fatalf("expected 1 send, got %d", len(adapter.sent))
|
|
}
|
|
if adapter.sent[0].Target != "123" || adapter.sent[0].Message.PlainText() != "ok" {
|
|
t.Fatalf("unexpected outbound message: %+v", adapter.sent[0])
|
|
}
|
|
}
|
|
|
|
func TestManagerSendUsesBinding(t *testing.T) {
|
|
t.Parallel()
|
|
|
|
log := slog.New(slog.NewTextHandler(io.Discard, &slog.HandlerOptions{}))
|
|
store := &fakeConfigStore{
|
|
effectiveConfig: ChannelConfig{
|
|
ID: "cfg-1",
|
|
BotID: "bot-1",
|
|
ChannelType: ChannelType("test"),
|
|
Credentials: map[string]any{"botToken": "token"},
|
|
UpdatedAt: time.Now(),
|
|
},
|
|
channelIdentityConfig: ChannelIdentityBinding{
|
|
ID: "binding-1",
|
|
Config: map[string]any{"target": "alice"},
|
|
},
|
|
}
|
|
reg := NewRegistry()
|
|
adapter := &fakeAdapter{channelType: ChannelType("test")}
|
|
manager := NewManager(log, reg, store, &fakeInboundProcessorIntegration{})
|
|
manager.RegisterAdapter(adapter)
|
|
|
|
err := manager.Send(context.Background(), "bot-1", ChannelType("test"), SendRequest{
|
|
ChannelIdentityID: "user-1",
|
|
Message: Message{
|
|
Text: "hello",
|
|
},
|
|
})
|
|
if err != nil {
|
|
t.Fatalf("expected no error, got %v", err)
|
|
}
|
|
|
|
adapter.mu.Lock()
|
|
defer adapter.mu.Unlock()
|
|
if len(adapter.sent) != 1 {
|
|
t.Fatalf("expected 1 send, got %d", len(adapter.sent))
|
|
}
|
|
if adapter.sent[0].Target != "resolved:alice" || adapter.sent[0].Message.PlainText() != "hello" {
|
|
t.Fatalf("unexpected outbound message: %+v", adapter.sent[0])
|
|
}
|
|
}
|
|
|
|
func TestManagerReconcileStartsAndStops(t *testing.T) {
|
|
t.Parallel()
|
|
|
|
log := slog.New(slog.NewTextHandler(io.Discard, &slog.HandlerOptions{}))
|
|
store := &fakeConfigStore{}
|
|
reg := NewRegistry()
|
|
adapter := &fakeAdapter{channelType: ChannelType("test")}
|
|
manager := NewManager(log, reg, store, &fakeInboundProcessorIntegration{})
|
|
manager.RegisterAdapter(adapter)
|
|
|
|
cfg := ChannelConfig{
|
|
ID: "cfg-1",
|
|
BotID: "bot-1",
|
|
ChannelType: ChannelType("test"),
|
|
Credentials: map[string]any{"botToken": "token"},
|
|
UpdatedAt: time.Now(),
|
|
}
|
|
manager.reconcile(context.Background(), []ChannelConfig{cfg})
|
|
manager.reconcile(context.Background(), nil)
|
|
|
|
adapter.mu.Lock()
|
|
defer adapter.mu.Unlock()
|
|
if len(adapter.started) != 1 {
|
|
t.Fatalf("expected 1 start, got %d", len(adapter.started))
|
|
}
|
|
if adapter.stops != 1 {
|
|
t.Fatalf("expected 1 stop, got %d", adapter.stops)
|
|
}
|
|
}
|