fix: missing AP champions, localized spell names, ally spell display

- Add Mel, Aurora, Hwei, Zac, Malphite, and other missing AP champions
  to CHAMP_DAMAGE_TYPE so damage composition bar classifies them correctly
- Fix Ignite (and other spells) not rendering for non-English LoL clients
  by matching on internal keys (SummonerDot, etc.) in addition to English
  names — rawDisplayName returns the internal key, not the translation
- Show summoner spells for ally team during live game too, not just enemies
  (ally icons are static; enemy icons remain clickable for cooldown timers)

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: ishtartec

src-tauri/src/lcu.rs

@@ -1788,18 +1788,21 @@ pub fn compute_gold_timeline(
 }
 
 fn spell_name_to_id(name: &str) -> i64 {
+    // Live Client Data returns `rawDisplayName` as the internal key like
+    // "GeneratedTip_SummonerSpell_SummonerDot_DisplayName" (Ignite). Match on
+    // both the internal key and the English display name to survive localization.
     match name {
-        s if s.contains("Flash") => 4,
-        s if s.contains("Ignite") => 14,
-        s if s.contains("Teleport") => 12,
-        s if s.contains("Exhaust") => 3,
-        s if s.contains("Heal") => 7,
-        s if s.contains("Barrier") => 21,
-        s if s.contains("Cleanse") => 1,
-        s if s.contains("Ghost") => 6,
-        s if s.contains("Smite") => 11,
-        s if s.contains("Clarity") => 13,
-        s if s.contains("Mark") => 32, // ARAM snowball
+        s if s.contains("SummonerFlash") || s.contains("Flash") => 4,
+        s if s.contains("SummonerDot") || s.contains("Ignite") => 14,
+        s if s.contains("SummonerTeleport") || s.contains("Teleport") => 12,
+        s if s.contains("SummonerExhaust") || s.contains("Exhaust") => 3,
+        s if s.contains("SummonerHeal") || s.contains("Heal") => 7,
+        s if s.contains("SummonerBarrier") || s.contains("Barrier") => 21,
+        s if s.contains("SummonerBoost") || s.contains("Cleanse") => 1,
+        s if s.contains("SummonerHaste") || s.contains("Ghost") => 6,
+        s if s.contains("SummonerSmite") || s.contains("Smite") => 11,
+        s if s.contains("SummonerMana") || s.contains("Clarity") => 13,
+        s if s.contains("SummonerSnowball") || s.contains("Mark") => 32, // ARAM snowball
         _ => 0,
     }
 }

src/App.tsx

@@ -420,7 +420,7 @@ const TRAIT_SHIELDERS = new Set([
 // Map champion ID to primary damage type: "ap" or "ad"
 const CHAMP_DAMAGE_TYPE: Record<number, "ap" | "ad"> = {};
 // AP champions
-[1,3,4,7,8,9,10,13,17,25,26,27,28,30,31,34,37,38,40,42,43,45,50,55,61,63,68,69,74,76,79,82,84,85,90,96,99,101,103,105,112,113,115,117,127,131,134,142,143,147,150,161,163,245,246,267,268,350,353,360,427,432,497,518,526,555,685,711,876,887,901,902,950].forEach(id => CHAMP_DAMAGE_TYPE[id] = "ap");
+[1,3,4,7,8,9,10,13,16,17,25,26,27,28,30,31,34,37,38,40,42,43,45,50,54,55,57,60,61,63,68,69,74,76,79,82,84,85,90,96,99,101,103,105,112,113,115,117,127,131,134,136,142,143,147,150,154,161,163,245,246,267,268,350,353,360,427,432,497,517,518,526,555,685,711,800,876,887,888,893,901,902,910,950].forEach(id => CHAMP_DAMAGE_TYPE[id] = "ap");
 // AD champions (rest default to AD for simplicity)
 
 interface ItemRec {
@@ -2060,6 +2060,17 @@ function SpellCdIcon({ spellId, playerName, gameTime, spellTimers, onSpellClick
   );
 }
 
+function SpellStaticIcon({ spellId }: { spellId: number }) {
+  const spellKey = SPELL_KEYS[spellId];
+  const spellName = SPELL_NAMES[spellId] || "Spell";
+  if (!spellKey) return <div className="spell-cd-wrap" title={spellName} />;
+  return (
+    <div className="spell-cd-wrap" title={spellName}>
+      <img src={spellIconUrl(spellId)} alt={spellName} className="spell-cd-img" />
+    </div>
+  );
+}
+
 function LiveGamePlayerCard({ p, onViewPlayer, isEnemy, spellCd }: { p: LiveGamePlayer; onViewPlayer?: (puuid: string) => void; isEnemy?: boolean; spellCd?: SpellCdProps }) {
   const totalGames = p.ranked_wins + p.ranked_losses;
   const champWr = p.champ_games > 0 ? (p.champ_wins / p.champ_games * 100) : 0;
@@ -2098,10 +2109,19 @@ function LiveGamePlayerCard({ p, onViewPlayer, isEnemy, spellCd }: { p: LiveGame
           ))}
         </span>
       </div>
-      {isEnemy && live && spellCd && (
+      {live && (live.spell1_id > 0 || live.spell2_id > 0) && (
         <div className="lg-spells">
-          <SpellCdIcon spellId={live.spell1_id} playerName={p.summoner_name} {...spellCd} />
-          <SpellCdIcon spellId={live.spell2_id} playerName={p.summoner_name} {...spellCd} />
+          {isEnemy && spellCd ? (
+            <>
+              <SpellCdIcon spellId={live.spell1_id} playerName={p.summoner_name} {...spellCd} />
+              <SpellCdIcon spellId={live.spell2_id} playerName={p.summoner_name} {...spellCd} />
+            </>
+          ) : (
+            <>
+              <SpellStaticIcon spellId={live.spell1_id} />
+              <SpellStaticIcon spellId={live.spell2_id} />
+            </>
+          )}
         </div>
       )}
       {live ? (