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.
241 lines
7.7 KiB
Go
241 lines
7.7 KiB
Go
package bind_test
|
|
|
|
import (
|
|
"context"
|
|
"encoding/json"
|
|
"errors"
|
|
"fmt"
|
|
"log/slog"
|
|
"os"
|
|
"testing"
|
|
"time"
|
|
|
|
"github.com/jackc/pgx/v5/pgtype"
|
|
"github.com/jackc/pgx/v5/pgxpool"
|
|
|
|
"github.com/memohai/memoh/internal/bind"
|
|
"github.com/memohai/memoh/internal/channel/identities"
|
|
"github.com/memohai/memoh/internal/db"
|
|
"github.com/memohai/memoh/internal/db/sqlc"
|
|
)
|
|
|
|
func setupBindIntegrationTest(t *testing.T) (*sqlc.Queries, *identities.Service, *bind.Service, func()) {
|
|
t.Helper()
|
|
|
|
dsn := os.Getenv("TEST_POSTGRES_DSN")
|
|
if dsn == "" {
|
|
t.Skip("skip integration test: TEST_POSTGRES_DSN is not set")
|
|
}
|
|
|
|
ctx := context.Background()
|
|
pool, err := pgxpool.New(ctx, dsn)
|
|
if err != nil {
|
|
t.Skipf("skip integration test: cannot connect to database: %v", err)
|
|
}
|
|
if err := pool.Ping(ctx); err != nil {
|
|
pool.Close()
|
|
t.Skipf("skip integration test: database ping failed: %v", err)
|
|
}
|
|
|
|
queries := sqlc.New(pool)
|
|
logger := slog.New(slog.NewTextHandler(os.Stdout, &slog.HandlerOptions{Level: slog.LevelDebug}))
|
|
channelIdentitySvc := identities.NewService(logger, queries)
|
|
bindSvc := bind.NewService(logger, pool, queries)
|
|
return queries, channelIdentitySvc, bindSvc, func() { pool.Close() }
|
|
}
|
|
|
|
func createUserForBind(ctx context.Context, queries *sqlc.Queries) (string, error) {
|
|
row, err := queries.CreateUser(ctx, sqlc.CreateUserParams{
|
|
IsActive: true,
|
|
Metadata: []byte("{}"),
|
|
})
|
|
if err != nil {
|
|
return "", err
|
|
}
|
|
return row.ID.String(), nil
|
|
}
|
|
|
|
func createBotForBind(ctx context.Context, queries *sqlc.Queries, ownerUserID string) (string, error) {
|
|
pgOwnerID, err := db.ParseUUID(ownerUserID)
|
|
if err != nil {
|
|
return "", err
|
|
}
|
|
meta, err := json.Marshal(map[string]any{"source": "bind-integration-test"})
|
|
if err != nil {
|
|
return "", err
|
|
}
|
|
row, err := queries.CreateBot(ctx, sqlc.CreateBotParams{
|
|
OwnerUserID: pgOwnerID,
|
|
Type: "personal",
|
|
DisplayName: pgtype.Text{String: "bind-test-bot", Valid: true},
|
|
IsActive: true,
|
|
Metadata: meta,
|
|
})
|
|
if err != nil {
|
|
return "", err
|
|
}
|
|
return row.ID.String(), nil
|
|
}
|
|
|
|
func TestIntegrationConsumeBindCodeSuccessAndSingleUse(t *testing.T) {
|
|
queries, channelIdentitySvc, bindSvc, cleanup := setupBindIntegrationTest(t)
|
|
defer cleanup()
|
|
|
|
ctx := context.Background()
|
|
ownerUserID, err := createUserForBind(ctx, queries)
|
|
if err != nil {
|
|
t.Fatalf("create owner user failed: %v", err)
|
|
}
|
|
sourceChannelIdentity, err := channelIdentitySvc.Create(ctx, "feishu", fmt.Sprintf("bind-success-%d", time.Now().UnixNano()), "source")
|
|
if err != nil {
|
|
t.Fatalf("create source channel identity failed: %v", err)
|
|
}
|
|
|
|
code, err := bindSvc.Issue(ctx, ownerUserID, "feishu", 10*time.Minute)
|
|
if err != nil {
|
|
t.Fatalf("issue bind code failed: %v", err)
|
|
}
|
|
if err := bindSvc.Consume(ctx, code, sourceChannelIdentity.ID); err != nil {
|
|
t.Fatalf("consume bind code failed: %v", err)
|
|
}
|
|
|
|
after, err := bindSvc.Get(ctx, code.Token)
|
|
if err != nil {
|
|
t.Fatalf("get bind code failed: %v", err)
|
|
}
|
|
if after.UsedAt.IsZero() {
|
|
t.Fatal("expected used_at to be set after consume")
|
|
}
|
|
if after.UsedByChannelIdentityID != sourceChannelIdentity.ID {
|
|
t.Fatalf("expected used_by_channel_identity_id=%s, got %s", sourceChannelIdentity.ID, after.UsedByChannelIdentityID)
|
|
}
|
|
|
|
linkedUserID, err := channelIdentitySvc.GetLinkedUserID(ctx, sourceChannelIdentity.ID)
|
|
if err != nil {
|
|
t.Fatalf("get linked user failed: %v", err)
|
|
}
|
|
if linkedUserID != ownerUserID {
|
|
t.Fatalf("expected linked user=%s, got %s", ownerUserID, linkedUserID)
|
|
}
|
|
|
|
if err := bindSvc.Consume(ctx, code, sourceChannelIdentity.ID); !errors.Is(err, bind.ErrCodeUsed) {
|
|
t.Fatalf("expected ErrCodeUsed on second consume, got %v", err)
|
|
}
|
|
}
|
|
|
|
func TestIntegrationConsumeBindCodeRollbackOnLinkConflict(t *testing.T) {
|
|
queries, channelIdentitySvc, bindSvc, cleanup := setupBindIntegrationTest(t)
|
|
defer cleanup()
|
|
|
|
ctx := context.Background()
|
|
ownerUserID, err := createUserForBind(ctx, queries)
|
|
if err != nil {
|
|
t.Fatalf("create owner user failed: %v", err)
|
|
}
|
|
otherUserID, err := createUserForBind(ctx, queries)
|
|
if err != nil {
|
|
t.Fatalf("create other user failed: %v", err)
|
|
}
|
|
sourceChannelIdentity, err := channelIdentitySvc.Create(ctx, "feishu", fmt.Sprintf("bind-rollback-%d", time.Now().UnixNano()), "source")
|
|
if err != nil {
|
|
t.Fatalf("create source channel identity failed: %v", err)
|
|
}
|
|
if err := channelIdentitySvc.LinkChannelIdentityToUser(ctx, sourceChannelIdentity.ID, otherUserID); err != nil {
|
|
t.Fatalf("pre-link source channel identity failed: %v", err)
|
|
}
|
|
|
|
code, err := bindSvc.Issue(ctx, ownerUserID, "feishu", 10*time.Minute)
|
|
if err != nil {
|
|
t.Fatalf("issue bind code failed: %v", err)
|
|
}
|
|
if err := bindSvc.Consume(ctx, code, sourceChannelIdentity.ID); !errors.Is(err, bind.ErrLinkConflict) {
|
|
t.Fatalf("expected ErrLinkConflict, got %v", err)
|
|
}
|
|
|
|
after, err := bindSvc.Get(ctx, code.Token)
|
|
if err != nil {
|
|
t.Fatalf("get bind code failed: %v", err)
|
|
}
|
|
if !after.UsedAt.IsZero() {
|
|
t.Fatal("expected used_at to remain empty when consume fails")
|
|
}
|
|
}
|
|
|
|
func TestIntegrationConsumeLinksChannelIdentityToIssuerUser(t *testing.T) {
|
|
queries, channelIdentitySvc, bindSvc, cleanup := setupBindIntegrationTest(t)
|
|
defer cleanup()
|
|
|
|
ctx := context.Background()
|
|
ownerUserID, err := createUserForBind(ctx, queries)
|
|
if err != nil {
|
|
t.Fatalf("create owner user failed: %v", err)
|
|
}
|
|
sourceChannelIdentity, err := channelIdentitySvc.ResolveByChannelIdentity(ctx, "feishu", fmt.Sprintf("bind-src-%d", time.Now().UnixNano()), "source")
|
|
if err != nil {
|
|
t.Fatalf("create source channelIdentity failed: %v", err)
|
|
}
|
|
code, err := bindSvc.Issue(ctx, ownerUserID, "feishu", 10*time.Minute)
|
|
if err != nil {
|
|
t.Fatalf("issue bind code failed: %v", err)
|
|
}
|
|
if err := bindSvc.Consume(ctx, code, sourceChannelIdentity.ID); err != nil {
|
|
t.Fatalf("consume bind code failed: %v", err)
|
|
}
|
|
|
|
after, err := bindSvc.Get(ctx, code.Token)
|
|
if err != nil {
|
|
t.Fatalf("get bind code failed: %v", err)
|
|
}
|
|
if after.UsedAt.IsZero() {
|
|
t.Fatal("expected code used_at set after consume")
|
|
}
|
|
if after.UsedByChannelIdentityID != sourceChannelIdentity.ID {
|
|
t.Fatalf("expected used_by_channel_identity_id=%s, got %s", sourceChannelIdentity.ID, after.UsedByChannelIdentityID)
|
|
}
|
|
|
|
linkedUserID, err := channelIdentitySvc.GetLinkedUserID(ctx, sourceChannelIdentity.ID)
|
|
if err != nil {
|
|
t.Fatalf("get linked user failed: %v", err)
|
|
}
|
|
if linkedUserID != ownerUserID {
|
|
t.Fatalf("expected linked user=%s, got %s", ownerUserID, linkedUserID)
|
|
}
|
|
}
|
|
|
|
func TestIntegrationConsumeConflictDoesNotMarkUsed(t *testing.T) {
|
|
queries, channelIdentitySvc, bindSvc, cleanup := setupBindIntegrationTest(t)
|
|
defer cleanup()
|
|
|
|
ctx := context.Background()
|
|
issuerUserID, err := createUserForBind(ctx, queries)
|
|
if err != nil {
|
|
t.Fatalf("create issuer user failed: %v", err)
|
|
}
|
|
otherUserID, err := createUserForBind(ctx, queries)
|
|
if err != nil {
|
|
t.Fatalf("create other user failed: %v", err)
|
|
}
|
|
sourceChannelIdentity, err := channelIdentitySvc.ResolveByChannelIdentity(ctx, "feishu", fmt.Sprintf("bind-conflict-%d", time.Now().UnixNano()), "source")
|
|
if err != nil {
|
|
t.Fatalf("create source channelIdentity failed: %v", err)
|
|
}
|
|
if err := channelIdentitySvc.LinkChannelIdentityToUser(ctx, sourceChannelIdentity.ID, otherUserID); err != nil {
|
|
t.Fatalf("pre-link source channelIdentity failed: %v", err)
|
|
}
|
|
code, err := bindSvc.Issue(ctx, issuerUserID, "feishu", 10*time.Minute)
|
|
if err != nil {
|
|
t.Fatalf("issue bind code failed: %v", err)
|
|
}
|
|
if err := bindSvc.Consume(ctx, code, sourceChannelIdentity.ID); !errors.Is(err, bind.ErrLinkConflict) {
|
|
t.Fatalf("expected ErrLinkConflict, got %v", err)
|
|
}
|
|
|
|
after, err := bindSvc.Get(ctx, code.Token)
|
|
if err != nil {
|
|
t.Fatalf("get bind code failed: %v", err)
|
|
}
|
|
if !after.UsedAt.IsZero() {
|
|
t.Fatal("expected code to remain unused after conflict")
|
|
}
|
|
}
|