run prettier

src/aspects/cfn-include-reporter.test.ts

@@ -22,7 +22,7 @@ describe("CfnIncludeReporter aspect", () => {
     app.synth();
 
     expect(warning).toHaveBeenCalledWith(
-      `As you're migrating a YAML/JSON template to ${LibraryInfo.NAME}, be sure to check for any stateful resources! See https://github.com/guardian/cdk/blob/main/docs/stateful-resources.md.`,
+      `As you're migrating a YAML/JSON template to ${LibraryInfo.NAME}, be sure to check for any stateful resources! See https://github.com/guardian/cdk/blob/main/docs/stateful-resources.md.`
     );
   });
 

src/aspects/cfn-include-reporter.ts

@@ -11,7 +11,7 @@ export class CfnIncludeReporter implements IAspect {
   public visit(node: IConstruct) {
     if (node instanceof CfnInclude) {
       Annotations.of(node).addWarning(
-        `As you're migrating a YAML/JSON template to ${LibraryInfo.NAME}, be sure to check for any stateful resources! See https://github.com/guardian/cdk/blob/main/docs/stateful-resources.md.`,
+        `As you're migrating a YAML/JSON template to ${LibraryInfo.NAME}, be sure to check for any stateful resources! See https://github.com/guardian/cdk/blob/main/docs/stateful-resources.md.`
       );
     }
   }

src/aspects/cfn-parameter-reporter.test.ts

@@ -19,7 +19,7 @@ describe("CfnParameterReporter aspect", () => {
     app.synth();
 
     expect(info).toHaveBeenCalledWith(
-      "Stack reads the SSM Parameter '/TEST/test/my-app/database-password'. Ensure it exists prior to deployment.",
+      "Stack reads the SSM Parameter '/TEST/test/my-app/database-password'. Ensure it exists prior to deployment."
     );
   });
 });

src/aspects/cfn-parameter-reporter.ts

@@ -14,7 +14,7 @@ export class CfnParameterReporter implements IAspect {
 
       if (isSsmParameter && hasDefault) {
         Annotations.of(node.stack).addInfo(
-          `Stack reads the SSM Parameter '${defaultValue}'. Ensure it exists prior to deployment.`,
+          `Stack reads the SSM Parameter '${defaultValue}'. Ensure it exists prior to deployment.`
         );
       }
     }

src/bin/aws-credential-provider.ts

@@ -2,7 +2,7 @@ import type { CredentialProviderChain } from "aws-sdk";
 import AWS from "aws-sdk";
 
 export const awsCredentialProviderChain: (profile: string | undefined) => CredentialProviderChain = (
-  profile: string | undefined,
+  profile: string | undefined
 ) => {
   const envCredentials = () => new AWS.EnvironmentCredentials("AWS");
 

src/bin/commands/account-readiness/index.ts

@@ -18,7 +18,7 @@ export const accountReadinessCommand = async (props: AwsConfig): CliCommandRespo
 
   if (report.parametersFound === 0) {
     console.log(
-      `It looks like this account has not been set up for @guardian/cdk yet. See https://github.com/guardian/aws-account-setup`,
+      `It looks like this account has not been set up for @guardian/cdk yet. See https://github.com/guardian/aws-account-setup`
     );
 
     return 1;

src/bin/commands/account-readiness/ssm.ts

@@ -42,7 +42,7 @@ const ssmParamReadiness = async ({ credentialProvider, region }: AwsConfig): Pro
     });
 
   const ssmParametersMatchingDefaultVpc = ssmParamsFromAccount.filter(
-    ({ Value }) => Value && defaultVpcs.includes(Value),
+    ({ Value }) => Value && defaultVpcs.includes(Value)
   );
 
   const defaultVPCReferencePaths = ssmParametersMatchingDefaultVpc
@@ -77,8 +77,8 @@ export const report = async (props: AwsConfig): Promise<Report> => {
     errs.set(
       "Default VPC referenced in VPC parameters",
       `you should use a custom VPC instead. Default VPC is referenced in paths: ${output.defaultVPCReferences.join(
-        ", ",
-      )}.`,
+        ", "
+      )}.`
     );
   }
 

src/bin/commands/new-project/index.ts

@@ -147,7 +147,7 @@ export const newCdkProject = async (props: NewProjectProps): CliCommandResponse
     ["lib/** bin/**", "--ext .ts", "--no-error-on-unmatched-pattern", "--fix"],
     {
       cwd: config.cdkDir,
-    },
+    }
   );
 
   ux.action.start(chalk.yellow("Running lint check..."));

src/bin/commands/new-project/utils/app.ts

@@ -40,7 +40,7 @@ export class AppBuilder {
 
     stages.forEach((stage) => {
       this.code.line(
-        `new ${appName.pascal}(app, "${appName.pascal}-${stage}", { stack: "${stack.kebab}", stage: "${stage}" });`,
+        `new ${appName.pascal}(app, "${appName.pascal}-${stage}", { stack: "${stack.kebab}", stage: "${stage}" });`
       );
     });
 

src/bin/commands/new-project/utils/snapshot.ts

@@ -48,7 +48,7 @@ export class TestBuilder {
 
     this.code.line("const app = new App();");
     this.code.line(
-      `const stack = new ${appName.pascal}(app, "${appName.pascal}", { stack: "${stackName.kebab}", stage: "TEST" });`,
+      `const stack = new ${appName.pascal}(app, "${appName.pascal}", { stack: "${stackName.kebab}", stage: "TEST" });`
     );
 
     this.code.line(`const template = Template.fromStack(stack);`);

src/bin/commands/new-project/utils/utils.ts

@@ -18,7 +18,7 @@ async function runTask(packageManager: PackageManager, cwd: string, task: string
 
 export function getCommands(
   packageManager: PackageManager,
-  cwd: string,
+  cwd: string
 ): {
   installDependencies: () => Promise<string>;
   lint: () => Promise<string>;

src/bin/index.ts

@@ -29,7 +29,7 @@ const parseCommandLineArguments = () => {
       .command(Commands.AccountReadiness, "Perform checks on an AWS account to see if it is GuCDK ready", (yargs) =>
         yargs
           .option("profile", { type: "string", description: "AWS profile" })
-          .option("region", { type: "string", description: "AWS region", default: "eu-west-1" }),
+          .option("region", { type: "string", description: "AWS region", default: "eu-west-1" })
       )
       .command(
         Commands.New,
@@ -67,14 +67,14 @@ const parseCommandLineArguments = () => {
                 "The Node package manager to use. Match this to the repository (package-lock.json = npm, yarn.lock = yarn). If the repository has neither file, and there is no strong convention in your team, we recommend npm.",
               choices: ["npm", "yarn"],
               demandOption: true,
-            }),
+            })
       )
       .version(
-        `${LibraryInfo.VERSION} (using aws-cdk-lib ${LibraryInfo.AWS_CDK_VERSION}, constructs ${LibraryInfo.CONSTRUCTS_VERSION})`,
+        `${LibraryInfo.VERSION} (using aws-cdk-lib ${LibraryInfo.AWS_CDK_VERSION}, constructs ${LibraryInfo.CONSTRUCTS_VERSION})`
       )
       .demandCommand(1, "") // just print help
       .help()
-      .alias("h", "help").argv,
+      .alias("h", "help").argv
   );
 };
 

src/constants/ssm-parameter-paths.ts

@@ -77,5 +77,5 @@ export const NAMED_SSM_PARAMETER_PATHS: NamedSsmParameterPaths = {
 };
 
 export const ALL_SSM_PARAMETER_PATHS: SsmParameterPath[] = Object.values(
-  NAMED_SSM_PARAMETER_PATHS,
+  NAMED_SSM_PARAMETER_PATHS
 ) as SsmParameterPath[];

src/constructs/autoscaling/asg.ts

@@ -98,7 +98,7 @@ export class GuAutoScalingGroup extends GuAppAwareConstruct(AutoScalingGroup) {
     // Ensure min and max are defined in the same way. Throwing an `Error` when necessary. For example when min is defined via a Mapping, but max is not.
     if (Token.isUnresolved(minimumInstances) && !Token.isUnresolved(maximumInstances)) {
       throw new Error(
-        "minimumInstances is defined via a Mapping, but maximumInstances is not. Create maximumInstances via a Mapping too.",
+        "minimumInstances is defined via a Mapping, but maximumInstances is not. Create maximumInstances via a Mapping too."
       );
     }
 
@@ -131,7 +131,7 @@ export class GuAutoScalingGroup extends GuAppAwareConstruct(AutoScalingGroup) {
     // Note: Launch templates via CDK allow specifying only one SG, so use connections
     // https://github.com/aws/aws-cdk/issues/18712
     [GuWazuhAccess.getInstance(scope, vpc), ...additionalSecurityGroups].forEach((sg) =>
-      launchTemplate.connections.addSecurityGroup(sg),
+      launchTemplate.connections.addSecurityGroup(sg)
     );
 
     const asgProps = {

src/constructs/cloudwatch/alarm.test.ts

@@ -71,7 +71,7 @@ describe("The GuAlarm class", () => {
       {
         OkActions: snsActionsCFN,
       },
-      0,
+      0
     );
   });
 

src/constructs/core/stack.test.ts

@@ -37,7 +37,7 @@ describe("The GuStack construct", () => {
       {
         stack: "test",
         stage: "TEST",
-      },
+      }
     );
 
     new Role(stack, "MyRole", {
@@ -61,7 +61,7 @@ describe("The GuStack construct", () => {
       {
         stack: "test",
         stage: "TEST",
-      },
+      }
     );
 
     expect(stack.repositoryName).toEqual("guardian/my-repository");

src/constructs/core/stack.ts

@@ -179,7 +179,7 @@ export class GuStack extends Stack implements StackStageIdentity {
       return gitUrlParse(repositoryUrl).full_name;
     } catch {
       console.info(
-        `Unable to find git repository name. Set the ${ContextKeys.REPOSITORY_URL} context value or configure a git remote`,
+        `Unable to find git repository name. Set the ${ContextKeys.REPOSITORY_URL} context value or configure a git remote`
       );
       return undefined;
     }

src/constructs/dns/dns-records.test.ts

@@ -36,7 +36,7 @@ describe("The GuDnsRecordSet construct", () => {
         ttl: Duration.hours(1),
       });
     }).toThrowError(
-      "According to RFC, a CNAME record should not return multiple answers. Doing so may cause problems during resolution.",
+      "According to RFC, a CNAME record should not return multiple answers. Doing so may cause problems during resolution."
     );
   });
 });

src/constructs/dns/dns-records.ts

@@ -49,7 +49,7 @@ export class GuDnsRecordSet extends Construct {
        */
       if (resourceRecords.length !== 1) {
         throw new Error(
-          "According to RFC, a CNAME record should not return multiple answers. Doing so may cause problems during resolution.",
+          "According to RFC, a CNAME record should not return multiple answers. Doing so may cause problems during resolution."
         );
       }
     }

src/constructs/ec2/vpc.ts

@@ -42,7 +42,7 @@ export class GuVpc {
      * TODO: Understand VPCs and Subnets better and develop a better solution to this problem
      */
     return subnets.map((subnetId) =>
-      Subnet.fromSubnetAttributes(scope, `subnet-${subnetId}`, { subnetId, routeTableId: " " }),
+      Subnet.fromSubnetAttributes(scope, `subnet-${subnetId}`, { subnetId, routeTableId: " " })
     );
   }
 

src/constructs/ecs/ecs-task.ts

@@ -164,7 +164,7 @@ export class GuEcsTask extends Construct {
 
     if (storage && storage < 21) {
       throw new Error(
-        "Storage must be at least 21. See https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-properties-ecs-taskdefinition-ephemeralstorage.html",
+        "Storage must be at least 21. See https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-properties-ecs-taskdefinition-ephemeralstorage.html"
       );
     }
 
@@ -228,7 +228,7 @@ export class GuEcsTask extends Construct {
       const alarmTopic = Topic.fromTopicArn(
         scope,
         AppIdentity.suffixText(props, "AlarmTopic"),
-        monitoringConfiguration.snsTopicArn,
+        monitoringConfiguration.snsTopicArn
       );
       const alarms = [
         {

src/constructs/iam/policies/base-policy.test.ts

@@ -10,7 +10,7 @@ describe("GuAllowPolicy", () => {
       new GuAllowPolicy(stack, "AllowS3GetObject", {
         actions: ["s3:GetObject"],
         resources: ["*"],
-      }),
+      })
     );
 
     Template.fromStack(stack).hasResourceProperties("AWS::IAM::Policy", {
@@ -34,7 +34,7 @@ describe("GuAllowPolicy", () => {
       new GuAllowPolicy(stack, "AllowS3GetObject", {
         actions: ["s3:GetObject", "s3:ListBucket"],
         resources: ["*"],
-      }),
+      })
     );
 
     Template.fromStack(stack).hasResourceProperties("AWS::IAM::Policy", {
@@ -60,7 +60,7 @@ describe("GuDenyPolicy", () => {
       new GuDenyPolicy(stack, "DenyS3GetObject", {
         actions: ["s3:GetObject"],
         resources: ["*"],
-      }),
+      })
     );
 
     Template.fromStack(stack).hasResourceProperties("AWS::IAM::Policy", {
@@ -84,7 +84,7 @@ describe("GuDenyPolicy", () => {
       new GuDenyPolicy(stack, "DenyS3GetObject", {
         actions: ["s3:GetObject", "s3:ListBucket"],
         resources: ["*"],
-      }),
+      })
     );
 
     Template.fromStack(stack).hasResourceProperties("AWS::IAM::Policy", {

src/constructs/iam/policies/base-policy.ts

@@ -27,7 +27,7 @@ export class GuAllowPolicy extends GuPolicy {
         effect: Effect.ALLOW,
         resources: props.resources,
         actions: props.actions,
-      }),
+      })
     );
   }
 }
@@ -41,7 +41,7 @@ export class GuDenyPolicy extends GuPolicy {
         effect: Effect.DENY,
         resources: props.resources,
         actions: props.actions,
-      }),
+      })
     );
   }
 }

src/constructs/iam/roles/github-actions.ts

@@ -93,11 +93,11 @@ export class GuGithubActionsRole extends GuRole {
         {
           StringLike: {
             [`${GITHUB_ACTIONS_ID_TOKEN_REQUEST_DOMAIN}:sub`]: GuGithubActionsRepositoryCondition.toString(
-              props.condition ?? ALL_GUARDIAN_REPOSITORIES,
+              props.condition ?? ALL_GUARDIAN_REPOSITORIES
             ),
           },
         },
-        "sts:AssumeRoleWithWebIdentity",
+        "sts:AssumeRoleWithWebIdentity"
       ),
     });
 

src/constructs/loadbalancing/alb/application-listener.test.ts

@@ -49,7 +49,7 @@ describe("The GuApplicationListener class", () => {
 
     GuTemplate.fromStack(stack).hasResourceWithLogicalId(
       "AWS::ElasticLoadBalancingV2::Listener",
-      /ApplicationListenerTesting.+/,
+      /ApplicationListenerTesting.+/
     );
   });
 
@@ -115,7 +115,7 @@ describe("The GuHttpsApplicationListener class", () => {
 
     GuTemplate.fromStack(stack).hasResourceWithLogicalId(
       "AWS::ElasticLoadBalancingV2::Listener",
-      /^HttpsApplicationListenerTesting.+/,
+      /^HttpsApplicationListenerTesting.+/
     );
   });
 

src/constructs/loadbalancing/alb/application-load-balancer.test.ts

@@ -22,7 +22,7 @@ describe("The GuApplicationLoadBalancer class", () => {
 
     GuTemplate.fromStack(stack).hasResourceWithLogicalId(
       "AWS::ElasticLoadBalancingV2::LoadBalancer",
-      /^ApplicationLoadBalancerTesting.+/,
+      /^ApplicationLoadBalancerTesting.+/
     );
   });
 

src/constructs/loadbalancing/alb/application-target-group.test.ts

@@ -23,7 +23,7 @@ describe("The GuApplicationTargetGroup class", () => {
 
     GuTemplate.fromStack(stack).hasResourceWithLogicalId(
       "AWS::ElasticLoadBalancingV2::TargetGroup",
-      /^ApplicationTargetGroupTesting.+/,
+      /^ApplicationTargetGroupTesting.+/
     );
   });
 

src/constructs/ses/index.test.ts

@@ -34,7 +34,7 @@ describe("The GuEmailIdentity construct", () => {
         app: "test",
       });
     }).toThrowError(
-      "Auto verification is only supported for certain domains. my-service.theguardian.com is not supported.",
+      "Auto verification is only supported for certain domains. my-service.theguardian.com is not supported."
     );
   });
 

src/constructs/stack-set/stack-set.test.ts

@@ -30,7 +30,7 @@ describe("The GuStackSet construct", () => {
       theStackSetInstance,
       "central-topic",
       new GuStringParameter(theStackSetInstance, "CentralSnsTopicArn", { allowedPattern: RegexPattern.ARN })
-        .valueAsString,
+        .valueAsString
     );
 
     const awsOrgId = "o-12345abcde";

src/constructs/vpc/vpc.ts

@@ -57,7 +57,7 @@ export class GuVpc extends Vpc {
   private static setAvailabilityZoneContext({ account, node }: GuStack) {
     if (Token.isUnresolved(account)) {
       throw new Error(
-        `Account ID not set - the resulting VPC might not be shaped how you'd expect. See https://docs.aws.amazon.com/cdk/api/v2/docs/aws-cdk-lib.StackProps.html#env`,
+        `Account ID not set - the resulting VPC might not be shaped how you'd expect. See https://docs.aws.amazon.com/cdk/api/v2/docs/aws-cdk-lib.StackProps.html#env`
       );
     }
 

src/experimental/constructs/iam/roles/fastly-kinesis-log.ts

@@ -46,7 +46,7 @@ export class GuFastlyKinesisLogRoleExperimental extends GuRole {
       policyName ?? "GuKinesisPutRecordsPolicyExperimental",
       {
         stream,
-      },
+      }
     );
 
     policy.attachToRole(this);

src/experimental/patterns/kinesis-lambda.ts

@@ -94,11 +94,11 @@ export class GuKinesisLambdaExperimental extends GuLambdaFunction {
       ? Stream.fromStreamArn(
           scope,
           existingKinesisStream.externalKinesisStreamName,
-          `arn:aws:kinesis:${region}:${account}:stream/${existingKinesisStream.externalKinesisStreamName}`,
+          `arn:aws:kinesis:${region}:${account}:stream/${existingKinesisStream.externalKinesisStreamName}`
         )
       : AppIdentity.taggedConstruct(
           props,
-          new GuKinesisStream(scope, "KinesisStream", { encryption: StreamEncryption.MANAGED, ...kinesisStreamProps }),
+          new GuKinesisStream(scope, "KinesisStream", { encryption: StreamEncryption.MANAGED, ...kinesisStreamProps })
         );
 
     const errorHandlingPropsToAwsProps = toAwsErrorHandlingProps(props.errorHandlingConfiguration);