Skip to content

Conversation

@sestinj
Copy link
Contributor

@sestinj sestinj commented Jan 6, 2026

Summary

  • Fixes JavaScript parsing errors in the continue-agents.yml workflow when agent output contains special characters
  • Changes the "Update Check Run" step to use environment variables instead of direct string interpolation

Problem

The workflow was failing with errors like SyntaxError: Unexpected identifier 'mobile' because agent output was being interpolated directly into JavaScript template literals:

const output = `${{ steps.run.outputs.output }}`;  // ❌ Breaks with special chars

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:

env:
  AGENT_OUTPUT: ${{ steps.run.outputs.output }}
const output = process.env.AGENT_OUTPUT || '';  // ✅ Safe handling

Environment variables are passed as-is without string parsing issues.

Test plan

  • Verified locally that env var approach correctly handles: multi-line text, backticks, template literal syntax, quotes
  • Re-run the failing workflow in remote-config-server to confirm the fix

🤖 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.

  • Bug Fixes
    • Pass outputs as env vars: AGENT_OUTPUT, AGENT_ERROR, AGENT_SUCCESS, CHECK_RUN_ID.
    • Read values via process.env and parse CHECK_RUN_ID as an integer.
    • Remove template-literal interpolation to avoid syntax errors from special characters.

Written for commit fe1a35a. Summary will update on new commits.

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>
@sestinj sestinj requested a review from a team as a code owner January 6, 2026 22:07
@sestinj sestinj requested review from RomneyDa and removed request for a team January 6, 2026 22:07
@continue
Copy link
Contributor

continue bot commented Jan 6, 2026

All Green - Keep your PRs mergeable

Learn more

All Green is an AI agent that automatically:

✅ Addresses code review comments

✅ Fixes failing CI checks

✅ Resolves merge conflicts


Unsubscribe from All Green comments

2 similar comments
@continue-staging
Copy link

All Green - Keep your PRs mergeable

Learn more

All Green is an AI agent that automatically:

✅ Addresses code review comments

✅ Fixes failing CI checks

✅ Resolves merge conflicts


Unsubscribe from All Green comments

@continue-development-app
Copy link

All Green - Keep your PRs mergeable

Learn more

All Green is an AI agent that automatically:

✅ Addresses code review comments

✅ Fixes failing CI checks

✅ Resolves merge conflicts


Unsubscribe from All Green comments

@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Jan 6, 2026
@github-actions
Copy link

github-actions bot commented Jan 6, 2026

✅ Review Complete

Code Review Summary

⚠️ AI review failed. Please check the Continue API key and configuration.

Troubleshooting

  • Verify the CONTINUE_API_KEY secret is set correctly
  • Check that the organization and config path are valid
  • Ensure the Continue service is accessible

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a 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

@sestinj sestinj merged commit 22041c5 into main Jan 6, 2026
57 of 60 checks passed
@sestinj sestinj deleted the fix/continue-agents-js-parsing branch January 6, 2026 22:30
@github-project-automation github-project-automation bot moved this from Todo to Done in Issues and PRs Jan 6, 2026
@github-actions github-actions bot locked and limited conversation to collaborators Jan 6, 2026
@sestinj
Copy link
Contributor Author

sestinj commented Jan 13, 2026

🎉 This PR is included in version 1.38.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@sestinj
Copy link
Contributor Author

sestinj commented Jan 14, 2026

🎉 This PR is included in version 1.37.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

released size:S This PR changes 10-29 lines, ignoring generated files.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants