Skip to content

Fix Docker sandbox exec for AgentBeats runtime#345

Open
Yiminnn wants to merge 4 commits into
mainfrom
codex/agentbeats-a2a-adapter-audit
Open

Fix Docker sandbox exec for AgentBeats runtime#345
Yiminnn wants to merge 4 commits into
mainfrom
codex/agentbeats-a2a-adapter-audit

Conversation

@Yiminnn
Copy link
Copy Markdown
Collaborator

@Yiminnn Yiminnn commented May 22, 2026

Summary

  • disable pseudo-TTY allocation for Docker sandbox exec commands with
  • keep multi-service exec behavior intact
  • unblocks SkillsBench AgentBeats worker runs from nested/non-interactive runners

Verification

  • ...... [100%]
    6 passed in 0.25s
  • .........sss.....ssssssssss..s....................... [100%]
    39 passed, 14 skipped in 0.30s

This branch is used by via a pinned commit for the AgentBeats runtime image.


Open in Devin Review

@Yiminnn
Copy link
Copy Markdown
Collaborator Author

Yiminnn commented May 22, 2026

This branch is now the BenchFlow dependency for SkillsBench AgentBeats runtime-first smoke.

Evidence:

The functional runtime fix remains the Docker compose exec -T change; the latest commit only updates a scene metadata test fixture for the A2A role metadata already emitted by the SDK.

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 6 additional findings.

Open in Devin Review

@bingran-you bingran-you added the review:pending PR is ready-for-review, no reviewer engagement yet. label May 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review:pending PR is ready-for-review, no reviewer engagement yet.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants