Populating PlayerGui via LocalScript in StarterPlayerScripts?

What would be the proper way to go about doing this; because I have a feeling this is bad practice.

sidenote: The goal of this script is basically to add a new frame inside a Gui for every weapon in the players backpack. And don't mind all the prints, I tried using them to debug

Edit: I was originally having a problem with this code not working on a server but I've since fixed that issue

local replicatedStorage = game:GetService("ReplicatedStorage")
local equipToolEvent = replicatedStorage:FindFirstChild("EquipToolEvent")
local unequipToolsEvent = replicatedStorage:WaitForChild("UnequipToolsEvent")

local player = game.Players.LocalPlayer
player:WaitForChild("Backpack")
local character = player.Character or player.CharacterAdded:wait()

function updateWeapons(tool)
    wait()
    local weapons = player.PlayerGui.InventoryGui.Inventory.Weapons
    local template = weapons:FindFirstChild("WeaponTemplate")

    clearInventory(weapons)

    local weaponTable = getWeapons(tool)

    for index, weapon in pairs(weaponTable) do
        local weaponFrame = template:Clone()
        if weapon.Values.Equipped.Value then
            weaponFrame.EquipButton.Visible = false
            weaponFrame.UnequipButton.Visible = true
        end
        weaponFrame.Name = weapon.Name
        weaponFrame.WeaponNameLabel.Text = weapon.Name
        weaponFrame.DamageLabel.Text = "Damage: " .. weapon.Values.Damage.Value
        weaponFrame.Position = weaponFrame.Position + UDim2.new(0, 0, 0, (index - 1) * 30)
        weaponFrame.Parent = weapons
        weaponFrame.Visible = true
    end
end

------------------------------------------------------------

function clearInventory(weapons)
    if weapons then
        for _, weapon in pairs(weapons:GetChildren()) do
            if weapon.Name ~= "WeaponTemplate" then
                weapon:Destroy()
            end
            print("cleared inventory")
        end
    else
        print("!weapons clear")
    end
end

------------------------------------------------------------

function getWeapons(tool)
    local backpack = player.Backpack
    if backpack then
        local weaponTable = {}
        if tool then
            print("tool")
            table.insert(weaponTable, tool)
        else
            print("!tool")
        end
        for _, weapon in pairs(backpack:GetChildren()) do
            print("inserting " .. weapon.Name)
            table.insert(weaponTable, weapon)
        end
        return weaponTable
    else
        print("!backpack")
    end
end

------------------------------------------------------------

updateWeapons()
equipToolEvent.OnClientEvent:Connect(function(tool) updateWeapons(tool) end)
unequipToolsEvent.OnClientEvent:Connect(function(tool) updateWeapons(tool) end)