diff options
| author | Adam Soltys <[email protected]> | 2026-08-16 00:23:28 -0700 |
|---|---|---|
| committer | Adam Soltys <[email protected]> | 2026-08-16 00:23:28 -0700 |
| commit | 984d2c3efeca25c37522c6fb17a9863717b44bc2 (patch) | |
| tree | 8a710128db8e0e5aa63510e2fb51021ae0d222e7 /patches/README.md | |
| parent | 7c6805decc508b009673e1a5651b2a86ebdef717 (diff) | |
| download | coinos-server-984d2c3efeca25c37522c6fb17a9863717b44bc2.tar.gz coinos-server-984d2c3efeca25c37522c6fb17a9863717b44bc2.zip | |
deps: patch @asoltys/clightning-client to fix cl reconnect
Register patches/@asoltys+clightning-client+0.1.10.patch via patchedDependencies
so bun install re-applies it. The library's reconnect() reused an already-dead
socket and never recovered after a cl restart (looping connect ENOENT, leaking a
socket+timer); the patch rebuilds a fresh socket + readline each reconnection and
clears its retry guard before reconnecting. Companion to the app-side
dropClient()/listener-recovery fixes.
Co-Authored-By: Claude Opus 4.8 <[email protected]>
Diffstat (limited to 'patches/README.md')
| -rw-r--r-- | patches/README.md | 41 |
1 files changed, 41 insertions, 0 deletions
diff --git a/patches/README.md b/patches/README.md new file mode 100644 index 00000000..b3bf1322 --- /dev/null +++ b/patches/README.md @@ -0,0 +1,41 @@ +# patches + +## `@asoltys+clightning-client+0.1.10.patch` + +Fixes the reconnect logic in `@asoltys/clightning-client` (v0.1.10) so a dropped +cl RPC connection actually recovers. + +**Bug:** `reconnect()` called `this.client.connect()` on the *same* socket that +had already emitted `error`/`close`. A destroyed socket can't be reconnected, so +after a `cl` restart the client wedged, looping `connect ENOENT …/lightning-rpc` +forever — spamming logs and leaking a socket+timer each time — and never +recovered. (If `connect()` threw, `reconnectTimeout` was also left set, which +permanently blocked every future `reconnect()`.) + +**Fix:** factor the socket+readline+handlers setup into `_setup()` which builds a +**fresh** socket each (re)connection and renews `clientConnectionPromise` (so +`call()` blocks until the reconnect is live). `reconnect()` now clears its guard +*before* reconnecting and retries on failure. + +This is defense-in-depth: the app already neutralises the broken loop in +`lib/ln.ts` (`dropClient()` tears the dead client down and the proxy makes a fresh +`LightningClient`). This patch fixes the library itself for any path that relies +on its own reconnect. + +### How to wire it in (bun) + +`bun patch` couldn't run in this checkout (node_modules link EEXIST), so this +patch was generated by hand and verified to apply cleanly (`git apply --check`). +To register it so `bun install` re-applies it, in a clean checkout: + +```sh +bun patch @asoltys/clightning-client +# copy patches/clightning-client-index.fixed.js over the printed editable +# node_modules/@asoltys/clightning-client/index.js +bun patch --commit node_modules/@asoltys/clightning-client +``` + +That writes bun's own `patches/…` file and the `patchedDependencies` entry in +`package.json`. Alternatively, apply this patch directly in the image build: +`git apply patches/@asoltys+clightning-client+0.1.10.patch` against the installed +package. `clightning-client-index.fixed.js` is the full patched file for reference. |
