Why Won't This Script Clone "Fireball" Into PlayerScripts and Backpack?

I've noticed some grammar mistakes, lmao. Maybe some terminology errors too, I hope I helped though

repeat wait()
Player = game.Players.LocalPlayer
Button = script.Parent
Magic = game.ServerScriptService:FindFirstChild("Fireball")
until

Button.MouseButton1Down:connect(function(purchase)
    if 1+1==2 then
        local a = Magic:Clone()
        a.Parent = Player.PlayerScripts
    end
end)

This would not work at all. Before I go on, just a tip of advice:

if 1 + 1 == 2 then

is completely unnecessary...it doesn't do anything either.

Basically you can't do until for until a signal is connected to a function. Yeah, that didn't even make sense or sound right when I said it. Now I'll go on.

repeat wait() 

and until is pretty useless. You can just incorporate these values in the beginning of the script, like so:

local Player = game.Players.LocalPlayer
local Button = script.Parent
local Magic = game.ServerScriptService:FindFirstChild("Fireball")

I mean, there's no point of repeating this until a button is clicked. You could then have a BindableFunction in the players Backpack, so put one in StarterGui if you don't want to use a script to insert it.

local Func = Player.Backpack.Function

Now let's invoke Func when the signal is called (or whatever, I haven't been scripting for a while now, please excuse my vocabulary and terminology if it is incorrect)

Like this:

Button.MouseButton1Down:connect(function(purchase)
    --if 1+1==2 then
        local a = Magic:Clone()
        a.Parent = Player.PlayerScripts 
        Func:Invoke(true)
    -- end | Not needed one bit, if true then could work better, both aren't needed I believe 
end)

Now let's receive the Invoke with a simple function, and then do whatever you wanted after. I assume you wanted to do something after the button was clicked. You tried to make it repeat something (not needed) until the button was clicked, anyways:

I recommend making a "MainFunction" to do whatever you wanted to do after the buttons clicked. Example:

local function MainFunction()
        print(1 + 1)
end

local function isn't needed, don't you dare criticize me on that.

function Func.OnInvoke(whateveryouwanthereitcanbeusefulLater)
             MainFunction()
-- or any code here
end

Simple as that, but there are easier ways to do this like with a value. Now you may be saying what if they press the button more times, no they won't just delete the button or make a boolValue that must be true to OnInvoke and then turn it off after the press.

So all together:

local Player = game.Players.LocalPlayer
local Button = script.Parent
local Magic = game.ServerScriptService:FindFirstChild("Fireball")
local Func = Player.Function

local function MainFunction()
        print(1 + 1)
end

function Func.OnInvoke(whateveryouwanthereitcanbeusefulLater)
             MainFunction()
-- or any code here
end

Button.MouseButton1Down:connect(function(purchase)
    --if 1+1==2 then
        local a = Magic:Clone()
        a.Parent = Player.PlayerScripts 
        Func:Invoke(true)
    -- end | Not needed one bit, if true then could work better, both aren't needed I believe 
end)

I really hope that I helped, if I didn't I at least I hope I educated you on some other stuff in coding

Hope I helped, have a great day. Oh and if my vocabulary is off or if nothing makes sense I'm sorry feel free to make fun of me in comments :(

You seem to be using both local and server only script items.

game.Players.LocalPlayer is only accessible from the client ServerScriptService is only accessible from the server

Magic is probably nil because you're trying to access ServerScriptService from a local script.

If this is a server script, Magic is probably nil because "Fireball" doesn't exist in ServerScriptService at the beginning of the game. Try using :WaitForChild() instead of :FindFirstChild()

Also: Your repeat loop and if statement are both completely unnecessary.