Don't send sdk version in the dev build (internal-1924)

* Don't send sdk version in dev build

build/rollup_plugins.js

@@ -16,14 +16,17 @@ import minifyStyleSpec from './rollup_plugin_minify_style_spec.js';
 // Common set of plugins/transformations shared across different rollup
 // builds (main mapboxgl bundle, style-spec package, benchmarks bundle)
 
-export const plugins = ({minified, production, test, bench, keepClassNames}) => [
+export const plugins = ({mode, minified, production, test, bench, keepClassNames}) => [
     minifyStyleSpec(),
     esbuild({
         // We target `esnext` and disable minification so esbuild
         // doesn't transform the code, which we'll minify later with the terser
         target: 'esnext',
         minify: false,
         sourceMap: true,
+        define: {
+            'import.meta.env': JSON.stringify({mode}),
+        },
     }),
     json({
         exclude: 'src/style-spec/reference/v8.json'

rollup.config.csp.js

@@ -15,7 +15,7 @@ const config = (input, file, format) => ({
         banner
     },
     treeshake: true,
-    plugins: plugins({minified: true, production: true, keepClassNames: true, test: false, bench: false})
+    plugins: plugins({minified: true, production: true, keepClassNames: true, test: false, bench: false, mode: 'production'})
 });
 
 export default [

rollup.config.js

@@ -52,7 +52,7 @@ export default ({watch}) => {
             },
             preset: "recommended"
         } : false,
-        plugins: plugins({minified, production, bench, test: false, keepClassNames: false})
+        plugins: plugins({minified, production, bench, test: false, keepClassNames: false, mode: BUILD})
     }, {
         // Next, bundle together the three "chunks" produced in the previous pass
         // into a single, final bundle. See rollup/bundle_prelude.js and

src/types/import-meta.d.ts

@@ -0,0 +1,6 @@
+interface ImportMeta {
+    env: {
+        mode?: 'dev' | 'production' | 'bench';
+        [key: string]: unknown;
+    };
+}

src/util/mapbox.ts

@@ -26,6 +26,7 @@ import type {RequestParameters, ResourceType as ResourceTypeEnum} from './ajax';
 import type {Cancelable} from '../types/cancelable';
 import type {TileJSON} from '../types/tilejson';
 import type {Map as MapboxMap} from "../ui/map";
+import '../types/import-meta.d';
 
 export type ResourceType = keyof typeof ResourceTypeEnum;
 export type RequestTransformFunction = (url: string, resourceTypeEnum?: ResourceType) => RequestParameters;
@@ -76,7 +77,9 @@ export class RequestManager {
     normalizeStyleURL(url: string, accessToken?: string): string {
         if (!isMapboxURL(url)) return url;
         const urlObject = parseUrl(url);
-        urlObject.params.push(`sdk=js-${sdkVersion}`);
+        if (import.meta.env.mode !== 'dev') {
+            urlObject.params.push(`sdk=js-${sdkVersion}`);
+        }
         urlObject.path = `/styles/v1${urlObject.path}`;
         return this._makeAPIURL(urlObject, this._customAccessToken || accessToken);
     }

tsconfig.json

@@ -36,7 +36,7 @@
     "./test/usvg/**/*",
     "./test/util/**/*",
     "./vitest.config.*",
-    "./rollup.config.*",
+    "./rollup.config.*"
   ],
   "exclude": [
     "./dist",