fix: plan too long for status api

.github/workflows/test.yml

@@ -40,6 +40,7 @@ jobs:
           token: ${{ secrets.GITHUB_TOKEN }}
           working-directory: ${{ env.TERRAFORM_WORKING_DIRECTORY }}
           report-title: Single Argument Run
+
 
       - uses: ./
         with:
@@ -70,6 +71,12 @@ jobs:
           token: ${{ secrets.GITHUB_TOKEN }}
           working-directory: ${{ env.TERRAFORM_WORKING_DIRECTORY }}
           report-title: Single Argument Run Second Job
+          save-directory: ./artifacts
+
+      - uses: actions/upload-artifact@v2
+        with:
+          name: terraform-output
+          path: ./artifacts/*
 
   release:
     runs-on: ubuntu-latest

action.yml

@@ -23,6 +23,10 @@ inputs:
     required: false
     description: 'The name of the terraform plan to use when posting the results to the Pull Request'
     default: 'Terraform Plan Report'
+  save-directory: 
+    required: false
+    description: 'The directory path to store the full stdout / stderr from the terraform plan execution.  A std.out and std.err file will be created with the data.'
+    default: ''
 runs:
   using: 'node12'
   main: 'dist/index.js'

src/github.ts

@@ -28,6 +28,10 @@ export async function createStatusCheck(
 \`\`\``;
   }
 
+  if (details.length > 65000) {
+    details = `${details.substr(0, 65400)} ... truncated`;
+  }
+
   const context = github.context;
   const pr = context.payload.pull_request;
   await octokit.checks.create({
@@ -47,14 +51,24 @@ export async function createStatusCheck(
 }
 
 function getPlanSummary(output: string): string {
-  const planLineStart = output.indexOf('Plan:');
-  if (planLineStart > 0) {
-    const endOfPlanLine = output.indexOf('\n', planLineStart);
-    if (endOfPlanLine > 0) {
-      return output.substr(planLineStart, endOfPlanLine - planLineStart);
+  let summary = findLine('Plan:', output);
+  if (summary === '') {
+    summary = findLine('No changes.', output);
+  }
+
+  return summary;
+}
+
+function findLine(wording: string, output: string): string {
+  const wordingIndex = output.indexOf('Plan:');
+  if (wordingIndex > 0) {
+    const endOfLineIndex = output.indexOf('\n', wordingIndex);
+    if (endOfLineIndex > 0) {
+      return output.substring(wordingIndex, endOfLineIndex);
     } else {
-      return output.substr(planLineStart);
+      return output.substr(wordingIndex);
     }
   }
+
   return '';
 }

src/main.ts

@@ -1,5 +1,8 @@
 import * as core from '@actions/core';
 import { exec, ExecOptions } from '@actions/exec';
+import io from '@actions/io';
+import fs from 'fs';
+import path from 'path';
 import { createStatusCheck } from './github';
 import { TerraformResults } from './terraform-results';
 
@@ -14,6 +17,7 @@ async function run(): Promise<void> {
     const continueOnError: boolean =
       (core.getInput('continue-on-error', { required: false }) || 'false') ===
       'true';
+    const saveDirectory = core.getInput('save-directory');
 
     const stdOut: Buffer[] = [];
     const stdErr: Buffer[] = [];
@@ -59,6 +63,10 @@ async function run(): Promise<void> {
     core.debug(error);
     core.debug(' ------ Standard Error from Plan -----');
 
+    if (saveDirectory !== undefined) {
+      writeFile(saveDirectory, output, error);
+    }
+
     createStatusCheck(
       githubToken,
       reportTitle,
@@ -73,6 +81,16 @@ async function run(): Promise<void> {
   }
 }
 
+async function writeFile(
+  directory: string,
+  output: string,
+  error: string
+): Promise<void> {
+  io.mkdirP(directory);
+  await fs.promises.writeFile(path.join(directory, 'std.out'), output);
+  await fs.promises.writeFile(path.join(directory, 'std.err'), error);
+}
+
 function writeBufferToString(data: Buffer[]): string {
   return data.map((buffer) => buffer.toString()).join('');
 }