Освен нещата, които ти е казал Жоро, виждам и още един проблем:

std::vector<std::shared_ptr<FileSystemObject>> dirs;

Горното е дефинирано преди тялото на основния цикъл, а вътре само дабавяш елементи към него.

Ако има само една вложена папка, това ще работи (тя ще си сложи нейните елементи вътре и ще извика getTreeView с тях), но ако се случи да има две или повече папки на едно ниво в дървото, те ще извикат getTreeView за всички предишни + техните си елементи.

Златно правило* е променливите да се дефинират там, където се използват, т.е. с възможно най-локален скоуп.

В конкретния пример, това е в тялото на if (dir != nullptr) { ... }. Ако го беше спазил, нямаше да имаш този проблем :)

*Друго златно правило е, променливите да се инициализират в момента на декларирането им винаги, когато това е възможно.

В конкретната задача това e не само възможно, но и доста по-лесно и кратко (буквално е един ред) от инициализацията в цикъл, която си направил. Погледни какви конструктори има vector и помисли кой от тях е най-лесен за позване в случая.

Между другото, по условие в задачата се иска да сортираме елементите във всяка директория по азбучен ред. Това липсва в текущото ти решение. Вариант е да сортираш вектора след като го инициализираш и преди да го използваш рекурсивно в getTreeView.

Hint: Има и по-хитър вариант с използване на друга структура вместо vector, която ще сортира елементите за нас, но за да го приложиш, ще ти трябва допълнителен метод, който да бъде викан рекурсивно вместо getTreeView, защото последния приема  vector като параметър.