The underlying std::string is freed?

My slightly different two cents: Returning const string& is a micro-optimization, and you should return _string_ everywhere. Not string_view. You have zero idea of the lifetime of a non-owning return parameter (e.g. what happens if someone else on a different thread starts accessing the object, does that invalidate the object you are holding?), and people are notoriously bad at documenting them.