fix: improve Devin check status with session URL and blocked state handling (#2081)

- Add Devin session URL to check summary for easy access from GitHub check page
- Add isDevinSessionActive helper to handle both 'working' and 'blocked' states consistently
- Update devin-status.ts to use isDevinSessionActive instead of isDevinSessionWorking
- Show appropriate title ('Devin is working' vs 'Devin is blocked') based on session state

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

Author: yujonglee

apps/bot/src/devin/detail.ts

@@ -45,3 +45,10 @@ export async function getDevinSessionDetail(
 export function isDevinSessionWorking(detail: DevinSessionDetail): boolean {
   return detail.status_enum === DevinSessionStatus.Working;
 }
+
+export function isDevinSessionActive(detail: DevinSessionDetail): boolean {
+  return (
+    detail.status_enum === DevinSessionStatus.Working ||
+    detail.status_enum === DevinSessionStatus.Blocked
+  );
+}

apps/bot/src/devin/poller.ts

@@ -211,35 +211,38 @@ export class DevinStatusPoller {
           ) {
             this.trackPR(trackedPR);
           } else if (detail.status_enum === DevinSessionStatus.Finished) {
+            const sessionUrl = `https://app.devin.ai/sessions/${session.session_id}`;
             await this.updateCheckStatus(
               trackedPR,
               "completed",
               "success",
               {
                 title: "Devin finished",
-                summary: `Devin session ${session.session_id} has completed.`,
+                summary: `Devin session has completed.\n\nView session: ${sessionUrl}`,
               },
               session.session_id,
             );
           } else if (detail.status_enum === DevinSessionStatus.Expired) {
+            const sessionUrl = `https://app.devin.ai/sessions/${session.session_id}`;
             await this.updateCheckStatus(
               trackedPR,
               "completed",
               "cancelled",
               {
                 title: "Devin session expired",
-                summary: `Devin session ${session.session_id} has expired.`,
+                summary: `Devin session has expired.\n\nView session: ${sessionUrl}`,
               },
               session.session_id,
             );
           } else if (detail.status_enum) {
+            const sessionUrl = `https://app.devin.ai/sessions/${session.session_id}`;
             await this.updateCheckStatus(
               trackedPR,
               "completed",
               "neutral",
               {
                 title: "Devin session ended",
-                summary: `Devin session ${session.session_id} ended with status: ${detail.status_enum}`,
+                summary: `Devin session ended with status: ${detail.status_enum}\n\nView session: ${sessionUrl}`,
               },
               session.session_id,
             );
@@ -336,14 +339,15 @@ export class DevinStatusPoller {
       // Verify the actual session status before marking as complete
       try {
         const detail = await getDevinSessionDetail(pr.sessionId);
+        const sessionUrl = `https://app.devin.ai/sessions/${pr.sessionId}`;
         if (detail.status_enum === DevinSessionStatus.Finished) {
           await this.updateCheckStatus(
             pr,
             "completed",
             "success",
             {
               title: "Devin finished",
-              summary: `Devin session ${pr.sessionId} has completed.`,
+              summary: `Devin session has completed.\n\nView session: ${sessionUrl}`,
             },
             pr.sessionId,
           );
@@ -354,7 +358,7 @@ export class DevinStatusPoller {
             "cancelled",
             {
               title: "Devin session expired",
-              summary: `Devin session ${pr.sessionId} has expired.`,
+              summary: `Devin session has expired.\n\nView session: ${sessionUrl}`,
             },
             pr.sessionId,
           );
@@ -368,7 +372,7 @@ export class DevinStatusPoller {
             "neutral",
             {
               title: "Devin session ended",
-              summary: `Devin session ${pr.sessionId} ended with status: ${detail.status_enum}`,
+              summary: `Devin session ended with status: ${detail.status_enum}\n\nView session: ${sessionUrl}`,
             },
             pr.sessionId,
           );
@@ -383,6 +387,7 @@ export class DevinStatusPoller {
     }
 
     const detail = await getDevinSessionDetail(session.session_id);
+    const sessionUrl = `https://app.devin.ai/sessions/${session.session_id}`;
 
     if (detail.status_enum === DevinSessionStatus.Working) {
       await this.updateCheckStatus(
@@ -391,7 +396,7 @@ export class DevinStatusPoller {
         undefined,
         {
           title: "Devin is working",
-          summary: `Devin session ${session.session_id} is currently working on this PR.`,
+          summary: `Devin session is currently working on this PR.\n\nView session: ${sessionUrl}`,
         },
         session.session_id,
       );
@@ -405,7 +410,7 @@ export class DevinStatusPoller {
         undefined,
         {
           title: "Devin is blocked",
-          summary: `Devin session ${session.session_id} is blocked and waiting for input.`,
+          summary: `Devin session is blocked and waiting for input.\n\nView session: ${sessionUrl}`,
         },
         session.session_id,
       );
@@ -419,7 +424,7 @@ export class DevinStatusPoller {
         "success",
         {
           title: "Devin finished",
-          summary: `Devin session ${session.session_id} has completed.`,
+          summary: `Devin session has completed.\n\nView session: ${sessionUrl}`,
         },
         session.session_id,
       );
@@ -434,7 +439,7 @@ export class DevinStatusPoller {
         "cancelled",
         {
           title: "Devin session expired",
-          summary: `Devin session ${session.session_id} has expired.`,
+          summary: `Devin session has expired.\n\nView session: ${sessionUrl}`,
         },
         session.session_id,
       );

apps/bot/src/features/devin-status.ts

@@ -1,10 +1,11 @@
 import { Probot } from "probot";
 
 import {
+  DevinSessionStatus,
   findRunningSessionForPR,
   getDevinSessionDetail,
   getDevinStatusPoller,
-  isDevinSessionWorking,
+  isDevinSessionActive,
 } from "../devin/index.js";
 import { createOrUpdateCheck, ProbotContext } from "../github/check.js";
 
@@ -57,7 +58,7 @@ async function checkDevinSession(
   }
 
   const detail = await getDevinSessionDetail(session.session_id);
-  if (!isDevinSessionWorking(detail)) {
+  if (!isDevinSessionActive(detail)) {
     return;
   }
 
@@ -74,16 +75,21 @@ async function checkDevinSession(
     });
   }
 
+  const sessionUrl = `https://app.devin.ai/sessions/${session.session_id}`;
+  const isBlocked = detail.status_enum === DevinSessionStatus.Blocked;
+
   await createOrUpdateCheck(context, {
     owner,
     repo,
     name: CHECK_NAME,
     head_sha: headSha,
     status: "in_progress",
-    details_url: `https://app.devin.ai/sessions/${session.session_id}`,
+    details_url: sessionUrl,
     output: {
-      title: "Devin is working",
-      summary: `Devin session ${session.session_id} is currently working on this PR.`,
+      title: isBlocked ? "Devin is blocked" : "Devin is working",
+      summary: isBlocked
+        ? `Devin session is blocked and waiting for input.\n\nView session: ${sessionUrl}`
+        : `Devin session is currently working on this PR.\n\nView session: ${sessionUrl}`,
     },
   });
 }