Ahh because instance in launch_run is new (https://github.com/dagster-io/dagster/commit/f711a5bce92f08c18f56fa20999366690c0b2788#diff-d959f2aebbc66e2c809f152cde30aaa341fa3da77c8bed38f2b31611bf5a84bb) and SyncInMemoryLauncher was just never fixed 😄

Having an instance available in initialize() is the newer change actually (that PR is an update to make it no longer a breaking change). RunLaunchers have access to an instance in initialize if they want because they also control termination and you might want to terminate a run (which could require an instance) before you launch a run. The instances should always be the same (prefer the initialize one probably, at some point the one in launch_run could be removed but we'll give plenty of warning if we do)

Ahh gotcha