bug: auto-buy purchases cheapest adventurer rather than most powerful affordable one #150

Closed
opened 2026-03-25 15:10:08 -07:00 by hikari · 0 comments
Owner

Description

When multiple adventurers are affordable, auto-buy selects the one with the lowest cost rather than the most powerful (highest tier) one the player can afford.

Example

  • Peasant costs 100 gold
  • All other affordable adventurers cost 150 gold
  • Auto-buy purchases a Peasant instead of the highest-tier 150-gold adventurer

Expected Behaviour

Auto-buy should prioritise purchasing the most powerful (highest combat power or highest tier) adventurer the player can afford, not the cheapest one. Buying weaker adventurers when stronger ones are available at the same approximate cost is not optimal.

Resolution

Fixed in commit 4c297f1 — the sort key was changed from adventurerB.combatPower - adventurerA.combatPower to adventurerB.level - adventurerA.level, ensuring the highest-tier affordable unlocked adventurer is always purchased first.

This issue was created with help from Hikari~ 🌸

## Description When multiple adventurers are affordable, auto-buy selects the one with the lowest cost rather than the most powerful (highest tier) one the player can afford. ## Example - Peasant costs 100 gold - All other affordable adventurers cost 150 gold - Auto-buy purchases a Peasant instead of the highest-tier 150-gold adventurer ## Expected Behaviour Auto-buy should prioritise purchasing the most powerful (highest combat power or highest tier) adventurer the player can afford, not the cheapest one. Buying weaker adventurers when stronger ones are available at the same approximate cost is not optimal. ## Resolution Fixed in commit `4c297f1` — the sort key was changed from `adventurerB.combatPower - adventurerA.combatPower` to `adventurerB.level - adventurerA.level`, ensuring the highest-tier affordable unlocked adventurer is always purchased first. ✨ This issue was created with help from Hikari~ 🌸
Sign in to join this conversation.
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: nhcarrigan/elysium#150