Skip to content

Passthrough _pydevd_bundle to sandbox when debug mode is enabled#1550

Closed
elidlocke wants to merge 2 commits into
temporalio:mainfrom
elidlocke:rebase-1249
Closed

Passthrough _pydevd_bundle to sandbox when debug mode is enabled#1550
elidlocke wants to merge 2 commits into
temporalio:mainfrom
elidlocke:rebase-1249

Conversation

@elidlocke
Copy link
Copy Markdown

Rebase of #1249, original work by @VegetarianOrc. The original PR was approved
on 2026-01-05 with green CI, but has been stale since February with merge
conflicts against main. This rebases it on top of current main with no
behavioral changes to the original commit; the only diff vs. the original
PR is:

  • Resolved the import-block conflict in temporalio/worker/_workflow.py
    (kept all four imports added on both sides).
  • Resolved the end-of-file conflict in tests/worker/test_worker.py (kept
    the new tests added to main, then test_worker_debug_mode after them).

Also adds a short docs note under "Workflow Sandbox" -> "Debugging Workflows
in the Sandbox" pointing users at debug_mode=True for IDE-debugger support.

The original commit by @VegetarianOrc is preserved as-is with author attribution.

Opening as a draft to confirm CI passes before requesting review.

VegetarianOrc and others added 2 commits May 21, 2026 11:02
…ugpy is used by the Microsoft authored VSCode python debugger
Adds a short subsection to the Workflow Sandbox documentation pointing
users at debug_mode=True (or the TEMPORAL_DEBUG env var) for attaching
IDE debuggers to sandboxed workflow code.
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 21, 2026

CLA assistant check
All committers have signed the CLA.

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.

3 participants