Fix StarAgentSetting.OnLoaded overwriting user-cleared Services config with example entries

[Copilot]

StarAgentSetting.OnLoaded unconditionally populated 4 example services whenever Services was null or empty, causing every subsequent Save() call (e.g. via WebPanel) to persist those examples — silently overwriting an intentionally empty user config.

Changes

• StarAgent/Setting.cs: Gate example-service population on IsNew so it only runs on first-time config creation, not on every load of an empty/cleared config
• Add Services ??= [] in the else branch to ensure Services is never null after load

// Before
if (Services == null || Services.Length == 0)
{
    Services = [si, si2, si3, si4];
}

// After
if ((Services == null || Services.Length == 0) && IsNew)
{
    Services = [si, si2, si3, si4];
}
else
{
    Services ??= [];
}

First-run behavior (example services as reference) is preserved; existing configs with a cleared <Services> node are left untouched.

[Copilot]

Pull request overview

This PR prevents StarAgentSetting.OnLoaded() from repopulating example Services entries after a user has intentionally cleared the services configuration, avoiding silent reintroduction of sample services on subsequent Save() calls (e.g., via WebPanel).

Changes:

• Gate example service population behind IsNew, so sample services are only added on first-time config creation.
• Ensure Services is never left null after load by initializing it to an empty array when appropriate.

---

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