update

Author: KarhouTam

torchci/lib/bot/crcrOncallBot.ts

@@ -80,31 +80,22 @@ export default function crcrOncallBot(app: Probot): void {
     }
 
     // Get the PRs this check run belongs to.
+    // checkRun.pull_requests is empty for cross-fork PRs (most pytorch
+    // contributions), so fall back to extracting the PR number from the
+    // check run's output.summary, which the Lambda side embeds in the format
+    // "{downstream_repo} workflow for PR {pr_number}: {details_url}".
+    // If neither yields a PR, fall back to the Search API as a last resort.
+    // The commits-pulls API (listPullRequestsAssociatedWithCommit) also returns
+    // empty for cross-fork PRs on large repos like pytorch/pytorch, but the
+    // Search API indexes commits across forks and reliably finds them.
     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) {