JavaScript — заполняем нишу между микросервисами и объектами — «нано-сервисы»

Если у вас большой размер кода и нужно грамотно разделять зоны ответственностей модулей, то самое время внедрить Typescript, а не устраивать валидацию в рантайме.

Про Proxy: во-первых, это уже часть стандарта языка, поддерживается в stable версии Node. Во-вторых, несмотря на использование Proxy, код остаётся типизированным и предсказуемым. В нем всегда можно проследить где метод декларируется и где вызывается. А в вашем подходе найти концы не представляется возможным

Ну смотрите, вот вы делаете присваивание this.service[service][funcName] = this.proxyUpcomingExecute.bind(this, service, funcName). Но в функции proxyUpcomingExecute первые два параметра при этом используются вот так:

let grantedItem = this.requireDict[serviceName][action] // action = funcName
return grantedItem.layer.executeAction(grantedItem.ticket, serviceName, action, ...args)

Это означает, что можно было бы биндить непосредственно grantedItem.layer.executeAction вместо proxyUpcomingExecute. Более того, у вас requireDict заполняется в той же функции где и биндится proxyUpcomingExecute — то есть можно было бы вообще использовать непосредственно переменную lookUpRes (которая после становится grantedItem) в той ветке где она уже имеется (а там где ее еще нет — можно отложить заполнение словаря service, избавившись тем самым от requireDict полностью).

Теперь разберемся что такое ваш grantedItem, он же lookUpRes. А это структура из двух свойств, которая формируется вот так: { ticket, layer: this }, при этом единственным ее предназначением является вызов вида x.layer.executeAction(x.ticket, serviceName, action, ...args). Но в таком случае можно было вместо структуры возвращать простую функцию, this.executeAction.bind(this, ticket)!

Это бы одновременно избавило вас от проверки this.grantedTickets.has(ticket) внутрях executeAction — потому что вызов этой функции с невалидным тикетом стал бы невозможным в принципе — вы же this нигде в явном виде сервисам не раскрываете (а еще можно перейти на Typescript и объявить ее приватной).

Продолжаем копать дальше. Первой же строчкой в функции executeAction идет let service = this.registeredWorkers[serviceName] — тот самый который был найден внутри serviceLookup. Его тоже можно забиндить.

Наконец, doExecute. Все что делает этот метод — вызывает this.service[action]. Поскольку action — опять константа, его тоже можно упростить.

В целом, если хорошо по-упрощать ваш код, его можно привести вот примерно к такому виду.

processImports(lookupFn) {
    for (const [serviceName, actions] of Object.entries(workerConfig.require || {})) {
        const imports = this.service[service] = {}

        for (const action of actions) {
            const resolved = lookupFn(serviceName, action)
            if (resolved.kind == "success")
                imports[action] = resolved.fn
        }
    }
}

И это вся необходимая обработка импортов!

При этом ту часть что лежит в layer.js можно написать как-то так:

serviceLookup (callerLayer, callerName, callerGroup, serviceName, action) {
    const ticket = { callerGroup, callerName, callerLayer }
    const service = this.registeredWorkers[serviceName]
    if (!service) {
        //  логи, логи
        return
    }

    const { code, fn } = service.tryResolve(callerGroup, action)
    if (code != "success") {
        //  логи, логи
        return
    }

    return (...args) => {
        if (!this.isReady()) /* Ошибочка */;
        if (/*включен аудит*/) /* аудитим */;
        fn(...args);
    }
}