I think you need to define what this method can return in the docs. I assume $cmd->{NoAction} returns a 0 or 1 (or maybe undef/1 ?), but $out is a string, which means that if $out = "0" then anyone testing this return code for true/false could get a false value when something actually happened. I think. Defining the return semantics would help a lot in resolving my confusion here. :)

By extension of this, I wonder if the comment based documentation at the top of this method should be replaced with POD instead? While this does look like an internal method, it also looks suitably useful for other contributors of this component that POD is a good idea?