fix: update action output handling for closed events to ensure accurate module release information (#273)

Author: virgofx

README.md

@@ -395,6 +395,10 @@ The following outputs are available from this action:
 }
 ```
 
+> **Note**: When using outputs during the `closed` event, the `changed-modules-map` output will show `latestTag` as the
+> previous release version, while `releaseTag` and `releaseType` reflect the new release that was just created. To
+> reference the newly published version, use the value of `releaseTag`.
+
 ## Inspiration
 
 This action was inspired by the blog post

src/main.ts

@@ -210,13 +210,17 @@ export async function run(): Promise<void> {
     const releasesToDelete = TerraformModule.getReleasesToDelete(allReleases, terraformModules);
     const tagsToDelete = TerraformModule.getTagsToDelete(allTags, terraformModules);
 
+    // Important: Let's set the action outputs prior to performing the closed/merge request release.
+    // This is because the changed modules filters on [module.needsRelease()] which will be false
+    // after the release is created. By setting the outputs here, we ensure they accurately reflect
+    // the modules that were changed and needed release at this point in time.
+    setActionOutputs(terraformModules);
+
     if (context.isPrMergeEvent) {
       await handlePullRequestMergedEvent(config, terraformModules, releasesToDelete, tagsToDelete);
     } else {
       await handlePullRequestEvent(terraformModules, releasesToDelete, tagsToDelete);
     }
-
-    setActionOutputs(terraformModules);
   } catch (error) {
     if (error instanceof Error) {
       setFailed(error.message);