Stuff like this is really great "...Side note: If you are looking at the values directly in zookeeper, remember that hbase.zookeeper.RecoverableZookeeper adds metadata to znode values." Belongs in more general section on zk.
Maybe allow somehow that it may not be /hbase/replication on your cluster (because /hbase is actually settable in zk config). Is that important? (Maybe you can just add a little note to your section on 'Configuration Parametters'
We should fix it so you don't need peer-state, so peers has attribute on whether or not enabled.... but thats not this issue.