Мертвый код в конструкторе класса Application

[MrWaip]

Я пытался понять, кем заполняется массив starts в Application и у меня есть два вопроса.

1. В конструкторе Application есть ошибка, new Modules('domain', this); создается повторно, никуда не сохраняется и не используется.
   application.js - строка 35 и 37

2. Как-то очень непонятно получается, что Modules заполняет массивы starts класса Application. Как будто нарушен принцип "Information Expert".
   modules.js

[MrWaip]

class Application extends events.EventEmitter {
  constructor() {
    super();
    this.initialization = true;
    this.finalization = false;
    this.root = process.cwd();
    this.path = path.join(this.root, 'application');

    this.schemas = new Schemas('schemas', this);
    this.static = new Resources('static', this);
    this.resources = new Resources('resources', this);
    this.api = new Interfaces('api', this);
    this.lib = new Modules('lib', this);
    this.domain = new Modules('domain', this);

    new Modules('domain', this);
    this.starts = [];

    this.Application = Application;
    this.Error = Error;
    this.cert = null;
    this.config = null;
    this.logger = null;
    this.console = null;
    this.auth = null;
    this.watcher = null;
  }
  ...

[tshemsedinov]

Да, строка лишняя как-то при ребейзе, наверное, попала. Спасибо, удалил. #1598

[tshemsedinov]

А вот с нарушением GRASP принципов это нормальное дело, их можно и нужно нарушать, более того, часто невозможно все вместе удовлетворить. Если тут сделать код менее зацепленным, нужно будет все стартовые обработчики сложить в коллекцию в модуле и после загрузки (а их же две будет, lib и domain) перекладывать из lib и domain в Application. Кода становится больше, а его понятность не увеличивается, т.к. Application будет лезть теперь в Modules, это лучше, но ненамного.

[MrWaip]

Можно starts заполнять через событие в Modules, так будет явно. Мне пришлось прошерстить весь код, чтобы найти где же этот массив заполняется

[tshemsedinov]

Не, это ни в коем случае, классы, не должны использовать события для взаимодействия со своими структурными частями. Тут можно самое правильное создавать Modules через фабрику, которая возвращает структуру:

const { instance: lib, starts: libStarts } = Modules.create('lib', this);
const { instance: domain, starts: domainStarts } = Modules.create('domain', this);
this.lib = lib;
this.domain = domain;
this.starts = [...libStarts, ...domainStarts];

[tshemsedinov]

Но starts заполняется после await load, а он иногда вызывается через Promise.allSettled, в общем, тут все усложняется. 2 секунды занимает найти по кодовой базе .starts., я не вижу, как тут сделать красивее пока что и такое решение меня устраивает.