-
Notifications
You must be signed in to change notification settings - Fork 4.1k
fix: use env vars in Write job summary step for safe string handling #9427
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
The Write job summary step had the same issue - backticks in the agent output were being interpreted as bash command substitution, causing 'syntax error near unexpected token' failures. Use environment variables and printf instead of echo to safely handle all special characters in the output. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Learn moreAll Green is an AI agent that automatically: ✅ Addresses code review comments ✅ Fixes failing CI checks ✅ Resolves merge conflicts |
1 similar comment
Learn moreAll Green is an AI agent that automatically: ✅ Addresses code review comments ✅ Fixes failing CI checks ✅ Resolves merge conflicts |
Learn moreAll Green is an AI agent that automatically: ✅ Addresses code review comments ✅ Fixes failing CI checks ✅ Resolves merge conflicts |
|
✅ Review Complete Code Review SummaryTroubleshooting
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No issues found across 1 file
- continue-agents.yml: Use env var for agent name in Create Check Run step (prevents JS template literal breakage with backticks in filenames) - run-continue-agent.yml: Use jq to construct JSON body instead of string interpolation (prevents JSON syntax errors when prompt contains quotes, newlines, backslashes, or other special characters) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
|
🎉 This PR is included in version 1.38.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 1.37.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |

Summary
printfinstead of direct interpolation withechoProblem
After #9425 fixed the "Update Check Run" step, the "Write job summary" step was still failing with:
This happened because backticks in the agent output (like
`<!-- Testing -->`) were being interpreted as bash command substitution when using:Solution
Same pattern as the previous fix:
printf '%s\n' "$VAR"instead ofecho "$VAR"to avoid any special character interpretationTest plan
printfcorrectly handles backticks, quotes, and other special characters🤖 Generated with Claude Code
Summary by cubic
Fix string handling in GitHub workflows to safely handle backticks, quotes, and other special characters and prevent syntax errors.
The job summary now uses env vars and printf; the Check Run step reads the agent name from env; and the agents API request uses jq to build JSON safely.
Written for commit 2d21472. Summary will update on new commits.