diff --git a/.github/actions/cora-review/action.yml b/.github/actions/cora-review/action.yml index 22e56a54..04a86b56 100644 --- a/.github/actions/cora-review/action.yml +++ b/.github/actions/cora-review/action.yml @@ -53,7 +53,7 @@ runs: done if [ -z "$VERSION" ]; then echo "::warning::Failed to resolve latest cora-cli version from GitHub API, using fallback" - VERSION="v0.1.7" + VERSION="v0.2.0" fi else VERSION="$INPUT_VERSION" @@ -116,11 +116,17 @@ runs: --format sarif \ --severity "$INPUT_SEVERITY" \ --quiet \ - > cora-results.sarif 2>cora-stderr.log || true - echo "Cora review complete ($(wc -c < cora-results.sarif) bytes)" + > cora-results.sarif 2>cora-stderr.log; EXIT_CODE=$? + SARIF_BYTES=$(wc -c < cora-results.sarif) + echo "Cora review complete ($SARIF_BYTES bytes, exit=$EXIT_CODE)" if [ -s cora-stderr.log ]; then echo "::warning::Cora stderr: $(cat cora-stderr.log)" fi + # Fail if cora exited non-zero AND produced empty result (LLM API failure) + if [ "$EXIT_CODE" -ne 0 ] && [ "$SARIF_BYTES" -lt 10 ]; then + echo "::error::Cora review failed (exit=$EXIT_CODE, $SARIF_BYTES bytes). LLM API may be unavailable." + exit 1 + fi - name: Upload SARIF to GitHub Code Scanning if: inputs.upload-sarif == 'true' diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 2d309f5a..81c14327 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -9,6 +9,7 @@ on: env: CARGO_TERM_COLOR: always RUST_BACKTRACE: 1 + FORCE_JAVASCRIPT_ACTIONS_TO_NODE24: true permissions: contents: read diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml index 3ec9d555..b48bd0b2 100644 --- a/.github/workflows/release.yml +++ b/.github/workflows/release.yml @@ -7,6 +7,7 @@ on: env: CARGO_TERM_COLOR: always + FORCE_JAVASCRIPT_ACTIONS_TO_NODE24: true jobs: build-release: @@ -78,7 +79,7 @@ jobs: 7z a ${{ matrix.artifact }}-${{ steps.version.outputs.VERSION }}.${{ matrix.ext }} ${{ matrix.artifact }}.exe - name: Upload artifact - uses: actions/upload-artifact@v4 + uses: actions/upload-artifact@v7 with: name: ${{ matrix.artifact }}-${{ steps.version.outputs.VERSION }} path: release/${{ matrix.artifact }}-${{ steps.version.outputs.VERSION }}.${{ matrix.ext }}