loader: Expect visit_date as an optional date in constructors

This avoids side-effect in the constructor and also aligns how it's done in other
loaders (e.g git).

Related to T1410