d-e-s-o / apcacli Goto Github PK

It would be really nice to add extension support to the application. What I mean by that is basically having a way to write scripts/programs that can be invoked as an apcacli sub-command, similar to how you can write git or cargo sub-commands.

We have written something like that for nitrocli. From my perspective, most of that could be copied to apcacli. A discussion thread with most of the design discussions is d-e-s-o/nitrocli#50 (though not everything is applicable) and the main part of the implementation d-e-s-o/nitrocli#112 (likely more relevant). I don't recall all the details off the top of my head, so I don't know exactly how much is transferable and whether there needs to be some design work.

Anyway, this would be really nice to have and would allow us to move functionality that is currently stand-alone into the repository. E.g., I have a program that is useful for mostly manual trading, which basically ensures that stop orders are present for all positions that are above a certain percentage gain (which I see I haven't even published yet).

We may not want to emit colors when the input is not a TTY. E.g., when redirecting the output to a file. I think we should add a switch to force usage of color, though. There is also NO_COLOR, though off hand I am not convinced that we need to follow it.

We should convert the crate from using structopt to using clap v3, which is the new kid on the block for argument parsing (well, sort of, v2 has been around for ages and was the de-facto standard, but v3 does a few more things). I recall there being some kind of tutorial specifically aimed at conversions like this, though I couldn't find it off hand. May be useful to check for that first. My notes reference this, but I was under the impression there was a more formal document.

It would be nice if we were honoring the current locale for formatting of prices and other decimal values. Well, and potentially dates. And whatever else may be locale-dependent.

Unfortunately, last I checked there basically weren't any crates around that provided such functionality. Need to check first whether that has changed.