Skip to content

Commit ffc0ec5

Browse files
ovitrifclaude
andcommitted
fix: simplify channel manager migration comment
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 1531af7 commit ffc0ec5

1 file changed

Lines changed: 1 addition & 7 deletions

File tree

src/builder.rs

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1347,13 +1347,7 @@ where
13471347
K: EntropySource + SignerProvider<EcdsaSigner = InMemorySigner>,
13481348
{
13491349
if let Some(manager_bytes) = &migration.channel_manager {
1350-
// Existence-only check (no deserialization). Unlike monitors, we cannot
1351-
// deserialize the channel manager here because ChannelManagerReadArgs
1352-
// requires already-deserialized channel_monitor_references, which are
1353-
// loaded later in build(). This means corrupt existing data will block
1354-
// migration — that is intentional: overwriting a potentially valid
1355-
// channel manager with stale FS data risks fund loss, while a node that
1356-
// refuses to start can be recovered by clearing the KV store.
1350+
// Skip if KV store already has a channel manager (existence-only check).
13571351
let should_write = match runtime.block_on(KVStore::read(
13581352
&**kv_store,
13591353
CHANNEL_MANAGER_PERSISTENCE_PRIMARY_NAMESPACE,

0 commit comments

Comments
 (0)