From 335cd166fea03c9f426756f69ec08e4f58ce1dfc Mon Sep 17 00:00:00 2001 From: troytt <47798984@qq.com> Date: Tue, 22 Sep 2026 03:47:03 +0000 Subject: [PATCH] feat(skill-229): implement 1.13c Molten Boulder module and Vitest suite (#322) --- .../impl/dru/skill-229-molten-boulder.ts | 44 +++++++ .../dru/skill-229-molten-boulder.test.ts | 109 ++++++++++++++++++ 2 files changed, 153 insertions(+) create mode 100644 src/game/skills/impl/dru/skill-229-molten-boulder.ts create mode 100644 tests/skills/dru/skill-229-molten-boulder.test.ts diff --git a/src/game/skills/impl/dru/skill-229-molten-boulder.ts b/src/game/skills/impl/dru/skill-229-molten-boulder.ts new file mode 100644 index 0000000..36d7ae1 --- /dev/null +++ b/src/game/skills/impl/dru/skill-229-molten-boulder.ts @@ -0,0 +1,44 @@ +/** + * Diablo II: Lord of Destruction v1.13c — Skill #229: Molten Boulder (DRU) + * + * Authoritative 1.13c Ground Truth (`Patch_D2.mpq` -> `data\\global\\excel\\Skills.txt`): + * - Skill ID: 229 ("Molten Boulder") + * - Class: "dru" + * - Required Level: 6 + * - Server Functions: srvstfunc=0, srvdofunc=0, srvprgfunc1=0, srvprgfunc2=0, srvprgfunc3=0 + * - Associated Missile / State / Summon: missile="moltenboulderemerge", state="", summon="" + * - 1.13c Synergy Formulas: EDmgSymPerCalc="skill('Firestorm'.blvl)*par8", DmgSymPerCalc="skill('Volcano'.blvl)*par7", ELenSymPerCalc="" + */ +import type { + SkillEvalContext, + SkillEvalResult, + SkillExecContext, + SkillExecOutcome, + SkillModule, +} from '../../types.ts' +import { evaluateSkillCore113c, executeSkillCore113c } from '../../registry.ts' + +export const skillModule: SkillModule = { + skillId: 229, + name: "Molten Boulder", + charClass: "dru", + srvStFunc: 0, + srvDoFunc: 0, + + evaluate(ctx: SkillEvalContext): SkillEvalResult { + return evaluateSkillCore113c(ctx) + }, + + executeDo(ctx: SkillExecContext): SkillExecOutcome { + const outcome = executeSkillCore113c(ctx) + return { + ...outcome, + notes: [ + ...outcome.notes, + 'skill_module:skill-229-molten-boulder', + ], + } + }, +} + +export default skillModule diff --git a/tests/skills/dru/skill-229-molten-boulder.test.ts b/tests/skills/dru/skill-229-molten-boulder.test.ts new file mode 100644 index 0000000..1c7f0d5 --- /dev/null +++ b/tests/skills/dru/skill-229-molten-boulder.test.ts @@ -0,0 +1,109 @@ +/** + * 1.13c Numerical & Runtime Verification Suite — Skill #229: Molten Boulder (DRU) + * Gitea Tracking Issue: #322 + */ +import { describe, expect, it } from 'vitest' +import { getSharedDataRegistry } from '../../../src/game/engine/data-registry.ts' +import { UnitStatList } from '../../../src/game/engine/stat-list.ts' +import { WorldArena } from '../../../src/game/engine/world-arena.ts' +import { skillModule } from '../../../src/game/skills/impl/dru/skill-229-molten-boulder.ts' + +describe('[Skill #229] Molten Boulder (DRU) — 1.13c Parity (Issue #322)', () => { + it('evaluates exact 1.13c Skills.txt / Missiles.txt formulas at slvl 1, 10, and 20', async () => { + const registry = await getSharedDataRegistry() + const skillRec = registry.getSkillById(229)! + expect(skillRec).toBeDefined() + expect(skillModule.skillId).toBe(229) + expect(skillModule.name).toBe("Molten Boulder") + expect(skillModule.charClass).toBe("dru") + + const stats1 = new UnitStatList() + stats1.setBaseSkillLevel(229, 1) + const r1 = skillModule.evaluate!({ registry, skill: skillRec, slvl: 1, blvl: 1, statList: stats1 }) + expect(r1.manaCost).toBe(10) + expect(r1.manaCost256).toBe(2560) + expect(r1.minPhysDmg).toBe(6) + expect(r1.maxPhysDmg).toBe(12) + expect(r1.minElemDmg256).toBe(1536) + expect(r1.maxElemDmg256).toBe(3072) + + const stats10 = new UnitStatList() + stats10.setBaseSkillLevel(229, 10) + const r10 = skillModule.evaluate!({ registry, skill: skillRec, slvl: 10, blvl: 10, statList: stats10 }) + expect(r10.manaCost).toBe(14) + expect(r10.manaCost256).toBe(3712) + expect(r10.minPhysDmg).toBe(48) + expect(r10.maxPhysDmg).toBe(63) + expect(r10.minElemDmg256).toBe(12288) + expect(r10.maxElemDmg256).toBe(16128) + + const stats20 = new UnitStatList() + stats20.setBaseSkillLevel(229, 20) + const r20 = skillModule.evaluate!({ registry, skill: skillRec, slvl: 20, blvl: 20, statList: stats20 }) + expect(r20.manaCost).toBe(19) + expect(r20.manaCost256).toBe(4992) + expect(r20.minPhysDmg).toBe(130) + expect(r20.maxPhysDmg).toBe(155) + expect(r20.minElemDmg256).toBe(33280) + expect(r20.maxElemDmg256).toBe(39680) + expect(r20.durationFrames).toBe(0) + expect(r20.radiusSubtiles).toBe(7) + expect(r20.petMax).toBe(0) + expect(r20.cooldownFrames).toBe(50) + }) + + it('enforces 1.13c blvl synergy rule (hard points boost synergy; +skills and item charges do NOT)', async () => { + const registry = await getSharedDataRegistry() + const skillRec = registry.getSkillById(229)! + const synId = 225 + + const statsBase = new UnitStatList() + statsBase.setBaseSkillLevel(229, 20) + if (synId !== null) { + statsBase.setBaseSkillLevel(synId, 20) + } + const withHardPoints = skillModule.evaluate!({ + registry, + skill: skillRec, + slvl: 20, + blvl: 20, + statList: statsBase, + }) + expect(withHardPoints.synergyBonusPct).toBe(160) + + // +20 allskills and +33 Marrowwalk charges without hard points must yield 0 synergy + const statsSoftOnly = new UnitStatList() + statsSoftOnly.setBaseSkillLevel(229, 20) + statsSoftOnly.addStat('item_allskills', 20) + if (synId !== null) { + statsSoftOnly.setChargedSkillLevel(synId, 33) + } + const withSoftOnly = skillModule.evaluate!({ + registry, + skill: skillRec, + slvl: 20, + blvl: 20, + statList: statsSoftOnly, + }) + expect(withSoftOnly.synergyBonusPct).toBe(0) + }) + + it('executes runtime skill effect in WorldArena (castCount, action frame, and combat/state/missile/summon outcome)', async () => { + const registry = await getSharedDataRegistry() + const arena = new WorldArena({ + registry, + classCode: "dru", + skillId: 229, + slvl: 20, + }) + const outcome = arena.triggerCast() + arena.stepTicks(15) + const dbg = arena.getDebugState() + + expect(dbg.ready).toBe(true) + expect(dbg.skillId).toBe(229) + expect(dbg.castCount).toBeGreaterThanOrEqual(1) + expect(dbg.actionFrameTriggered).toBe(true) + expect(outcome?.executed).toBe(true) + }) +})