I've got a few suggestions on it, which mainly are to use the sendOnSteCommand() over sendOnStepCommandBlind (As most of that returns 0/1) and test for it. That will let us add a check so if it doesn't support it to not do it again, but I'll issue a pull request to you in the next day or two if you want.

I still think that it being chatty isn't an issue except that it often runs into some of the timeouts if something isn't supported, and if we keep doing it, and triggering the timeout, or leaving things in the buffer compared to: If we check it once the timeout happens near enough to startup. We can also gate that with OnStep version check, but I haven't dug into things enough to know when those were added.

Read More...