refactor: rename handleWikiOperations to handleReleasePlanComment for clarity

Author: virgofx

.github/workflows/ci.yml

@@ -59,27 +59,36 @@ jobs:
           echo "Testing action outputs..."
           
           # Test if outputs are set
-          if [[ -n "${{ steps.test-action.outputs.changed_module_names }}" ]]; then
-            echo "✅ Changed module names: ${{ steps.test-action.outputs.changed_module_names }}"
+          if [[ -n "${{ steps.test-action.outputs.changed-module-names }}" ]]; then
+            echo "✅ Output 'changed-modules-names': ${{ steps.test-action.outputs.changed-module-names }}"
           else
             echo "❌ No changed module names found"
           fi
           
-          if [[ -n "${{ steps.test-action.outputs.changed_module_paths }}" ]]; then
-            echo "✅ Changed module paths: ${{ steps.test-action.outputs.changed_module_paths }}"
+          if [[ -n "${{ steps.test-action.outputs.changed-module-paths }}" ]]; then
+            echo "✅ Output 'changed-module-paths': ${{ steps.test-action.outputs.changed-module-paths }}"
           else
             echo "❌ No changed module paths found"
           fi
           
-          if [[ -n "${{ steps.test-action.outputs.changed_modules_map }}" ]]; then
-            echo "✅ Changed modules map:"
-            echo "${{ steps.test-action.outputs.changed_modules_map }}"
+          if [[ -n "${{ steps.test-action.outputs.changed-modules-map }}" ]]; then
+            echo "✅ Output 'changed-modules-map':"
+            echo '${{ steps.test-action.outputs.changed-modules-map }}' | jq -r '
+              "Found \(length) changed module(s):",
+              (to_entries[] | 
+                "• \(.key):",
+                "  - Path: \(.value.path)",
+                "  - Current Tag: \(.value.currentTag)",
+                "  - Next Tag: \(.value.nextTag)",
+                "  - Release Type: \(.value.releaseType)"
+              )
+            '
           else
             echo "❌ No changed modules map found"
           fi
-          
-          # Validate JSON format of changed_modules_map
-          if ! echo "${{ steps.test-action.outputs.changed_modules_map }}" | jq '.' >/dev/null 2>&1; then
-            echo "❌ Changed modules map is not valid JSON"
+
+          # Silently validate JSON structure is an object
+          echo '${{ steps.test-action.outputs.changed-modules-map }}' | jq -e 'type == "object"' > /dev/null || {
+            echo "❌ Expected object type in changed-modules-map"
             exit 1
-          fi
+          }

.node-version

@@ -1,2 +1 @@
-22
-# Used by CI only
+22

src/main.ts

@@ -14,7 +14,7 @@ import type {
   TerraformModule,
 } from '@/types';
 import { WikiStatus, checkoutWiki, commitAndPushWikiChanges, generateWikiFiles } from '@/wiki';
-import { info, setFailed, setOutput } from '@actions/core';
+import { endGroup, info, setFailed, setOutput, startGroup } from '@actions/core';
 
 /**
  * Initializes and returns the configuration and context objects.
@@ -37,7 +37,7 @@ function initialize(): { config: Config; context: Context } {
  * @param {string[]} terraformModuleNamesToRemove - List of Terraform module names to remove.
  * @returns {Promise<void>} Resolves when wiki-related operations are completed.
  */
-async function handleWikiOperations(
+async function handleReleasePlanComment(
   config: Config,
   terraformChangedModules: TerraformChangedModule[],
   terraformModuleNamesToRemove: string[],
@@ -129,7 +129,7 @@ export async function run(): Promise<void> {
     const terraformModuleNamesToRemove = getTerraformModulesToRemove(allTags, terraformModules);
 
     if (!context.isPrMergeEvent) {
-      await handleWikiOperations(config, terraformChangedModules, terraformModuleNamesToRemove);
+      await handleReleasePlanComment(config, terraformChangedModules, terraformModuleNamesToRemove);
     } else {
       await handleMergeEvent(
         config,
@@ -154,14 +154,15 @@ export async function run(): Promise<void> {
     );
 
     // Log the changes for debugging
+    startGroup('Outputs');
     info(`Changed module names: ${JSON.stringify(changedModuleNames)}`);
     info(`Changed module paths: ${JSON.stringify(changedModulePaths)}`);
     info(`Changed modules map: ${JSON.stringify(changedModulesMap, null, 2)}`);
+    endGroup();
 
-
-    setOutput('changed_module_names', changedModuleNames);
-    setOutput('changed_module_paths', changedModulePaths);
-    setOutput('changed_modules_map', changedModulesMap);
+    setOutput('changed-module-names', changedModuleNames);
+    setOutput('changed-module-paths', changedModulePaths);
+    setOutput('changed-modules-map', changedModulesMap);
   } catch (error) {
     if (error instanceof Error) {
       setFailed(error.message);