Skip to content

feat: support lazy environment activation in NodeEnvManager#2841

Merged
idoros merged 1 commit intomainfrom
ido/lazy-env-activation
Mar 12, 2026
Merged

feat: support lazy environment activation in NodeEnvManager#2841
idoros merged 1 commit intomainfrom
ido/lazy-env-activation

Conversation

@idoros
Copy link
Contributor

@idoros idoros commented Mar 12, 2026

This PR Split worker environment initialization into two phases to support lazy startup:

  1. preLoad - spawn worker and set up communication
  2. activate - send activate rpc, wait for ready

Changes

  • Add lazy parameter to autoLaunch - when true, environments are pre-loaded but not activated until new activateEnvs() is called explicitly
  • Fire a new activate rpc call to notify environment
  • keep backwards compatibility with WorkerThreadInitializer2.initialize() as a convenience that calls both preLoad and activate

@idoros idoros requested a review from barak007 March 12, 2026 08:36
@idoros idoros self-assigned this Mar 12, 2026
@idoros idoros merged commit a402585 into main Mar 12, 2026
6 checks passed
@idoros idoros deleted the ido/lazy-env-activation branch March 12, 2026 10:24
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.

2 participants