-
Notifications
You must be signed in to change notification settings - Fork 4.1k
fix: use env vars for safe string handling in continue-agents workflow #9425
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 "Update Check Run" step was interpolating step outputs directly into JavaScript template literals, which breaks when the agent output contains backticks, template literal interpolation sequences, or other special characters that interfere with JS string parsing. This caused failures like "SyntaxError: Unexpected identifier 'mobile'" when the agent output contained multi-line text with certain keywords. Fix by passing outputs as environment variables and accessing them via process.env, which safely handles all string content without parsing issues. 🤖 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 |
2 similar comments
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
|
🎉 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
continue-agents.ymlworkflow when agent output contains special charactersProblem
The workflow was failing with errors like
SyntaxError: Unexpected identifier 'mobile'because agent output was being interpolated directly into JavaScript template literals:When the output contains backticks,
${}sequences, or certain line breaks, it breaks the JavaScript syntax.Solution
Pass outputs as environment variables and access them via
process.env:Environment variables are passed as-is without string parsing issues.
Test plan
🤖 Generated with Claude Code
Summary by cubic
Use environment variables in the continue-agents workflow to safely handle agent output and prevent JavaScript parsing errors. This avoids failures when output includes backticks, ${}, or multi-line content and ensures the check run updates reliably.
Written for commit fe1a35a. Summary will update on new commits.