aboutsummaryrefslogtreecommitdiffstats
path: root/patches/README.md
diff options
context:
space:
mode:
authorAdam Soltys <[email protected]>2026-08-16 00:23:28 -0700
committerAdam Soltys <[email protected]>2026-08-16 00:23:28 -0700
commit984d2c3efeca25c37522c6fb17a9863717b44bc2 (patch)
tree8a710128db8e0e5aa63510e2fb51021ae0d222e7 /patches/README.md
parent7c6805decc508b009673e1a5651b2a86ebdef717 (diff)
downloadcoinos-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.md41
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.