Problem with these 2 admin commands?

Your problem was with the second command. When you checked to see if the player was an admin, you used "or" without imbedding it in an "and". This means that if the player's name is equal to any name other than the first name statted, it will give them infinite gold anyways. To fix this, just put an open parenthesis after "and" and a closed parentheses before "then". However, we can make the script more efficient using a for loop to check if the player is an admin. Here's the finished script:

First Command:

PlayerAdded:connect(function(player)
PlayerChatted:connect(function(msg)
if msg == ":cmds" then --If you don't want this to be case--sensitive, then you can change this line to *if string.lower(msg) == ":cmds" then*
local clone = script.Parent:Clone()
clone.Parent = player.PlayerGui
end
end)
end)

Second Command:

local admins = {"Player", "Player1", "dragonmaster4111", "LimitedGoneWrong", "groovydino", "aquathorn321"}

game.Players.PlayerAddded:connect(function(player)
for i,v in pairs(admins) do
if v == player.Name then
player.Chatted:connect(function(msg)
if msg ==":InfGold" then --If you don't want this to be case-sensitive, then you can change this line to *if string.lower(msg) == ":infgold" then*
local stats = player:FindFIrstChild("leaderstats") --We don't need if statements checking to see if stats or gold exists, because the code will return an error and stop running on its own.
local gold = stats:FindFirstChild("Gold")
gold.Value == 999999999
end
end
break --end the loop if the player is recognized as an admin
end
end
end)