Remove Mergeable check and fix Devin status polling (#2072)

- Remove bot-ci.ts which created the 'Mergeable: bot_ci' check
- Remove related tests and fixtures for bot_ci handler
- Fix Devin status polling to reconcile stuck checks on startup
  - Now fetches all sessions (not just running) on startup
  - Updates checks for finished/expired sessions immediately
  - Only tracks working/blocked sessions for ongoing polling
- Update vitest config to pass with no tests

Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>

Author: yujonglee

apps/bot/src/devin/poller.ts

@@ -168,7 +168,7 @@ export class DevinStatusPoller {
     this.logger.info("Discovering existing Devin sessions with open PRs...");
 
     try {
-      const { sessions } = await listDevinSessions({ status: "running" });
+      const { sessions } = await listDevinSessions({});
 
       for (const session of sessions) {
         if (!session.pull_request?.url) {
@@ -182,9 +182,6 @@ export class DevinStatusPoller {
         }
 
         const detail = await getDevinSessionDetail(session.session_id);
-        if (detail.status_enum !== DevinSessionStatus.Working) {
-          continue;
-        }
 
         try {
           const octokit = await this.createOctokit();
@@ -198,15 +195,55 @@ export class DevinStatusPoller {
             continue;
           }
 
-          this.trackPR({
+          const trackedPR: TrackedPR = {
             owner: parsed.owner,
             repo: parsed.repo,
             prNumber: parsed.prNumber,
             prUrl,
             headSha: pr.head.sha,
             sessionId: session.session_id,
             addedAt: Date.now(),
-          });
+          };
+
+          if (
+            detail.status_enum === DevinSessionStatus.Working ||
+            detail.status_enum === DevinSessionStatus.Blocked
+          ) {
+            this.trackPR(trackedPR);
+          } else if (detail.status_enum === DevinSessionStatus.Finished) {
+            await this.updateCheckStatus(
+              trackedPR,
+              "completed",
+              "success",
+              {
+                title: "Devin finished",
+                summary: `Devin session ${session.session_id} has completed.`,
+              },
+              session.session_id,
+            );
+          } else if (detail.status_enum === DevinSessionStatus.Expired) {
+            await this.updateCheckStatus(
+              trackedPR,
+              "completed",
+              "cancelled",
+              {
+                title: "Devin session expired",
+                summary: `Devin session ${session.session_id} has expired.`,
+              },
+              session.session_id,
+            );
+          } else if (detail.status_enum) {
+            await this.updateCheckStatus(
+              trackedPR,
+              "completed",
+              "neutral",
+              {
+                title: "Devin session ended",
+                summary: `Devin session ${session.session_id} ended with status: ${detail.status_enum}`,
+              },
+              session.session_id,
+            );
+          }
         } catch {
           this.logger.error(`Failed to fetch PR details for ${prUrl}`);
         }

apps/bot/src/features/bot-ci.ts

@@ -1,4 +1,3 @@
-export * from "./bot-ci.js";
 export * from "./devin-status.js";
 export * from "./fix-merge-conflict.js";
 export * from "./pr-closed.js";

apps/bot/src/features/index.ts

@@ -2,7 +2,6 @@ import { ApplicationFunctionOptions, Probot } from "probot";
 
 import { startDevinStatusPoller } from "./devin/index.js";
 import {
-  registerBotCiHandler,
   registerDevinStatusHandler,
   registerFixMergeConflictHandler,
   registerPrClosedHandler,
@@ -17,13 +16,10 @@ export default (app: Probot, { getRouter }: ApplicationFunctionOptions) => {
     });
   }
 
-  // Start the poller before registering handlers to avoid race conditions
-  // where handlers might call getDevinStatusPoller() before it's initialized
   if (process.env.NODE_ENV !== "test") {
     startDevinStatusPoller(app);
   }
 
-  registerBotCiHandler(app);
   registerDevinStatusHandler(app);
   registerFixMergeConflictHandler(app);
   registerPrClosedHandler(app);