avoid unnecessary string copies in function parameters

[IamRezaMousavi]

Change std::string parameters to std::string const& in a few functions where the string is only read.

[arvidn]

I believe some of those implement interfaces, and changing them would require changing the interface and all other implementations as well.

Also, if possible, I think std::string_view is generally a better substitute for by-value std::string.

[arvidn]

it's advisable to make sure the changes compile locally, before pushing the PR.
Also, your agent should monitor the CI status, so it can react to if failing