update

Author: KarhouTam

torchci/lib/bot/crcrOncallBot.ts

@@ -81,30 +81,13 @@ export default function crcrOncallBot(app: Probot): void {
 
     // Get the PRs this check run belongs to.
     let prNumbers: number[] = [];
-    if (checkRun.output?.summary) {
+    if (checkRun.pull_requests && checkRun.pull_requests.length > 0) {
+      prNumbers = checkRun.pull_requests.map((pr) => pr.number);
+    } else if (checkRun.output?.summary) {
       const match = checkRun.output.summary.match(/for PR (\d+)/);
       if (match) {
         prNumbers = [parseInt(match[1], 10)];
       }
-    } else if (checkRun.pull_requests && checkRun.pull_requests.length > 0) {
-      prNumbers = checkRun.pull_requests.map((pr) => pr.number);
-    }
-
-    // Fall back to Search API if still no PR found (e.g., pr_number was empty
-    // on the Lambda side).
-    if (prNumbers.length === 0 && checkRun.head_sha) {
-      try {
-        const result = await ctx.octokit.rest.search.issuesAndPullRequests({
-          q: `${checkRun.head_sha} type:pr repo:${owner}/${repo}`,
-        });
-        prNumbers = result.data.items.map((item: any) => item.number);
-      } catch (err) {
-        ctx.log(
-          { err },
-          `crcrOncall: failed to resolve PRs for commit ${checkRun.head_sha}, skipping`
-        );
-        return;
-      }
     }
 
     if (prNumbers.length === 0) {

torchci/test/crcrOncallBot.test.ts

@@ -177,13 +177,8 @@ L3:
   });
 
   test("does not comment when output has no PR number", async () => {
-    // output.summary exists but regex doesn't match (empty pr_number);
-    // falls through to Search API which also returns nothing
-    const scope = nock("https://api.github.com")
-      .get("/search/issues")
-      .query(true)
-      .reply(200, { total_count: 0, items: [] });
-
+    // output.summary exists but regex doesn't match (no PR number
+    // after "for PR"), so the bot returns early without any API call.
     await probot.receive({
       name: "check_run" as any,
       payload: checkRunPayload({
@@ -195,7 +190,6 @@ L3:
       }) as any,
       id: "8",
     });
-    handleScope(scope);
   });
 
   test("posts comment when output contains PR number for cross-fork PR", async () => {