LL
← All projectsLightning Labs

LND

Go implementation of a complete Lightning Network node.

BitcoinLightning NetworkNormal
Repository coverage

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

118security candidates168second-pass queue1236AI analyses
55commits · 30 days
171commits · 60 days
830commits · 180 days
1220commits · 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
279Strong · 80–100
564Adequate · 60–79
367Thin · 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 Osuntokun19940198474
Elle Mouton26832268167
ziggie41015408164
Jared Tobin909171
Erick Cestari26726070
bitromortac46546067
yyforyongyu55355065
saubyk15215070
Oli727057
Suheb323075
Yong323073
elnosh323071
Analysis record

Published AI watches

Last scanned 22 minutes ago

Moderate 59 AI analysisMessage 73 · Adequate
LL Lightning LabsLND BitcoinLightning Network

Merge pull request #11023 from ellemouton/update-fee-log

This commit hardens how LND handles Lightning channel fee updates and mailbox message admission. It enforces the protocol rule that only the channel opener can send fee updates, prevents uncommitted fee updates from piling up in the update…

BOLT 2 role validation for UpdateFee senderReplacement of uncommitted fee updates to prevent log bloat and index gapsBounded mailbox wire-message queue (count and bytes)
7f56541dby Yong+750−138 files
No security note in commit
Moderate 57 AI analysisMessage 73 · Adequate
LL Lightning LabsLND BitcoinLightning Network

Merge pull request #11019 from Roasbeef/coop-close-race-public

This update fixes two related problems in LND's cooperative channel-closing code. First, it eliminates a data race where the same state machine was being advanced by two different internal goroutines at once, which could cause crashes or i…

Data race in legacy cooperative close state machine driven concurrently by link and peer goroutinesMissing or inconsistent validation of remote delivery script when no upfront shutdown script is recordedpanic() in DustLimitForSize on unrecognized script sizes replaced with safe fallback
f4ae5656by Yong+463−609 files
Vendor flagged security relevance
Moderate 63 AI analysisMessage 58 · Thin
LL Lightning LabsLND BitcoinLightning Network

Merge pull request #11024 from ziggie1984/invoices-update-validation

This commit tightens how LND processes invoice payments, especially for newer multi-path (MPP) and AMP invoices, keysend payments, and replayed payments. It adds checks that ensure the right payment preimage is used for each invoice type a…

Preimage validation added for AMP and regular invoice replaysAMP invoices forced into MPP update path, preventing legacy-path processingLegacy settlement now fails when invoice-level preimage is missing
61f3b5a7by ziggieXXX+482−114 files
No security note in commit
Moderate 59 AI analysisMessage 35 · Opaque
LL Lightning LabsLND BitcoinLightning Network

invoices: refine update handling

This commit tightens how LND handles invoice payment updates, especially for newer 'AMP' invoices and old-style 'keysend' payments. It adds checks so that the wrong kind of payment cannot be processed against the wrong kind of invoice, and…

Added preimage/hash mismatch checks for both regular and AMP invoice replaysAMP records now require an MPP payload, preventing AMP processing on the legacy pathLegacy path now rejects AMP invoices and invoices missing invoice-level preimages
6be6350eby ziggie+476−113 files
No security note in commit
Informational 15 AI analysisMessage 57 · Thin
LL Lightning LabsLND BitcoinLightning Network

docs: update 0.21.2 release notes

This commit only updates the release notes document for version 0.21.2. It adds a bullet point describing a previously merged code change about invoice handling and adds a contributor name to the credits list. No actual code was changed, s…

758bbb8eby ziggie+6−01 file
No security note in commit
Moderate 60 AI analysisMessage 88 · Strong
LL Lightning LabsLND BitcoinLightning Network

peer+lnwallet/chancloser: advance the legacy closer from one goroutine

This commit fixes a concurrency bug in how LND negotiates cooperative channel closures. Previously, two different goroutines could update the same channel-closing state machine at the same time, which could corrupt internal data or cause a…

Data race on ChanCloser state field detected under go test -raceConcurrent access to priorFeeOffers map and signing step by two goroutinesSingle-goroutine ownership invariant now documented on ChanCloser type
e5e134ddby Olaoluwa Osuntokun+292−384 files
Vendor flagged security relevance
Low 29 AI analysisMessage 73 · Adequate
LL Lightning LabsLND BitcoinLightning Network

lnwallet/chancloser: record the remote close output only when accepted

This change fixes a bookkeeping bug in how LND records the other party's preferred closing address during a channel close. Previously, a late or unexpected Shutdown message could overwrite the correct address with one from a message that w…

State variable written before validationRejected message could overwrite accepted close outputFix prevents stale/incorrect close metadata
fb89732dby Olaoluwa Osuntokun+7−21 file
No security note in commit
High 72 AI analysisMessage 68 · Adequate
LL Lightning LabsLND BitcoinLightning Network

htlcswitch: bound peer-controlled channel ingress

This commit hardens LND's channel message handling against a peer that tries to overwhelm the node. It adds limits on how many ordered messages and how much data a peer can queue up for a channel, and it disconnects the peer if either limi…

Adds resource bounds on peer-controlled channel ingress (count + bytes)Disconnects peer on mailbox admission failure instead of silently dropping ordered channel messagesPrevents unauthorized UpdateFee from channel initiator before fee-exposure calculations
ef24f2c5by Elle Mouton+507−94 files
Vendor flagged security relevance
Moderate 64 AI analysisMessage 83 · Strong
LL Lightning LabsLND BitcoinLightning Network

lnwallet/chancloser: validate delivery scripts in the RBF closer

This commit fixes a bug in LND's cooperative channel-closing code where a peer could supply an invalid, empty, or even money-burning Bitcoin address during a close, and LND would accept it without checking. The fix makes the newer 'RBF' cl…

Missing input validation on remote-supplied shutdown/delivery scriptEmpty/zero-length script accepted as valid due to nil-treatment in validatorMid-negotiation script swap via ClosingComplete not validated
a8e2a0f7by Olaoluwa Osuntokun+131−142 files
Vendor flagged security relevance
Moderate 51 AI analysisMessage 73 · Adequate
LL Lightning LabsLND BitcoinLightning Network

lnwallet: make DustLimitForSize total over the sizes it can be handed

This change removes a deliberate program crash (panic) in LND's fee/dust-limit helper and replaces it with a safe fallback. Previously, if the helper received a script length it didn't explicitly recognize, it would panic and potentially c…

Removal of a panic/default branch in favor of a conservative fallbackDefensive handling of arbitrary and future witness-version script sizesPotential denial-of-service vector eliminated: unhandled script lengths no longer crash the helper
f80f92dcby Olaoluwa Osuntokun+23−62 files
No security note in commit
Informational 15 AI analysisMessage 57 · Thin
LL Lightning LabsLND BitcoinLightning Network

docs: move gossip bounds note to v0.21.2

This commit only moves an existing release-note entry from one version's notes (v0.22.0) to another (v0.21.2). It does not change any code, build files, configuration, or tests. The change is purely editorial documentation bookkeeping.

83e7eb8cby ziggie+11−112 files
No security note in commit
High 72 AI analysisMessage 68 · Adequate
LL Lightning LabsLND BitcoinLightning Network

discovery: bound channel range reply buffering

This change fixes a way a malicious or buggy peer could make an LND node use an unpredictable amount of memory while syncing the Lightning channel graph. Before the fix, the node only limited the size of one compressed reply message, but a…

Resource exhaustion / unbounded memory growth in gossip syncRemote peer can influence memory allocation via ReplyChannelRange streamMissing aggregate limit on decoded working set
ceff94faby Olaoluwa Osuntokun+335−153 files
Vendor flagged security relevance
Low 33 AI analysisMessage 47 · Thin
LL Lightning LabsLND BitcoinLightning Network

docs: update release notes

This commit updates the release notes for LND 0.21.2 to describe a fix for a bug in the HTLC interceptor RPC. The bug caused LND to fail to forward payments in blinded routes when the next hop was identified by node ID instead of channel I…

Blinded route forwarding failure could disrupt payment routing or be used to probe path structure.Sentinel value change in RPC API could cause client misclassification of forwards if not handled.Release notes describe the change as a fix for a forwarding issue, not as a security vulnerability.
f42b4298by bitromortac+20−01 file
No security note in commit
Informational 14 AI analysisMessage 83 · Strong
LL Lightning LabsLND BitcoinLightning Network

itest: cover blinded route next_node_id forwarding

This commit only adds new integration tests for an existing feature in LND: forwarding blinded payments where the next hop is identified by node ID rather than channel ID. It does not change production code, fix a bug, or introduce any new…

da6a40c0by bitromortac+432−02 files
No security note in commit
Low 25 AI analysisMessage 73 · Adequate
LL Lightning LabsLND BitcoinLightning Network

witness beacon: report node-ID next hop to the on-chain HTLC interceptor

This change fixes a reporting gap in LND's 'witness beacon,' a component that watches on-chain transactions and tells the HTLC interceptor where a payment should go next. Previously, when the next hop was identified by a node ID rather tha…

Missing security-critical metadata in on-chain interception pathBehavioral parity between on-chain and off-chain HTLC interceptionPotential misclassification of node-ID forwards as final receives
9c4b8bfeby bitromortac+59−52 files
No security note in commit
Moderate 59 AI analysisMessage 81 · Strong
LL Lightning LabsLND BitcoinLightning Network

htlcswitch: forward node-ID blinded hops via non-strict forwarding

This commit fixes a bug in LND's payment forwarding for 'blinded routes'—a privacy feature in the Lightning Network. Previously, if the next hop was identified by node ID rather than a specific channel ID, the payment could not be forwarde…

Fixes a functional forwarding failure for blinded-route payments using node-ID next hops (issue #10937).Prevents private channel SCID leakage in failure messages for node-ID blinded hops by returning FailUnknownNextPeer instead of a channel_update.Adds circular-route filtering for node-ID next hops before non-strict forwarding selection.
dbc57040by bitromortac+334−416 files
No security note in commit
Informational 19 AI analysisMessage 73 · Adequate
LL Lightning LabsLND BitcoinLightning Network

htlcswitch+lnrpc: report node-ID next hop to the off-chain HTLC interceptor

This change improves how the Lightning Network Daemon (LND) reports forwarding details to external plugins that inspect HTLCs (payment packets). For a new type of blinded route where the next hop is identified by node ID rather than a spec…

Information-correctness fix for HTLC interceptor APIPrevents misclassification of node-ID forwards as exit hopsAdds explicit sentinel to disambiguate zero SCID semantics
32373b76by bitromortac+29−13 files
No security note in commit
Low 25 AI analysisMessage 73 · Adequate
LL Lightning LabsLND BitcoinLightning Network

htlcswitch: classify a node-ID forward as a forward event

This commit fixes a bookkeeping bug in LND's HTLC event stream. When a payment is forwarded to a 'blinded' next hop identified only by a node public key (not a channel ID), and it fails before the switch picks an outgoing channel, the even…

Event misclassification in HTLC event stream (forward reported as receive)Blinded/node-ID forwarding path introduced prior to this fixFailure and resolution packets dropped next-hop metadata before fix
a4844ef5by bitromortac+149−04 files
No security note in commit
Low 29 AI analysisMessage 68 · Adequate
LL Lightning LabsLND BitcoinLightning Network

htlcswitch/hop: decode next_node_id blinded hops

This commit adds support in LND for a different way of identifying the next hop in a private ('blinded') Lightning payment route. Previously LND expected the next hop to be identified by a channel ID; now it also accepts the next node's pu…

New validation enforces BOLT 4 mutual exclusivity of short_channel_id and next_node_id in blinded route dataPreviously ambiguous or unhandled next_node_id forwarding case now explicitly decodedNo memory-safety, cryptographic, or authorization changes observed
4fd4289aby bitromortac+471−86 files
No security note in commit
Moderate 60 AI analysisMessage 78 · Adequate
LL Lightning LabsLND BitcoinLightning Network

lnwire: cap decoded short channel IDs

This commit tightens a safety limit in LND's Lightning wire protocol parser. Previously, a compressed network message could decompress into an unbounded number of short channel IDs, potentially consuming far more memory than the wire messa…

Resource-limit bypass: compressed-input bound did not bound decoded outputMemory-exhaustion risk from zlib decompression of peer-supplied dataTruncated/corrupt zlib streams previously accepted as valid partial results
d1622919by Olaoluwa Osuntokun+271−502 files
No security note in commit
Repository ledger

Explore captured commits

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

AI review queuedlnwallet: add field `CommitTxBlockHeight` to ResolutionReqby ffranr · 3d2b0d70 · Nov 25, 2025 · 3 filesMessage 73 · AdequateLow 25Details
Commit message · ffranr

lnwallet: add field `CommitTxBlockHeight` to ResolutionReq

Introduce `CommitTxBlockHeight` field to the `ResolutionReq` structure
and related methods. This field records the block height where a
commitment transaction has confirmed.

73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Low 25/100

This commit adds a new piece of information—the block height at which a commitment transaction was confirmed—to the data structures used when resolving Lightning Network channel closes. It does not change any security-critical logic by itself; it is a data plumbing change that makes the confirmation height available to auxiliary contract resolvers. There is no direct vulnerability visible in the diff, but it could be a prerequisite for a future fix or feature that depends on knowing this height accurately.

Lower-prioritydocs: add release-notes for LND 20.1by ziggie · db3add1c · Nov 22, 2025 · 1 fileMessage 57 · ThinInformational 15Details
Commit message · ziggie

docs: add release-notes for LND 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 line to the release notes document for LND version 0.20.1. It describes a bug fix where resolving certain channel outputs could take too long, but it does not change any program code, configuration, or cryptographic logic. There is no direct security relevance visible in the commit itself.

Lower-prioritycontractcourt: use confheight instead of rescanning the chainby ziggie · 79b92a9f · Nov 22, 2025 · 2 filesMessage 50 · ThinLow 33Details
Commit message · ziggie

contractcourt: use confheight instead of rescanning the chain

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

This change removes a live blockchain rescan that waited for the commitment transaction to confirm, and instead uses a previously known confirmation height stored in the resolver. The main risk is that if the stored confirmation height is wrong or stale, the resolver may compute the wrong unlock time for locked funds, potentially causing premature or delayed sweeps. There is no direct evidence in the commit message or diff that this fixes an active security bug; it appears to be a simplification/optimization.

AI review queuedcontracourt: fix commentby ziggie · 55388032 · Nov 22, 2025 · 1 fileMessage 28 · OpaqueInformational 15Details
Commit message · ziggie

contracourt: fix comment

28/100 · OpaqueMessage clarity
✓ Subject identifies a change! No meaningful explanatory body
Why it was queued
second-pass: opaque commit message
AI analysis · Informational 15/100

This commit changes only a single line of code comment text in the contractcourt package. It rewords a comment to more accurately describe that a report field is being updated with a calculated maturity height, rather than a confirmation height. No executable code, logic, or data flow was modified.

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

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

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.

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

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.