thanks for the check will update
Lakshya Singh (f470aa2d) at 06 Mar 17:03
Lakshya Singh (f5ff6edb) at 06 Mar 17:03
Merge branch 'king-11/agent-code' into 'main'
... and 3 more commits
AGENTS.md file for the coding agents to refer our internal READMEsAGENTS.md so added a separate CLAUDE.md which just imports the standard AGENTS.md
ready for review @DanielFeichtinger @SulaimanAminuBarkindo
@DanielFeichtinger ping here
Lakshya Singh (631df27a) at 05 Mar 14:57
feat: add checks for cln rest progress in CI
A Debian Trixie (stable) now packages rustc 1.85 we can update our MSRV to match. This also updated the embedded Dockerfile and upload script as that was last run at 1.84. The new CI image is 1.95. I also simplified the update-all script for bumping deps as the resolver in 1.85 handles MSRV aware updates out of the box. Finally, this bumps all dependencies in lock files.
There is an CVE for the version of the time crate that
we use and the fix requires 1.88, but my review turned up only local usages, never parsing incoming data so I marked that CVE as ignored.
Changelog-Changed: Increase MSRV to 1.85
Lakshya Singh (08c07de8) at 13 Feb 17:29
feat: add checks for cln rest progress in CI
Lakshya Singh (f99b5cce) at 13 Feb 17:22
feat: add checks for cln rest progress in CI
Lakshya Singh (bffcf764) at 13 Feb 17:09
feat: add checks for cln rest progress in CI
Lakshya Singh (7523c4dc) at 13 Feb 06:13
feat: add checks for cln rest progress in CI
Lakshya Singh (37713f3c) at 13 Feb 05:23
feat: add checks for cln rest progress in CI
Lakshya Singh (8e252b49) at 12 Feb 17:10
feat: add checks for cln rest progress in CI
Lakshya Singh (0f6c1ee2) at 12 Feb 16:49
docs: add details about CLN rest plugin
added example of using rune and clnrest as well
tested locally that regtest setup is able to come up and cln rest endpoints are accessible
docker compose --profile vls -f docker-compose.yml -f docker-compose.regtest.yml up
docker container exec lightningd-regtest lightning-cli --regtest createrune -k "restrictions"="readonly"
curl -k -X POST 'https://localhost:3010/v1/getinfo' -H 'Rune: <rune>'
@DanielFeichtinger shall we merge this?
Lakshya Singh (2e7a2bb6) at 12 Feb 16:43
cln: v25.02
we should remove the function
This MR fixes our timeout issues with lnrod integration tests. The issue seemed to be that occasionally the route from Alice to Charlie through Bob wasn't being broadcast/discovered. I didn't get to the root of why that that only happening intermittently, but adding a peer connection from Alice's node to Charlie's node fixes the problem. This fix may be a bit of a band-aid over the real issue so I'm open for feedback on this as a solution or if we should open an issue to full resolve.
I also took the opportunity to tidy up some of the process management and logging output in the tests, which makes the commit a bit noisy. The salient line for the fix is really just 305.