plugin-git-manager 1.1.9 → 1.1.12

package/dist/server/actions/gitlab-api.js

@@ -32,6 +32,10 @@ __export(gitlab_api_exports, {
 });
 module.exports = __toCommonJS(gitlab_api_exports);
 var import_gitlab_url = require("../utils/gitlab-url");
+function getActionParams(ctx) {
+  var _a, _b;
+  return { ...ctx.action.params, ...(_a = ctx.action.params) == null ? void 0 : _a.values, ...((_b = ctx.request) == null ? void 0 : _b.body) || {} };
+}
 async function gitlabFetch(apiBase, endpoint, pat, params) {
   const url = new URL(`${apiBase}${endpoint}`);
   if (params) {
@@ -57,8 +61,7 @@ async function gitlabFetch(apiBase, endpoint, pat, params) {
   return { data, totalPages: totalPages ? parseInt(totalPages, 10) : null, total: total ? parseInt(total, 10) : null };
 }
 async function getRepoApiContext(ctx) {
-  var _a;
-  const params = { ...ctx.action.params, ...(_a = ctx.action.params) == null ? void 0 : _a.values, ...ctx.request.body || {} };
+  const params = getActionParams(ctx);
   const { repositoryId } = params;
   const repo = await ctx.db.getRepository("gitRepositories").findOne({
     filterByTk: repositoryId
@@ -105,9 +108,8 @@ async function githubFetch(endpoint, pat, params) {
   return { data, totalPages, total: null };
 }
 async function mergeRequests(ctx, next) {
-  var _a;
   const { pat, apiBase, encodedProject, projectPath, isGitHub } = await getRepoApiContext(ctx);
-  const params = { ...ctx.action.params, ...(_a = ctx.action.params) == null ? void 0 : _a.values, ...ctx.request.body || {} };
+  const params = getActionParams(ctx);
   const {
     state = "opened",
     search,
@@ -138,14 +140,14 @@ async function mergeRequests(ctx, next) {
       result.total = null;
     }
     items = pullRequests.map((pr) => {
-      var _a2, _b;
+      var _a, _b;
       return {
         id: pr.id,
         iid: pr.number,
         title: pr.title,
         description: pr.body,
         state: pr.state === "open" ? "opened" : pr.merged_at ? "merged" : "closed",
-        sourceBranch: (_a2 = pr.head) == null ? void 0 : _a2.ref,
+        sourceBranch: (_a = pr.head) == null ? void 0 : _a.ref,
         targetBranch: (_b = pr.base) == null ? void 0 : _b.ref,
         author: pr.user ? { name: pr.user.login, username: pr.user.login, avatarUrl: pr.user.avatar_url } : null,
         assignees: (pr.assignees || []).map((a) => ({ name: a.login, username: a.login, avatarUrl: a.avatar_url })),
@@ -215,9 +217,9 @@ async function mergeRequests(ctx, next) {
   await next();
 }
 async function mergeRequestDetail(ctx, next) {
-  var _a, _b, _c, _d;
+  var _a, _b, _c;
   const { pat, apiBase, encodedProject, projectPath, isGitHub } = await getRepoApiContext(ctx);
-  const params = { ...ctx.action.params, ...(_a = ctx.action.params) == null ? void 0 : _a.values, ...ctx.request.body || {} };
+  const params = getActionParams(ctx);
   const { mrIid } = params;
   if (!mrIid) {
     ctx.throw(400, "mrIid is required");
@@ -243,8 +245,8 @@ async function mergeRequestDetail(ctx, next) {
       title: pr.title,
       description: pr.body,
       state: pr.state === "open" ? "opened" : pr.merged_at ? "merged" : "closed",
-      sourceBranch: (_b = pr.head) == null ? void 0 : _b.ref,
-      targetBranch: (_c = pr.base) == null ? void 0 : _c.ref,
+      sourceBranch: (_a = pr.head) == null ? void 0 : _a.ref,
+      targetBranch: (_b = pr.base) == null ? void 0 : _b.ref,
       author: pr.user ? { name: pr.user.login, username: pr.user.login, avatarUrl: pr.user.avatar_url } : null,
       assignees: (pr.assignees || []).map((a) => ({ name: a.login, username: a.login, avatarUrl: a.avatar_url })),
       reviewers: (pr.requested_reviewers || []).map((r) => ({ name: r.login, username: r.login, avatarUrl: r.avatar_url })),
@@ -276,7 +278,7 @@ async function mergeRequestDetail(ctx, next) {
       gitlabFetch(apiBase, `/projects/${encodedProject}/merge_requests/${mrIid}/changes`, pat).catch(() => ({ data: {} }))
     ]);
     const mr = mrResult.data;
-    const changes = (((_d = changesResult.data) == null ? void 0 : _d.changes) || []).map((c) => ({
+    const changes = (((_c = changesResult.data) == null ? void 0 : _c.changes) || []).map((c) => ({
       oldPath: c.old_path,
       newPath: c.new_path,
       newFile: c.new_file,
@@ -318,9 +320,8 @@ async function mergeRequestDetail(ctx, next) {
   await next();
 }
 async function mergeRequestNotes(ctx, next) {
-  var _a;
   const { pat, apiBase, encodedProject, projectPath, isGitHub } = await getRepoApiContext(ctx);
-  const params = { ...ctx.action.params, ...(_a = ctx.action.params) == null ? void 0 : _a.values, ...ctx.request.body || {} };
+  const params = getActionParams(ctx);
   const { mrIid, page = 1, perPage = 50 } = params;
   if (!mrIid) {
     ctx.throw(400, "mrIid is required");

package/dist/server/actions/review.d.ts

@@ -1,5 +1,6 @@
 import { Context } from '@nocobase/actions';
 import type { Application } from '@nocobase/server';
+export declare const WORKER_JOB_GIT_REVIEW_PROCESS = "git-review:process";
 interface TriggerArgs {
     flowId?: number | null;
     repositoryId: number;
@@ -14,8 +15,8 @@ interface TriggerArgs {
 }
 /**
  * Trigger an AI-driven code review for an MR / commit / branch.
- * The review record is upserted synchronously, then AIEmployee.invoke runs in
- * the background. The action returns immediately with the reviewId.
+ * The review record is upserted synchronously, then queued for an available
+ * git-review worker. The action returns immediately with the reviewId.
  */
 export declare function triggerReview(ctx: Context, next: () => Promise<void>): Promise<void>;
 /**
@@ -23,6 +24,8 @@ export declare function triggerReview(ctx: Context, next: () => Promise<void>):
  * Returns the reviewId of the upserted record.
  */
 export declare function triggerReviewInternal(app: Application, args: TriggerArgs): Promise<number>;
+export declare function registerReviewQueue(app: Application): void;
+export declare function unregisterReviewQueue(app: Application): void;
 /**
  * Mark a review as approved and post its content to GitLab as an MR note.
  */

package/dist/server/actions/review.js

@@ -26,17 +26,34 @@ var __copyProps = (to, from, except, desc) => {
 var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: true }), mod);
 var review_exports = {};
 __export(review_exports, {
+  WORKER_JOB_GIT_REVIEW_PROCESS: () => WORKER_JOB_GIT_REVIEW_PROCESS,
   branchMatches: () => branchMatches,
   pickFlowMatchingBranch: () => pickFlowMatchingBranch,
   recoverStuckReviews: () => recoverStuckReviews,
+  registerReviewQueue: () => registerReviewQueue,
   reviewApprovePost: () => reviewApprovePost,
   reviewReject: () => reviewReject,
   triggerReview: () => triggerReview,
-  triggerReviewInternal: () => triggerReviewInternal
+  triggerReviewInternal: () => triggerReviewInternal,
+  unregisterReviewQueue: () => unregisterReviewQueue
 });
 module.exports = __toCommonJS(review_exports);
 var import_gitlab_url = require("../utils/gitlab-url");
 var import_redact = require("../utils/redact");
+const WORKER_JOB_GIT_REVIEW_PROCESS = "git-review:process";
+const REVIEW_QUEUE_CHANNEL = "plugin-git-manager.review";
+const REVIEW_QUEUE_CONCURRENCY = Math.max(
+  1,
+  Number.parseInt(process.env.GIT_REVIEW_QUEUE_CONCURRENCY || process.env.GIT_REVIEW_MAX_CONCURRENCY || "3", 10) || 3
+);
+const REVIEW_QUEUE_TIMEOUT_MS = Math.max(
+  6e4,
+  Number.parseInt(process.env.GIT_REVIEW_QUEUE_TIMEOUT_MS || "", 10) || 10 * 60 * 1e3
+);
+function getActionParams(ctx) {
+  var _a, _b;
+  return { ...ctx.action.params, ...(_a = ctx.action.params) == null ? void 0 : _a.values, ...((_b = ctx.request) == null ? void 0 : _b.body) || {} };
+}
 function targetKey(args) {
   if (args.targetType === "mr") return `${args.repositoryId}:mr:${args.mrIid}`;
   if (args.targetType === "commit") return `${args.repositoryId}:commit:${args.commitSha}`;
@@ -48,8 +65,8 @@ async function withTriggerLock(app, key, fn) {
   return app.lockManager.runExclusive(lockKey, fn, TRIGGER_LOCK_TTL_MS);
 }
 async function triggerReview(ctx, next) {
-  var _a, _b, _c;
-  const params = { ...ctx.action.params, ...(_a = ctx.action.params) == null ? void 0 : _a.values, ...ctx.request.body || {} };
+  var _a, _b;
+  const params = getActionParams(ctx);
   const {
     flowId,
     repositoryId,
@@ -65,7 +82,7 @@ async function triggerReview(ctx, next) {
   if (targetType === "mr" && !mrIid) ctx.throw(400, "mrIid is required for MR review");
   if (targetType === "commit" && !commitSha) ctx.throw(400, "commitSha is required for commit review");
   if (targetType === "branch" && !branch) ctx.throw(400, "branch is required for branch review");
-  const userId = (_c = (_b = ctx.state) == null ? void 0 : _b.currentUser) == null ? void 0 : _c.id;
+  const userId = (_b = (_a = ctx.state) == null ? void 0 : _a.currentUser) == null ? void 0 : _b.id;
   try {
     const reviewId = await triggerReviewInternal(ctx.app, {
       flowId,
@@ -138,13 +155,12 @@ async function triggerReviewInternalLocked(app, args) {
     latestSha: headSha || existingLatestSha || null,
     triggeredBy: args.triggeredBy || "manual",
     status: "pending",
-    // Stamp startedAt synchronously so `recoverStuckReviews` can sweep rows
-    // that get stuck in `pending` (process died before runReview ran).
-    // runReview's own update will refresh this on actual start.
-    startedAt: /* @__PURE__ */ new Date(),
+    // `startedAt` is stamped by the queue worker when execution actually
+    // starts. Pending rows may be legitimately waiting in Redis.
+    startedAt: null,
     finishedAt: null,
     durationMs: null,
-    postStatus: flow.get("postMode") === "disabled" ? "skipped" : "pending_approval",
+    postStatus: getInitialPostStatus(flow, args.targetType),
     error: null
   };
   let reviewId;
@@ -162,29 +178,133 @@ async function triggerReviewInternalLocked(app, args) {
     const review = await reviewsRepo.create({ values: baseValues });
     reviewId = review.get("id");
   }
-  setImmediate(
-    () => runReview(app, {
-      reviewId,
+  await enqueueReview(app, {
+    reviewId,
+    repositoryId: args.repositoryId,
+    targetType: args.targetType,
+    mrIid: args.targetType === "mr" ? args.mrIid : null,
+    commitSha: args.targetType === "commit" ? args.commitSha : null,
+    branch: args.branch || void 0,
+    headSha,
+    aiEmployeeUsername,
+    extraInstructions: args.extraInstructions,
+    userId: args.userId ?? null,
+    flowSnapshot: createFlowSnapshot(flow)
+  });
+  return reviewId;
+}
+function registerReviewQueue(app) {
+  app.eventQueue.subscribe(REVIEW_QUEUE_CHANNEL, {
+    concurrency: REVIEW_QUEUE_CONCURRENCY,
+    idle: () => isGitReviewWorker(app),
+    process: async (message) => {
+      await processQueuedReview(app, message);
+    }
+  });
+}
+function unregisterReviewQueue(app) {
+  app.eventQueue.unsubscribe(REVIEW_QUEUE_CHANNEL);
+}
+function createFlowSnapshot(flow) {
+  return {
+    id: Number(flow.get("id")),
+    name: flow.get("name"),
+    postMode: flow.get("postMode"),
+    llmService: flow.get("llmService"),
+    model: flow.get("model"),
+    instructions: flow.get("instructions")
+  };
+}
+function createFlowFromSnapshot(snapshot, fallback) {
+  return {
+    get(name) {
+      var _a;
+      if (snapshot && Object.prototype.hasOwnProperty.call(snapshot, name)) {
+        return snapshot[name];
+      }
+      return (_a = fallback == null ? void 0 : fallback.get) == null ? void 0 : _a.call(fallback, name);
+    }
+  };
+}
+function isGitReviewWorker(app) {
+  const workerMode = process.env.WORKER_MODE || "";
+  return app.serving(WORKER_JOB_GIT_REVIEW_PROCESS) || workerMode === "worker" || workerMode === "task" || process.env.APP_ROLE === "worker";
+}
+async function enqueueReview(app, message) {
+  try {
+    await app.eventQueue.publish(REVIEW_QUEUE_CHANNEL, message, {
+      timeout: REVIEW_QUEUE_TIMEOUT_MS,
+      maxRetries: 1
+    });
+  } catch (err) {
+    const safeMessage = (0, import_redact.redactPat)((err == null ? void 0 : err.message) || String(err));
+    await app.db.getRepository("gitCodeReviews").update({
+      filterByTk: message.reviewId,
+      values: {
+        status: "failed",
+        error: `Failed to enqueue review: ${safeMessage}`,
+        finishedAt: /* @__PURE__ */ new Date()
+      }
+    });
+    throw err;
+  }
+}
+async function failQueuedReview(app, reviewId, err) {
+  const safeMessage = (0, import_redact.redactPat)((err == null ? void 0 : err.message) || String(err));
+  await app.db.getRepository("gitCodeReviews").update({
+    filterByTk: reviewId,
+    values: {
+      status: "failed",
+      error: safeMessage,
+      finishedAt: /* @__PURE__ */ new Date()
+    }
+  });
+}
+async function processQueuedReview(app, message) {
+  var _a, _b, _c, _d, _e, _f, _g;
+  const db = app.db;
+  const reviewsRepo = db.getRepository("gitCodeReviews");
+  const review = await reviewsRepo.findOne({ filterByTk: message.reviewId });
+  if (!review) {
+    (_b = (_a = app.log) == null ? void 0 : _a.warn) == null ? void 0 : _b.call(_a, `git review queue: review ${message.reviewId} not found, skipping`);
+    return;
+  }
+  if (review.get("status") !== "pending") {
+    (_d = (_c = app.log) == null ? void 0 : _c.info) == null ? void 0 : _d.call(_c, `git review queue: review ${message.reviewId} is ${review.get("status")}, skipping`);
+    return;
+  }
+  try {
+    const repo = await db.getRepository("gitRepositories").findOne({
+      filterByTk: message.repositoryId || review.get("repositoryId")
+    });
+    if (!repo) throw new Error("Repository not found");
+    const storedFlow = await db.getRepository("gitReviewFlows").findOne({
+      filterByTk: ((_e = message.flowSnapshot) == null ? void 0 : _e.id) || review.get("flowId")
+    });
+    const flow = createFlowFromSnapshot(message.flowSnapshot, storedFlow);
+    const aiEmployeeUsername = message.aiEmployeeUsername || flow.get("aiEmployeeUsername");
+    if (!aiEmployeeUsername) throw new Error("Flow has no AI employee configured");
+    await runReview(app, {
+      reviewId: message.reviewId,
       flow,
       repo,
-      targetType: args.targetType,
-      mrIid: args.targetType === "mr" ? args.mrIid : null,
-      commitSha: args.targetType === "commit" ? args.commitSha : null,
-      branch: args.branch || void 0,
-      headSha,
+      targetType: message.targetType || review.get("targetType"),
+      mrIid: message.targetType === "mr" ? message.mrIid ?? Number(review.get("mrIid")) : null,
+      commitSha: message.targetType === "commit" ? message.commitSha || review.get("commitSha") : null,
+      branch: message.branch || review.get("branch"),
+      headSha: message.headSha || review.get("headSha"),
       aiEmployeeUsername,
-      extraInstructions: args.extraInstructions,
-      userId: args.userId ?? null
-    }).catch((err) => {
-      var _a, _b;
-      (_b = (_a = app.log) == null ? void 0 : _a.error) == null ? void 0 : _b.call(_a, "runReview background error", err);
-    })
-  );
-  return reviewId;
+      extraInstructions: message.extraInstructions,
+      userId: message.userId ?? null
+    });
+  } catch (err) {
+    (_g = (_f = app.log) == null ? void 0 : _f.error) == null ? void 0 : _g.call(_f, "git review queue: failed before review execution", err);
+    await failQueuedReview(app, message.reviewId, err);
+  }
 }
 async function reviewApprovePost(ctx, next) {
-  var _a, _b, _c;
-  const params = { ...ctx.action.params, ...(_a = ctx.action.params) == null ? void 0 : _a.values, ...ctx.request.body || {} };
+  var _a, _b;
+  const params = getActionParams(ctx);
   const { reviewId, editedMarkdown } = params;
   if (!reviewId) ctx.throw(400, "reviewId is required");
   const reviewsRepo = ctx.db.getRepository("gitCodeReviews");
@@ -199,7 +319,7 @@ async function reviewApprovePost(ctx, next) {
   });
   if (!repo) ctx.throw(404, "Repository not found");
   const noteId = await postNoteToGitLab(repo, Number(review.get("mrIid")), markdown);
-  const userId = (_c = (_b = ctx.state) == null ? void 0 : _b.currentUser) == null ? void 0 : _c.id;
+  const userId = (_b = (_a = ctx.state) == null ? void 0 : _a.currentUser) == null ? void 0 : _b.id;
   await reviewsRepo.update({
     filterByTk: reviewId,
     values: {
@@ -207,21 +327,22 @@ async function reviewApprovePost(ctx, next) {
       postStatus: "posted",
       postedNoteId: String(noteId),
       approvedBy: userId ? String(userId) : null,
-      approvedAt: /* @__PURE__ */ new Date()
+      approvedAt: /* @__PURE__ */ new Date(),
+      error: null
     }
   });
   ctx.body = { success: true, data: { reviewId, postedNoteId: noteId } };
   await next();
 }
 async function reviewReject(ctx, next) {
-  var _a, _b, _c;
-  const params = { ...ctx.action.params, ...(_a = ctx.action.params) == null ? void 0 : _a.values, ...ctx.request.body || {} };
+  var _a, _b;
+  const params = getActionParams(ctx);
   const { reviewId, reason } = params;
   if (!reviewId) ctx.throw(400, "reviewId is required");
   const reviewsRepo = ctx.db.getRepository("gitCodeReviews");
   const review = await reviewsRepo.findOne({ filterByTk: reviewId });
   if (!review) ctx.throw(404, "Review not found");
-  const userId = (_c = (_b = ctx.state) == null ? void 0 : _b.currentUser) == null ? void 0 : _c.id;
+  const userId = (_b = (_a = ctx.state) == null ? void 0 : _a.currentUser) == null ? void 0 : _b.id;
   await reviewsRepo.update({
     filterByTk: reviewId,
     values: {
@@ -353,9 +474,10 @@ async function runReview(app, args) {
     }
     const finishedAt = /* @__PURE__ */ new Date();
     const durationMs = finishedAt.getTime() - startedAt.getTime();
-    const postMode = args.flow.get("postMode");
-    let postStatus = "pending_approval";
+    const postMode = getFlowPostMode(args.flow);
+    let postStatus = getInitialPostStatus(args.flow, args.targetType);
     let postedNoteId = null;
+    let autoPostError = null;
     if (postMode === "disabled") {
       postStatus = "skipped";
     } else if (postMode === "auto" && args.targetType === "mr" && args.mrIid) {
@@ -363,6 +485,8 @@ async function runReview(app, args) {
         postedNoteId = String(await postNoteToGitLab(args.repo, args.mrIid, content));
         postStatus = "posted";
       } catch (err) {
+        autoPostError = (0, import_redact.redactPat)((err == null ? void 0 : err.message) || String(err));
+        postStatus = "post_failed";
         (_c = (_b = app.log) == null ? void 0 : _b.error) == null ? void 0 : _c.call(_b, "Auto-post review note failed", err);
       }
     }
@@ -380,11 +504,14 @@ async function runReview(app, args) {
         finishedAt,
         postStatus,
         postedNoteId,
+        error: autoPostError ? `Auto-post failed: ${autoPostError}` : null,
         metadata: {
           flowName: args.flow.get("name"),
           aiEmployeeUsername: args.aiEmployeeUsername,
           llmService,
-          model
+          model,
+          postMode,
+          autoPostError
         }
       }
     });
@@ -555,6 +682,23 @@ async function postNoteToGitLab(repo, mrIid, body) {
 }
 const MAX_BRANCH_FILTER_LENGTH = 200;
 const loggedBadFilters = /* @__PURE__ */ new Set();
+function getFlowPostMode(flow) {
+  var _a;
+  const rawValue = (_a = flow == null ? void 0 : flow.get) == null ? void 0 : _a.call(flow, "postMode");
+  const value = rawValue && typeof rawValue === "object" && "value" in rawValue ? rawValue.value : rawValue;
+  const normalized = String(value || "manual").trim().toLowerCase().replace(/[\s-]+/g, "_");
+  if (["auto", "auto_post", "autopost", "auto_post_to_mr"].includes(normalized)) return "auto";
+  if (["disabled", "disable", "do_not_post", "dont_post", "none", "skip", "skipped"].includes(normalized)) {
+    return "disabled";
+  }
+  return "manual";
+}
+function getInitialPostStatus(flow, targetType) {
+  const postMode = getFlowPostMode(flow);
+  if (postMode === "disabled") return "skipped";
+  if (postMode === "auto" && targetType !== "mr") return "skipped";
+  return "pending_approval";
+}
 function warnInvalidBranchFilter(filter, reason) {
   if (loggedBadFilters.has(filter)) return;
   loggedBadFilters.add(filter);
@@ -597,7 +741,7 @@ async function recoverStuckReviews(app) {
     const cutoff = new Date(Date.now() - STUCK_REVIEW_CUTOFF_MS);
     const stuck = await reviewsRepo.find({
       filter: {
-        status: { $in: ["running", "pending"] },
+        status: "running",
         startedAt: { $lt: cutoff }
       }
     });
@@ -625,11 +769,14 @@ async function recoverStuckReviews(app) {
 }
 // Annotate the CommonJS export names for ESM import in node:
 0 && (module.exports = {
+  WORKER_JOB_GIT_REVIEW_PROCESS,
   branchMatches,
   pickFlowMatchingBranch,
   recoverStuckReviews,
+  registerReviewQueue,
   reviewApprovePost,
   reviewReject,
   triggerReview,
-  triggerReviewInternal
+  triggerReviewInternal,
+  unregisterReviewQueue
 });

package/dist/server/ai-tools.js

@@ -97,12 +97,14 @@ function registerGitReviewAiTools(app) {
     return obj;
   };
   const runResourceAction = async (ctx, handler, params, gate) => {
+    var _a2;
     enforceAcl(ctx, gate.resource, gate.action, params);
     const synthCtx = {
       ...ctx,
       app: ctx.app,
       db: ctx.db,
       action: { params },
+      request: { ...ctx.request || {}, body: ((_a2 = ctx.request) == null ? void 0 : _a2.body) || {} },
       throw: (status, message) => {
         const err = new Error(message);
         err.status = status;

package/dist/server/collections/gitCodeReviews.js

@@ -166,6 +166,7 @@ var gitCodeReviews_default = (0, import_database.defineCollection)({
           { value: "pending_approval", label: "Pending Approval" },
           { value: "approved", label: "Approved" },
           { value: "posted", label: "Posted" },
+          { value: "post_failed", label: "Post Failed" },
           { value: "skipped", label: "Skipped" },
           { value: "rejected", label: "Rejected" }
         ]

package/dist/server/collections/gitRepositories.js

@@ -87,6 +87,18 @@ var gitRepositories_default = (0, import_database.defineCollection)({
       interface: "checkbox",
       uiSchema: { title: "Auto Review", type: "boolean", "x-component": "Checkbox" }
     },
+    {
+      type: "belongsTo",
+      name: "autoReviewFlow",
+      target: "gitReviewFlows",
+      foreignKey: "autoReviewFlowId",
+      interface: "m2o",
+      uiSchema: {
+        title: "Primary Auto Review Flow",
+        "x-component": "AssociationField",
+        "x-component-props": { fieldNames: { label: "name", value: "id" } }
+      }
+    },
     {
       type: "date",
       name: "lastPolledAt",

package/dist/server/migrations/20260508000000-add-auto-review-flow-id.d.ts

@@ -0,0 +1,6 @@
+import { Migration } from '@nocobase/server';
+export default class AddAutoReviewFlowIdMigration extends Migration {
+    on: string;
+    up(): Promise<void>;
+    down(): Promise<void>;
+}

package/dist/server/migrations/20260508000000-add-auto-review-flow-id.js

@@ -0,0 +1,57 @@
+/**
+ * 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 add_auto_review_flow_id_exports = {};
+__export(add_auto_review_flow_id_exports, {
+  default: () => AddAutoReviewFlowIdMigration
+});
+module.exports = __toCommonJS(add_auto_review_flow_id_exports);
+var import_server = require("@nocobase/server");
+var import_sequelize = require("sequelize");
+class AddAutoReviewFlowIdMigration extends import_server.Migration {
+  on = "afterLoad";
+  async up() {
+    var _a;
+    const queryInterface = this.db.sequelize.getQueryInterface();
+    const tablePrefix = ((_a = this.db.options) == null ? void 0 : _a.tablePrefix) || "";
+    const tableName = `${tablePrefix}gitRepositories`;
+    const tableInfo = await queryInterface.describeTable(tableName).catch(() => null);
+    if (!tableInfo || tableInfo.autoReviewFlowId) return;
+    await queryInterface.addColumn(tableName, "autoReviewFlowId", {
+      type: import_sequelize.DataTypes.INTEGER,
+      allowNull: true
+    });
+  }
+  async down() {
+    var _a;
+    const queryInterface = this.db.sequelize.getQueryInterface();
+    const tablePrefix = ((_a = this.db.options) == null ? void 0 : _a.tablePrefix) || "";
+    const tableName = `${tablePrefix}gitRepositories`;
+    const tableInfo = await queryInterface.describeTable(tableName).catch(() => null);
+    if (!(tableInfo == null ? void 0 : tableInfo.autoReviewFlowId)) return;
+    await queryInterface.removeColumn(tableName, "autoReviewFlowId");
+  }
+}

package/dist/server/plugin.d.ts

@@ -1,8 +1,12 @@
 import { Plugin } from '@nocobase/server';
 export declare class PluginGitManagerServer extends Plugin {
+    app: any;
+    db: any;
+    beforeLoad(): Promise<void>;
     load(): Promise<void>;
     install(): Promise<void>;
     beforeDisable(): Promise<void>;
     beforeUnload(): Promise<void>;
 }
+export declare function ensureAutoReviewFlowSchema(app: any): Promise<void>;
 export default PluginGitManagerServer;