chore: remove sdk-test-scenes.decentraland.zone

[decentraland-bot]

Summary

• Removes all hardcoded references to `sdk-test-scenes.decentraland.zone` — the SDK6 Fargate service being shut down (`decentraland/sdk-test-scenes`, archived 2+ years, no updates)
• Redirects `InitialRealm.TestScenes` to `SDK_TEST_SCENES_URL` (SDK7 CDN, still maintained) to avoid breaking any serialized Unity assets
• Removes the now-dead `"test"` chat teleport shortcut

Changes

File	Change
IRealmNavigator.cs	Remove TEST_SCENES_URL constant
MainSceneLoader.cs	Remove sdk-test-scenes.decentraland.zone from trusted realm allowlist
DynamicSceneLoaderSettings.asset	Remove URL from realm list
ChatTeleporter.cs	Remove "test" shortcut (pointed to dead URL)
RealmUrls.cs	Redirect InitialRealm.TestScenes → SDK_TEST_SCENES_URL
RealmLaunchSettingsDrawer.cs	Remove TestScenes from TEST_REALMS and isolation conditions

Testing

No functional change for users — the removed URL was an SDK6 dev realm that has been offline. The InitialRealm.TestScenes enum value is preserved (redirected) to avoid Unity serialization issues.

Closes

#9569

---

🤖 Created via Slack with Claude
Requested by Mikhail Agapov (U04A7TYN13L)

[github-actions]

Windows and Mac build successful in Unity Cloud! You can find a link to the downloadable artifact below.

Name	Link
Commit	9bc4bf9
Logs	https://github.com/decentraland/unity-explorer/actions/runs/30819380726
Download Windows	https://github.com/decentraland/unity-explorer/suites/83565621495/artifacts/8860078767
Download Windows S3	https://explorer-artifacts.decentraland.org/@dcl/unity-explorer/branch/fix/9569-remove-sdk-test-scenes-url/pr-24463-9bc4bf9/Decentraland_windows64.zip
Download Mac	https://github.com/decentraland/unity-explorer/suites/83565621495/artifacts/8859785776
Download Mac S3	https://explorer-artifacts.decentraland.org/@dcl/unity-explorer/branch/fix/9569-remove-sdk-test-scenes-url/pr-24463-9bc4bf9/Decentraland_macos.zip
Built on	2026-08-03T14:46:34Z

[github-actions]

Warnings not reduced: 13975 => 13975 — remove at least 1 warning to merge.

Warnings/errors in files changed by this PR (6)

Assets/DCL/Chat/Commands/ChatTeleporter.cs:47  InconsistentNaming  Name 'realmURL' does not match rule 'non_public_members_should_be_camel_case'. Suggested name is 'realmUrl'.
Assets/DCL/Chat/Commands/ChatTeleporter.cs:87  InconsistentNaming  Name 'realmURL' does not match rule 'non_public_members_should_be_camel_case'. Suggested name is 'realmUrl'.
Assets/DCL/Chat/Commands/ChatTeleporter.cs:114  InconsistentNaming  Name 'realmURL' does not match rule 'parameters_should_be_camel_case'. Suggested name is 'realmUrl'.
Assets/DCL/Chat/Commands/ChatTeleporter.cs:128  InconsistentNaming  Name 'realmURL' does not match rule 'parameters_should_be_camel_case'. Suggested name is 'realmUrl'.
Assets/DCL/Chat/Commands/ChatTeleporter.cs:95  RedundantArgumentDefaultValue  The parameter 'allowsSpawnPointerOverride' has the same default value
Assets/DCL/Infrastructure/Global/Editor/RealmLaunchSettingsDrawer.cs:248  RedundantAssignment  Value assigned is not used in any execution path

[github-actions]

Lint in progress, come back later!

[github-actions]

All Unity tests passed ✅

TESTS SUITE	Result	Passed	Failed	Skipped
EditMode	✅ Passed	24332	0	13
PlayMode	✅ Passed	236	0	5

[decentraland-bot]

STEP 1 — Context & scope

• Loaded CLAUDE.md, docs/README.md, and the repo review instructions.
• Checked out PR head fix/9569-remove-sdk-test-scenes-url against dev.
• Reviewed changed files and surrounding realm/navigation/editor drawer code.
• Repo search evidence: InitialRealm.TestScenes has 0 matches; TestScenes has 0 matches; sdk-test-scenes.decentraland.zone still has 1 match in Explorer/Assets/Scenes/LocomotionTestScene.unity:3929.

STEP 2 — Root-cause check

• Problem: remove dependency on the archived SDK6 sdk-test-scenes.decentraland.zone Fargate and remove the obsolete InitialRealm.TestScenes enum value.
• Result: FAIL — the C# enum/constant/switch cleanup addresses most references, but the deprecated URL remains serialized in a Unity scene, so the diff does not fully remove the dependency.

STEP 3 — Design & integration

• No new long-lived unit, system, manager, service, controller, subscription, connection, room, buffer, or measurement is introduced; owner search is not applicable.
• Existing owners reviewed: realm startup mapping (RealmUrls.cs), launch settings serialization (RealmLaunchSettings.cs/InitialRealm.cs), editor selection (RealmLaunchSettingsDrawer.cs), trusted realm validation (MainSceneLoader.cs), chat aliases (ChatTeleporter.cs), and serialized realm lists (DynamicSceneLoaderSettings.asset, scene assets).
• Integration gap: serialized Unity scene data bypasses the IRealmNavigator constants, and LocomotionTestScene.unity still contains the old endpoint.

STEP 4 — Member audit

• Changed public member: InitialRealm enum removed TestScenes and keeps World = 5 to preserve serialized numeric values. Consumers found via rg: RealmLaunchSettings, RealmUrls, RealmLaunchSettingsDrawer, and edit-mode tests. No remaining InitialRealm.TestScenes consumers.
• Changed public constant: IRealmNavigator.TEST_SCENES_URL removed. Consumers found via rg: no remaining C# consumers.

STEP 5 — Findings

[P1] Explorer/Assets/Scenes/LocomotionTestScene.unity:3929 still contains the deprecated endpoint.

Problem: the PR removes the URL from C# and DynamicSceneLoaderSettings.asset, but the same URL remains in serialized scene data:

- https://sdk-test-scenes.decentraland.zone

Fix: replace it with the maintained SDK7 test-scenes CDN URL, or remove the entry if LocomotionTestScene should no longer expose SDK test scenes. Concrete suggested replacement:

  - https://sdk-team-cdn.decentraland.org/ipfs/sdk7-test-scenes-main-latest

Why: after the Fargate shutdown, opening/running this test scene can still point developers/QA at the removed service, so the old dependency is not fully eliminated.

Security review: No security issues found. The PR does not add secrets, auth changes, or broaden URL trust; it reduces stale endpoint exposure, except for the remaining serialized scene reference above.

Consumer impact analysis: IRealmNavigator.TEST_SCENES_URL is a public constant in the Unity project, but repo-wide search found no remaining consumers. No external published package/API consumer was identified for this Unity-internal constant.

STEP 6 — Complexity assessment

• COMPLEX: the diff touches 7 files, runtime realm/navigation configuration, a shared interface constant, editor launch settings, and serialized Unity asset data.

STEP 7 — QA assessment

• QA_REQUIRED: YES because runtime code under Explorer/ and realm/navigation behavior are touched.

STEP 8 — Non-blocking warnings

• Main scene was not modified.
• CI is not green yet: semantic title check is failing, QA/DEV approval is pending, and Unity build/test/lint jobs are still pending.

STEP 9 — Verdict
REVIEW_RESULT: FAIL ❌
COMPLEXITY: COMPLEX
COMPLEXITY_REASON: Touches runtime realm/navigation configuration, a shared interface constant, editor launch settings, and serialized Unity asset data across more than three files.
QA_REQUIRED: YES

---

Reviewed by Jarvis 🤖 · Requested by Mikhail Agapov (<@U04A7TYN13L>) via Slack