Przede wszystkim tego nie da się czytać bo kod ma miliard znaków szerokości. Nie możesz tego poformatować? 80 - 120 znaków na linijkę to max. Po to używamy eslinta, żeby unikać takiego kodu. Po drugie czemu App nie jest w components ?. Gdzie są propTypes ? W Colors js zmienna list to chyba oddzielny komponent. To samo w similarColors. W app.js - getRanodomColor wydziel do service, to samo z uniqueID, generateHex. Raz używasz "", a innym razem ''. Czemu ? Nie można tego ujednolicić? To tak na szybko.