Archivable property

All instances have an Archivable property. This determines if an object can be cloned, or saved

By default roblox characters have this set to false and when you try cloning without setting it to true, instance.Clone will return nil.

Also your script has lots of "fluff", lots of what you are doing is redundant.

ds is a misleading variable name, it does not clearly show its purpose in the code. character was one of your variables but you never used it.

What you can do is something like this:

local button = script.Parent;
local client = game:GetService("Players").LocalPlayer; --// Get services using :GetService(serviceName)
local character = client.Character or client.CharacterAdded:Wait(); --// You have the order wrong.
character.Archivable = true;
 
--[[ 
	/*
	Check for character first. 
	If it exists, assign the character variable to the character model. 
	If not, wait for the character. Yours is waiting first when it is possible they loaded in already.
	*/
--]]

button.Activated:Connect(function() --// Activated works for other devices too
	local clone = character:Clone();
	clone:SetPrimaryPartCFrame(character.HumanoidRootPart.CFrame));
end);

There is no point in multiplying it by CFrame.new(0, 0, 0). It won't do anything.

Finally, please, please, please indent. Separating each statement with a line of whitespace is not a substitute for indenting properly and doesn't make the code any more readable.

If indenting is truly hard for you, I have made a tutorial which covers when and when not to indent. You can find it here.

I hope I helped you, and if I did, then don't forget to hit that "Accept Answer" button. It helps the both of us out.