As you mentioned the migration guide: it is explicitly mentioned there, as there is a section “what about .data ?” 
But in general I agree with you.
@albanD isn’t it possible to replace all .data calls with .detach() in the internals?