diff --git a/.github/workflows/claude-review.yml b/.github/workflows/claude-review.yml index ce5eddf936ac..877d65d637f6 100644 --- a/.github/workflows/claude-review.yml +++ b/.github/workflows/claude-review.yml @@ -68,7 +68,7 @@ jobs: - name: Setup Claude review toolkit if: steps.set-authorized.outputs.IS_AUTHORIZED == 'true' id: toolkit - uses: Expensify/GitHub-Actions/.github/actions/claude-review-toolkit@43c1b4348cd46637e44064b02f5d316bfab990ee + uses: Expensify/GitHub-Actions/.github/actions/claude-review-toolkit@2b751e9a86d8690f53443cce0ce886079eccea76 - name: Mark review as in progress if: steps.set-authorized.outputs.IS_AUTHORIZED == 'true' @@ -99,22 +99,7 @@ jobs: env: GH_TOKEN: ${{ github.token }} STRUCTURED_OUTPUT: ${{ steps.code-review.outputs.structured_output }} - run: | - if [ -z "$STRUCTURED_OUTPUT" ]; then - echo "::error::Claude Code Action returned empty structured output" - exit 1 - fi - COUNT=$(echo "$STRUCTURED_OUTPUT" | jq '.violations | length') - if [ "$COUNT" -eq 0 ]; then - addPrReaction.sh "$PR_NUMBER" "+1" - else - echo "$STRUCTURED_OUTPUT" | jq -c '.violations[]' | while IFS= read -r v; do - PATH_ARG=$(echo "$v" | jq -r '.path') - BODY_ARG=$(echo "$v" | jq -r '.body') - LINE_ARG=$(echo "$v" | jq -r '.line') - createInlineComment.sh "$PR_NUMBER" "$PATH_ARG" "$BODY_ARG" "$LINE_ARG" || true - done - fi + run: postCodeReviewResults.sh "$PR_NUMBER" - name: Run Claude Code (docs) if: steps.set-authorized.outputs.IS_AUTHORIZED == 'true' && steps.filter.outputs.docs == 'true'