Two identical declarations of NodeId make it ambiguous for static tools such as IWYU which header should provide the symbol. The first commit moves the declaration into a header of its own.
The second commit fixes the #includes of the net and node/eviction modules, which the first commit touched, and adds the headers that other files previously received transitively from them.
A prerequisite for #36808.
Two identical declarations of `NodeId` make it ambiguous for static
tools such as IWYU which header should provide the symbol. Move the
declaration into a header of its own.
`net.h` re-exports it, since `NodeId` is part of its interface.
The `#include`s added to other files replace ones they previously
received transitively via `net.h` and `node/eviction.h`.
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--006a51241073e994b41acfe9ec718e94-->
<!--021abf342d371248e50ceaed478a90ca-->
See the guideline and AI policy for information on the review process.
| Type | Reviewers |
|---|---|
| ACK | jpk68 |
If your review is incorrectly listed, please copy-paste <code><!--meta-tag:bot-skip--></code> into the comment that the bot should ignore.
<!--5faf32d7da4f0f540f40219e4f7537a3-->
ACK 144b4abf6a877d0ddab54fef823b4119aede7969