Don't use toArray(String[]::new), use toArray(new String[0]) instead when converting from a List<String> (especially an ArrayList<String>) to a String[]

Just some quick looks at the code. Only little tweaks that don't really have a big impact either way, so you could totally just resolve the comments and not use them if you think the current way is fine.

Sorry, I couldn't get to the documentation because I have to go now. I can just suggest looking through the GitHub Actions failures, which is complaining about the spacing between some of the lines (all lines should have 1 empty line between them).

Documentation reviewed as requested. I've requested a couple changes, notably:

• A bit of re-organizing of the CommandArguments page by removing the table-of-contents and the "What terms are used?" sections
• Changing the format of "Access the inner structure directly" to match the way other pages in the documentation explain similar concepts

There's a lot of repetition with the node name and accessing by index under each section! After a long think about it, I'm going to approve that - I don't think it needs changing, but ensure that when it comes to writing examples you keep them concise - an example isn't necessary for every single method and once you've given an example of one concept you don't have to repeat it for every one (for example, you can write an example for get(String) and get(int), an example for getOrDefault(String, Object/Supplier) and getOptional(String) in the "Access arguments" section. In the "access raw arguments" section, an example of just accessing getRaw(String) would be sufficient and likewise with the "access unsafe arguments" section - an example of getUnchecked(String) would be sufficient).

An explanation at the top of "access raw arguments" is necessary to let users know what we're talking about (refactoring the explanation in "What terms are used" to this section would make a lot of sense here). Likewise with unsafe arguments.

Hopefully this review isn't too long!

Other than the issue with JavaDocs for the CommandArguments record, we're ready to merge.