Цена композиции в Javascript-мире

Не совсем корректно сравнивать реализации композиции и наследования. Ведь композицию можно сделать иначе — без пересоздания объекта на каждый чих.
А если сделать объекты без методов? Пусть будут stateless сервисы по колдунству, что умеют работать с колдунами и паладинами, а с рыцарями пусть не работают? Взамен они будут возвращать или измененный объект или новый инстанс объекта с измененными свойствами.

Вечная проблема с этим воображаемым противостоянием композиции и наследования и грубые примеры из разряда «я ехал на машине, не останавливаясь вышел покурить и сламал себе ноги, давайте лучше ездить на велосипедах». И, конечно, примеры из разряда: «давайте сделаем как можно повыё, типа функционально у нас все, потом за смузи будет обсуждать этих идиотов джавистов, которые создают гориллу, жующую банан».

Во-первых, нафиг такое усложнение с Object.assign, которое ломает автодополнение?

function castSpell () {
  console.log(`${this.name} casts ${spell}!`);
  this.mana--;
}

function fight () {
  console.log(`${this.name} slashes at the foe!`);
  this.stamina--;
}

const fighter = (name) => ({{
    name,
    health: 100,
    stamina: 100,
    fight
})

const mage = (name) => ({
  name,
  health: 100,
  mana: 100,
  castSpell
})

const paladin = (name) => ({
  name,
  health: 100,
  mana: 100,
  stamina: 100,
  fight,
  castSpell
});

Но про такую фигню статью не напишешь, да? Да и бгомеркий this, который используют только грязныё джависты. О, кстати, давайте соблюдать DRY.

const character = (name) => ({
  name,
  health: 100,
});

const fighter = (name) => ({
  ...character(name),
  stamina: 100,
  fight
})

const mage = (name) => ({
  ...character(name),
  mana: 100,
  castSpell
})

const paladin = (name) => ({
  ...character(name),
  mana: 100,
  fight,
  castSpell
});

Опс, наследование получилось! Фигня какая. Это ведь почти горила-жрущая-банан! Упс. Тут или как мерзкие джависты, или по-модному, копипастя самого себя три раза в каждом объекте.
Я уж молчу, что ваше предложение вообще не гарантирует, что добавив fight в очередной инстанс программист добавит так же stamina.

найти ложку дегтя в предложенном решении...

… достаточно зачерпнуть ложкой в любом месте вашего решения.

Я уж молчу, что последнее решение точно так же делается на классах, как и без них.

А как гейм-девелопер скажу, что решение — отвратительное и вообще не близко к практике. У мага и паладина вообще разные спелы, а какой-то спел может использовать стамину вместо маны. Более того, персонаж может получить возможность кастовать спелы, подняв «посох безумного огня» и потом внезапно её потерять, когда в этом посохе закончатся заряды. А ещё персонаж может использовать магию со свитков, которые тоже не затрачивают ману, но требуют определенного уровня интеллекта. Я уж молчу о том, что нельзя просто взять и изменить «стамину» — она должна измениться через анимацию.

Потому такие действия делают отдельными событиями-классами, на которые можно подписаться через эвент-систему и которые статически типизируются, потому что в отличии от клепания одинаковых формочек на редаксе в геймдеве такой быдлокод не пройдет.

Ну вот, к примеру: habr.com/ru/company/pixonic/blog/413729

А наследование и композиция просто должны применятся в разных местах. ОБА этих инструмента по-своему полезны.

Починил ваш код, чтобы компилятор смог его cоптимизировать:

const canCast = (state) => {
  state.mana = 100
  state.cast = function() {
    console.log(`${this.name} casts fireball!`);
    this.mana  --  ;
  }
}

const character = (state) => {
  state.health = 100
}

const mage = (name) => {
  let state = { name }
  character(state)
  canCast(state)
  return state;
}

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

но я практически уверен, что можно сделать еще лучше

Мне кажется есть проблемма с пониманием JS и фабрик. Попрубуйте так:

const allProperties = {
  addCommon(state, name) {
    state.name = name;
    state.health = 100;
  },
  addMagic(state) {
    state.mana = 100;
  },
  addFight(state, name) {
    state.stamina = 100;
  }
};

const allMethods = {
  cast(spell) {
    console.log(`${this.name} casts ${spell}!`);
    this.mana--;
  },
  fight(spell) {
    console.log(`${this.name} slashes at the foe!`);
    this.stamina--;
  },
};

function Mage2(name) {
  allProperties.addCommon(this, name);
  allProperties.addMagic(this);
}
Mage2.prototype.cast = allMethods.cast;

function Fighter2(name) {
  allProperties.addCommon(this, name);
  allProperties.addFight(this);
}
Fighter2.prototype.fight = allMethods.fight;

Результат 57 против 496. То есть в 6ть раз быстрее.

а чем плох вариант наследования, описанный в MDN, последний пример (mix-ins)?