@mapbox/mapbox-gl-style-spec 14.31.0 → 14.32.0-rc.1

package/migrate/v8.ts

@@ -1,81 +1,85 @@
-/* eslint-disable @typescript-eslint/ban-ts-comment */
-// @ts-nocheck
 import {eachSource, eachLayer, eachProperty} from '../visit';
 
-function eachLayout(layer, callback) {
+type LegacyObject = Record<string, unknown>;
+type LegacyFunctionStop = [unknown, unknown];
+type LegacyFunction = {stops: LegacyFunctionStop[]};
+type LegacySource = LegacyObject & {type?: string; url?: unknown; coordinates?: unknown[][]};
+type LegacyLayer = LegacyObject & {type?: string; layout?: LegacyObject};
+type LegacyStyle = {
+    version: number;
+    constants?: LegacyObject;
+    glyphs?: string;
+    layers?: LegacyLayer[];
+};
+type Migrated<T> = Omit<T, 'version'> & {version: 8};
+type Each<T> = (style: LegacyStyle, callback: (value: T) => void) => void;
+type EachProperty = (
+    style: LegacyStyle,
+    options: {paint?: boolean; layout?: boolean},
+    callback: (property: {value: unknown; set: (value: unknown) => void}) => void
+) => void;
+
+function eachLayout(layer: LegacyLayer, callback: (layout: LegacyObject, key: string) => void): void {
     for (const k in layer) {
         if (k.indexOf('layout') === 0) {
-            // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
-            callback(layer[k], k);
+            callback(layer[k] as LegacyObject, k);
         }
     }
 }
 
-function eachPaint(layer, callback) {
+function eachPaint(layer: LegacyLayer, callback: (paint: LegacyObject, key: string) => void): void {
     for (const k in layer) {
         if (k.indexOf('paint') === 0) {
-            // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
-            callback(layer[k], k);
+            callback(layer[k] as LegacyObject, k);
         }
     }
 }
 
-function resolveConstant(style, value) {
+function resolveConstant(style: LegacyStyle, value: unknown): unknown {
     if (typeof value === 'string' && value[0] === '@') {
-        // eslint-disable-next-line @typescript-eslint/no-unsafe-return, @typescript-eslint/no-unsafe-member-access
-        return resolveConstant(style, style.constants[value]);
+        return resolveConstant(style, style.constants![value]);
     } else {
-        // eslint-disable-next-line @typescript-eslint/no-unsafe-return
         return value;
     }
 }
 
-function isFunction(value) {
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
-    return Array.isArray(value.stops);
+function isFunction(value: unknown): value is LegacyFunction {
+    return Array.isArray((value as LegacyFunction).stops);
 }
 
-function renameProperty(obj, from, to) {
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-unsafe-member-access
+function renameProperty(obj: LegacyObject, from: string, to: string): void {
     obj[to] = obj[from]; delete obj[from];
 }
 
-export default function (style) {
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+export default function <T extends LegacyStyle>(style: T): Migrated<T> {
     style.version = 8;
 
     // Rename properties, reverse coordinates in source and layers
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-argument
-    eachSource(style, (source) => {
+    (eachSource as unknown as Each<LegacySource>)(style, (source) => {
         if (source.type === 'video' && source.url !== undefined) {
             renameProperty(source, 'url', 'urls');
         }
         if (source.type === 'video') {
-            source.coordinates.forEach((coord) => {
+            source.coordinates!.forEach((coord) => {
                 return coord.reverse();
             });
         }
     });
 
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-argument
-    eachLayer(style, (layer) => {
+    (eachLayer as unknown as Each<LegacyLayer>)(style, (layer) => {
         eachLayout(layer, (layout) => {
-            // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
             if (layout['symbol-min-distance'] !== undefined) {
                 renameProperty(layout, 'symbol-min-distance', 'symbol-spacing');
             }
         });
 
         eachPaint(layer, (paint) => {
-            // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
             if (paint['background-image'] !== undefined) {
                 renameProperty(paint, 'background-image', 'background-pattern');
             }
-            // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
             if (paint['line-image'] !== undefined) {
                 renameProperty(paint, 'line-image', 'line-pattern');
             }
-            // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
             if (paint['fill-image'] !== undefined) {
                 renameProperty(paint, 'fill-image', 'fill-pattern');
             }
@@ -83,65 +87,49 @@ export default function (style) {
     });
 
     // Inline Constants
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-argument
-    eachProperty(style, {paint: true, layout: true}, (property) => {
-        // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
+    (eachProperty as unknown as EachProperty)(style, {paint: true, layout: true}, (property) => {
         const value = resolveConstant(style, property.value);
 
         if (isFunction(value)) {
-            // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
             value.stops.forEach((stop) => {
-                // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-unsafe-member-access
                 stop[1] = resolveConstant(style, stop[1]);
             });
         }
 
         property.set(value);
     });
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
     delete style.constants;
 
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-argument
-    eachLayer(style, (layer) => {
+    (eachLayer as unknown as Each<LegacyLayer>)(style, (layer) => {
         // get rid of text-max-size, icon-max-size
         // turn text-size, icon-size into layout properties
         // https://github.com/mapbox/mapbox-gl-style-spec/issues/255
 
         eachLayout(layer, (layout) => {
-            // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
             delete layout['text-max-size'];
-            // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
             delete layout['icon-max-size'];
         });
 
         eachPaint(layer, (paint) => {
-            // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
             if (paint['text-size']) {
                 if (!layer.layout) layer.layout = {};
-                // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-unsafe-member-access
                 layer.layout['text-size'] = paint['text-size'];
-                // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
                 delete paint['text-size'];
             }
 
-            // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
             if (paint['icon-size']) {
                 if (!layer.layout) layer.layout = {};
-                // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-unsafe-member-access
                 layer.layout['icon-size'] = paint['icon-size'];
-                // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
                 delete paint['icon-size'];
             }
         });
     });
 
-    function migrateFontstackURL(input) {
-        // eslint-disable-next-line @typescript-eslint/no-unsafe-argument
+    function migrateFontstackURL(input: string): string {
         const inputParsed = new URL(input);
         const inputPathnameParts = inputParsed.pathname.split('/');
 
         if (inputParsed.protocol !== 'mapbox:') {
-            // eslint-disable-next-line @typescript-eslint/no-unsafe-return
             return input;
 
         } else if (inputParsed.hostname === 'fontstack') {
@@ -150,52 +138,43 @@ export default function (style) {
 
         } else if (inputParsed.hostname === 'fonts') {
             assert(inputPathnameParts[1] === 'v1');
-            assert(decodeURI(inputPathnameParts[3]) === '{fontstack}');
-            assert(decodeURI(inputPathnameParts[4]) === '{range}.pbf');
-            return `mapbox://fonts/${inputPathnameParts[2]}/{fontstack}/{range}.pbf`;
+            assert(decodeURI(inputPathnameParts[3]!) === '{fontstack}');
+            assert(decodeURI(inputPathnameParts[4]!) === '{range}.pbf');
+            return `mapbox://fonts/${inputPathnameParts[2]!}/{fontstack}/{range}.pbf`;
 
         } else {
             assert(false);
         }
 
-        function assert(predicate) {
+        function assert(predicate: boolean): asserts predicate {
             if (!predicate) {
                 throw new Error(`Invalid font url: "${input}"`);
             }
         }
     }
 
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
     if (style.glyphs) {
-        // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-unsafe-member-access
         style.glyphs = migrateFontstackURL(style.glyphs);
     }
 
-    function migrateFontStack(font) {
-        function splitAndTrim(string) {
-            // eslint-disable-next-line @typescript-eslint/no-unsafe-return, @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
+    function migrateFontStack(font: unknown): unknown {
+        function splitAndTrim(string: string): string[] {
             return string.split(',').map((s) => {
-                // eslint-disable-next-line @typescript-eslint/no-unsafe-return, @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
                 return s.trim();
             });
         }
 
         if (Array.isArray(font)) {
             // Assume it's a previously migrated font-array.
-            // eslint-disable-next-line @typescript-eslint/no-unsafe-return
             return font;
 
         } else if (typeof font === 'string') {
-            // eslint-disable-next-line @typescript-eslint/no-unsafe-return
             return splitAndTrim(font);
 
         } else if (typeof font === 'object') {
-            // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
-            font.stops.forEach((stop) => {
-                // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-unsafe-member-access
-                stop[1] = splitAndTrim(stop[1]);
+            (font as LegacyFunction).stops.forEach((stop) => {
+                stop[1] = splitAndTrim(stop[1] as string);
             });
-            // eslint-disable-next-line @typescript-eslint/no-unsafe-return
             return font;
 
         } else {
@@ -203,12 +182,9 @@ export default function (style) {
         }
     }
 
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-argument
-    eachLayer(style, (layer) => {
+    (eachLayer as unknown as Each<LegacyLayer>)(style, (layer) => {
         eachLayout(layer, (layout) => {
-            // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
             if (layout['text-font']) {
-                // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-unsafe-member-access
                 layout['text-font'] = migrateFontStack(layout['text-font']);
             }
         });
@@ -228,24 +204,17 @@ export default function (style) {
     // Symbol layers that are below another type (line, fill) of layer preserve their draw order.
 
     let firstSymbolLayer = 0;
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
-    for (let i = style.layers.length - 1; i >= 0; i--) {
-        // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-unsafe-member-access
-        const layer = style.layers[i];
-        // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+    for (let i = style.layers!.length - 1; i >= 0; i--) {
+        const layer = style.layers![i]!;
         if (layer.type !== 'symbol') {
             firstSymbolLayer = i + 1;
             break;
         }
     }
 
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
-    const symbolLayers = style.layers.splice(firstSymbolLayer);
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
+    const symbolLayers = style.layers!.splice(firstSymbolLayer);
     symbolLayers.reverse();
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-unsafe-call
-    style.layers = style.layers.concat(symbolLayers);
+    style.layers = style.layers!.concat(symbolLayers);
 
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-return
-    return style;
+    return style as Migrated<T>;
 }

package/migrate/v9.ts

@@ -1,33 +1,33 @@
-/* eslint-disable @typescript-eslint/ban-ts-comment */
-// @ts-nocheck
 import deref from '../deref';
 
-function eachLayer(style, callback) {
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+import type {LayerSpecification} from '../types';
+
+type LegacyLayer = Record<string, unknown> & {id: string};
+type LegacyStyle = {version: number; layers: LegacyLayer[]};
+
+type Migrated<T> = Omit<T, 'version'> & {version: 9};
+
+function eachLayer(style: LegacyStyle, callback: (layer: LegacyLayer) => void): void {
+    // eslint-disable-next-line @typescript-eslint/no-for-in-array
     for (const k in style.layers) {
-        // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
-        callback(style.layers[k]);
+        callback(style.layers[k as unknown as number]!);
     }
 }
 
-export default function (style) {
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+export default function <T extends LegacyStyle>(style: T): Migrated<T> {
     style.version = 9;
 
     // remove user-specified refs
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-unsafe-argument
-    style.layers = deref(style.layers);
+    style.layers = deref(style.layers as unknown as Array<LayerSpecification>);
 
     // remove class-specific paint properties
     eachLayer(style, (layer) => {
         for (const k in layer) {
             if (k.includes('paint.')) {
-                // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
                 delete layer[k];
             }
         }
     });
 
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-return
-    return style;
+    return style as Migrated<T>;
 }

package/migrate.ts

@@ -20,8 +20,7 @@ export default function (style: {version: 7} | StyleSpecification): StyleSpecifi
     let migrated = false;
 
     if (style.version === 7) {
-        // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
-        style = migrateToV8(style);
+        style = migrateToV8(style) as StyleSpecification;
         migrated = true;
     }
 
@@ -34,5 +33,5 @@ export default function (style: {version: 7} | StyleSpecification): StyleSpecifi
         throw new Error(`Cannot migrate from ${style.version}`);
     }
 
-    return style as StyleSpecification;
+    return style;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mapbox/mapbox-gl-style-spec",
-  "version": "14.31.0",
+  "version": "14.32.0-rc.1",
   "description": "a specification for mapbox gl styles",
   "author": "Mapbox",
   "license": "SEE LICENSE IN LICENSE.txt",

package/reference/v8.json

@@ -43,14 +43,14 @@
       "default": 0,
       "period": 360,
       "units": "degrees",
-      "doc": "Default bearing, in degrees. The bearing is the compass direction that is \"up\"; for example, a bearing of 90° orients the map so that east is up. This value will be used only if the map has not been positioned by other means (e.g. map options or user interaction).",
+      "doc": "Default bearing. The bearing is the compass direction that is \"up\"; for example, a bearing of 90° orients the map so that east is up. This value will be used only if the map has not been positioned by other means (e.g. map options or user interaction).",
       "example": 29
     },
     "pitch": {
       "type": "number",
       "default": 0,
       "units": "degrees",
-      "doc": "Default pitch, in degrees. Zero is perpendicular to the surface, for a look straight down at the map, while a greater value like 60 looks ahead towards the horizon. The style pitch will be used only if the map has not been positioned by other means (e.g. map options or user interaction).",
+      "doc": "Default pitch. Zero is perpendicular to the surface, for a look straight down at the map, while a greater value like 60 looks ahead towards the horizon. The style pitch will be used only if the map has not been positioned by other means (e.g. map options or user interaction).",
       "example": 50
     },
     "light": {
@@ -1894,9 +1894,9 @@
       "doc": "If true, the models will be reduced in density based on the zoom level. This is useful for large datasets that may be slow to render.",
       "sdk-support": {
         "basic functionality": {
-          "js": "0.10.0",
-          "android": "2.0.1",
-          "ios": "2.0.0"
+          "js": "3.18.0",
+          "android": "11.20.0",
+          "ios": "11.18.0"
         }
       },
       "property-type": "data-constant"
@@ -2230,7 +2230,8 @@
       "default": 0,
       "minimum": 0,
       "maximum": 1,
-      "doc": "Radius of a fill extrusion edge in meters. If not zero, rounds extrusion edges for a smoother appearance.",
+      "doc": "Radius of a fill extrusion edge. If not zero, rounds extrusion edges for a smoother appearance.",
+      "units": "meters",
       "sdk-support": {
         "basic functionality": {
           "js": "3.0.0",
@@ -2352,6 +2353,7 @@
       "maximum": 20.0,
       "default": 3.1,
       "doc": "Width of a floor. Think of this as measure how wide each unit should be. This effectively determines the number of units per each floor. Note that this does not affect the ground level facades (i.e. number of windows).",
+      "units": "meters",
       "private": true,
       "property-type": "data-driven",
       "sdk-support": {
@@ -2516,7 +2518,7 @@
       "units": "meters",
       "default": 0,
       "minimum": 0,
-      "doc": "The extent of the flood light effect on the walls of the buildings in meters.",
+      "doc": "The extent of the flood light effect on the walls of the buildings.",
       "requires": [
         "lights"
       ],
@@ -2547,7 +2549,7 @@
       "private": true,
       "units": "meters",
       "default": 0,
-      "doc": "The extent of the flood light effect on the ground beneath the buildings in meters.",
+      "doc": "The extent of the flood light effect on the ground beneath the buildings.",
       "requires": [
         "lights"
       ],
@@ -2576,7 +2578,6 @@
       "property-type": "data-driven",
       "type": "boolean",
       "private": true,
-      "units": "meters",
       "default": false,
       "doc": "Flips the orientation of the roofs for the buildings. This only affects simple geometries. Namely buildings whose footprints form a quadrilateral. By default (false), the roof ridge takes the direction of the longer edge of the quadrilateral.",
       "transition": true,
@@ -2757,8 +2758,9 @@
     },
     "line-z-offset": {
       "type": "number",
-      "doc": "Vertical offset from ground, in meters. Not supported for globe projection at the moment.",
+      "doc": "Vertical offset from ground. Not supported for globe projection at the moment.",
       "default": 0,
+      "units": "meters",
       "requires": [
         "line-elevation-reference"
       ],
@@ -6119,7 +6121,8 @@
         ],
         "relaxZoomRestriction": true
       },
-      "doc": "An array of two number values, specifying the vertical range, measured in meters, over which the fog should gradually fade out. When both parameters are set to zero, the fog will be rendered without any vertical constraints.",
+      "doc": "An array of two number values, specifying the vertical range over which the fog should gradually fade out. When both parameters are set to zero, the fog will be rendered without any vertical constraints.",
+      "units": "meters",
       "sdk-support": {
         "basic functionality": {
           "js": "3.0.0",
@@ -7217,7 +7220,7 @@
         ]
       },
       "requires": [
-        "line-pattern"
+        "fill-pattern"
       ],
       "sdk-support": {
         "basic functionality": {
@@ -7256,7 +7259,8 @@
     },
     "fill-z-offset": {
       "type": "number",
-      "doc": "Specifies an uniform elevation in meters. Note: If the value is zero, the layer will be rendered on the ground. Non-zero values will elevate the layer from the sea level, which can cause it to be rendered below the terrain.",
+      "doc": "Specifies an uniform elevation. Note: If the value is zero, the layer will be rendered on the ground. Non-zero values will elevate the layer from the sea level, which can cause it to be rendered below the terrain.",
+      "units": "meters",
       "default": 0,
       "minimum": 0,
       "transition": true,
@@ -7491,7 +7495,7 @@
         ]
       },
       "requires": [
-        "line-pattern"
+        "fill-extrusion-pattern"
       ],
       "sdk-support": {
         "basic functionality": {
@@ -7718,7 +7722,8 @@
         ]
       },
       "transition": true,
-      "doc": "The extent of the ambient occlusion effect on the ground beneath the extruded buildings in meters.",
+      "doc": "The extent of the ambient occlusion effect on the ground beneath the extruded buildings.",
+      "units": "meters",
       "requires": [
         "lights"
       ],
@@ -7816,7 +7821,7 @@
       "units": "meters",
       "default": 0,
       "minimum": 0,
-      "doc": "The extent of the flood light effect on the walls of the extruded buildings in meters.",
+      "doc": "The extent of the flood light effect on the walls of the extruded buildings.",
       "requires": [
         "lights"
       ],
@@ -7847,7 +7852,7 @@
       "experimental": true,
       "units": "meters",
       "default": 0,
-      "doc": "The extent of the flood light effect on the ground beneath the extruded buildings in meters. Note: this experimental property is evaluated once per tile, during tile initialization. Changing the property value could trigger tile reload. The `feature-state` styling is deprecated and will get removed soon.",
+      "doc": "The extent of the flood light effect on the ground beneath the extruded buildings. Note: this experimental property is evaluated once per tile, during tile initialization. Changing the property value could trigger tile reload. The `feature-state` styling is deprecated and will get removed soon.",
       "requires": [
         "lights"
       ],
@@ -8148,7 +8153,8 @@
         ]
       },
       "transition": true,
-      "doc": "The extent of the ambient occlusion effect on the ground beneath the procedural buildings in meters.",
+      "doc": "The extent of the ambient occlusion effect on the ground beneath the procedural buildings.",
+      "units": "meters",
       "sdk-support": {
         "basic functionality": {
           "js": "3.16.0",
@@ -8640,7 +8646,7 @@
       "minimum": 0,
       "transition": true,
       "units": "pixels",
-      "doc": "Blur applied to the line, in pixels.",
+      "doc": "Blur applied to the line.",
       "sdk-support": {
         "basic functionality": {
           "js": "0.10.0",
@@ -8925,6 +8931,7 @@
       "default": 0.0,
       "minimum": 0.0,
       "transition": true,
+      "units": "pixels",
       "sdk-support": {
         "basic functionality": {
           "js": "3.0.0",
@@ -9088,7 +9095,7 @@
       "default": 0,
       "minimum": 0,
       "units": "meters",
-      "doc": "Depth offset along the camera, in meters. When set to a non-zero value, applied cutout opacity is 0.",
+      "doc": "Depth offset along the camera. When set to a non-zero value, applied cutout opacity is 0.",
       "experimental": true,
       "private": true,
       "transition": true,
@@ -9458,7 +9465,7 @@
       "minimum": 1,
       "transition": true,
       "units": "pixels",
-      "doc": "Radius of influence of one heatmap point in pixels. Increasing the value makes the heatmap smoother, but less detailed. `queryRenderedFeatures` on heatmap layers will return points within this radius.",
+      "doc": "Radius of influence of one heatmap point. Increasing the value makes the heatmap smoother, but less detailed. `queryRenderedFeatures` on heatmap layers will return points within this radius.",
       "sdk-support": {
         "basic functionality": {
           "js": "0.41.0",
@@ -10294,7 +10301,8 @@
     },
     "symbol-z-offset": {
       "type": "number",
-      "doc": "Specifies an uniform elevation from the ground, in meters.",
+      "doc": "Specifies an uniform elevation from the ground.",
+      "units": "meters",
       "default": 0,
       "minimum": 0,
       "transition": true,
@@ -10722,10 +10730,11 @@
     },
     "raster-elevation": {
       "type": "number",
-      "doc": "Defines an uniform elevation from the base specified in raster-elevation-reference, in meters.",
+      "doc": "Defines an uniform elevation from the base specified in raster-elevation-reference.",
       "default": 0,
       "minimum": 0,
       "transition": true,
+      "units": "meters",
       "experimental": true,
       "sdk-support": {
         "basic functionality": {
@@ -10898,10 +10907,11 @@
     },
     "raster-particle-elevation": {
       "type": "number",
-      "doc": "Defines an uniform elevation from sea level, in meters.",
+      "doc": "Defines an uniform elevation from sea level.",
       "default": 0,
       "minimum": 0,
       "transition": true,
+      "units": "meters",
       "sdk-support": {
         "basic functionality": {
           "js": "3.7.0",
@@ -10925,6 +10935,7 @@
       "minimum": 0,
       "maximum": 359,
       "doc": "The direction of the light source used to generate the hillshading with 0 as the top of the viewport if `hillshade-illumination-anchor` is set to `viewport` and due north if `hillshade-illumination-anchor` is set to `map` and no 3d lights enabled. If `hillshade-illumination-anchor` is set to `map` and 3d lights enabled, the direction from 3d lights is used instead.",
+      "units": "degrees",
       "transition": false,
       "sdk-support": {
         "basic functionality": {
@@ -11327,7 +11338,8 @@
       "minimum": 0,
       "maximum": 180,
       "transition": false,
-      "doc": "The angular distance (measured in degrees) from `sky-gradient-center` up to which the gradient extends. A value of 180 causes the gradient to wrap around to the opposite direction from `sky-gradient-center`.",
+      "doc": "The angular distance from `sky-gradient-center` up to which the gradient extends. A value of 180 causes the gradient to wrap around to the opposite direction from `sky-gradient-center`.",
+      "units": "degrees",
       "sdk-support": {
         "basic functionality": {
           "js": "2.0.0",
@@ -11546,7 +11558,8 @@
         }
       },
       "transition": true,
-      "doc": "The translation of the model in meters in form of [longitudal, latitudal, altitude] offsets."
+      "units": "meters",
+      "doc": "The translation of the model in form of [longitudal, latitudal, altitude] offsets."
     },
     "model-color": {
       "type": "color",

package/validate/validate_appearance.ts

@@ -3,7 +3,6 @@ import ValidationError from '../error/validation_error';
 import validateProperty from './validate_property';
 import {unbundle} from '../util/unbundle_jsonlint';
 import validateExpression from './validate_expression';
-import latest from '../reference/latest';
 
 import type {StyleSpecification, LayerSpecification, AppearanceSpecification} from '../types';
 import type {StyleReference} from '../reference/latest';
@@ -91,7 +90,7 @@ function validateCondition(options: AppearanceValidatorOptions): Array<Validatio
     errors.push(...validateExpression({
         key: options.key,
         value: condition,
-        valueSpec: (latest['appearance'] as Record<string, Partial<StylePropertySpecification>>)['condition'],
+        valueSpec: (options.styleSpec['appearance'] as Record<string, Partial<StylePropertySpecification>>)['condition'],
         expressionContext: 'appearance'
     }));