Skip to content

Conversation

@turuslan
Copy link
Contributor

Signed-off-by: turuslan <[email protected]>
Signed-off-by: turuslan <[email protected]>
Signed-off-by: turuslan <[email protected]>
@kamilsa kamilsa requested a review from Copilot September 22, 2025 08:50
Copy link
Contributor

@kamilsa kamilsa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe also rename lean_node to qlean?

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Adds a new CLI subcommand to generate a node key and fixes a minor log message typo.

  • Introduces lean_node key generate-node-key to output a Secp256k1 private key (hex) and PeerId (Base58)
  • Fixes grammar in a log line and updates help output to list the new command

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 7 comments.

File Description
src/modules/production/production.cpp Fixes a typo in a log message string
src/executable/lean_node.cpp Wires a new CLI subcommand and minimal argument parsing helper
src/executable/cmd_key_generate_node_key.hpp Implements the key generation command (secp256k1 + PeerId)
src/app/configurator.cpp Updates help output to mention the new subcommand

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Signed-off-by: turuslan <[email protected]>
Signed-off-by: turuslan <[email protected]>
Signed-off-by: turuslan <[email protected]>
@turuslan turuslan enabled auto-merge (squash) September 23, 2025 03:09
@turuslan turuslan merged commit 0e3e6bc into master Sep 23, 2025
2 checks passed
@turuslan turuslan deleted the feature/key-generate-node-key branch September 23, 2025 04:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Node key generator

4 participants