Add encoding field to vector sources (#1251)

* Add encoding field to vector sources

* formatting

* fix copy/paste error

* Use integration instead of unit tests

* apply review suggestions

* Add issue references

* remove comment

* Adjust references to appease mkdocs

* indicate existing support

* add test case

Author: TimSylvester

CHANGELOG.md

@@ -1,6 +1,7 @@
 ## main
 
 ### ✨ Features and improvements
+- Add optional `encoding` property to vector `source` to allow for MapLibre Tile vector sources
 - _...Add new stuff here..._
 
 ### 🐞 Bug fixes

src/reference/v8.json

@@ -364,6 +364,31 @@
         }
       }
     },
+    "encoding": {
+      "type": "enum",
+      "values": {
+        "mvt": {
+          "doc": "Mapbox Vector Tiles. See http://github.com/mapbox/vector-tile-spec for more info."
+        },
+        "mlt": {
+          "doc": "MapLibre Vector Tiles. See https://github.com/maplibre/maplibre-tile-spec for more info."
+        }
+      },
+      "default": "mvt",
+      "doc": "The encoding used by this source. Mapbox Vector Tiles encoding is used by default.",
+      "sdk-support": {
+        "mvt": {
+          "android": "supported",
+          "ios": "supported",
+          "js": "supported"
+        },
+        "mlt": {
+          "android": "https://github.com/maplibre/maplibre-native/issues/3721",
+          "ios": "https://github.com/maplibre/maplibre-native/issues/3721",
+          "js": "https://github.com/maplibre/maplibre-gl-js/issues/6258"
+        }
+      }
+    },
     "*": {
       "type": "*",
       "doc": "Other keys to configure the data source."

test/integration/style-spec/tests/sources.input.json

@@ -64,6 +64,47 @@
       "tiles": ["http://example.com/{x}/{y}/{z}.png"],
       "valueOf": null,
       "toLocaleString": 6
+    },
+    "source-vector-default": {
+      "type": "vector",
+      "url": "https://demotiles.maplibre.org/tiles/tiles.json"
+    },
+    "source-vector-mvt": {
+      "type": "vector",
+      "url": "https://demotiles.maplibre.org/tiles/tiles.json",
+      "encoding": "mvt"
+    },
+    "source-vector-mlt-tilejson": {
+      "type": "vector",
+      "url": "https://demotiles.maplibre.org/tiles/tiles.json",
+      "encoding": "mlt"
+    },
+    "source-vector-mlt-tiles": {
+      "type": "vector",
+      "url": "https://demotiles.maplibre.org/tiles/tiles/{z}/{x}/{y}.mlt",
+      "encoding": "mlt"
+    },
+    "source-vector-custom": {
+      "type": "vector",
+      "url": "https://demotiles.maplibre.org/tiles/tiles.json",
+      "encoding": "custom"
+    },
+    "source-vector-blank": {
+      "type": "vector",
+      "url": "https://demotiles.maplibre.org/tiles/tiles.json",
+      "encoding": ""
+    },
+    "source-vector-numeric": {
+      "type": "vector",
+      "url": "https://demotiles.maplibre.org/tiles/tiles.json",
+      "encoding": 3
+    },
+    "source-vector-array": {
+      "type": "vector",
+      "url": "https://demotiles.maplibre.org/tiles/tiles.json",
+      "encoding": [
+        "mvt"
+      ]
     }
   },
   "layers": []

test/integration/style-spec/tests/sources.output.json

@@ -38,5 +38,21 @@
   {
     "message": "sources.cluster-properties.state.map: \"zoom\" and \"feature-state\" expressions are not supported with cluster properties.",
     "line": 50
+  },
+  {
+    "message": "sources.source-vector-custom.encoding: expected one of [mvt, mlt], \"custom\" found",
+    "line": 90
+  },
+  {
+    "message": "sources.source-vector-blank.encoding: expected one of [mvt, mlt], \"\" found",
+    "line": 95
+  },
+  {
+    "message": "sources.source-vector-numeric.encoding: expected one of [mvt, mlt], 3 found",
+    "line": 100
+  },
+  {
+    "message": "sources.source-vector-array.encoding: expected one of [mvt, mlt], [\"mvt\"] found",
+    "line": 105
   }
 ]