EP
← All projectsElements Project

Core Lightning

Modular C implementation of the Lightning Network protocol and node stack.

BitcoinLightning NetworkActive incident
Repository coverage

2009 commits in the local evidence base

Every captured commit receives deterministic security triage and a separate communication-quality score. Security candidates and broader second-pass signals receive full-patch Ollama analysis.

184security candidates280second-pass queue1894AI analyses
56commits · 30 days
211commits · 60 days
683commits · 180 days
1671commits · 365 days
Backfill bands
Aug 5 → Feb 61049 seen37 candidatesComplete
Feb 6 → Jun 6579 seen10 candidatesComplete
Jun 6 → Jul 678 seen1 candidatesComplete
Jul 6 → Aug 5147 seen16 candidatesComplete
Commit communication

Does the history explain itself?

Message quality measures whether a commit identifies its scope, purpose, rationale, testing, and supporting references. It does not change the security-severity score.

70/100 average clarity
489Strong · 80–100
1231Adequate · 60–79
259Thin · 40–59
30Opaque · 0–39
Read the scoring rubric →
Developer activity

Who is changing the project?

Public Git author strings; identities are not independently verified.

DeveloperCommitsCandidatesAnalyzedHigh riskMessage avg.
cdecker14910384
Níckolas Goline63844277
Rusty Russell96475958172
daywalker90118895165
Lagrang386578170
Vincenzo Palazzo13411182
dovgopoly15015164
Sangbida Chaudhuri14931146066
Dusty Daemon821378066
ShahanaFarooqui931093063
Peter Neuroth1024101071
Christian Decker36436072
Analysis record

Published AI watches

Last scanned 23 minutes ago

Informational 15 AI analysisMessage 73 · Adequate
EP Elements ProjectCore Lightning BitcoinLightning Network

contrib: add jaonoctus's signing key

This commit simply adds a new person's PGP public-key fingerprint to the list of trusted release signers in two documentation files. It does not change any code, fix any bug, or alter any security behavior of the software itself.

41278b4eby jaonoctus+2−02 files
No security note in commit
Informational 15 AI analysisMessage 73 · Adequate
EP Elements ProjectCore Lightning BitcoinLightning Network

wallet: fixed timestamps for invented bookkeeper deposits

This is a tiny internal fix to make a bookkeeping migration produce stable event ordering. It changes the timestamp used when creating historical 'deposit' records during a one-time database migration, so the records sort consistently with…

498e5086by daywalker90+2−01 file
No security note in commit
Informational 21 AI analysisMessage 85 · Strong
EP Elements ProjectCore Lightning BitcoinLightning Network

docs: document missing and fix erroneous JSON-RPC error codes

This commit is a documentation-only cleanup of the JSON-RPC command help files. It corrects which numeric error codes are listed for each command so the published schemas match what the software actually returns. No program logic, validati…

Documentation-only change with no executable code modificationsCorrects RPC schema error-code metadata to match actual handler behaviorNo change to input parsing, authorization, cryptography, or network behavior
d1b6963bby daywalker90+204−1224 files
No security note in commit
Low 46 AI analysisMessage 73 · Adequate
EP Elements ProjectCore Lightning BitcoinLightning Network

channeld: initialize tx_sigs_allowed on startup

This commit fixes a bug where a setting that controls whether unexpected transaction signatures are allowed was not initialized when the channel daemon starts. If a peer sent such signatures before the channel was fully ready, the program …

use of uninitialized variableundefined behavior (invalid bool load)network-triggered code path
c09c4520by Erick Cestari+1−01 file
No security note in commit
Informational 15 AI analysisMessage 100 · Strong
EP Elements ProjectCore Lightning BitcoinLightning Network

ci: fix the coverage nightly's profile handling

This commit fixes the project's internal nightly code-coverage CI workflow. It changes how test coverage files are collected, ensures the same LLVM compiler version is used to generate and merge coverage data, and uploads a Codecov-compati…

5685c9a3by cdecker+12−211 file
No security note in commit
Low 37 AI analysisMessage 78 · Adequate
EP Elements ProjectCore Lightning BitcoinLightning Network

schemas: make required fields match the code

This commit is a large cleanup of Core Lightning's API schemas, generated RPC bindings, and related plugin code. The stated goal is to make the documented 'required' fields match what the C code actually always produces or expects. In prac…

Large schema-only change with no accompanying security advisory or CVEOne semantic change to plugin hook response: invoice_payment hook can now reject with only failure_message and no resultMany fields change from optional to required in public RPC/protobuf interfaces
ddbd2963by daywalker90+1786−173950 files
No security note in commit
Informational 15 AI analysisMessage 73 · Adequate
EP Elements ProjectCore Lightning BitcoinLightning Network

doc: document error 313 on every command which can return it

This commit only updates documentation. It adds error code 313 to the documented error lists for several Core Lightning commands (fundpsbt, utxopsbt, txprepare, multiwithdraw, and upgradewallet). The error code already existed in the code …

a34b9758by Peter Neuroth+30−86 files
No security note in commit
Low 25 AI analysisMessage 88 · Strong
EP Elements ProjectCore Lightning BitcoinLightning Network

tools/build-release.sh: name the checksum file when verifying the signature

This commit fixes a release-script check that verifies the cryptographic signature on a file of checksums. Previously, the script only told GPG to verify the signature file itself. If someone replaced that signature file with an inline-sig…

Incorrect cryptographic verification logic in release toolingPotential false-positive signature verification with inline-signed .asc substitutionRelease-integrity hardening
8e495707by Peter Neuroth+5−21 file
No security note in commit
Low 42 AI analysisMessage 95 · Strong
EP Elements ProjectCore Lightning BitcoinLightning Network

doc: pass the manifest explicitly to gpg --verify

This commit updates Core Lightning's release documentation to tell users and release managers to run gpg --verify with both the signature file and the manifest file named explicitly. The old one-argument form can silently succeed even if t…

Verification bypass risk in release artifact validationgpg --verify single-argument form can exit 0 without reading the intended manifestDocumentation-only hardening of release process
2dee8699by Peter Neuroth+9−62 files
Vendor flagged security relevance
Low 25 AI analysisMessage 58 · Thin
EP Elements ProjectCore Lightning BitcoinLightning Network

Cargo.lock: update dependencies

This commit is a routine update to the Rust dependency lock file (Cargo.lock), bumping many third-party libraries to newer patch or minor versions. The commit message gives no security reason for the update, and no verified references link…

Routine dependency refresh with no stated security rationaleUpdates to security-sensitive transitive crates (rustls, hyper, h2, tokio, webpki-roots) but no evidence these versions fix known vulnerabilitiesNo source-code changes or patch-specific fixes visible in the diff
c1551c55by daywalker90+298−2761 file
No security note in commit
High 80 AI analysisMessage 85 · Strong
EP Elements ProjectCore Lightning BitcoinLightning Network

common: bound JSON nesting depth when parsing

This commit fixes a stack-overflow risk in Core Lightning's JSON parser. Before the fix, an attacker could send a valid JSON-RPC message containing thousands of nested brackets or braces. The parser's own helper functions used recursion fo…

Stack-overflow via deeply nested JSONRecursive JSON traversal without depth boundDenial-of-service vector in JSON-RPC input parsing
83f92e4bby Níckolas Goline+90−22 files
Vendor flagged security relevance
Informational 15 AI analysisMessage 28 · Opaque
EP Elements ProjectCore Lightning BitcoinLightning Network

common: minor typo fixes

This commit only fixes typos and comment style. It changes two C-style comments from // to /* */ and corrects a grammar error in a documentation comment ('element' to 'elements'). There are no code behavior changes, no bug fixes, and no se…

273d7f02by w3lld1+3−32 files
No security note in commit
Informational 20 AI analysisMessage 90 · Strong
EP Elements ProjectCore Lightning BitcoinLightning Network

pyln-testing: truncate long testnames

This change fixes a test-infrastructure bug in Core Lightning's Python testing helpers. When running tests against a PostgreSQL database, very long test names could be silently shortened by PostgreSQL, causing different test runs or nodes …

No security-relevant signal: change is in test framework code onlyFixes a test reliability issue, not a runtime vulnerabilityNo input sanitization, authentication, cryptography, or network changes
c4fe4cbaby Peter Neuroth+59−12 files
No security note in commit
Moderate 60 AI analysisMessage 73 · Adequate
EP Elements ProjectCore Lightning BitcoinLightning Network

wallet: exclude immature coinbase outputs from fee rescue selection

This fix prevents Core Lightning from trying to use freshly created bitcoins (immature coinbase rewards) as emergency funds for fee-bump transactions. Such a transaction would be invalid under Bitcoin's rules and would be rejected by the n…

Consensus-invalid transaction generation preventedFee rescue / CPFP failure mode mitigatedCoinbase maturity check added to coin selection
34883ff7by cdecker+5−12 files
Vendor flagged security relevance
Moderate 59 AI analysisMessage 91 · Strong
EP Elements ProjectCore Lightning BitcoinLightning Network

askrene: fix crash loading node bias with description

This commit fixes a bug in Core Lightning's askrene plugin that could prevent a node from restarting. When a saved routing layer contained a node bias with a description, the plugin accidentally freed the description's memory while using i…

Use-after-free / double-take of a tal-allocated string during plugin startupDenial-of-service-like symptom: lightningd aborts before replying to init, node cannot restartFixes publicly reported issue #9433 by endothermicdev
4681177dby Vincenzo Palazzo+2−32 files
No security note in commit
Informational 12 AI analysisMessage 91 · Strong
EP Elements ProjectCore Lightning BitcoinLightning Network

tests: restart the node which owns the layer in node bias test

This commit only fixes a test case so it actually exercises the intended code path. It does not change any production code, so it cannot introduce or fix a real-world security vulnerability by itself. The test change is a reproducer for a …

0f3f67a0by Vincenzo Palazzo+3−21 file
No security note in commit
High 78 AI analysisMessage 68 · Adequate
EP Elements ProjectCore Lightning BitcoinLightning Network

dualopend: bound the feerates a peer opens at

This commit fixes a bug in Core Lightning's experimental dual-funded channel feature. When another node tried to open a channel, Core Lightning was not checking whether the proposed transaction fees were reasonable. A peer could request a …

Missing input validation on wire-parsed feerate fieldsPeer could induce signing and storage of feerate == 0RBF remote path allowed unbounded upward feerate walks
ba08356fby cdecker+140−04 files
Vendor flagged security relevance
High 72 AI analysisMessage 100 · Strong
EP Elements ProjectCore Lightning BitcoinLightning Network

tests: regression coverage for out-of-range feerates

This commit adds regression tests for three related bugs where wildly wrong Bitcoin transaction feerates could enter Core Lightning. In the worst case, a malicious or broken fee source could make the node think a feerate was zero (due to a…

Integer overflow in feerate conversion (u32 wrap from 0xFFFFFFFF perkb to 0 perkw)Absurd feerate from external fee source bypassing sanity ceilingDatabase-stored out-of-range feerate causing startup abort/crash loop
65bdbc47by cdecker+137−02 files
No security note in commit
Moderate 64 AI analysisMessage 86 · Strong
EP Elements ProjectCore Lightning BitcoinLightning Network

lightningd: don't assert on funding feerates read from the db

This update fixes a crash bug in Core Lightning. When the software tried to list details of a channel opening in progress, it could crash if a stored fee rate was extremely large or zero. The crash happened because the code used an interna…

Integer overflow in RBF escalation (u32 * 25 / 24) leading to assertion failureAssertion failure in read-only introspection RPC (listpeerchannels) causing crash-loop at startupDatabase value treated as invariant despite originating from external fee estimator
d67c4936by cdecker+72−227 files
Vendor flagged security relevance
Moderate 53 AI analysisMessage 86 · Strong
EP Elements ProjectCore Lightning BitcoinLightning Network

wallet: clamp absurd inflight funding feerates on upgrade

This commit fixes a bug where Core Lightning nodes could get stuck in a crash loop. If a node had previously stored an extremely high or zero fee rate for an in-progress channel funding operation (a 'splice' or dual-funded channel RBF), a …

Integer overflow in fee-rate calculation (u32 overflow when multiplying by 25/24)Assertion failure leading to daemon crash loop at startupDatabase migration clamps out-of-range stored funding feerates
f057c692by cdecker+34−01 file
Vendor flagged security relevance
Repository ledger

Explore captured commits

Expand any commit for its author, full message, clarity score, changed files, triage signals, analysis, and source link.

Lower-prioritylightningd: fail in-flight HTLCs upstream when dropping withheld channelby Peter Neuroth · 1c1e83e4 · Mar 19, 2026 · 2 filesMessage 85 · StrongModerate 60Details
Commit message · Peter Neuroth

lightningd: fail in-flight HTLCs upstream when dropping withheld channel

When a withheld channel is force-closed (e.g. due to CLTV timeout),
in-flight HTLCs were being freed without failing them back to the
upstream sender. This caused the upstream peer to eventually
force-close its own channel waiting for a resolution that would
never come.

Iterate over all offered HTLCs on the withheld channel in
drop_to_chain() and fail them back upstream with
permanent_channel_failure before freeing them.

Changelog-fixed: withheld channel now fail back incoming inflight htlc

85/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Explains rationale or failure mode
AI analysis · Moderate 60/100

This commit fixes a bug in Core Lightning where money-transfer promises (HTLCs) stuck in a special 'withheld' channel were silently dropped when that channel was force-closed. Previously, the upstream sender was never told the payment failed, so it would eventually force-close its own channel waiting for a resolution that would never arrive. The patch now explicitly fails those HTLCs back upstream before freeing them, preventing unnecessary channel force-closures and fund lockups.

Lower-prioritylightningd: add preimage to forward_event notificationby Peter Neuroth · 13608c5a · Mar 19, 2026 · 5 filesMessage 73 · AdequateInformational 19Details
Commit message · Peter Neuroth

lightningd: add preimage to forward_event notification

Adds the preimage to the forward_event notification when status is
settled. This can be useful to gather some context when listening to
this event

Changelog-Changed: forward_event notification no has preimage set if
status is settled.

Signed-off-by: Peter Neuroth <pet.v.ne@gmail.com>

73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Informational 19/100

This change adds the payment preimage to a plugin notification that fires when a forwarded Lightning payment succeeds. The preimage is already known to the node at that moment, so nothing secret is being exposed that wasn't already available internally. It is a feature enhancement, not a security fix or vulnerability.

Security candidatelightningd: unreserve UTXOs from withheld funding PSBT when dropping channelby Peter Neuroth · 05736f96 · Mar 19, 2026 · 2 filesMessage 85 · StrongLow 28Details
Commit message · Peter Neuroth

lightningd: unreserve UTXOs from withheld funding PSBT when dropping channel

When a channel using a withheld funding PSBT is dropped, we need to
unreserve the UTXOs from the PSBT. Otherwise those funds will remain
blocked until the reservation expires, delaying their availability for
other transactions.

Changelog-Fixed: lightningd: unreserve UTXOs from withheld funding PSBT
on channel close.

85/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Explains rationale or failure mode
Why it was queued
signing boundary
AI analysis · Low 28/100

This fix ensures that when a user opens a 'zero-confirmation' channel whose funding transaction is deliberately withheld and then the channel is closed/aborted, any bitcoin inputs that had been set aside (reserved) for that funding transaction are released back to the wallet promptly. Before this fix, those coins could stay locked up until an automatic timeout expired, making them unavailable for other payments or channel opens.

Lower-priorityreckless: fail reckless rpc if executable failsby Lagrang3 · daa994f6 · Mar 18, 2026 · 1 fileMessage 68 · AdequateInformational 21Details
Commit message · Lagrang3

reckless: fail reckless rpc if executable fails

If the execution of reckless fails for some reason recklessrpc plugin
should return an rpc failure message instead of hanging.

Changelog-Fixed: JSON-RPC: reckless command no longer hangs if reckless executable is not found in PATH.

Signed-off-by: Lagrang3 <lagrang3@protonmail.com>

68/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Informational 21/100

This commit fixes a bug in Core Lightning's 'reckless' plugin command. Previously, if the external 'reckless' program could not be launched (for example, because it was missing from the system PATH), the RPC command would hang indefinitely instead of reporting an error. The patch makes the plugin return a clear failure message when the program cannot be executed.

Lower-priorityfixup! reestablish: Send announcement sigs when askedby Dusty Daemon · 9fb10b89 · Mar 17, 2026 · 1 fileMessage 50 · ThinInformational 23Details
Commit message · Dusty Daemon

fixup! reestablish: Send announcement sigs when asked

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI analysis · Informational 23/100

This is a small code cleanup in Core Lightning's channel gossip handling. It changes two helper functions so they no longer return a true/false success value. The actual behavior—when to send announcement signatures and when to stop on errors—appears unchanged. The commit title says 'fixup!' which typically means it is polishing an earlier commit rather than fixing a new bug. There is no direct evidence this is a security fix.

Lower-prioritysplice: Handle CHANNEL_READY during splice resumeby Dusty Daemon · 7068c301 · Mar 17, 2026 · 1 fileMessage 68 · AdequateLow 34Details
Commit message · Dusty Daemon

splice: Handle CHANNEL_READY during splice resume

In some situations Eclair sends CHANNEL_READY before resuming the splice in reestablish. This update allows one of these messages to be processed if it comes in during the splice resume process.

Changelog-None

68/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Low 34/100

This change fixes a compatibility issue between Core Lightning and the Eclair Lightning node implementation during a channel-splicing reconnect. Previously, Core Lightning would fail the connection if Eclair sent a CHANNEL_READY message while a splice was being resumed. The patch teaches Core Lightning to accept and process that out-of-order message instead of treating it as an error. It is a protocol-handling fix rather than a memory-safety or cryptographic bug, and the commit message frames it as interoperability, not security.

Lower-priorityfixup! splice: Handle CHANNEL_READY during splice resumeby Dusty Daemon · 3acbaf14 · Mar 17, 2026 · 1 fileMessage 50 · ThinLow 34Details
Commit message · Dusty Daemon

fixup! splice: Handle CHANNEL_READY during splice resume

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI analysis · Low 34/100

This is a small follow-up fix to a recent splice-resume feature in Core Lightning's channel daemon. It removes a hard abort when an unexpected ERROR message arrives during a splice, and instead lets the daemon handle certain allowed out-of-order messages (like CHANNEL_READY) while waiting for the expected splice messages. The change is defensive and appears aimed at preventing the node from crashing during legitimate splice resume flows.

AI review queuedsplice: RPC error handling fixby Dusty Daemon · d871ff3b · Mar 17, 2026 · 1 fileMessage 80 · StrongModerate 51Details
Commit message · Dusty Daemon

splice: RPC error handling fix

Splice commands can fail because of external reasons, for example our peer rejecting the splice. These external reasons eat the active `splice_command` so we need to stop storing the `splice_command` in callbacks and instead pull it from the `splice_commands` list which automatically removes the object when its consumed.

This corrects a lightningd crash under certain RPC circumstances.

Changelog-None

80/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Explains rationale or failure mode
Why it was queued
second-pass: broader security terminology
AI analysis · Moderate 51/100

This patch fixes a crash in Core Lightning's lightningd daemon that could occur during splice-related RPC operations. The root cause was that splice command state could be consumed by external failures (like a peer rejecting the splice) while callbacks still held a stale pointer to it. The fix stops storing the splice command pointer inside callback data structures and instead looks it up fresh from the active list when needed, avoiding use-after-free-style crashes.

Security candidatereestablish: Send announcement sigs when askedby Dusty Daemon · 60cd23e3 · Mar 17, 2026 · 7 filesMessage 76 · AdequateLow 28Details
Commit message · Dusty Daemon

reestablish: Send announcement sigs when asked

A new TLV in `channel_reestablish` lets our peer request announcement signatures. This updates CLN to (re)send them when requested.

As per splice spec PR https://github.com/lightning/bolts/pull/1160/changes#diff-ed04ca2c673fd6aabde69389511fa9ee60cb44d6b2ef6c88b549ffaa753d6afeR3303-R3308

Changelog-None

76/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Links an issue, advisory, or supporting reference
Why it was queued
signing boundarysigning or wallet path
AI analysis · Low 28/100

This commit changes how Core Lightning handles reconnection with another Lightning node. It adds a way for the peer to ask for 'announcement signatures' again after reconnecting, and makes the node resend them when asked. This is part of a protocol update for splicing. It is not a traditional security fix; it is a protocol compliance/robustness change that could help prevent channel announcement issues after reconnections or splices.

Lower-prioritysplice: Fix log messageby Dusty Daemon · 5a910996 · Mar 17, 2026 · 1 fileMessage 43 · ThinInformational 15Details
Commit message · Dusty Daemon

splice: Fix log message

Update log message to match the behavior of the check its logging about.

Changelog-None

43/100 · ThinMessage clarity
✓ Subject identifies a change✓ Provides an explanatory body
AI analysis · Informational 15/100

This commit fixes a log message in Core Lightning's channel daemon so it correctly lists all three message types the code accepts. It also adds a check during peer reconnection and aborts if an error message is received. The changes are defensive hardening and logging corrections, not a fix for an exploitable security vulnerability.

Lower-priorityaskrene: log reservation failures during getroutesby Lagrang3 · e34c4a6e · Mar 17, 2026 · 1 fileMessage 73 · AdequateLow 31Details
Commit message · Lagrang3

askrene: log reservation failures during getroutes

Log when a reservation removal failures during getroutes computation.
Failed reservation removals can lead to reservation leaks.

Changelog-None

Signed-off-by: Lagrang3 <lagrang3@protonmail.com>

73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Low 31/100

This change adds logging when the Core Lightning routing plugin (askrene) fails to remove a temporary channel-capacity reservation. Previously these failures were silently ignored, which could cause reservations to 'leak' and remain in place longer than intended. The patch only adds diagnostics; it does not fix the underlying removal failure or the leak itself.

Lower-priorityaskrene-unreserve: add dev_remove_all optionby Lagrang3 · f7553137 · Mar 17, 2026 · 11 filesMessage 60 · AdequateInformational 22Details
Commit message · Lagrang3

askrene-unreserve: add dev_remove_all option

Changelog-None

Signed-off-by: Lagrang3 <lagrang3@protonmail.com>

60/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
AI analysis · Informational 22/100

This commit adds a hidden developer-only option called dev_remove_all to the askrene-unreserve command. When enabled, it clears all capacity reservations managed by the Askrene plugin instead of only unreserving the specific path provided. The feature is marked as hidden and intended for development/testing use. There is no indication in the commit that this fixes a security vulnerability; it appears to be a convenience tool for developers.

Lower-priorityaskrene: add test for reservation leaksby Lagrang3 · b8bc77c4 · Mar 17, 2026 · 1 fileMessage 70 · AdequateInformational 12Details
Commit message · Lagrang3

askrene: add test for reservation leaks

Changelog-None.

Signed-off-by: Lagrang3 <lagrang3@protonmail.com>

70/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body✓ Mentions testing or verification
AI analysis · Informational 12/100

This commit only adds a new automated test to check that a routing-reservation leak does not occur. It does not change any production code, so by itself it cannot introduce or fix a security vulnerability in running software. The test may be related to a known bug, but the commit message and diff do not describe a fix or disclose security relevance.

AI review queuedpyln-testing: move mnemonic dependency directly into pyln-testingby daywalker90 · 2566266b · Mar 17, 2026 · 3 filesMessage 75 · AdequateInformational 20Details
Commit message · daywalker90

pyln-testing: move mnemonic dependency directly into pyln-testing

When installing only pyln-testing directly it is missing the mnemonic
dependency.

Changelog-None

75/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body✓ Mentions testing or verification
Why it was queued
second-pass: broader security terminology
AI analysis · Informational 20/100

This commit is a routine packaging fix. It moves a Python library called 'mnemonic' from the main project's dependency list into the dependency list of a smaller sub-package (pyln-testing). This ensures that anyone installing only pyln-testing gets the missing library. There is no security vulnerability or code behavior change here.

Lower-prioritycommon: add a new WILL_SEND flag to HTLC states.by Rusty Russell · 795b4bc7 · Mar 17, 2026 · 2 filesMessage 68 · AdequateInformational 13Details
Commit message · Rusty Russell

common: add a new WILL_SEND flag to HTLC states.

This is useful for splicing: given an HTLC state, do we need to send
more messages to get it into a non-pending state?

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>

68/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Informational 13/100

This commit is a straightforward internal code change in Core Lightning's bookkeeping for payment commitments (HTLCs). It adds a new flag called WILL_SEND to track whether more messages still need to be exchanged before a payment update is fully settled. The change is described by the developer as groundwork for the 'splicing' feature and contains no obvious security fix or vulnerability.

Lower-prioritypytest: fix flake in test_misc.by Rusty Russell · 9810c4e1 · Mar 17, 2026 · 2 filesMessage 78 · AdequateInformational 13Details
Commit message · Rusty Russell

pytest: fix flake in test_misc.

I got a flake, but all I see is:
```

> assert suspended == set()
E AssertionError: assert {'c26485f2839c5a27'} == set()
E
E Extra items in the left set:
E 'c26485f2839c5a27'
E
E Full diff:
E - set()
E + {
E + 'c26485f2839c5a27',
E + }

tests/test_misc.py:5049: AssertionError
```

Add some more diagnostics, but it's also clear that something is being
suspended and not terminating before we finish.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>

78/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Mentions testing or verification
AI analysis · Informational 13/100

This is a test-only reliability fix. A developer saw an occasional test failure where one background tracing call had not finished by the time the test ended. The patch adds a diagnostic line to the trace output and relaxes the test assertion to allow exactly one suspended call at shutdown, rather than requiring zero. There is no indication this affects real users, funds, or network security.

Lower-prioritychanneld: allow outgoing messages through even if they have send STFU.by Rusty Russell · 8e8027d4 · Mar 17, 2026 · 1 fileMessage 85 · StrongLow 43Details
Commit message · Rusty Russell

channeld: allow outgoing messages through even if they have send STFU.

Otherwise, we can hang: we don't send commitment_signed, and they're
waiting to receive it.

1. We defer fee updates, blockheight updates and master requests
(adding and closing htlcs) if we're *trying* or *started* to quiesce.
2. We only stop actually sending commitment_signed if we have sent
STFU.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Changelog-EXPERIMENTAL: Protocol: avoid an occasional hang when splicing with a pending closing HTLC.

85/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Explains rationale or failure mode
AI analysis · Low 43/100

This patch fixes a hang in Core Lightning's experimental splicing feature. Previously, once a node decided to enter 'quiet' mode (STFU) for splicing, it would stop sending all outgoing channel update messages, including the commitment_signed message the other side was waiting for. The fix lets messages that are already in flight go out, and only blocks brand-new activity once STFU has actually been sent. This prevents the two peers from deadlocking during a splice when a closing payment is still pending.

Lower-prioritychanneld: simplify pending_updates() using HTLC_F_WILL_SENDby Rusty Russell · 5197c441 · Mar 17, 2026 · 3 filesMessage 65 · AdequateLow 27Details
Commit message · Rusty Russell

channeld: simplify pending_updates() using HTLC_F_WILL_SEND

And remove `uncommitted_ok` flag which was always false.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>

65/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
AI analysis · Low 27/100

This commit simplifies how Core Lightning decides whether a channel still has unfinished updates before entering a quiet 'STFU' mode. It removes an 'uncommitted_ok' option that was never actually used as true, and replaces a hand-rolled check with a clearer flag named HTLC_F_WILL_SEND. The change also adds temporary debug logging that prints each HTLC's state when STFU is considered. There is no direct evidence in the commit that this fixes a security bug, but any change to state-machine logic around pending updates can affect protocol correctness.

Lower-prioritychore: replace v26.03 version mentions with v26.04by daywalker90 · e63b0afd · Mar 17, 2026 · 8 filesMessage 77 · AdequateInformational 17Details
Commit message · daywalker90

chore: replace v26.03 version mentions with v26.04

Allowing deprecated API's should actually allow them in test_sql_deprecated

77/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
AI analysis · Informational 17/100

This is a routine release-version bump from v26.03 to v26.04 across documentation, generated schema files, and one source file. It also fixes a test so that a deprecated database column is actually allowed when deprecated APIs are enabled. There is no security vulnerability here; it is ordinary release housekeeping.

Lower-prioritymsggen: propagate added and deprecated fieldsby daywalker90 · 2044a91e · Mar 17, 2026 · 17 filesMessage 68 · AdequateInformational 19Details
Commit message · daywalker90

msggen: propagate added and deprecated fields

Whenever we want to add a new rpc command or notification in msggen's `util.py` we get an error during `make`
that the `added` field of that new command or notification is missing even when it's clearly not.
We had to uncomment 2 lines in `patch.py` and override any missing `added` annotation to get it running.
We now pass any `added` or `deprecated` fields from the parent object correctly so we no longer have to do this.

Unfortunately we have commited wrong metadata to `.msggen.json` in the past.
This commit also includes missing `added` annotation for commands added after the `pre-v0.10.1` cutoff and
manual fixes of the metadata in `.msggen.json`

68/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Informational 19/100

This commit fixes a code-generator bug in Core Lightning's message generator (msggen). Previously, newly added RPC commands and fields could fail to build because version metadata wasn't inherited from parent objects. The patch makes the generator propagate 'added' and 'deprecated' annotations correctly and updates the generated RPC model files, gRPC proto, and JSON schema metadata to match. It is a build/tooling correctness fix, not a runtime security patch.

Lower-prioritymsggen: raise oldest supported version to v24.11by daywalker90 · c25183f8 · Mar 17, 2026 · 6 filesMessage 45 · ThinInformational 19Details
Commit message · daywalker90

msggen: raise oldest supported version to v24.11

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI analysis · Informational 19/100

This commit updates Core Lightning's automatic message-schema generator (msggen) so that the oldest supported protocol/API version is raised from v0.10.1 to v24.11. As a result, many fields that were previously marked optional in the generated gRPC/JSON-RPC bindings are now marked required, because modern Core Lightning always returns them. This is a routine compatibility and code-generation cleanup, not a fix for an exploitable vulnerability.

Lower-prioritylighting-cli: fix invalid jsonby Lagrang3 · d365b6d0 · Mar 16, 2026 · 2 filesMessage 68 · AdequateLow 37Details
Commit message · Lagrang3

lighting-cli: fix invalid json

Changelog-Fixed: lightning-cli: fix invalid json requests when input includes a numeric string with leading zeroes

Signed-off-by: Lagrang3 <lagrang3@protonmail.com>

68/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Low 37/100

This commit fixes a bug in the lightning-cli command-line tool. Previously, if a user typed a numeric string starting with zero (like '0123'), the tool would send it to the lightningd server as a bare number. In JSON, numbers with leading zeros are invalid, so plugins written in Rust or Python could crash when parsing the request. The fix forces such strings to be quoted, making them valid JSON strings instead of invalid numbers. A previously-skipped test that checks for valid JSON from the CLI is re-enabled.

Lower-prioritytests: add test to validate jsonrpcby Lagrang3 · 3b58201c · Mar 16, 2026 · 2 filesMessage 78 · AdequateInformational 11Details
Commit message · Lagrang3

tests: add test to validate jsonrpc

Add test to validate jsonrpc from lightning-cli requests in the case of
numeric strings with leading zeroes.

Changelog-None

Signed-off-by: Lagrang3 <lagrang3@protonmail.com>

78/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Mentions testing or verification
Why it was queued
defensive validation
AI analysis · Informational 11/100

This commit only adds a new test case (currently skipped) and a tiny helper plugin. It does not change any production code, so it cannot by itself fix or introduce a security vulnerability. The test is meant to check that the lightning-cli command-line tool sends JSON values correctly when a numeric-looking string has a leading zero, which some JSON parsers reject as invalid integers.

Lower-priorityxpay: handle pay ignoring unknown argumentsby Lagrang3 · 7c9fb7e0 · Mar 16, 2026 · 2 filesMessage 68 · AdequateLow 26Details
Commit message · Lagrang3

xpay: handle pay ignoring unknown arguments

Changelog-Fixed: xpay: handle payment redirected from "pay" even if we don't recognize some arguments.

Signed-off-by: Lagrang3 <lagrang3@protonmail.com>

68/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Low 26/100

This change makes the 'xpay' payment plugin more willing to take over payments that were originally sent to the older 'pay' command. Previously, if 'pay' received an argument that xpay did not understand, xpay would refuse to handle the payment and the older 'pay' logic would run instead. Now xpay logs that it does not recognize the argument, ignores it, and still redirects the payment through xpay. This is a behavior change, not a clear-cut security fix, and could in theory cause payments to be processed with options that xpay silently drops.

Lower-prioritycln-grpc: fix txids encoding in close commandby daywalker90 · 6fa4d320 · Mar 13, 2026 · 3 filesMessage 45 · ThinInformational 24Details
Commit message · daywalker90

cln-grpc: fix txids encoding in close command

Changelog-None

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI analysis · Informational 24/100

This commit fixes how transaction IDs (txids) are encoded when the 'close' command response is sent over the gRPC interface in Core Lightning. Previously, the txids were likely passed through as plain strings instead of being decoded from hex into raw bytes, which could cause the gRPC layer to produce malformed or unexpected data for clients. The fix also updates the code generator so future txid fields are handled correctly in both directions.