@bigbinary/neeto-playwright-commons 4.8.1 → 4.8.2

package/index.d.ts

@@ -10085,8 +10085,8 @@ declare const currencyUtils: {
  * test("should fail because of a bug in the product", async () => {
  *   test.fail(
  *     await isGithubIssueOpen([
- *       "github.com/bigbinary/neeto-app-web/1234",
- *       "github.com/bigbinary/neeto-app-web/1235",
+ *       "https://github.com/bigbinary/neeto-app-web/issues/1234",
+ *       "https://github.com/bigbinary/neeto-app-web/issues/1235",
  *     ]),
  *     "This test will fail until these bugs are fixed."
  *   );
@@ -10097,8 +10097,8 @@ declare const currencyUtils: {
  * test("should fail because of a bug in the product", async () => {
  *   test.fixme(
  *     await isGithubIssueOpen([
- *       "github.com/bigbinary/neeto-app-web/1234",
- *       "github.com/bigbinary/neeto-app-web/1235",
+ *       "https://github.com/bigbinary/neeto-app-web/issues/1234",
+ *       "https://github.com/bigbinary/neeto-app-web/issues/1235",
  *     ]),
  *     "This test should be fixed until these bugs are fixed."
  *   );

package/index.js

@@ -119020,26 +119020,38 @@ class Octokit {
   auth;
 }
 
-const isGithubIssueOpen = async (issueLinks) => {
-    const octokit = new Octokit({
-        auth: process.env.GITHUB_TOKEN,
+const issueStatusCache = new Map();
+let octokit;
+const getOctokit = () => (octokit ??= new Octokit({ auth: process.env.GITHUB_TOKEN }));
+const fetchIsIssueOpen = async (owner, repo, issueNumber) => {
+    const { data } = await getOctokit().request("GET /repos/{owner}/{repo}/issues/{issue_number}", {
+        owner,
+        repo,
+        issue_number: issueNumber,
+        headers: { "X-GitHub-Api-Version": "2022-11-28" },
     });
-    const issueStatuses = await Promise.all(issueLinks.map(issueLink => {
-        const issueSuffix = issueLink.split("github.com/").at(-1);
-        console.log(issueSuffix);
-        if (!issueSuffix)
-            return null;
-        const [owner, repo, _, issueNumber] = issueSuffix.split("/");
-        return octokit.request(`GET /repos/{owner}/{repo}/issues/{issueNumber}`, {
-            owner,
-            repo,
-            issueNumber,
-            headers: {
-                "X-GitHub-Api-Version": "2022-11-28",
-            },
-        });
-    }));
-    return issueStatuses.some(issueStatus => issueStatus?.data?.state === "open");
+    return data.state === "open";
+};
+const isIssueOpen = (issueLink) => {
+    const [owner, repo, ...rest] = (issueLink.split(/[?#]/)[0].split("github.com/").at(-1) ?? "")
+        .split("/")
+        .filter(Boolean);
+    const issueNumber = Number(rest.at(-1));
+    if (!owner || !repo || !Number.isInteger(issueNumber)) {
+        throw new Error(`[isGithubIssueOpen] Invalid GitHub issue link: ${issueLink}`);
+    }
+    const cacheKey = `${owner}/${repo}#${issueNumber}`;
+    const cachedStatus = issueStatusCache.get(cacheKey);
+    if (cachedStatus)
+        return cachedStatus;
+    const status = fetchIsIssueOpen(owner, repo, issueNumber);
+    issueStatusCache.set(cacheKey, status);
+    status.catch(() => issueStatusCache.delete(cacheKey));
+    return status;
+};
+const isGithubIssueOpen = async (issueLinks) => {
+    const issueStatuses = await Promise.all(issueLinks.map(isIssueOpen));
+    return issueStatuses.some(Boolean);
 };
 
 class HelpAndProfilePage {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bigbinary/neeto-playwright-commons",
-  "version": "4.8.1",
+  "version": "4.8.2",
   "description": "A package encapsulating common playwright code across neeto projects.",
   "repository": "git@github.com:bigbinary/neeto-playwright-commons.git",
   "license": "apache-2.0",