Skip to content

refactor(desktop): simplify remote window plumbing#29367

Open
chrisnojima wants to merge 1 commit into
nojima/HOTPOT-sharing-cleanupfrom
nojima/HOTPOT-remote-window-clean
Open

refactor(desktop): simplify remote window plumbing#29367
chrisnojima wants to merge 1 commit into
nojima/HOTPOT-sharing-cleanupfrom
nojima/HOTPOT-remote-window-clean

Conversation

@chrisnojima

Copy link
Copy Markdown
Contributor

The main process now owns a remote-window registry and a cache of the last props sent per window. Windows pull cached props when they load, which replaces the wants-props handshake through the main window (and its config-store counter map) and the racy find-window-by-URL retry loop.

  • darkMode now rides along in the serialized props envelope and is applied by the component loader, instead of being hand-plumbed through every proxy and wrapper
  • remote action handlers live in a module-level map instead of a zustand store, so they survive the store reset at logout (the old registry silently dropped pinentry/tracker handlers there)
  • all remote windows share one webpack entry (remote.html?component=) with per-component lazy chunks, replacing four separate entries

The main process now owns a remote-window registry and a cache of the
last props sent per window. Windows pull cached props when they load,
which replaces the wants-props handshake through the main window (and
its config-store counter map) and the racy find-window-by-URL retry
loop.

- darkMode now rides along in the serialized props envelope and is
  applied by the component loader, instead of being hand-plumbed
  through every proxy and wrapper
- remote action handlers live in a module-level map instead of a
  zustand store, so they survive the store reset at logout (the old
  registry silently dropped pinentry/tracker handlers there)
- all remote windows share one webpack entry (remote.html?component=)
  with per-component lazy chunks, replacing four separate entries
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