Seems fine for me.
Just from curiosity - it possible that CardDAV will also support incremental
sync ? (I'm not sure if this will be as much usable as in case of PBAP)