updates

frontend/src/components/PlayerModals.jsx

@@ -51,9 +51,9 @@ export const PlayerEditModal = ({
   };
 
   return (
-    <div className="fixed inset-0 z-[150] flex items-center justify-center bg-black/80 backdrop-blur-sm p-4">
-      <div className="bg-slate-950 border border-slate-800 w-full max-w-2xl rounded-2xl shadow-2xl overflow-hidden animate-in zoom-in-95 duration-200 flex flex-col">
-        <div className="bg-slate-900 p-5 flex justify-between items-center border-b border-slate-800">
+    <div className="fixed inset-0 z-[150] flex items-center justify-center bg-black/80 backdrop-blur-sm p-2 sm:p-4">
+      <div className="bg-slate-950 border border-slate-800 w-full max-w-2xl max-h-[90vh] sm:max-h-none overflow-y-auto sm:overflow-visible rounded-xl sm:rounded-2xl shadow-2xl animate-in zoom-in-95 duration-200 flex flex-col">
+        <div className="bg-slate-900 p-4 sm:p-5 flex justify-between items-center border-b border-slate-800 sticky top-0 z-20">
           <div className="flex flex-col">
             <h3 className="font-black text-2xl text-slate-100 uppercase tracking-tight">{livePlayer.Name}</h3>
             <div className="flex gap-3 text-sm font-bold text-slate-500">
@@ -64,7 +64,7 @@ export const PlayerEditModal = ({
           </div>
           <button onClick={() => setSelectedPlayer(null)} className="text-slate-500 hover:text-white transition-colors bg-slate-900 p-2 rounded-full border border-slate-800">✕</button>
         </div>
-        <div className="p-6 flex flex-col gap-6">
+        <div className="p-3 sm:p-6 flex flex-col gap-4 sm:gap-6">
           <div className="flex gap-4">
             {[
               { label: `GW${activeGW} xG`, val: livePlayer[`${activeGW}_xG`] ?? livePlayer.xG ?? "-" },
@@ -206,6 +206,8 @@ export const PlayerSearchModal = ({
   itb,
   handleAddPlayer,
 }) => {
+  const cleanString = (str) => str ? str.normalize("NFD").replace(/[\u0300-\u036f]/g, "").toLowerCase() : "";
+  const cleanSearch = cleanString(searchQuery);
   return (
     <div className="fixed inset-0 z-[150] flex items-center justify-center bg-black/80 backdrop-blur-sm p-4">
       <div className="bg-slate-950 border border-slate-800 w-full max-w-lg rounded-2xl shadow-2xl overflow-hidden flex flex-col animate-in zoom-in-95 duration-200">
@@ -238,10 +240,11 @@ export const PlayerSearchModal = ({
           </button>
         </div>
 
-        <div className="max-h-[400px] overflow-y-auto p-2">
+        {/* THE FIX: Changed max-h to 50vh on mobile so it doesn't span the whole screen */}
+        <div className="max-h-[50vh] sm:max-h-[400px] overflow-y-auto p-1 sm:p-2 custom-scrollbar">
           {globalPlayers
-            // THE FIX: Removed the restrictive 'replacedPlayer' ban and added defensive FPL ID type-checking
-            .filter((p) => !ownedPlayerIds.has(p.ID) && !ownedPlayerIds.has(String(p.ID)) && !ownedPlayerIds.has(Number(p.ID)) && String(p.ID) !== String(selectedPlayer.replacedPlayer?.ID) && p.Pos === selectedPlayer.Pos && p.Name.toLowerCase().includes(searchQuery.toLowerCase()))
+            // THE FIX: Apply cleanString to both the player name and the search query
+            .filter((p) => !ownedPlayerIds.has(p.ID) && !ownedPlayerIds.has(String(p.ID)) && !ownedPlayerIds.has(Number(p.ID)) && String(p.ID) !== String(selectedPlayer.replacedPlayer?.ID) && p.Pos === selectedPlayer.Pos && cleanString(p.Name).includes(cleanSearch))
             .sort((a, b) => {
               let valA = sortConfig.key === "ev" ? Number(a[`${activeGW}_Pts`] || 0) : getPlayerPrice(a);
               let valB = sortConfig.key === "ev" ? Number(b[`${activeGW}_Pts`] || 0) : getPlayerPrice(b);
@@ -251,7 +254,6 @@ export const PlayerSearchModal = ({
             })
             .slice(0, 50)
             .map((p) => {
-              // THE FIX: Your true FPL purchasing power includes the money freed up by selling the outgoing player
               const sellingPrice = getPlayerPrice(selectedPlayer) || 0;
               const maxBudget = itb + sellingPrice;
               const cost = getPlayerPrice(p);
@@ -262,19 +264,21 @@ export const PlayerSearchModal = ({
                   key={p.ID}
                   disabled={!isAffordable}
                   onClick={() => handleAddPlayer(p)}
-                  className={`w-full flex items-center justify-between p-3 border-b border-slate-800/30 transition-colors group ${isAffordable ? "hover:bg-slate-900 cursor-pointer" : "opacity-40 cursor-not-allowed"}`}
+                  // THE FIX: Shrunk padding on mobile (p-2 instead of p-3)
+                  className={`w-full flex items-center justify-between p-2 sm:p-3 border-b border-slate-800/30 transition-colors group ${isAffordable ? "hover:bg-slate-900 cursor-pointer" : "opacity-40 cursor-not-allowed"}`}
                 >
                   <div className="flex flex-col items-start text-left">
-                    <span className="font-bold text-slate-200 text-sm">{p.Name}</span>
-                    <span className="text-[10px] text-slate-500 font-bold uppercase tracking-wider">{p.Team} • {p.Pos}</span>
+                    {/* THE FIX: Shrunk font sizes on mobile */}
+                    <span className="font-bold text-slate-200 text-xs sm:text-sm">{p.Name}</span>
+                    <span className="text-[9px] sm:text-[10px] text-slate-500 font-bold uppercase tracking-wider">{p.Team} • {p.Pos}</span>
                   </div>
-                  <div className="flex items-center gap-4 text-right">
+                  <div className="flex items-center gap-2 sm:gap-4 text-right">
                     <div className="flex flex-col items-end">
-                      <span className="text-xs font-mono text-emerald-400 font-bold">EV: {Number(p[`${activeGW}_Pts`] || 0).toFixed(2)}</span>
-                      <span className="text-[10px] font-mono text-slate-400">{p[`${activeGW}_xMins`] || 0} xMins</span>
+                      <span className="text-[10px] sm:text-xs font-mono text-emerald-400 font-bold">EV: {Number(p[`${activeGW}_Pts`] || 0).toFixed(2)}</span>
+                      <span className="text-[8px] sm:text-[10px] font-mono text-slate-400">{p[`${activeGW}_xMins`] || 0} xMins</span>
                     </div>
-                    <span className={`text-sm font-mono font-bold ${isAffordable ? "text-slate-300" : "text-red-400"}`}>£{cost.toFixed(1)}m</span>
-                    <Plus className={`transition-colors ${isAffordable ? "text-slate-600 group-hover:text-luigi-400" : "text-slate-800"}`} size={18} />
+                    <span className={`text-xs sm:text-sm font-mono font-bold ${isAffordable ? "text-slate-300" : "text-red-400"}`}>£{cost.toFixed(1)}m</span>
+                    <Plus className={`transition-colors ${isAffordable ? "text-slate-600 group-hover:text-luigi-400" : "text-slate-800"}`} size={16} />
                   </div>
                 </button>
               );

solver_engine.py

@@ -6,8 +6,10 @@ def _norm_id_list(raw) -> list[int]:
         return []
     out = []
     for x in raw:
-        if isinstance(x, dict) and "id" in x:
-            out.append(int(x["id"]))
+        # THE FIX: Safely extract the ID from React's dictionary payload
+        if isinstance(x, dict):
+            pid = x.get("ID") or x.get("id", 0)
+            out.append(int(pid))
         else:
             out.append(int(x))
     return out
@@ -110,8 +112,12 @@ def prep_solver_data(payload_data: dict):
         for x in raw:
             if isinstance(x, list) and len(x) == 2:
                 out.append((int(x[0]), int(x[1])))
+            elif isinstance(x, dict):
+                # THE FIX: Extract the ID and strictly bind it to the FIRST gameweek of the horizon!
+                pid = x.get("ID") or x.get("id", 0)
+                out.append((int(pid), horizon_gws[0]))
             else:
-                out.append((int(x), None))
+                out.append((int(x), horizon_gws[0]))
         return out
 
     settings["banned_next_gw"] = norm_temporal_list(