lightning-terminal icon indicating copy to clipboard operation
lightning-terminal copied to clipboard

improve litcli documentation clarity

Open AndySchroder opened this issue 2 years ago • 10 comments
trafficstars

We have

$ litcli -h
NAME:
   litcli - control plane for your Lightning Terminal (lit) daemon

USAGE:
   litcli [global options] command [command options] [arguments...]

VERSION:
   0.10.1-alpha commit=v0.10.1-alpha

COMMANDS:
   actions  List actions performed on the Litd server
   help, h  Shows a list of commands or help for one command

   Accounts:
     accounts, a  manage accounts

   Autopilot:
     autopilot  manage autopilot sessions

   LiT:
     stop     shutdown the LiT daemon
     getinfo  Returns basic information related to the active daemon.

   Privacy:
     privacy, p  Access the real-pseudo string pairs of the privacy mapper

   Sessions:
     sessions, s  manage Terminal Web sessions

GLOBAL OPTIONS:
   --rpcserver value          lit daemon address host:port (default: "localhost:8443")
   --network value, -n value  the network litd is running on e.g. mainnet, testnet, etc. (default: "mainnet")
   --basedir value            path to lit's base directory (default: "/home/lnd/.lit")
   --tlscertpath value        path to lit's TLS certificate (default: "/home/lnd/.lit/tls.cert")
   --macaroonpath value       path to lit's macaroon file (default: "/home/lnd/.lit/mainnet/lit.macaroon")
   --help, -h                 show help
   --version, -v              print the version

however

  • I can't find any explanation of what the "privacy mapper" is and,
  • sessions seems to manage more than just terminal web sessions, it seems to manage all lightning node connect sessions.

AndySchroder avatar Jul 08 '23 02:07 AndySchroder

I would like to contribute to this issue. Can you tell how do I get started?

daredevil3435 avatar Aug 05 '23 10:08 daredevil3435

I think some cleaning up of the descriptions of the help text would be a good starting point:

  • Make sure all short descriptions use the same style (capitalization and punctuation?)
  • Rename "Terminal Web sessions" to "Lightning Node Connect"
  • Add extended descriptions for each command

Though I'm not sure the first point is easy to address, as the privacy mapper isn't really documented anywhere outside of the code.

guggero avatar Aug 07 '23 07:08 guggero

Hi Andy! You're right, the Builder's Guide didn't make any mention of the word "Privacy Mapper", and we've corrected that. In short, it's a way for litd to obfuscate your channel IDs, transaction IDs and pubkeys. There's a paragraph explaining it here.

Liongrass avatar Sep 12 '23 19:09 Liongrass

Thanks Leo! Will close this for now. Feel free to re-open for any further questions if needed.

levmi avatar Sep 12 '23 19:09 levmi

Has everything in https://github.com/lightninglabs/lightning-terminal/issues/591#issuecomment-1667304656 been addressed? Seems like https://github.com/lightninglabs/lightning-terminal/issues/591#issuecomment-1716269453 only addresses part of the issue.

AndySchroder avatar Sep 12 '23 20:09 AndySchroder

Ah yes, thank you for pointing that out! This was likely a pre-emptive close on my part. Apologies, will re-open until we fix everything :)

levmi avatar Sep 12 '23 20:09 levmi

I think some cleaning up of the descriptions of the help text would be a good starting point:

  • Make sure all short descriptions use the same style (capitalization and punctuation?)
  • Rename "Terminal Web sessions" to "Lightning Node Connect"
  • Add extended descriptions for each command

Though I'm not sure the first point is easy to address, as the privacy mapper isn't really documented anywhere outside of the code.

I am interested in getting involved with this project. I have experience with system-level programming including go and c/c++, in-depth understanding of the bitcoin system and wouldn't mind enhancing some documentation in order to get familiar with some of the source code.

Is there a specific capitalization and punctuation style for the short descriptions, or is it only necessary that they are consistent?

nonfungible-human avatar Oct 25 '23 12:10 nonfungible-human

Hi @nonfungible-human ! That would be great, we'd love to see that pull request! We don't have an official style guide, but we do try to have a consistent style on the Builder's Guide and you can always check there for references. I'll review your writing once the pull request is up, though you might have to ping me to make sure I notice! You can also always find me on the LND slack with questions!

Liongrass avatar Oct 25 '23 19:10 Liongrass

@Liongrass Okay. I'll probably start with just a few improvements like the short descriptions first. How do I join the LND slack?

nonfungible-human avatar Oct 25 '23 19:10 nonfungible-human

You can find the latest invite link here

Liongrass avatar Oct 25 '23 19:10 Liongrass