plugin-git-manager 1.0.10 → 1.0.12

package/dist/externalVersion.js

@@ -11,8 +11,8 @@ module.exports = {
   "react": "18.2.0",
   "antd": "5.24.2",
   "@ant-design/icons": "5.6.1",
-  "@nocobase/client": "2.0.46",
-  "@nocobase/server": "2.0.46",
-  "@nocobase/actions": "2.0.46",
-  "@nocobase/database": "2.0.46"
+  "@nocobase/client": "2.0.47",
+  "@nocobase/server": "2.0.47",
+  "@nocobase/actions": "2.0.47",
+  "@nocobase/database": "2.0.47"
 };

package/dist/server/actions/git-actions.js

@@ -53,7 +53,8 @@ module.exports = __toCommonJS(git_actions_exports);
 var import_simple_git = __toESM(require("simple-git"));
 var path = __toESM(require("path"));
 var fs = __toESM(require("fs"));
-const REF_PATTERN = /^[a-zA-Z0-9._\-\/]+$/;
+var import_redact = require("../utils/redact");
+const REF_PATTERN = /^(?!-)[a-zA-Z0-9._\-\/]+$/;
 const repoLocks = /* @__PURE__ */ new Map();
 function acquireLock(key) {
   const prev = repoLocks.get(key) || Promise.resolve();
@@ -89,14 +90,16 @@ function validateRepoUrl(repoUrl) {
     throw new Error("Only HTTPS repository URLs are allowed");
   }
 }
-async function withAuth(git, repoUrl, pat, fn, username) {
-  const lockKey = repoUrl;
+async function withAuth(git, localPath, repoUrl, pat, fn, username) {
+  const lockKey = localPath;
   const lock = acquireLock(lockKey);
   await lock.promise;
   const authUrl = getAuthUrl(repoUrl, pat, username);
   await git.remote(["set-url", "origin", authUrl]);
   try {
     return await fn();
+  } catch (err) {
+    throw (0, import_redact.redactError)(err);
   } finally {
     try {
       await git.remote(["set-url", "origin", repoUrl]);
@@ -105,8 +108,8 @@ async function withAuth(git, repoUrl, pat, fn, username) {
         await git.remote(["set-url", "origin", repoUrl]);
       } catch {
         console.error(
-          `[plugin-git-manager] CRITICAL: failed to remove PAT from remote URL for ${repoUrl}. Manual cleanup of .git/config may be required.`,
-          cleanupErr
+          `[plugin-git-manager] CRITICAL: failed to remove PAT from remote URL for ${(0, import_redact.redactPat)(repoUrl)}. Manual cleanup of .git/config may be required.`,
+          (0, import_redact.redactError)(cleanupErr)
         );
       }
     }
@@ -114,9 +117,9 @@ async function withAuth(git, repoUrl, pat, fn, username) {
   }
 }
 function getAuthUrl(repoUrl, pat, username) {
-  const url = new URL(repoUrl);
-  url.username = username || "oauth2";
-  url.password = pat;
+  const url = new URL(repoUrl.trim());
+  url.username = (username || "oauth2").trim();
+  url.password = pat.trim();
   return url.toString();
 }
 function getGit(localPath) {
@@ -144,10 +147,12 @@ function validateLocalPath(localPath) {
 async function clone(ctx, next) {
   const repo = await getRepo(ctx);
   const localPath = validateLocalPath(repo.get("localPath"));
-  const repoUrl = repo.get("repoUrl");
-  const pat = repo.get("pat");
-  const username = repo.get("username");
+  const repoUrl = (repo.get("repoUrl") || "").trim();
+  const pat = (repo.get("pat") || "").trim();
+  const username = (repo.get("username") || "").trim();
+  const defaultBranch = (repo.get("defaultBranch") || "main").trim() || "main";
   validateRepoUrl(repoUrl);
+  validateBranch(defaultBranch);
   if (fs.existsSync(localPath)) {
     ctx.throw(400, "Directory already exists. Remove it before cloning again.");
   }
@@ -156,7 +161,7 @@ async function clone(ctx, next) {
   }
   const authUrl = getAuthUrl(repoUrl, pat, username);
   try {
-    await (0, import_simple_git.default)().clone(authUrl, localPath, ["--branch", repo.get("defaultBranch") || "main"]);
+    await (0, import_simple_git.default)().clone(authUrl, localPath, ["--branch", defaultBranch]);
     await (0, import_simple_git.default)(localPath).remote(["set-url", "origin", repoUrl]);
     await ctx.db.getRepository("gitRepositories").update({
       filterByTk: repo.get("id"),
@@ -168,40 +173,40 @@ async function clone(ctx, next) {
       filterByTk: repo.get("id"),
       values: { status: "error" }
     });
-    throw err;
+    throw (0, import_redact.redactError)(err);
   }
   await next();
 }
 async function pull(ctx, next) {
   const repo = await getRepo(ctx);
   const localPath = validateLocalPath(repo.get("localPath"));
-  const pat = repo.get("pat");
-  const repoUrl = repo.get("repoUrl");
-  const username = repo.get("username");
+  const pat = (repo.get("pat") || "").trim();
+  const repoUrl = (repo.get("repoUrl") || "").trim();
+  const username = (repo.get("username") || "").trim();
   const git = getGit(localPath);
-  const result = await withAuth(git, repoUrl, pat, () => git.pull(), username);
+  const result = await withAuth(git, localPath, repoUrl, pat, () => git.pull(), username);
   ctx.body = { success: true, data: result };
   await next();
 }
 async function push(ctx, next) {
   const repo = await getRepo(ctx);
   const localPath = validateLocalPath(repo.get("localPath"));
-  const pat = repo.get("pat");
-  const repoUrl = repo.get("repoUrl");
-  const username = repo.get("username");
+  const pat = (repo.get("pat") || "").trim();
+  const repoUrl = (repo.get("repoUrl") || "").trim();
+  const username = (repo.get("username") || "").trim();
   const git = getGit(localPath);
-  const result = await withAuth(git, repoUrl, pat, () => git.push(), username);
+  const result = await withAuth(git, localPath, repoUrl, pat, () => git.push(), username);
   ctx.body = { success: true, data: result };
   await next();
 }
 async function fetch(ctx, next) {
   const repo = await getRepo(ctx);
   const localPath = validateLocalPath(repo.get("localPath"));
-  const pat = repo.get("pat");
-  const repoUrl = repo.get("repoUrl");
-  const username = repo.get("username");
+  const pat = (repo.get("pat") || "").trim();
+  const repoUrl = (repo.get("repoUrl") || "").trim();
+  const username = (repo.get("username") || "").trim();
   const git = getGit(localPath);
-  const result = await withAuth(git, repoUrl, pat, () => git.fetch(), username);
+  const result = await withAuth(git, localPath, repoUrl, pat, () => git.fetch(), username);
   ctx.body = { success: true, data: result };
   await next();
 }

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

@@ -66,8 +66,8 @@ async function getRepoApiContext(ctx) {
   if (!repo) {
     ctx.throw(404, "Repository not found");
   }
-  const pat = repo.get("pat");
-  const repoUrl = repo.get("repoUrl");
+  const pat = (repo.get("pat") || "").trim();
+  const repoUrl = (repo.get("repoUrl") || "").trim();
   const isGitHub = repoUrl.includes("github.com");
   const { apiBase, encodedProject, projectPath } = (0, import_gitlab_url.parseGitLabProject)(repoUrl);
   return { repo, pat, apiBase, encodedProject, projectPath, isGitHub };
@@ -128,8 +128,14 @@ async function mergeRequests(ctx, next) {
       direction: sort
     });
     let pullRequests = result.data || [];
+    let mergedFilterApplied = false;
     if (state === "merged") {
       pullRequests = pullRequests.filter((pr) => pr.merged_at);
+      mergedFilterApplied = true;
+    }
+    if (mergedFilterApplied) {
+      result.totalPages = null;
+      result.total = null;
     }
     items = pullRequests.map((pr) => {
       var _a2, _b;
@@ -147,18 +153,18 @@ async function mergeRequests(ctx, next) {
         labels: (pr.labels || []).map((l) => l.name),
         draft: pr.draft || false,
         mergedBy: null,
-        // Not returned in list API
+        // Not returned by the PR list endpoint
         mergedAt: pr.merged_at,
         createdAt: pr.created_at,
         updatedAt: pr.updated_at,
-        userNotesCount: 0,
-        // Not returned in pull list API
+        userNotesCount: typeof pr.comments === "number" ? pr.comments : 0,
         upvotes: 0,
         downvotes: 0,
         webUrl: pr.html_url,
-        hasConflicts: false,
-        // Not guaranteed in list
-        changesCount: 0
+        // GitHub's PR list does not include mergeability info — surface as
+        // `null` (unknown) so the UI doesn't render a misleading "no conflicts".
+        hasConflicts: null,
+        changesCount: typeof pr.changed_files === "number" ? pr.changed_files : null
       };
     });
   } else {
@@ -252,8 +258,15 @@ async function mergeRequestDetail(ctx, next) {
       // Not always readily available on GitHub without extra call
       closedAt: pr.closed_at,
       webUrl: pr.html_url,
-      hasConflicts: pr.mergeable_state === "dirty",
-      diffStats: { additions: pr.additions },
+      // `mergeable_state === 'unknown'` when GitHub is still computing — surface
+      // `null` instead of `false` so the UI can distinguish "no conflicts" from
+      // "not yet known".
+      hasConflicts: pr.mergeable_state === "dirty" ? true : pr.mergeable_state === "unknown" || pr.mergeable === null ? null : false,
+      diffStats: {
+        additions: typeof pr.additions === "number" ? pr.additions : null,
+        deletions: typeof pr.deletions === "number" ? pr.deletions : null,
+        changedFiles: typeof pr.changed_files === "number" ? pr.changed_files : null
+      },
       changes
     };
   } else {

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

@@ -33,4 +33,5 @@ export declare function reviewApprovePost(ctx: Context, next: () => Promise<void
 export declare function reviewReject(ctx: Context, next: () => Promise<void>): Promise<void>;
 export declare function branchMatches(flow: any, branch: string): boolean;
 export declare function pickFlowMatchingBranch(flows: any[], branch?: string): any | null;
+export declare function recoverStuckReviews(app: Application): Promise<number>;
 export {};

package/dist/server/actions/review.js

@@ -28,6 +28,7 @@ var review_exports = {};
 __export(review_exports, {
   branchMatches: () => branchMatches,
   pickFlowMatchingBranch: () => pickFlowMatchingBranch,
+  recoverStuckReviews: () => recoverStuckReviews,
   reviewApprovePost: () => reviewApprovePost,
   reviewReject: () => reviewReject,
   triggerReview: () => triggerReview,
@@ -35,6 +36,17 @@ __export(review_exports, {
 });
 module.exports = __toCommonJS(review_exports);
 var import_gitlab_url = require("../utils/gitlab-url");
+var import_redact = require("../utils/redact");
+function targetKey(args) {
+  if (args.targetType === "mr") return `${args.repositoryId}:mr:${args.mrIid}`;
+  if (args.targetType === "commit") return `${args.repositoryId}:commit:${args.commitSha}`;
+  return `${args.repositoryId}:branch:${args.branch}`;
+}
+const TRIGGER_LOCK_TTL_MS = 3e4;
+async function withTriggerLock(app, key, fn) {
+  const lockKey = `git-review:trigger:${key}`;
+  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 || {} };
@@ -73,6 +85,9 @@ async function triggerReview(ctx, next) {
   await next();
 }
 async function triggerReviewInternal(app, args) {
+  return withTriggerLock(app, targetKey(args), () => triggerReviewInternalLocked(app, args));
+}
+async function triggerReviewInternalLocked(app, args) {
   const db = app.db;
   const flowsRepo = db.getRepository("gitReviewFlows");
   let flow = null;
@@ -123,12 +138,19 @@ async function triggerReviewInternal(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(),
+    finishedAt: null,
+    durationMs: null,
     postStatus: flow.get("postMode") === "disabled" ? "skipped" : "pending_approval",
     error: null
   };
   let reviewId;
   if (existing) {
-    if (existing.get("status") === "running") {
+    const st = existing.get("status");
+    if (st === "running" || st === "pending") {
       return existing.get("id");
     }
     await reviewsRepo.update({
@@ -251,6 +273,7 @@ async function runReview(app, args) {
       db,
       state: { currentUser: args.userId ? { id: args.userId } : null },
       req: { headers: { "x-timezone": "UTC", "x-locale": "en-US" } },
+      log: app.logger || console,
       get(name) {
         return this.req.headers[String(name).toLowerCase()];
       },
@@ -356,11 +379,12 @@ async function runReview(app, args) {
     }
   } catch (err) {
     const finishedAt = /* @__PURE__ */ new Date();
+    const safeMessage = (0, import_redact.redactPat)((err == null ? void 0 : err.message) || String(err));
     await reviewsRepo.update({
       filterByTk: args.reviewId,
       values: {
         status: "failed",
-        error: (err == null ? void 0 : err.message) || String(err),
+        error: safeMessage,
         finishedAt,
         durationMs: finishedAt.getTime() - startedAt.getTime()
       }
@@ -409,18 +433,39 @@ function buildReviewPrompt(args) {
   return lines.join("\n");
 }
 function extractLastAiMessageContent(result) {
-  var _a;
-  if (!(result == null ? void 0 : result.messages) || !Array.isArray(result.messages)) return "";
-  for (let i = result.messages.length - 1; i >= 0; i--) {
-    const msg = result.messages[i];
-    if (!msg) continue;
-    const className = (_a = msg == null ? void 0 : msg.constructor) == null ? void 0 : _a.name;
-    if (className === "HumanMessage" || className === "ToolMessage") continue;
+  const messages = result == null ? void 0 : result.messages;
+  if (!Array.isArray(messages)) return "";
+  const isAiMessage = (msg) => {
+    var _a;
+    if (!msg) return false;
+    if (typeof msg._getType === "function") {
+      try {
+        return msg._getType() === "ai";
+      } catch {
+      }
+    }
+    if (typeof msg.role === "string") {
+      const r = msg.role.toLowerCase();
+      return r === "assistant" || r === "ai";
+    }
+    if (typeof msg.type === "string" && msg.type.toLowerCase() === "ai") return true;
+    const name = (_a = msg == null ? void 0 : msg.constructor) == null ? void 0 : _a.name;
+    if (name === "AIMessage" || name === "AIMessageChunk") return true;
+    return false;
+  };
+  const getContent = (msg) => {
     if (typeof msg.content === "string") return msg.content;
     if (Array.isArray(msg.content)) {
-      const textBlock = msg.content.find((c) => c.type === "text");
-      if (textBlock == null ? void 0 : textBlock.text) return textBlock.text;
+      const textBlock = msg.content.find((c) => (c == null ? void 0 : c.type) === "text");
+      if (typeof (textBlock == null ? void 0 : textBlock.text) === "string") return textBlock.text;
     }
+    return "";
+  };
+  for (let i = messages.length - 1; i >= 0; i--) {
+    const msg = messages[i];
+    if (!isAiMessage(msg)) continue;
+    const content = getContent(msg);
+    if (content) return content;
   }
   return "";
 }
@@ -495,14 +540,26 @@ async function postNoteToGitLab(repo, mrIid, body) {
   }
 }
 const MAX_BRANCH_FILTER_LENGTH = 200;
+const loggedBadFilters = /* @__PURE__ */ new Set();
+function warnInvalidBranchFilter(filter, reason) {
+  if (loggedBadFilters.has(filter)) return;
+  loggedBadFilters.add(filter);
+  console.warn(
+    `[plugin-git-manager] branchFilter rejected (${reason}): ${JSON.stringify(filter)}. Flow will not match any branch.`
+  );
+}
 function branchMatches(flow, branch) {
   const filter = flow.get("branchFilter");
   if (!filter) return true;
-  if (filter.length > MAX_BRANCH_FILTER_LENGTH) return true;
+  if (filter.length > MAX_BRANCH_FILTER_LENGTH) {
+    warnInvalidBranchFilter(filter, `too long (>${MAX_BRANCH_FILTER_LENGTH} chars)`);
+    return false;
+  }
   try {
     return new RegExp(filter).test(branch);
-  } catch {
-    return true;
+  } catch (err) {
+    warnInvalidBranchFilter(filter, `invalid regex: ${(err == null ? void 0 : err.message) || err}`);
+    return false;
   }
 }
 function pickFlowMatchingBranch(flows, branch) {
@@ -518,10 +575,45 @@ function throwHttp(status, message) {
   err.status = status;
   throw err;
 }
+const STUCK_REVIEW_CUTOFF_MS = 10 * 60 * 1e3;
+async function recoverStuckReviews(app) {
+  var _a, _b, _c, _d;
+  try {
+    const reviewsRepo = app.db.getRepository("gitCodeReviews");
+    const cutoff = new Date(Date.now() - STUCK_REVIEW_CUTOFF_MS);
+    const stuck = await reviewsRepo.find({
+      filter: {
+        status: { $in: ["running", "pending"] },
+        startedAt: { $lt: cutoff }
+      }
+    });
+    if (!(stuck == null ? void 0 : stuck.length)) return 0;
+    const finishedAt = /* @__PURE__ */ new Date();
+    for (const review of stuck) {
+      const startedAt = review.get("startedAt");
+      const durationMs = startedAt ? finishedAt.getTime() - new Date(startedAt).getTime() : null;
+      await reviewsRepo.update({
+        filterByTk: review.get("id"),
+        values: {
+          status: "failed",
+          error: "Review interrupted by application restart",
+          finishedAt,
+          durationMs
+        }
+      });
+    }
+    (_b = (_a = app.log) == null ? void 0 : _a.info) == null ? void 0 : _b.call(_a, `plugin-git-manager: marked ${stuck.length} stuck review(s) as failed after restart`);
+    return stuck.length;
+  } catch (err) {
+    (_d = (_c = app.log) == null ? void 0 : _c.error) == null ? void 0 : _d.call(_c, `plugin-git-manager: recoverStuckReviews failed: ${err == null ? void 0 : err.message}`);
+    return 0;
+  }
+}
 // Annotate the CommonJS export names for ESM import in node:
 0 && (module.exports = {
   branchMatches,
   pickFlowMatchingBranch,
+  recoverStuckReviews,
   reviewApprovePost,
   reviewReject,
   triggerReview,

package/dist/server/ai-tools.js

@@ -49,7 +49,31 @@ function registerGitReviewAiTools(app) {
     (_c = (_b = app.log) == null ? void 0 : _b.warn) == null ? void 0 : _c.call(_b, "plugin-git-manager: AIManager.toolsManager not available; skipping AI tool registration");
     return;
   }
-  const runResourceAction = async (ctx, handler, params) => {
+  const enforceAcl = (ctx, resource, action) => {
+    var _a2, _b2, _c2, _d2;
+    const user = (_a2 = ctx == null ? void 0 : ctx.state) == null ? void 0 : _a2.currentUser;
+    if (!(user == null ? void 0 : user.id)) {
+      const err = new Error("AI tool requires an authenticated user context");
+      err.status = 401;
+      throw err;
+    }
+    const acl = (_b2 = ctx.app) == null ? void 0 : _b2.acl;
+    if (!(acl == null ? void 0 : acl.can)) return;
+    const role = ((_c2 = ctx == null ? void 0 : ctx.state) == null ? void 0 : _c2.currentRole) || Array.isArray(user.roles) && ((_d2 = user.roles[0]) == null ? void 0 : _d2.name) || null;
+    if (!role) {
+      const err = new Error("AI tool requires a resolvable role on the current user");
+      err.status = 403;
+      throw err;
+    }
+    const allowed = acl.can({ role, resource, action });
+    if (!allowed) {
+      const err = new Error(`Permission denied: ${resource}:${action}`);
+      err.status = 403;
+      throw err;
+    }
+  };
+  const runResourceAction = async (ctx, handler, params, gate) => {
+    enforceAcl(ctx, gate.resource, gate.action);
     const synthCtx = {
       ...ctx,
       app: ctx.app,
@@ -80,7 +104,10 @@ function registerGitReviewAiTools(app) {
         })
       },
       invoke: async (ctx, args) => {
-        const body = await runResourceAction(ctx, gitlabApi.mergeRequestDetail, args);
+        const body = await runResourceAction(ctx, gitlabApi.mergeRequestDetail, args, {
+          resource: "gitManager",
+          action: "mergeRequestDetail"
+        });
         return (body == null ? void 0 : body.data) ?? body;
       }
     },
@@ -102,7 +129,10 @@ function registerGitReviewAiTools(app) {
         })
       },
       invoke: async (ctx, args) => {
-        const body = await runResourceAction(ctx, gitlabApi.mergeRequests, args);
+        const body = await runResourceAction(ctx, gitlabApi.mergeRequests, args, {
+          resource: "gitManager",
+          action: "mergeRequests"
+        });
         return (body == null ? void 0 : body.data) ?? body;
       }
     },
@@ -121,7 +151,10 @@ function registerGitReviewAiTools(app) {
         })
       },
       invoke: async (ctx, args) => {
-        const body = await runResourceAction(ctx, gitlabApi.mergeRequestNotes, args);
+        const body = await runResourceAction(ctx, gitlabApi.mergeRequestNotes, args, {
+          resource: "gitManager",
+          action: "mergeRequestNotes"
+        });
         return (body == null ? void 0 : body.data) ?? body;
       }
     },
@@ -140,7 +173,10 @@ function registerGitReviewAiTools(app) {
         })
       },
       invoke: async (ctx, args) => {
-        const body = await runResourceAction(ctx, gitActions.commitDetail, args);
+        const body = await runResourceAction(ctx, gitActions.commitDetail, args, {
+          resource: "gitManager",
+          action: "commitDetail"
+        });
         return (body == null ? void 0 : body.data) ?? body;
       }
     },
@@ -161,7 +197,10 @@ function registerGitReviewAiTools(app) {
         })
       },
       invoke: async (ctx, args) => {
-        const body = await runResourceAction(ctx, gitActions.diff, args);
+        const body = await runResourceAction(ctx, gitActions.diff, args, {
+          resource: "gitManager",
+          action: "diff"
+        });
         return (body == null ? void 0 : body.data) ?? body;
       }
     },
@@ -181,7 +220,10 @@ function registerGitReviewAiTools(app) {
         })
       },
       invoke: async (ctx, args) => {
-        const body = await runResourceAction(ctx, gitActions.fileContent, args);
+        const body = await runResourceAction(ctx, gitActions.fileContent, args, {
+          resource: "gitManager",
+          action: "fileContent"
+        });
         return (body == null ? void 0 : body.data) ?? body;
       }
     },
@@ -197,7 +239,10 @@ function registerGitReviewAiTools(app) {
         schema: import_zod.z.object({ repositoryId: import_zod.z.number() })
       },
       invoke: async (ctx, args) => {
-        const body = await runResourceAction(ctx, gitActions.branches, args);
+        const body = await runResourceAction(ctx, gitActions.branches, args, {
+          resource: "gitManager",
+          action: "branches"
+        });
         return (body == null ? void 0 : body.data) ?? body;
       }
     },
@@ -217,7 +262,10 @@ function registerGitReviewAiTools(app) {
         })
       },
       invoke: async (ctx, args) => {
-        const body = await runResourceAction(ctx, gitActions.log, args);
+        const body = await runResourceAction(ctx, gitActions.log, args, {
+          resource: "gitManager",
+          action: "log"
+        });
         return (body == null ? void 0 : body.data) ?? body;
       }
     }

package/dist/server/collections/gitRepositories.js

@@ -62,7 +62,7 @@ var gitRepositories_default = (0, import_database.defineCollection)({
       uiSchema: { title: "Local Path", type: "string", "x-component": "Input" }
     },
     {
-      type: "password",
+      type: "string",
       name: "pat",
       interface: "password",
       uiSchema: { title: "Personal Access Token", type: "string", "x-component": "Password" }

package/dist/server/plugin.js

@@ -46,6 +46,7 @@ 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_review = require("./actions/review");
 var import_ai_tools = require("./ai-tools");
 var import_poller = require("./poller");
 class PluginGitManagerServer extends import_server.Plugin {
@@ -78,8 +79,26 @@ class PluginGitManagerServer extends import_server.Plugin {
         pollerStatus: pollerActions.pollerStatus
       }
     });
+    this.app.use(async (ctx, next) => {
+      if (ctx.logger && ctx.logger.warn) {
+        const originalWarn = ctx.logger.warn.bind(ctx.logger);
+        ctx.logger.warn = (message, ...args) => {
+          if (typeof message === "string" && message.includes("[Workflow") && message.includes("collection") && message.includes("not found")) {
+            return ctx.logger;
+          }
+          return originalWarn(message, ...args);
+        };
+      }
+      return next();
+    });
     (0, import_ai_tools.registerGitReviewAiTools)(this.app);
     this.app.on("afterStart", () => {
+      (0, import_review.recoverStuckReviews)(this.app).catch(
+        (err) => {
+          var _a, _b;
+          return (_b = (_a = this.app.log) == null ? void 0 : _a.error) == null ? void 0 : _b.call(_a, "plugin-git-manager: recoverStuckReviews error", err);
+        }
+      );
       (0, import_poller.startPoller)(this.app);
     });
     this.app.on("beforeStop", () => {