What the developer wrote
Authored by Hennadii Stepanov
doc: Document IWYU workaround
What changed, and why it matters
This commit only adds a comment explaining why a particular code header must be kept. It does not change any program logic, data handling, or security behavior. There is no security issue here.
No security action needed. This is a documentation-only change.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff adds three lines of documentation in src/core_io.cpp above the #include
Changed components
src/core_io.cppInspect captured patch +3 / −0
diff --git a/src/core_io.cpp b/src/core_io.cpp
index a789d5ca..58989115 100644
--- a/src/core_io.cpp
+++ b/src/core_io.cpp
@@ -10,6 +10,9 @@
#include <consensus/consensus.h>
#include <consensus/validation.h>
#include <key_io.h>
+// IWYU incorrectly suggests replacing this header
+// with forward declarations.
+// See https://github.com/include-what-you-use/include-what-you-use/issues/1886.
#include <primitives/block.h> // IWYU pragma: keep
#include <primitives/transaction.h>
#include <script/descriptor.h>
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.