Is it generally acceptable to allow a Visitor to modify state of the Receiver, or should that be a Command pattern instead?
The purpose of the visitor pattern is to allow new operations to be added to a class heirarchy without modification to that heirarchy. I've never seen anyone suggesting that only read-only operations are acceptable. The only limitation is that the added operations should only use the public interface of the class heirarchy.