plugin-git-manager 1.0.3 → 1.0.10

package/dist/server/plugin.js

@@ -43,6 +43,11 @@ module.exports = __toCommonJS(plugin_exports);
 var import_server = require("@nocobase/server");
 var import_path = require("path");
 var gitActions = __toESM(require("./actions/git-actions"));
+var gitlabApi = __toESM(require("./actions/gitlab-api"));
+var reviewActions = __toESM(require("./actions/review"));
+var pollerActions = __toESM(require("./actions/poller"));
+var import_ai_tools = require("./ai-tools");
+var import_poller = require("./poller");
 class PluginGitManagerServer extends import_server.Plugin {
   async load() {
     await this.db.import({
@@ -62,21 +67,47 @@ class PluginGitManagerServer extends import_server.Plugin {
         checkout: gitActions.checkout,
         fileTree: gitActions.fileTree,
         fileContent: gitActions.fileContent,
-        commitDetail: gitActions.commitDetail
+        commitDetail: gitActions.commitDetail,
+        mergeRequests: gitlabApi.mergeRequests,
+        mergeRequestDetail: gitlabApi.mergeRequestDetail,
+        mergeRequestNotes: gitlabApi.mergeRequestNotes,
+        triggerReview: reviewActions.triggerReview,
+        reviewApprovePost: reviewActions.reviewApprovePost,
+        reviewReject: reviewActions.reviewReject,
+        pollNow: pollerActions.pollNow,
+        pollerStatus: pollerActions.pollerStatus
       }
     });
+    (0, import_ai_tools.registerGitReviewAiTools)(this.app);
+    this.app.on("afterStart", () => {
+      (0, import_poller.startPoller)(this.app);
+    });
+    this.app.on("beforeStop", () => {
+      (0, import_poller.stopPoller)();
+    });
+    this.app.on("beforeDestroy", () => {
+      (0, import_poller.stopPoller)();
+    });
     this.app.acl.registerSnippet({
       name: `pm.${this.name}.read`,
       actions: [
         "gitRepositories:list",
         "gitRepositories:get",
+        "gitReviewFlows:list",
+        "gitReviewFlows:get",
+        "gitCodeReviews:list",
+        "gitCodeReviews:get",
         "gitManager:status",
         "gitManager:log",
         "gitManager:branches",
         "gitManager:diff",
         "gitManager:fileTree",
         "gitManager:fileContent",
-        "gitManager:commitDetail"
+        "gitManager:commitDetail",
+        "gitManager:mergeRequests",
+        "gitManager:mergeRequestDetail",
+        "gitManager:mergeRequestNotes",
+        "gitManager:pollerStatus"
       ]
     });
     this.app.acl.registerSnippet({
@@ -85,13 +116,35 @@ class PluginGitManagerServer extends import_server.Plugin {
         "gitRepositories:create",
         "gitRepositories:update",
         "gitRepositories:destroy",
+        "gitReviewFlows:create",
+        "gitReviewFlows:update",
+        "gitReviewFlows:destroy",
+        "gitCodeReviews:create",
+        "gitCodeReviews:update",
+        "gitCodeReviews:destroy",
         "gitManager:clone",
         "gitManager:pull",
         "gitManager:push",
         "gitManager:fetch",
-        "gitManager:checkout"
+        "gitManager:checkout",
+        "gitManager:triggerReview",
+        "gitManager:reviewApprovePost",
+        "gitManager:reviewReject",
+        "gitManager:pollNow"
       ]
     });
+    this.app.resourceManager.use(async (ctx, next) => {
+      var _a, _b, _c, _d, _e;
+      if (((_a = ctx.action) == null ? void 0 : _a.resourceName) === "gitRepositories" && ["create", "update"].includes((_b = ctx.action) == null ? void 0 : _b.actionName)) {
+        if (((_d = (_c = ctx.action.params) == null ? void 0 : _c.values) == null ? void 0 : _d.pat) === "\u2022\u2022\u2022\u2022\u2022\u2022\u2022\u2022") {
+          delete ctx.action.params.values.pat;
+        }
+        if (((_e = ctx.request.body) == null ? void 0 : _e.pat) === "\u2022\u2022\u2022\u2022\u2022\u2022\u2022\u2022") {
+          delete ctx.request.body.pat;
+        }
+      }
+      return next();
+    });
     this.app.resourceManager.use(async (ctx, next) => {
       var _a, _b;
       if (((_a = ctx.action) == null ? void 0 : _a.resourceName) !== "gitRepositories") {
@@ -112,6 +165,12 @@ class PluginGitManagerServer extends import_server.Plugin {
   }
   async install() {
   }
+  async beforeDisable() {
+    (0, import_poller.stopPoller)();
+  }
+  async beforeUnload() {
+    (0, import_poller.stopPoller)();
+  }
 }
 var plugin_default = PluginGitManagerServer;
 // Annotate the CommonJS export names for ESM import in node:

package/dist/server/poller.d.ts

@@ -0,0 +1,30 @@
+import type { Application } from '@nocobase/server';
+/**
+ * Start the background poller. Idempotent — calling twice is a no-op.
+ * The poller scans repos with autoReview=true, lists their open MRs from GitLab,
+ * and triggers a review if no record exists for the MR's current head SHA.
+ */
+export declare function startPoller(app: Application): void;
+export declare function stopPoller(): void;
+export declare function getPollerStatus(): {
+    running: boolean;
+    polling: boolean;
+    lastTickAt: string;
+    lastError: string;
+    intervalSec: number;
+};
+/**
+ * Run one poll cycle for all auto-enabled repos. Public so that the
+ * gitManager:pollNow action can force a tick on demand.
+ */
+export declare function pollAllRepos(app: Application): Promise<{
+    scanned: number;
+    triggered: number;
+}>;
+/**
+ * Poll a single repo for MR changes. Returns counts.
+ */
+export declare function pollOneRepo(app: Application, repo: any): Promise<{
+    scanned: number;
+    triggered: number;
+}>;

package/dist/server/poller.js

@@ -0,0 +1,241 @@
+/**
+ * This file is part of the NocoBase (R) project.
+ * Copyright (c) 2020-2024 NocoBase Co., Ltd.
+ * Authors: NocoBase Team.
+ *
+ * This project is dual-licensed under AGPL-3.0 and NocoBase Commercial License.
+ * For more information, please refer to: https://www.nocobase.com/agreement.
+ */
+
+var __defProp = Object.defineProperty;
+var __getOwnPropDesc = Object.getOwnPropertyDescriptor;
+var __getOwnPropNames = Object.getOwnPropertyNames;
+var __hasOwnProp = Object.prototype.hasOwnProperty;
+var __export = (target, all) => {
+  for (var name in all)
+    __defProp(target, name, { get: all[name], enumerable: true });
+};
+var __copyProps = (to, from, except, desc) => {
+  if (from && typeof from === "object" || typeof from === "function") {
+    for (let key of __getOwnPropNames(from))
+      if (!__hasOwnProp.call(to, key) && key !== except)
+        __defProp(to, key, { get: () => from[key], enumerable: !(desc = __getOwnPropDesc(from, key)) || desc.enumerable });
+  }
+  return to;
+};
+var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: true }), mod);
+var poller_exports = {};
+__export(poller_exports, {
+  getPollerStatus: () => getPollerStatus,
+  pollAllRepos: () => pollAllRepos,
+  pollOneRepo: () => pollOneRepo,
+  startPoller: () => startPoller,
+  stopPoller: () => stopPoller
+});
+module.exports = __toCommonJS(poller_exports);
+var import_review = require("./actions/review");
+var import_gitlab_url = require("./utils/gitlab-url");
+const POLL_INTERVAL_MS = 5 * 60 * 1e3;
+const MR_PAGE_SIZE = 50;
+const STALE_POLL_TIMEOUT_MS = 10 * 60 * 1e3;
+let timer = null;
+let isPolling = false;
+let pollStartedAt = null;
+let lastTickAt = null;
+let lastError = null;
+function startPoller(app) {
+  var _a, _b;
+  if (timer) return;
+  (_b = (_a = app.log) == null ? void 0 : _a.info) == null ? void 0 : _b.call(_a, "plugin-git-manager: starting MR poller (interval 5min)");
+  timer = setInterval(() => tick(app).catch((e) => {
+    var _a2, _b2;
+    return (_b2 = (_a2 = app.log) == null ? void 0 : _a2.error) == null ? void 0 : _b2.call(_a2, "poller tick error", e);
+  }), POLL_INTERVAL_MS);
+}
+function stopPoller() {
+  if (timer) {
+    clearInterval(timer);
+    timer = null;
+  }
+}
+function getPollerStatus() {
+  return {
+    running: !!timer,
+    polling: isPolling,
+    lastTickAt: (lastTickAt == null ? void 0 : lastTickAt.toISOString()) ?? null,
+    lastError,
+    intervalSec: POLL_INTERVAL_MS / 1e3
+  };
+}
+async function tryAcquirePollerLock(app) {
+  var _a, _b, _c;
+  try {
+    const sequelize = app.db.sequelize;
+    if (!sequelize) return true;
+    const dialect = (_a = sequelize.getDialect) == null ? void 0 : _a.call(sequelize);
+    const lockKey = 777042;
+    if (dialect === "postgres") {
+      const [results] = await sequelize.query(`SELECT pg_try_advisory_lock(${lockKey}) AS locked`);
+      return ((_b = results == null ? void 0 : results[0]) == null ? void 0 : _b.locked) === true;
+    }
+    if (dialect === "mysql" || dialect === "mariadb") {
+      const [results] = await sequelize.query(`SELECT GET_LOCK('git_poller', 0) AS locked`);
+      return ((_c = results == null ? void 0 : results[0]) == null ? void 0 : _c.locked) === 1;
+    }
+    return true;
+  } catch {
+    return true;
+  }
+}
+async function releasePollerLock(app) {
+  var _a;
+  try {
+    const sequelize = app.db.sequelize;
+    if (!sequelize) return;
+    const dialect = (_a = sequelize.getDialect) == null ? void 0 : _a.call(sequelize);
+    const lockKey = 777042;
+    if (dialect === "postgres") {
+      await sequelize.query(`SELECT pg_advisory_unlock(${lockKey})`);
+    } else if (dialect === "mysql" || dialect === "mariadb") {
+      await sequelize.query(`SELECT RELEASE_LOCK('git_poller')`);
+    }
+  } catch {
+  }
+}
+async function pollAllRepos(app) {
+  var _a, _b, _c, _d, _e, _f, _g, _h;
+  if (isPolling) {
+    if (pollStartedAt && Date.now() - pollStartedAt > STALE_POLL_TIMEOUT_MS) {
+      (_b = (_a = app.log) == null ? void 0 : _a.warn) == null ? void 0 : _b.call(_a, "poller: stale isPolling flag detected \u2014 forcibly resetting");
+      isPolling = false;
+      pollStartedAt = null;
+    } else {
+      return { scanned: 0, triggered: 0 };
+    }
+  }
+  const lockAcquired = await tryAcquirePollerLock(app);
+  if (!lockAcquired) {
+    (_d = (_c = app.log) == null ? void 0 : _c.debug) == null ? void 0 : _d.call(_c, "poller: another node holds the advisory lock \u2014 skipping");
+    return { scanned: 0, triggered: 0 };
+  }
+  isPolling = true;
+  pollStartedAt = Date.now();
+  lastTickAt = /* @__PURE__ */ new Date();
+  lastError = null;
+  let scanned = 0;
+  let triggered = 0;
+  try {
+    const reposRepo = app.db.getRepository("gitRepositories");
+    const repos = await reposRepo.find({ filter: { autoReview: true } });
+    for (const repo of repos) {
+      try {
+        const result = await pollOneRepo(app, repo);
+        scanned += result.scanned;
+        triggered += result.triggered;
+      } catch (err) {
+        (_f = (_e = app.log) == null ? void 0 : _e.error) == null ? void 0 : _f.call(_e, `poller: repo ${repo.get("id")} failed: ${err == null ? void 0 : err.message}`);
+      }
+    }
+  } catch (err) {
+    lastError = (err == null ? void 0 : err.message) || String(err);
+    (_h = (_g = app.log) == null ? void 0 : _g.error) == null ? void 0 : _h.call(_g, "poller: cycle failed", err);
+  } finally {
+    isPolling = false;
+    pollStartedAt = null;
+    await releasePollerLock(app);
+  }
+  return { scanned, triggered };
+}
+async function tick(app) {
+  await pollAllRepos(app);
+}
+async function pollOneRepo(app, repo) {
+  var _a, _b;
+  const flowsRepo = app.db.getRepository("gitReviewFlows");
+  const reviewsRepo = app.db.getRepository("gitCodeReviews");
+  const reposRepo = app.db.getRepository("gitRepositories");
+  const flows = await flowsRepo.find({
+    filter: {
+      enabled: true,
+      triggerMode: { $in: ["onMergeRequestCreated", "both"] },
+      $or: [{ repositoryId: repo.get("id") }, { repositoryId: null }]
+    },
+    sort: ["-repositoryId"]
+  });
+  if (!(flows == null ? void 0 : flows.length)) return { scanned: 0, triggered: 0 };
+  const pat = repo.get("pat");
+  if (!pat) return { scanned: 0, triggered: 0 };
+  const lastPolledAt = repo.get("lastPolledAt");
+  const mrs = await listMergeRequests(repo, lastPolledAt);
+  let triggered = 0;
+  for (const mr of mrs) {
+    const flow = flows.find((f) => (0, import_review.branchMatches)(f, mr.source_branch));
+    if (!flow) continue;
+    const existing = await reviewsRepo.findOne({
+      filter: {
+        repositoryId: repo.get("id"),
+        targetType: "mr",
+        mrIid: mr.iid
+      }
+    });
+    if (existing) {
+      if (existing.get("latestSha") !== mr.sha) {
+        await reviewsRepo.update({
+          filterByTk: existing.get("id"),
+          values: { latestSha: mr.sha }
+        });
+      }
+      continue;
+    }
+    try {
+      await (0, import_review.triggerReviewInternal)(app, {
+        flowId: flow.get("id"),
+        repositoryId: repo.get("id"),
+        targetType: "mr",
+        mrIid: mr.iid,
+        branch: mr.source_branch,
+        headSha: mr.sha,
+        triggeredBy: "poll"
+      });
+      triggered++;
+    } catch (err) {
+      (_b = (_a = app.log) == null ? void 0 : _a.error) == null ? void 0 : _b.call(_a, `poller: triggerReviewInternal failed for MR !${mr.iid}: ${err == null ? void 0 : err.message}`);
+    }
+  }
+  await reposRepo.update({
+    filterByTk: repo.get("id"),
+    values: { lastPolledAt: /* @__PURE__ */ new Date() }
+  });
+  return { scanned: mrs.length, triggered };
+}
+async function listMergeRequests(repo, updatedAfter) {
+  const repoUrl = repo.get("repoUrl");
+  const pat = repo.get("pat");
+  const { apiBase, encodedProject } = (0, import_gitlab_url.parseGitLabProject)(repoUrl);
+  const params = new URLSearchParams({
+    state: "opened",
+    per_page: String(MR_PAGE_SIZE),
+    order_by: "updated_at",
+    sort: "desc"
+  });
+  if (updatedAfter) {
+    const safeBound = new Date(updatedAfter.getTime() - 1e3);
+    params.set("updated_after", safeBound.toISOString());
+  }
+  const response = await fetch(`${apiBase}/projects/${encodedProject}/merge_requests?${params.toString()}`, {
+    headers: { "PRIVATE-TOKEN": pat, Accept: "application/json" }
+  });
+  if (!response.ok) {
+    const body = await response.text().catch(() => "");
+    throw new Error(`GitLab API error ${response.status}: ${body}`);
+  }
+  return response.json();
+}
+// Annotate the CommonJS export names for ESM import in node:
+0 && (module.exports = {
+  getPollerStatus,
+  pollAllRepos,
+  pollOneRepo,
+  startPoller,
+  stopPoller
+});

package/dist/server/utils/gitlab-url.d.ts

@@ -0,0 +1,21 @@
+/**
+ * Shared utility for parsing GitLab repository URLs.
+ * Extracted to avoid duplicating URL-parsing logic across
+ * gitlab-api.ts, poller.ts, and review.ts.
+ */
+export interface GitLabProject {
+    /** Full API base URL, e.g. https://gitlab.com/api/v4 */
+    apiBase: string;
+    /** Raw project path, e.g. group/sub/project */
+    projectPath: string;
+    /** URL-encoded project path for use in API endpoints */
+    encodedProject: string;
+}
+/**
+ * Extract GitLab API base URL and project path from a repository URL.
+ * Supports: https://gitlab.com/group/project.git
+ *           https://self-hosted.gitlab.com/group/sub/project.git
+ *
+ * @throws Error if the URL cannot be parsed or has no project path
+ */
+export declare function parseGitLabProject(repoUrl: string): GitLabProject;

package/dist/server/utils/gitlab-url.js

@@ -0,0 +1,47 @@
+/**
+ * This file is part of the NocoBase (R) project.
+ * Copyright (c) 2020-2024 NocoBase Co., Ltd.
+ * Authors: NocoBase Team.
+ *
+ * This project is dual-licensed under AGPL-3.0 and NocoBase Commercial License.
+ * For more information, please refer to: https://www.nocobase.com/agreement.
+ */
+
+var __defProp = Object.defineProperty;
+var __getOwnPropDesc = Object.getOwnPropertyDescriptor;
+var __getOwnPropNames = Object.getOwnPropertyNames;
+var __hasOwnProp = Object.prototype.hasOwnProperty;
+var __export = (target, all) => {
+  for (var name in all)
+    __defProp(target, name, { get: all[name], enumerable: true });
+};
+var __copyProps = (to, from, except, desc) => {
+  if (from && typeof from === "object" || typeof from === "function") {
+    for (let key of __getOwnPropNames(from))
+      if (!__hasOwnProp.call(to, key) && key !== except)
+        __defProp(to, key, { get: () => from[key], enumerable: !(desc = __getOwnPropDesc(from, key)) || desc.enumerable });
+  }
+  return to;
+};
+var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: true }), mod);
+var gitlab_url_exports = {};
+__export(gitlab_url_exports, {
+  parseGitLabProject: () => parseGitLabProject
+});
+module.exports = __toCommonJS(gitlab_url_exports);
+function parseGitLabProject(repoUrl) {
+  const url = new URL(repoUrl);
+  const projectPath = url.pathname.replace(/\.git$/, "").replace(/^\//, "");
+  if (!projectPath) {
+    throw new Error("Cannot extract project path from repository URL");
+  }
+  return {
+    apiBase: `${url.protocol}//${url.host}/api/v4`,
+    projectPath,
+    encodedProject: encodeURIComponent(projectPath)
+  };
+}
+// Annotate the CommonJS export names for ESM import in node:
+0 && (module.exports = {
+  parseGitLabProject
+});

package/package.json

@@ -3,7 +3,7 @@
   "displayName": "Git Manager",
   "displayName.zh-CN": "Git 管理器",
   "description": "Manage Git repositories with PAT authentication - pull, push, fetch, diff, file browsing",
-  "version": "1.0.3",
+  "version": "1.0.10",
   "license": "Apache-2.0",
   "main": "dist/server/index.js",
   "files": [
@@ -26,6 +26,11 @@
     "@nocobase/database": "2.x",
     "@nocobase/test": "2.x"
   },
+  "optionalPeerDependencies": {
+    "@nocobase/ai": "2.x",
+    "@nocobase/plugin-ai": "2.x",
+    "@nocobase/plugin-field-markdown-vditor": "2.x"
+  },
   "dependencies": {
     "simple-git": "^3.27.0"
   }

package/src/client/ai-context.tsx

@@ -0,0 +1,114 @@
+import React from 'react';
+import { Space } from 'antd';
+import { BranchesOutlined, MergeOutlined, NodeIndexOutlined } from '@ant-design/icons';
+
+/**
+ * WorkContext options exported as plain objects so we can register them
+ * regardless of whether plugin-ai is installed at compile time. The plugin
+ * checks `app.aiManager` at runtime before registering.
+ */
+
+export const GitRepositoryWorkContext: any = {
+  name: 'git-repository',
+  menu: {
+    icon: <BranchesOutlined />,
+    Component: () => <span>Git Repository</span>,
+  },
+  tag: {
+    Component: ({ item }: any) => (
+      <Space size={4}>
+        <BranchesOutlined />
+        <span>{item?.title || `Repo #${item?.uid}`}</span>
+      </Space>
+    ),
+  },
+  getContent: async (app: any, { uid }: any) => {
+    const response = await app.apiClient.request({
+      url: 'gitRepositories:get',
+      params: { filterByTk: uid },
+    });
+    const repo = response?.data?.data;
+    if (!repo) return null;
+    return {
+      type: 'git-repository',
+      repositoryId: repo.id,
+      name: repo.name,
+      repoUrl: repo.repoUrl,
+      defaultBranch: repo.defaultBranch,
+      status: repo.status,
+    };
+  },
+};
+
+export const GitMergeRequestWorkContext: any = {
+  name: 'git-merge-request',
+  menu: {
+    icon: <MergeOutlined />,
+    Component: () => <span>Git Merge Request</span>,
+  },
+  tag: {
+    Component: ({ item }: any) => (
+      <Space size={4}>
+        <MergeOutlined />
+        <span>{item?.title || `MR ${item?.uid}`}</span>
+      </Space>
+    ),
+  },
+  getContent: async (app: any, { uid }: any) => {
+    // uid format: "<repositoryId>:<mrIid>"
+    const [repoIdStr, mrIidStr] = String(uid || '').split(':');
+    const repositoryId = Number(repoIdStr);
+    const mrIid = Number(mrIidStr);
+    if (!repositoryId || !mrIid) return null;
+    const response = await app.apiClient.request({
+      url: 'gitManager:mergeRequestDetail',
+      params: { repositoryId, mrIid },
+    });
+    const data = response?.data?.data;
+    return {
+      type: 'git-merge-request',
+      repositoryId,
+      mrIid,
+      title: data?.title,
+      sourceBranch: data?.sourceBranch,
+      targetBranch: data?.targetBranch,
+      author: data?.author,
+      changes: data?.changes,
+      description: data?.description,
+      webUrl: data?.webUrl,
+    };
+  },
+};
+
+export const GitCommitWorkContext: any = {
+  name: 'git-commit',
+  menu: {
+    icon: <NodeIndexOutlined />,
+    Component: () => <span>Git Commit</span>,
+  },
+  tag: {
+    Component: ({ item }: any) => (
+      <Space size={4}>
+        <NodeIndexOutlined />
+        <span>{item?.title || `Commit ${item?.uid}`}</span>
+      </Space>
+    ),
+  },
+  getContent: async (app: any, { uid }: any) => {
+    // uid format: "<repositoryId>:<commitSha>"
+    const [repoIdStr, commitSha] = String(uid || '').split(':');
+    const repositoryId = Number(repoIdStr);
+    if (!repositoryId || !commitSha) return null;
+    const response = await app.apiClient.request({
+      url: 'gitManager:commitDetail',
+      params: { repositoryId, commitHash: commitSha },
+    });
+    const data = response?.data?.data;
+    return {
+      type: 'git-commit',
+      repositoryId,
+      commitSha,
+      ...data,
+    };
+  },
+};

package/src/client/components/AIEmployeeSelect.tsx

@@ -0,0 +1,63 @@
+import React, { useEffect, useState } from 'react';
+import { Select } from 'antd';
+import { useAPIClient } from '@nocobase/client';
+
+interface AIEmployeeOption {
+  username: string;
+  nickname?: string;
+  avatar?: string;
+}
+
+/**
+ * Select dropdown listing AI employees the current user has access to.
+ * Calls `aiEmployees:listByUser` from plugin-ai.
+ */
+export const AIEmployeeSelect: React.FC<{
+  value?: string;
+  onChange?: (value: string) => void;
+  allowClear?: boolean;
+  size?: 'small' | 'middle' | 'large';
+  style?: React.CSSProperties;
+  placeholder?: string;
+}> = ({ value, onChange, allowClear = true, size, style, placeholder }) => {
+  const api = useAPIClient();
+  const [employees, setEmployees] = useState<AIEmployeeOption[]>([]);
+  const [loading, setLoading] = useState(false);
+
+  useEffect(() => {
+    let cancelled = false;
+    setLoading(true);
+    api
+      .request({ url: 'aiEmployees:listByUser' })
+      .then((res) => {
+        if (cancelled) return;
+        const list = res?.data?.data || [];
+        setEmployees(Array.isArray(list) ? list : []);
+      })
+      .catch(() => undefined)
+      .finally(() => {
+        if (!cancelled) setLoading(false);
+      });
+    return () => {
+      cancelled = true;
+    };
+  }, [api]);
+
+  return (
+    <Select
+      value={value}
+      onChange={onChange}
+      allowClear={allowClear}
+      loading={loading}
+      size={size}
+      style={style}
+      placeholder={placeholder || 'Select AI employee'}
+      options={employees.map((e) => ({
+        value: e.username,
+        label: `${e.nickname || e.username} (@${e.username})`,
+      }))}
+      showSearch
+      optionFilterProp="label"
+    />
+  );
+};

package/src/client/components/CommitHistory.tsx

@@ -12,6 +12,7 @@ import {
 } from '@ant-design/icons';
 import { useAPIClient } from '@nocobase/client';
 import { useGitManager } from '../context/GitManagerContext';
+import { RunReviewButton } from './RunReviewButton';
 import { useT } from '../locale';
 
 const { Text } = Typography;
@@ -210,6 +211,19 @@ export const CommitHistory: React.FC = () => {
           setDiffFile(null);
         }}
         width={800}
+        extra={
+          selectedCommit && selectedRepo ? (
+            <RunReviewButton
+              target={{
+                type: 'commit',
+                repositoryId: selectedRepo.id,
+                commitSha: selectedCommit.hash,
+                title: selectedCommit.message,
+              }}
+              size="small"
+            />
+          ) : null
+        }
       >
         {detailLoading ? (
           <Spin />