Functions that take a `std::string_view` but construct a `std::string` internally should just take a `std::string const&`

[TartanLlama]

This will avoid incurring additional allocations at the binding boundaries