fix(players): invalidate all player queries on create/delete

useCreatePlayer and useDeletePlayer were only invalidating
playerKeys.lists() which doesn't include listWithSkillData().

The practice page uses usePlayersWithSkillData() so newly created
players weren't appearing until page reload.

Now invalidates playerKeys.all to catch all player-related queries.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

apps/web/src/hooks/useUserPlayers.ts

@@ -176,7 +176,8 @@ export function useCreatePlayer() {
     },
     onSettled: () => {
       // Always refetch after error or success
-      queryClient.invalidateQueries({ queryKey: playerKeys.lists() })
+      // Invalidate ALL player queries (including listWithSkillData used by practice page)
+      queryClient.invalidateQueries({ queryKey: playerKeys.all })
     },
   })
 }
@@ -276,7 +277,8 @@ export function useDeletePlayer() {
     },
     onSettled: () => {
       // Refetch after error or success
-      queryClient.invalidateQueries({ queryKey: playerKeys.lists() })
+      // Invalidate ALL player queries (including listWithSkillData used by practice page)
+      queryClient.invalidateQueries({ queryKey: playerKeys.all })
     },
   })
 }