reckless: rust rewrite by daywalker90 · Pull Request #9239 · ElementsProject/lightning

daywalker90 · 2026-06-19T16:48:24Z

Not ready for code review yet, we have not decided on some important aspects. Right now it is a drop-in replacement of the python reckless version.

Should
Fixes: #8439
Fixes: #8761
Fixes: #8775
Fixes: #8776
Fixes: #8794
and should supercede #8630

Key differences:

it is now a plugin only, the only use case to have a standalone binary, that i could come up with, is a broken plugin install where CLN won't start, but if you can call a binary you can uncomment a line in a config, right?
as a consequence, alot of options are no longer necessary and reckless-dir is a proper plugin option now
except for the help we always output json, so no more --json option
the rewrite does not adhere to the reckless schema in all places, e.g. listavailable, an array of strings is just not nice here
a tip function was added to tip plugin authors that have an offer in their manifest.json

vincenzopalazzo

Nice to see that the manifest idea is taking over. Hope you are also using git for versioning, otherwise this would be another good thing to take from https://github.com/coffee-tools/coffee

vincenzopalazzo · 2026-06-19T18:46:40Z

Would be nice to reference the original reference implementation https://coffee-docs.netlify.app/support-coffee

Would be nice to reference the original reference implementation https://coffee-docs.netlify.app/support-coffee

I was planning on giving full credit for the manifest/tip function to coffee in the README. But i want to lock down the functionality first before writing documentation.

to save credit was also good to just include coffee, I do not think there are more features than coffee, we were also supporting ws (palanning to) to support interactive UX inside the web applications

But these days cloning is easy!

daywalker90 · 2026-06-19T18:50:18Z

Nice to see that the manifest idea is taking over. Hope you are also using git for versioning, otherwise this would be another good thing to take from https://github.com/coffee-tools/coffee

If you mean installing a plugin with a git ref then yes, that is supported.

vincenzopalazzo · 2026-06-19T18:53:28Z

If you mean installing a plugin with a git ref then yes, that is supported.

Yeah, otherwise it's insanity! Nice, it would be nice to have two lines that reference coffee because the development died because Blockstream was moving to reckless. Referencing a good idea is what is left in this AI era, where rebuilding is easy

daywalker90 · 2026-07-14T13:24:22Z

~~Some tests will fail for now because of #9283. If we decide not to fix it, i can do a workaround.~~

ScuttoZ · 2026-08-05T14:06:19Z

+    let line = format!("creating venv at: {}", venv_dir.display());
+    logger.log(&line, LogLevel::INFO).await?;
+
+    let mut command = Command::new("python");


Is this command assuming the presence of python-is-python3? Couldn't we invoke python3 directly?

Good catch. Outside a venv it's best to call python3 on unix platforms.

ScuttoZ · 2026-08-05T14:24:22Z

+            .map(std::borrow::ToOwned::to_owned),
+    };
+
+    match enable_plugin(plugin.clone(), rl_plugin, install_args.options, &mut logger).await {


reckless install plugin-a plugin-b is not possible anymore, as clearly stated in the README and --help, but if a user attempts it no error is raised. Since this was possible in the old version, I feel like we should somehow enforce this new behaviour, or at least raise an error if attempted.

A second plugin name is currently not distinguishable from a config flag, since we don't require plugin config options to prefix with something like --. That is why you don't see an error on a command like you wrote. It will try to install plugin-a with the boolean flag plugin-b which at the end will fail since no such option exists in the plugin-a manifest:

l1-cli reckless install summary historian ... "INFO: plugin installed: /tmp/l1/reckless/summary/summary.py", "WARNING: option historian not found in manifest", "WARNING: summary failed to start, it may require options, read the logs!"

Shahana said, that breaking with old reckless input/output format was ok since nobody really used it.

alright, then I guess the docs mentions are enough.

Changelog-Changed: reckless: complete rewrite with new subcommands `listinstalled` and `tip`. Replaced `search` with `listavailable`.

If the timestamp of the python file does not differ after an update it will load the bytecode of the old version. We remove the pycache to make sure this does not happen. Mostly a test environment issue as in practive the timestamps should always differ.

…rning an error

…e seconds

…sources

…ecision

After ElementsProject@fce9d6e we can now use it. Also fix writing flag options to config file.

daywalker90 force-pushed the reckless-rs branch 2 times, most recently from 8644166 to 6889d36 Compare June 19, 2026 17:21

vincenzopalazzo reviewed Jun 19, 2026

View reviewed changes

madelinevibes added this to the v26.09 milestone Jun 22, 2026

madelinevibes assigned daywalker90 Jun 22, 2026

daywalker90 force-pushed the reckless-rs branch 3 times, most recently from 6658af8 to f6bbfb5 Compare July 1, 2026 17:09

daywalker90 force-pushed the reckless-rs branch 5 times, most recently from 5833d8b to 853fe7d Compare July 8, 2026 15:34

daywalker90 force-pushed the reckless-rs branch 4 times, most recently from f4513c8 to bf9e3e2 Compare July 14, 2026 12:39

daywalker90 marked this pull request as ready for review July 14, 2026 13:11

daywalker90 requested a review from cdecker as a code owner July 14, 2026 13:11

daywalker90 force-pushed the reckless-rs branch from bf9e3e2 to 2dde507 Compare July 15, 2026 14:59

madelinevibes added the QA Blockstream QA team have reproduced, or a test has been created! Look for the linked PR/Issue label Jul 17, 2026

daywalker90 force-pushed the reckless-rs branch 2 times, most recently from 59f0030 to a0ca360 Compare July 21, 2026 11:04

madelinevibes unassigned daywalker90 Jul 23, 2026

madelinevibes added the Status::Ready for Review The work has been completed and is now awaiting evaluation or approval. label Jul 23, 2026

madelinevibes requested review from ShahanaFarooqui and enaples July 23, 2026 07:59

daywalker90 mentioned this pull request Jul 29, 2026

reckless returned invalid character in json output daywalker90/summars#55

Closed

daywalker90 force-pushed the reckless-rs branch 2 times, most recently from 4cbd454 to fc99024 Compare August 4, 2026 13:02

ScuttoZ reviewed Aug 5, 2026

View reviewed changes

daywalker90 force-pushed the reckless-rs branch 3 times, most recently from a81ce21 to 9f45807 Compare August 6, 2026 15:02

daywalker90 mentioned this pull request Aug 8, 2026

reckless install gives ERROR: error: failed to parse manifest at... daywalker90/cln-nip47#32

Closed

daywalker90 and others added 20 commits August 14, 2026 16:17

reckless: rust rewrite

110b180

Changelog-Changed: reckless: complete rewrite with new subcommands `listinstalled` and `tip`. Replaced `search` with `listavailable`.

typo: changed "lintintalled" instances to "listinstalled"

6fec586

fixup! typo: changed "lintintalled" instances to "listinstalled"

11fc2f0

fix repo owner dir removal when removing source

64e3a2f

Makefile: add cln-rpc and cln-plugin as dependency of reckless

ac1866b

tests: add reckless test for update

c791a23

tests: add reckless tests for listinstalled

0375509

reckless: listinstalled: check if reckless dir exists instead of retu…

1a97d28

…rning an error

reckless: return error if enable detects plugin is already running

306648c

reckless: fix update_config_file returning commented option names

0d43a2c

reckless: fix unix_timestamp in notification, it was missing the whol…

369fd43

…e seconds

reckless: replace two unwraps with ok_or_else

dba7015

reckless: make plugin search case insensitive

56bd1b1

pytest: add test for reckless_log notification

3be850f

reckless: only attempt to remove source cache after it was found in .…

b33c306

…sources

reckless: use platform specific python outside venv

4bfc78f

pytests: fix iso parser in python 3.10 not working with nanosecond pr…

a0e2bfc

…ecision

reckless: improve cli help texts

a1d01ab

reckless: use .call_typed when starting a plugin with options

4a983dd

After ElementsProject@fce9d6e we can now use it. Also fix writing flag options to config file.

daywalker90 force-pushed the reckless-rs branch from 9f45807 to 4a983dd Compare August 14, 2026 14:50

Navigation Menu

Uh oh!

reckless: rust rewrite - #9239

reckless: rust rewrite#9239
daywalker90 wants to merge 20 commits into
ElementsProject:masterfrom
daywalker90:reckless-rs

daywalker90 commented Jun 19, 2026

Uh oh!

vincenzopalazzo left a comment

Uh oh!

vincenzopalazzo Jun 19, 2026

Uh oh!

daywalker90 Jun 19, 2026

Uh oh!

vincenzopalazzo Jun 19, 2026

Uh oh!

daywalker90 commented Jun 19, 2026

Uh oh!

vincenzopalazzo commented Jun 19, 2026

Uh oh!

daywalker90 commented Jul 14, 2026 •

edited

Loading

Uh oh!

ScuttoZ Aug 5, 2026

Uh oh!

daywalker90 Aug 5, 2026

Uh oh!

ScuttoZ Aug 5, 2026

Uh oh!

daywalker90 Aug 5, 2026

Uh oh!

ScuttoZ Aug 5, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Uh oh!

Conversation

daywalker90 commented Jun 19, 2026

Uh oh!

vincenzopalazzo left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

daywalker90 commented Jun 19, 2026

Uh oh!

vincenzopalazzo commented Jun 19, 2026

Uh oh!

daywalker90 commented Jul 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

daywalker90 commented Jul 14, 2026 •

edited

Loading