Just wanted to note that this hook gets called after the help command is run. If the help command is run, this hook never fires.

Part of me thinks this should be run before the help command. @bryannielsen can you weigh in on this?

@matthewjohns0n yea I would agree that if we're calling this a boot hook I would move it as early as possible in the process and definitely before anything can be sent as output.

FYI - I ended up splitting this into 2 hooks, which both seem to have application here. cli_boot is the hook that is run before anything happens with the CLI command. This allows an add-on dev to potentially hook into commands and adjust parameters or something along those lines.

cli_before_handle is the command that runs right before the CLI handle() function is run. At this point, the command may have failed due to not being found or other processing may happen instead, like the help command. We only run this hook directly before actually running a CLI command.

@bryannielsen @matthewjohns0n do we need second review on this?

@matthewjohns0n the order of parameters in extensions generator was wrong, I fixed it.

I also discovered that cli_boot hook does not really allow you to "hook into commands and adjust parameters" as per your comment. I could only display extra stuff, not modify parameters or anything... Which really makes the usability of it questionable. So perhaps we should just remove it?

I also updated the docs, leaving that for you to review

Thanks for fixing that parameter order @intoeetive. I'm going to test the hooks tomorrow morning and I'll loop back on this one.