fix: replace searchAPI usage with GraphQL in findSRIssue util

lib/definitions/constants.js

@@ -1,3 +1,5 @@
 export const ISSUE_ID = "<!-- semantic-release:github -->";
 
 export const RELEASE_NAME = "GitHub release";
+
+export const RELEASE_FAIL_LABEL = "release-failing";

lib/fail.js

@@ -2,7 +2,7 @@ import { template } from "lodash-es";
 import debugFactory from "debug";
 
 import parseGithubUrl from "./parse-github-url.js";
-import { ISSUE_ID } from "./definitions/constants.js";
+import { ISSUE_ID, RELEASE_FAIL_LABEL } from "./definitions/constants.js";
 import resolveConfig from "./resolve-config.js";
 import { toOctokitOptions } from "./octokit.js";
 import findSRIssues from "./find-sr-issues.js";
@@ -57,7 +57,14 @@ export default async function fail(pluginConfig, context, { Octokit }) {
     const body = failComment
       ? template(failComment)({ branch, errors })
       : getFailComment(branch, errors);
-    const [srIssue] = await findSRIssues(octokit, failTitle, owner, repo);
+    const [srIssue] = await findSRIssues(
+      octokit,
+      logger,
+      failTitle,
+      labels,
+      owner,
+      repo,
+    );
 
     const canCommentOnOrCreateIssue = failCommentCondition
       ? template(failCommentCondition)({ ...context, issue: srIssue })
@@ -85,7 +92,7 @@ export default async function fail(pluginConfig, context, { Octokit }) {
         repo,
         title: failTitle,
         body: `${body}\n\n${ISSUE_ID}`,
-        labels: labels || [],
+        labels: (labels || []).concat([RELEASE_FAIL_LABEL]),
         assignees,
       };
       debug("create issue: %O", newIssue);

lib/find-sr-issues.js

@@ -1,11 +1,61 @@
-import { ISSUE_ID } from "./definitions/constants.js";
+import { uniqBy } from "lodash-es";
+import { ISSUE_ID, RELEASE_FAIL_LABEL } from "./definitions/constants.js";
+
+export default async (octokit, logger, title, labels, owner, repo) => {
+  let issues = [];
 
-export default async (octokit, title, owner, repo) => {
   const {
-    data: { items: issues },
-  } = await octokit.request("GET /search/issues", {
-    q: `in:title+repo:${owner}/${repo}+type:issue+state:open+${title}`,
+    repository: {
+      issues: { nodes: issueNodes },
+    },
+  } = await octokit.graphql(loadGetSRIssuesQuery, {
+    owner,
+    repo,
+    filter: {
+      labels: (labels || []).concat([RELEASE_FAIL_LABEL]),
+    },
   });
 
-  return issues.filter((issue) => issue.body && issue.body.includes(ISSUE_ID));
+  issues.push(...issueNodes);
+
+  /**
+   * BACKWARD COMPATIBILITY: Fallback to the search API if the issue was not found in the GraphQL response.
+   * This fallback will be removed in a future release
+   */
+  try {
+    const {
+      data: { items: backwardIssues },
+    } = await octokit.request("GET /search/issues", {
+      q: `in:title+repo:${owner}/${repo}+type:issue+state:open+${title}`,
+    });
+    issues.push(...backwardIssues);
+  } catch (error) {
+    logger.log(
+      "An error occured fetching issue via fallback (with GH SearchAPI)",
+    );
+  }
+
+  const uniqueSRIssues = uniqBy(
+    issues.filter((issue) => issue.body && issue.body.includes(ISSUE_ID)),
+    "number",
+  );
+
+  return uniqueSRIssues;
 };
+
+/**
+ * GraphQL Query to et the semantic-release issues for a repository.
+ */
+const loadGetSRIssuesQuery = `#graphql
+  query getSRIssues($owner: String!, $repo: String!, $filter: IssueFilters) {
+    repository(owner: $owner, name: $repo) {
+      issues(first: 100, states: OPEN, filterBy: $filter) {
+        nodes {
+          number
+          title
+          body
+        }
+      }
+    }
+  }
+`;

lib/resolve-config.js

@@ -38,6 +38,7 @@ export default function resolveConfig(
       : failTitle,
     failComment,
     failCommentCondition,
+    // ATTN??? We opinionated :(???
     labels: isNil(labels)
       ? ["semantic-release"]
       : labels === false

lib/success.js

@@ -28,6 +28,7 @@ export default async function success(pluginConfig, context, { Octokit }) {
     githubApiPathPrefix,
     githubApiUrl,
     proxy,
+    labels,
     successComment,
     successCommentCondition,
     failTitle,
@@ -266,7 +267,14 @@ export default async function success(pluginConfig, context, { Octokit }) {
   if (failComment === false || failTitle === false) {
     logger.log("Skip closing issue.");
   } else {
-    const srIssues = await findSRIssues(octokit, failTitle, owner, repo);
+    const srIssues = await findSRIssues(
+      octokit,
+      logger,
+      failTitle,
+      labels,
+      owner,
+      repo,
+    );
 
     debug("found semantic-release issues: %O", srIssues);
 

test/fail.test.js

@@ -3,7 +3,7 @@ import sinon from "sinon";
 import test from "ava";
 import fetchMock from "fetch-mock";
 
-import { ISSUE_ID } from "../lib/definitions/constants.js";
+import { ISSUE_ID, RELEASE_FAIL_LABEL } from "../lib/definitions/constants.js";
 import { TestOctokit } from "./helpers/test-octokit.js";
 
 /* eslint camelcase: ["error", {properties: "never"}] */
@@ -36,6 +36,13 @@ test("Open a new issue with the list of errors", async (t) => {
     .getOnce("https://api.github.local/repos/test_user/test_repo", {
       full_name: `${redirectedOwner}/${redirectedRepo}`,
     })
+    .postOnce("https://api.github.local/graphql", {
+      data: {
+        repository: {
+          issues: { nodes: [] },
+        },
+      },
+    })
     .getOnce(
       `https://api.github.local/search/issues?q=${encodeURIComponent(
         "in:title",
@@ -59,7 +66,7 @@ test("Open a new issue with the list of errors", async (t) => {
           data.body,
           /---\n\n### Error message 1\n\nError 1 details\n\n---\n\n### Error message 2\n\nError 2 details\n\n---\n\n### Error message 3\n\nError 3 details\n\n---/,
         );
-        t.deepEqual(data.labels, ["semantic-release"]);
+        t.deepEqual(data.labels, ["semantic-release", RELEASE_FAIL_LABEL]);
         return true;
       },
       {
@@ -117,6 +124,13 @@ test("Open a new issue with the list of errors and custom title and comment", as
       full_name: `${owner}/${repo}`,
       clone_url: `https://api.github.local/${owner}/${repo}.git`,
     })
+    .postOnce("https://api.github.local/graphql", {
+      data: {
+        repository: {
+          issues: { nodes: [] },
+        },
+      },
+    })
     .getOnce(
       `https://api.github.local/search/issues?q=${encodeURIComponent(
         "in:title",
@@ -132,7 +146,7 @@ test("Open a new issue with the list of errors and custom title and comment", as
         body: {
           title: failTitle,
           body: `branch master Error message 1 Error message 2 Error message 3\n\n${ISSUE_ID}`,
-          labels: ["semantic-release"],
+          labels: ["semantic-release", RELEASE_FAIL_LABEL],
         },
       },
     );
@@ -185,6 +199,13 @@ test("Open a new issue with assignees and the list of errors", async (t) => {
       full_name: `${owner}/${repo}`,
       clone_url: `https://api.github.local/${owner}/${repo}.git`,
     })
+    .postOnce("https://api.github.local/graphql", {
+      data: {
+        repository: {
+          issues: { nodes: [] },
+        },
+      },
+    })
     .getOnce(
       `https://api.github.local/search/issues?q=${encodeURIComponent(
         "in:title",
@@ -203,7 +224,7 @@ test("Open a new issue with assignees and the list of errors", async (t) => {
           data.body,
           /---\n\n### Error message 1\n\nError 1 details\n\n---\n\n### Error message 2\n\nError 2 details\n\n---/,
         );
-        t.deepEqual(data.labels, ["semantic-release"]);
+        t.deepEqual(data.labels, ["semantic-release", RELEASE_FAIL_LABEL]);
         t.deepEqual(data.assignees, ["user1", "user2"]);
         return true;
       },
@@ -258,6 +279,13 @@ test("Open a new issue without labels and the list of errors", async (t) => {
       full_name: `${owner}/${repo}`,
       clone_url: `https://api.github.local/${owner}/${repo}.git`,
     })
+    .postOnce("https://api.github.local/graphql", {
+      data: {
+        repository: {
+          issues: { nodes: [] },
+        },
+      },
+    })
     .getOnce(
       `https://api.github.local/search/issues?q=${encodeURIComponent(
         "in:title",
@@ -276,7 +304,7 @@ test("Open a new issue without labels and the list of errors", async (t) => {
           data.body,
           /---\n\n### Error message 1\n\nError 1 details\n\n---\n\n### Error message 2\n\nError 2 details\n\n---/,
         );
-        t.deepEqual(data.labels, []);
+        t.deepEqual(data.labels, [RELEASE_FAIL_LABEL]);
         return true;
       },
       { html_url: "https://github.com/issues/1", number: 1 },
@@ -335,6 +363,13 @@ test("Update the first existing issue with the list of errors", async (t) => {
       full_name: `${owner}/${repo}`,
       clone_url: `https://api.github.local/${owner}/${repo}.git`,
     })
+    .postOnce("https://api.github.local/graphql", {
+      data: {
+        repository: {
+          issues: { nodes: issues },
+        },
+      },
+    })
     .getOnce(
       `https://api.github.local/search/issues?q=${encodeURIComponent(
         "in:title",
@@ -501,6 +536,18 @@ test('Does not post comments on existing issues when "failCommentCondition" is "
     .getOnce(`https://api.github.local/repos/${owner}/${repo}`, {
       full_name: `${owner}/${repo}`,
     })
+    .postOnce(
+      (url, { body }) =>
+        url === "https://api.github.local/graphql" &&
+        JSON.parse(body).query.includes("query getSRIssues("),
+      {
+        data: {
+          repository: {
+            issues: { nodes: issues },
+          },
+        },
+      },
+    )
     .getOnce(
       `https://api.github.local/search/issues?q=${encodeURIComponent(
         "in:title",
@@ -551,6 +598,18 @@ test(`Post new issue if none exists yet, but don't comment on existing issues wh
     .getOnce(`https://api.github.local/repos/${owner}/${repo}`, {
       full_name: `${owner}/${repo}`,
     })
+    .postOnce(
+      (url, { body }) =>
+        url === "https://api.github.local/graphql" &&
+        JSON.parse(body).query.includes("query getSRIssues("),
+      {
+        data: {
+          repository: {
+            issues: { nodes: [] },
+          },
+        },
+      },
+    )
     .getOnce(
       `https://api.github.local/search/issues?q=${encodeURIComponent(
         "in:title",