refactor: rename HasDepsDigests to HasCanonicalAssets

The flag signals what the converter actually guarantees: only scenes
fetch files[], and only scene bundles upload to the canonical assets/
prefix (asset-reuse is gated on entity.type === 'scene'). The name now
says so, and the flag fires for any injected files[] — a reuse-converted
scene can legitimately list only 2-part names and still live under
assets/.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Author: dalkia

Explorer/Assets/DCL/Infrastructure/ECS/Unity/StreamableLoading/AssetBundles/PrepareAssetBundleLoadingParametersSystemBase.cs

@@ -83,10 +83,10 @@ private URLAddress GetAssetBundleURL(AssetBundleManifestVersion manifest, string
         {
             string version = manifest.GetAssetBundleManifestVersion();
 
-            // Digest-mapped bundles live under the canonical assets/ prefix (no entity segment) — requesting it
-            // directly skips the edge rewrite. Manifests without the map (wearables/emotes, pre-v49 scenes) carry
-            // digest-less hashes that only resolve through the entity path, so they keep the legacy shapes.
-            if (manifest.HasDepsDigests())
+            // Canonical-assets bundles live under the assets/ prefix (no entity segment) — requesting it directly
+            // skips the edge rewrite. Entity-scoped bundles (wearables/emotes, pre-v49 scenes) only resolve through
+            // the entity path, so they keep the legacy shapes.
+            if (manifest.HasCanonicalAssets())
                 return assetBundlesURL.Append(new URLPath($"{version}/assets/{hash}"));
 
             if (manifest.HasHashInPath())

Explorer/Assets/DCL/Infrastructure/ECS/Unity/StreamableLoading/AssetBundles/Tests/EditModeTests/DepsDigestCacheKeyShould.cs

@@ -57,18 +57,19 @@ public void TranslateBareHashCaseInsensitivelyToManifestCasing()
         }
 
         [Test]
-        public void ReportDepsDigestsOnlyWhenMapWasInjected()
+        public void ReportCanonicalAssetsOnlyWhenFilesWereInjected()
         {
-            // The cache-key dispatch hinges on this: only scene manifests receive files[]; wearables/emotes must keep buildDate keying.
-            var withoutMap = AssetBundleManifestVersion.CreateFromFallback("v49", "2026-05-01");
-            Assert.That(withoutMap.HasDepsDigests(), Is.False);
+            // The URL shape and cache-key dispatch hinge on this: only scenes fetch files[], and only scene bundles
+            // are stored under the canonical assets/ prefix; wearables/emotes must keep entity-path URLs and buildDate keying.
+            var withoutFiles = AssetBundleManifestVersion.CreateFromFallback("v49", "2026-05-01");
+            Assert.That(withoutFiles.HasCanonicalAssets(), Is.False);
 
-            AssetBundleManifestVersion withMap = CreateV49Manifest($"{HASH_A}_{DIGEST_A}_mac");
-            Assert.That(withMap.HasDepsDigests(), Is.True);
+            AssetBundleManifestVersion withDigestFiles = CreateV49Manifest($"{HASH_A}_{DIGEST_A}_mac");
+            Assert.That(withDigestFiles.HasCanonicalAssets(), Is.True);
 
-            // A manifest whose files[] contained only legacy 2-part names carries no map either.
+            // Any injected files[] counts — a reuse-converted scene can legitimately list only 2-part names.
             AssetBundleManifestVersion onlyLegacyFiles = CreateV49Manifest($"{HASH_LEGACY}_mac");
-            Assert.That(onlyLegacyFiles.HasDepsDigests(), Is.False);
+            Assert.That(onlyLegacyFiles.HasCanonicalAssets(), Is.True);
         }
 
         [Test]
@@ -107,7 +108,7 @@ public void ResolveQmContentCasingThroughTheSameMap()
             manifest.InjectContent("Qmf7DaJZRygoayfNn5Jq6QAykrhFpQUr2us2VFvjREiajk",
                 new[] { new ContentDefinition { file = "model.glb", hash = "QmaBrb8WisG9b4Szzt6ACHgaJdyULTEjpzmTwDi4RCEtZV" } });
 
-            Assert.That(manifest.HasDepsDigests(), Is.False, "Content casing entries must not switch cache keying to the v49 scheme");
+            Assert.That(manifest.HasCanonicalAssets(), Is.False, "Content casing entries must not switch the URL shape or cache keying to the canonical scheme");
 
             Assert.That(manifest.GetCdnRequestHash("QmaBrb8WisG9b4Szzt6ACHgaJdyULTEjpzmTwDi4RCEtZV"),
                 Is.EqualTo($"qmabrb8wisg9b4szzt6achgajdyultejpzmtwdi4rcetzv{platform}").IgnoreCase);

Explorer/Assets/DCL/NetworkDefinitions/AssetBundleManifestVersion.cs

@@ -36,7 +36,7 @@ public class AssetBundleManifestVersion
 
         //Bare hash → canonical CDN file name; fed by InjectDepsDigests (digest-bearing names) and InjectContent (Qm casing fixes).
         private Dictionary<string, string>? cdnFiles;
-        private bool hasDepsDigests;
+        private bool hasCanonicalAssets;
 
         public bool HasHashInPath()
         {
@@ -76,6 +76,7 @@ private static bool TryParseVersionNumber(string? version, out int parsed)
         public void InjectDepsDigests(string[]? files)
         {
             if (files == null || files.Length == 0) return;
+            hasCanonicalAssets = true;
 
             foreach (string file in files)
             {
@@ -86,13 +87,12 @@ public void InjectDepsDigests(string[]? files)
 
                 cdnFiles ??= new Dictionary<string, string>(new UrlHashComparer());
                 cdnFiles[parts[0]] = file;
-                hasDepsDigests = true;
             }
         }
 
-        /// <summary>True when digest-bearing files were injected — only scene manifests fetch <c>files[]</c>; wearables/emotes never do and keep buildDate cache keying.</summary>
-        public bool HasDepsDigests() =>
-            hasDepsDigests;
+        /// <summary>True when the manifest's <c>files[]</c> were injected — only scenes fetch them, and only scene bundles are stored under the canonical <c>assets/</c> prefix. Wearables/emotes stay entity-scoped and keep buildDate cache keying.</summary>
+        public bool HasCanonicalAssets() =>
+            hasCanonicalAssets;
 
         /// <summary>Translates a bare hash to the hash requested from the CDN: the canonical manifest file name when known (digest-bearing, correctly cased), otherwise the platform-suffixed bare hash.</summary>
         public string GetCdnRequestHash(string bareHash) =>
@@ -102,9 +102,9 @@ public string GetCdnRequestHash(string bareHash) =>
         public string ComposeCacheKey(string hash) =>
             TryGetCdnFileName(hash, out string fileName) ? fileName : hash;
 
-        /// <summary>Computes the Unity-cache key for a CDN request hash: scene manifests (deps map present) key on version+hash — the digest travels inside the hash — while wearables/emotes keep buildDate keying, as their bundles are republished in place.</summary>
+        /// <summary>Computes the Unity-cache key for a CDN request hash: canonical-assets manifests key on version+hash — the digest travels inside the hash — while wearables/emotes keep buildDate keying, as their bundles are republished in place.</summary>
         public Hash128 ComputeCacheHash(string hash) =>
-            HasDepsDigests()
+            HasCanonicalAssets()
                 ? ComputeHashV49(hash, GetAssetBundleManifestVersion()!)
                 : ComputeHashLegacy(hash, GetAssetBundleManifestBuildDate()!);