будет 1) читабельней 2) поддерживаемо

а можно пример?

у меня в запросе идет добавление записи, но одно из значений должно быть получено поиском по другой таблице

сморя чего ты юзаешь для доступа к бд

например  тебя есть  функция которой ты делаешь поиск по таблице1 .

и она выглядит както так

find('select * from table where id = 11',  function(result){
  здесь мы получили данные в переменной result
});

верно ?

нет

а

так

сек я в редактор перенесу

knex('subscriptions')
     .insert(
       knex.raw(
       '(login, room_id)'+
         ' values (:login ,' +
         ' (select id from '+
         ' chat_rooms where `name`=:room_name))',
       {login:login,
        room_name:room_name
       }
     ))
     .catch(function(e) {
       logger.error(e);
     });
 };
 this.subscribe = subscribe;
};

вон тот вложеный селект  он же разбирается примерно так?:
knex('chat_rooms').where({name: ':room_name'});

если я правильно понял документацию то эта конструкция на выходе получит промис.

то есть у нас из неё вырастет then и catch

knex('chat_rooms').where({name: ':room_name'}).select('id')

скорее всего вот так нужно кастануть

следовательно делаеш так:

function getChats (){

  return knex('chat_rooms').where({name: ':room_name'});

}

function insertDataToSubscriptions(data) {

  data = _.pick(data, ['login', 'room_id']);

  return knex('subscriptions').insert(data);

}

getChats()

  .then(insertDataToSubscriptions)

  .then(() => { console.log('FINISH')})

  .catch(() => { console.log('FAIL')})

это примерчик.  Возможно knex умеет более оптимально делать запрос

вероятно оно даже медленней будет чем твой вариант.
или медленней чем просто строка для sql

я заклинание нифага не осознал