Skip to content

fix(examples): repair multi-agent notepad launch - #4807

Merged
TaylorMutch merged 2 commits into
mainfrom
fix/3995-repair-multi-agent-notepad-demo/TaylorMutch
Oct 21, 2026
Merged

TaylorMutch merged 2 commits into
mainfrom
fix/3995-repair-multi-agent-notepad-demo/TaylorMutch

Conversation

@TaylorMutch

@TaylorMutch TaylorMutch commented Oct 21, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Repair the multi-agent notepad demo by creating and uploading each sandbox before starting its worker or synthesis command with sandbox exec. The shipped command previously combined sandbox create --upload with a trailing command, which the CLI rejects before provisioning.

Related Issue

Closes #3995

Changes

  • detach after creating and uploading each demo sandbox
  • run the uploaded worker or synthesis payload through sandbox exec

Release Notes

The multi-agent shared-notepad example now launches on the current CLI.

Testing

  • Checks appropriate to the affected code and behavior pass
  • Unit tests added/updated (not applicable for this example-only command split)
  • E2E tests added/updated (requires a gateway, credentials, and a suitable Codex agent image)

Validated with:

  • bash -n examples/multi-agent-notepad/demo.sh
  • shellcheck -e SC2059 examples/multi-agent-notepad/demo.sh (the excluded informational findings predate this change)
  • git diff --check

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)
  • Architecture docs updated (not applicable)

Signed-off-by: Taylor Mutch <taylormutch@gmail.com>
@github-actions

Copy link
Copy Markdown

Signed-off-by: Taylor Mutch <taylormutch@gmail.com>
@TaylorMutch
TaylorMutch requested a review from purp October 21, 2026 19:23

@purp purp left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM 🚢

@TaylorMutch
TaylorMutch enabled auto-merge October 21, 2026 19:26
@TaylorMutch
TaylorMutch added this pull request to the merge queue Oct 21, 2026
Merged via the queue into main with commit 1d880c9 Oct 21, 2026
81 checks passed
@TaylorMutch
TaylorMutch deleted the fix/3995-repair-multi-agent-notepad-demo/TaylorMutch branch October 21, 2026 19:50
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.

The multi-agent-notepad demo.sh fails because sandbox create rejects --upload together with a command.

2 participants