Add changeset status check

Author: hsubox76

.github/workflows/check-changeset.yml

@@ -22,43 +22,38 @@ jobs:
     - name: Run changeset script
       run: yarn ts-node-script scripts/check_changeset.ts
       id: check-changeset
-    - name: Read output
-      run: echo "${{steps.check-changeset.outputs.MISSING_PACKAGES}}"
+    - name: Print changeset checker output
+      run: echo "${{steps.check-changeset.outputs.CHANGESET_ERROR_MESSAGE}}"
     - name: Find Comment
       uses: peter-evans/find-comment@v1
       id: fc
       with:
         issue-number: ${{github.event.number}}
         body-includes: Changeset File Check
     - name: Create comment (missing packages)
-      if: ${{!steps.fc.outputs.comment-id && steps.check-changeset.outputs.MISSING_PACKAGES}}
+      if: ${{!steps.fc.outputs.comment-id && steps.check-changeset.outputs.CHANGESET_ERROR_MESSAGE}}
       uses: peter-evans/create-or-update-comment@v1
       with:
         issue-number: ${{github.event.number}}
         body: |
           ### Changeset File Check :warning:
-          Warning: This PR modifies files in the following packages but they have not been included in the changeset file:
-          ${{steps.check-changeset.outputs.MISSING_PACKAGES}}
-          
-          Make sure this was intentional.
+          ${{steps.check-changeset.outputs.CHANGESET_ERROR_MESSAGE}}
     - name: Update comment (missing packages)
       if: ${{steps.fc.outputs.comment-id}}
       uses: peter-evans/create-or-update-comment@v1
       with:
-        comment-id: ${{steps.fc.outputs.comment-id}} && steps.check-changeset.outputs.MISSING_PACKAGES}}
+        comment-id: ${{steps.fc.outputs.comment-id}} && steps.check-changeset.outputs.CHANGESET_ERROR_MESSAGE}}
         edit-mode: replace
         body: |
           ### Changeset File Check :warning:
-          Warning: This PR modifies files in the following packages but they have not been included in the changeset file:
-          ${{steps.check-changeset.outputs.MISSING_PACKAGES}}
-
-          Make sure this was intentional.
+          ${{steps.check-changeset.outputs.CHANGESET_ERROR_MESSAGE}}
     - name: Update comment (no missing packages)
-      if: ${{steps.fc.outputs.comment-id && !steps.check-changeset.outputs.MISSING_PACKAGES}}
+      if: ${{steps.fc.outputs.comment-id && !steps.check-changeset.outputs.CHANGESET_ERROR_MESSAGE}}
       uses: peter-evans/create-or-update-comment@v1
       with:
         comment-id: ${{steps.fc.outputs.comment-id}}
         edit-mode: replace
         body: |
           ### Changeset File Check :white_check_mark:
-          No modified packages are missing from the changeset file.
+          - No modified packages are missing from the changeset file.
+          - No changeset formatting errors detected.

scripts/check_changeset.ts

@@ -17,6 +17,7 @@
 
 import { resolve } from 'path';
 import { existsSync } from 'fs';
+import { exec } from 'child-process-promise';
 import chalk from 'chalk';
 import simpleGit from 'simple-git/promise';
 import fs from 'mz/fs';
@@ -83,6 +84,21 @@ async function parseChangesetFile(changesetFile: string) {
 }
 
 async function main() {
+  let formattedStatusError: string = '';
+  let missingPackagesError: string = '';
+  try {
+    await exec('yarn changeset status');
+  } catch (e) {
+    const messageLines = e.message.replace(/🦋  error /g, '').split('\n');
+    formattedStatusError = 'Changeset formatting error in following file:%0A';
+    formattedStatusError += messageLines
+      .filter(
+        (line: string) => !line.match(/^    at [\w\.]+ \(.+:[0-9]+:[0-9]+\)/)
+      )
+      .filter((line: string) => !line.includes('Command failed'))
+      .filter((line: string) => !line.includes('exited with error code 1'))
+      .join('%0A');
+  }
   try {
     const diffData = await getDiffData();
     if (diffData == null) {
@@ -94,23 +110,33 @@ async function main() {
         changedPkg => !changesetPackages.includes(changedPkg)
       );
       if (missingPackages.length > 0) {
-        /**
-         * Sets Github Actions output for a step. Pass missing package list to next
-         * step. See:
-         * https://github.com/actions/toolkit/blob/master/docs/commands.md#set-outputs
-         */
-        console.log(
-          `::set-output name=MISSING_PACKAGES::${missingPackages
-            .map(pkg => `- ${pkg}`)
-            .join('%0A')}`
-        );
+        missingPackagesError = `Warning: This PR modifies files in the following packages but they have not been included in the changeset file:%0A
+        ${missingPackages.map(pkg => `- ${pkg}`).join('%0A')}
+        %0A
+        Make sure this was intentional.%0A`;
       }
-      process.exit();
     }
   } catch (e) {
     console.error(chalk`{red ${e}}`);
     process.exit(1);
   }
+
+  /**
+   * Sets Github Actions output for a step. Pass changeset error message to next
+   * step. See:
+   * https://github.com/actions/toolkit/blob/master/docs/commands.md#set-outputs
+   */
+  console.log(
+    `::set-output name=CHANGESET_ERROR_MESSAGE::${[
+      formattedStatusError,
+      missingPackagesError
+    ].join('%0A')}`
+  );
+  if (formattedStatusError) {
+    process.exit(1);
+  } else {
+    process.exit();
+  }
 }
 
 main();