decentraland / asset-bundle-registry / 28173690751

72%

Build Type

push

github

Committed by web-flow

Commit Message

chore: remove webgl platform support (#127)

* chore: remove webgl platform from asset bundle registry

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

* style: format platform guard imports

* fix: keep webgl fields in API responses for backward compatibility

WebGL was decommissioned internally, but dropping it from the API
responses is a breaking change for existing consumers. Re-add the
`webgl` fields at the response boundary with default values:

- /entities/active and /entities/versions: inject `webgl` into bundles
  and versions, preserving historical values for already-converted
  entities and defaulting to `pending` / empty version otherwise
- /queues/status: always return `webglPendingJobs` as an empty array

The fields are documented as deprecated in the OpenAPI spec. The
internal model, processing pipeline and DB writes remain free of WebGL.

* refactor: centralize platform typing on SupportedPlatform

Address review feedback on the WebGL decommission:

- textures-handler: use the guard-narrowed `eventMetadata.platform`
  instead of an `as keyof Registry.Bundles['assets']` cast, tying the
  type to the runtime guard and removing a latent mismatch risk
- replace `platform: string` and inline 'windows' | 'mac' unions with
  SupportedPlatform across the db adapter, registry component, queue
  manager, entity-status-fetcher and the status/textures handlers
- add tests asserting stale `webgl` events are dropped gracefully by
  both the status and textures handlers
- delete the unused key-generator util (dead duplicate of the queue
  manager's private cache-key helper)

---------

Co-authored-by: LautaroPetaccio <lausuper@gmail.com>

Coverage Stats

637 of 928 branches covered (68.64%)

Branch coverage included in aggregate %.

38 of 43 new or added lines in 10 files covered. (88.37%)

1 existing line in 1 file now uncovered.

1663 of 2272 relevant lines covered (73.2%)

40.26 hits per line