7 Commits

Author	SHA1	Message	Date
prosolis	bae83271fe	Combat engine §6: a caster casts in the rooms, not just at the boss ... §6 was filed as "clerics are weak in solo — lift the trailer". It is not a balance problem and a tuning dial would have buried it. Re-baselined on HEAD: cleric still last, 46.4% vs fighter 81.4%. But cleric *dies* less than mage, sorcerer or warlock. Its entire deficit is FLEEING — 167 of 500 runs end with the player alive and the expedition over, where fighter and ranger flee zero. Instrumenting the stopEnded sites: every one of those runs ends in an ORDINARY room fight. Not a patrol, not an elite, not the boss. An ordinary room is runZoneCombatRoster → SimulateCombat on an 8-round clock: one breath, no turn engine, no action picker. The only spell that could ever land there was a PendingCast the player queued BY HAND before walking in. On autopilot nobody queues one — so for every room of every expedition, a caster fought with a weapon and nothing else. A cleric has no Extra Attack, a mace, and its whole kit unusable: it grinds the 8 rounds out, and a wounded one starts losing fights a fighter never loses. One room loss ends the whole expedition. The tell is that dnd_class_balance.go — the harness the class corpus was tuned on — ALWAYS hands a caster their best damage spell before it simulates. The numbers that say "cleric is a weak class" modelled a cleric who casts. The live room modelled one who does not. The corpus and the game disagreed about what a cleric is. Same defect as the rest of this plan: the action model is narrower than the kit. §1 (the picker never healed), §3 (the companion never acted), Pete (LoadDnDCharacter → nil → "attack"), and now every caster in every room. The spell is additive pre-damage, exactly as a hand-queued PendingCast has always been, so this stays comparable to the corpus instead of being a new mechanic. It is slot-aware and NOT free. pickBestDamageSpell reads slotsForClassLevel — the theoretical class table — so reusing it would have cast an unlimited leveled spell every room: the same "no row to persist onto, so it arrives fresh" free lunch that gave the companion an infinite body. The new picker reads the seat's real remaining slots and spends one. It never upcasts. The big slots are what the elite and the boss are for and the turn engine spends them there; a picker that nukes a goblin with a 5th-level slot leaves the caster swinging a stick at the thing that matters. Both goldens byte-identical — they pin the engine, and this changes its inputs at the zone layer. Martials provably untouched. UNMEASURED: the verification sweep is still in flight. Read cleric's fled count, and watch bard/druid for overshoot — they get the same buff and were not the problem. Claude-Session: https://claude.ai/code/session_01J5SQZWoLmL3M3mw2XmHHdy	2026-07-11 16:30:41 -07:00
prosolis	055f07d3c0	Combat engine §2(a): the enemy charges for what a seat brings ... Both scaling levers counted seats. partyEnemyHPScale gave +15% boss HP for any roster >= 2, and partyActionExpectation lifted the enemy from 1 to 2.4 attacks a round. A seat COUNT charges the same for an under-levelled friend, a hired NPC, and a true peer — so a below-median body cost a full seat's worth of boss and did not give a full seat's worth back. Measured, once the companion's free full-heal was taken away and he became honest: hiring him was WORSE than going alone (66.1% against solo's 69.0%). That is this bug, and it has been live for every under-levelled friend anyone has ever invited. Seats now carry a SeatWeight, and both levers scale on the summed weight of the LIVING seats rather than on a head count. The weight is level-based, priced against the leader, times a discount for a hireling (no subclass, no magic items, gear that is never Masterwork — the layers a player accrues and a hireling never will). Level, and deliberately not a power score: an HP-x-damage proxy would rank a cleric below a fighter and quietly make every mixed HUMAN party easier, which is a difficulty regression smuggled in under a bug fix. The safety argument is one property: **a peer weighs exactly 1.0**. So the curves interpolate between the integer knots the P8 sweep tuned — (1, 1.0), (2, 2.4), 2n-1 from 3 up — and every integer input returns exactly what it always returned. Solo is byte-identical, a party of same-level humans is byte-identical, both goldens hold unmoved, and only an UNEQUAL roster lands between the knots. That is the entire point of the change. It also finishes §2(b): a seat that is down now buys the enemy nothing. §2(b) fixed the head-count half; a corpse still carried its full weight until this. Measured, 640 runs/arm, same grid: solo 69.0% (unchanged — corpus intact) + Pete 76.8% (+7.8pp) + a human cleric peer 77.6% (+8.6pp) band solo +Pete lift trailing (<40%) 10.0% 31.0% +21.0pp middle 58.9% 76.8% +17.9pp leading (>=70%) 93.5% 99.2% +5.7pp Help, never a carry: he rescues the players who were drowning and barely moves the ones who were already fine — and he stays below a real human of the leader's level, which is the invariant a hireling must never break. Claude-Session: https://claude.ai/code/session_01J5SQZWoLmL3M3mw2XmHHdy	2026-07-11 15:20:49 -07:00
prosolis	27b9de5936	Companion: he carries his wounds, rations his slots, and heals himself ... Three defects, all the same mistake, all found by sweep and not by tests: the companion has no database row for a thing to persist onto, so the thing "arrives fresh next time" — which for a resource means infinite. 1. His spell slots refilled every fight. The ledger went on his combat SEAT, and a seat is per-session. A human rations one pool across a 30-room run and gets it back at camp; rationing it IS the caster's game. Now on expedition_party.companion_slots_used, refreshed at camp. (Worth ~0pp alone — a run holds only ~2 real fights, so the pool never binds. I predicted this was the whole answer. It was not.) 2. His BODY refilled every fight. buildFightSeats seated him at Stats.MaxHP and the close-out skipped him — "he arrives fresh next time", said the comment. That is an infinite body: he soaked a share of every fight's incoming and then reset, while the humans beside him bled all the way to camp. THIS was the carry. Now expedition_party.companion_hp; healed at camp; a dropped companion returns on 1 HP rather than as a corpse, because there is no companion-death rule and inventing one inside a bug fix would be a second feature. 3. No autopiloted caster had ever healed ITSELF. simPickAllyHeal skipped `i == seat` and bailed on !IsParty(), so a solo cleric carried cure_wounds for a whole run and never once cast it. Now simPickHeal: heal whoever is worst off, which is sometimes you. Measured, 640 runs/arm, like-for-like (the leaders whose role-fill gives Pete a Cleric, against a human Cleric follower of the leader's own level): solo 69.0% + a human cleric 77.6% (+8.6pp) + Pete 66.1% (-2.9pp) The reference arm is the point. Against SOLO even a mace-only Pete looked like a carry — but parties are designed to be safer, so solo is the wrong yardstick. Against a human peer the real bug appeared: a gearless, level-penalized hireling was out-clearing a fully-geared human cleric of the leader's own level by 15pp, because he was the only combatant in the game who healed to full between fights. With the free lunches gone he is honest, and honestly a net negative — which is exactly the plan's §2 diagnosis, unmasked: a below-median seat cannot pay for its own enemy scaling (+15% boss HP and 2.4 enemy actions a round instead of 1). §2(a) is next, and the sweep now argues FOR it; before this commit it would have made things worse. Self-heal moved solo 66.1% -> 66.2%, so the balance corpus is undisturbed and no re-baseline is owed. It is also NOT the answer to §6 — casters reach for a healing consumable first and the sim stocks them, so a human rarely falls through to the spell. Pete carries no consumables, so it is his only heal. Claude-Session: https://claude.ai/code/session_01J5SQZWoLmL3M3mw2XmHHdy	2026-07-11 14:56:19 -07:00
prosolis	d538f91cf7	Combat engine: pay off the N-body debt, and hire Pete ... N3 widened the combat *roster* from 1 to N but never widened the action model, the scaling model, or the test net to match. Building the hireable companion walked into all three. Only one of the four defects found was the companion's; the rest have been live in prod for every human party since N3. The party golden did not exist (§5) Solo combat is pinned exhaustively (7468 lines); the entire N-body layer had nothing. That is why a healer class that cannot heal shipped without a test going red. Adds party_characterization.golden (9 scenarios x 5 seeds, incl. weak and dying seats) and TestPartyCharacterization_OneSeatIsStillSolo, so the N-body path can never quietly stop being a superset of the balance corpus. Regenerate only on purpose: -update-party. No action could target another seat (§1) Every heal in the engine was self-scoped. A party cleric could not put one hit point on a friend. Adds turnActionEffect.AllyHeal/AllySeat, `!cast <spell> @user` and `--target @user` -- the latter has been advertised in !help and silently swallowed by parseCombatCast since SP2 ("reserved for SP3"). The auto picker uses it too (simPickAllyHeal), so away-players and engine-driven healers behave like competent ones. It will not raise the dead. Corpses kept buffing the boss (§2b) enemyActionsThisRound counted len(st.actors), dead included -- so a party that lost a member kept paying for them, and the survivor faced a boss still swinging at two-player cadence, alone. A death spiral with the arrow pointing the wrong way. Now counts livingActors(). Party golden moved deliberately for this. An engine-driven seat was a bool any command could clear (§3) autoDriveCombat drives a party by dispatching each seat's turn AS that seat, so a companion's own auto-played move arrived at beginCombatTurn looking like a player returning to the keyboard and cleared the latch that was moving him. He then stood in the fight doing nothing while the boss he had inflated killed everyone. ActorStatuses.EngineDriven is now a persisted seat property that no command clears, and the driver calls driveEngineSeat instead of impersonating. "The party" could be empty (§4) A solo expedition has no expedition_party rows, so asking the roster who was in the party answered "nobody" -- and every caller fell back to something plausible. That is how the companion got hired at level 1 for exactly the player the feature exists for. expeditionParty()/partyHumans() always include the owner. The companion himself (!expedition hire [class] / !expedition dismiss) Day 1, leader only, costs coins, role-fills the gap, globally exclusive. He is an NPC seat and must never become a player: no player_meta, no dnd_character, no inventory, no DM room -- mint him a player_meta row and ensureDnDCharacterForCombat will auto-build the news bot a real character, and he starts appearing in the graveyard and filing death notices about himself. Mail and seats are different sets: he fights, he does not get written to. Measured on millenia, n=750/arm. Before these fixes he was -28pp -- worse than no companion at all. After: solo 48.5% -> 63.9% clear (+15.3pp), with +28.0pp for trailing players and +2.0pp for leaders. Help, never a carry. The solo golden is byte-identical throughout: solo combat provably did not move, and the balance corpus is intact. Known gap: the companion cannot cast (castActionForSeat loads a sheet from the DB and he has none by design), so a hired Cleric is still just a bad fighter. Claude-Session: https://claude.ai/code/session_01J5SQZWoLmL3M3mw2XmHHdy	2026-07-11 12:39:01 -07:00
prosolis	d76c63be0c	Review follow-ups A + B: armed abilities survive the fight, supply pool serialized ... A. An armed ability lasted one round of a turn-based fight. buildZoneCombatants called applyArmedAbility, which applies an ability's mods *and* clears ArmedAbility and saves the sheet. The turn engine calls that builder again on every !attack / !cast / !consume, so round 1 fired the ability and disarmed the character, and every later round rebuilt them with none of its mods. A Berserker paid stamina for a single round of BerserkerRage / RageMeleeDmg / PhysicalResistRage / FrenzyDmgBonus. Every entry in dndActiveAbilities had the same shape. mods.BerserkerRage was not merely unread at close-out — by then it no longer existed. Split arming into its two halves: consumeArmedAbility(c) mutates: disarms, saves, returns the id. Once, at fight start. applyAbilityByID(c, id, mods) pure: no DB write, no disarm. Safe on every rebuild. (No ability's Apply writes to the character, so this really is pure.) armAbilityForFight(c, mods) consume + apply, for the auto-resolve callers that build and fight in one breath. buildZoneCombatants now takes the already-consumed id and re-applies it. The id rides on ActorStatuses.ArmedAbility, seeded per seat at fight start, so partyCombatantsForSession reproduces the ability every rebuild and the close-out can still see that a rage fired. The close-out itself: postCombatBookkeeping now carries grantCombatAchievements + persistDnDPostCombatSubclass, and all four close-outs route through it — runDungeonCombat, runZoneCombatRoster, finishCombatSession, finishPartyCombatSession. It fires on every terminal status, not just a win: a Berserker who rages and loses is still exhausted, which is what auto-resolve always did. Also: buildFightSeats and runZoneCombatRoster consumed the ability before the checks that could sit a seat out, so a downed member was disarmed for a fight they never joined. The refusals now run first. B. Six unlocked read-modify-writes against the shared supply pool. updateSupplies rewrites supplies_json wholesale, so a caller folding its delta onto an *Expedition it read earlier discards whatever landed in between. Handlers run one goroutine per event, so those writers genuinely interleave. All six now go through withExpeditionSupplies, which takes advExpeditionLock, re-reads the row, hands the closure the fresh copy and persists what it returns: nightRolloverBurn (forage + burn in one write), grantTwoWeeksCache, advanceToNextRegion's transit burn, campPitch, pitchAutopilotCamp, and the ambient pack-rat drain. expeditionCmdAccept's hand-rolled lock folds onto the same helper. expedition_sim.go is left alone: single-threaded, takes no locks. Known consequence, for the balance track: trySimAutoArm used to live inside the rebuild, so a simulated Fighter (second_wind) or Cleric (healing_word) re-armed and re-spent a resource every round of every elite/boss fight. expedition-sim drives those through the turn engine, so every prior expedition-sim corpus overstates those two classes. Re-baseline after this, not before. Claude-Session: https://claude.ai/code/session_017mEwUmmS7aQTP2NQXj6rUa	2026-07-10 08:01:51 -07:00
prosolis	1f211564d9	Review fixes: party close-out, member soft-lock, supply race, elite loot, season crown ... Five bugs found reviewing n1-restoration end to end. beginCombatTurn settles any phase the engine owes before reading whose turn it is. That settle can end the fight — and the old code then answered "you're not in a fight" and returned. The terminal status was already persisted, so nothing ever paid the party out: no XP, no loot, no death recorded, no run teardown. The reaper cannot recover it either, because listExpiredCombatSessions filters on status='active'. Close the fight out there, the way the !fight start path and the reaper already do. A party member was permanently soft-locked when their leader extracted and never resumed. seatedExpeditionFor (the guard) spans 'extracting'; expeditionForMember (what !expedition leave resolved through) saw only 'active'. So the member was refused any new adventure by the guard and told "No active expedition" by the command the guard points them at, with nothing sweeping stale rows and only the leader able to clear one. Resolve the exit through the same lookup as the gate. updateSupplies overwrites supplies_json wholesale, and expeditionCmdAccept folded a member's packs onto a snapshot read before the coin debit, unlocked. Handlers run one goroutine per event, so two invitees accepting genuinely interleave and one member's packs vanish. advUserLock cannot help — it is keyed by sender, so racing members take different mutexes. Add advExpeditionLock and re-read the pool under it. Closes accept-vs-accept; the six other updateSupplies callers still race and are written up separately. runHarvestInterrupt picked an elite enemy and elite narration off a local `elite` flag, then passed a hardcoded false as isElite to closeOutZoneWin. dropZoneLoot gates masterwork on isBoss||isElite, so beating an elite interrupt skipped the masterwork roll and took standard treasure weight — while the same elite fought via !zone paid out correctly. arenaSeasonRollover marked its job complete even when recordArenaSeasonTitle failed, and JobCompleted short-circuits every later run for that quarter, so a transient SQLite BUSY lost the crown forever. Defer completion on failure; the insert is ON CONFLICT DO NOTHING against PRIMARY KEY (season, kind) and a past season's data is frozen, so the retry is safe. Also: drop dead partySurvivors, collapse the zoneCombatRoster alias into fightRoster, route partyCasualtyLine through joinNames, fold four copies of the expedition column projection into expeditionSelectCols, stop replyDM sending a blank DM, and correct two doc comments describing a path that no longer exists. Deliberately not fixed, with reasons, in gogobee_code_review_followups.md — most notably that both turn-based close-outs skip grantCombatAchievements and persistDnDPostCombatSubclass, which the auto-resolve paths run.	2026-07-10 07:20:14 -07:00
prosolis	08d3053368	N3/P6e: a party that fights every room ... Only elite and boss doorways seated a roster. Everything else -- exploration rooms, patrol encounters, harvest interrupts -- resolved through SimulateCombat against ctx.Sender, and P6d made the walk commands leader-only. So on a 38-room T5 expedition a party of three fought together twice and the leader soloed the other ~35, then died alone while two untouched members stood at full HP. The plan said the N-body core was already there and only the callers passed one player. It wasn't: SimulateCombat built a one-seat roster internally. But the resolution primitives already read st.c -- the cursor's Combatant -- because the turn engine has called them that way since P3. Only the round loop needed widening. combat_engine_party.go carries it: simulateParty, simulatePartyRound, roundInitiative, enemyTargetSeat. Every roster short-circuit collapses for one seat, copying P3's solo exemptions, so the RNG draw order is unchanged and SimulateCombat is now simulateParty([]Combatant{p}, ...).Seats[0]. TestCombatCharacterization is byte-identical; TestSimulateCombat_IsTheOneSeatPartyCase pins the delegation event-for-event across 40 seeds. zone_combat_party.go carries the callers' half: runZoneCombatRoster fans out the character-scoped close-out (HP, XP, achievements, subclass, heal items burned, Misty's repair) per seat, while loot, threat, kill records and death stay with whoever knows the room. runZoneCombat remains the explicit solo entry point -- the arena calls it, and an arena bout must never drag in a party. Death is read per seat off HP, never off the fight's terminal status: a timed-out party can still have lost somebody, and a solo player at 0 HP has already ended the fight, so PlayerEndHP <= 0 is exactly the old !TimedOut rule. Preserved deliberately: a solo player can win at 0 HP (a retaliate aura kills the swinger on the killing blow, and resolvePlayerAttack returns before enemyDown is consumed) and is not marked dead. A party marks its downed seats dead on a win, which is what finishPartyWin always did. Solo T5 re-sweep is unregressed (fighter 47-73%, cleric 20-33%). Party of 3 now clears 100% of every T5 cell, which is P8's problem: the enemy takes one turn per round and swings at one seat, so a party of N deals xN damage and each member takes ~1/N^2 of the solo incoming. An HP scalar cannot close that -- it restores the fight's duration, not the enemy's action economy.	2026-07-10 06:32:29 -07:00