fix: prevent channel monitor overwrite in migrations

AGENTS.md

@@ -21,20 +21,56 @@ cargo build --profile=release-smaller
 ```
 
 ### Testing
+
+#### Unit tests (no external infrastructure needed)
+```bash
+cargo test --lib
+```
+
+#### Standard integration tests
+Uses the `electrsd` crate to auto-download and spawn `bitcoind` + `electrs` binaries.
+To use pre-downloaded binaries, set `BITCOIND_EXE` and `ELECTRS_EXE` env vars.
+
+On macOS, the default file descriptor limit (256) is too low for spawning `bitcoind` +
+`electrs`. Raise it before running integration tests:
+
+Test files: `tests/integration_tests_rust.rs`, `tests/reorg_test.rs`,
+`tests/multi_address_types_tests.rs`
+
 ```bash
-# Run all tests
-cargo test
+ulimit -n 10240 && cargo test
+```
 
-# Run a specific test
+#### Run a specific test
+```bash
 cargo test test_name
+```
 
-# Run tests with specific features
+#### Run tests with UniFFI bindings feature
+```bash
 cargo test --features "uniffi"
+```
+
+#### CLN integration tests
+Requires Docker and `socat` for RPC socket forwarding.
+```bash
+docker compose -f docker-compose-cln.yml up -d
+RUSTFLAGS="--cfg cln_test" cargo test --test integration_tests_cln
+```
 
-# Integration tests with specific backends
-cargo test --cfg cln_test  # Core Lightning tests
-cargo test --cfg lnd_test  # LND tests
-cargo test --cfg vss_test  # VSS (Versioned Storage Service) tests
+#### LND integration tests
+Requires Docker and CMake (>=3.5, <4.0) for `lnd_grpc_rust`.
+Set `LND_DATA_DIR`, `LND_CERT_PATH`, and `LND_MACAROON_PATH` env vars.
+```bash
+docker compose -f docker-compose-lnd.yml up -d
+RUSTFLAGS="--cfg lnd_test" cargo test --test integration_tests_lnd
+```
+
+#### VSS integration tests
+Requires PostgreSQL and a running VSS server (from `lightningdevkit/vss-server`).
+Set `TEST_VSS_BASE_URL` env var.
+```bash
+RUSTFLAGS="--cfg vss_test" cargo test --test integration_tests_vss
 ```
 
 ### Code Quality

CHANGELOG.md

@@ -1,4 +1,4 @@
-# 0.7.0-rc.32 (Synonym Fork)
+# 0.7.0-rc.33 (Synonym Fork)
 
 ## Bug Fixes
 
@@ -29,6 +29,11 @@
 
 ## Synonym Fork Additions
 
+- Fixed channel monitor migration from filesystem store to KV store overwriting newer state.
+  During FS→KV migration, the code now checks if the KV store already has a channel monitor
+  with a newer `update_id` before writing, preventing stale migration data from overwriting
+  current state on repeated migrations or restarts. Errors reading or deserializing existing
+  monitors now fail-closed (abort migration) to avoid silent data loss.
 - Added `OnchainPayment::calculate_send_all_fee()` to preview the fee for a drain / send-all
   transaction before broadcasting (fee-calculation counterpart of `send_all_to_address`)
 - Added runtime APIs for dynamic address type management:

Cargo.toml

@@ -4,7 +4,7 @@ exclude = ["bindings/uniffi-bindgen"]
 
 [package]
 name = "ldk-node"
-version = "0.7.0-rc.32"
+version = "0.7.0-rc.33"
 authors = ["Elias Rohrer <dev@tnull.de>"]
 homepage = "https://lightningdevkit.org/"
 license = "MIT OR Apache-2.0"

Package.swift

@@ -3,8 +3,8 @@
 
 import PackageDescription
 
-let tag = "v0.7.0-rc.32"
-let checksum = "89d987f390e0f8fae370c6d549aacd8b8e1c3c95fb0d6e9ccfcf216cbd263b38"
+let tag = "v0.7.0-rc.33"
+let checksum = "235e276d2a39ed41abb4034b027f72bf876b65eb40a2a584cc1d06208ad7a58a"
 let url = "https://github.com/synonymdev/ldk-node/releases/download/\(tag)/LDKNodeFFI.xcframework.zip"
 
 let package = Package(

bindings/kotlin/ldk-node-android/gradle.properties

@@ -3,4 +3,4 @@ android.useAndroidX=true
 android.enableJetifier=true
 kotlin.code.style=official
 group=com.synonym
-version=0.7.0-rc.32
+version=0.7.0-rc.33

bindings/kotlin/ldk-node-jvm/gradle.properties

@@ -1,4 +1,4 @@
 org.gradle.jvmargs=-Xmx1536m
 kotlin.code.style=official
 group=com.synonym
-version=0.7.0-rc.32
+version=0.7.0-rc.33

bindings/python/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "ldk_node"
-version = "0.7.0-rc.32"
+version = "0.7.0-rc.33"
 authors = [
   { name="Elias Rohrer", email="dev@tnull.de" },
 ]