Add missing drop tables and combat definitions

[HarleyGilpin]

What changed?

• Closes the gap between attackable NPCs and their config coverage: drop tables first, then combat definitions and stats.
• NPCs that are not currently spawned in-game have been untouched, we will add them later as we implement them.

Drop tables

Before this branch, 577 spawned NPCs (938 total) resolved no drop table at kill time and died to nothing; ~30 NPC families with tables were missing their charm drops.

• Charm sub-tables for ~35 families (goblin, cow, skeleton, zombie, hill giant, hobgoblin, guards, demons, KBD, gargoyle, nechryael, tzhaar, wolves, spiders...). Rates computed from runescape.wiki charm log data (Module:Data/<npc>) at 1/100000 precision, matching the existing bat/scorpion table format; cave slime, outlaw, rockslug, unicorn and tzhaar fall back to rounded community rates where no charm log exists. KBD drops charms four at a time (amount = 4).
• 124 new drop tables generated from RuneScape wiki page revisions dated Jan–Mar 2011 (era-accurate for rev 634). Rarity words map to: very common ~1/5, common ~1/16, uncommon ~1/32, rare ~1/171, very rare ~1/512 on roll = 1024.
• 577 NPC defs wired to tables: 169 to existing tables they should have pointed at (e.g. green_dragon_1-4 dropped nothing because clones don't inherit the implicit string-id table match), 19 to bones, 63 to empty for NPCs that legitimately drop nothing.
• Removes hobgoblins.drops.toml, a stale duplicate of hobgoblin/hobgoblin.drops.toml.

Combat definitions & stats

An NPC whose combat_def doesn't resolve never swings, never retaliates, and plays no defend/death animation (Attack.kt:52, Combat.kt:146). 681 spawned NPCs were in that state; 304 also had no hitpoints.

• 468 NPCs wired to existing combat definitions by species (Guard variants → guard, skeletons by wielded weapon → skeleton_sword_shield/skeleton_mace_shield, etc.).
• Stats for 336 NPCs (hitpoints, att/str/def/mage/range, attack_bonus, max_hit_<style>, ×10 lifepoints scale) from OSRS wiki Monster Examine data, filtered per-version to monsters released before March 2007 so values match rev 634 (e.g. Bandit's 2002 level-22 version is used, its 2014 level-130 version is rejected). Only absent keys are filled — no existing values touched.
• 106 minimal combat definitions authored from the same data (attack_speed + target_hit with correct offense style and max hit). Attack anims intentionally omitted until cache anim ids are sourced — an omitted anim is a silent no-op, so these NPCs now fight correctly but without bespoke animations.
• Fixes five pre-existing dangling combat_def values (bear_cub, king_scorpion, zombie_dragging, tzhar_ket typo, gardener).
• Second pass for post-2007 content the OSRS wiki can't cover, using runescape.wiki December 2011 revisions (Scabaras dungeon, gnoeals, mutated bloodveld, ancient warriors, warped creatures, cursed ivory bosses): 41 more combat definitions and stats for ~65 NPCs. 2011 infoboxes carry LP/style/speed/max hit directly on the lifepoints scale; att/str/def levels were never published pre-EoC, so they are derived by reversing the standard formulas (rune-server: (503+) NPC Combat Numbers Explained): strength from max hit (P = (max/10 − 0.5)·10 − 9, zero equipment bonus as the 634 cache lacks npc equipment stats), then defence from combat level (DEF = 4·(CB − S) − HP, S = 0.325·(att+str) melee / 0.4875·level ranged/magic). Verified against existing guard data. NPCs with no published max hit use a combined estimate instead.

Aggression & poison (2011 wiki)

194 spawned NPCs marked hunt_mode = "cowardly" (level-checked aggression) per December 2011 wiki infoboxes (only where no hunt_mode existed; explicit disables untouched, conditional aggression skipped). Seven combat definitions gain impact_poison with wiki start damage (ancient mage 88, frog 68, sea snake hatchling 50, spinolyp 20, big snake 10, confusion beast 2), ten poisonous species already had it. Stronghold of Player Safety muggers excluded (not aggressive, source in toml comment). The Kill rat with range test pins npc.huntMode = "" since giant rats are aggressive and now charge the shooter mid-test.

Combat animations & sounds

The combat definitions authored in this branch initially had no attack/defend/death anims. 114 of 149 now have them, 266 animation + 57 sound entries added beside each combat definition, with per-style attack anims where the source distinguishes magic/ranged. The remaining 35 post-2009 definitions were since filled: real anim ids where reference data existed (chaos dwogre, ogre chieftain, soul wars avatars, blood reaver, ancient mage/ranger, luminescent icefiend, Temple Trekking shadows' shared death anim) and species-appropriate reuse of existing void anims for the rest (quest humans → human, elite dark ranger → archer, trolls, rock crab, demon, dagannoth, ghost, scarab). Only baby tanglefoot and the three cursed Ivory zogre bosses remain bare — no source data exists in any reference.

Verification

• Zero unresolved drop_table/combat_def references; combat def loader hard-fails on unknown keys at boot and boot passes.
• WorldTest boot green; combat suites (CombatTest, CombatMovementTest) and engine drop table tests pass.
• Cache audit: spawned NPCs missing drop tables 577 → 0; missing combat definitions 681 → 0; missing hitpoints 304 → 0.

---

Continues #1143 - that PR could not be reopened after the branch was recreated

[GregHib]

Gave it a quick scan and looks good, if there's any issues they can be resolved over time

[HarleyGilpin]

Closes #1141