fix(ng-dev): close linked issues when merging via autosquash into main

ng-dev/pr/merge/strategies/autosquash-merge.ts

@@ -131,12 +131,12 @@ export class AutosquashMergeStrategy extends MergeStrategy {
         pull_number: pullRequest.prNumber,
         state: 'closed',
       });
-
-      // When a pull request is merged, GitHub automatically closes any linked issues.
-      // This is not the case when the pull request is not merged into the main branch.
-      // This is why we need to manually close the linked issues.
-      await this.closeLinkedIssues(pullRequest);
     }
+
+    // When a pull request is merged, GitHub automatically closes any linked issues.
+    // This is not the case when the pull request is not merged into the main branch or closed via commit message.
+    // This is why we need to manually close the linked issues.
+    await this.closeLinkedIssues(pullRequest);
   }
 }
 

ng-dev/pr/merge/strategies/strategy.ts

@@ -246,18 +246,8 @@ export abstract class MergeStrategy {
    * because the PR's target branch is *not* the repository's main branch.
    *
    * It updates any open linked issues to a `closed` state with a `completed` reason.
-   *
-   * @note The method is a **no-op** if the PR target is the main branch,
-   * as GitHub handles issue closure automatically in that case.
    */
-  protected async closeLinkedIssues({
-    closingIssuesReferences,
-    githubTargetBranch,
-  }: PullRequest): Promise<void> {
-    if (githubTargetBranch === this.git.mainBranchName) {
-      return;
-    }
-
+  protected async closeLinkedIssues({closingIssuesReferences}: PullRequest): Promise<void> {
     for (const {number: issue_number, state} of closingIssuesReferences) {
       if (state === 'CLOSED') {
         continue;