Clearing site data makes total sense. Otherwise after a year of web usage you have multiple GB of stored website data that the user does not know what it is being used for or how to clear it.
And the majority of uses for local data is to cache fetched data or store temporary preferences not worthy of being in the server side database.
This makes total sense to me, and I’m happy Facebook can’t store tracking data in my Safari for longer than 7 days.