Fix CI workflow shell script and Playwright errors (#13)

- Remove invalid --screenshot=on option from Playwright E2E test command
- Fix multi-line string assignments in all 4 review jobs (requirements,
  code-quality, context, ui) that caused shell parsing errors
- Use printf for simple string concatenation
- Use heredocs for multi-line comment templates

Fixes "Requirements: command not found" and "exist: command not found"
errors caused by YAML indentation issues in multi-line shell strings.

Closes #12

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>

Author: sylvansys

.github/workflows/ci-pipeline.yml

@@ -193,7 +193,7 @@ jobs:
       - name: Run E2E tests
         run: |
           mkdir -p .claude/screenshots
-          npx playwright test --project=chromium --screenshot=on
+          npx playwright test --project=chromium
 
       - uses: actions/upload-artifact@v4
         if: always()
@@ -297,20 +297,22 @@ jobs:
             OLD_BODY=$(echo "$EXISTING" | jq -r '.body')
             # Append new row
             NEW_BODY=$(echo "$OLD_BODY" | sed "/^| $REVIEW |/d")
-            NEW_BODY="$NEW_BODY
-          $ROW"
+            NEW_BODY=$(printf '%s\n%s' "$NEW_BODY" "$ROW")
             gh api --method PATCH "/repos/${{ github.repository }}/issues/comments/$COMMENT_ID" -f body="$NEW_BODY"
           else
             # Create new comment
-            COMMENT="$MARKER
+            COMMENT=$(cat <<EOF
+          $MARKER
           ## Reviews
 
           | Review | Status | Summary |
           |--------|--------|---------|
           $ROW
 
           ---
-          *Claude Code CI*"
+          *Claude Code CI*
+          EOF
+          )
             gh pr comment "$PR" --body "$COMMENT"
           fi
 
@@ -386,20 +388,22 @@ jobs:
             COMMENT_ID=$(echo "$EXISTING" | jq -r '.id')
             OLD_BODY=$(echo "$EXISTING" | jq -r '.body')
             # Append new row before the footer
-            NEW_BODY=$(echo "$OLD_BODY" | sed "/---/i\\
-          $ROW")
+            NEW_BODY=$(echo "$OLD_BODY" | sed "/---/i\\$ROW")
             gh api --method PATCH "/repos/${{ github.repository }}/issues/comments/$COMMENT_ID" -f body="$NEW_BODY"
           else
             # Create new comment (shouldn't happen as requirements runs first)
-            COMMENT="$MARKER
+            COMMENT=$(cat <<EOF
+          $MARKER
           ## Reviews
 
           | Review | Status | Summary |
           |--------|--------|---------|
           $ROW
 
           ---
-          *Claude Code CI*"
+          *Claude Code CI*
+          EOF
+          )
             gh pr comment "$PR" --body "$COMMENT"
           fi
 
@@ -499,20 +503,22 @@ jobs:
             COMMENT_ID=$(echo "$EXISTING" | jq -r '.id')
             OLD_BODY=$(echo "$EXISTING" | jq -r '.body')
             # Append new row before the footer
-            NEW_BODY=$(echo "$OLD_BODY" | sed "/---/i\\
-          $ROW")
+            NEW_BODY=$(echo "$OLD_BODY" | sed "/---/i\\$ROW")
             gh api --method PATCH "/repos/${{ github.repository }}/issues/comments/$COMMENT_ID" -f body="$NEW_BODY"
           else
             # Create new comment (shouldn't happen as requirements runs first)
-            COMMENT="$MARKER
+            COMMENT=$(cat <<EOF
+          $MARKER
           ## Reviews
 
           | Review | Status | Summary |
           |--------|--------|---------|
           $ROW
 
           ---
-          *Claude Code CI*"
+          *Claude Code CI*
+          EOF
+          )
             gh pr comment "$PR" --body "$COMMENT"
           fi
 
@@ -602,19 +608,21 @@ jobs:
             COMMENT_ID=$(echo "$EXISTING" | jq -r '.id')
             OLD_BODY=$(echo "$EXISTING" | jq -r '.body')
             # Append new row before the footer
-            NEW_BODY=$(echo "$OLD_BODY" | sed "/---/i\\
-          $ROW")
+            NEW_BODY=$(echo "$OLD_BODY" | sed "/---/i\\$ROW")
             gh api --method PATCH "/repos/${{ github.repository }}/issues/comments/$COMMENT_ID" -f body="$NEW_BODY"
           else
             # Create new comment (shouldn't happen as requirements runs first)
-            COMMENT="$MARKER
+            COMMENT=$(cat <<EOF
+          $MARKER
           ## Reviews
 
           | Review | Status | Summary |
           |--------|--------|---------|
           $ROW
 
           ---
-          *Claude Code CI*"
+          *Claude Code CI*
+          EOF
+          )
             gh pr comment "$PR" --body "$COMMENT"
           fi

PLAN.md

@@ -1 +1 @@
-/home/ben/.claude/plans/compiled-puzzling-crane.md
+/home/ben/.claude/plans/robust-tickling-starlight.md