✓ Accepted answer
what you have is probably ok if you’re just looking for one thing every one second, but the most optimal way to do these kinds of things is with a signal. a signal meaning, using Connect(). with connections you can forget about constant checks, the connect will do it for you without loops, which can save a lot of lag.
it looks like you’re using an Instance. luckily, roblox has a handy-dandy Instance.ChildAdded event. you can tap into that. if you’re only doing it once, you could even use Instance.ChildAdded:Wait().