fix: add missing environments field to uiConfigSchema (#1736)

* fix: add missing environment field to uiConfigSchema

* refactor: avoid throwing when unleash-server is a dependency in dev mode

src/lib/openapi/spec/ui-config-schema.test.ts

@@ -7,6 +7,7 @@ test('uiConfigSchema', () => {
         version: 'a',
         unleashUrl: 'a',
         baseUriPath: 'a',
+        environment: 'a',
         disablePasswordAuth: false,
         segmentValuesLimit: 0,
         strategySegmentsLimit: 0,

src/lib/openapi/spec/ui-config-schema.ts

@@ -24,6 +24,9 @@ export const uiConfigSchema = {
         version: {
             type: 'string',
         },
+        environment: {
+            type: 'string',
+        },
         unleashUrl: {
             type: 'string',
         },

src/lib/services/openapi-service.ts

@@ -77,11 +77,7 @@ export class OpenApiService {
         const errors = validateSchema(schema, data);
 
         if (errors) {
-            if (process.env.NODE_ENV === 'development') {
+            this.logger.warn('Invalid response:', errors);
-                throw new Error(JSON.stringify(errors, null, 2));
-            } else {
-                this.logger.warn('Invalid response:', errors);
-            }
         }
 
         res.status(status).json(data);

src/test/e2e/api/openapi/__snapshots__/openapi.e2e.test.ts.snap

@@ -1052,6 +1052,9 @@ Object {
           "disablePasswordAuth": Object {
             "type": "boolean",
           },
+          "environment": Object {
+            "type": "string",
+          },
           "flags": Object {
             "additionalProperties": Object {
               "type": "boolean",