@gradientedge/cdk-utils 8.121.0 → 8.123.0

package/src/lib/aws/construct/graphql-api-lambda/main.ts

@@ -5,6 +5,7 @@ import { AssetCode, IFunction, ILayerVersion, LayerVersion } from 'aws-cdk-lib/a
 import { IHostedZone } from 'aws-cdk-lib/aws-route53'
 import { ISecret } from 'aws-cdk-lib/aws-secretsmanager'
 import { Construct } from 'constructs'
+import _ from 'lodash'
 import { CommonConstruct } from '../../common'
 import { GraphQlApiLambdaEnvironment, GraphQlApiLambdaProps } from './types'
 
@@ -151,7 +152,7 @@ export class GraphQLApiLambda extends CommonConstruct {
 
     if (!this.props.graphqlApiLambdaLayerSources) return
 
-    this.props.graphqlApiLambdaLayerSources.forEach((source: AssetCode, index: number) => {
+    _.forEach(this.props.graphqlApiLambdaLayerSources, (source: AssetCode, index: number) => {
       layers.push(this.lambdaManager.createLambdaLayer(`${this.id}-layer-${index}`, this, source))
     })
 
@@ -205,8 +206,8 @@ export class GraphQLApiLambda extends CommonConstruct {
    */
   protected createApiBasePathMapping() {
     const apiRootPaths = this.props.apiRootPaths
-    if (apiRootPaths && apiRootPaths.length > 0) {
-      apiRootPaths.forEach((apiRootPath: string) => {
+    if (apiRootPaths && !_.isEmpty(apiRootPaths)) {
+      _.forEach(apiRootPaths, (apiRootPath: string) => {
         this.graphQLApiBasePathMappings.push(
           new BasePathMapping(this, `${this.id}-base-bath-mapping-${apiRootPath}`, {
             basePath: apiRootPath,

package/src/lib/aws/construct/graphql-api-lambda-with-cache/main.ts

@@ -3,6 +3,7 @@ import { ISecurityGroup, IVpc, Peer, Port, SecurityGroup } from 'aws-cdk-lib/aws
 import { CfnReplicationGroup } from 'aws-cdk-lib/aws-elasticache'
 import { ManagedPolicy } from 'aws-cdk-lib/aws-iam'
 import { Construct } from 'constructs'
+import _ from 'lodash'
 import { GraphQLApiLambda } from '..'
 import * as utils from '../../utils'
 import { GraphQlApiLambdaWithCacheProps } from './types'
@@ -91,7 +92,7 @@ export class GraphQLApiLambdaWithCache extends GraphQLApiLambda {
       `${this.id}-elasticache`,
       this,
       this.props.graphQLElastiCache,
-      this.graphQLVpc.privateSubnets.map(subnet => subnet.subnetId),
+      _.map(this.graphQLVpc.privateSubnets, subnet => subnet.subnetId),
       [this.graphQLSecurityGroup.securityGroupId]
     )
   }

package/src/lib/aws/construct/lambda-with-iam-access/main.ts

@@ -4,6 +4,7 @@ import { CfnAccessKey, Policy, PolicyDocument, PolicyStatement, Role, User } fro
 import { AssetCode, IFunction, ILayerVersion, LayerVersion } from 'aws-cdk-lib/aws-lambda'
 import { CfnSecret, Secret } from 'aws-cdk-lib/aws-secretsmanager'
 import { Construct } from 'constructs'
+import _ from 'lodash'
 import { CommonConstruct } from '../../common'
 import { LambdaWithIamAccessEnvironment, LambdaWithIamAccessProps } from './types'
 
@@ -112,7 +113,7 @@ export class LambdaWithIamAccess extends CommonConstruct {
 
     if (!this.props.lambdaLayerSources) return
 
-    this.props.lambdaLayerSources.forEach((source: AssetCode, index: number) => {
+    _.forEach(this.props.lambdaLayerSources, (source: AssetCode, index: number) => {
       layers.push(this.lambdaManager.createLambdaLayer(`${this.id}-layer-${index}`, this, source))
     })
 

package/src/lib/aws/construct/rest-api-lambda/main.ts

@@ -5,6 +5,7 @@ import { AssetCode, IFunction, ILayerVersion, LayerVersion } from 'aws-cdk-lib/a
 import { IHostedZone } from 'aws-cdk-lib/aws-route53'
 import { ISecret } from 'aws-cdk-lib/aws-secretsmanager'
 import { Construct } from 'constructs'
+import _ from 'lodash'
 import { CommonConstruct } from '../../common'
 import { RestApiLambdaEnvironment, RestApiLambdaProps } from './types'
 
@@ -152,7 +153,7 @@ export abstract class RestApiLambda extends CommonConstruct {
 
     if (!this.props.restApiLambdaLayerSources) return
 
-    this.props.restApiLambdaLayerSources.forEach((source: AssetCode, index: number) => {
+    _.forEach(this.props.restApiLambdaLayerSources, (source: AssetCode, index: number) => {
       layers.push(this.lambdaManager.createLambdaLayer(`${this.id}-layer-${index}`, this, source))
     })
 
@@ -208,8 +209,8 @@ export abstract class RestApiLambda extends CommonConstruct {
    */
   protected createApiBasePathMapping() {
     const apiRootPaths = this.props.apiRootPaths
-    if (apiRootPaths && apiRootPaths.length > 0) {
-      apiRootPaths.forEach((apiRootPath: string) => {
+    if (apiRootPaths && !_.isEmpty(apiRootPaths)) {
+      _.forEach(apiRootPaths, (apiRootPath: string) => {
         this.restApiBasePathMappings.push(
           new BasePathMapping(this, `${this.id}-base-bath-mapping-${apiRootPath}`, {
             basePath: apiRootPath,

package/src/lib/aws/construct/rest-api-lambda-with-cache/main.ts

@@ -3,6 +3,7 @@ import { ISecurityGroup, IVpc, Peer, Port, SecurityGroup } from 'aws-cdk-lib/aws
 import { CfnReplicationGroup } from 'aws-cdk-lib/aws-elasticache'
 import { ManagedPolicy } from 'aws-cdk-lib/aws-iam'
 import { Construct } from 'constructs'
+import _ from 'lodash'
 import { RestApiLambda } from '..'
 import { createCfnOutput } from '../../utils'
 import { RestApiLambdaWithCacheProps } from './types'
@@ -94,7 +95,7 @@ export abstract class RestApiLambdaWithCache extends RestApiLambda {
       `${this.id}-elasticache`,
       this,
       this.props.restApiCache,
-      this.restApivpc.privateSubnets.map(subnet => subnet.subnetId),
+      _.map(this.restApivpc.privateSubnets, subnet => subnet.subnetId),
       [this.restApiSecurityGroup.securityGroupId]
     )
 

package/src/lib/aws/construct/site-with-ecs-backend/main.ts

@@ -405,15 +405,14 @@ export class SiteWithEcsBackend extends CommonConstruct {
         name: `${this.id}-fs`,
       })
 
-      if (this.props.siteTask.mountPoints && this.props.siteTask.mountPoints.length > 0) {
-        this.props.siteTask.mountPoints.forEach(
-          mountPoint =>
-            this.siteEcsTaskDefinition.defaultContainer?.addMountPoints({
-              containerPath: mountPoint.containerPath,
-              readOnly: mountPoint.readOnly,
-              sourceVolume: `${this.id}-fs`,
-            })
-        )
+      if (this.props.siteTask.mountPoints && !_.isEmpty(this.props.siteTask.mountPoints)) {
+        _.forEach(this.props.siteTask.mountPoints, mountPoint => {
+          this.siteEcsTaskDefinition.defaultContainer?.addMountPoints({
+            containerPath: mountPoint.containerPath,
+            readOnly: mountPoint.readOnly,
+            sourceVolume: `${this.id}-fs`,
+          })
+        })
       }
     }
 

package/src/lib/aws/construct/static-asset-deployment/main.ts

@@ -1,6 +1,7 @@
 import { IBucket } from 'aws-cdk-lib/aws-s3'
 import { BucketDeployment } from 'aws-cdk-lib/aws-s3-deployment'
 import { Construct } from 'constructs'
+import _ from 'lodash'
 import { CommonConstruct } from '../../common'
 import { StaticAssetDeploymentProps } from './types'
 
@@ -62,7 +63,7 @@ export class StaticAssetDeployment extends CommonConstruct {
     if (!staticAssetsForExport) return
 
     /* optional additional exports needed for asset urls */
-    staticAssetsForExport.forEach(asset => {
+    _.forEach(staticAssetsForExport, asset => {
       this.addCfnOutput(asset.key, this.staticAssetBucket.s3UrlForObject(asset.value))
     })
   }

package/src/lib/aws/services/api-gateway/main.ts

@@ -13,6 +13,7 @@ import {
 } from 'aws-cdk-lib/aws-apigateway'
 import { ICertificate } from 'aws-cdk-lib/aws-certificatemanager'
 import { IFunction } from 'aws-cdk-lib/aws-lambda'
+import _ from 'lodash'
 import { CommonConstruct } from '../../common'
 import { createCfnOutput } from '../../utils'
 import { LambdaRestApiProps } from './types'
@@ -83,8 +84,8 @@ export class ApiManager {
       retainDeployments: props.retainDeployments,
     })
 
-    if (props.tags && props.tags.length > 0) {
-      props.tags.forEach(tag => {
+    if (props.tags && !_.isEmpty(props.tags)) {
+      _.forEach(props.tags, tag => {
         Tags.of(api).add(tag.key, tag.value)
       })
     }
@@ -153,12 +154,13 @@ export class ApiManager {
         allowOrigins: allowedOrigins ?? Cors.ALL_ORIGINS,
       },
     })
-    methods.forEach(method =>
+
+    _.forEach(methods, method => {
       resource.addMethod(method, integration, {
         authorizer,
         requestParameters: methodRequestParameters,
       })
-    )
+    })
     createCfnOutput(`${id}-${path}ResourceId`, scope, resource.resourceId)
 
     if (addProxy) {
@@ -170,12 +172,13 @@ export class ApiManager {
           allowOrigins: allowedOrigins ?? Cors.ALL_ORIGINS,
         },
       })
-      methods.forEach(method =>
+
+      _.forEach(methods, method => {
         resourceProxy.addMethod(method, proxyIntegration ?? integration, {
           authorizer,
           requestParameters: methodRequestParameters,
         })
-      )
+      })
       createCfnOutput(`${id}-${path}ProxyResourceId`, scope, resourceProxy.resourceId)
     }
 

package/src/lib/aws/services/cloudfront/main.ts

@@ -26,6 +26,7 @@ import { CommonConstruct, CommonStack } from '../../common'
 import { createCfnOutput } from '../../utils'
 import { LambdaEdgeProps } from '../lambda'
 import { CloudFrontProps, CloudfrontFunctionProps, DistributionProps } from './types'
+import _ from 'lodash'
 
 /**
  * @classdesc Provides operations on AWS
@@ -115,8 +116,8 @@ export class CloudFrontManager {
       webACLId: props.webACLId,
     })
 
-    if (props.tags && props.tags.length > 0) {
-      props.tags.forEach(tag => {
+    if (props.tags && !_.isEmpty(props.tags)) {
+      _.forEach(props.tags, tag => {
         Tags.of(distribution).add(tag.key, tag.value)
       })
     }
@@ -180,8 +181,8 @@ export class CloudFrontManager {
       webAclId: props.webAclId,
     })
 
-    if (props.tags && props.tags.length > 0) {
-      props.tags.forEach(tag => {
+    if (props.tags && !_.isEmpty(props.tags)) {
+      _.forEach(props.tags, tag => {
         Tags.of(distribution).add(tag.key, tag.value)
       })
     }
@@ -244,8 +245,8 @@ export class CloudFrontManager {
       webAclId: props.webAclId,
     })
 
-    if (props.tags && props.tags.length > 0) {
-      props.tags.forEach(tag => {
+    if (props.tags && !_.isEmpty(props.tags)) {
+      _.forEach(props.tags, tag => {
         Tags.of(distribution).add(tag.key, tag.value)
       })
     }
@@ -306,7 +307,7 @@ export class CloudFrontManager {
       vpc: vpc,
     })
 
-    if (props.tags && props.tags.length > 0) {
+    if (props.tags && !_.isEmpty(props.tags)) {
       props.tags.forEach(tag => {
         Tags.of(edgeFunction).add(tag.key, tag.value)
       })

package/src/lib/aws/services/cloudwatch/logs.ts

@@ -1,7 +1,8 @@
 import * as cdk from 'aws-cdk-lib'
 import * as logs from 'aws-cdk-lib/aws-logs'
-import * as utils from '../../utils'
+import _ from 'lodash'
 import { CommonConstruct } from '../../common'
+import * as utils from '../../utils'
 import { LogProps, MetricFilterProps } from './types'
 
 /**
@@ -66,8 +67,8 @@ export class LogManager {
       retentionInDays: props.retention,
     })
 
-    if (props.tags && props.tags.length > 0) {
-      props.tags.forEach(tag => {
+    if (props.tags && !_.isEmpty(props.tags)) {
+      _.forEach(props.tags, tag => {
         cdk.Tags.of(logGroup).add(tag.key, tag.value)
       })
     }
@@ -93,8 +94,8 @@ export class LogManager {
       retention: props.retention,
     })
 
-    if (props.tags && props.tags.length > 0) {
-      props.tags.forEach(tag => {
+    if (props.tags && !_.isEmpty(props.tags)) {
+      _.forEach(props.tags, tag => {
         cdk.Tags.of(logGroup).add(tag.key, tag.value)
       })
     }

package/src/lib/aws/services/cloudwatch/main.ts

@@ -1,8 +1,9 @@
 import * as cdk from 'aws-cdk-lib'
 import * as watch from 'aws-cdk-lib/aws-cloudwatch'
 import { IMetric } from 'aws-cdk-lib/aws-cloudwatch'
-import * as utils from '../../utils'
+import _ from 'lodash'
 import { CommonConstruct } from '../../common'
+import * as utils from '../../utils'
 import {
   AlarmProps,
   AlarmStatusWidgetProps,
@@ -56,9 +57,10 @@ export class CloudWatchManager {
     if (!props.metricProps) throw `Could not find metricProps for Alarm props for id:${id}`
 
     const metrics: any = {}
-    this.determineMetrics(scope, props.metricProps).map(
-      (metric: watch.IMetric, index: number) => (metrics[`m${index}`] = metric)
-    )
+    _.map(this.determineMetrics(scope, props.metricProps), (metric: watch.IMetric, index: number) => {
+      metrics[`m${index}`] = metric
+    })
+
     const expression = new watch.MathExpression({
       expression: props.expression,
       period: props.periodInSecs ? cdk.Duration.seconds(props.periodInSecs) : cdk.Duration.minutes(5),
@@ -139,7 +141,9 @@ export class CloudWatchManager {
     if (!props || props.length == 0) throw `Widget props undefined`
 
     const widgets: any = []
-    props.forEach((widgetProps: any) => widgets.push(this.createWidget(widgetProps.id, scope, widgetProps)))
+    _.forEach(props, (widgetProps: any) => {
+      widgets.push(this.createWidget(widgetProps.id, scope, widgetProps))
+    })
 
     return widgets
   }
@@ -188,9 +192,7 @@ export class CloudWatchManager {
     const metricProps: any[] = props.metricProps
     return this.createWidget(id, scope, {
       ...props,
-      ...{
-        metricProps: metricProps.map(metricProp => ({ ...metricProp, ...{ distributionId: distributionId } })),
-      },
+      metricProps: _.map(metricProps, { ..._, distributionId }),
     })
   }
 
@@ -206,9 +208,7 @@ export class CloudWatchManager {
     const metricProps: any[] = props.metricProps
     return this.createWidget(id, scope, {
       ...props,
-      ...{
-        metricProps: metricProps.map(metricProp => ({ ...metricProp, ...{ stateMachineArn: stateMachineArn } })),
-      },
+      metricProps: _.map(metricProps, { ..._, stateMachineArn }),
     })
   }
 
@@ -225,12 +225,7 @@ export class CloudWatchManager {
     const metricProps: any[] = props.metricProps
     return this.createWidget(id, scope, {
       ...props,
-      ...{
-        metricProps: metricProps.map(metricProp => ({
-          ...metricProp,
-          ...{ eventBusName: eventBusName, ruleName: ruleName },
-        })),
-      },
+      metricProps: _.map(metricProps, { ..._, eventBusName, ruleName }),
     })
   }
 
@@ -246,9 +241,7 @@ export class CloudWatchManager {
     const metricProps: any[] = props.metricProps
     return this.createWidget(id, scope, {
       ...props,
-      ...{
-        metricProps: metricProps.map(metricProp => ({ ...metricProp, ...{ apiName: apiName } })),
-      },
+      metricProps: _.map(metricProps, { ..._, apiName }),
     })
   }
 
@@ -264,9 +257,7 @@ export class CloudWatchManager {
     const metricProps: any[] = props.metricProps
     return this.createWidget(id, scope, {
       ...props,
-      ...{
-        metricProps: metricProps.map(metricProp => ({ ...metricProp, ...{ functionName: functionName } })),
-      },
+      metricProps: _.map(metricProps, { ..._, functionName }),
     })
   }
 
@@ -282,9 +273,7 @@ export class CloudWatchManager {
     const metricProps: any[] = props.metricProps
     return this.createWidget(id, scope, {
       ...props,
-      ...{
-        metricProps: metricProps.map(metricProp => ({ ...metricProp, ...{ service: service } })),
-      },
+      metricProps: _.map(metricProps, { ..._, service }),
     })
   }
 
@@ -300,9 +289,7 @@ export class CloudWatchManager {
     const metricProps: any[] = props.metricProps
     return this.createWidget(id, scope, {
       ...props,
-      ...{
-        metricProps: metricProps.map(metricProp => ({ ...metricProp, ...{ clusterName: clusterName } })),
-      },
+      metricProps: _.map(metricProps, { ..._, clusterName }),
     })
   }
 
@@ -325,12 +312,7 @@ export class CloudWatchManager {
     const metricProps: any[] = props.metricProps
     return this.createWidget(id, scope, {
       ...props,
-      ...{
-        metricProps: metricProps.map(metricProp => ({
-          ...metricProp,
-          ...{ clusterName: clusterName, serviceName: serviceName },
-        })),
-      },
+      metricProps: _.map(metricProps, { ..._, clusterName, serviceName }),
     })
   }
 
@@ -346,9 +328,7 @@ export class CloudWatchManager {
     const metricProps: any[] = props.metricProps
     return this.createWidget(id, scope, {
       ...props,
-      ...{
-        metricProps: metricProps.map(metricProp => ({ ...metricProp, ...{ loadBalancer: loadBalancer } })),
-      },
+      metricProps: _.map(metricProps, { ..._, loadBalancer }),
     })
   }
 
@@ -364,9 +344,7 @@ export class CloudWatchManager {
     const metricProps: any[] = props.metricProps
     return this.createWidget(id, scope, {
       ...props,
-      ...{
-        metricProps: metricProps.map(metricProp => ({ ...metricProp, ...{ cacheClusterId: cacheClusterId } })),
-      },
+      metricProps: _.map(metricProps, { ..._, cacheClusterId }),
     })
   }
 
@@ -541,101 +519,77 @@ export class CloudWatchManager {
         if (metricProp.functionName) {
           metricDimensions = {
             ...metricProp.dimensionsMap,
-            ...{
-              FunctionName: `${metricProp.functionName}`,
-            },
+            FunctionName: `${metricProp.functionName}`,
           }
         }
         if (metricProp.serviceName && metricProp.clusterName) {
           metricDimensions = {
             ...metricProp.dimensionsMap,
-            ...{
-              ClusterName: `${metricProp.clusterName}`,
-              ServiceName: `${metricProp.serviceName}`,
-            },
+            ClusterName: `${metricProp.clusterName}`,
+            ServiceName: `${metricProp.serviceName}`,
           }
         }
         if (!metricProp.serviceName && metricProp.clusterName) {
           metricDimensions = {
             ...metricProp.dimensionsMap,
-            ...{
-              ClusterName: `${metricProp.clusterName}`,
-            },
+            ClusterName: `${metricProp.clusterName}`,
           }
         }
         if (metricProp.serviceName && !metricProp.clusterName) {
           metricDimensions = {
             ...metricProp.dimensionsMap,
-            ...{
-              ServiceName: `${metricProp.serviceName}`,
-            },
+            ServiceName: `${metricProp.serviceName}`,
           }
         }
         if (metricProp.loadBalancer) {
           metricDimensions = {
             ...metricProp.dimensionsMap,
-            ...{
-              LoadBalancer: `${metricProp.loadBalancer}`,
-            },
+            LoadBalancer: `${metricProp.loadBalancer}`,
           }
         }
         if (metricProp.service) {
           metricDimensions = {
             ...metricProp.dimensionsMap,
-            ...{
-              service: `${metricProp.service}`,
-            },
+            service: `${metricProp.service}`,
           }
         }
         if (metricProp.distributionId) {
           metricDimensions = {
             ...metricProp.dimensionsMap,
-            ...{
-              DistributionId: `${metricProp.distributionId}`,
-              Region: `Global`,
-            },
+            DistributionId: `${metricProp.distributionId}`,
+            Region: `Global`,
           }
         }
         if (metricProp.apiName) {
           metricDimensions = {
             ...metricProp.dimensionsMap,
-            ...{
-              ApiName: `${metricProp.apiName}`,
-            },
+            ApiName: `${metricProp.apiName}`,
           }
         }
         if (metricProp.cacheClusterId) {
           metricDimensions = {
             ...metricProp.dimensionsMap,
-            ...{
-              CacheClusterId: `${metricProp.cacheClusterId}`,
-            },
+            CacheClusterId: `${metricProp.cacheClusterId}`,
           }
         }
         if (metricProp.dbClusterIdentifier) {
           metricDimensions = {
             ...metricProp.dimensionsMap,
-            ...{
-              DBClusterIdentifier: `${metricProp.dbClusterIdentifier}`,
-            },
+            DBClusterIdentifier: `${metricProp.dbClusterIdentifier}`,
           }
         }
         if (metricProp.stateMachineArn) {
           metricDimensions = {
             ...metricProp.dimensionsMap,
-            ...{
-              StateMachineArn: `${metricProp.stateMachineArn}`,
-            },
+            StateMachineArn: `${metricProp.stateMachineArn}`,
           }
         }
 
         if (metricProp.eventBusName && metricProp.ruleName) {
           metricDimensions = {
             ...metricProp.dimensionsMap,
-            ...{
-              EventBusName: `${metricProp.eventBusName}`,
-              RuleName: `${metricProp.ruleName}`,
-            },
+            EventBusName: `${metricProp.eventBusName}`,
+            RuleName: `${metricProp.ruleName}`,
           }
         }
         const metric = new watch.Metric({
@@ -679,7 +633,7 @@ export class CloudWatchManager {
   private determineAlarms(id: string, scope: CommonConstruct, alarmProps: watch.AlarmProps[]) {
     const alarms: watch.IAlarm[] = []
     if (alarmProps) {
-      alarmProps.forEach((alarmProp: watch.AlarmProps) => {
+      _.forEach(alarmProps, (alarmProp: watch.AlarmProps) => {
         if (!alarmProp.alarmName) throw `Alarm name undefined for ${id}`
         const alarmArn = `arn:aws:cloudwatch:${cdk.Stack.of(scope).region}:${cdk.Stack.of(scope).account}:alarm:${
           alarmProp.alarmName

package/src/lib/aws/services/dynamodb/main.ts

@@ -1,5 +1,6 @@
 import { Tags } from 'aws-cdk-lib'
 import { Table } from 'aws-cdk-lib/aws-dynamodb'
+import _ from 'lodash'
 import { CommonConstruct } from '../../common'
 import { createCfnOutput } from '../../utils'
 import { TableProps } from './types'
@@ -50,8 +51,8 @@ export class DynamodbManager {
       writeCapacity: props.writeCapacity,
     })
 
-    if (props.tags && props.tags.length > 0) {
-      props.tags.forEach(tag => {
+    if (props.tags && !_.isEmpty(props.tags)) {
+      _.forEach(props.tags, tag => {
         Tags.of(table).add(tag.key, tag.value)
       })
     }

package/src/lib/aws/services/elastic-container-service/main.ts

@@ -14,6 +14,7 @@ import {
 import { ApplicationLoadBalancedFargateService } from 'aws-cdk-lib/aws-ecs-patterns'
 import { Role } from 'aws-cdk-lib/aws-iam'
 import { ILogGroup } from 'aws-cdk-lib/aws-logs'
+import _ from 'lodash'
 import { CommonConstruct } from '../../common'
 import { createCfnOutput } from '../../utils'
 import { EcsApplicationLoadBalancedFargateServiceProps, EcsClusterProps, EcsTaskProps } from './types'
@@ -55,8 +56,8 @@ export class EcsManager {
       vpc: vpc,
     })
 
-    if (props.tags && props.tags.length > 0) {
-      props.tags.forEach(tag => {
+    if (props.tags && !_.isEmpty(props.tags)) {
+      _.forEach(props.tags, tag => {
         Tags.of(ecsCluster).add(tag.key, tag.value)
       })
     }
@@ -132,8 +133,8 @@ export class EcsManager {
       secrets: secrets,
     })
 
-    if (props.tags && props.tags.length > 0) {
-      props.tags.forEach(tag => {
+    if (props.tags && !_.isEmpty(props.tags)) {
+      _.forEach(props.tags, tag => {
         Tags.of(ecsTask).add(tag.key, tag.value)
       })
     }

package/src/lib/aws/services/elastic-file-system/main.ts

@@ -1,6 +1,7 @@
 import { RemovalPolicy } from 'aws-cdk-lib'
 import { ISecurityGroup, IVpc, SubnetSelection } from 'aws-cdk-lib/aws-ec2'
 import { FileSystem, LifecyclePolicy, OutOfInfrequentAccessPolicy, PerformanceMode } from 'aws-cdk-lib/aws-efs'
+import _ from 'lodash'
 import { CommonConstruct } from '../../common'
 import { createCfnOutput } from '../../utils'
 import { EfsAccessPointOptions, EfsFileSystemProps } from './types'
@@ -71,7 +72,7 @@ export class EfsManager {
     createCfnOutput(`${id}-fileSystemId`, scope, fileSystem.fileSystemId)
 
     /* provision access points if specified */
-    if (accessPointOptions && accessPointOptions.length > 0) {
+    if (accessPointOptions && !_.isEmpty(accessPointOptions)) {
       for (const [index, accessPointOption] of accessPointOptions.entries()) {
         if (!accessPointOption.path) throw `Undefined access point path for option: [${accessPointOption}], id: [${id}]`
         const accessPoint = fileSystem.addAccessPoint(`${id}-ap-${index}`, {

package/src/lib/aws/services/elasticache/main.ts

@@ -1,5 +1,6 @@
 import { Tags } from 'aws-cdk-lib'
 import { CfnCacheCluster, CfnReplicationGroup, CfnSubnetGroup } from 'aws-cdk-lib/aws-elasticache'
+import _ from 'lodash'
 import { CommonConstruct } from '../../common'
 import { createCfnOutput } from '../../utils'
 import { ElastiCacheProps, ReplicatedElastiCacheProps } from './types'
@@ -80,8 +81,8 @@ export class ElastiCacheManager {
 
     elasticacheCluster.addDependency(subnetGroup)
 
-    if (props.tags && props.tags.length > 0) {
-      props.tags.forEach(tag => {
+    if (props.tags && !_.isEmpty(props.tags)) {
+      _.forEach(props.tags, tag => {
         Tags.of(elasticacheCluster).add(tag.key, tag.value)
       })
     }

package/src/lib/aws/services/eventbridge/main.ts

@@ -6,6 +6,7 @@ import { CfnPermission, IFunction } from 'aws-cdk-lib/aws-lambda'
 import { CfnPipe } from 'aws-cdk-lib/aws-pipes'
 import { IQueue } from 'aws-cdk-lib/aws-sqs'
 import { IStateMachine } from 'aws-cdk-lib/aws-stepfunctions'
+import _ from 'lodash'
 import { CommonConstruct } from '../../common'
 import { createCfnOutput } from '../../utils'
 import { EventBusProps, EventRuleProps, RuleProps, SqsToSfnPipeProps } from './types'
@@ -72,14 +73,14 @@ export class EventManager {
       schedule: props.schedule,
     })
 
-    if (targets && targets.length > 0) {
-      targets.forEach(target => {
+    if (targets && !_.isEmpty(targets)) {
+      _.forEach(targets, target => {
         rule.addTarget(target)
       })
     }
 
-    if (props.tags && props.tags.length > 0) {
-      props.tags.forEach(tag => {
+    if (props.tags && !_.isEmpty(props.tags)) {
+      _.forEach(props.tags, tag => {
         Tags.of(rule).add(tag.key, tag.value)
       })
     }