Skip to content

refactor(devtools): make the hydration property of DevToolsNode optional#68872

Open
csmick wants to merge 1 commit into
angular:mainfrom
csmick:optional-hydration
Open

refactor(devtools): make the hydration property of DevToolsNode optional#68872
csmick wants to merge 1 commit into
angular:mainfrom
csmick:optional-hydration

Conversation

@csmick
Copy link
Copy Markdown
Contributor

@csmick csmick commented May 21, 2026

Make the hydration`` property of DevToolsNode`` optional as frameworks like Client-Only Wiz don't have the same hydration concept as Angular and will not set one for nodes returned from getComponentForest.

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • angular.dev application / infrastructure changes
  • Other... Please describe:

What is the current behavior?

The hydration property is required in the DevToolsNode interface.

Issue Number: N/A

What is the new behavior?

The hydration property is optional in the DevToolsNode interface, allowing other frameworks like Client-Only Wiz to create DevToolsNode objects without it.

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@csmick csmick requested a review from dgp1130 May 21, 2026 20:25
@ngbot ngbot Bot added this to the Backlog milestone May 21, 2026
@csmick csmick added the target: rc This PR is targeted for the next release-candidate label May 21, 2026
@csmick csmick force-pushed the optional-hydration branch from 06666bc to 67eb134 Compare May 21, 2026 21:50
Make the `hydration`` property of `DevToolsNode`` optional as frameworks like Client-Only Wiz don't have the same hydration concept as Angular and will not set one for nodes returned from `getComponentForest`.
@csmick csmick force-pushed the optional-hydration branch from 67eb134 to c983927 Compare May 21, 2026 21:51
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Wow that's a lot of mock data!

@JeanMeche JeanMeche added the action: merge The PR is ready for merge by the caretaker label May 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: devtools target: rc This PR is targeted for the next release-candidate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants