@forge/manifest 10.0.1-next.7-experimental-effab31 → 10.1.0-next.8-experimental-05d33ea

package/CHANGELOG.md

@@ -1,10 +1,11 @@
 # @forge/manifest
 
-## 10.0.1-next.7-experimental-effab31
+## 10.1.0-next.8-experimental-05d33ea
 
 ### Minor Changes
 
 - effab31: Bumped cheerio from 0.22 to 1.1
+- 46daa27: Added support for default environment variable declaration in the manifest
 
 ### Patch Changes
 
@@ -17,8 +18,15 @@
 - 1479d2d: Update manifest definitions
 - ca7e661: Update manifest definitions
 - 5ab2c49: Update manifest definitions
+- 609f68b: Update manifest definitions
 - Updated dependencies [195411c]
-  - @forge/i18n@0.0.7-next.0-experimental-effab31
+  - @forge/i18n@0.0.7-next.0-experimental-05d33ea
+
+## 10.1.0-next.8
+
+### Minor Changes
+
+- 46daa27: Added support for default environment variable declaration in the manifest
 
 ## 10.0.1-next.7
 

package/out/interpolator/environment-variable-interpolator.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"environment-variable-interpolator.d.ts","sourceRoot":"","sources":["../../src/interpolator/environment-variable-interpolator.ts"],"names":[],"mappings":"AACA,OAAO,EACL,gBAAgB,EAChB,gBAAgB,EAChB,oBAAoB,EACpB,yBAAyB,EAC1B,MAAM,yBAAyB,CAAC;AACjC,OAAO,EAAE,cAAc,EAAE,MAAM,oBAAoB,CAAC;AAIpD,qBAAa,oCAAqC,SAAQ,yBAAyB;gBACrE,OAAO,EAAE,MAAM;CAI5B;AAKD,qBAAa,+BAAgC,YAAW,oBAAoB;IAS9D,OAAO,CAAC,QAAQ,CAAC,aAAa;IAR1C,IAAI,EAAE,gBAAgB,CAAyC;IAI/D,6BAA6B,SAAqC;IAElE,OAAO,CAAC,gBAAgB,CAAW;gBAEN,aAAa,GAAE,MAAM,CAAC,MAAM,EAAE,MAAM,GAAG,SAAS,CAAM;IAInF,WAAW,CAAC,CAAC,EAAE,EAAE,QAAQ,EAAE,EAAE,gBAAgB,CAAC,CAAC,CAAC,GAAG,cAAc;IAKjE,OAAO,CAAC,0BAA0B,CAkBhC;CACH"}
+{"version":3,"file":"environment-variable-interpolator.d.ts","sourceRoot":"","sources":["../../src/interpolator/environment-variable-interpolator.ts"],"names":[],"mappings":"AACA,OAAO,EACL,gBAAgB,EAChB,gBAAgB,EAChB,oBAAoB,EACpB,yBAAyB,EAC1B,MAAM,yBAAyB,CAAC;AACjC,OAAO,EAAE,cAAc,EAAE,MAAM,oBAAoB,CAAC;AAIpD,qBAAa,oCAAqC,SAAQ,yBAAyB;gBACrE,OAAO,EAAE,MAAM;CAI5B;AAUD,qBAAa,+BAAgC,YAAW,oBAAoB;IAS9D,OAAO,CAAC,QAAQ,CAAC,aAAa;IAR1C,IAAI,EAAE,gBAAgB,CAAyC;IAI/D,6BAA6B,SAAqC;IAElE,OAAO,CAAC,gBAAgB,CAAsB;gBAEjB,aAAa,GAAE,MAAM,CAAC,MAAM,EAAE,MAAM,GAAG,SAAS,CAAM;IAInF,WAAW,CAAC,CAAC,EAAE,EAAE,QAAQ,EAAE,EAAE,gBAAgB,CAAC,CAAC,CAAC,GAAG,cAAc;IAmBjE,OAAO,CAAC,0BAA0B,CAoBhC;CACH"}

package/out/interpolator/environment-variable-interpolator.js

@@ -23,7 +23,22 @@ class EnvironmentVariableInterpolator {
         this.envVarsToReplace = [];
     }
     interpolate({ manifest }) {
-        this.envVarsToReplace = manifest.environment?.variables ?? [];
+        this.envVarsToReplace = [];
+        if (manifest.environment && manifest.environment.variables) {
+            this.envVarsToReplace = manifest.environment.variables.map((manifestEnvVar) => {
+                if (typeof manifestEnvVar === 'string') {
+                    return {
+                        key: manifestEnvVar
+                    };
+                }
+                else {
+                    return {
+                        key: manifestEnvVar.key,
+                        default: manifestEnvVar.default
+                    };
+                }
+            });
+        }
         return (0, yaml_1.parse)(JSON.stringify(manifest), this.environmentVariableReviver);
     }
     environmentVariableReviver = (_, value) => {
@@ -31,10 +46,11 @@ class EnvironmentVariableInterpolator {
             return value;
         }
         value = value.replace(this.POSSIBLE_ENVIRONMENT_VARIABLE, (original, envVarName) => {
-            if (!this.envVarsToReplace.includes(envVarName)) {
+            const envVarToReplace = this.envVarsToReplace.find((envVar) => envVar.key === envVarName);
+            if (!envVarToReplace) {
                 return original;
             }
-            const envVar = this.envVarsRecord[envVarName];
+            const envVar = this.envVarsRecord[envVarName] ?? envVarToReplace.default;
             if (envVar === undefined) {
                 throw new EnvironmentVariableInterpolatorError(text_1.errors.schema.missingEnvironmentVariable(envVarName));
             }

package/out/schema/manifest-schema.json

@@ -27020,9 +27020,22 @@
                                 "type": "string",
                                 "description": "The name of the connector."
                             },
-                            "icon": {
-                                "type": "string",
-                                "description": "The icon for the connector."
+                            "icons": {
+                                "type": "object",
+                                "properties": {
+                                    "light": {
+                                        "type": "string",
+                                        "description": "The icon for the connector when in Light mode."
+                                    },
+                                    "dark": {
+                                        "type": "string",
+                                        "description": "The icon for the connector when in Dark mode."
+                                    }
+                                },
+                                "required": [
+                                    "light",
+                                    "dark"
+                                ]
                             },
                             "entityTypes": {
                                 "type": "array",
@@ -27077,7 +27090,7 @@
                         },
                         "required": [
                             "name",
-                            "icon",
+                            "icons",
                             "entityTypes",
                             "key"
                         ],
@@ -27676,8 +27689,16 @@
                     "title": "variables",
                     "description": "Manifest environment variables",
                     "items": {
-                        "type": "string",
-                        "pattern": "^[a-zA-Z_][a-zA-Z0-9_]*$"
+                        "oneOf": [
+                            {
+                                "$ref": "#/definitions/EnvVarString",
+                                "title": "EnvVarString"
+                            },
+                            {
+                                "$ref": "#/definitions/EnvVarObject",
+                                "title": "EnvVarObject"
+                            }
+                        ]
                     }
                 }
             }
@@ -27835,6 +27856,29 @@
                 }
             }
         },
+        "EnvVarString": {
+            "type": "string",
+            "pattern": "^[a-zA-Z_][a-zA-Z0-9_]*$",
+            "description": "An environment variable name, which can be used in the manifest"
+        },
+        "EnvVarObject": {
+            "type": "object",
+            "additionalProperties": false,
+            "required": [
+                "key",
+                "default"
+            ],
+            "properties": {
+                "key": {
+                    "$ref": "#/definitions/EnvVarString",
+                    "description": "The name of the environment variable"
+                },
+                "default": {
+                    "type": "string",
+                    "description": "The default value of the environment variable"
+                }
+            }
+        },
         "AuthProviderPredefined": {
             "type": "object",
             "additionalProperties": false,

package/out/schema/manifest.d.ts

@@ -721,10 +721,14 @@ export type OAuth2ScopeName = string;
  * External authentication providers
  */
 export type Auth = (AuthProviderCustom | AuthProviderPredefined)[];
+/**
+ * An environment variable name, which can be used in the manifest
+ */
+export type EnvVarString = string;
 /**
  * Manifest environment variables
  */
-export type Variables = string[];
+export type Variables = (EnvVarString | EnvVarObject)[];
 export type TranslationsResources = [
   {
     key: ForgeSupportedLocaleCode;
@@ -23223,10 +23227,17 @@ export interface Modules {
        * The name of the connector.
        */
       name: string;
-      /**
-       * The icon for the connector.
-       */
-      icon: string;
+      icons: {
+        /**
+         * The icon for the connector when in Light mode.
+         */
+        light: string;
+        /**
+         * The icon for the connector when in Dark mode.
+         */
+        dark: string;
+        [k: string]: unknown;
+      };
       /**
        * Entities that the connector can provide.
        */
@@ -23291,10 +23302,17 @@ export interface Modules {
        * The name of the connector.
        */
       name: string;
-      /**
-       * The icon for the connector.
-       */
-      icon: string;
+      icons: {
+        /**
+         * The icon for the connector when in Light mode.
+         */
+        light: string;
+        /**
+         * The icon for the connector when in Dark mode.
+         */
+        dark: string;
+        [k: string]: unknown;
+      };
       /**
        * Entities that the connector can provide.
        */
@@ -72678,6 +72696,16 @@ export interface AuthProviderPredefined {
 export interface Environment {
   variables?: Variables;
 }
+export interface EnvVarObject {
+  /**
+   * The name of the environment variable
+   */
+  key: string;
+  /**
+   * The default value of the environment variable
+   */
+  default: string;
+}
 export interface Translations {
   resources: TranslationsResources;
   fallback: TranslationsFallback;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@forge/manifest",
-  "version": "10.0.1-next.7-experimental-effab31",
+  "version": "10.1.0-next.8-experimental-05d33ea",
   "description": "Definitions and validations of the Forge manifest",
   "main": "out/index.js",
   "scripts": {
@@ -24,7 +24,7 @@
   "author": "Atlassian",
   "license": "SEE LICENSE IN LICENSE.txt",
   "dependencies": {
-    "@forge/i18n": "0.0.7-next.0-experimental-effab31",
+    "@forge/i18n": "0.0.7-next.0-experimental-05d33ea",
     "@sentry/node": "7.106.0",
     "ajv": "^8.12.0",
     "ajv-formats": "2.1.1",