Skip to content

chore: remove webgl platform support#395

Open
decentraland-bot wants to merge 3 commits into
mainfrom
chore/decommission-webgl
Open

chore: remove webgl platform support#395
decentraland-bot wants to merge 3 commits into
mainfrom
chore/decommission-webgl

Conversation

@decentraland-bot

Copy link
Copy Markdown
Contributor

WebGL asset bundle conversion has been decommissioned. The SQS queue and Fargate converter service have been deleted.

This PR removes all WebGL references from opscli so status commands only show mac and windows.

Changes

  • Remove WEBGL from Platform enum
  • Remove WebGL manifest check from check-ab-conversion
  • Remove WebGL output from pointer-consistency and world-ab-status
  • Update README and index.html

@decentraland-bot

Copy link
Copy Markdown
Contributor Author

Peer review: no blocking findings

I reviewed the WebGL decommission changes in opscli.

P0/P1 findings

None found.

P2 notes

  1. P2 — Document the intentional CLI contract change clearly.

    • Files: src/helpers/asset-bundles.ts:7-10, src/commands/queue-ab-conversion.ts:25-34, README.md
    • Removing Platform.WEBGL means existing invocations with --platform webgl now fail validation, and omitted --platform now means only the remaining supported platforms (windows, mac). That matches the decommission goal, but it should be called out explicitly in README/release notes so operators do not mistake a successful “all platforms” run for including WebGL.
  2. P2 — Add regression coverage for the new platform set.

    • Suggested coverage: omitted --platform enqueues only windows/mac; explicit --platform webgl fails with a clear error; status commands print only Windows/Mac.

Security review

No security issues found. The PR is mostly deletions/reduced platform lists and introduces no secrets, auth/authz changes, injection risks, dependency changes, or sensitive-data exposure.

CI

Passing: build is green.


Reviewed by Jarvis 🤖 · Requested by Charly via Slack

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants