Skip to content

Remove engineCommandManager, rustContext from singletons#10252

Merged
franknoirot merged 10 commits intomainfrom
franknoirot/9956/remove-ecm-rc-singletons
Mar 6, 2026
Merged

Remove engineCommandManager, rustContext from singletons#10252
franknoirot merged 10 commits intomainfrom
franknoirot/9956/remove-ecm-rc-singletons

Conversation

@franknoirot
Copy link
Contributor

They now hang off of the currently-executing editor (kclManager is what we call that today), so just replace uses of these two subsystems with them through kclManager. Split out from #10173 as this is the noisy "change a bunch of files" update after that hard logic migration.

franknoirot and others added 4 commits February 26, 2026 19:08
… return

Separated out from #10119 because it's so dang noisy. Towards #9956, and
evaporating `buildSingletons()` into nothing. Simply accesses these
systems through `kclManager`, which was always close at hand wherever
they were anyway.
@vercel
Copy link

vercel bot commented Mar 2, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
modeling-app Ready Ready Preview, Comment Mar 6, 2026 5:04pm

Request Review

@franknoirot franknoirot force-pushed the franknoirot/9956/remove-ecm-rc-singletons branch from 6e3567a to 5202e5f Compare March 4, 2026 03:14
They're available off of the currently-executing editor (`kclManager` is
what we call that today).
@franknoirot franknoirot changed the base branch from main to franknoirot/9956/update-sceneInfra-sceneentities-use March 4, 2026 05:08
@franknoirot franknoirot force-pushed the franknoirot/9956/remove-ecm-rc-singletons branch from 5202e5f to 8893b24 Compare March 4, 2026 05:08
@franknoirot franknoirot marked this pull request as ready for review March 5, 2026 01:30
@franknoirot franknoirot requested a review from a team as a code owner March 5, 2026 01:30
Base automatically changed from franknoirot/9956/update-sceneInfra-sceneentities-use to main March 6, 2026 13:56
@franknoirot franknoirot merged commit 4a03327 into main Mar 6, 2026
58 checks passed
@franknoirot franknoirot deleted the franknoirot/9956/remove-ecm-rc-singletons branch March 6, 2026 18:08
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