From d2510fc860628ea8c10c30da8d9fb7a6d2d86297 Mon Sep 17 00:00:00 2001 From: Tam Nhu Tran Date: Sat, 28 Mar 2026 20:24:46 -0400 Subject: [PATCH] fix(ai-review): use printf for subagent prompt outputs, match existing pattern Replace echo with printf '%s\n' for subagent prompt GITHUB_OUTPUT writes to match the existing main prompt pattern (line 216). Prevents edge case where echo misinterprets leading -n/-e as flags. --- .github/workflows/ai-review.yml | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/.github/workflows/ai-review.yml b/.github/workflows/ai-review.yml index 3a536a0a..95f11c93 100644 --- a/.github/workflows/ai-review.yml +++ b/.github/workflows/ai-review.yml @@ -244,22 +244,22 @@ jobs: SECURITY_DELIM="SECURITY_$(openssl rand -hex 16)" echo "security_prompt<<${SECURITY_DELIM}" >> "$GITHUB_OUTPUT" - echo "$SECURITY_PROMPT" >> "$GITHUB_OUTPUT" + printf '%s\n' "$SECURITY_PROMPT" >> "$GITHUB_OUTPUT" echo "${SECURITY_DELIM}" >> "$GITHUB_OUTPUT" QUALITY_DELIM="QUALITY_$(openssl rand -hex 16)" echo "quality_prompt<<${QUALITY_DELIM}" >> "$GITHUB_OUTPUT" - echo "$QUALITY_PROMPT" >> "$GITHUB_OUTPUT" + printf '%s\n' "$QUALITY_PROMPT" >> "$GITHUB_OUTPUT" echo "${QUALITY_DELIM}" >> "$GITHUB_OUTPUT" CCS_DELIM="CCS_$(openssl rand -hex 16)" echo "ccs_prompt<<${CCS_DELIM}" >> "$GITHUB_OUTPUT" - echo "$CCS_PROMPT" >> "$GITHUB_OUTPUT" + printf '%s\n' "$CCS_PROMPT" >> "$GITHUB_OUTPUT" echo "${CCS_DELIM}" >> "$GITHUB_OUTPUT" ADVERSARIAL_DELIM="ADVERSARIAL_$(openssl rand -hex 16)" echo "adversarial_prompt<<${ADVERSARIAL_DELIM}" >> "$GITHUB_OUTPUT" - echo "$ADVERSARIAL_PROMPT" >> "$GITHUB_OUTPUT" + printf '%s\n' "$ADVERSARIAL_PROMPT" >> "$GITHUB_OUTPUT" echo "${ADVERSARIAL_DELIM}" >> "$GITHUB_OUTPUT" - name: Run Claude Code Review