fix(clients): populate sdkId in serviceId and default to use arnNamespace as signingName (#1786)

* fix: move signingName inference to endpointsGenerator

This change also change client serivceId config from arnNamespace
to sdkId

* fix(clients-*): generate sdkId in serviceId and default to use arnNamespace as signingName

Co-authored-by: Trivikram Kamat <[email protected]>

Author: AllanZhengYP

Diff for: clients/client-accessanalyzer/endpoints.ts

@@ -77,5 +77,5 @@ export const defaultRegionInfoProvider: RegionInfoProvider = (region: string, op
         };
       }
   }
-  return Promise.resolve(regionInfo);
+  return Promise.resolve({ signingService: "access-analyzer", ...regionInfo });
 };

Diff for: clients/client-accessanalyzer/runtimeConfig.shared.ts

@@ -9,5 +9,5 @@ export const ClientSharedValues = {
   disableHostPrefix: false,
   logger: {} as __Logger,
   regionInfoProvider: defaultRegionInfoProvider,
-  serviceId: "access-analyzer",
+  serviceId: "AccessAnalyzer",
 };

Diff for: clients/client-acm-pca/endpoints.ts

@@ -197,5 +197,5 @@ export const defaultRegionInfoProvider: RegionInfoProvider = (region: string, op
         };
       }
   }
-  return Promise.resolve(regionInfo);
+  return Promise.resolve({ signingService: "acm-pca", ...regionInfo });
 };

Diff for: clients/client-acm-pca/runtimeConfig.shared.ts

@@ -9,5 +9,5 @@ export const ClientSharedValues = {
   disableHostPrefix: false,
   logger: {} as __Logger,
   regionInfoProvider: defaultRegionInfoProvider,
-  serviceId: "acm-pca",
+  serviceId: "ACM PCA",
 };

Diff for: clients/client-acm/endpoints.ts

@@ -197,5 +197,5 @@ export const defaultRegionInfoProvider: RegionInfoProvider = (region: string, op
         };
       }
   }
-  return Promise.resolve(regionInfo);
+  return Promise.resolve({ signingService: "acm", ...regionInfo });
 };

Diff for: clients/client-acm/runtimeConfig.shared.ts

@@ -9,5 +9,5 @@ export const ClientSharedValues = {
   disableHostPrefix: false,
   logger: {} as __Logger,
   regionInfoProvider: defaultRegionInfoProvider,
-  serviceId: "acm",
+  serviceId: "ACM",
 };

Diff for: clients/client-alexa-for-business/endpoints.ts

@@ -83,5 +83,5 @@ export const defaultRegionInfoProvider: RegionInfoProvider = (region: string, op
         };
       }
   }
-  return Promise.resolve(regionInfo);
+  return Promise.resolve({ signingService: "a4b", ...regionInfo });
 };

Diff for: clients/client-alexa-for-business/runtimeConfig.shared.ts

@@ -9,5 +9,5 @@ export const ClientSharedValues = {
   disableHostPrefix: false,
   logger: {} as __Logger,
   regionInfoProvider: defaultRegionInfoProvider,
-  serviceId: "a4b",
+  serviceId: "Alexa For Business",
 };

Diff for: clients/client-amplify/endpoints.ts

@@ -77,5 +77,5 @@ export const defaultRegionInfoProvider: RegionInfoProvider = (region: string, op
         };
       }
   }
-  return Promise.resolve(regionInfo);
+  return Promise.resolve({ signingService: "amplify", ...regionInfo });
 };

Diff for: clients/client-amplify/runtimeConfig.shared.ts

@@ -9,5 +9,5 @@ export const ClientSharedValues = {
   disableHostPrefix: false,
   logger: {} as __Logger,
   regionInfoProvider: defaultRegionInfoProvider,
-  serviceId: "amplify",
+  serviceId: "Amplify",
 };

Diff for: clients/client-amplifybackend/endpoints.ts

@@ -77,5 +77,5 @@ export const defaultRegionInfoProvider: RegionInfoProvider = (region: string, op
         };
       }
   }
-  return Promise.resolve(regionInfo);
+  return Promise.resolve({ signingService: "amplifybackend", ...regionInfo });
 };

Diff for: clients/client-amplifybackend/runtimeConfig.shared.ts

@@ -9,5 +9,5 @@ export const ClientSharedValues = {
   disableHostPrefix: false,
   logger: {} as __Logger,
   regionInfoProvider: defaultRegionInfoProvider,
-  serviceId: "amplifybackend",
+  serviceId: "AmplifyBackend",
 };

Diff for: clients/client-api-gateway/endpoints.ts

@@ -215,5 +215,5 @@ export const defaultRegionInfoProvider: RegionInfoProvider = (region: string, op
         };
       }
   }
-  return Promise.resolve(regionInfo);
+  return Promise.resolve({ signingService: "apigateway", ...regionInfo });
 };

Diff for: clients/client-api-gateway/runtimeConfig.shared.ts

@@ -9,5 +9,5 @@ export const ClientSharedValues = {
   disableHostPrefix: false,
   logger: {} as __Logger,
   regionInfoProvider: defaultRegionInfoProvider,
-  serviceId: "apigateway",
+  serviceId: "API Gateway",
 };

Diff for: clients/client-apigatewaymanagementapi/endpoints.ts

@@ -77,5 +77,5 @@ export const defaultRegionInfoProvider: RegionInfoProvider = (region: string, op
         };
       }
   }
-  return Promise.resolve(regionInfo);
+  return Promise.resolve({ signingService: "apigateway", ...regionInfo });
 };

Diff for: clients/client-apigatewaymanagementapi/runtimeConfig.shared.ts

@@ -9,5 +9,5 @@ export const ClientSharedValues = {
   disableHostPrefix: false,
   logger: {} as __Logger,
   regionInfoProvider: defaultRegionInfoProvider,
-  serviceId: "apigateway",
+  serviceId: "ApiGatewayManagementApi",
 };

Diff for: clients/client-apigatewayv2/endpoints.ts

@@ -215,5 +215,5 @@ export const defaultRegionInfoProvider: RegionInfoProvider = (region: string, op
         };
       }
   }
-  return Promise.resolve(regionInfo);
+  return Promise.resolve({ signingService: "apigateway", ...regionInfo });
 };

Diff for: clients/client-apigatewayv2/runtimeConfig.shared.ts

@@ -9,5 +9,5 @@ export const ClientSharedValues = {
   disableHostPrefix: false,
   logger: {} as __Logger,
   regionInfoProvider: defaultRegionInfoProvider,
-  serviceId: "apigateway",
+  serviceId: "ApiGatewayV2",
 };

Diff for: clients/client-app-mesh/endpoints.ts

@@ -161,5 +161,5 @@ export const defaultRegionInfoProvider: RegionInfoProvider = (region: string, op
         };
       }
   }
-  return Promise.resolve(regionInfo);
+  return Promise.resolve({ signingService: "appmesh", ...regionInfo });
 };

Diff for: clients/client-app-mesh/runtimeConfig.shared.ts

@@ -9,5 +9,5 @@ export const ClientSharedValues = {
   disableHostPrefix: false,
   logger: {} as __Logger,
   regionInfoProvider: defaultRegionInfoProvider,
-  serviceId: "appmesh",
+  serviceId: "App Mesh",
 };

Diff for: clients/client-appconfig/endpoints.ts

@@ -77,5 +77,5 @@ export const defaultRegionInfoProvider: RegionInfoProvider = (region: string, op
         };
       }
   }
-  return Promise.resolve(regionInfo);
+  return Promise.resolve({ signingService: "appconfig", ...regionInfo });
 };

Diff for: clients/client-appconfig/runtimeConfig.shared.ts

@@ -9,5 +9,5 @@ export const ClientSharedValues = {
   disableHostPrefix: false,
   logger: {} as __Logger,
   regionInfoProvider: defaultRegionInfoProvider,
-  serviceId: "appconfig",
+  serviceId: "AppConfig",
 };

Diff for: clients/client-appflow/endpoints.ts

@@ -77,5 +77,5 @@ export const defaultRegionInfoProvider: RegionInfoProvider = (region: string, op
         };
       }
   }
-  return Promise.resolve(regionInfo);
+  return Promise.resolve({ signingService: "appflow", ...regionInfo });
 };

Diff for: clients/client-appflow/runtimeConfig.shared.ts

@@ -9,5 +9,5 @@ export const ClientSharedValues = {
   disableHostPrefix: false,
   logger: {} as __Logger,
   regionInfoProvider: defaultRegionInfoProvider,
-  serviceId: "appflow",
+  serviceId: "Appflow",
 };

Diff for: clients/client-appintegrations/endpoints.ts

@@ -77,5 +77,5 @@ export const defaultRegionInfoProvider: RegionInfoProvider = (region: string, op
         };
       }
   }
-  return Promise.resolve(regionInfo);
+  return Promise.resolve({ signingService: "app-integrations", ...regionInfo });
 };

Diff for: clients/client-appintegrations/runtimeConfig.shared.ts

@@ -9,5 +9,5 @@ export const ClientSharedValues = {
   disableHostPrefix: false,
   logger: {} as __Logger,
   regionInfoProvider: defaultRegionInfoProvider,
-  serviceId: "app-integrations",
+  serviceId: "AppIntegrations",
 };

Diff for: clients/client-application-auto-scaling/endpoints.ts

@@ -221,5 +221,5 @@ export const defaultRegionInfoProvider: RegionInfoProvider = (region: string, op
         };
       }
   }
-  return Promise.resolve(regionInfo);
+  return Promise.resolve({ signingService: "application-autoscaling", ...regionInfo });
 };

Diff for: clients/client-application-auto-scaling/runtimeConfig.shared.ts

@@ -9,5 +9,5 @@ export const ClientSharedValues = {
   disableHostPrefix: false,
   logger: {} as __Logger,
   regionInfoProvider: defaultRegionInfoProvider,
-  serviceId: "application-autoscaling",
+  serviceId: "Application Auto Scaling",
 };

Diff for: clients/client-application-discovery-service/endpoints.ts

@@ -89,5 +89,5 @@ export const defaultRegionInfoProvider: RegionInfoProvider = (region: string, op
         };
       }
   }
-  return Promise.resolve(regionInfo);
+  return Promise.resolve({ signingService: "discovery", ...regionInfo });
 };

Diff for: clients/client-application-discovery-service/runtimeConfig.shared.ts

@@ -9,5 +9,5 @@ export const ClientSharedValues = {
   disableHostPrefix: false,
   logger: {} as __Logger,
   regionInfoProvider: defaultRegionInfoProvider,
-  serviceId: "discovery",
+  serviceId: "Application Discovery Service",
 };

Diff for: clients/client-application-insights/endpoints.ts

@@ -77,5 +77,5 @@ export const defaultRegionInfoProvider: RegionInfoProvider = (region: string, op
         };
       }
   }
-  return Promise.resolve(regionInfo);
+  return Promise.resolve({ signingService: "applicationinsights", ...regionInfo });
 };

Diff for: clients/client-application-insights/runtimeConfig.shared.ts

@@ -9,5 +9,5 @@ export const ClientSharedValues = {
   disableHostPrefix: false,
   logger: {} as __Logger,
   regionInfoProvider: defaultRegionInfoProvider,
-  serviceId: "applicationinsights",
+  serviceId: "Application Insights",
 };

Diff for: clients/client-appstream/endpoints.ts

@@ -150,5 +150,5 @@ export const defaultRegionInfoProvider: RegionInfoProvider = (region: string, op
         };
       }
   }
-  return Promise.resolve(regionInfo);
+  return Promise.resolve({ signingService: "appstream", ...regionInfo });
 };

Diff for: clients/client-appstream/runtimeConfig.shared.ts

@@ -9,5 +9,5 @@ export const ClientSharedValues = {
   disableHostPrefix: false,
   logger: {} as __Logger,
   regionInfoProvider: defaultRegionInfoProvider,
-  serviceId: "appstream",
+  serviceId: "AppStream",
 };

Diff for: clients/client-appsync/endpoints.ts

@@ -143,5 +143,5 @@ export const defaultRegionInfoProvider: RegionInfoProvider = (region: string, op
         };
       }
   }
-  return Promise.resolve(regionInfo);
+  return Promise.resolve({ signingService: "appsync", ...regionInfo });
 };

Diff for: clients/client-appsync/runtimeConfig.shared.ts

@@ -9,5 +9,5 @@ export const ClientSharedValues = {
   disableHostPrefix: false,
   logger: {} as __Logger,
   regionInfoProvider: defaultRegionInfoProvider,
-  serviceId: "appsync",
+  serviceId: "AppSync",
 };

Diff for: clients/client-athena/endpoints.ts

@@ -191,5 +191,5 @@ export const defaultRegionInfoProvider: RegionInfoProvider = (region: string, op
         };
       }
   }
-  return Promise.resolve(regionInfo);
+  return Promise.resolve({ signingService: "athena", ...regionInfo });
 };

Diff for: clients/client-athena/runtimeConfig.shared.ts

@@ -9,5 +9,5 @@ export const ClientSharedValues = {
   disableHostPrefix: false,
   logger: {} as __Logger,
   regionInfoProvider: defaultRegionInfoProvider,
-  serviceId: "athena",
+  serviceId: "Athena",
 };

Diff for: clients/client-auditmanager/endpoints.ts

@@ -77,5 +77,5 @@ export const defaultRegionInfoProvider: RegionInfoProvider = (region: string, op
         };
       }
   }
-  return Promise.resolve(regionInfo);
+  return Promise.resolve({ signingService: "auditmanager", ...regionInfo });
 };

Diff for: clients/client-auditmanager/runtimeConfig.shared.ts

@@ -9,5 +9,5 @@ export const ClientSharedValues = {
   disableHostPrefix: false,
   logger: {} as __Logger,
   regionInfoProvider: defaultRegionInfoProvider,
-  serviceId: "auditmanager",
+  serviceId: "AuditManager",
 };

Diff for: clients/client-auto-scaling-plans/endpoints.ts

@@ -155,5 +155,5 @@ export const defaultRegionInfoProvider: RegionInfoProvider = (region: string, op
         };
       }
   }
-  return Promise.resolve(regionInfo);
+  return Promise.resolve({ signingService: "autoscaling-plans", ...regionInfo });
 };

Diff for: clients/client-auto-scaling-plans/runtimeConfig.shared.ts

@@ -9,5 +9,5 @@ export const ClientSharedValues = {
   disableHostPrefix: false,
   logger: {} as __Logger,
   regionInfoProvider: defaultRegionInfoProvider,
-  serviceId: "autoscaling-plans",
+  serviceId: "Auto Scaling Plans",
 };

Diff for: clients/client-auto-scaling/endpoints.ts

@@ -221,5 +221,5 @@ export const defaultRegionInfoProvider: RegionInfoProvider = (region: string, op
         };
       }
   }
-  return Promise.resolve(regionInfo);
+  return Promise.resolve({ signingService: "autoscaling", ...regionInfo });
 };

Diff for: clients/client-auto-scaling/runtimeConfig.shared.ts

@@ -9,5 +9,5 @@ export const ClientSharedValues = {
   disableHostPrefix: false,
   logger: {} as __Logger,
   regionInfoProvider: defaultRegionInfoProvider,
-  serviceId: "autoscaling",
+  serviceId: "Auto Scaling",
 };

Diff for: clients/client-backup/endpoints.ts

@@ -185,5 +185,5 @@ export const defaultRegionInfoProvider: RegionInfoProvider = (region: string, op
         };
       }
   }
-  return Promise.resolve(regionInfo);
+  return Promise.resolve({ signingService: "backup", ...regionInfo });
 };

Diff for: clients/client-backup/runtimeConfig.shared.ts

@@ -9,5 +9,5 @@ export const ClientSharedValues = {
   disableHostPrefix: false,
   logger: {} as __Logger,
   regionInfoProvider: defaultRegionInfoProvider,
-  serviceId: "backup",
+  serviceId: "Backup",
 };

Diff for: clients/client-batch/endpoints.ts

@@ -197,5 +197,5 @@ export const defaultRegionInfoProvider: RegionInfoProvider = (region: string, op
         };
       }
   }
-  return Promise.resolve(regionInfo);
+  return Promise.resolve({ signingService: "batch", ...regionInfo });
 };

Diff for: clients/client-batch/runtimeConfig.shared.ts

@@ -9,5 +9,5 @@ export const ClientSharedValues = {
   disableHostPrefix: false,
   logger: {} as __Logger,
   regionInfoProvider: defaultRegionInfoProvider,
-  serviceId: "batch",
+  serviceId: "Batch",
 };

Diff for: clients/client-braket/endpoints.ts

@@ -77,5 +77,5 @@ export const defaultRegionInfoProvider: RegionInfoProvider = (region: string, op
         };
       }
   }
-  return Promise.resolve(regionInfo);
+  return Promise.resolve({ signingService: "braket", ...regionInfo });
 };

Diff for: clients/client-braket/runtimeConfig.shared.ts

@@ -9,5 +9,5 @@ export const ClientSharedValues = {
   disableHostPrefix: false,
   logger: {} as __Logger,
   regionInfoProvider: defaultRegionInfoProvider,
-  serviceId: "braket",
+  serviceId: "Braket",
 };

Diff for: clients/client-budgets/endpoints.ts

@@ -78,5 +78,5 @@ export const defaultRegionInfoProvider: RegionInfoProvider = (region: string, op
         return defaultRegionInfoProvider("aws-global");
       }
   }
-  return Promise.resolve(regionInfo);
+  return Promise.resolve({ signingService: "budgets", ...regionInfo });
 };

Diff for: clients/client-budgets/runtimeConfig.shared.ts

@@ -9,5 +9,5 @@ export const ClientSharedValues = {
   disableHostPrefix: false,
   logger: {} as __Logger,
   regionInfoProvider: defaultRegionInfoProvider,
-  serviceId: "budgets",
+  serviceId: "Budgets",
 };

Diff for: clients/client-chime/endpoints.ts

@@ -78,5 +78,5 @@ export const defaultRegionInfoProvider: RegionInfoProvider = (region: string, op
         return defaultRegionInfoProvider("aws-global");
       }
   }
-  return Promise.resolve(regionInfo);
+  return Promise.resolve({ signingService: "chime", ...regionInfo });
 };

Diff for: clients/client-chime/runtimeConfig.shared.ts

@@ -9,5 +9,5 @@ export const ClientSharedValues = {
   disableHostPrefix: false,
   logger: {} as __Logger,
   regionInfoProvider: defaultRegionInfoProvider,
-  serviceId: "chime",
+  serviceId: "Chime",
 };

Diff for: clients/client-cloud9/endpoints.ts

@@ -119,5 +119,5 @@ export const defaultRegionInfoProvider: RegionInfoProvider = (region: string, op
         };
       }
   }
-  return Promise.resolve(regionInfo);
+  return Promise.resolve({ signingService: "cloud9", ...regionInfo });
 };

Diff for: clients/client-cloud9/runtimeConfig.shared.ts

@@ -9,5 +9,5 @@ export const ClientSharedValues = {
   disableHostPrefix: false,
   logger: {} as __Logger,
   regionInfoProvider: defaultRegionInfoProvider,
-  serviceId: "cloud9",
+  serviceId: "Cloud9",
 };

Diff for: clients/client-clouddirectory/endpoints.ts

@@ -137,5 +137,5 @@ export const defaultRegionInfoProvider: RegionInfoProvider = (region: string, op
         };
       }
   }
-  return Promise.resolve(regionInfo);
+  return Promise.resolve({ signingService: "clouddirectory", ...regionInfo });
 };