feat(packages/twilio-run): regionalize toolkit config and api (#433)

* feat(packages/twilio-run): regionalize the .twiliodeployinfo

* feat: regional support for config files

* chore(runtime-handler/packager.json): override a check

* chore: typescript override for lodash types

* refactor: using optional chaining

* fix: passing missing region param to related functions

Author: mbeldartwilio

package.json

@@ -39,7 +39,8 @@
   },
   "overrides": {
     "@types/prettier": "2.6.0",
-    "@types/express-serve-static-core": "ts3.9"
+    "@types/express-serve-static-core": "ts3.9",
+    "@types/lodash": "ts3.9"
   },
   "lint-staged": {
     "*.{js,jsx,ts,tsx}": [

packages/runtime-handler/package.json

@@ -77,7 +77,8 @@
   },
   "overrides": {
     "@types/prettier": "2.6.0",
-    "@types/express-serve-static-core": "ts3.9"
+    "@types/express-serve-static-core": "ts3.9",
+    "@types/lodash": "ts3.9"
   },
   "gitHead": "6db273648ed19474f4125042556b10c051529912"
 }

packages/serverless-api/package.json

@@ -72,7 +72,8 @@
     "upath": "^1.1.2"
   },
   "overrides": {
-    "@types/prettier": "2.6.0"
+    "@types/prettier": "2.6.0",
+    "@types/lodash": "ts3.9"
   },
   "gitHead": "6db273648ed19474f4125042556b10c051529912"
 }

packages/serverless-api/src/api/utils/__tests__/api-client.test.ts

@@ -62,4 +62,18 @@ describe('getApiUrl', () => {
     const url = getApiUrl(DEFAULT_TEST_CLIENT_CONFIG);
     expect(url).toBe('https://serverless.sydney.au2.twilio.com/v1');
   });
+
+  test('handles edge with only eligible region with variable', () => {
+    process.env.TWILIO_REGION = 'au1';
+    const url = getApiUrl(DEFAULT_TEST_CLIENT_CONFIG);
+    expect(url).toBe('https://serverless.sydney.au1.twilio.com/v1');
+  });
+
+  test('handles edge with only eligible region with params', () => {
+    const url = getApiUrl({
+      ...DEFAULT_TEST_CLIENT_CONFIG,
+      region: 'us1',
+    });
+    expect(url).toBe('https://serverless.ashburn.us1.twilio.com/v1');
+  });
 });

packages/serverless-api/src/api/utils/api-client.ts

@@ -1,14 +1,29 @@
 import { ClientConfig } from '../../types';
 
+const regionEdgeMap: { [index: string]: string } = {
+  us1: 'ashburn',
+  au1: 'sydney',
+  ie1: 'dublin',
+  'stage-us1': 'ashburn',
+  'stage-au1': 'sydney',
+};
+
 export function getApiUrl(
   config: ClientConfig,
   product = 'serverless',
   apiVersion = 'v1'
 ): string {
   const configEdge = config.edge || process.env.TWILIO_EDGE;
   const configRegion = config.region || process.env.TWILIO_REGION;
+  const region = configRegion ? `${configRegion}.` : '';
+
+  if (!configEdge && configRegion) {
+    const defaultEdge = regionEdgeMap[configRegion]
+      ? `${regionEdgeMap[configRegion]}.`
+      : '';
+    return `https://${product}.${defaultEdge}${region}twilio.com/${apiVersion}`;
+  }
 
   const edge = configEdge ? `${configEdge}.` : '';
-  const region = configRegion ? `${configRegion}.` : '';
   return `https://${product}.${edge}${region}twilio.com/${apiVersion}`;
 }

packages/twilio-run/__tests__/config/global.test.ts

@@ -145,4 +145,69 @@ describe('readSpecializedConfig', () => {
       env: '.env.stage',
     });
   });
+
+  test('account + region config override', () => {
+    __setTestConfig({
+      serviceSid: 'ZS11112222111122221111222211112222',
+      env: '.env.example',
+      commands: {
+        deploy: {
+          functionsFolder: '/tmp/functions',
+        },
+      },
+      environments: {
+        prod: {
+          serviceSid: 'ZS11112222111122221111222211112223',
+          env: '.env.prod',
+        },
+      },
+      projects: {
+        'AC11112222111122221111222211114444:au1': {
+          serviceSid: 'ZS11112222111122221111222211114444',
+        },
+        'AC11112222111122221111222211114444:ie1': {
+          serviceSid: 'ZS11112222111122221111222211114445',
+        },
+        AC11112222111122221111222211114444: {
+          serviceSid: 'ZS11112222111122221111222211114446',
+        },
+      },
+    });
+
+    expect(
+      readSpecializedConfig('/tmp', '.twilioserverlessrc', 'deploy', {
+        environmentSuffix: 'prod',
+        username: 'AC11112222111122221111222211114444',
+        region: 'ie1',
+      })
+    ).toEqual({
+      serviceSid: 'ZS11112222111122221111222211114445',
+      functionsFolder: '/tmp/functions',
+      env: '.env.prod',
+    });
+
+    expect(
+      readSpecializedConfig('/tmp', '.twilioserverlessrc', 'deploy', {
+        environmentSuffix: 'prod',
+        username: 'AC11112222111122221111222211114444',
+        region: 'au1',
+      })
+    ).toEqual({
+      serviceSid: 'ZS11112222111122221111222211114444',
+      functionsFolder: '/tmp/functions',
+      env: '.env.prod',
+    });
+
+    expect(
+      readSpecializedConfig('/tmp', '.twilioserverlessrc', 'deploy', {
+        environmentSuffix: 'prod',
+        username: 'AC11112222111122221111222211114444',
+        region: 'us1',
+      })
+    ).toEqual({
+      serviceSid: 'ZS11112222111122221111222211114446',
+      functionsFolder: '/tmp/functions',
+      env: '.env.prod',
+    });
+  });
 });

packages/twilio-run/package.json

@@ -126,7 +126,8 @@
     "node": ">=12.22.1"
   },
   "overrides": {
-    "@types/express-serve-static-core": "ts3.9"
+    "@types/express-serve-static-core": "ts3.9",
+    "@types/lodash": "ts3.9"
   },
   "gitHead": "6db273648ed19474f4125042556b10c051529912"
 }

packages/twilio-run/src/commands/deploy.ts

@@ -134,7 +134,8 @@ export async function handler(
       buildSid,
       config.username.startsWith('AC')
         ? config.username
-        : externalCliOptions && externalCliOptions.accountSid
+        : externalCliOptions && externalCliOptions.accountSid,
+      config.region
     );
   } catch (err) {
     handleError(err, spinner, flags, config);

packages/twilio-run/src/config/deploy.ts

@@ -74,6 +74,7 @@ export async function getConfigFromFlags(
       (externalCliOptions && externalCliOptions.accountSid) ||
       undefined,
     environmentSuffix: flags.environment,
+    region: flags.region,
   });
 
   flags = mergeFlagsAndConfig<DeployCliFlags>(configFlags, flags, cliInfo);
@@ -98,7 +99,8 @@ export async function getConfigFromFlags(
         ? flags.username
         : username.startsWith('AC')
         ? username
-        : externalCliOptions?.accountSid
+        : externalCliOptions?.accountSid,
+      flags.region
     ));
 
   const pkgJson = await readPackageJsonContent(flags);

packages/twilio-run/src/config/env/env-get.ts

@@ -49,6 +49,7 @@ export async function getConfigFromFlags(
       (externalCliOptions && externalCliOptions.accountSid) ||
       undefined,
     environmentSuffix: flags.environment,
+    region: flags.region,
   });
 
   flags = mergeFlagsAndConfig<EnvGetFlags>(configFlags, flags, cliInfo);
@@ -71,7 +72,8 @@ export async function getConfigFromFlags(
         ? flags.username
         : username.startsWith('AC')
         ? username
-        : externalCliOptions?.accountSid
+        : externalCliOptions?.accountSid,
+      flags.region
     ));
 
   let serviceName = await getServiceNameFromFlags(flags);