extract run npm script ci logic

Author: sarasvoss

.github/actions/run-npm-script/action.yml

@@ -0,0 +1,43 @@
+name: 'Run NPM Script'
+description: 'Run an npm script if present, and output status'
+
+inputs:
+  script:
+    description: 'The npm script to run (e.g., build, lint:check, test)'
+    required: true
+    default: ''
+  working-directory:
+    description: 'Directory containing package.json (optional)'
+    required: false
+    default: '.'
+
+outputs:
+  status:
+    description: 'success, failure, or notpresent'
+
+runs:
+  using: 'composite'
+  steps:
+    - name: Run npm script
+      id: run-script
+      shell: bash
+      env:
+        WORKING_DIRECTORY: ${{ inputs.working-directory }}
+        SCRIPT: ${{ inputs.script }}
+      run: |
+        cd "$WORKING_DIRECTORY"
+        if ! [ -f package.json ]; then
+          echo "No package.json found. Skipping $SCRIPT."
+          echo "status=notpresent" >> "$GITHUB_OUTPUT"
+          exit 0
+        fi
+        if ! jq -e --arg s "$SCRIPT" '.scripts[$s]' package.json > /dev/null; then
+          echo "script `$SCRIPT` not found in package.json. Skipping."
+          echo "status=notpresent" >> "$GITHUB_OUTPUT"
+          exit 0
+        fi
+        if npm run "$SCRIPT"; then
+          echo "status=success" >> "$GITHUB_OUTPUT"
+        else
+          echo "status=failure" >> "$GITHUB_OUTPUT"
+        fi

.github/workflows/run_npm_ci_scripts.yml

@@ -52,74 +52,42 @@ jobs:
         continue-on-error: true
 
       - name: Build
+        uses: ./.github/actions/run-npm-script
         id: build
-        continue-on-error: true
-        run: |
-          if ! [ -f package.json ] || ! jq -e '.scripts.build' package.json > /dev/null; then
-            echo "No build script found in package.json."
-            echo "build_status=notpresent" >> $GITHUB_OUTPUT
-            exit 0
-          fi
-          if npm run build; then
-            echo "build_status=success" >> $GITHUB_OUTPUT
-          else
-            echo "build_status=failure" >> $GITHUB_OUTPUT
-          fi
+        with:
+          working-directory: '.'
+          script: build
 
       - name: Lint
+        uses: ./.github/actions/run-npm-script
         id: lint
-        continue-on-error: true
-        run: |
-          if ! [ -f package.json ] || ! jq -e '.scripts.["lint:check"]' package.json > /dev/null; then
-            echo "No lint script found in package.json."
-            echo "lint_status=notpresent" >> $GITHUB_OUTPUT
-            exit 0
-          fi
-          if npm run lint:check; then
-            echo "lint_status=success" >> $GITHUB_OUTPUT
-          else
-            echo "lint_status=failure" >> $GITHUB_OUTPUT
-          fi
+        with:
+          working-directory: '.'
+          script: lint:check
 
       - name: Format
+        uses: ./.github/actions/run-npm-script
         id: format
-        continue-on-error: true
-        run: |
-          if ! [ -f package.json ] || ! jq -e '.scripts.["format:check"]' package.json > /dev/null; then
-            echo "No format script found in package.json."
-            echo "format_status=notpresent" >> $GITHUB_OUTPUT
-            exit 0
-          fi
-          if npm run format:check; then
-            echo "format_status=success" >> $GITHUB_OUTPUT
-          else
-            echo "format_status=failure" >> $GITHUB_OUTPUT
-          fi
+        with:
+          working-directory: '.'
+          script: format:check
 
       - name: Test
+        uses: ./.github/actions/run-npm-script
         id: test
-        continue-on-error: true
-        run: |
-          if ! [ -f package.json ] || ! jq -e '.scripts.test' package.json > /dev/null; then
-            echo "No test script found in package.json."
-            echo "test_status=notpresent" >> $GITHUB_OUTPUT
-            exit 0
-          fi
-          if npm test; then
-            echo "test_status=success" >> $GITHUB_OUTPUT
-          else
-            echo "test_status=failure" >> $GITHUB_OUTPUT
-          fi
+        with:
+          working-directory: '.'
+          script: test
 
       - name: Job Summary
         if: always()
         env:
           AUDIT_STATUS: ${{ steps.audit-npm.outputs.gate_passed == 'true' && 'success' || 'failure' }}
           AUDIT_SUMMARY: ${{ steps.audit-npm.outputs.gate_summary }}
-          BUILD_STATUS: ${{ steps.build.outputs.build_status || steps.build.outcome }}
-          LINT_STATUS: ${{ steps.lint.outputs.lint_status || steps.lint.outcome }}
-          FORMAT_STATUS: ${{ steps.format.outputs.format_status || steps.format.outcome }}
-          TEST_STATUS: ${{ steps.test.outputs.test_status || steps.test.outcome }}
+          BUILD_STATUS: ${{ steps.build.outputs.status || steps.build.outcome }}
+          LINT_STATUS: ${{ steps.lint.outputs.status || steps.lint.outcome }}
+          FORMAT_STATUS: ${{ steps.format.outputs.status || steps.format.outcome }}
+          TEST_STATUS: ${{ steps.test.outputs.status || steps.test.outcome }}
         run: |
           status_emoji () {
             case "$1" in