Skip to content

refactor: move subprocess worker to workers/ directory#340

Merged
Blackgaurd merged 1 commit into
mainfrom
move-subprocess-worker
May 16, 2026
Merged

refactor: move subprocess worker to workers/ directory#340
Blackgaurd merged 1 commit into
mainfrom
move-subprocess-worker

Conversation

@Blackgaurd
Copy link
Copy Markdown
Collaborator

What changed

Moved runtime/isolated_worker.pyworkers/subprocess_worker.py and the corresponding test file from tests/runtime/tests/workers/.

Updated runtime/runner.py to point WORKER_PATH at the new location.

Why

isolated_worker.py lived in runtime/ alongside server-side infrastructure (config loading, registry, venv management), but it's fundamentally different: it's a standalone subprocess entry point that runs in a separate Python interpreter, never imported by the server. A dedicated workers/ directory makes this distinction explicit.

Benefit

Clearer separation of concerns in the directory structure. runtime/ is now purely server-side infrastructure; workers/ holds subprocess entry points.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 16, 2026

@Blackgaurd Blackgaurd merged commit a20135b into main May 16, 2026
4 checks passed
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