LL
← All projectsLightning Labs

LND

Go implementation of a complete Lightning Network node.

BitcoinLightning NetworkNormal
Repository coverage

1407 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.

142security candidates195second-pass queue1287AI analyses
94commits · 30 days
194commits · 60 days
617commits · 180 days
1248commits · 365 days
Backfill bands
Aug 5 → Feb 6405 seen5 candidatesComplete
Feb 6 → Jun 6665 seen28 candidatesComplete
Jun 6 → Jul 689 seen4 candidatesComplete
Jul 6 → Aug 569 seen0 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.

67/100 average clarity
324Strong · 80–100
661Adequate · 60–79
391Thin · 40–59
31Opaque · 0–39
1security candidate with opaque commit messaging
Read the scoring rubric →
Developer activity

Who is changing the project?

Public Git author strings; identities are not independently verified.

DeveloperCommitsCandidatesAnalyzedHigh riskMessage avg.
Olaoluwa Osuntokun20942204474
Elle Mouton26832268167
ziggie45016413165
Yong28513171
LNBiG111173
Jared Tobin909171
bitromortac661153071
yyforyongyu83861068
Erick Cestari26726070
Andras Banki-Horvath11311072
Gijs van Dam33230073
ziggieXXX2226070
Analysis record

Published AI watches

Last scanned 31 minutes ago

Moderate 62 AI analysisMessage 58 · Thin
LL Lightning LabsLND BitcoinLightning Network

Merge pull request #11212 from ziggie1984/disable-legacy-channels

This change stops LND from opening new Lightning channels using the old 'legacy' commitment format. The legacy format makes it harder to recover funds if something goes wrong, because the money owed to you is tied to a secret key that chan…

Prevents opening new channels with the legacy commitment type, whose tweaked to_remote output complicates data-loss recoveryCloses a negotiation path where an empty channel_type TLV bypassed feature checks and forced a legacy channelAdds explicit RPC and wire rejection with a dedicated error code
112cb5f3by ziggieXXX+265−4716 files
Vendor flagged security relevance
Moderate 56 AI analysisMessage 73 · Adequate
LL Lightning LabsLND BitcoinLightning Network

Merge pull request #11223 from gijswijs/legacy-dust-retribution-fix

This update fixes a bug in how LND handles old-style punishment transactions when a channel partner tries to cheat. Previously, tiny (dust) HTLCs were left as blank placeholder entries in the punishment data, which could cause the node to …

nil-pointer dereference risk in breach retribution pathlegacy revocation log handling inconsistency with modern formatdefensive hardening added for trimmed/dust HTLCs
500ca51fby ziggieXXX+168−146 files
No security note in commit
Moderate 62 AI analysisMessage 73 · Adequate
LL Lightning LabsLND BitcoinLightning Network

Merge pull request #11190 from Roasbeef/zpay32-reject-duplicate-payment-hashes

This change tightens how LND reads Lightning invoices (BOLT 11). Previously, if an invoice contained more than one payment hash field, LND would silently keep the first one and ignore the rest. Now it rejects the invoice entirely. The rele…

Behavior change from 'use first duplicate field' to 'reject duplicate fields'New error type ErrDuplicatePaymentHash returned on duplicate payment hash fieldsMalformed/unsupported-length duplicate payment hash now treated as duplicate and rejected
86306f89by Yong+147−106 files
No security note in commit
Moderate 62 AI analysisMessage 98 · Strong
LL Lightning LabsLND BitcoinLightning Network

zpay32: reject duplicate payment hash fields

This change tightens how LND reads Lightning invoices. Previously, if an invoice contained more than one payment hash field, LND would silently keep the first valid one and ignore the rest. Now it rejects such invoices outright. This preve…

Behavioral change from silent first-field acceptance to explicit rejection of duplicate payment hash fieldsPrevents invoice parsing from depending on field ordering when multiple payment hashes are presentCovers malformed first field + valid second field, closing a potential bypass
e2f27063by Olaoluwa Osuntokun+147−106 files
Vendor flagged security relevance
Moderate 53 AI analysisMessage 58 · Thin
LL Lightning LabsLND BitcoinLightning Network

Merge pull request #11132 from yyforyongyu/task-11129-pong

This change fixes how LND answers network 'ping' messages from other Lightning nodes. Previously, LND would sometimes silently ignore valid pings to limit reply traffic, which violates the Lightning protocol (BOLT 1) and could make a peer …

Protocol compliance fix: BOLT 1 requires a Pong for every valid PingRemoved silent suppression of Pong replies that could cause peer timeout/misbehaviorUnified rate limiting now accounts for amplification bandwidth, not just ping count
d3d1f1d0by Olaoluwa Osuntokun+274−1715 files
Vendor flagged security relevance
Low 33 AI analysisMessage 90 · Strong
LL Lightning LabsLND BitcoinLightning Network

itest: stop creating legacy channels

This commit updates LND's integration tests to stop using an obsolete 'legacy' channel type and adds a test confirming that LND now rejects that legacy type. The commit message says the real fix is elsewhere in the same pull request: previ…

Deprecated commitment format (legacy) is now rejected by RPCTest added to verify legacy channel type is refusedUnused legacy node configuration flag removed
03d8b09dby ziggie+24−85 files
No security note in commit
Moderate 60 AI analysisMessage 73 · Adequate
LL Lightning LabsLND BitcoinLightning Network

Merge pull request #11198 from gijswijs/amp-setlocal-reconstruction-fix

This change fixes a bug in LND's handling of AMP (Atomic Multi-Path) payments. Previously, if one partial payment set failed to reconstruct its secret preimage, the entire invoice was canceled, including unrelated partial payment sets from…

Denial-of-service against reusable static AMP invoices by causing unrelated accepted HTLC sets to be canceledIncorrect invoice state transition from set-local failure to global invoice cancellationAvailability impact on multi-payer AMP invoice scenarios
88959aecby Olaoluwa Osuntokun+155−44 files
Vendor flagged security relevance
Low 29 AI analysisMessage 73 · Adequate
LL Lightning LabsLND BitcoinLightning Network

Merge pull request #11200 from ziggie1984/build-go-1.27.1

This commit updates the Go programming language version used to build LND from 1.25.13/1.26.6 to 1.26.8 (language version) and 1.27.1 (toolchain). It also regenerates many automatically generated RPC files and adjusts linter settings. The …

Go version bump may indirectly address unlisted runtime/standard-library security fixesLarge regenerated protobuf files suggest dependency/toolchain refresh rather than logic changeLinter configuration expanded to suppress new checks, consistent with a toolchain upgrade
ef4820e5by Olaoluwa Osuntokun+4932−412670 files
No security note in commit
Informational 15 AI analysisMessage 92 · Strong
LL Lightning LabsLND BitcoinLightning Network

build: adopt focused golangci-lint v2.13 checks

This commit is a routine build-maintenance change. It updates the project's Go linter configuration to a newer version and fixes the resulting style warnings: a WebSocket header spelling and several documentation comments that didn't start…

bada7ea3by ziggie+27−1911 files
No security note in commit
Informational 15 AI analysisMessage 68 · Adequate
LL Lightning LabsLND BitcoinLightning Network

lnrpc: regenerate protos with Go 1.27

This commit is a housekeeping change that regenerates automatically generated RPC binding files using a newer Go compiler version. The only differences are in code comments (doc-comment formatting), not in actual program logic or API behav…

dd523108by ziggie+4501−369826 files
No security note in commit
Informational 20 AI analysisMessage 100 · Strong
LL Lightning LabsLND BitcoinLightning Network

build: fix Go 1.27 compatibility

This commit is a routine build-and-test maintenance patch to make the LND codebase compile and pass tests with the upcoming Go 1.27 release. It upgrades the project's custom linter tool, refreshes many indirect tool dependencies, adjusts o…

No production code changesNo vulnerability fix or CVE reference presentDependency-only refresh in tools module (development tooling)
6b54a387by ziggie+365−3706 files
No security note in commit
Informational 15 AI analysisMessage 80 · Strong
LL Lightning LabsLND BitcoinLightning Network

build: bump Go language and toolchain versions

This commit is a routine maintenance update that changes which version of the Go programming language is used to build the lnd project. It does not change any application logic, fix a bug, or address a security vulnerability. The project i…

b1f9ca66by ziggie+39−3930 files
No security note in commit
Moderate 60 AI analysisMessage 83 · Strong
LL Lightning LabsLND BitcoinLightning Network

invoices: cancel only the failing AMP set on reconstruction failure

This commit fixes a bug in LND's invoice handling for reusable AMP (Atomic Multi-Path) invoices. Previously, if one payment set failed to reconstruct its preimage, the entire invoice was canceled, even though other valid payment sets on th…

Denial-of-service-like availability impact: a malicious or buggy payer could cause a whole reusable AMP invoice to be canceled, blocking legitimate concurrent payersSet-local failure incorrectly escalated to invoice-global cancellationRegression test added for concurrent-set survival after reconstruction failure
d74ab6faby Gijs van Dam+141−42 files
No security note in commit
Informational 15 AI analysisMessage 68 · Adequate
LL Lightning LabsLND BitcoinLightning Network

bolt12: describe the lnwire dependency

This is a documentation-only change. A package comment was updated to accurately describe that the bolt12 codec library does depend on LND's lnwire package for low-level message types, while clarifying it still does not interact with the r…

5f53171dby bitromortac+3−21 file
No security note in commit
Moderate 60 AI analysisMessage 91 · Strong
LL Lightning LabsLND BitcoinLightning Network

bolt12: cover the amount overflow guard

This commit adds a test to ensure that when a BOLT 12 invoice is validated against a request, the multiplication of the offer amount by the requested quantity cannot overflow and wrap around to zero. Without this guard, a malicious or malf…

Integer overflow / wraparound in amount calculationMissing test coverage for security-critical branchBOLT 12 invoice amount validation
3c8e2bcbby bitromortac+47−01 file
Vendor flagged security relevance
Low 34 AI analysisMessage 73 · Adequate
LL Lightning LabsLND BitcoinLightning Network

Merge pull request #11061 from bitromortac/2604-bolt12-1f

This commit adds the missing cryptographic signature layer for BOLT 12 offers in LND. It builds a Merkle tree from the message fields, signs that tree with a BIP-340 Schnorr signature, and now rejects invoice requests and invoices whose si…

Adds BIP-340 Schnorr signature verification for BOLT 12 invoice_request and invoice messagesMerkle tree commits to canonical re-encoded TLV records; non-minimal feature/amount encodings now rejected to preserve byte-exactnessSignature TLV type 240 and reserved range 240-1000 excluded from the signed Merkle root
1049d301by Yong+2076−7815 files
Vendor flagged security relevance
Low 35 AI analysisMessage 73 · Adequate
LL Lightning LabsLND BitcoinLightning Network

Merge pull request #11125 from bhandras/codex/reorg-aware-output-lock-20260825

This commit adds a new optional wallet feature to LND that lets users lock UTXOs until the transaction spending them reaches a chosen confirmation depth, instead of relying on a wall-clock timeout. It is a feature addition with safety chec…

New RPC fields release_after_spend_confs and confirmed_spend_height added to LeaseOutput and FundPsbtCustom lock ID enforced as required for confirmation-controlled FundPsbt leases; all-zero and reserved internal IDs rejectedCapability check (ResolveOutputLeaser) prevents silent downgrade to time-only lease when confirmation control is requested
80dde144by Yong+1293−10021 files
No security note in commit
Informational 19 AI analysisMessage 58 · Thin
LL Lightning LabsLND BitcoinLightning Network

Merge pull request #10812 from ziggie1984/chanstate-kv-store-move

This commit is a large internal code reorganization for the LND Lightning node. It moves channel state storage logic from the `channeldb` package into a new `chanstate` package, while keeping the same database keys, serialization formats, …

Large refactoring with no change to cryptographic or consensus-critical logicCodec serialization moved verbatim to new package; wire format unchangedDatabase bucket keys and TLV record types preserved
62b0a073by Olaoluwa Osuntokun+3923−310924 files
No security note in commit
Low 25 AI analysisMessage 73 · Adequate
LL Lightning LabsLND BitcoinLightning Network

Merge pull request #10804 from GeorgeTsagk/close-immediately-itest-lnd-7c38

This change fixes a data-handling bug in LND's channel-opening code. When opening a Lightning channel using an optional 'auxiliary funding controller' (used for custom channel types such as Taproot Assets), the funding manager was previous…

Data-correctness fix in funding handshake: negotiated channel configs now exposed to auxiliary funding controllerAuxiliary funding controller receives incomplete channel state before patch, complete state after patchNo explicit security claim, CVE, or advisory referenced in commit or supplied materials
1e127653by Olaoluwa Osuntokun+291−254 files
No security note in commit
Moderate 60 AI analysisMessage 88 · Strong
LL Lightning LabsLND BitcoinLightning Network

bolt12: verify invoice_request and invoice signatures on read

This change makes LND's BOLT 12 invoice and invoice-request reader actually check that the digital signature is valid, instead of only checking that a signature field exists. Before, an attacker could craft or modify a BOLT 12 message and …

Missing cryptographic verification on parsed BOLT 12 messages replaced with BIP-340 Schnorr signature verificationReader accepted any 64-byte placeholder signature before the patchNew negative tests verify rejection of wrong-signer and tampered Merkle-root fields
aa8bd19fby bitromortac+272−723 files
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-prioritycontracourt: rename broadcastHeight to confirmHeightby ziggie · 88242446 · Nov 22, 2025 · 2 filesMessage 85 · StrongInformational 15Details
Commit message · ziggie

contracourt: rename broadcastHeight to confirmHeight

The broadcastHeight was misleading because the commit resolver
is only created when the commitment transaction is confirmed.

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

This commit is a simple variable rename from broadcastHeight to confirmHeight, plus updated comments, in the Lightning Network Daemon (LND) contract court code. It does not change any program logic, behavior, or security properties. The rename makes the code easier to understand because the value actually stores the block height at which the commitment transaction was confirmed, not the height it was broadcast.

Lower-prioritylncli: remove dead codeby Boris Nagaev · 031c792b · Nov 20, 2025 · 1 fileMessage 51 · ThinInformational 15Details
Commit message · Boris Nagaev

lncli: remove dead code

Variable chanPoints was accumulated but never used. Found with staticcheck tool.

The code is dead since 8f5d78c875b8eca436f7ee2e86e743afee262386

51/100 · ThinMessage clarity
✓ Subject identifies a change✓ Provides detailed explanatory context
AI analysis · Informational 15/100

This commit simply deletes unused code in the lncli command-line tool. A variable named chanPoints was being built up from channel backup data but never read or used afterward. Removing it has no functional or security effect.

Lower-prioritygraph: add regression test for the fixed behaviourby ziggie · 5225b9bb · Nov 19, 2025 · 1 fileMessage 60 · AdequateModerate 64Details
Commit message · ziggie

graph: add regression test for the fixed behaviour

60/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Mentions testing or verification! No meaningful explanatory body
Why it was queued
fuzzing or regression evidence
AI analysis · Moderate 64/100

This commit only adds a regression test for a previously fixed bug in LND's routing graph cache. The bug caused channels whose two routing policies were initially disabled to be left out of an in-memory cache. Later, if one policy became enabled, the cache could not be updated, so the channel stayed invisible for routing. The test documents the bug and verifies the fix, but does not itself change production code.

Lower-prioritydocs: add release-notes for 20.1by ziggie · 4f40d45d · Nov 19, 2025 · 1 fileMessage 57 · ThinInformational 15Details
Commit message · ziggie

docs: add release-notes for 20.1

57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

This commit only adds a new release notes document for version 0.20.1. It contains no code changes and does not alter how the software behaves. The document is mostly a template with empty sections and one bug-fix note describing a routing issue that was fixed in a separate pull request.

Lower-prioritygraph: fix graph cache population for channels with both policies disabledby ziggie · 575766c5 · Nov 19, 2025 · 1 fileMessage 73 · AdequateModerate 59Details
Commit message · ziggie

graph: fix graph cache population for channels with both policies disabled

Fix a bug where channels with both policies disabled were not added to
the graph cache during startup. When a policy update later re-enabled
one of the directions, the update would succeed in the database but fail
to update the graph cache (since the channel structure was never added),
preventing the channel from being used for routing.

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

This commit fixes a bug in LND's routing graph cache. Previously, if a Lightning channel had both routing policies disabled when the node started up, the channel was never loaded into the in-memory routing cache. Later, if a policy update re-enabled one direction, the database would record the change but the cache would stay out of sync, so the channel could not actually be used for routing. The fix always loads the channel structure into the cache and only skips attaching the disabled policies, allowing later updates to enable it correctly.

Lower-prioritylnwire: implement SerializedSize for onion messagesby ziggie · 84504fa8 · Nov 16, 2025 · 1 fileMessage 50 · ThinInformational 19Details
Commit message · ziggie

lnwire: implement SerializedSize for onion messages

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

This commit adds a missing helper method that reports how many bytes an onion message occupies when serialized. It is a small interface-compliance change with no direct security fix visible in the diff. Any security relevance would come from how downstream code uses this size information, but that behavior is not shown here.

Lower-prioritydocs: move release notes from 0.20.0 to 0.21.0by Boris Nagaev · 86e3b1b0 · Nov 13, 2025 · 2 filesMessage 72 · AdequateInformational 15Details
Commit message · Boris Nagaev

docs: move release notes from 0.20.0 to 0.21.0

Remove the chain notifier RPC note from the 0.20 release notes and add it to
0.21.

72/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

This commit only reorganizes documentation. It moves a release note about a chain notifier RPC behavior change from the 0.20.0 release notes file to the 0.21.0 release notes file. No code was changed, and no security issue is introduced or fixed by this commit itself.

Lower-prioritydocs: add release notesby Nishant Bansal · 36fb79ec · Nov 13, 2025 · 1 fileMessage 55 · ThinInformational 15Details
Commit message · Nishant Bansal

docs: add release notes

Signed-off-by: Nishant Bansal <nishant.bansal.282003@gmail.com>

55/100 · ThinMessage clarity
✓ Subject identifies a change✓ Uses a recognizable type or scope✓ Provides an explanatory body
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

This commit only adds a release note describing a previously merged feature. It does not change any code, configuration defaults, or security behavior. There is no security issue in this documentation update.

Lower-prioritymulti: add new config option `upfront-shutdown-address`by Nishant Bansal · e4c4d946 · Nov 13, 2025 · 5 filesMessage 73 · AdequateInformational 19Details
Commit message · Nishant Bansal

multi: add new config option `upfront-shutdown-address`

Introduced a new config value `upfront-shutdown-address`
in the `lnd.conf` file. This ensures that channel close
funds are transferred to the specified shutdown address.
The value applies to both the funder and the fundee but
can be overridden by the value specified during
`openchannel` or by the `channel acceptor`.

NOTE: If this field is set when opening a channel with a
peer that does not advertise support for upfront shutdown
feature, the channel open will fail.

Signed-off-by: Nishant Bansal <nishant.bansal.282003@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 commit adds a new optional configuration setting, upfront-shutdown-address, to the Lightning Network Daemon (LND). When set, it tells LND to send cooperative channel-closing funds to a specific address by default. It only takes effect if the other peer also supports the upfront shutdown feature; otherwise channel opens fail. The change is a feature addition, not a fix for an active vulnerability, and it does not appear to introduce a security flaw on its own.

Lower-priorityitest: add itest for `upfront-shutdown-address` configby Nishant Bansal · 4c3cff14 · Nov 13, 2025 · 2 filesMessage 65 · AdequateInformational 15Details
Commit message · Nishant Bansal

itest: add itest for `upfront-shutdown-address` config

Signed-off-by: Nishant Bansal <nishant.bansal.282003@gmail.com>

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

This commit only adds a new integration test to the LND project. It checks that when two users set an 'upfront shutdown address' before opening a Lightning channel, their funds correctly go to those addresses when the channel closes. There is no change to production code, no bug fix, and no security patch.

Lower-prioritygraph/db: freeze sql migration queriesby Elle Mouton · 352b4d62 · Nov 12, 2025 · 11 filesMessage 45 · ThinInformational 17Details
Commit message · Elle Mouton

graph/db: freeze sql migration queries

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

This commit is a code-maintenance refactor for LND's graph database migration. It copies a set of auto-generated SQL query helpers from the main SQL package into the migration1 package and updates imports so the migration uses its own frozen copy. The goal is to prevent future changes to the main SQL queries from accidentally breaking the one-time graph-to-SQL migration. There is no direct security fix or vulnerability being patched in the diff itself.

Lower-prioritydocs: update release notesby Elle Mouton · 95b84a89 · Nov 12, 2025 · 1 fileMessage 47 · ThinInformational 15Details
Commit message · Elle Mouton

docs: update release notes

47/100 · ThinMessage clarity
✓ Descriptive subject✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

This commit only updates the release notes document. It adds a bullet point describing a previous code change that froze a database migration, and it adds a contributor's name. There is no actual code change, no bug fix, and no security-related content.

Lower-prioritygo.mod: replace local sqldbby Elle Mouton · 5db82a42 · Nov 12, 2025 · 2 filesMessage 45 · ThinInformational 15Details
Commit message · Elle Mouton

go.mod: replace local sqldb

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

This commit is a routine build-file change. It tells the Go build system to use a local copy of an internal database package (sqldb) instead of fetching a published version from the internet. The commit message explicitly frames this as a temporary workaround for an upcoming feature (gossip V2 sqldb changes). There is no indication of a security fix, vulnerability, or behavior change in the compiled program.

Lower-prioritygraph/db: remove TestPopulateViaMigrationby Elle Mouton · 07d0f084 · Nov 12, 2025 · 1 fileMessage 68 · AdequateInformational 15Details
Commit message · Elle Mouton

graph/db: remove TestPopulateViaMigration

This tests was a temporary helper to let devs test the graph SQL
migration before it was plugged in to LND. But that migration has now
shipped and so we can remove this.

68/100 · AdequateMessage clarity
✓ Descriptive subject✓ Provides detailed explanatory context✓ Mentions testing or verification
AI analysis · Informational 15/100

This commit simply deletes a developer-only helper test that was used to manually test a database migration before it was released. The removed code was never run by default, never shipped to users, and did not change any production logic. It is not a security fix and has no security relevance.

Lower-prioritygraph/db: remove outdated commentby Elle Mouton · 33c3809e · Nov 12, 2025 · 1 fileMessage 45 · ThinInformational 15Details
Commit message · Elle Mouton

graph/db: remove outdated comment

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

This commit only deletes a four-line code comment in a single file. No program logic, data handling, or security behavior changed. It is a documentation cleanup with no security relevance.

Security candidategraph/db: simplify auth proof and edge infoby Elle Mouton · edc021f7 · Nov 12, 2025 · 4 filesMessage 65 · AdequateInformational 15Details
Commit message · Elle Mouton

graph/db: simplify auth proof and edge info

Remove various unused fields and methods.

65/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body✓ Names security-relevant behavior explicitly
Why it was queued
authentication path
AI analysis · Informational 15/100

This commit is a routine internal cleanup in LND's channel graph database code. It removes unused cached signature and public-key fields and their accessor methods, and switches some functions from returning structs by value to returning pointers. There is no change to security-sensitive behavior, no bug fix, and no externally reported issue.

Security candidatemulti: freeze graph SQL migration logicby Elle Mouton · 125325f6 · Nov 12, 2025 · 17 filesMessage 68 · AdequateLow 34Details
Commit message · Elle Mouton

multi: freeze graph SQL migration logic

Copy over all the code that the graph SQL migration needs to a
separate folder. This will let us advance the main graph SQL CRUD code
without worrying about changing the sql migration code. It will also let
us change the SQL queries without changing the migration. In this
commit, only the migration logic is "frozen" but in an upcoming commit,
the sqlc queries & models will be frozen too.

68/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
Why it was queued
authentication path
AI analysis · Low 34/100

This commit is a large internal refactoring of the Lightning Network Daemon (LND) database migration code. It copies existing graph database migration logic into a new, separate package called 'migration1' so future changes to the main graph code won't accidentally alter how old data is migrated. The change itself does not add a new user-facing feature or fix a known security bug; it is preparation work to make future database upgrades safer and easier to maintain.

Lower-prioritymulti: add models.Node V1 constructorby Elle Mouton · 80e70096 · Nov 12, 2025 · 18 filesMessage 60 · AdequateInformational 12Details
Commit message · Elle Mouton

multi: add models.Node V1 constructor

Add a version field to models.Node and a V1 constructor for it.

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

This commit is a large internal refactoring of how Lightning Network node records are created in the LND codebase. It introduces a version field and a new constructor for node objects, then updates many tests and internal callers to use that constructor. There is no direct security fix or vulnerability being patched; it is structural preparation for future gossip protocol versions.

AI review queuedgraph: remove DB interfaceby Elle Mouton · 855e579c · Nov 12, 2025 · 3 filesMessage 35 · OpaqueInformational 15Details
Commit message · Elle Mouton

graph: remove DB interface

35/100 · OpaqueMessage clarity
✓ Descriptive subject! No meaningful explanatory body
Why it was queued
second-pass: opaque commit message
AI analysis · Informational 15/100

This commit is a routine code cleanup: it removes an unused Go interface called DB from the graph package and replaces its one remaining use with a direct pointer to the concrete ChannelGraph type. There is no change to program logic, data handling, or security behavior.

Lower-prioritymodels: simplify models.Nodeby Elle Mouton · a394938d · Nov 12, 2025 · 4 filesMessage 50 · ThinInformational 15Details
Commit message · Elle Mouton

models: simplify models.Node

Simplify the struct by removing un-used methods and outdated comments.

50/100 · ThinMessage clarity
✓ Descriptive subject✓ Provides an explanatory body
AI analysis · Informational 15/100

This is a routine code cleanup in the LND Lightning Network daemon. It removes unused helper methods and outdated comments from the internal graph database node model, and updates callers to use direct field assignments instead. There is no security-relevant change.

Lower-prioritydocs: update release notes for 0.21.0by Gijs van Dam · 93c50bc6 · Nov 12, 2025 · 1 fileMessage 57 · ThinInformational 15Details
Commit message · Gijs van Dam

docs: update release notes for 0.21.0

57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
documentation-only discount
AI analysis · Informational 15/100

This commit only updates release notes documentation. It adds a description of a new feature (onion messaging forwarding) to the 0.21.0 release notes file. No code changes are present, so there is no direct security impact from this patch itself.

Security candidatemulti: remove HaveNodeAnnouncement field from Nodeby Elle Mouton · 8f205e2d · Nov 12, 2025 · 15 filesMessage 70 · AdequateLow 28Details
Commit message · Elle Mouton

multi: remove HaveNodeAnnouncement field from Node

Remove the 2 sources of truth here. If we have a signature for the
node, then we have the announcement.

70/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body✓ Names security-relevant behavior explicitly
Why it was queued
signing boundary
AI analysis · Low 28/100

This commit removes a redundant flag called HaveNodeAnnouncement from LND's internal node records and replaces it with a simple rule: if a node has a stored signature, it has an announcement. It is a code-quality/refactoring change that eliminates two conflicting ways of answering the same question. There is no direct evidence in the commit that this fixes an active security vulnerability, but inconsistent state can historically lead to subtle bugs in how node information is stored, forwarded, or validated.

Lower-prioritymulti: endpoints for onion messagesby Gijs van Dam · 07dc74e1 · Nov 12, 2025 · 9 filesMessage 68 · AdequateLow 28Details
Commit message · Gijs van Dam

multi: endpoints for onion messages

This commit creates the necessary endpoints for onion messages.
Specifically, it adds the following:

- `SendOnionMessage` endpoint to send onion messages.
- `SubscribeOnionMessages` endpoint to subscribe to incoming onion
messages.

It uses the `msgmux` package to handle the onion messages.

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

This commit adds new RPC endpoints to LND that allow users to send and receive 'onion messages'—a type of privacy-preserving Lightning network message. The change is a feature addition, not a bug fix. There is no indication in the commit or supplied references that this resolves a security vulnerability or was disclosed as a security issue.

AI review queuedlnrpc: SendOnionMessage rpc endpointby Gijs van Dam · 2ee07029 · Nov 12, 2025 · 7 filesMessage 35 · OpaqueLow 25Details
Commit message · Gijs van Dam

lnrpc: SendOnionMessage rpc endpoint

35/100 · OpaqueMessage clarity
✓ Descriptive subject! No meaningful explanatory body
Why it was queued
second-pass: opaque commit message
AI analysis · Low 25/100

This commit adds a new gRPC API endpoint called SendOnionMessage to the LND Lightning node, along with a matching subscription stream for incoming onion messages. It only changes the protocol definition and generated code; it does not include the actual server-side implementation that would process these messages. Because the implementation is missing, the change by itself does not create or fix a security vulnerability, but it does expose a new network-facing capability that will need careful review once the backend logic lands.

Lower-prioritychore: thread context through to SendCustomMessageby Gijs van Dam · c37b18cf · Nov 12, 2025 · 2 filesMessage 85 · StrongLow 25Details
Commit message · Gijs van Dam

chore: thread context through to SendCustomMessage

The only way to unblock SendCustomMessage is if the peer activates,
disconnects or the server shuts down. This means that if the context is
cancelled, we will still wait until one of those other events happen.

With this commit we thread the context through to SendCustomMessage, so
that if the context is cancelled, we can return early. This improves the
cancellation semantics.

85/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context
AI analysis · Low 25/100

This change fixes a minor responsiveness bug in LND's 'SendCustomMessage' RPC. Previously, if a caller cancelled the request, the server would still keep waiting until the peer became active, disconnected, or the whole server shut down. Now the cancellation signal is passed through and the call returns immediately when cancelled. It is a cleanup/quality improvement rather than a clear security vulnerability.