Build/Test Tools: Remove in:body from pull request search query.

When searching for pull requests using the GraphQL API, `in:body` is not being respected in search queries. Because it’s common for Trac tickets to be referenced in the discussions happening in comments and reviews, some pull requests are being closed out incorrectly.

This removes `in:body` from the search query introduced in [59661] and instead checks the `bodyText` for references to the fixed tickets.

The `state` field is also being removed because it is unused.

Props dd32, peterwilsoncc, johnbillion, joedolson, westonruter, dmsnell.
Fixes #64175.

git-svn-id: https://develop.svn.wordpress.org/trunk@61100 602fd350-edb4-49c9-b593-d223f7449a82

Author: desrosj

.github/workflows/reusable-cleanup-pull-requests.yml

@@ -54,28 +54,36 @@ jobs:
 
             for (const ticket of fixedList) {
               const tracTicketUrl = `https://core.trac.wordpress.org/ticket/${ticket}`;
-              const corePrefix = `Core-${ticket}`;
+              const corePrefix = `core-${ticket}`;
 
               const query = `
                 query($searchQuery: String!) {
                   search(query: $searchQuery, type: ISSUE_ADVANCED, first: 20) {
                     nodes {
                       ... on PullRequest {
                         number
-                        state
+                        bodyText
                       }
                     }
                   }
                 }
               `;
 
-              const searchQuery = `repo:${context.repo.owner}/${context.repo.repo} is:pr is:open in:body ( "${tracTicketUrl}" OR "${corePrefix}" )`;
+              const searchQuery = `repo:${context.repo.owner}/${context.repo.repo} is:pr is:open ( "${tracTicketUrl}" OR "${corePrefix}" )`;
 
               const result = await github.graphql(query, {
                 searchQuery,
               });
 
-              prNumbers.push(...result.search.nodes.map(pr => pr.number));
+              // Since search queries will match anywhere for any activity on a pull request, the body specifically needs to be manually checked.
+              const matchingPRs = result.search.nodes
+                .filter(
+                  const bodyLower = pr.bodyText.toLowerCase();
+                  pr => pr.bodyText.includes(tracTicketUrl) || pr.bodyText.includes(corePrefix)
+                )
+                .map(pr => pr.number);
+
+              prNumbers.push(...matchingPRs);
             }
 
             return prNumbers;