Skip to content

Conversation

@sestinj
Copy link
Contributor

@sestinj sestinj commented Jan 5, 2026

Summary

  • Fixed workflow escaping issues that caused failures when agent output contains backticks
  • Changed from direct GitHub expression interpolation to environment variables throughout the workflow

Problem

When agent output contains markdown with backticks (e.g., LogsTabContent.tsx), the workflow failed with errors like:

LogsTabContent.tsx: command not found
SyntaxError: Unexpected identifier 'LogsTabContent'

This happened because:

  1. Shell issue: Backticks in echo statements triggered command substitution
  2. JavaScript issue: Backticks in template literals broke JavaScript syntax

Solution

Pass all dynamic values via env: blocks instead of direct interpolation. Environment variables are properly escaped by GitHub Actions, preventing both shell and JavaScript parsing issues.

Test plan

  • Trigger workflow with agent output containing backticks in markdown
  • Verify job summary renders correctly
  • Verify check run updates successfully

🤖 Generated with Claude Code


Summary by cubic

Prevented failures in the continue-agents workflow when agent output includes backticks by routing all dynamic values through environment variables. This ensures proper escaping, fixes shell substitution and JS template literal errors, and keeps job summaries and check runs working.

  • Bug Fixes
    • Updated steps to read values from env: Discover agents, Extract agent name, Create Check Run, Run agent, Write job summary, Update Check Run.
    • Use process.env in github-script and env vars in shell to avoid unsafe interpolation.
    • Job summary and check run updates now safely handle markdown and backticks.

Written for commit 98254b1. Summary will update on new commits.

Agent output containing backticks (e.g., `LogsTabContent.tsx`) caused
workflow failures due to:
1. Shell command substitution in echo statements
2. JavaScript template literal syntax errors

This fix passes all dynamic values via environment variables instead
of direct GitHub expression interpolation, which properly escapes
special characters.

🤖 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 5, 2026 22:44
@sestinj sestinj requested review from Patrick-Erichsen and removed request for a team January 5, 2026 22:44
@continue
Copy link
Contributor

continue bot commented Jan 5, 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

@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

@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Jan 5, 2026
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 closed this Jan 6, 2026
@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
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

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