Skip to content

fix: make SSR hydration interactions reliable - #83

Merged
smiggleworth merged 2 commits into
mainfrom
fix/askr-windows-ssr-hydration
Jul 25, 2026
Merged

fix: make SSR hydration interactions reliable#83
smiggleworth merged 2 commits into
mainfrom
fix/askr-windows-ssr-hydration

Conversation

@smiggleworth

Copy link
Copy Markdown
Contributor

Fixes #82.

Hydrated event handlers now publish as direct element listeners during the SSR-to-client handoff. Client-created and subsequent re-rendered elements retain delegated listeners. This avoids browser-host differences in delegated-container propagation while preserving normal delegation after hydration.

Validation: full unit, checks, jsdom, browser, lint/typecheck, build, and installed type gates pass locally.

Copilot AI review requested due to automatic review settings July 25, 2026 21:04

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adjusts SSR hydration listener installation to improve cross-browser reliability (notably addressing Windows CI failures) by ensuring hydrated event handlers are attached as direct element listeners during the SSR-to-client handoff.

Changes:

  • Add a forceDirect path to addTrackedListener and use it during hydration publication.
  • Prevent hydration-staged listeners from publishing as delegated listeners (even when delegation is enabled), to avoid delegated-container propagation inconsistencies.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/renderer/prop-bindings.ts
@smiggleworth
smiggleworth force-pushed the fix/askr-windows-ssr-hydration branch from 8d4896d to c29134a Compare July 25, 2026 21:21
@smiggleworth
smiggleworth merged commit 2ce44d1 into main Jul 25, 2026
7 of 9 checks passed
@smiggleworth
smiggleworth deleted the fix/askr-windows-ssr-hydration branch July 25, 2026 21:39
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.

fix: Windows SSR navigation hydration does not attach interactive handlers

2 participants