LND

This commit is a hardening and cleanup of a GitHub Actions workflow that automatically labels pull requests by severity. It does not change any LND node code, wallet logic, or network protocol. Instead, it splits the workflow into two jobs: a read-only 'classify' job that runs an AI model to decide the severity, and a separate 'apply' job that actually sets the label and posts the comment. The change reduces security risk by keeping write permissions out of the job that processes untrusted pull-request text, pins the external AI action to a fixed commit hash, disables unnecessary git credentials, and adds input sanitization for the model-generated comment. It is a defensive improvement, not a vulnerability fix.

This commit updates a GitHub Actions workflow for an optional code-review bot called 'gateway' from version 0.4.4 to 0.5.0. It adds support for replying to inline review comments (not just regular PR comments) and pins the new action and runtime to specific, immutable commit hashes. The change is administrative and does not touch LND's core Lightning node code, wallet logic, or network protocol handling.

This change fixes how the Lightning Network Daemon (LND) decides which pending payments (HTLCs) are considered 'active' in a payment channel. Previously, it matched HTLCs between the two sides of a channel by hashing the onion routing blob, which is not a unique identifier and can be duplicated by accident or on purpose. Now it matches by the HTLC's channel-level index and direction, which is a reliable identifier. The old behavior could have caused LND to misidentify active HTLCs when onion blobs were duplicated, potentially affecting channel state decisions.

This commit fixes a bug in LND's code that creates a copy of an open Lightning payment channel. The copy function was accidentally leaving out two pieces of channel data: the height at which the channel was confirmed as closed, and a database reference. That meant any code working on a copy could see slightly different channel state than the original, which could lead to wrong decisions about closing, settling, or recording channels. The fix simply copies those two missing fields. There is no direct evidence this is exploitable as an attack, but silent data divergence in financial software is a security-relevant bug.

This is a one-line code cleanup change. It swaps which internal Go package provides an empty 'OpenChannel' data structure when building a restored channel shell. The actual fields being filled in are identical, and there is no functional change to how data is read, written, or validated.

This commit is a straightforward internal code cleanup: it changes the contractcourt package to use a newly introduced chanstate.OpenChannel type instead of the older channeldb.OpenChannel alias. There is no functional change to how the Lightning node operates, and nothing in the commit suggests a security fix or vulnerability.

This commit is a straightforward internal code cleanup: it changes the lnwallet package to use the OpenChannel type from a dedicated chanstate package instead of getting it indirectly through the channeldb package. There is no change to user-facing behavior, network protocol handling, cryptographic operations, or database logic. It is purely a refactoring move to make the code's type ownership clearer.

This commit is a routine internal code cleanup in the LND lightning node software. It changes the peer package to use a new dedicated channel-state type (chanstate.OpenChannel) instead of an older alias from the channeldb package. There are no user-facing changes, no bug fixes, and no security-related behavior changes visible in the diff.

This commit only changes test helper code in LND. It swaps one internal type name (channeldb.OpenChannel) for another (chanstate.OpenChannel) inside test utilities. There is no change to production code, no user-facing behavior change, and no security fix.

This is a routine code cleanup in LND's RPC server. It changes several internal helper functions to accept a more specific 'chanstate.OpenChannel' type instead of the broader 'channeldb.OpenChannel' alias. The functions only read channel state, so this is purely a type-refactoring change with no visible behavior or security effect.

This commit is a straightforward internal code cleanup. It changes one part of the Lightning Network Daemon (LND) to use a newer, more focused package called `chanstate` for representing open channel data, instead of the older, broader `channeldb` package. The behavior of the program is intended to stay exactly the same; only the names and locations of the data types used in the code have changed. There is no indication this fixes or introduces a security problem.

This commit is a small internal code cleanup in LND's payment routing layer. It swaps some uses of the older `channeldb` package for a newer `chanstate` package when passing around channel type information. There is no change to user-facing behavior, no bug fix, and no security patch.

This commit is a straightforward internal code cleanup in LND's channel-opening machinery. It swaps one internal type name (channeldb.OpenChannel) for another (chanstate.OpenChannel) across function signatures in the funding manager and its tests. There is no change to user-facing behavior, network protocol, cryptography, or data handling, and nothing in the commit suggests a security fix or vulnerability.

This commit is a straightforward internal code cleanup in LND. It changes three places in server.go to use a new internal type name (chanstate.OpenChannel) instead of an old alias (channeldb.OpenChannel). The actual behavior of the code is unchanged; it simply updates wiring after a previous refactoring in the funding package. There is no security-relevant change visible in the diff.

This is a routine internal code cleanup in LND's watchtower subsystem. It changes which Go package the watchtower code uses to reference channel types and channel close summaries, moving from the older 'channeldb' package to a newer 'chanstate' package. There is no functional change visible in the diff, no bug fix, and no security patch.

This is a small internal code cleanup in LND's channel gossip subsystem. It changes which internal data type the gossiper uses when looking up a channel, switching from a database-specific type to a newer channel-state type. There is no user-facing behavior change and no security fix.

This is a small internal code cleanup change. It swaps one internal type reference (channeldb.OpenChannel) for another internal type reference (chanstate.OpenChannel) inside a peer-related data structure. The commit message says the goal is to keep the peer-facing type independent from a compatibility alias while keeping the same field layout for existing callers. There is no security-relevant behavior change visible in the diff.

This is a small internal code cleanup in LND's wallet RPC server. It changes one helper function to use a newer internal package type (chanstate.OpenChannel) instead of an older compatibility alias (channeldb.OpenChannel). There is no user-facing behavior change, no bug fix, and no security-relevant change visible in the diff.

This commit is a straightforward internal code cleanup: it changes which Go package provides the data structures used to describe open and closed channels inside the HTLC switch. There is no change to how data is validated, stored, or transmitted, and no security-relevant behavior is altered.

This commit is a straightforward internal code cleanup: it changes the channel status manager to use a newer internal package (chanstate) instead of an older compatibility alias (channeldb) for representing open channels. There is no user-facing behavior change, no bug fix, and no security-relevant change visible in the diff.

This is a routine internal code cleanup in LND's channel backup system. It replaces references to the older `channeldb` package with a newer `chanstate` package for channel type definitions. The actual backup data, encryption, and recovery logic are unchanged. There is no indication this fixes a security bug or introduces a vulnerability.

This is a small internal code cleanup in LND. It changes one function parameter type from channeldb.OpenChannel to chanstate.OpenChannel inside the channel backup notifier. There is no visible security fix, behavior change, or bug correction in the diff. It appears to be part of a larger refactoring to move channel-state logic into a new chanstate package.

This commit is a straightforward internal code cleanup. It changes which Go package name is used to refer to common channel data types (OpenChannel, ChannelCloseSummary, etc.) in two parts of the codebase. Previously these types were referenced through an older compatibility alias in the channeldb package; now they are referenced directly through the newer chanstate package. The actual data structures and behavior are unchanged, and there is no indication this fixes or introduces a security issue.

This commit is a small internal code cleanup. It changes one function to use a different internal data type for HTLC (payment channel transaction) information, moving from a database-specific type to a channel-state type. There is no security-relevant change visible in the diff.

This commit is a straightforward internal code cleanup. It changes three interface definitions to reference a new package (chanstate.OpenChannel) instead of an older one (channeldb.OpenChannel). There is no functional change to how the software behaves, no bug fix, and no security-related change.