How do I make a player finder script which uses magnitude to add a object value to a folder?

My code is this the name of the script is player finder

local Attacks = require(script.Parent.Attacks)
game.Players.PlayerAdded:Connect(function(plr)
    wait(3)
    plr.CharacterAdded:Connect(function(char)
        local Torso = char:FindFirstChild("Torso") or char:FindFirstChild("UpperTorso")
        local mag = (Torso.Position - script.Parent.UpperTorso.Position)
        while wait() do
            if mag >= 23 then
                if not Attacks:IsPlayerHere(plr.Name) then
    local obj = Instance.new("ObjectValue")
    obj.Value = plr
    obj.Name = plr.Name --Name doesn't matter
    obj.Parent = script.Parent.Players
    end
    end
    end
    end)
end)

it's supposed to add a value to a folder and the module script has a player checking system and returns true or false

local Attacks = {}
function Attacks:IsPlayerHere(plrinquestion)
if script.Parent.Players:FindFirstChild(plrinquestion) then
return true
else
return false
end

is the loop not working for the magnitude?