feat(review): allow UI rereview via ai-review team request

Author: keithah

docs/runbooks/review-requested-debug.md

@@ -53,6 +53,7 @@ For the same `deliveryId`, check review handler gate logs.
 
 Expected outcomes:
 - Accepted path: `Accepted review_requested event for kodiai reviewer`
+- Accepted path (team-based rereview): `Accepted review_requested event for rereview team` (team `ai-review`)
 - Skip path with reason:
   - `non-kodiai-reviewer`
   - `team-only-request`

src/handlers/review.test.ts

@@ -229,13 +229,55 @@ describe("createReviewHandler review_requested gating", () => {
 
     await handlers.get("pull_request.review_requested")!(
       buildReviewRequestedEvent({
-        requested_team: { name: "backend" },
+        requested_team: { name: "backend", slug: "backend" },
       }),
     );
 
     expect(enqueueCount).toBe(0);
   });
 
+  test("accepts team-based rereview requests for ai-review", async () => {
+    const handlers = new Map<string, (event: WebhookEvent) => Promise<void>>();
+    const enqueued: Array<{ installationId: number }> = [];
+
+    const eventRouter: EventRouter = {
+      register: (eventKey, handler) => {
+        handlers.set(eventKey, handler);
+      },
+      dispatch: async () => undefined,
+    };
+
+    const jobQueue: JobQueue = {
+      enqueue: async <T>(installationId: number) => {
+        enqueued.push({ installationId });
+        return undefined as T;
+      },
+      getQueueSize: () => 0,
+      getPendingCount: () => 0,
+    };
+
+    createReviewHandler({
+      eventRouter,
+      jobQueue,
+      workspaceManager: {} as WorkspaceManager,
+      githubApp: { getAppSlug: () => "kodiai" } as GitHubApp,
+      executor: {} as never,
+      logger: createNoopLogger(),
+    });
+
+    const handler = handlers.get("pull_request.review_requested");
+    expect(handler).toBeDefined();
+
+    await handler!(
+      buildReviewRequestedEvent({
+        requested_team: { name: "ai-review", slug: "ai-review" },
+      }),
+    );
+
+    expect(enqueued).toHaveLength(1);
+    expect(enqueued[0]?.installationId).toBe(42);
+  });
+
   test("skips malformed reviewer payloads without throwing", async () => {
     const handlers = new Map<string, (event: WebhookEvent) => Promise<void>>();
     let enqueueCount = 0;

src/handlers/review.ts

@@ -44,6 +44,8 @@ function normalizeReviewerLogin(login: string): string {
  *
  * Trigger model: initial review events plus explicit re-request only.
  * Re-requested reviews run only when kodiai itself is the requested reviewer.
+ * Additionally, a team-based re-request is supported for the special team slug/name "ai-review"
+ * to enable UI-only re-review without a comment.
  * Clones the repo, builds a review prompt, runs Claude via the executor,
  * and optionally submits a silent approval if no issues were found.
  */
@@ -57,6 +59,8 @@ export function createReviewHandler(deps: {
 }): void {
   const { eventRouter, jobQueue, workspaceManager, githubApp, executor, logger } = deps;
 
+  const rereviewTeamSlugs = new Set(["ai-review"]);
+
   async function handleReview(event: WebhookEvent): Promise<void> {
     const payload = event.payload as unknown as
       | PullRequestOpenedEvent
@@ -110,6 +114,10 @@ export function createReviewHandler(deps: {
         typeof requestedTeam?.name === "string"
           ? requestedTeam.name
           : undefined;
+      const requestedTeamSlug =
+        typeof (requestedTeam as { slug?: unknown } | undefined)?.slug === "string"
+          ? (requestedTeam as { slug: string }).slug
+          : undefined;
       const appSlug = githubApp.getAppSlug();
       const normalizedAppSlug = normalizeReviewerLogin(appSlug);
 
@@ -144,18 +152,38 @@ export function createReviewHandler(deps: {
           "Accepted review_requested event for kodiai reviewer",
         );
       } else if (requestedTeamName) {
+        const normalizedTeamName = requestedTeamName.trim().toLowerCase();
+        const normalizedTeamSlug = (requestedTeamSlug ?? "").trim().toLowerCase();
+        const matchedTeam = rereviewTeamSlugs.has(normalizedTeamSlug) || rereviewTeamSlugs.has(normalizedTeamName);
+
+        if (!matchedTeam) {
+          logger.info(
+            {
+              ...baseLog,
+              gate: "review_requested_reviewer",
+              gateResult: "skipped",
+              skipReason: "team-only-request",
+              requestedReviewer: null,
+              requestedTeam: requestedTeamName,
+              requestedTeamSlug: requestedTeamSlug ?? null,
+            },
+            "Skipping review_requested event because only a non-rereview team was requested",
+          );
+          return;
+        }
+
         logger.info(
           {
             ...baseLog,
             gate: "review_requested_reviewer",
-            gateResult: "skipped",
-            skipReason: "team-only-request",
+            gateResult: "accepted",
             requestedReviewer: null,
             requestedTeam: requestedTeamName,
+            requestedTeamSlug: requestedTeamSlug ?? null,
+            rereviewTeam: true,
           },
-          "Skipping review_requested event because only a team was requested",
+          "Accepted review_requested event for rereview team",
         );
-        return;
       } else {
         logger.warn(
           {