maplibre-gl 2.3.0 → 2.4.0

package/dist/style-spec/index.d.ts

@@ -207,8 +207,6 @@ export declare type ExpressionSpecification = [
 	ExpressionInputType | ExpressionSpecification
 ] | [
 	"coalesce",
-	ExpressionInputType | ExpressionSpecification,
-	ExpressionInputType | ExpressionSpecification,
 	...(ExpressionInputType | ExpressionSpecification)[]
 ] | [
 	"match",
@@ -224,27 +222,20 @@ export declare type ExpressionSpecification = [
 	"interpolate",
 	InterpolationSpecification,
 	number | ExpressionSpecification,
-	number | ExpressionSpecification,
-	ExpressionInputType | ExpressionSpecification,
-	...(number | ExpressionInputType | ExpressionSpecification)[]
+	...(number | number[] | ColorSpecification)[]
 ] | [
 	"interpolate-hcl",
 	InterpolationSpecification,
 	number | ExpressionSpecification,
-	number | ExpressionSpecification,
-	ExpressionInputType | ExpressionSpecification,
-	...(number | ColorSpecification | ExpressionSpecification)[]
+	...(number | ColorSpecification)[]
 ] | [
 	"interpolate-lab",
 	InterpolationSpecification,
 	number | ExpressionSpecification,
-	number | ExpressionSpecification,
-	ExpressionInputType | ExpressionSpecification,
-	...(number | ColorSpecification | ExpressionSpecification)[]
+	...(number | ColorSpecification)[]
 ] | [
 	"step",
 	number | ExpressionSpecification,
-	number | ExpressionSpecification,
 	ExpressionInputType | ExpressionSpecification,
 	...(number | ExpressionInputType | ExpressionSpecification)[]
 ] | [
@@ -257,8 +248,6 @@ export declare type ExpressionSpecification = [
 	string
 ] | [
 	"concat",
-	ExpressionInputType | ExpressionSpecification,
-	ExpressionInputType | ExpressionSpecification,
 	...(ExpressionInputType | ExpressionSpecification)[]
 ] | [
 	"downcase",
@@ -309,8 +298,6 @@ export declare type ExpressionSpecification = [
 	number | ExpressionSpecification
 ] | [
 	"+",
-	number | ExpressionSpecification,
-	number | ExpressionSpecification,
 	...(number | ExpressionSpecification)[]
 ] | [
 	"abs",

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "maplibre-gl",
   "description": "BSD licensed community fork of mapbox-gl, a WebGL interactive maps library",
-  "version": "2.3.0",
+  "version": "2.4.0",
   "main": "dist/maplibre-gl.js",
   "style": "dist/maplibre-gl.css",
   "license": "BSD-3-Clause",
@@ -28,6 +28,7 @@
     "earcut": "^2.2.4",
     "geojson-vt": "^3.2.1",
     "gl-matrix": "^3.4.3",
+    "global-prefix": "^3.0.0",
     "murmurhash-js": "^1.0.0",
     "pbf": "^3.2.1",
     "potpack": "^1.0.2",
@@ -39,7 +40,7 @@
   "devDependencies": {
     "@mapbox/gazetteer": "^5.1.0",
     "@mapbox/mapbox-gl-rtl-text": "^0.2.3",
-    "@mapbox/mvt-fixtures": "^3.9.0",
+    "@mapbox/mvt-fixtures": "^3.10.0",
     "@rollup/plugin-commonjs": "^22.0.2",
     "@rollup/plugin-json": "^4.1.0",
     "@rollup/plugin-node-resolve": "^13.3.0",
@@ -47,15 +48,14 @@
     "@rollup/plugin-strip": "^2.1.0",
     "@rollup/plugin-typescript": "^8.3.4",
     "@types/benchmark": "^2.1.1",
-    "@types/browserify": "^12.0.37",
     "@types/cssnano": "^5.0.0",
     "@types/d3": "^7.4.0",
     "@types/diff": "^5.0.2",
     "@types/earcut": "^2.1.1",
-    "@types/eslint": "^8.4.5",
+    "@types/eslint": "^8.4.6",
     "@types/gl": "^4.1.1",
     "@types/glob": "^7.2.0",
-    "@types/jest": "^28.1.6",
+    "@types/jest": "^28.1.7",
     "@types/jsdom": "^20.0.0",
     "@types/minimist": "^1.2.2",
     "@types/murmurhash-js": "^1.0.3",
@@ -63,31 +63,30 @@
     "@types/offscreencanvas": "^2019.7.0",
     "@types/pixelmatch": "^5.2.4",
     "@types/pngjs": "^6.0.1",
-    "@types/react": "^18.0.15",
+    "@types/react": "^18.0.17",
     "@types/react-dom": "^18.0.6",
     "@types/request": "^2.48.8",
     "@types/rollup-plugin-json": "^3.0.3",
     "@types/shuffle-seed": "^1.1.0",
     "@types/supercluster": "^7.1.0",
     "@types/window-or-global": "^1.0.4",
-    "@typescript-eslint/eslint-plugin": "^5.32.0",
-    "@typescript-eslint/parser": "^5.32.0",
+    "@typescript-eslint/eslint-plugin": "^5.34.0",
+    "@typescript-eslint/parser": "^5.33.1",
     "acorn-import-assertions": "^1.8.0",
     "address": "^1.2.0",
     "benchmark": "^2.1.4",
-    "browserify": "^17.0.0",
     "canvas": "^2.9.3",
-    "cssnano": "^5.1.12",
+    "cssnano": "^5.1.13",
     "d3": "^7.6.1",
     "d3-queue": "^3.0.7",
     "diff": "^5.1.0",
-    "documentation": "14.0.0-alpha.1",
+    "documentation": "14.0.0",
     "dts-bundle-generator": "^6.12.0",
-    "eslint": "^8.21.0",
+    "eslint": "^8.22.0",
     "eslint-config-mourner": "^3.0.0",
     "eslint-plugin-html": "^7.1.0",
     "eslint-plugin-import": "^2.26.0",
-    "eslint-plugin-jest": "^26.7.0",
+    "eslint-plugin-jest": "^26.8.7",
     "eslint-plugin-jsdoc": "^39.3.4",
     "eslint-plugin-react": "^7.30.1",
     "gl": "^5.0.3",
@@ -106,9 +105,9 @@
     "node-plantuml": "^0.9.0",
     "npm-font-open-sans": "^1.1.0",
     "npm-run-all": "^4.1.5",
-    "pdf-merger-js": "^3.4.0",
+    "pdf-merger-js": "^4.0.0",
     "pixelmatch": "^5.3.0",
-    "playwright": "^1.24.2",
+    "playwright": "^1.25.0",
     "pngjs": "^6.0.0",
     "postcss": "^8.4.14",
     "postcss-cli": "^10.0.0",
@@ -117,7 +116,7 @@
     "react": "^18.2.0",
     "react-dom": "^18.2.0",
     "request": "^2.88.0",
-    "rollup": "^2.77.2",
+    "rollup": "^2.78.1",
     "rollup-plugin-import-assert": "^2.1.0",
     "rollup-plugin-sourcemaps": "^0.6.3",
     "rollup-plugin-terser": "^7.0.2",
@@ -126,12 +125,11 @@
     "shuffle-seed": "^1.1.6",
     "source-map-explorer": "^2.5.2",
     "st": "^3.0.0",
-    "stylelint": "^14.9.1",
-    "stylelint-config-standard": "^26.0.0",
-    "ts-jest": "^28.0.7",
+    "stylelint": "^14.11.0",
+    "stylelint-config-standard": "^28.0.0",
+    "ts-jest": "^28.0.8",
     "ts-node": "^10.9.1",
-    "typescript": "^4.7.4",
-    "unassert-rollup-plugin": "^2.0.0"
+    "typescript": "^4.7.4"
   },
   "scripts": {
     "generate-shaders": "node --loader ts-node/esm --experimental-specifier-resolution=node build/generate-shaders.ts",
@@ -141,6 +139,7 @@
     "generate-typings": "node --loader ts-node/esm --experimental-specifier-resolution=node build/generate-typings.ts",
     "generate-query-test-fixtures": "node --loader ts-node/esm --experimental-specifier-resolution=node build/generate-query-test-fixtures.ts",
     "generate-debug-index-file": "node --loader ts-node/esm --experimental-specifier-resolution=node build/generate-debug-index-file.ts",
+    "build-dist": "npm run generate-typings && npm run build-dev && npm run build-prod && npm run build-csp && npm run build-css",
     "build-dev": "rollup --configPlugin @rollup/plugin-typescript -c --environment BUILD:dev",
     "watch-dev": "rollup --configPlugin @rollup/plugin-typescript -c --environment BUILD:dev --watch",
     "build-prod": "rollup --configPlugin @rollup/plugin-typescript -c --environment BUILD:production",

package/src/data/array_types.g.ts

@@ -1,6 +1,5 @@
 // This file is generated. Edit build/generate-struct-arrays.ts, then run `npm run codegen`.
 
-import assert from 'assert';
 import {Struct, StructArray} from '../util/struct_array';
 import {register} from '../util/web_worker_transfer';
 import Point from '@mapbox/point-geometry';
@@ -877,7 +876,6 @@ export class CollisionBoxArray extends StructArrayLayout6i1ul2ui20 {
      * @private
      */
     get(index: number): CollisionBoxStruct {
-        assert(!this.isTransferred);
         return new CollisionBoxStruct(this, index);
     }
 }
@@ -922,7 +920,6 @@ export class PlacedSymbolArray extends StructArrayLayout2i2ui3ul3ui2f3ub1ul1i48
      * @private
      */
     get(index: number): PlacedSymbolStruct {
-        assert(!this.isTransferred);
         return new PlacedSymbolStruct(this, index);
     }
 }
@@ -976,7 +973,6 @@ export class SymbolInstanceArray extends StructArrayLayout8i15ui1ul4f68 {
      * @private
      */
     get(index: number): SymbolInstanceStruct {
-        assert(!this.isTransferred);
         return new SymbolInstanceStruct(this, index);
     }
 }
@@ -1024,7 +1020,6 @@ export class FeatureIndexArray extends StructArrayLayout1ul2ui8 {
      * @private
      */
     get(index: number): FeatureIndexStruct {
-        assert(!this.isTransferred);
         return new FeatureIndexStruct(this, index);
     }
 }

package/src/data/bucket/fill_bucket.ts

@@ -6,7 +6,6 @@ import {ProgramConfigurationSet} from '../program_configuration';
 import {LineIndexArray, TriangleIndexArray} from '../index_array_type';
 import earcut from 'earcut';
 import classifyRings from '../../util/classify_rings';
-import assert from 'assert';
 const EARCUT_MAX_RINGS = 500;
 import {register} from '../../util/web_worker_transfer';
 import {hasPattern, addPatternDependencies} from './pattern_bucket_features';
@@ -211,7 +210,6 @@ class FillBucket implements Bucket {
             }
 
             const indices = earcut(flattened, holeIndices);
-            assert(indices.length % 3 === 0);
 
             for (let i = 0; i < indices.length; i += 3) {
                 this.indexArray.emplaceBack(

package/src/data/bucket/fill_extrusion_bucket.ts

@@ -9,7 +9,6 @@ import earcut from 'earcut';
 import mvt from '@mapbox/vector-tile';
 const vectorTileFeatureTypes = mvt.VectorTileFeature.types;
 import classifyRings from '../../util/classify_rings';
-import assert from 'assert';
 const EARCUT_MAX_RINGS = 500;
 import {register} from '../../util/web_worker_transfer';
 import {hasPattern, addPatternDependencies} from './pattern_bucket_features';
@@ -264,7 +263,6 @@ class FillExtrusionBucket implements Bucket {
             }
 
             const indices = earcut(flattened, holeIndices);
-            assert(indices.length % 3 === 0);
 
             for (let j = 0; j < indices.length; j += 3) {
                 // Counter-clockwise winding order.

package/src/data/feature_position_map.ts

@@ -1,6 +1,5 @@
 import murmur3 from 'murmurhash-js';
 import {register} from '../util/web_worker_transfer';
-import assert from 'assert';
 
 type SerializedFeaturePositionMap = {
     ids: Float64Array;
@@ -31,7 +30,7 @@ export default class FeaturePositionMap {
     }
 
     getPositions(id: unknown): Array<FeaturePosition> {
-        assert(this.indexed);
+        if (!this.indexed) throw new Error('Trying to get index, but feature positions are not indexed');
 
         const intId = getNumericId(id);
 

package/src/gl/framebuffer.ts

@@ -1,5 +1,4 @@
 import {ColorAttachment, DepthAttachment} from './value';
-import assert from 'assert';
 
 import type Context from './context';
 
@@ -22,7 +21,9 @@ class Framebuffer {
         if (hasDepth) {
             this.depthAttachment = new DepthAttachment(context, fbo);
         }
-        assert(gl.checkFramebufferStatus(gl.FRAMEBUFFER) === gl.FRAMEBUFFER_COMPLETE);
+        if (gl.checkFramebufferStatus(gl.FRAMEBUFFER) !== gl.FRAMEBUFFER_COMPLETE) {
+            throw new Error('Framebuffer is not complete');
+        }
     }
 
     destroy() {

package/src/gl/index_buffer.ts

@@ -1,4 +1,3 @@
-import assert from 'assert';
 
 import type {StructArray} from '../util/struct_array';
 import type {TriangleIndexArray, LineIndexArray, LineStripIndexArray} from '../data/index_array_type';
@@ -34,7 +33,7 @@ class IndexBuffer {
 
     updateData(array: StructArray) {
         const gl = this.context.gl;
-        assert(this.dynamicDraw);
+        if (!this.dynamicDraw) throw new Error('Attempted to update data while not in dynamic mode.');
         // The right VAO will get this buffer re-bound later in VertexArrayObject#bind
         // See https://github.com/mapbox/mapbox-gl-js/issues/5620
         this.context.unbindVAO();

package/src/gl/vertex_buffer.ts

@@ -1,4 +1,3 @@
-import assert from 'assert';
 
 import type {
     StructArray,
@@ -62,7 +61,7 @@ class VertexBuffer {
     }
 
     updateData(array: StructArray) {
-        assert(array.length === this.length);
+        if (array.length !== this.length) throw new Error(`Length of new data is ${array.length}, which doesn't match current length of ${this.length}`);
         const gl = this.context.gl;
         this.bind();
         gl.bufferSubData(gl.ARRAY_BUFFER, 0, array.arrayBuffer);

package/src/index.ts

@@ -1,4 +1,3 @@
-import assert from 'assert';
 import {supported} from '@mapbox/mapbox-gl-supported';
 import packageJSON from '../package.json' assert {type: 'json'};
 import Map from './ui/map';
@@ -262,5 +261,4 @@ Debug.extend(exported, {isSafari, getPerformanceMetrics: PerformanceUtils.getPer
  */
 
 export default exported;
-// canary assert: used to confirm that asserts have been removed from production build
-assert(true, 'canary assert');
+

package/src/render/draw_debug.ts

@@ -66,7 +66,7 @@ function drawDebug(painter: Painter, sourceCache: SourceCache, coords: Array<Ove
     }
 }
 
-function drawDebugTile(painter, sourceCache, coord: OverscaledTileID) {
+function drawDebugTile(painter: Painter, sourceCache: SourceCache, coord: OverscaledTileID) {
     const context = painter.context;
     const gl = context.gl;
 

package/src/render/draw_fill.ts

@@ -1,6 +1,7 @@
 import Color from '../style-spec/util/color';
 import DepthMode from '../gl/depth_mode';
 import CullFaceMode from '../gl/cull_face_mode';
+import ColorMode from '../gl/color_mode';
 import {
     fillUniformValues,
     fillPatternUniformValues,
@@ -56,7 +57,14 @@ function drawFill(painter: Painter, sourceCache: SourceCache, layer: FillStyleLa
     }
 }
 
-function drawFillTiles(painter, sourceCache, layer, coords, depthMode, colorMode, isOutline) {
+function drawFillTiles(
+    painter: Painter,
+    sourceCache: SourceCache,
+    layer: FillStyleLayer,
+    coords: Array<OverscaledTileID>,
+    depthMode: Readonly<DepthMode>,
+    colorMode: Readonly<ColorMode>,
+    isOutline: boolean) {
     const gl = painter.context.gl;
 
     const patternProperty = layer.paint.get('fill-pattern');

package/src/render/draw_fill_extrusion.ts

@@ -45,7 +45,14 @@ function draw(painter: Painter, source: SourceCache, layer: FillExtrusionStyleLa
     }
 }
 
-function drawExtrusionTiles(painter, source, layer, coords, depthMode, stencilMode, colorMode) {
+function drawExtrusionTiles(
+    painter: Painter,
+    source: SourceCache,
+    layer: FillExtrusionStyleLayer,
+    coords: OverscaledTileID[],
+    depthMode: DepthMode,
+    stencilMode: Readonly<StencilMode>,
+    colorMode: Readonly<ColorMode>) {
     const context = painter.context;
     const gl = context.gl;
     const patternProperty = layer.paint.get('fill-extrusion-pattern');

package/src/render/draw_heatmap.ts

@@ -4,6 +4,8 @@ import DepthMode from '../gl/depth_mode';
 import StencilMode from '../gl/stencil_mode';
 import ColorMode from '../gl/color_mode';
 import CullFaceMode from '../gl/cull_face_mode';
+import Context from '../gl/context';
+import Framebuffer from '../gl/framebuffer';
 import {
     heatmapUniformValues,
     heatmapTextureUniformValues
@@ -67,7 +69,7 @@ function drawHeatmap(painter: Painter, sourceCache: SourceCache, layer: HeatmapS
     }
 }
 
-function bindFramebuffer(context, painter, layer) {
+function bindFramebuffer(context: Context, painter: Painter, layer: HeatmapStyleLayer) {
     const gl = context.gl;
     context.activeTexture.set(gl.TEXTURE1);
 
@@ -94,7 +96,7 @@ function bindFramebuffer(context, painter, layer) {
     }
 }
 
-function bindTextureToFramebuffer(context, painter, texture, fbo) {
+function bindTextureToFramebuffer(context: Context, painter: Painter, texture: WebGLTexture, fbo: Framebuffer) {
     const gl = context.gl;
     // Use the higher precision half-float texture where available (producing much smoother looking heatmaps);
     // Otherwise, fall back to a low precision texture
@@ -103,7 +105,7 @@ function bindTextureToFramebuffer(context, painter, texture, fbo) {
     fbo.colorAttachment.set(texture);
 }
 
-function renderTextureToMap(painter, layer) {
+function renderTextureToMap(painter: Painter, layer: HeatmapStyleLayer) {
     const context = painter.context;
     const gl = context.gl;
 

package/src/render/draw_hillshade.ts

@@ -2,6 +2,8 @@ import Texture from './texture';
 import StencilMode from '../gl/stencil_mode';
 import DepthMode from '../gl/depth_mode';
 import CullFaceMode from '../gl/cull_face_mode';
+import ColorMode from '../gl/color_mode';
+import Tile from '../source/tile';
 import {
     hillshadeUniformValues,
     hillshadeUniformPrepareValues
@@ -37,7 +39,14 @@ function drawHillshade(painter: Painter, sourceCache: SourceCache, layer: Hillsh
     context.viewport.set([0, 0, painter.width, painter.height]);
 }
 
-function renderHillshade(painter, coord, tile, layer, depthMode, stencilMode, colorMode) {
+function renderHillshade(
+    painter: Painter,
+    coord: OverscaledTileID,
+    tile: Tile,
+    layer: HillshadeStyleLayer,
+    depthMode: Readonly<DepthMode>,
+    stencilMode: Readonly<StencilMode>,
+    colorMode: Readonly<ColorMode>) {
     const context = painter.context;
     const gl = context.gl;
     const fbo = tile.fbo;
@@ -58,7 +67,13 @@ function renderHillshade(painter, coord, tile, layer, depthMode, stencilMode, co
 
 // hillshade rendering is done in two steps. the prepare step first calculates the slope of the terrain in the x and y
 // directions for each pixel, and saves those values to a framebuffer texture in the r and g channels.
-function prepareHillshade(painter, tile, layer, depthMode, stencilMode, colorMode) {
+function prepareHillshade(
+    painter: Painter,
+    tile: Tile,
+    layer: HillshadeStyleLayer,
+    depthMode: Readonly<DepthMode>,
+    stencilMode: Readonly<StencilMode>,
+    colorMode: Readonly<ColorMode>) {
     const context = painter.context;
     const gl = context.gl;
     const dem = tile.dem;

package/src/render/image_manager.ts

@@ -5,7 +5,6 @@ import {Event, ErrorEvent, Evented} from '../util/evented';
 import {RGBAImage} from '../util/image';
 import {ImagePosition} from './image_atlas';
 import Texture from './texture';
-import assert from 'assert';
 import {renderStyleImage} from '../style/style_image';
 import {warnOnce} from '../util/util';
 
@@ -88,7 +87,7 @@ class ImageManager extends Evented {
     }
 
     addImage(id: string, image: StyleImage) {
-        assert(!this.images[id]);
+        if (this.images[id]) throw new Error(`Image id ${id} already exist, use updateImage instead`);
         if (this._validate(id, image)) {
             this.images[id] = image;
         }
@@ -135,16 +134,15 @@ class ImageManager extends Evented {
 
     updateImage(id: string, image: StyleImage) {
         const oldImage = this.images[id];
-        assert(oldImage);
-        assert(oldImage.data.width === image.data.width);
-        assert(oldImage.data.height === image.data.height);
+        if (oldImage.data.width !== image.data.width || oldImage.data.height !== image.data.height) {
+            throw new Error(`size mismatch between old image (${oldImage.data.width}x${oldImage.data.height}) and new image (${image.data.width}x${image.data.height}).`);
+        }
         image.version = oldImage.version + 1;
         this.images[id] = image;
         this.updatedImages[id] = true;
     }
 
     removeImage(id: string) {
-        assert(this.images[id]);
         const image = this.images[id];
         delete this.images[id];
         delete this.patterns[id];
@@ -295,7 +293,7 @@ class ImageManager extends Evented {
             this.callbackDispatchedThisFrame[id] = true;
 
             const image = this.images[id];
-            assert(image);
+            if (!image) warnOnce(`Image with ID: "${id}" was not found`);
 
             const updated = renderStyleImage(image);
             if (updated) {

package/src/render/program/line_program.ts

@@ -186,7 +186,7 @@ function calculateTileRatio(tile: Tile, transform: Transform) {
     return 1 / pixelsToTileUnits(tile, 1, transform.tileZoom);
 }
 
-function calculateMatrix(painter, tile, layer, coord) {
+function calculateMatrix(painter: Painter, tile: Tile, layer: LineStyleLayer, coord: OverscaledTileID) {
     return painter.translatePosMatrix(
         coord ? coord.posMatrix : tile.tileID.posMatrix,
         tile,

package/src/render/program/pattern.ts

@@ -1,4 +1,3 @@
-import assert from 'assert';
 import {
     Uniform1i,
     Uniform1f,
@@ -74,7 +73,6 @@ function bgPatternUniformValues(
 ): UniformValues<BackgroundPatternUniformsType> {
     const imagePosA = painter.imageManager.getPattern(image.from.toString());
     const imagePosB = painter.imageManager.getPattern(image.to.toString());
-    assert(imagePosA && imagePosB);
     const {width, height} = painter.imageManager.getPixelSize();
 
     const numTiles = Math.pow(2, tile.tileID.overscaledZ);

package/src/render/program.ts

@@ -1,5 +1,4 @@
 import shaders from '../shaders/shaders';
-import assert from 'assert';
 import ProgramConfiguration from '../data/program_configuration';
 import VertexArrayObject from './vertex_array_object';
 import Context from '../gl/context';
@@ -84,7 +83,6 @@ class Program<Us extends UniformBindings> {
         }
         gl.shaderSource(fragmentShader, fragmentSource);
         gl.compileShader(fragmentShader);
-        assert(gl.getShaderParameter(fragmentShader, gl.COMPILE_STATUS), (gl.getShaderInfoLog(fragmentShader) as any));
         gl.attachShader(this.program, fragmentShader);
 
         const vertexShader = gl.createShader(gl.VERTEX_SHADER);
@@ -94,7 +92,6 @@ class Program<Us extends UniformBindings> {
         }
         gl.shaderSource(vertexShader, vertexSource);
         gl.compileShader(vertexShader);
-        assert(gl.getShaderParameter(vertexShader, gl.COMPILE_STATUS), (gl.getShaderInfoLog(vertexShader) as any));
         gl.attachShader(this.program, vertexShader);
 
         this.attributes = {};
@@ -110,7 +107,6 @@ class Program<Us extends UniformBindings> {
         }
 
         gl.linkProgram(this.program);
-        assert(gl.getProgramParameter(this.program, gl.LINK_STATUS), (gl.getProgramInfoLog(this.program) as any));
 
         gl.deleteShader(vertexShader);
         gl.deleteShader(fragmentShader);
@@ -178,11 +174,18 @@ class Program<Us extends UniformBindings> {
             configuration.setUniforms(context, this.binderUniforms, currentProperties, {zoom: (zoom as any)});
         }
 
-        const primitiveSize = {
-            [gl.LINES]: 2,
-            [gl.TRIANGLES]: 3,
-            [gl.LINE_STRIP]: 1
-        }[drawMode];
+        let primitiveSize = 0;
+        switch (drawMode) {
+            case gl.LINES:
+                primitiveSize = 2;
+                break;
+            case gl.TRIANGLES:
+                primitiveSize = 3;
+                break;
+            case gl.LINE_STRIP:
+                primitiveSize = 1;
+                break;
+        }
 
         for (const segment of segments.get()) {
             const vaos = segment.vaos || (segment.vaos = {});

package/src/render/vertex_array_object.ts

@@ -1,4 +1,3 @@
-import assert from 'assert';
 
 import type Program from './program';
 import type VertexBuffer from '../gl/vertex_buffer';
@@ -119,9 +118,8 @@ class VertexArrayObject {
             // Disable all attributes from the previous program that aren't used in
             // the new program. Note: attribute indices are *not* program specific!
             for (let i = numNextAttributes; i < numPrevAttributes; i++) {
-                // WebGL breaks if you disable attribute 0.
+                // WebGL breaks if you disable attribute 0, so if i == 0.
                 // http://stackoverflow.com/questions/20305231
-                assert(i !== 0);
                 gl.disableVertexAttribArray(i);
             }
         }

package/src/source/geojson_worker_source.ts

@@ -6,7 +6,6 @@ import GeoJSONWrapper from './geojson_wrapper';
 import vtpbf from 'vt-pbf';
 import Supercluster from 'supercluster';
 import geojsonvt from 'geojson-vt';
-import assert from 'assert';
 import VectorTileWorkerSource from './vector_tile_worker_source';
 import {createExpression} from '../style-spec/expression';
 
@@ -292,9 +291,6 @@ function getSuperclusterOptions({superclusterOptions, clusterProperties}: { supe
         const reduceExpressionParsed = createExpression(
             typeof operator === 'string' ? [operator, ['accumulated'], ['get', key]] : operator);
 
-        assert(mapExpressionParsed.result === 'success');
-        assert(reduceExpressionParsed.result === 'success');
-
         mapExpressions[key] = mapExpressionParsed.value;
         reduceExpressions[key] = reduceExpressionParsed.value;
     }

package/src/source/query_features.ts

@@ -6,7 +6,6 @@ import type {RetainedQueryData} from '../symbol/placement';
 import type {FilterSpecification} from '../style-spec/types.g';
 import type {MapGeoJSONFeature} from '../util/vectortile_to_geojson';
 import type Point from '@mapbox/point-geometry';
-import assert from 'assert';
 import {mat4} from 'gl-matrix';
 
 /*
@@ -137,8 +136,6 @@ export function queryRenderedSymbols(styleLayers: {[_: string]: StyleLayer},
                     // we sort each feature based on the first matching symbol instance.
                     const sortedA = featureSortOrder.indexOf(a.featureIndex);
                     const sortedB = featureSortOrder.indexOf(b.featureIndex);
-                    assert(sortedA >= 0);
-                    assert(sortedB >= 0);
                     return sortedB - sortedA;
                 } else {
                     // Bucket hasn't been re-sorted based on angle, so use the