using events for character customisation?

The first problem with this code is that you use local skin = script.Parent.Value.Value this will store the value in the variable named skin which is never updated so the value will be the same.

The second problem is that you set the colours on the client side meaning the changes will not replicate to other players. You will need to use a remote event if this uses user input to update the skin colour.

Youtube vid on remote events wiki info on remote events

All value object have a Changed event which also passes the new value.

Example:-

script.Parent.Value.Changed:Connect(function(newVal)
    -- update body colours
end)

With your current code it may not run when the player spawns. local scripts under the StarterCharacterScripts will be parented to the playe each time the spawn. The alternative is to use the CharacterAdded event (note both methods are with no remote events and will only be visible to the client).

Using lists

As you have a list of possible brick colours using a table with a array of BrickColor is probably a good option as you can edit it easily.

Example:-

local tbl = {
    BrickColor.new("Pastel brown"),
    BrickColor.new("Nougat"),
    BrickColor.new("Dirt brow"),
    BrickColor.new("Tawny"),
}

script.Parent.Value.Changed:Connect(function(newVal)
    if tbl[newVal] then -- bl[newVal] will return nil if index has no value ie no brickcolor
        print("found", tbl[newVal])
        local newBrickColor = tbl[newVal] -- variable for the brick color
        -- set brick color
    else
        print("index not found")
    end
end)

Final code (with no remote events)

local tbl = {
    BrickColor.new("Pastel brown"), -- value 1
    BrickColor.new("Nougat"), -- value 2
    BrickColor.new("Dirt brow"), -- ect...
    BrickColor.new("Tawny"),
}
-- pls use get service as the names can be changed 
local player = game:GetService("Players").LocalPlayer

local function updateBodyColor(val)
    if tbl[newVal] then return end -- not a valid index

    if player.Character then -- if the player has spawned ie not nil
        local bodyColor = player.Character:WaitForChild("Body Colors")
        local brickColor = tbl[newVal] 
        bodyColors.HeadColor = brickColor 
        -- ect....
    end
end


script.Parent.Value.Changed:Connect(function(newVal)
    updateBodyColor(newVal) -- run the local function with the value
end)

-- if the script does not run when the player spawns
player.CharacterAdded:Connect(function(charModel)
    -- you should give the value object a appropriate name Value.Value is confusing
    updateBodyColor(script.Parent.Value.Value) -- run the same function
end)

I hope this helps. If you want this to be visible to all players you should look at how remote events work and move the updateBodyColor function to a server script so that the server can update the bpdy colors.