Hotfix: check org and repo for linked issue

src/handlers/assign/auto.ts

@@ -18,43 +18,45 @@ export const checkPullRequests = async () => {
 
   // Loop through the pull requests and assign them to their respective issues if needed
   for (const pull of pulls) {
-    const linkedIssue = await gitLinkedIssueParser({
+    const linkedIssues = await gitLinkedIssueParser({
       owner: payload.repository.owner.login,
       repo: payload.repository.name,
       pull_number: pull.number,
     });
 
-    // if pullRequestLinked is empty, continue
-    if (linkedIssue == "" || !pull.user || !linkedIssue) {
-      continue;
-    }
+    for (const linkedIssue of linkedIssues) {
+      // if pullRequestLinked is empty, continue
+      if (linkedIssue == "" || !pull.user || !linkedIssue) {
+        continue;
+      }
 
-    const connectedPull = await getPullByNumber(context, pull.number);
+      const connectedPull = await getPullByNumber(context, pull.number);
 
-    // Newly created PULL (draft or direct) pull does have same `created_at` and `updated_at`.
-    if (connectedPull?.created_at !== connectedPull?.updated_at) {
-      logger.debug("It's an updated Pull Request, reverting");
-      continue;
-    }
+      // The new PR, whether it's in draft or in direct form, it has identical `created_at` and `updated_at` timestamps.
+      if (connectedPull?.created_at !== connectedPull?.updated_at) {
+        logger.debug("Skipping because it's not a new PR");
+        continue;
+      }
 
-    const linkedIssueNumber = linkedIssue.substring(linkedIssue.lastIndexOf("/") + 1);
+      const linkedIssueNumber = linkedIssue.substring(linkedIssue.lastIndexOf("/") + 1);
 
-    // Check if the pull request opener is assigned to the issue
-    const opener = pull.user.login;
+      // Check if the pull request opener is assigned to the issue
+      const opener = pull.user.login;
 
-    const issue = await getIssueByNumber(context, +linkedIssueNumber);
-    if (!issue?.assignees) continue;
+      const issue = await getIssueByNumber(context, +linkedIssueNumber);
+      if (!issue?.assignees) continue;
 
-    // if issue is already assigned, continue
-    if (issue.assignees.length > 0) {
-      logger.debug(`Issue already assigned, ignoring...`);
-      continue;
-    }
+      // if issue is already assigned, continue
+      if (issue.assignees.length > 0) {
+        logger.debug(`Issue already assigned, ignoring...`);
+        continue;
+      }
 
-    const assignedUsernames = issue.assignees.map((assignee) => assignee.login);
-    if (!assignedUsernames.includes(opener)) {
-      await addAssignees(+linkedIssueNumber, [opener]);
-      logger.debug(`Assigned pull request #${pull.number} opener to issue ${linkedIssueNumber}.`);
+      const assignedUsernames = issue.assignees.map((assignee) => assignee.login);
+      if (!assignedUsernames.includes(opener)) {
+        await addAssignees(+linkedIssueNumber, [opener]);
+        logger.debug(`Assigned pull request #${pull.number} opener to issue ${linkedIssueNumber}.`);
+      }
     }
   }
 };

src/helpers/parser.ts

@@ -19,22 +19,38 @@ export interface LinkedPR {
 
 export const gitLinkedIssueParser = async ({ owner, repo, pull_number }: GitParser) => {
   const logger = getLogger();
+  const linkedIssueUrls = [];
   try {
     const { data } = await axios.get(`https://github.com/${owner}/${repo}/pull/${pull_number}`);
     const dom = parse(data);
     const devForm = dom.querySelector("[data-target='create-branch.developmentForm']") as HTMLElement;
     const linkedIssues = devForm.querySelectorAll(".my-1");
 
-    if (linkedIssues.length === 0) {
-      return null;
-    }
+    if (linkedIssues.length === 0) return [];
+
+    for (const linkedIssue of linkedIssues) {
+      const issueUrl = linkedIssue.querySelector("a")?.attrs?.href;
+
+      if (!issueUrl) continue;
+
+      const parts = issueUrl.split("/");
+
+      // check if array size is at least 4
+      if (parts.length < 4) continue;
+
+      // extract the organization name and repo name from the link:(e.g. "
+      const issueOrganization = parts[parts.length - 4];
+      const issueRepository = parts[parts.length - 3];
 
-    const issueUrl = linkedIssues[0].querySelector("a")?.attrs?.href || "";
-    return issueUrl;
+      if (`${issueOrganization}/${issueRepository}` !== `${owner}/${repo}`) continue;
+
+      linkedIssueUrls.push(issueUrl);
+    }
   } catch (error) {
     logger.error(`${JSON.stringify(error)}`);
-    return null;
   }
+
+  return linkedIssueUrls;
 };
 
 export const gitLinkedPrParser = async ({ owner, repo, issue_number }: GitParser): Promise<LinkedPR[]> => {