1) Предпочтительнее использовать string из std, в этом случаи вам не надо заниматься явным приведением типов.
2) Если вы используете константы для всего приложения, перенесите их на уровень вашей текущей единицы трансляции. Снабдите коментариями для ясности.
3) fmt::format здесь лишняя зависимость. Вы можете воспользоваться сразу Си-строками, или сформировать строку через конкатинацию.
4) Если вы всегда возвращаете true, тогда зачем вообще что-либо возвращать? void должно быть достаточно хорошо.
