fix: parse NoArgumentConverter correctly

[AstreaTSS]

What type of pull request is this?

• Non-breaking code change
• Breaking code change
• Documentation change/addition
• Tests change

Description

This PR fixes a number of issues that NoArgumentConverter has, detecting if it's a class and filling NoArgumentConverters in even after all of the arguments have been exhausted.

...this is what hybrid commands do to you.

Changes

• Move out NoArgumentConverter checking to its own function.
• Add in an extra check to see if the annotation was possibly a class and a subclass of NoArgumentConverter.
• Fill in NoArgumentConverter even after all of the parameters have been "exhausted".

Checklist

• I've formatted my code with Black
• I've ensured my code works on Python 3.10.x
• I've tested my code