ci: security review: increase timeout: 20 -> 60 min
What changed, and why it matters
This commit simply increases the timeout for an automated security review script from 20 minutes to 60 minutes. It does not change any wallet, networking, or cryptographic code that could affect users. There is no security vulnerability here.
No action needed. This is a benign CI configuration change.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The change modifies a single constant, CLAUDE_TIMEOUT_SECONDS, in contrib/ci/claude_security_review.py from 2060 to 6060. This script is part of the CI pipeline and invokes an LLM-based security review. The change only affects how long the CI job waits for the LLM response before failing. It does not alter Electrum’s runtime behavior, consensus logic, transaction handling, or any user-facing functionality.
Changed components
contrib/ci/claude_security_review.pyInspect captured patch +1 / −1
diff --git a/contrib/ci/claude_security_review.py b/contrib/ci/claude_security_review.py
index 1579afe..b0d1d8b 100644
--- a/contrib/ci/claude_security_review.py
+++ b/contrib/ci/claude_security_review.py
@@ -34,7 +34,7 @@ SCRIPT_DIR = os.path.dirname(os.path.abspath(__file__))
PROMPT_FILE = os.path.join(SCRIPT_DIR, "security_review_prompt.md")
MAX_DIFF_CHARS = 800_000
-CLAUDE_TIMEOUT_SECONDS = 20 * 60
+CLAUDE_TIMEOUT_SECONDS = 60 * 60
CLAUDE_MODEL = "claude-opus-4-7"
CLAUDE_EFFORT = "max"
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.