Skip to content

Backmerge 1.39.0 - #3323

Merged
ipdae merged 2 commits into
developmentfrom
release/1.39.0
Oct 16, 2026
Merged

ipdae merged 2 commits into
developmentfrom
release/1.39.0

Conversation

@ipdae

@ipdae ipdae commented Oct 16, 2026

Copy link
Copy Markdown
Contributor

release/1.39.0 의 릴리즈 커밋을 development 로 백머지합니다.

🤖 Generated with Claude Code

ipdae and others added 2 commits October 12, 2026 19:43
* fix(boost): cap world boss and adventure boss reward boosts at 5x

A BoostScheduleSheet row for WORLD_BOSS_BATTLE_REWARD, WORLD_BOSS_KILL_REWARD,
WORLD_BOSS_RANK_REWARD, ADVENTURE_BOSS_FLOOR_REWARD or
ADVENTURE_BOSS_FIRST_CLEAR_REWARD had no upper bound. A mistyped value (e.g. a
block index pasted into the value cell) made ClaimRaidReward add several
int.MaxValue circle counts to one inventory slot, overflowing it to a negative
count.

Each boosted amount is now capped at 5 times the drawn amount, ADD included,
computed in 64 bits (WorldBossHelper.MaxWorldBossRewardBoostFactor,
AdventureBossHelper.MaxAdventureBossRewardBoostFactor). Lower bounds are
unchanged (world boss 0, adventure boss 1) and an amount of 0 is still never
boosted.

Also make SummonHelperTest.GetSummonRecipeIdsWithGradeGuarantee_AppliesBoost
fail when the boost is dropped: assert the last 7 summons are all grade 4+
and that the unboosted run with the same seed is not.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* docs: document constants next to the boost reward caps

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
@ipdae
ipdae merged commit 9759223 into development Oct 16, 2026
40 of 41 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant