Help on this script?

Firstly: prepare yourself for some horrendous scripting that may seriously make you cry, I am quite new. This script is basically meant to make a box (target box) then move the smoke to the target destination. When you press Q, in a tool.

local tool = script.Parent
	local spawned = false
	 game.Players.LocalPlayer:GetMouse().KeyDown:connect(function(key)
    if string.lower(key) ==  "q" then
	   if player.script.Parent.Parent.Parent.Parent.Parent.leaderstats.Lvl.Value>=2 and player.script.Parent.Parent.Mana.Value>=20 then
 if spawned then return end

	    local target = mouse.Target
	    local a = game.Lighting.union:Clone()
	    a.Parent = Workspace
	    a.Size = Vector3.new(3,3,3)
	    a.Anchored = true
	    a.CanCollide = False
	    spawned = true
	    while spawned do
	        wait()
	        a.CFrame = CFrame.new(Vector3.new(tool.Parent.Humanoid.TargetPoint.X,0,tool.Parent.Humanoid.TargetPoint.Z))
	    end
	end
end)
	 
function onButton1Down(mouse)
game.Workspace.Union.BrickColor = BrickColor.new(29)
a.Anchored=False --So it can't move (I think)
local smoke=game.lighting.Smoke:clone()
smoke.parent=Workspace
 game.Workspace.smoke.CFrame = game.Workspace.union.CFrame+ Vector3.new(0, 4, 0)


wait (5)
a:remove()
smoke:remove()
function onEquipped(mouse)
	
function onEquipped(mouse)
	  mouse.Button1Down:connect(function () onButton1Down(mouse) end)
	end
	 

tool.Equipped:connect(onEquipped)
	
	

I probably missed a LOT of ends and I must apologies for my incompetence. Thank you in advance. Please bear with me in your explanations. Just so you know, I have been helped with a script that has inserted two IntValues into the player called Mana, and MAXMana, for your reference it can be found here: https://scriptinghelpers.org/questions/10533/help-on-this-script-with-intvalues

Here's your script, copy this using the 'copy to clipboard' tool you see in the top right corner when hovering your mouse over the script. I don't know if this script works like you want it to, but I cleaned it up a bit. Really, look at how i structured the code and try to use that same structure too. Don't use spacebar in front, just hit tab once. This gives you a much better overview of what you're doing.

local tool = script.Parent
local spawned = false
local player = game.Players.LocalPlayer
local mouse = player:GetMouse()
local a = nil
-- I added some local variables so they're easier to call from within the script

mouse.KeyDown:connect(function(key)
	if string.lower(key) ==  "q" then
		if player.leaderstats.Lvl.Value>=2 and player.Mana.Value>=20 then -- Changed to new variables
			if spawned then return end
	        local target = mouse.Target
	        a = game.Lighting.union:Clone()
	        a.Parent = Workspace
	        a.Size = Vector3.new(3,3,3)
	        a.Anchored = true
	        a.CanCollide = False
	        spawned = true
	        while spawned do
	            wait()
	            a.CFrame = CFrame.new(Vector3.new(tool.Parent.Humanoid.TargetPoint.X,0,tool.Parent.Humanoid.TargetPoint.Z))
	        end
		end -- This 'end' was missing
    end
end)

function onButton1Down(mouse)
	a.BrickColor = BrickColor.new(1022) -- Updated
	a.Anchored = false
	local smoke = game.lighting.Smoke:clone()
	smoke.parent = Workspace
	game.Workspace.smoke.CFrame = game.Workspace.union.CFrame+ Vector3.new(0, 4, 0)
	wait (5)
	a:remove()
	smoke:remove()
end
-- Here was an extra function onEquipped(mouse) that didn't end
function onEquipped(mouse)
      mouse.Button1Down:connect(function () onButton1Down(mouse) end)
end


tool.Equipped:connect(onEquipped(mouse))

Also make sure that you're running this inside a LocalScript. If not

-- Change:
local Player = game.Players.LocalPlayer
-- To:
local Player = game.Players:GetPlayerFromCharacter(script.Parent.Parent)