fix: Quick fixes for reviewer comments

Phase 1 fixes from issue #56:
- Use ⏭️ emoji for skipped status (instead of ⚠️)
- Fix 404 agent prompt links by using SHA instead of branch name
- Remove split test - use emoji format for all reviewers
- Update intro line: "PR feedback updates from your reviewer agents..."
- Add CI failure (exit 1) when checks fail

Closes #56

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

Author: sylvansys

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

@@ -197,3 +197,9 @@ runs:
         echo "checks_passed=$CHECKS_PASSED" >> $GITHUB_OUTPUT
         echo "checks_failed=$CHECKS_FAILED" >> $GITHUB_OUTPUT
         echo "checks_skipped=$CHECKS_SKIPPED" >> $GITHUB_OUTPUT
+
+        # Fail CI if any checks failed (but not if just skipped)
+        if [ "$CHECKS_FAILED" -gt 0 ]; then
+          echo "::error::$CHECKS_FAILED check(s) failed"
+          exit 1
+        fi

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

@@ -250,3 +250,9 @@ runs:
         echo "checks_passed=$CHECKS_PASSED" >> $GITHUB_OUTPUT
         echo "checks_failed=$CHECKS_FAILED" >> $GITHUB_OUTPUT
         echo "checks_skipped=$CHECKS_SKIPPED" >> $GITHUB_OUTPUT
+
+        # Fail CI if any checks failed (but not if just skipped)
+        if [ "$CHECKS_FAILED" -gt 0 ]; then
+          echo "::error::$CHECKS_FAILED check(s) failed"
+          exit 1
+        fi

.github/actions/review-comment/action.yml

@@ -71,22 +71,16 @@ runs:
           STATUS_EMOJI="❌"
         else
           # All skipped
-          STATUS_EMOJI="⚠️"
+          STATUS_EMOJI="⏭️"
         fi
 
-        # Build summary line - split test: Requirements uses words only, others use emoji+number
-        if [ "$REVIEW" = "Requirements" ]; then
-          # Format A: Words only (no emoji)
-          SUMMARY_PARTS="$CHECKS_PASSED passed • $CHECKS_FAILED failed • $CHECKS_SKIPPED skipped"
-        else
-          # Format B: Emoji + number only (compact)
-          SUMMARY_PARTS="✅ $CHECKS_PASSED • ❌ $CHECKS_FAILED • ⚠️ $CHECKS_SKIPPED"
-        fi
+        # Build summary line with emojis
+        SUMMARY_PARTS="✅ $CHECKS_PASSED • ❌ $CHECKS_FAILED • ⏭️ $CHECKS_SKIPPED"
 
         # Build prompt link
         PROMPT_LINK=""
         if [ -n "$PROMPT_FILE" ]; then
-          PROMPT_LINK="[View agent prompt](https://github.com/${{ github.repository }}/blob/${{ github.head_ref || github.ref_name }}/$PROMPT_FILE)"
+          PROMPT_LINK="[View agent prompt](https://github.com/${{ github.repository }}/blob/${{ inputs.sha }}/$PROMPT_FILE)"
         fi
 
         # Build checks table header
@@ -105,7 +99,7 @@ runs:
           case "$CHECK_STATUS" in
             "passed") CHECK_EMOJI="✅" ;;
             "failed") CHECK_EMOJI="❌" ;;
-            *) CHECK_EMOJI="⚠️" ;;
+            *) CHECK_EMOJI="⏭️" ;;
           esac
 
           # Get files for this check
@@ -194,7 +188,7 @@ runs:
           # Create new comment
           {
             echo "$MARKER"
-            echo "PR review updates from your Constellos agents. Based on linked issue(s), context files, and committed changes."
+            echo "PR feedback updates from your reviewer agents. Based on linked issues, context files, and committed changes."
             echo ""
             echo "$AGENT_BLOCK"
           } > /tmp/comment.md