fix: Extract JSON from all assistant messages, not just last

Claude Code may output text in multiple turns. The JSON result could
be in any message, not necessarily the last one. Join all text content
and search for the JSON block in the combined output.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: sylvansys

.github/actions/code-quality-reviewer/action.yml

@@ -158,8 +158,10 @@ runs:
 
         EXEC="${{ steps.review.outputs.execution_file }}"
         if [ -f "$EXEC" ]; then
-          TEXT=$(jq -r '[.[] | select(.type=="assistant") | .message.content[]? | select(.type=="text") | .text] | last // ""' "$EXEC" 2>/dev/null || echo "")
-          JSON=$(echo "$TEXT" | sed -n '/```json/,/```/{/```/d;p;}' | head -50)
+          # Get ALL text from assistant messages, joined together
+          ALL_TEXT=$(jq -r '[.[] | select(.type=="assistant") | .message.content[]? | select(.type=="text") | .text] | join("\n")' "$EXEC" 2>/dev/null || echo "")
+          # Extract JSON block from anywhere in the combined text
+          JSON=$(echo "$ALL_TEXT" | sed -n '/```json/,/```/{/```/d;p;}' | head -100)
           if echo "$JSON" | jq . >/dev/null 2>&1; then
             # Count checks by status
             CHECKS_PASSED=$(echo "$JSON" | jq '[.checks[]? | select(.status == "passed")] | length')

.github/actions/requirements-reviewer/action.yml

@@ -186,18 +186,11 @@ runs:
         RESULT='{"checks":[],"message":"Review failed to complete"}'
 
         EXEC="${{ steps.review.outputs.execution_file }}"
-        echo "::group::Debug extraction"
-        echo "EXEC path: $EXEC"
-        echo "File exists: $([ -f "$EXEC" ] && echo yes || echo no)"
         if [ -f "$EXEC" ]; then
-          echo "File size: $(wc -c < "$EXEC") bytes"
-          TEXT=$(jq -r '[.[] | select(.type=="assistant") | .message.content[]? | select(.type=="text") | .text] | last // ""' "$EXEC" 2>/dev/null || echo "")
-          echo "TEXT length: ${#TEXT} chars"
-          echo "TEXT preview: ${TEXT:0:200}"
-          JSON=$(echo "$TEXT" | sed -n '/```json/,/```/{/```/d;p;}' | head -50)
-          echo "JSON length: ${#JSON} chars"
-          echo "JSON preview: ${JSON:0:200}"
-          echo "::endgroup::"
+          # Get ALL text from assistant messages, joined together
+          ALL_TEXT=$(jq -r '[.[] | select(.type=="assistant") | .message.content[]? | select(.type=="text") | .text] | join("\n")' "$EXEC" 2>/dev/null || echo "")
+          # Extract JSON block from anywhere in the combined text
+          JSON=$(echo "$ALL_TEXT" | sed -n '/```json/,/```/{/```/d;p;}' | head -100)
           if echo "$JSON" | jq . >/dev/null 2>&1; then
             # Count checks by status
             CHECKS_PASSED=$(echo "$JSON" | jq '[.checks[]? | select(.status == "passed")] | length')
@@ -212,12 +205,7 @@ runs:
             fi
 
             RESULT=$(echo "$JSON" | jq -c '.')
-            echo "Extraction succeeded: passed=$PASSED, checks_passed=$CHECKS_PASSED"
-          else
-            echo "JSON parsing failed"
           fi
-        else
-          echo "Execution file not found"
         fi
 
         # Always output