fix: add runtime guards for stale webgl events and fix type errors

Author: decentraland-bot

docs/openapi.yaml

@@ -660,7 +660,10 @@ components:
         - assetBundles
     PlatformStatus:
       type: object
-      description: Conversion status per platform (Windows and Mac)
+      description: |
+        Conversion status per platform (Windows and Mac).
+        Note: WebGL platform support was removed as part of the WebGL asset bundle decommission.
+        Only Windows and Mac platforms are supported going forward.
       properties:
         mac:
           type: string
@@ -754,7 +757,9 @@ components:
         - lods
     BundlePlatformStatus:
       type: object
-      description: Conversion status for each platform
+      description: |
+        Conversion status for each platform.
+        Note: WebGL platform was removed as part of the WebGL asset bundle decommission.
       properties:
         windows:
           type: string

src/logic/handlers/status-handler.ts

@@ -1,5 +1,5 @@
 import { DeploymentToSqs } from '@dcl/schemas/dist/misc/deployments-to-sqs'
-import { AppComponents, IEventHandlerComponent, EventHandlerName, EventHandlerResult } from '../../types'
+import { AppComponents, IEventHandlerComponent, EventHandlerName, EventHandlerResult, isSupportedPlatform } from '../../types'
 import { AssetBundleConversionManuallyQueuedEvent, Events } from '@dcl/schemas'
 
 export const createStatusEventHandler = ({
@@ -19,8 +19,12 @@ export const createStatusEventHandler = ({
     if (event.type === Events.Type.ASSET_BUNDLE && event.subType === Events.SubType.AssetBundle.MANUALLY_QUEUED) {
       const { metadata } = event as AssetBundleConversionManuallyQueuedEvent
 
+      if (!isSupportedPlatform(metadata.platform)) {
+        return { entityId: '', platforms: [], isLods: false }
+      }
+
       entityId = metadata.entityId
-      platforms.push(metadata.platform as 'windows' | 'mac')
+      platforms.push(metadata.platform)
       isLods = metadata.isLods
     } else {
       const deploymentEvent = event as DeploymentToSqs
@@ -42,6 +46,11 @@ export const createStatusEventHandler = ({
       try {
         const { entityId, platforms, isLods } = getEventProperties(event)
 
+        if (platforms.length === 0) {
+          logger.warn('Ignoring event for unsupported platform')
+          return { ok: true, handlerName: HANDLER_NAME }
+        }
+
         if (isLods) {
           logger.info('Skipping processing status for LODs', { entityId, platforms: platforms.join(', ') })
           return { ok: true, handlerName: HANDLER_NAME }

src/logic/handlers/textures-handler.ts

@@ -1,5 +1,5 @@
 import { AssetBundleConversionFinishedEvent, Entity } from '@dcl/schemas'
-import { AppComponents, IEventHandlerComponent, EventHandlerName, EventHandlerResult, Registry } from '../../types'
+import { AppComponents, IEventHandlerComponent, EventHandlerName, EventHandlerResult, Registry, isSupportedPlatform } from '../../types'
 import { ManifestStatusCode } from '../entity-status-fetcher'
 
 export const createTexturesEventHandler = ({
@@ -21,6 +21,15 @@ export const createTexturesEventHandler = ({
     handle: async (event: AssetBundleConversionFinishedEvent): Promise<EventHandlerResult> => {
       try {
         const eventMetadata = event.metadata
+
+        if (!isSupportedPlatform(eventMetadata.platform)) {
+          logger.warn('Ignoring event for unsupported platform', {
+            entityId: eventMetadata.entityId,
+            platform: eventMetadata.platform
+          })
+          return { ok: true, handlerName: HANDLER_NAME }
+        }
+
         let entity: Registry.DbEntity | null = await db.getRegistryById(eventMetadata.entityId)
 
         // Track if the entity was originally OBSOLETE to preserve its status later

src/types/types.ts

@@ -151,3 +151,15 @@ export type MessageProcessorResult = {
   ok: boolean
   failedHandlers: EventHandlerName[]
 }
+
+/**
+ * Platforms supported for asset bundle conversion.
+ * WebGL was decommissioned — stale events with platform 'webgl' must be
+ * filtered out at the handler entry point.
+ */
+export const SUPPORTED_PLATFORMS = ['windows', 'mac'] as const
+export type SupportedPlatform = (typeof SUPPORTED_PLATFORMS)[number]
+
+export function isSupportedPlatform(p: string): p is SupportedPlatform {
+  return (SUPPORTED_PLATFORMS as readonly string[]).includes(p)
+}