Полезно ли всегда возвращать обещание

Нет смысла возвращать промис в синхронных методах.

Обещания обеспечивают абстракцию параллелизма. Когда параллелизм не задействован, например, при предоставлении массива. Возврат промиса ухудшает управление потоком и значительно замедляет его.

Это также передает неправильное сообщение. На самом деле обещание чего-то без причины — довольно распространенный антипаттерн.

Один из случаев, когда он полезен, — это когда метод может быть асинхронным, например: получение чего-либо из кеша или выполнение запроса, если его там нет:

function getData(id){
     if(cache.has(id) return Promise.cast(cache.get(id));
     return AsyncService.fetch(id).tap(cache.put);
}

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

function parseArray(someArray) {
    var result;
    // synchronous implementation
    return Promise.resolve(result);           
}

Эта функция — беспорядок, потому что она может генерировать синхронно, но также возвращает промис. Функция, которая возвращает промис, никогда не должна бросать вызов, иначе вы потеряете большую часть преимуществ промисов, потому что теперь вам придется использовать и try-catch, и .catch().

Правильный способ реализации - "аннотировать" функцию с помощью .method:

var parseArray = Promise.method(function() { 
    var result;
    //Promise.resolve(result) is unnecessary now
    return result;
});

Теперь функция гарантированно никогда не будет вызываться синхронно и может использоваться согласованным образом.