Рекомендация к использованию синхронного метода для работы с такими жирными данными как изображения да ещё с необходимостью руками чистить память вместо асинхронного и более простого выглядит как начало новой книги Вредных советов Григория Остера.

Есть ли пруфы, что так лучше? Бенчмарки там, ещё что-нибудь?

Несколько вопросов по data-state

Не понимаю зачем хранить dataStates вне comments?
Ах наверное что бы делать лишние тело движения comments.dataState = dataStates.loading;
Реализацию лучше скрыть внутри comments , думаю следующий код откроет вам глаза где слабое место в вашем примере.

const comments = (dataStates => {
    const hiddenVar = {
        dataState: dataStates.notAsked,
        errorText: '',
        data: [],
    };

    return {
        set data(val) {
            hiddenVar.dataState = dataStates.loaded; // 🔥
            hiddenVar.data = val
        },
        get data() {
            return hiddenVar.data.slice(0);
        },

        set errorText(val) {
            hiddenVar.dataState = dataStates.failed; // 🔥
            hiddenVar.errorText = val;
        },
        get errorText() {
            return hiddenVar.errorText;
        },

        set loading(val) {
            if(val){
                hiddenVar.dataState = dataStates.loading; // 🔥
            }
        },

        get isLoading() {
            return hiddenVar.dataState === dataStates.loading;
        },
        get isLoaded() {
            return hiddenVar.dataState === dataStates.loaded;
        },
        get isError() {
            return hiddenVar.dataState === dataStates.failed;
        }
    }
})({
    notAsked: 'notAsked',
    loading: 'loading',
    loaded: 'loaded',
    failed: 'failed',
});



 // Более лаконичный вариант 
 function loadComments() {
     comments.loading = true;
     fetch('/comments').then((response)=>response.json()).then((data)=>{
         comments.data = data;
     }).catch((error)=>{
         comments.errorText = error.message;
     });
 }

покуда нет private свойств вынес в отдельный объект hiddenVar скрытые переменные.
заменить значения dataStates на числа
можно пойти дальше и добавить в реализацию onChange что бы при изменении hiddenVar.dataState вызывались слушатели(listeners) этого объекта comments

Currently, the user must click the readme.md file manually .if you follow the naming as README.md its resolve the issue

Если сами файлы называть однообразно index.js и index.spec.js:

было

utils
├── chunk
│   ├── chunk.js
│   └── chunk.spec.js
├── flat-map
│   ├── flat-map.js
│   └── flat-map.spec.js
└── pluck
    ├── pluck.js
    └── pluck.spec.js

стало

utils
├── chunk
│   ├── index.js
│   └── index.spec.js
├── flat-map
│   ├── index.js
│   └── index.spec.js
└── pluck
    ├── index.js
    └── index.spec.js

то для подключения утилиты не придется дважды дублировать ее название:

было

import chunk from './chunk/chunk';

стало

import chunk from './chunk';

Как бонус, для переименования утилиты достаточно переименовать папку, не переименоввывая файлы.

В совете про target blank достаточно указать rel="noopener". Проблему безопасности это решает.

noreferrer лучше не добавлять, потому что это будет ломать статистику переходов (для сайта, на который стоит ссылка). Системы аналитики будут думать, что посетители пришли не с какого-то сайта, а набрали адрес вручную (или перешли из приложения и т.п.)