Skip to content

Commit 7b0b712

Browse files
committed
minor updates
1 parent 6df385a commit 7b0b712

1 file changed

Lines changed: 9 additions & 9 deletions

File tree

docs/repro-channel-monitor-desync.md

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ Failed to start wallet
4343

4444
---
4545

46-
## Repro: Blocktank channel (staging regtest)
46+
## Repro Case #1: Blocktank channel (staging regtest)
4747

4848
Reproduces the bug using a Blocktank LSP channel opened via "transfer to spending".
4949

@@ -58,7 +58,7 @@ Reproduces the bug using a Blocktank LSP channel opened via "transfer to spendin
5858
### Steps
5959

6060
1. Install v1.1.6 (RN), create wallet, fund on-chain, open Lightning channel (transfer to spending), make 1 LN payment
61-
2. Install v2.0.6/v2.0.3 (native) **over** RN app — migration runs automatically
61+
2. Install v2.0.6 (iOS) or v2.0.3 (Android) **over** RN app — migration runs automatically
6262
3. Make 21+ Lightning payments on native:
6363
```bash
6464
# iOS
@@ -70,7 +70,7 @@ Reproduces the bug using a Blocktank LSP channel opened via "transfer to spendin
7070

7171
---
7272

73-
## Repro: 3rd-party channel (local docker)
73+
## Repro Case #2: 3rd-party channel (local docker)
7474

7575
Reproduces the bug using a manually opened channel to the local docker LND node.
7676

@@ -82,7 +82,7 @@ Reproduces the bug using a manually opened channel to the local docker LND node.
8282

8383
### Build Notes
8484

85-
RN v1.1.6 local builds use `.env.test.template` (regtest + localhost Electrum). For release builds, `react-native-dotenv` reads `.env.production`, so that file must be overwritten with the local config.
85+
RN v1.1.6 local builds use `.env.test.template` (regtest + localhost Electrum). For release builds, `react-native-dotenv` reads `.env.production`, so that file must be overwritten with the local regtest config.
8686

8787
**Critical**: The RN app's `.env.production` must point the backup server to **staging** (not localhost), because the native apps have `rnBackupServerHost` hardcoded to staging. If the RN app pushes to `127.0.0.1:3003` but the native app queries `bitkit.stag0.blocktank.to`, it will never find the channel monitors and the bug won't trigger.
8888

@@ -106,8 +106,8 @@ All other settings (Electrum, network, etc.) stay local.
106106
./bitcoin-cli send 0.01 <address>
107107
./bitcoin-cli mine 6
108108
```
109-
4. In the app, go to **Settings > Advanced > Channels > + > Fund Custom > Manual** and enter the local LND connection:
110-
- Node ID: `02cfdfd683aca2561621870fe50ab9ef2d0c887b3729ce6797ff68fde6f044feb9`
109+
4. In the app, go to **Settings > Advanced > Channels > + > Fund Custom > Manual** and enter the local LND connection (get the node ID from `./bitcoin-cli getinfo`):
110+
- Node ID: LND's pubkey
111111
- Host: `0.0.0.0`
112112
- Port: `9735`
113113
5. Set amount (e.g. 50,000 sats) and confirm the channel open
@@ -116,7 +116,7 @@ All other settings (Electrum, network, etc.) stay local.
116116

117117
#### 2. Open channel from LND to the app
118118

119-
The app's channel has all balance on the app side. LND needs outbound liquidity to pay invoices to the app:
119+
The app's channel has all balance on the app side. LND needs outbound liquidity to pay invoices to the app. Get the app's node ID from **Settings > Advanced > Lightning Node Info**, then:
120120

121121
```bash
122122
./bitcoin-cli openchannel <app_node_id> 500000
@@ -185,7 +185,7 @@ Matrix of upgrade/recovery scenarios to validate v2.1.2. Each scenario should be
185185
| B1 | v2.0.6 (wallet with 21+ payment gap) → v2.1.0 → confirm broken | Reproduces |
186186
| B2 | Restore broken v2.1.0 wallet into v2.1.2 (clean install + restore) | ✅ Recovered |
187187
| B3 | Update broken v2.1.0 wallet to v2.1.2 (in-place upgrade) | ✅ Recovered |
188-
| B4 | v2.0.6 (wallet with gap) → v2.1.2 (skip v2.1.0) |Recovered |
188+
| B4 | v2.0.6 (wallet with gap) → v2.1.2 (skip v2.1.0) |No issues |
189189
| B5 | v2.0.6 (wallet with gap) → v2.1.1 → v2.1.2 | ✅ Recovered |
190190
| B6 | v2.1.0 healthy wallet (no gap) → v2.1.2 (regression check) | ✅ No issues |
191191
| B7 | v2.1.0 broken wallet + 600 blocks mined → v2.1.2 (stale chain state) | ✅ Recovered |
@@ -196,7 +196,7 @@ Matrix of upgrade/recovery scenarios to validate v2.1.2. Each scenario should be
196196
|---|----------|--------|
197197
| T1 | v2.0.6 (wallet with 30+ payment gap) → v2.1.0 → confirm broken | Reproduces |
198198
| T2 | Update broken v2.1.0 wallet to v2.1.2 (in-place upgrade) | ✅ Recovered |
199-
| T3 | v2.0.6 (wallet with gap) → v2.1.2 (skip v2.1.0) |Recovered |
199+
| T3 | v2.0.6 (wallet with gap) → v2.1.2 (skip v2.1.0) |No issues |
200200
| T4 | v2.1.0 healthy wallet (no gap) → v2.1.2 (regression check) | ✅ No issues |
201201
| T5 | v2.1.0 broken wallet + 600 blocks mined → v2.1.2 (stale chain state) | ✅ Recovered |
202202

0 commit comments

Comments
 (0)