fix: resolve bash variable indirection issue in workflow

EdwardAngert · claude · EdwardAngert · commit 815a74dbfd7a · 2025-04-24T13:04:58.000-04:00
- Replace dynamic variable indirection with explicit case statement - Fix 'invalid variable name' error during step output access - Improve compatibility with different shell environments 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
diff --git a/.github/docs/actions/docs-core/action.yaml b/.github/docs/actions/docs-core/action.yaml
@@ -1107,12 +1107,46 @@ runs:
           fi
           
           # Extract status and message from step outputs
-          STATUS_VAR="steps.${STEP_ID}.outputs.status"
-          MESSAGE_VAR="steps.${STEP_ID}.outputs.message"
+          # We'll use a more compatible approach than variable indirection
+          # Instead of dynamic variables, we'll use a case statement to look up the values
           
-          # Use status from step output or default to 'skipped'
-          STATUS="${!STATUS_VAR:-skipped}"
-          MESSAGE="${!MESSAGE_VAR:-Not run}"
+          # Default values
+          STATUS="skipped"
+          MESSAGE="Not run"
+          
+          # Set status based on step ID
+          case "$STEP_ID" in
+            "lint-markdown")
+              if [ -n "${{ steps.lint-markdown.outputs.status }}" ]; then
+                STATUS="${{ steps.lint-markdown.outputs.status }}"
+                MESSAGE="${{ steps.lint-markdown.outputs.message }}"
+              fi
+              ;;
+            "check-format")
+              if [ -n "${{ steps.check-format.outputs.status }}" ]; then
+                STATUS="${{ steps.check-format.outputs.status }}"
+                MESSAGE="${{ steps.check-format.outputs.message }}"
+              fi
+              ;;
+            "check-links")
+              if [ -n "${{ steps.check-links.outputs.status }}" ]; then
+                STATUS="${{ steps.check-links.outputs.status }}"
+                MESSAGE="${{ steps.check-links.outputs.message }}"
+              fi
+              ;;
+            "lint-vale")
+              if [ -n "${{ steps.lint-vale.outputs.status }}" ]; then
+                STATUS="${{ steps.lint-vale.outputs.status }}"
+                MESSAGE="${{ steps.lint-vale.outputs.message }}"
+              fi
+              ;;
+            "check-cross-references")
+              if [ -n "${{ steps.check-cross-references.outputs.status }}" ]; then
+                STATUS="${{ steps.check-cross-references.outputs.status }}"
+                MESSAGE="${{ steps.check-cross-references.outputs.message }}"
+              fi
+              ;;
+          esac
           
           # Add comma if not the first item
           if [ "$FIRST_ITEM" = true ]; then
