Warning: This build has drifted.
The coverage report for this pull request build may be inaccurate because its base commit is no longer the HEAD of its target branch.
This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

    • Learn more: For more information on this, see Tracking coverage changes for pull request builds.
    • Fix now: For a quick fix, rebase this PR at GitHub. Your next report should be accurate.
    • Prevent going forward: To avoid this issue with future PRs, see these Recommended CI Configurations.

fix: only treat COMPLETED signal as terminal success state

REVIEW_DONE and CODEX_REVIEW_DONE signals mark the end of review
passes, not the end of execution. Only COMPLETED (from ALL_TASKS_DONE)
and FAILED should be terminal signals that stop the status badge from
updating.

This fixes the e2e test TestSignalEventRendering where the badge was
showing "REVIEW" instead of "COMPLETED" because intermediate review
signals were incorrectly treated as terminal.