Add durable Strands sandbox support by brianstrauch · Pull Request #1768 · temporalio/sdk-python

brianstrauch · 2026-08-20T02:48:29Z

Summary

add an experimental activity-backed TemporalSandbox implementation for the Strands sandbox API
register lazy, worker-lifetime sandbox factories through StrandsPlugin
buffer execution streams for deterministic replay and optionally publish raw StreamChunk values through Workflow Streams
preserve Strands timeout and missing-path errors across the activity boundary
vend the standard sandbox bash and file-editor tools with user overrides
document lifecycle, payload, retry, streaming, and environment-value considerations

Testing

poe test -s -k strands
poe lint
git diff --check

Copilot

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

patbeqo · 2026-08-27T19:05:19Z

@codex

chatgpt-codex-connector · 2026-08-27T19:09:11Z

Codex Review: Didn't find any major issues. Swish!

Reviewed commit: ef2c383e3c

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

patbeqo · 2026-08-28T21:16:12Z

+# workflow
+agent = TemporalAgent(
+    sandbox=TemporalSandbox(
+        "build",


Now that we are using SandboxWorkflowContext do we need the name?

Suggested change

"build",

The name selects the corresponding sandbox defined in StrandsPlugin(sandboxes=...), whereas the context contains the Workflow ID.

Could the factory function handle all sandbox retrievals instead of name map?

TemporalAgent( sandbox=TemporalSandbox(start_to_close_timeout=timedelta(minutes=5)) ) ... plugins=[StrandsPlugin(sandboxes=build_sandbox)]

The name map is good because it's what we already do for MCP, and it allows us to have multiple sandboxes in the same environment.

patbeqo · 2026-09-01T14:15:36Z

@codex

chatgpt-codex-connector · 2026-09-01T14:15:52Z

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review	Status	Commit	Review trigger
📝 Code Review	✅ Completed 2026-09-01T14:23:58.272519Z	`18d709b`	Manual request

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 18d709b210

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

patbeqo · 2026-09-01T14:34:39Z

+    value: dict[str, Any]
+
+
+class _SandboxRecord:


Can the cache be replaced with a call to the user-provided factory for every activity? The factory can then resolve the correct sandbox using the current context and implement caching appropriate to its backend. This would simplify the lifecycle and cancellation logic, and keep caching policy with the component that understands sandbox identity and cleanup.

I like the idea in principle, but if you were making millions of tool calls you'd have millions of factory calls. It's possible that factory calls are expensive, which could force users to have to implement some type of caching mechanism in their factories to avoid the cost.

xumaple

Overall looks good but provided some comments

xumaple · 2026-09-04T17:38:01Z

+
+
+def _timeout_error(err: SandboxTimeoutError, timeout: float | None) -> ApplicationError:
+    # A timeout is the deterministic outcome the caller asked for, so retrying


A timeout is the deterministic outcome the caller asked for

It is? Isn't timeout the definition of something we should retry?

Agreed. Better to retry by default, and the user can just set a retry policy with 0 retries if they don't want it.

In the context of a sandbox, most commands are deterministic (writing to a file for instance) and seeing a timeout once usually means you'll see it again.

xumaple · 2026-09-04T17:39:29Z

+    ExecutionResult,
+    FileInfo,
+    Sandbox,
+    StreamChunk,


Raw StreamChunk values lack:

sandbox name;

workflow operation/execution ID;

Activity attempt;

chunk sequence;

command versus code execution.

That makes a shared topic difficult to consume correctly. “Use different topics” pushes correlation onto callers and
becomes awkward for concurrent agent tool calls.

I would publish an envelope such as:

SandboxStreamEvent(
sandbox_name=...,
execution_id=...,
attempt=...,
sequence=...,
chunk=...,
)

Will do! Making execution_id=workflow_run_id+activity_id so it can work across continue as new

# Conflicts: # pyproject.toml

brianstrauch requested review from a team as code owners August 20, 2026 02:48

brianstrauch requested a balanced review from Copilot August 20, 2026 03:02

Copilot AI reviewed Aug 20, 2026

patbeqo reviewed Aug 27, 2026

View reviewed changes

Comment thread temporalio/contrib/strands/README.md Outdated

patbeqo reviewed Aug 28, 2026

View reviewed changes

Comment thread temporalio/contrib/strands/_sandbox_activity.py

brianstrauch requested a review from patbeqo August 31, 2026 16:11

chatgpt-codex-connector Bot reviewed Sep 1, 2026

View reviewed changes

Comment thread temporalio/contrib/strands/_sandbox_activity.py Outdated

Comment thread temporalio/contrib/strands/_sandbox_activity.py Outdated

Comment thread temporalio/contrib/strands/_sandbox_activity.py Outdated

Comment thread temporalio/contrib/strands/_plugin.py Outdated

patbeqo reviewed Sep 1, 2026

View reviewed changes

brianstrauch requested a review from patbeqo September 1, 2026 21:01

patbeqo requested a review from a team September 4, 2026 13:11

xumaple reviewed Sep 4, 2026

View reviewed changes

brianstrauch requested a review from xumaple September 10, 2026 23:28

brianstrauch force-pushed the strands-temporal-sandbox branch from 195375b to 42920d2 Compare September 14, 2026 22:37

Add durable Strands sandbox support

6a92f65

brianstrauch force-pushed the strands-temporal-sandbox branch from 42920d2 to 6a92f65 Compare September 14, 2026 22:37

brianstrauch added 4 commits September 14, 2026 19:21

Merge remote-tracking branch 'origin/main' into strands-temporal-sandbox

e023802

# Conflicts: # pyproject.toml

Move Strands sandbox changelog entry to Unreleased

fa7ea9d

Remove Strands sandbox adapter cache

08bb73e

Merge remote-tracking branch 'origin/main' into strands-temporal-sandbox

48435c5

patbeqo approved these changes Sep 17, 2026

View reviewed changes

brianstrauch commented Sep 17, 2026

View reviewed changes

Comment thread CHANGELOG.md Outdated

Apply suggestion from @brianstrauch

bccce00

brianstrauch commented Sep 17, 2026

View reviewed changes

Comment thread CHANGELOG.md Outdated

Apply suggestion from @brianstrauch

0d4fb7b

tconley1428 approved these changes Sep 17, 2026

View reviewed changes

brianstrauch merged commit 46c6f25 into main Sep 17, 2026
46 of 49 checks passed

brianstrauch deleted the strands-temporal-sandbox branch September 17, 2026 22:51



		def _timeout_error(err: SandboxTimeoutError, timeout: float \| None) -> ApplicationError:
		# A timeout is the deterministic outcome the caller asked for, so retrying

Conversation

brianstrauch commented Aug 20, 2026

Summary

Testing

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

patbeqo commented Aug 27, 2026

Uh oh!

chatgpt-codex-connector Bot commented Aug 27, 2026

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

patbeqo commented Sep 1, 2026

Uh oh!

chatgpt-codex-connector Bot commented Sep 1, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codex Review Summary

Uh oh!

chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

xumaple left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

chatgpt-codex-connector Bot commented Sep 1, 2026 •

edited

Loading