From 1b825498bd662663b8f6591684bcfb6b7af5dce0 Mon Sep 17 00:00:00 2001 From: prosolis <5590409+prosolis@users.noreply.github.com> Date: Wed, 8 Apr 2026 17:22:04 -0700 Subject: [PATCH] Fix harvest actions remaining to use consistent harvestMax pattern MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The nudge message after an action was computing harvest remaining inline with maxHarvestActions + 1 for holidays instead of using the harvestMax++ pattern used everywhere else. Functionally equivalent but inconsistent — would break if the holiday bonus ever changed. Co-Authored-By: Claude Opus 4.6 --- internal/plugin/adventure.go | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/internal/plugin/adventure.go b/internal/plugin/adventure.go index b18e07f..b187099 100644 --- a/internal/plugin/adventure.go +++ b/internal/plugin/adventure.go @@ -930,10 +930,11 @@ func (p *AdventurePlugin) resolveActivity(ctx MessageContext, char *AdventureCha remaining = append(remaining, "combat") } if char.CanDoHarvest(isHol) { - harvestLeft := maxHarvestActions - char.HarvestActionsUsed + harvestMax := maxHarvestActions if isHol { - harvestLeft = maxHarvestActions + 1 - char.HarvestActionsUsed + harvestMax++ } + harvestLeft := harvestMax - char.HarvestActionsUsed remaining = append(remaining, fmt.Sprintf("%d harvest", harvestLeft)) } p.SendDM(ctx.Sender, fmt.Sprintf("Actions remaining today: %s. Type `!adv` for the menu.", strings.Join(remaining, ", ")))