Started by an SCM change3143a69634a3392781d22b3fcb63ef223b9d139c3df3a69634a3392781d22b3fcb63ef223b9d139c3dfrefs/remotes/origin/master3a69634a3392781d22b3fcb63ef223b9d139c3df3a69634a3392781d22b3fcb63ef223b9d139c3dfrefs/remotes/origin/master3a69634a3392781d22b3fcb63ef223b9d139c3df3a69634a3392781d22b3fcb63ef223b9d139c3dfrefs/remotes/origin/masterhttps://github.com/mcMMO-Dev/mcMMO.githttps://popicraft.net/jenkins/job/mcMMO/314/artifacthttps://popicraft.net/jenkins/job/mcMMO/changeshttps://popicraft.net/jenkins/job/mcMMO/314/https://popicraft.net/jenkins/job/mcMMO/314/testReportfalse#3144966846420mcMMO #314314falsefalse314146FAILURE1780202913343https://popicraft.net/jenkins/job/mcMMO/314/Buildersrc/main/java/com/gmail/nossr50/util/AttributeMapper.javasrc/main/java/com/gmail/nossr50/util/skills/CombatUtils.javaChangelog.txtsrc/test/java/com/gmail/nossr50/util/skills/CombatUtilsTest.java8e1b6deaa5a81fabe92b362e8ef58adfe8e354f81780200740000https://popicraft.net/jenkins/user/nossr50nossr50nossr50@gmail.comFix melee attack strength scale after Paper fixed attack cooldown bug in 26.1.2 Paper 26.1.2 fixed a bug where the player attack cooldown ticker was not resetting at the correct point during melee hits. mcMMO relied on the old (incorrect) order, reading the cooldown during the damage event, which always returned near-zero after Paper's fix. Attack strength scale is now back-derived from the raw event damage divided by the player's GENERIC_ATTACK_DAMAGE attribute value and clamped to [0.0, 1.0]. AttributeMapper now survives test environments where Bukkit registries are absent (catch Throwable in the registry reflection path). Fixes #5287. 2026-05-30 21:12:20 -07008e1b6deaa5a81fabe92b362e8ef58adfe8e354f8Fix melee attack strength scale after Paper fixed attack cooldown bug in 26.1.2editChangelog.txteditsrc/main/java/com/gmail/nossr50/util/AttributeMapper.javaeditsrc/test/java/com/gmail/nossr50/util/skills/CombatUtilsTest.javaeditsrc/main/java/com/gmail/nossr50/util/skills/CombatUtils.javasrc/main/java/com/gmail/nossr50/datatypes/player/PlayerProfile.javasrc/main/java/com/gmail/nossr50/runnables/player/ClearRegisteredXPGainTask.javasrc/main/java/com/gmail/nossr50/datatypes/experience/SkillXpGain.javasrc/main/java/com/gmail/nossr50/datatypes/experience/DiminishedReturnsCache.javasrc/main/java/com/gmail/nossr50/datatypes/experience/DiminishedReturnsState.javasrc/test/java/com/gmail/nossr50/datatypes/experience/DiminishedReturnsCacheTest.java3fb13694441fa58f02f464cebf02cc9c40b10d5d1780200750000https://popicraft.net/jenkins/user/nossr50nossr50nossr50@gmail.comRefactor diminished returns tracking into DiminishedReturnsCache/State Extracts per-player DR data out of PlayerProfile into two new classes: - DiminishedReturnsState: thread-safe XP gain queue and rolling totals - DiminishedReturnsCache: UUID-keyed server-side cache that keeps DR state alive across disconnect/reconnect to prevent window bypass Fixes a bug where evictExpired() would remove fresh states that had never registered any XP, orphaning the PlayerProfile reference and allowing reconnects to bypass the DR window. DR tracking is now skipped entirely (no state mutation) when the diminished returns config option is disabled. Fixes #3475. Fixes #3814. 2026-05-30 21:12:30 -07003fb13694441fa58f02f464cebf02cc9c40b10d5dRefactor diminished returns tracking into DiminishedReturnsCache/Stateeditsrc/main/java/com/gmail/nossr50/runnables/player/ClearRegisteredXPGainTask.javaaddsrc/main/java/com/gmail/nossr50/datatypes/experience/DiminishedReturnsState.javaaddsrc/main/java/com/gmail/nossr50/datatypes/experience/DiminishedReturnsCache.javaaddsrc/test/java/com/gmail/nossr50/datatypes/experience/DiminishedReturnsCacheTest.javaeditsrc/main/java/com/gmail/nossr50/datatypes/player/PlayerProfile.javaeditsrc/main/java/com/gmail/nossr50/datatypes/experience/SkillXpGain.javasrc/test/java/com/gmail/nossr50/database/FlatFileDatabaseManagerTest.java.gitignoresrc/test/java/com/gmail/nossr50/skills/tridents/TridentsTest.javasrc/main/java/com/gmail/nossr50/config/treasure/FishingTreasureConfig.java3a69634a3392781d22b3fcb63ef223b9d139c3df1780202550000https://popicraft.net/jenkins/user/nossr50nossr50nossr50@gmail.comFix pre-existing test failures and improve Docker test skip mechanism - FishingTreasureConfig.fixMooshroomEntityId was a public static method that called mcMMO.p.getLogger() directly, causing NPE when tests invoked it without a live plugin instance. Add a private static Logger field and use it in the static method instead. - TridentsTest.impaleDamageBonusShouldMatchRankOneBaseDamage had a stale expected value (1.0) left over from before commit 71868d9d fixed the Impale formula to use rank * multiplier. The correct expectation for rank 1 is base + 1 * multiplier = 1.5. Rename the test to match the new semantic. - Add @Tag(docker) to FlatFileDatabaseManagerTest so the existing skip-docker-tests Maven profile (-DskipDockerTests=true) excludes it alongside SQLDatabaseManagerTest. - Add generated config file names (experience.yml, config.yml, etc.) to .gitignore to prevent accidental staging when tests run without a properly mocked getDataFolder(). 2026-05-30 21:42:30 -07003a69634a3392781d22b3fcb63ef223b9d139c3dfFix pre-existing test failures and improve Docker test skip mechanismeditsrc/main/java/com/gmail/nossr50/config/treasure/FishingTreasureConfig.javaeditsrc/test/java/com/gmail/nossr50/database/FlatFileDatabaseManagerTest.javaedit.gitignoreeditsrc/test/java/com/gmail/nossr50/skills/tridents/TridentsTest.javagithttps://popicraft.net/jenkins/user/nossr50nossr50nossr50https://popicraft.net/jenkins/user/githubgithubgithub