Merge pull request #1 from 1stdibs/defnotrobbie/FI-1931

fix: FI-1931 update plugin

Author: defnotrobbie

index.js

@@ -35,6 +35,29 @@ class AwsAlias {
 		 */
 		this._stage = this._provider.getStage();
 		this._alias = this._options.alias || this._stage;
+
+		/**
+		 * (new) register custom variable source--preferred method to define
+		 * plugin variables
+		 * @see https://www.serverless.com/framework/docs/guides/plugins/custom-variables
+		 */
+		this.configurationVariablesSources = {
+			awsAlias: {
+				resolve: async ({ address }) =>
+					address === 'alias' ? { value: this._alias } : null,
+			},
+		};
+
+		/**
+		 * register alias property with provider schema to maintain old
+		 * functionality while meeting new variable resolution standards
+		 */
+		this._serverless.configSchemaHandler.schema.properties.provider.properties.alias =
+			{ type: 'string' };
+
+		/**
+		 * (old/bad) set alias property
+		 */
 		this._serverless.service.provider.alias = this._alias;
 
 		/**
@@ -86,12 +109,14 @@ class AwsAlias {
 							alias: {
 								usage: 'Name of the alias',
 								shortcut: 'a',
-								required: true
+								required: true,
+								type: 'string'
 							},
 							verbose: {
 								usage: 'Enable verbose output',
 								shortcut: 'v',
-								required: false
+								required: false,
+								type: 'boolean'
 							}
 						}
 					}

package.json

@@ -12,6 +12,12 @@
     "eslint": "node node_modules/eslint/bin/eslint.js --ext .js lib"
   },
   "author": "Frank Schmid <[email protected]>",
+  "contributors": [
+    {
+      "name": "Robbie Nichols",
+      "email": "[email protected]"
+    }
+  ],
   "nyc": {
     "exclude": [
       "test/**/*.*"
@@ -30,6 +36,9 @@
     "os": "^0.1.1",
     "semver": "^7.3.4"
   },
+  "peerDependencies": {
+    "serverless": "^2.72.3"
+  },
   "devDependencies": {
     "chai": "^4.2.0",
     "chai-as-promised": "^7.1.1",