Не работает $function с параметрами после build

[alexeyten]

После build-а функция уже с параметрами и получается вложенный массив.
{ '$function': [ [ 'test', { data: 42 } ], { ENV: 'test' } ] }. Надо склеивать параметры функции или обрабатывать вложенный массив.

'use strict';

const buildConfig = require('dollar-config/build');
const DollarConfig = require('dollar-config');

const source = {
    func: { $function: [ 'test', { data: 42 } ] }
};

const functions = {
    test: (params) => params.data
};

const good = new DollarConfig(source, { functions });
console.log('GOOD', good.get('func', {})); // => GOOD 42

const builded = buildConfig(source, { ENV: 'test' });
const bad = new DollarConfig(builded, { functions });

console.log('BAD', bad.get('func', {})); // Кидает ошибку
/*
.../node_modules/dollar-config/index.js:189
    path = path.split('.');
                ^
    TypeError: path.split is not a function
    at get (.../node_modules/dollar-config/index.js:189:17)
    at Object.$function (.../node_modules/dollar-config/index.js:44:16)
    at Config._resolve (.../node_modules/dollar-config/index.js:110:44)
    at Config.get (.../node_modules/dollar-config/index.js:80:30)
    at ...
*/

[razetdinov]

Возможность передавать параметры в функцию пока не открыта (и валидацию через ajv не пройдёт). Я думал над этим и задал себе такой вопрос: можно ли при вызове переопределять (доопределять) params?

[alexeyten]

Я хотел сделать примерно такую функцию getResHeader и использовать в логировании

getResHeader(req, name) {
    return req.res.getHeader(name) || '';
}

redirect_to: { $function: [ getResHeader, 'Location' ] }

[razetdinov]

То есть первый «аргумент» в вызове превращается во второй в декларации. Если так норм, можно сделать.

[alexeyten]

Мне ещё не нравится, что при вызове функции происходит склеивание параметров из build и текущего req. https://github.com/razetdinov/dollar-config/blob/master/index.js#L43

[razetdinov]

Это сделано на случай, если параметр, который мы инлайним, используется внутри функции. Заранее мы это знать не можем, поэтому мёржим параметры. Функция должна вести себя одинаково как со сборкой, так и без. Пока склоняюсь к такому решению. Первым элементом название функции, вторым — массив аргументов, третьим — собранные параметры (если есть).

До сборки:

redirect_to: { $function: [ getResHeader, [ Location ] ] }

После сборки:

redirect_to: { $function: [ getResHeader, [ Location ], { ENVIRONMENT: production } ] }

[alexeyten]

Тоже какая-то лажа. Надо ещё подумать

[alexeyten]

А обязательно массив?
Может как-то так?

$function:
  name: getResHeader
  args: [ Location ]
  bind: { ENVIRONMENT: production }