drwily/gogobee
1
0
Fork 0
mirror of https://github.com/prosolis/gogobee.git synced 2026-07-16 00:52:40 +00:00
Code Issues Packages Projects Releases Wiki Activity

Adventure: skip overlevel penalty when no higher tier is accessible

Browse Source 
Previously, advOverlevelMultiplier penalized every action where the
player's effective level exceeded the location's MinLevel by 4+, with
no escape valve. That punished players grinding at their max-available
tier (e.g., a level 50 character at the highest mining location they
qualify for, with nothing higher unlocked).

Now: the multiplier short-circuits to 1.0 when no accessible higher-tier
location of the same activity exists for that player. Signature changed
from (effectiveLevel, minLevel) to (effectiveLevel, *AdvLocation) so the
helper can scan the activity's location list. combat_bridge.go updated
to match.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
prosolis
2026-04-28 22:36:17 -07:00
parent 07ca5288c3
commit d93dc4faa4
2 changed files with 15 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
internal/plugin/adventure_activities.go
View File

@@ -603,11 +603,22 @@ func advCheckBrokenEquipment(equip map[EquipmentSlot]*AdvEquipment) []EquipmentS
// advOverlevelMultiplier returns a multiplier (0.05–1.0) that reduces XP and // advOverlevelMultiplier returns a multiplier (0.05–1.0) that reduces XP and
// loot when a character's effective level far exceeds the location's minimum. // loot when a character's effective level far exceeds the location's minimum.
// Gap 0-3: no penalty. Gap 4+: −15% per level over 3, floor 5%. // Gap 0-3: no penalty. Gap 4+: −15% per level over 3, floor 5%.
func advOverlevelMultiplier(effectiveLevel, minLevel int) float64 { // No penalty if no higher-tier location of the same activity is accessible.
gap := effectiveLevel - minLevel func advOverlevelMultiplier(effectiveLevel int, loc *AdvLocation) float64 {
gap := effectiveLevel - loc.MinLevel
if gap <= 3 { if gap <= 3 {
return 1.0 return 1.0
} }
hasHigherAccessible := false
for _, other := range allAdvLocations(loc.Activity) {
if other.MinLevel > loc.MinLevel && other.MinLevel <= effectiveLevel {
hasHigherAccessible = true
break
}
}
if !hasHigherAccessible {
return 1.0
}
mult := 1.0 - 0.15*float64(gap-3) mult := 1.0 - 0.15*float64(gap-3)
return math.Max(0.05, mult) return math.Max(0.05, mult)
} }
@@ -644,7 +655,7 @@ func resolveAdvAction(char *AdventureCharacter, equip map[EquipmentSlot]*AdvEqui
// Overlevel penalty — reduces loot and XP for farming low-tier content // Overlevel penalty — reduces loot and XP for farming low-tier content
skillLevel := advEffectiveSkill(char, loc.Activity, bonuses) skillLevel := advEffectiveSkill(char, loc.Activity, bonuses)
overlevelMult := advOverlevelMultiplier(skillLevel, loc.MinLevel) overlevelMult := advOverlevelMultiplier(skillLevel, loc)
// Roll outcome // Roll outcome
roll := rand.Float64() * 100 roll := rand.Float64() * 100

2
internal/plugin/combat_bridge.go
View File

@@ -524,7 +524,7 @@ func (p *AdventurePlugin) resolveDungeonAction(
// Overlevel penalty // Overlevel penalty
skillLevel := advEffectiveSkill(char, loc.Activity, bonuses) skillLevel := advEffectiveSkill(char, loc.Activity, bonuses)
overlevelMult := advOverlevelMultiplier(skillLevel, loc.MinLevel) overlevelMult := advOverlevelMultiplier(skillLevel, loc)
// Loot on success/exceptional // Loot on success/exceptional
if result.Outcome == AdvOutcomeSuccess || result.Outcome == AdvOutcomeExceptional { if result.Outcome == AdvOutcomeSuccess || result.Outcome == AdvOutcomeExceptional {