@mapbox/mapbox-gl-style-spec 14.16.0-beta.3 → 14.16.0

package/diff.ts

@@ -629,8 +629,7 @@ export default function diffStyles(before: StyleSpecification, after: StyleSpeci
         const sourcesRemoved: Record<string, true> = {};
 
         // First collect the {add,remove}Source commands
-        const removeOrAddSourceCommands = [];
-        // eslint-disable-next-line @typescript-eslint/no-unsafe-argument
+        const removeOrAddSourceCommands: Command[] = [];
         diffSources(before.sources, after.sources, removeOrAddSourceCommands, sourcesRemoved);
 
         // Push a removeLayer command for each style layer that depends on a
@@ -638,7 +637,7 @@ export default function diffStyles(before: StyleSpecification, after: StyleSpeci
         // Also, exclude any such layers them from the input to `diffLayers`
         // below, so that diffLayers produces the appropriate `addLayers`
         // command
-        const beforeLayers = [];
+        const beforeLayers: LayerSpecification[] = [];
         if (before.layers) {
             before.layers.forEach((layer) => {
                 if (layer.source && sourcesRemoved[layer.source]) {
@@ -667,7 +666,7 @@ export default function diffStyles(before: StyleSpecification, after: StyleSpeci
         }
 
         // Handle changes to `layers`
-        // eslint-disable-next-line @typescript-eslint/no-unsafe-argument
+
         diffLayers(beforeLayers, after.layers, commands);
     } catch (e) {
         // fall back to setStyle

package/dist/index.cjs

@@ -838,7 +838,7 @@
   	},
   	promoteId: {
   		type: "promoteId",
-  		doc: "A property to use as a feature id (for feature state). Either a property name, or an object of the form `{<sourceLayer>: <propertyName>}`. If specified as a string for a vector tile source, the same property is used across all its source layers. If specified as an object only specified source layers will have id overriden, others will fallback to original feature id"
+  		doc: "A property to use as a feature id (for feature state). It can be a property name, or an expression to evaluate as the ID, or an object of the form `{<sourceLayer>: <propertyName/expression>}`. The expression can only be feature dependent if it is used. If specified as a string for a vector tile source, the same property is used across all its source layers. If specified as an object only specified source layers will have id overriden, others will fallback to original feature id"
   	},
   	volatile: {
   		type: "boolean",
@@ -1204,7 +1204,7 @@
   	},
   	promoteId: {
   		type: "promoteId",
-  		doc: "A property to use as a feature id (for feature state). Either a property name, or an object of the form `{<sourceLayer>: <propertyName>}`."
+  		doc: "A property to use as a feature id (for feature state). Either a property name, an expression to evaluate as the ID, or an object of the form `{<sourceLayer>: <propertyName/expression>}`. The expression can only be feature dependent if it is used."
   	},
   	dynamic: {
   		type: "boolean",
@@ -1648,7 +1648,7 @@
   			"symbol"
   		],
   		"private": true,
-  		doc: "Conditional styling applied to layer features based on dynamic conditions. If multiple conditions are true, only the first matching appearance will be applied. Only properties marked with 'Can be used in appearances' are supported."
+  		doc: "Conditional styling applied to symbol layer features based on dynamic conditions. If multiple conditions are true, only the first matching appearance will be applied. Only properties marked with 'Works with appearances' are supported."
   	}
   };
   var appearance = {
@@ -10576,7 +10576,7 @@
   var promoteId = {
   	"*": {
   		type: "*",
-  		doc: "A feature property name to use as the ID, or an expression to evaluate as the ID for feature state."
+  		doc: "A feature property name to use as the ID, or an expression to evaluate as the ID for feature state. The expression can only be feature dependent if it is used."
   	}
   };
   var $doc = {
@@ -14129,8 +14129,7 @@
               }
               return new FormattedSection(// eslint-disable-next-line @typescript-eslint/no-unsafe-argument
               toString(evaluatedContent), null, // eslint-disable-next-line @typescript-eslint/no-unsafe-argument
-              section.scale ? section.scale.evaluate(ctx) : null, // eslint-disable-next-line @typescript-eslint/no-unsafe-argument, @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
-              section.font ? section.font.evaluate(ctx).join(',') : null, // eslint-disable-next-line @typescript-eslint/no-unsafe-argument
+              section.scale ? section.scale.evaluate(ctx) : null, section.font ? section.font.evaluate(ctx).join(',') : null, // eslint-disable-next-line @typescript-eslint/no-unsafe-argument
               section.textColor ? section.textColor.evaluate(ctx) : null);
           };
           return new Formatted(this.sections.map(evaluateSection));
@@ -16439,8 +16438,7 @@
       }
       const ruler = new CheapRuler(lngLatLines[0][0][1], 'meters');
       if (geometry.type === 'Point' || geometry.type === 'MultiPoint' || geometry.type === 'LineString') {
-          return pointSetToLinesDistance(geometry.type === 'Point' ? [geometry.coordinates] : geometry.coordinates, // eslint-disable-next-line @typescript-eslint/no-unsafe-argument
-          geometry.type === 'LineString', lngLatLines, ruler);
+          return pointSetToLinesDistance(geometry.type === 'Point' ? [geometry.coordinates] : geometry.coordinates, geometry.type === 'LineString', lngLatLines, ruler);
       }
       if (geometry.type === 'MultiLineString') {
           let dist = Infinity;
@@ -16477,8 +16475,7 @@
       }
       const ruler = new CheapRuler(lngLatPolygons[0][0][0][1], 'meters');
       if (geometry.type === 'Point' || geometry.type === 'MultiPoint' || geometry.type === 'LineString') {
-          return pointSetToPolygonsDistance(geometry.type === 'Point' ? [geometry.coordinates] : geometry.coordinates, // eslint-disable-next-line @typescript-eslint/no-unsafe-argument
-          geometry.type === 'LineString', lngLatPolygons, ruler);
+          return pointSetToPolygonsDistance(geometry.type === 'Point' ? [geometry.coordinates] : geometry.coordinates, geometry.type === 'LineString', lngLatPolygons, ruler);
       }
       if (geometry.type === 'MultiLineString') {
           let dist = Infinity;
@@ -16492,8 +16489,7 @@
           return dist;
       }
       if (geometry.type === 'Polygon' || geometry.type === 'MultiPolygon') {
-          return polygonsToPolygonsDistance(geometry.type === 'Polygon' ? [geometry.coordinates] : geometry.coordinates, // eslint-disable-next-line @typescript-eslint/no-unsafe-argument
-          lngLatPolygons, ruler);
+          return polygonsToPolygonsDistance(geometry.type === 'Polygon' ? [geometry.coordinates] : geometry.coordinates, lngLatPolygons, ruler);
       }
       return null;
   }
@@ -17432,11 +17428,8 @@
               outputType = outputType || parsed.type;
               parsedArgs.push(parsed);
           }
-          const needsAnnotation = expectedType && // eslint-disable-next-line @typescript-eslint/no-unsafe-argument, @typescript-eslint/no-unsafe-member-access
-          parsedArgs.some(arg => checkSubtype(expectedType, arg.type));
-          return needsAnnotation ? // eslint-disable-next-line @typescript-eslint/no-unsafe-argument
-          new Coalesce(ValueType, parsedArgs) : // eslint-disable-next-line @typescript-eslint/no-unsafe-argument
-          new Coalesce(outputType, parsedArgs);
+          const needsAnnotation = expectedType && parsedArgs.some(arg => checkSubtype(expectedType, arg.type));
+          return needsAnnotation ? new Coalesce(ValueType, parsedArgs) : new Coalesce(outputType, parsedArgs);
       }
       // eslint-disable-next-line @typescript-eslint/no-explicit-any
       evaluate(ctx) {
@@ -19370,6 +19363,9 @@
       constructor(expression, propertySpec, scope, options, iconImageUseTheme) {
           this.expression = expression;
           this._warningHistory = {};
+          this._scope = scope;
+          this._options = options;
+          this._iconImageUseTheme = iconImageUseTheme;
           this._evaluator = new EvaluationContext(scope, options, iconImageUseTheme);
           this._defaultValue = propertySpec ? getDefaultValue(propertySpec) : null;
           this._enumValues = propertySpec && propertySpec.type === 'enum' ? propertySpec.values : null;
@@ -19388,6 +19384,9 @@
           return this.expression.evaluate(this._evaluator);
       }
       evaluate(globals, feature, featureState, canonical, availableImages, formattedSection, featureTileCoord, featureDistanceData, iconImageUseTheme) {
+          if (!this._evaluator) {
+              this._evaluator = new EvaluationContext(this._scope, this._options, this._iconImageUseTheme);
+          }
           this._evaluator.globals = globals;
           this._evaluator.feature = feature || null;
           this._evaluator.featureState = featureState || null;
@@ -21779,7 +21778,8 @@ Use an identity property function instead: ${ example }.`)];
       } else if (options.layerType === 'model' && propertyType === 'paint' && layer && layer.layout && layer.layout.hasOwnProperty('model-id')) {
           if (supportsPropertyExpression(valueSpec) && (supportsLightExpression(valueSpec) || supportsZoomExpression(valueSpec))) {
               const expression = createPropertyExpression(deepUnbundle(value), valueSpec);
-              const expressionObj = expression.value.expression || expression.value._styleExpression.expression;
+              const expressionValue = expression.value;
+              const expressionObj = 'expression' in expressionValue && expressionValue.expression || '_styleExpression' in expressionValue && expressionValue._styleExpression && expressionValue._styleExpression.expression;
               if (expressionObj && !isGlobalPropertyConstant(expressionObj, ['measure-light'])) {
                   if (propertyKey !== 'model-emissive-strength' || (!isFeatureConstant(expressionObj) || !isStateConstant(expressionObj))) {
                       errors.push(new ValidationError(key, value, `${ propertyKey } does not support measure-light expressions when the model layer source is vector tile or GeoJSON.`));
@@ -21858,7 +21858,6 @@ Use an identity property function instead: ${ example }.`)];
       errors.push(...validateExpression({
           key: options.key,
           value: condition,
-          // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-unsafe-member-access
           valueSpec: v8['appearance']['condition'],
           expressionContext: 'appearance'
       }));
@@ -22161,10 +22160,11 @@ Use an identity property function instead: ${ example }.`)];
       }
   }
   function getSourceTypeValues(styleSpec) {
-      return styleSpec.source.reduce((memo, source) => {
+      const sourceArray = styleSpec.source;
+      return sourceArray.reduce((memo, source) => {
           const sourceType = styleSpec[source];
           if (sourceType.type.type === 'enum') {
-              memo = memo.concat(Object.keys(sourceType.type.values));
+              memo = memo.concat(Object.keys(sourceType.type.values || {}));
           }
           return memo;
       }, []);
@@ -22525,7 +22525,6 @@ Use an identity property function instead: ${ example }.`)];
               errors = errors.concat(validate({
                   key,
                   value: projection[key],
-                  // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-unsafe-member-access
                   valueSpec: projectionSpec[key],
                   style,
                   styleSpec
@@ -22666,7 +22665,8 @@ Use an identity property function instead: ${ example }.`)];
           if (elementValidators[elementSpecKey]) {
               continue;
           }
-          if (elementSpecs[elementSpecKey].required && elementSpecs[elementSpecKey]['default'] === void 0 && object[elementSpecKey] === void 0) {
+          const elementSpec = elementSpecs[elementSpecKey];
+          if (elementSpec.required && elementSpec['default'] === void 0 && object[elementSpecKey] === void 0) {
               errors.push(new ValidationError(key, object, `missing required property "${ elementSpecKey }"`));
           }
       }