From 8189f3c643971ddb0f8a53c30b494fd23c5baf61 Mon Sep 17 00:00:00 2001 From: troytt <47798984@qq.com> Date: Tue, 22 Sep 2026 03:44:42 +0000 Subject: [PATCH] feat(skill-090): implement 1.13c IronGolem module and Vitest suite (#244) --- .../skills/impl/nec/skill-090-irongolem.ts | 44 +++++++ tests/skills/nec/skill-090-irongolem.test.ts | 109 ++++++++++++++++++ 2 files changed, 153 insertions(+) create mode 100644 src/game/skills/impl/nec/skill-090-irongolem.ts create mode 100644 tests/skills/nec/skill-090-irongolem.test.ts diff --git a/src/game/skills/impl/nec/skill-090-irongolem.ts b/src/game/skills/impl/nec/skill-090-irongolem.ts new file mode 100644 index 0000000..93d9b78 --- /dev/null +++ b/src/game/skills/impl/nec/skill-090-irongolem.ts @@ -0,0 +1,44 @@ +/** + * Diablo II: Lord of Destruction v1.13c — Skill #090: IronGolem (NEC) + * + * Authoritative 1.13c Ground Truth (`Patch_D2.mpq` -> `data\\global\\excel\\Skills.txt`): + * - Skill ID: 90 ("IronGolem") + * - Class: "nec" + * - Required Level: 24 + * - Server Functions: srvstfunc=20, srvdofunc=57, srvprgfunc1=0, srvprgfunc2=0, srvprgfunc3=0 + * - Associated Missile / State / Summon: missile="", state="thorns", summon="IronGolem" + * - 1.13c Synergy Formulas: EDmgSymPerCalc="", DmgSymPerCalc="", ELenSymPerCalc="" + */ +import type { + SkillEvalContext, + SkillEvalResult, + SkillExecContext, + SkillExecOutcome, + SkillModule, +} from '../../types.ts' +import { evaluateSkillCore113c, executeSkillCore113c } from '../../registry.ts' + +export const skillModule: SkillModule = { + skillId: 90, + name: "IronGolem", + charClass: "nec", + srvStFunc: 20, + srvDoFunc: 57, + + evaluate(ctx: SkillEvalContext): SkillEvalResult { + return evaluateSkillCore113c(ctx) + }, + + executeDo(ctx: SkillExecContext): SkillExecOutcome { + const outcome = executeSkillCore113c(ctx) + return { + ...outcome, + notes: [ + ...outcome.notes, + 'skill_module:skill-090-irongolem', + ], + } + }, +} + +export default skillModule diff --git a/tests/skills/nec/skill-090-irongolem.test.ts b/tests/skills/nec/skill-090-irongolem.test.ts new file mode 100644 index 0000000..9f702aa --- /dev/null +++ b/tests/skills/nec/skill-090-irongolem.test.ts @@ -0,0 +1,109 @@ +/** + * 1.13c Numerical & Runtime Verification Suite — Skill #090: IronGolem (NEC) + * Gitea Tracking Issue: #244 + */ +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/nec/skill-090-irongolem.ts' + +describe('[Skill #090] IronGolem (NEC) — 1.13c Parity (Issue #244)', () => { + 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(90)! + expect(skillRec).toBeDefined() + expect(skillModule.skillId).toBe(90) + expect(skillModule.name).toBe("IronGolem") + expect(skillModule.charClass).toBe("nec") + + const stats1 = new UnitStatList() + stats1.setBaseSkillLevel(90, 1) + const r1 = skillModule.evaluate!({ registry, skill: skillRec, slvl: 1, blvl: 1, statList: stats1 }) + expect(r1.manaCost).toBe(35) + expect(r1.manaCost256).toBe(8960) + expect(r1.minPhysDmg).toBe(0) + expect(r1.maxPhysDmg).toBe(0) + expect(r1.minElemDmg256).toBe(0) + expect(r1.maxElemDmg256).toBe(0) + + const stats10 = new UnitStatList() + stats10.setBaseSkillLevel(90, 10) + const r10 = skillModule.evaluate!({ registry, skill: skillRec, slvl: 10, blvl: 10, statList: stats10 }) + expect(r10.manaCost).toBe(35) + expect(r10.manaCost256).toBe(8960) + expect(r10.minPhysDmg).toBe(0) + expect(r10.maxPhysDmg).toBe(0) + expect(r10.minElemDmg256).toBe(0) + expect(r10.maxElemDmg256).toBe(0) + + const stats20 = new UnitStatList() + stats20.setBaseSkillLevel(90, 20) + const r20 = skillModule.evaluate!({ registry, skill: skillRec, slvl: 20, blvl: 20, statList: stats20 }) + expect(r20.manaCost).toBe(35) + expect(r20.manaCost256).toBe(8960) + expect(r20.minPhysDmg).toBe(0) + expect(r20.maxPhysDmg).toBe(0) + expect(r20.minElemDmg256).toBe(0) + expect(r20.maxElemDmg256).toBe(0) + expect(r20.durationFrames).toBe(0) + expect(r20.radiusSubtiles).toBe(435) + expect(r20.petMax).toBe(1) + expect(r20.cooldownFrames).toBe(0) + }) + + 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(90)! + const synId = null + + const statsBase = new UnitStatList() + statsBase.setBaseSkillLevel(90, 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(0) + + // +20 allskills and +33 Marrowwalk charges without hard points must yield 0 synergy + const statsSoftOnly = new UnitStatList() + statsSoftOnly.setBaseSkillLevel(90, 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: "nec", + skillId: 90, + slvl: 20, + }) + const outcome = arena.triggerCast() + arena.stepTicks(15) + const dbg = arena.getDebugState() + + expect(dbg.ready).toBe(true) + expect(dbg.skillId).toBe(90) + expect(dbg.castCount).toBeGreaterThanOrEqual(1) + expect(dbg.actionFrameTriggered).toBe(true) + expect(outcome?.executed).toBe(true) + }) +})