@spaceflow/core 0.1.3 → 0.3.0

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

@@ -19,6 +19,7 @@ import type {
   EditPullRequestOption,
   User,
   RepositoryContent,
+  ResolvedThread,
 } from "./types";
 
 /** PR 列表查询选项 */
@@ -164,7 +165,15 @@ export interface GitProvider {
   ): Promise<PullReview>;
   /** 列出 PR 的所有 Reviews */
   listPullReviews(owner: string, repo: string, index: number): Promise<PullReview[]>;
-  /** 删除 PR Review */
+  /** 更新 PR Review 的内容 */
+  updatePullReview(
+    owner: string,
+    repo: string,
+    index: number,
+    reviewId: number,
+    body: string,
+  ): Promise<PullReview>;
+  /** 删除 PR Review（仅支持 PENDING 状态） */
   deletePullReview(owner: string, repo: string, index: number, reviewId: number): Promise<void>;
   /** 获取 PR Review 的行级评论列表 */
   listPullReviewComments(
@@ -173,10 +182,20 @@ export interface GitProvider {
     index: number,
     reviewId: number,
   ): Promise<PullReviewComment[]>;
+  /** 删除 PR Review 的单条行级评论 */
+  deletePullReviewComment(owner: string, repo: string, commentId: number): Promise<void>;
+  /** 获取 PR 中所有已解决的 review threads（含文件路径和行号） */
+  listResolvedThreads(owner: string, repo: string, index: number): Promise<ResolvedThread[]>;
 
   // ============ Reaction 操作 ============
   /** 获取 Issue/PR 评论的 reactions */
   getIssueCommentReactions(owner: string, repo: string, commentId: number): Promise<Reaction[]>;
+  /** 获取 PR Review 行级评论的 reactions */
+  getPullReviewCommentReactions(
+    owner: string,
+    repo: string,
+    commentId: number,
+  ): Promise<Reaction[]>;
   /** 获取 Issue/PR 的 reactions */
   getIssueReactions(owner: string, repo: string, index: number): Promise<Reaction[]>;
 

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,
@@ -27,6 +25,7 @@ import {
   type EditPullRequestOption,
   type User,
   type RepositoryContent,
+  type ResolvedThread,
 } from "./types";
 import { GiteaAdapter } from "./adapters/gitea.adapter";
 import { GithubAdapter } from "./adapters/github.adapter";
@@ -36,13 +35,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);
   }
 
@@ -265,6 +261,16 @@ export class GitProviderService implements GitProvider {
     return this.adapter.listPullReviews(owner, repo, index);
   }
 
+  async updatePullReview(
+    owner: string,
+    repo: string,
+    index: number,
+    reviewId: number,
+    body: string,
+  ): Promise<PullReview> {
+    return this.adapter.updatePullReview(owner, repo, index, reviewId, body);
+  }
+
   async deletePullReview(
     owner: string,
     repo: string,
@@ -283,6 +289,14 @@ export class GitProviderService implements GitProvider {
     return this.adapter.listPullReviewComments(owner, repo, index, reviewId);
   }
 
+  async deletePullReviewComment(owner: string, repo: string, commentId: number): Promise<void> {
+    return this.adapter.deletePullReviewComment(owner, repo, commentId);
+  }
+
+  async listResolvedThreads(owner: string, repo: string, index: number): Promise<ResolvedThread[]> {
+    return this.adapter.listResolvedThreads(owner, repo, index);
+  }
+
   // ============ Reaction 操作 ============
 
   async getIssueCommentReactions(
@@ -293,6 +307,14 @@ export class GitProviderService implements GitProvider {
     return this.adapter.getIssueCommentReactions(owner, repo, commentId);
   }
 
+  async getPullReviewCommentReactions(
+    owner: string,
+    repo: string,
+    commentId: number,
+  ): Promise<Reaction[]> {
+    return this.adapter.getPullReviewCommentReactions(owner, repo, commentId);
+  }
+
   async getIssueReactions(owner: string, repo: string, index: number): Promise<Reaction[]> {
     return this.adapter.getIssueReactions(owner, repo, index);
   }

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-provider/types.ts

@@ -318,6 +318,20 @@ export interface Issue {
 /** PR Review 事件类型 */
 export type ReviewStateType = "APPROVE" | "REQUEST_CHANGES" | "COMMENT" | "PENDING";
 
+/** PR Review 事件常量 */
+export const REVIEW_STATE = {
+  APPROVE: "APPROVE",
+  REQUEST_CHANGES: "REQUEST_CHANGES",
+  COMMENT: "COMMENT",
+  PENDING: "PENDING",
+} as const;
+
+/** PR Review 行级评论 Diff 方向 */
+export const DIFF_SIDE = {
+  RIGHT: "RIGHT",
+  LEFT: "LEFT",
+} as const;
+
 /** PR Review 行级评论 */
 export interface CreatePullReviewComment {
   /** 文件路径 */
@@ -381,6 +395,19 @@ export interface PullReviewComment {
   html_url?: string;
 }
 
+/** PR Review Thread 已解决信息 */
+export interface ResolvedThread {
+  /** 文件路径 */
+  path?: string;
+  /** 行号（新文件中的行号） */
+  line?: number;
+  /** 解决者 */
+  resolvedBy?: {
+    id?: number;
+    login?: string;
+  } | null;
+}
+
 /** 用户信息 */
 export interface User {
   id?: number;

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");