[FIX] Fix ScriptComponent clone not handling awaiting scripts

[willeastcott]

Description

When cloning an entity with a ScriptComponent, scripts in an "awaiting" state were not being properly included in the clone's script order.

Bug

The cloneComponent method was iterating over _scriptsIndex using for...in, but incorrectly checking properties on the string key instead of the script entry object:

for (const key in entity.script._scriptsIndex) {
    if (key.awaiting) {  // ❌ key is a string, key.awaiting is always undefined
        order.splice(key.ind, 0, key);
    }
}

Since key is a string (the property name), key.awaiting was always undefined (falsy), making this code block effectively dead code.

Fix

Access the actual script entry object stored at that key:

for (const key in entity.script._scriptsIndex) {
    const scriptEntry = entity.script._scriptsIndex[key];
    if (scriptEntry.awaiting) {  // ✅ correctly checks the script entry
        order.splice(scriptEntry.ind, 0, key);
    }
}

Fixes #2796

Checklist

• I have read the contributing guidelines
• My code follows the project's coding standards
• This PR focuses on a single change

[Copilot]

Pull request overview

This PR fixes a bug in the ScriptComponentSystem.cloneComponent method where scripts in an "awaiting" state (scripts that are referenced but not yet registered) were not being properly included in the cloned entity's script order due to incorrectly accessing properties on a string key instead of the script entry object.

• Fixed the for...in loop to correctly retrieve and check the script entry object instead of checking properties on the string key
• Now properly handles awaiting scripts by accessing scriptEntry.awaiting and scriptEntry.ind instead of key.awaiting and key.ind

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The fix correctly handles awaiting scripts during cloning, but there is no test coverage for this scenario. Consider adding a test that verifies cloning an entity with a ScriptComponent that has scripts in the "awaiting" state (i.e., scripts that are referenced but not yet registered). This would ensure the fix works as intended and prevent future regressions.