ci: check that security and docs are synced
What changed, and why it matters
This commit adds a new automated CI check that verifies two security policy documents stay in sync. It does not change any runtime code, fix a bug, or alter security behavior. It is purely a process improvement for documentation consistency.
No security action needed. This is a benign CI/documentation hygiene change.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The patch adds a check-security Makefile target that diffs SECURITY.md (skipping first 2 lines) against doc/contribute-to-core-lightning/security-policy.md (skipping first 8 lines), and includes that target in the existing check-source target. No source code, protocol handling, cryptography, or configuration is modified.
Changed components
MakefileCI/documentation consistency checksInspect captured patch +6 / −1
diff --git a/Makefile b/Makefile
index 72c8d775..ac1a1c8d 100644
--- a/Makefile
+++ b/Makefile
@@ -667,8 +667,13 @@ check-doc-examples: update-doc-examples
check-wire-format: extract-bolt-csv
git diff --exit-code HEAD
+# SECURITY.md and doc/contribute-to-core-lightning/security-policy.md must be
+# synced. So far we do this manually.
+check-security:
+ @bash -lc 'diff -u <(tail -n +3 -- SECURITY.md) <(tail -n +9 -- doc/contribute-to-core-lightning/security-policy.md)'
+
# This should NOT compile things!
-check-source: check-makefile check-whitespace check-spelling check-python-flake8 check-includes check-shellcheck check-setup_locale check-tmpctx check-discouraged-functions check-amount-access check-bad-sprintf check-wire-format check-source-bolt
+check-source: check-makefile check-whitespace check-spelling check-python-flake8 check-includes check-shellcheck check-setup_locale check-tmpctx check-discouraged-functions check-amount-access check-bad-sprintf check-wire-format check-source-bolt check-security
full-check: check check-source
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.