X Tutup
Skip to content

refactor(core): declare explicit reactive node prototypes types#67510

Open
JoostK wants to merge 1 commit intoangular:mainfrom
JoostK:signals/flow-reactive-node-type
Open

refactor(core): declare explicit reactive node prototypes types#67510
JoostK wants to merge 1 commit intoangular:mainfrom
JoostK:signals/flow-reactive-node-type

Conversation

@JoostK
Copy link
Member

@JoostK JoostK commented Mar 8, 2026

These type annotations allow TS to associate the object's properties with their corresponding declaration in the interfaces, enabling much better code navigation. For example, "Find all implementations" for ReactiveNode.producerRecomputeValue now finds the implementation in COMPUTED_NODE and LINKED_SIGNAL_NODE.

@JoostK JoostK added the target: patch This PR is targeted for the next patch release label Mar 8, 2026
@pullapprove pullapprove bot requested review from atscott and mturco March 8, 2026 10:53
@angular-robot angular-robot bot added area: core Issues related to the framework runtime requires: TGP This PR requires a passing TGP before merging is allowed labels Mar 8, 2026
@ngbot ngbot bot added this to the Backlog milestone Mar 8, 2026
These type annotations allow TS to associate the object's properties
with their corresponding declaration in the interfaces, enabling
much better code navigation. For example, "Find all implementations"
for `ReactiveNode.producerRecomputeValue` now finds the implementation
in `COMPUTED_NODE` and `LINKED_SIGNAL_NODE`.
@JoostK JoostK force-pushed the signals/flow-reactive-node-type branch from f352552 to 9c18598 Compare March 8, 2026 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: core Issues related to the framework runtime requires: TGP This PR requires a passing TGP before merging is allowed target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

X Tutup