better messaging (google-gemini#8794)

mattKorwel · nagendrareddy10 · commit 7bd156579733 · 2025-09-22T19:43:34.000+05:30
diff --git a/.github/workflows/release-patch-1-create-pr.yml b/.github/workflows/release-patch-1-create-pr.yml
@@ -71,17 +71,22 @@ jobs:
           GH_TOKEN: '${{ steps.generate_token.outputs.token }}'
         continue-on-error: true
         run: |
-          node scripts/releasing/create-patch-pr.js --commit=${{ github.event.inputs.commit }} --channel=${{ github.event.inputs.channel }} --dry-run=${{ github.event.inputs.dry_run }} > patch_output.log 2>&1
-          echo "EXIT_CODE=$?" >> "$GITHUB_OUTPUT"
-          cat patch_output.log
+          # Capture output directly to environment variable
+          {
+            node scripts/releasing/create-patch-pr.js --commit=${{ github.event.inputs.commit }} --channel=${{ github.event.inputs.channel }} --dry-run=${{ github.event.inputs.dry_run }}
+            echo "EXIT_CODE=$?" >> "$GITHUB_OUTPUT"
+          } 2>&1 | {
+            echo "LOG_CONTENT<<EOF" >> "$GITHUB_ENV"
+            cat >> "$GITHUB_ENV"
+            echo "EOF" >> "$GITHUB_ENV"
+          }
 
       - name: 'Comment on Original PR'
         if: '!inputs.dry_run && inputs.original_pr'
         env:
           GH_TOKEN: '${{ steps.generate_token.outputs.token }}'
           ORIGINAL_PR: '${{ github.event.inputs.original_pr }}'
           EXIT_CODE: '${{ steps.create_patch.outputs.EXIT_CODE }}'
-          OUTPUT_LOG: 'patch_output.log'
           COMMIT: '${{ github.event.inputs.commit }}'
           CHANNEL: '${{ github.event.inputs.channel }}'
           REPOSITORY: '${{ github.repository }}'
diff --git a/.github/workflows/release-patch-from-comment.yml b/.github/workflows/release-patch-from-comment.yml
@@ -52,6 +52,7 @@ jobs:
 
       - name: 'Dispatch if Merged'
         if: "steps.pr_status.outputs.STATE == 'MERGED'"
+        id: 'dispatch_patch'
         uses: 'actions/github-script@00f12e3e20659f42342b1c0226afda7f7c042325'
         env:
           COMMENT_BODY: '${{ github.event.comment.body }}'
@@ -82,7 +83,7 @@ jobs:
 
             console.log('Detected channel:', channel);
 
-            github.rest.actions.createWorkflowDispatch({
+            const response = await github.rest.actions.createWorkflowDispatch({
               owner: context.repo.owner,
               repo: context.repo.repo,
               workflow_id: 'release-patch-1-create-pr.yml',
@@ -93,7 +94,30 @@ jobs:
                 dry_run: 'false',
                 original_pr: '${{ github.event.issue.number }}'
               }
-            })
+            });
+
+            // Wait a moment for the workflow to be created, then find it
+            await new Promise(resolve => setTimeout(resolve, 2000));
+
+            const runs = await github.rest.actions.listWorkflowRuns({
+              owner: context.repo.owner,
+              repo: context.repo.repo,
+              workflow_id: 'release-patch-1-create-pr.yml',
+              per_page: 10
+            });
+
+            // Find the most recent run that matches our trigger
+            const dispatchedRun = runs.data.workflow_runs.find(run =>
+              run.event === 'workflow_dispatch' &&
+              new Date(run.created_at) > new Date(Date.now() - 10000) // Within last 10 seconds
+            );
+
+            if (dispatchedRun) {
+              core.setOutput('dispatched_run_id', dispatchedRun.id);
+              core.setOutput('dispatched_run_url', dispatchedRun.html_url);
+            }
+
+            core.setOutput('channel', channel);
 
       - name: 'Comment on Failure'
         if: "startsWith(github.event.comment.body, '/patch') && steps.pr_status.outputs.STATE != 'MERGED'"
@@ -102,3 +126,54 @@ jobs:
           issue-number: '${{ github.event.issue.number }}'
           body: |
             :x: The `/patch` command failed. This pull request must be merged before a patch can be created.
+
+      - name: 'Final Status Comment - Success'
+        if: "always() && startsWith(github.event.comment.body, '/patch') && steps.dispatch_patch.outcome == 'success' && steps.dispatch_patch.outputs.dispatched_run_url"
+        uses: 'peter-evans/create-or-update-comment@67dcc547d311b736a8e6c5c236542148a47adc3d'
+        with:
+          issue-number: '${{ github.event.issue.number }}'
+          body: |
+            ✅ **Patch workflow dispatched successfully!**
+
+            **📋 Details:**
+            - **Channel**: `${{ steps.dispatch_patch.outputs.channel }}`
+            - **Commit**: `${{ steps.pr_status.outputs.MERGE_COMMIT_SHA }}`
+
+            **🔗 Track Progress:**
+            - [Dispatched patch workflow](${{ steps.dispatch_patch.outputs.dispatched_run_url }})
+            - [This workflow run](https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }})
+
+      - name: 'Final Status Comment - Dispatch Success (No URL)'
+        if: "always() && startsWith(github.event.comment.body, '/patch') && steps.dispatch_patch.outcome == 'success' && !steps.dispatch_patch.outputs.dispatched_run_url"
+        uses: 'peter-evans/create-or-update-comment@67dcc547d311b736a8e6c5c236542148a47adc3d'
+        with:
+          issue-number: '${{ github.event.issue.number }}'
+          body: |
+            ✅ **Patch workflow dispatched successfully!**
+
+            **📋 Details:**
+            - **Channel**: `${{ steps.dispatch_patch.outputs.channel }}`
+            - **Commit**: `${{ steps.pr_status.outputs.MERGE_COMMIT_SHA }}`
+
+            **🔗 Track Progress:**
+            - [View patch workflows](https://github.com/${{ github.repository }}/actions/workflows/release-patch-1-create-pr.yml)
+            - [This workflow run](https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }})
+
+      - name: 'Final Status Comment - Failure'
+        if: "always() && startsWith(github.event.comment.body, '/patch') && (steps.dispatch_patch.outcome == 'failure' || steps.dispatch_patch.outcome == 'cancelled')"
+        uses: 'peter-evans/create-or-update-comment@67dcc547d311b736a8e6c5c236542148a47adc3d'
+        with:
+          issue-number: '${{ github.event.issue.number }}'
+          body: |
+            ❌ **Patch workflow dispatch failed!**
+
+            There was an error dispatching the patch creation workflow.
+
+            **🔍 Troubleshooting:**
+            - Check that the PR is properly merged
+            - Verify workflow permissions
+            - Review error logs in the workflow run
+
+            **🔗 Debug Links:**
+            - [This workflow run](https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }})
+            - [Patch workflow history](https://github.com/${{ github.repository }}/actions/workflows/release-patch-1-create-pr.yml)
diff --git a/scripts/releasing/patch-create-comment.js b/scripts/releasing/patch-create-comment.js
@@ -13,7 +13,6 @@
 
 import yargs from 'yargs';
 import { hideBin } from 'yargs/helpers';
-import { readFileSync } from 'node:fs';
 
 async function main() {
   const argv = await yargs(hideBin(process.argv))
@@ -27,11 +26,6 @@ async function main() {
       type: 'number',
       demandOption: !process.env.GITHUB_ACTIONS,
     })
-    .option('output-log', {
-      description: 'Path to the patch output log file',
-      type: 'string',
-      default: 'patch_output.log',
-    })
     .option('commit', {
       description: 'The commit SHA being patched',
       type: 'string',
@@ -86,8 +80,6 @@ async function main() {
     argv.exitCode !== undefined
       ? argv.exitCode
       : parseInt(process.env.EXIT_CODE || '1');
-  const outputLog =
-    argv.outputLog || process.env.OUTPUT_LOG || 'patch_output.log';
   const commit = argv.commit || process.env.COMMIT;
   const channel = argv.channel || process.env.CHANNEL;
   const repository =
@@ -111,7 +103,6 @@ async function main() {
     console.log('\n📋 Inputs:');
     console.log(`  - Original PR: ${originalPr}`);
     console.log(`  - Exit Code: ${exitCode}`);
-    console.log(`  - Output Log: ${outputLog}`);
     console.log(`  - Commit: ${commit}`);
     console.log(`  - Channel: ${channel} → npm tag: ${npmTag}`);
     console.log(`  - Repository: ${repository}`);
@@ -121,20 +112,17 @@ async function main() {
   let commentBody;
   let logContent = '';
 
-  // Try to read the output log
-  try {
-    if (testMode) {
-      // Create mock log content for testing
-      if (exitCode === 0) {
-        logContent = `Creating hotfix branch hotfix/v0.5.3/${channel}/cherry-pick-${commit.substring(0, 7)} from release/v0.5.3`;
-      } else {
-        logContent = 'Error: Failed to create patch';
-      }
+  // Get log content from environment variable or generate mock content for testing
+  if (testMode && !process.env.LOG_CONTENT) {
+    // Create mock log content for testing only if LOG_CONTENT is not provided
+    if (exitCode === 0) {
+      logContent = `Creating hotfix branch hotfix/v0.5.3/${channel}/cherry-pick-${commit.substring(0, 7)} from release/v0.5.3`;
     } else {
-      logContent = readFileSync(outputLog, 'utf8');
+      logContent = 'Error: Failed to create patch';
     }
-  } catch (error) {
-    console.log(`Could not read output log ${outputLog}:`, error.message);
+  } else {
+    // Use log content from environment variable
+    logContent = process.env.LOG_CONTENT || '';
   }
 
   if (logContent.includes('already has an open PR')) {
diff --git a/scripts/tests/patch-create-comment.test.js b/scripts/tests/patch-create-comment.test.js
