Skip to content

fix: log full deterministic step output on failure for cloud visibility#716

Merged
khaliqgant merged 1 commit into
mainfrom
fix/deterministic-step-logging
Apr 10, 2026
Merged

fix: log full deterministic step output on failure for cloud visibility#716
khaliqgant merged 1 commit into
mainfrom
fix/deterministic-step-logging

Conversation

@khaliqgant
Copy link
Copy Markdown
Member

@khaliqgant khaliqgant commented Apr 10, 2026

Summary

  • Log the command being run for every deterministic step
  • On failure, log full stdout/stderr via this.log() before throwing — lands in runner.log and is visible in agent-relay cloud logs
  • Covers both code paths: executor (cloud/Daytona) and local spawn

Problem

When a deterministic step fails in cloud, the user sees only:

Step "compile" failed: Command failed with exit code 1

The actual error output is truncated to 500 chars in the exception message and never written to runner.log, making cloud failures opaque.

After this fix

[workflow 00:01] [compile] Running: npm run build
[workflow 00:02] [compile] Command failed (exit code 1)
[workflow 00:02] [compile] stderr:
src/index.ts(42,5): error TS2322: Type 'string' is not assignable to type 'number'.

Test plan

  • Run a cloud workflow with a deterministic step that fails — full output now visible in agent-relay cloud logs
  • Run locally with a failing deterministic step — same improvement in terminal output

🤖 Generated with Claude Code


Open with Devin

When a deterministic step fails, the error message truncates output to
500 chars and never writes to runner.log. This makes cloud failures
opaque — `agent-relay cloud logs` shows only "Command failed with exit
code 1" with no detail.

Now logs the command being run, and on failure logs full stdout/stderr
via this.log() so it lands in runner.log and is visible in cloud logs.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

@khaliqgant khaliqgant merged commit 620e15f into main Apr 10, 2026
37 checks passed
@khaliqgant khaliqgant deleted the fix/deterministic-step-logging branch April 10, 2026 18:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant