Code used in Button.Run for compatability reasons is weird and evil.

[whoamimaybeme]

when the stuff for v6.1.2 is added, the script adds items to certain tables. good, right?
well
this actually is horrible code, as it adds the item multiple times to the array, which means that this happens

and also this code adds it to a table stored in the Button repository script and not within the run function or something, which means that if an older button is used on this, that means that stuff might change that actually shouldnt!

also this button repo script doesnt do anything to constraints during the versions when it was that specific way, so thats also broken if a newer button is used on the Button repo script!

my proposal to fix this

and then have code that works with this

[whoamimaybeme]

this is here for copy and paste

-- if value is a table, first string = first version it was allowed, second string = first version its disallowed
local COMPATIBILITY_SET_PROPERTIES_ENABLED = {
	Light = "v6.1.2",
	Highlight = "v6.1.2",
	Constraint = {"v6.1.2", "v6.1.3"}
}
local COMPATIBILITY_SET_PROPERTIES_ENABLED_OPTIONAL = {
	Constraint = "v6.1.3"
}
local function getCompatibleProperties(utility: _T.Utility, version: string, type: ("normal" | "optional")?): {string}
	local propertyTable: {[string]: string | {string}} = if type == "optional" 
		then COMPATIBILITY_SET_PROPERTIES_ENABLED_OPTIONAL else COMPATIBILITY_SET_PROPERTIES_ENABLED
	local compare = utility.ClientObjects.isVersionOrHigher
	
	local properties = {}
	for property, propertyver in propertyTable do
		if (typeof(propertyver) == "string" and compare(version,propertyver)) or 
			(typeof(propertyver) == "table" and compare(version, propertyver[1]) and 
				not compare(version, propertyver[2])) then
			table.insert(properties,property)
		end
	end
	
	return properties
end