Skip to content

Conversation

@sestinj
Copy link
Contributor

@sestinj sestinj commented Jan 6, 2026

Summary

  • Fixes the "Write job summary" step which was still failing with backticks in agent output
  • Uses environment variables + printf instead of direct interpolation with echo

Problem

After #9425 fixed the "Update Check Run" step, the "Write job summary" step was still failing with:

syntax error near unexpected token `newline'

This happened because backticks in the agent output (like `<!-- Testing -->`) were being interpreted as bash command substitution when using:

echo "${{ steps.run.outputs.output }}"

Solution

Same pattern as the previous fix:

  1. Pass outputs as environment variables
  2. Use printf '%s\n' "$VAR" instead of echo "$VAR" to avoid any special character interpretation

Test plan

  • Verified locally that printf correctly handles backticks, quotes, and other special characters
  • Re-run the failing workflow to confirm

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

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>
@sestinj sestinj requested a review from a team as a code owner January 6, 2026 23:52
@sestinj sestinj requested review from Patrick-Erichsen and removed request for a team January 6, 2026 23:52
@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

1 similar comment
@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

- 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>
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. and removed size:S This PR changes 10-29 lines, ignoring generated files. labels Jan 6, 2026
@sestinj sestinj merged commit a0fdf94 into main Jan 6, 2026
44 of 45 checks passed
@sestinj sestinj deleted the nate/fix-again branch January 6, 2026 23:57
@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:M This PR changes 30-99 lines, ignoring generated files.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants