@spaceflow/core 0.1.3 → 0.2.0

package/src/shared/feishu-sdk/feishu-sdk.service.ts

@@ -1,9 +1,7 @@
-import { Inject, Injectable, type OnModuleDestroy, type OnModuleInit } from "@nestjs/common";
-import { EventEmitter2 } from "@nestjs/event-emitter";
+import { EventEmitter } from "events";
 import * as lark from "@larksuiteoapi/node-sdk";
 import {
   FeishuModuleOptions,
-  FEISHU_MODULE_OPTIONS,
   FeishuUser,
   GetUserParams,
   FEISHU_CARD_ACTION_TRIGGER,
@@ -15,12 +13,39 @@ import {
 /**
  * 飞书 API 服务
  */
-@Injectable()
-export class FeishuSdkService implements OnModuleInit, OnModuleDestroy {
+export class FeishuSdkService {
   protected readonly client: lark.Client;
+  protected readonly eventEmitter: EventEmitter;
+  private eventDispatcher: lark.EventDispatcher | null = null;
 
-  onModuleInit(): void {
-    const eventDispatcher = new lark.EventDispatcher({
+  constructor(protected readonly options: FeishuModuleOptions) {
+    this.eventEmitter = new EventEmitter();
+    this.client = new lark.Client({
+      appId: options.appId,
+      appSecret: options.appSecret,
+      appType: options.appType === "store" ? lark.AppType.ISV : lark.AppType.SelfBuild,
+      domain: options.domain === "lark" ? lark.Domain.Lark : lark.Domain.Feishu,
+    });
+
+    this.initEventDispatcher();
+  }
+
+  /**
+   * 注册事件监听器
+   */
+  on(eventName: string, listener: (...args: any[]) => void): void {
+    this.eventEmitter.on(eventName, listener);
+  }
+
+  /**
+   * 移除事件监听器
+   */
+  off(eventName: string, listener: (...args: any[]) => void): void {
+    this.eventEmitter.off(eventName, listener);
+  }
+
+  private initEventDispatcher(): void {
+    this.eventDispatcher = new lark.EventDispatcher({
       encryptKey: "encrypt key",
     }).register<CardEvents>({
       [FEISHU_CARD_ACTION_TRIGGER]: async (data) => {
@@ -41,23 +66,10 @@ export class FeishuSdkService implements OnModuleInit, OnModuleDestroy {
     });
   }
 
-  onModuleDestroy(): void {
+  destroy(): void {
     this.eventEmitter.removeAllListeners();
   }
 
-  constructor(
-    @Inject(FEISHU_MODULE_OPTIONS)
-    protected readonly options: FeishuModuleOptions,
-    protected readonly eventEmitter: EventEmitter2,
-  ) {
-    this.client = new lark.Client({
-      appId: options.appId,
-      appSecret: options.appSecret,
-      appType: options.appType === "store" ? lark.AppType.ISV : lark.AppType.SelfBuild,
-      domain: options.domain === "lark" ? lark.Domain.Lark : lark.Domain.Feishu,
-    });
-  }
-
   /**
    * 验证飞书配置
    */

package/src/shared/feishu-sdk/fieshu-card.service.ts

@@ -1,4 +1,3 @@
-import { Injectable } from "@nestjs/common";
 import { FeishuSdkService } from "./feishu-sdk.service";
 import {
   SendCardParams,
@@ -8,25 +7,24 @@ import {
   CardContent,
   type CardActionTriggerCallback,
 } from "./types";
-import { OnEvent, type EventEmitter2 } from "@nestjs/event-emitter";
 import { FEISHU_CARD_ACTION_TRIGGER } from "./types";
 
 /**
  * 飞书卡片消息服务
  * 提供卡片消息的发送、回复、更新功能
  */
-@Injectable()
 export class FeishuCardService {
-  constructor(
-    protected readonly feishuSdkService: FeishuSdkService,
-    protected readonly eventEmitter: EventEmitter2,
-  ) {}
+  constructor(protected readonly feishuSdkService: FeishuSdkService) {
+    // 监听卡片动作触发事件
+    feishuSdkService.on(FEISHU_CARD_ACTION_TRIGGER, (event) => {
+      this.handleCardActionTrigger(event);
+    });
+  }
 
-  @OnEvent(FEISHU_CARD_ACTION_TRIGGER)
-  async handleCardActionTrigger(event: CardActionTriggerCallback) {
-    const event_hook = event.header.event_type ?? event.event.action.tag;
+  async handleCardActionTrigger(event: CardActionTriggerCallback): Promise<void> {
+    const _eventHook = event.header.event_type ?? event.event.action.tag;
 
-    // console.log(event_hook);
+    // console.log(_eventHook);
   }
 
   /**

package/src/shared/feishu-sdk/index.ts

@@ -1,4 +1,3 @@
-export * from "./feishu-sdk.module";
 export * from "./feishu-sdk.service";
 export * from "./fieshu-card.service";
 export * from "./types/index";

package/src/shared/git-provider/git-provider.service.ts

@@ -1,4 +1,3 @@
-import { Inject, Injectable } from "@nestjs/common";
 import type {
   GitProvider,
   LockBranchOptions,
@@ -6,7 +5,6 @@ import type {
 } from "./git-provider.interface";
 import {
   type GitProviderModuleOptions,
-  GIT_PROVIDER_MODULE_OPTIONS,
   type BranchProtection,
   type CreateBranchProtectionOption,
   type EditBranchProtectionOption,
@@ -36,13 +34,10 @@ import { GitlabAdapter } from "./adapters/gitlab.adapter";
  * Git Provider 统一服务
  * 根据配置的 provider 类型代理到对应的适配器实现
  */
-@Injectable()
 export class GitProviderService implements GitProvider {
   protected readonly adapter: GitProvider;
 
-  constructor(
-    @Inject(GIT_PROVIDER_MODULE_OPTIONS) protected readonly options: GitProviderModuleOptions,
-  ) {
+  constructor(protected readonly options: GitProviderModuleOptions) {
     this.adapter = this.createAdapter(options);
   }
 

package/src/shared/git-provider/index.ts

@@ -3,5 +3,4 @@ export * from "./detect-provider";
 export * from "./parse-repo-url";
 export * from "./git-provider.interface";
 export * from "./git-provider.service";
-export * from "./git-provider.module";
 export * from "./adapters";

package/src/shared/git-sdk/git-sdk.service.ts

@@ -1,9 +1,7 @@
-import { Injectable } from "@nestjs/common";
 import { spawn, execSync } from "child_process";
 import type { GitCommit, GitChangedFile, GitDiffFile, GitRunOptions } from "./git-sdk.types";
 import { mapGitStatus, parseDiffText } from "./git-sdk-diff.utils";
 
-@Injectable()
 export class GitSdkService {
   protected readonly defaultOptions: GitRunOptions = {
     cwd: process.cwd(),

package/src/shared/git-sdk/index.ts

@@ -1,4 +1,3 @@
 export * from "./git-sdk.types";
 export * from "./git-sdk-diff.utils";
 export { GitSdkService } from "./git-sdk.service";
-export { GitSdkModule } from "./git-sdk.module";

package/src/shared/llm-proxy/adapters/claude-code.adapter.spec.ts

@@ -1,18 +1,24 @@
-import { vi, type Mocked, type Mock } from "vitest";
-import { Test, TestingModule } from "@nestjs/testing";
+import { vi, type Mock } from "vitest";
 import { ClaudeCodeAdapter } from "./claude-code.adapter";
-import { ClaudeSetupService } from "../../claude-setup";
 import { LlmStreamEvent } from "../interfaces";
 
 vi.mock("@anthropic-ai/claude-agent-sdk", () => ({
   query: vi.fn(),
 }));
 
+vi.mock("../../claude-setup", () => ({
+  ClaudeSetupService: class MockClaudeSetupService {
+    configure = vi.fn().mockResolvedValue(undefined);
+    backup = vi.fn().mockResolvedValue(undefined);
+    restore = vi.fn().mockResolvedValue(undefined);
+    withTemporaryConfig = vi.fn();
+  },
+}));
+
 import { query } from "@anthropic-ai/claude-agent-sdk";
 
 describe("ClaudeAdapter", () => {
   let adapter: ClaudeCodeAdapter;
-  let claudeSetupService: Mocked<ClaudeSetupService>;
 
   const mockConfig = {
     claudeCode: {
@@ -21,24 +27,9 @@ describe("ClaudeAdapter", () => {
     },
   };
 
-  beforeEach(async () => {
-    const mockSetup = {
-      configure: vi.fn().mockResolvedValue(undefined),
-      backup: vi.fn().mockResolvedValue(undefined),
-      restore: vi.fn().mockResolvedValue(undefined),
-      withTemporaryConfig: vi.fn(),
-    };
-
-    const module: TestingModule = await Test.createTestingModule({
-      providers: [
-        ClaudeCodeAdapter,
-        { provide: "LLM_PROXY_CONFIG", useValue: mockConfig },
-        { provide: ClaudeSetupService, useValue: mockSetup },
-      ],
-    }).compile();
-
-    adapter = module.get<ClaudeCodeAdapter>(ClaudeCodeAdapter);
-    claudeSetupService = module.get(ClaudeSetupService);
+  beforeEach(() => {
+    vi.clearAllMocks();
+    adapter = new ClaudeCodeAdapter(mockConfig as any);
   });
 
   it("should be defined", () => {
@@ -51,15 +42,8 @@ describe("ClaudeAdapter", () => {
       expect(adapter.isConfigured()).toBe(true);
     });
 
-    it("should return false if claude config is missing", async () => {
-      const module: TestingModule = await Test.createTestingModule({
-        providers: [
-          ClaudeCodeAdapter,
-          { provide: "LLM_PROXY_CONFIG", useValue: {} },
-          { provide: ClaudeSetupService, useValue: { configure: vi.fn() } },
-        ],
-      }).compile();
-      const unconfiguredAdapter = module.get<ClaudeCodeAdapter>(ClaudeCodeAdapter);
+    it("should return false if claude config is missing", () => {
+      const unconfiguredAdapter = new ClaudeCodeAdapter({} as any);
       expect(unconfiguredAdapter.isConfigured()).toBe(false);
     });
   });
@@ -79,7 +63,6 @@ describe("ClaudeAdapter", () => {
         events.push(event);
       }
 
-      expect(claudeSetupService.configure).toHaveBeenCalled();
       expect(query).toHaveBeenCalledWith(
         expect.objectContaining({
           prompt: "hello",

package/src/shared/llm-proxy/adapters/claude-code.adapter.ts

@@ -1,4 +1,3 @@
-import { Injectable, Inject } from "@nestjs/common";
 import { query, type SpawnOptions } from "@anthropic-ai/claude-agent-sdk";
 import { spawn } from "child_process";
 import type { LlmAdapter } from "./llm-adapter.interface";
@@ -12,14 +11,14 @@ import type {
 import { ClaudeSetupService } from "../../claude-setup";
 import { shouldLog } from "../../verbose";
 
-@Injectable()
 export class ClaudeCodeAdapter implements LlmAdapter {
   readonly name = "claude-code";
+  private readonly claudeSetupService: ClaudeSetupService;
 
-  constructor(
-    @Inject("LLM_PROXY_CONFIG") private readonly config: LlmProxyConfig,
-    private readonly claudeSetupService: ClaudeSetupService,
-  ) {}
+  constructor(private readonly config: LlmProxyConfig) {
+    // 创建 ClaudeSetupService 实例，传入 LLM 配置
+    this.claudeSetupService = new ClaudeSetupService(config);
+  }
 
   isConfigured(): boolean {
     return !!this.config.claudeCode;

package/src/shared/llm-proxy/adapters/open-code.adapter.ts

@@ -1,4 +1,3 @@
-import { Injectable, Inject } from "@nestjs/common";
 import { createOpencode } from "@opencode-ai/sdk";
 import type { LlmAdapter } from "./llm-adapter.interface";
 import type {
@@ -11,11 +10,10 @@ import type {
 } from "../interfaces";
 import { shouldLog } from "../../verbose";
 
-@Injectable()
 export class OpenCodeAdapter implements LlmAdapter {
   readonly name = "open-code";
 
-  constructor(@Inject("LLM_PROXY_CONFIG") private readonly config: LlmProxyConfig) {}
+  constructor(private readonly config: LlmProxyConfig) {}
 
   isConfigured(): boolean {
     return !!this.config.openCode;

package/src/shared/llm-proxy/adapters/openai.adapter.spec.ts

@@ -1,5 +1,4 @@
 import { vi, type Mock } from "vitest";
-import { Test, TestingModule } from "@nestjs/testing";
 import { OpenAIAdapter } from "./openai.adapter";
 import OpenAI from "openai";
 
@@ -17,7 +16,8 @@ describe("OpenAIAdapter", () => {
     },
   };
 
-  beforeEach(async () => {
+  beforeEach(() => {
+    vi.clearAllMocks();
     mockOpenAIInstance = {
       chat: {
         completions: {
@@ -28,12 +28,7 @@ describe("OpenAIAdapter", () => {
     (OpenAI as unknown as Mock).mockImplementation(function () {
       return mockOpenAIInstance;
     });
-
-    const module: TestingModule = await Test.createTestingModule({
-      providers: [OpenAIAdapter, { provide: "LLM_PROXY_CONFIG", useValue: mockConfig }],
-    }).compile();
-
-    adapter = module.get<OpenAIAdapter>(OpenAIAdapter);
+    adapter = new OpenAIAdapter(mockConfig as any);
   });
 
   it("should be defined", () => {
@@ -46,11 +41,8 @@ describe("OpenAIAdapter", () => {
       expect(adapter.isConfigured()).toBe(true);
     });
 
-    it("should return false if openai config is missing", async () => {
-      const module: TestingModule = await Test.createTestingModule({
-        providers: [OpenAIAdapter, { provide: "LLM_PROXY_CONFIG", useValue: {} }],
-      }).compile();
-      const unconfiguredAdapter = module.get<OpenAIAdapter>(OpenAIAdapter);
+    it("should return false if openai config is missing", () => {
+      const unconfiguredAdapter = new OpenAIAdapter({} as any);
       expect(unconfiguredAdapter.isConfigured()).toBe(false);
     });
   });
@@ -156,10 +148,7 @@ describe("OpenAIAdapter", () => {
     });
 
     it("should yield error when openai not configured", async () => {
-      const module: TestingModule = await Test.createTestingModule({
-        providers: [OpenAIAdapter, { provide: "LLM_PROXY_CONFIG", useValue: {} }],
-      }).compile();
-      const unconfigured = module.get<OpenAIAdapter>(OpenAIAdapter);
+      const unconfigured = new OpenAIAdapter({} as any);
       const events: any[] = [];
       for await (const event of unconfigured.chatStream([{ role: "user", content: "hi" }] as any)) {
         events.push(event);
@@ -176,10 +165,7 @@ describe("OpenAIAdapter", () => {
     });
 
     it("should throw when openai not configured for chat", async () => {
-      const module: TestingModule = await Test.createTestingModule({
-        providers: [OpenAIAdapter, { provide: "LLM_PROXY_CONFIG", useValue: {} }],
-      }).compile();
-      const unconfigured = module.get<OpenAIAdapter>(OpenAIAdapter);
+      const unconfigured = new OpenAIAdapter({} as any);
       await expect(unconfigured.chat([{ role: "user", content: "hi" }] as any)).rejects.toThrow(
         "未配置 openai",
       );

package/src/shared/llm-proxy/adapters/openai.adapter.ts

@@ -1,4 +1,3 @@
-import { Injectable, Inject } from "@nestjs/common";
 import OpenAI from "openai";
 import type { LlmAdapter } from "./llm-adapter.interface";
 import type {
@@ -10,13 +9,12 @@ import type {
 } from "../interfaces";
 import { shouldLog } from "../../verbose";
 
-@Injectable()
 export class OpenAIAdapter implements LlmAdapter {
   readonly name = "openai";
 
   private client: OpenAI | null = null;
 
-  constructor(@Inject("LLM_PROXY_CONFIG") private readonly config: LlmProxyConfig) {}
+  constructor(private readonly config: LlmProxyConfig) {}
 
   isConfigured(): boolean {
     return !!this.config.openai;

package/src/shared/llm-proxy/index.ts

@@ -2,5 +2,4 @@ export * from "./interfaces";
 export * from "./adapters";
 export * from "./llm-session";
 export * from "./llm-proxy.service";
-export * from "./llm-proxy.module";
 export * from "./stream-logger";

package/src/shared/llm-proxy/interfaces/config.interface.ts

@@ -2,6 +2,7 @@ export interface ClaudeAdapterConfig {
   model?: string;
   baseUrl?: string;
   authToken?: string;
+  hasCompletedOnboarding?: boolean;
 }
 
 export interface OpenAIAdapterConfig {
@@ -22,11 +23,18 @@ export interface OpenCodeAdapterConfig {
 
 export type LLMMode = "claude-code" | "openai" | "gemini" | "open-code";
 
+export interface GeminiAdapterConfig {
+  model?: string;
+  baseUrl?: string;
+  apiKey?: string;
+}
+
 export interface LlmProxyConfig {
   defaultAdapter?: LLMMode;
   claudeCode?: ClaudeAdapterConfig;
   openai?: OpenAIAdapterConfig;
   openCode?: OpenCodeAdapterConfig;
+  gemini?: GeminiAdapterConfig;
 }
 
 export const LLM_PROXY_CONFIG = Symbol("LLM_PROXY_CONFIG");