declapract-typescript-ehmpathy 0.17.0

package/dist/practices/prettier/best-practice/package.json.declapract.ts

@@ -0,0 +1,3 @@
+import { FileCheckType } from 'declapract';
+
+export const check = FileCheckType.CONTAINS;

package/dist/practices/prettier/best-practice/prettier.config.js

@@ -0,0 +1,9 @@
+// ref: http://json.schemastore.org/prettierrc
+
+module.exports = {
+  trailingComma: 'all',
+  tabWidth: 2,
+  singleQuote: true,
+  importOrder: ['^[./]'],
+  importOrderSeparation: true,
+};

package/dist/practices/runtime-type-checking/bad-practices/joi-types/package.json

@@ -0,0 +1,5 @@
+{
+  "devDependencies": {
+    "@types/joi": "@declapract{check.minVersion('0.0.0')}"
+  }
+}

package/dist/practices/runtime-type-checking/bad-practices/joi-types/package.json.declapract.ts

@@ -0,0 +1,18 @@
+import { FileCheckType, FileFixFunction } from 'declapract';
+
+export const check = FileCheckType.CONTAINS;
+
+export const fix: FileFixFunction = (contents) => {
+  if (!contents) return { contents }; // do nothing if no contents
+  const packageJSON = JSON.parse(contents);
+  const updatedPackageJSON = {
+    ...packageJSON,
+    devDependencies: {
+      ...packageJSON.devDependencies,
+      '@types/joi': undefined,
+    },
+  };
+  return {
+    contents: JSON.stringify(updatedPackageJSON, null, 2),
+  };
+};

package/dist/practices/runtime-type-checking/bad-practices/old-joi-syntax-valid-input-cant-be-array/.declapract.readme.md

@@ -0,0 +1 @@
+joi updated their syntax and now throws an error if `.valid([])`, instead we need to give `.valid(...[])`

package/dist/practices/runtime-type-checking/bad-practices/old-joi-syntax-valid-input-cant-be-array/src/<star><star>/<star>.ts.declapract.ts

@@ -0,0 +1,15 @@
+import { FileCheckFunction, FileFixFunction } from 'declapract';
+
+export const check: FileCheckFunction = (contents) => {
+  if (contents?.includes('.valid(Object.values(')) return; // matches bad practice if this is found
+  throw new Error('passes, does not match bad practice');
+};
+export const fix: FileFixFunction = (contents) => {
+  if (!contents) return {};
+  return {
+    contents: contents.replace(
+      /\.valid\(Object\.values/g,
+      '.valid(...Object.values',
+    ),
+  };
+};

package/dist/practices/runtime-type-checking/best-practice/package.json

@@ -0,0 +1,5 @@
+{
+  "dependencies": {
+    "joi": "@declapract{check.minVersion('17.4.0')}"
+  }
+}

package/dist/practices/runtime-type-checking/best-practice/package.json.declapract.ts

@@ -0,0 +1,3 @@
+import { FileCheckType } from 'declapract';
+
+export const check = FileCheckType.CONTAINS;

package/dist/practices/serverless/bad-practices/generic-deploy-script/package.json

@@ -0,0 +1,8 @@
+{
+  "scripts": {
+    "deploy:release": "npm run build && sls deploy --verbose --stage $DEPLOYMENT_STAGE",
+    "deploy:send-notification": "curl -X POST -H 'Content-type: application/json' --data \"{\\\"text\\\":\\\"$([ -z $DEPLOYER_NAME ] && git config user.name || echo $DEPLOYER_NAME) has deployed $npm_package_name@v$npm_package_version:\nhttps://github.com/@declapract{variable.organizationName}/$npm_package_name/tree/v$npm_package_version\\\"}\" @declapract{variable.slackWebhookUrl}",
+    "deploy:dev": "DEPLOYMENT_STAGE=dev npm run deploy:release",
+    "deploy:prod": "DEPLOYMENT_STAGE=prod npm run deploy:release && npm run deploy:send-notification"
+  }
+}

package/dist/practices/serverless/bad-practices/generic-deploy-script/package.json.declapract.ts

@@ -0,0 +1,23 @@
+import { FileCheckFunction, FileFixFunction } from 'declapract';
+
+export const check: FileCheckFunction = (contents) => {
+  if (!contents) throw new Error('does not match bad practice'); // no contents -> fine
+  const packageJSON = JSON.parse(contents);
+  if (packageJSON.scripts.deploy) return; // bad practice
+  throw new Error('does not match bad practice');
+};
+
+export const fix: FileFixFunction = (contents) => {
+  if (!contents) return { contents }; // do nothing if no contents
+  const packageJSON = JSON.parse(contents);
+  const updatedPackageJSON = {
+    ...packageJSON,
+    scripts: {
+      ...packageJSON.scripts,
+      deploy: undefined, // remove the deploy script
+    },
+  };
+  return {
+    contents: JSON.stringify(updatedPackageJSON, null, 2),
+  };
+};

package/dist/practices/serverless/bad-practices/old-sls-plugins/package.json

@@ -0,0 +1,6 @@
+{
+  "devDependencies": {
+    "serverless-offline": "@declapract{check.minVersion('0.0.0')}",
+    "serverless-pseudo-parameters": "@declapract{check.minVersion('0.0.0')}"
+  }
+}

package/dist/practices/serverless/bad-practices/old-sls-plugins/package.json.declapract.ts

@@ -0,0 +1,19 @@
+import { FileCheckType, FileFixFunction } from 'declapract';
+
+export const check = FileCheckType.CONTAINS;
+
+export const fix: FileFixFunction = (contents) => {
+  if (!contents) return { contents }; // do nothing if no contents
+  const packageJSON = JSON.parse(contents);
+  const updatedPackageJSON = {
+    ...packageJSON,
+    devDependencies: {
+      ...packageJSON.devDependencies,
+      'serverless-offline': undefined, // remove the dep
+      'serverless-pseudo-parameters': undefined, // remove the deploy script
+    },
+  };
+  return {
+    contents: JSON.stringify(updatedPackageJSON, null, 2),
+  };
+};

package/dist/practices/serverless/best-practice/package.json

@@ -0,0 +1,11 @@
+{
+  "devDependencies": {
+    "serverless": "@declapract{check.minVersion('2.57.0')}"
+  },
+  "scripts": {
+    "deploy:release": "npm run build && sls deploy --verbose --stage $DEPLOYMENT_STAGE",
+    "deploy:send-notification": "curl -X POST -H 'Content-type: application/json' --data \"{\\\"text\\\":\\\"$([ -z $DEPLOYER_NAME ] && git config user.name || echo $DEPLOYER_NAME) has deployed $npm_package_name@v$npm_package_version:\nhttps://github.com/@declapract{variable.organizationName}/$npm_package_name/tree/v$npm_package_version\\\"}\" @declapract{variable.slackWebhookUrl}",
+    "deploy:dev": "DEPLOYMENT_STAGE=dev npm run deploy:release",
+    "deploy:prod": "DEPLOYMENT_STAGE=prod npm run deploy:release && npm run deploy:send-notification"
+  }
+}

package/dist/practices/serverless/best-practice/package.json.declapract.ts

@@ -0,0 +1,3 @@
+import { FileCheckType } from 'declapract';
+
+export const check = FileCheckType.CONTAINS;

package/dist/practices/serverless/best-practice/serverless.yml

@@ -0,0 +1,70 @@
+service: @declapract{variable.projectName}
+
+provider:
+  name: aws
+  runtime: nodejs16.x
+  memorySize: 1024 # optional, in MB, default is 1024
+  timeout: 10
+  stage: ${opt:stage}
+  stackTags:
+    app: @declapract{variable.organizationName}
+    environment: ${self:provider.stage}
+    product: ${self:service}
+  environment:
+    NODE_ENV: production # deploy with production optimizations of all resources, to make `dev` and `prod` stage deployments equivalent functionally (i.e., the same code paths in dev and prod)
+    DEPLOYMENT_STAGE: ${self:provider.stage} # deploy specifying which stage we're targeting, to enable targeting the correct config + resources (e.g., hit dev db -vs- prod db)
+    AWS_NODEJS_CONNECTION_REUSE_ENABLED: true # https://docs.aws.amazon.com/sdk-for-javascript/v2/developer-guide/node-reusing-connections.html
+  deploymentBucket: serverless-deployment-@declapract{variable.infrastructureNamespaceId}-${self:provider.stage}
+  vpc:
+    securityGroupIds:
+      - ${ssm:/tf/infrastructure/vpc/main/lambdaSecurityGroupId}
+    subnetIds:
+      - ${ssm:/tf/infrastructure/vpc/main/lambdaSubnet1Id}
+      - ${ssm:/tf/infrastructure/vpc/main/lambdaSubnet2Id}
+      - ${ssm:/tf/infrastructure/vpc/main/lambdaSubnet3Id}
+  iamRoleStatements:
+    # parameter store access
+    - Effect: 'Allow'
+      Action: 'ssm:DescribeParameters'
+      Resource: '*'
+    - Effect: 'Allow'
+      Action: 'ssm:GetParameters'
+      Resource: arn:aws:ssm:${aws:region}:${aws:accountId}:parameter/*
+    - Effect: 'Allow'
+      Action: 'kms:Decrypt'
+      Resource: arn:aws:kms:${aws:region}:${aws:accountId}:alias/aws/ssm*
+    # allow invocation of other lambdas
+    - Effect: Allow
+      Action:
+        - lambda:InvokeFunction
+        - lambda:InvokeAsync
+      Resource: '*'
+    # allow reading and writing to s3 buckets namespaced by this service
+    - Effect: Allow
+      Action:
+        - s3:ListObject
+        - s3:GetObject
+        - s3:PutObject
+      Resource: arn:aws:s3:${aws:region}:${aws:accountId}:${self:service}-*
+    # allow reading and writing to sqs queues namespaced to this service
+    - Effect: Allow
+      Action:
+        - sqs:SendMessage
+        - sqs:ReceiveMessage
+        - sqs:DeleteMessage
+        - sqs:GetQueueAttributes
+      Resource: arn:aws:sqs:${aws:region}:${aws:accountId}:${self:service}-*
+    # allow reading and writing to dynamodb tables namespaced to this service
+    - Effect: Allow
+      Action:
+        - dynamodb:DescribeTable
+        - dynamodb:Query
+        - dynamodb:Scan
+        - dynamodb:GetItem
+        - dynamodb:PutItem
+      Resource: arn:aws:dynamodb:${aws:region}:${aws:accountId}:table/${self:service}-*
+    # allow publishing to sns topics namespaced by this service
+    - Effect: Allow
+      Action:
+        - SNS:Publish
+      Resource: arn:aws:sns:${aws:region}:${aws:accountId}:${self:service}-*

package/dist/practices/serverless/best-practice/serverless.yml.declapract.ts

@@ -0,0 +1,26 @@
+import { FileCheckType } from 'declapract';
+import { FileFixFunction } from 'declapract/dist/domain';
+
+export const check = FileCheckType.CONTAINS; // i.e., check that the contents of the file contains what's declared (default is equals)
+
+export const fix: FileFixFunction = (contents) => {
+  if (!contents) return { contents }; // do nothing if file dne; // TODO: update to provision file from declared contents
+  return {
+    contents: contents
+      .replace(/runtime\: nodejs\d\d.x/, 'runtime: nodejs14.x')
+      .replace('\nplugins:\n', '')
+      .replace(/  - serverless-offline .*\n/, '') // a plugin we no longer use (never used it, no need to have it)
+      .replace(/  - serverless-pseudo-parameters .*\n/, '') // a plugin we no longer use (serverless supports variables natively now)
+      .replace(/\#\{AWS\:\:Region\}/g, '${aws:region}') // use the serverless native variables, instead of the pseudo-parameters format
+      .replace(/\#\{AWS\:\:AccountId\}/g, '${aws:accountId}') // use the serverless native variables, instead of the pseudo-parameters format
+      .replace('## paramstore access', '# parameter store access')
+      .replace(
+        '## allow invocation of other lambdas',
+        '# allow invocation of other lambdas',
+      )
+      .replace(
+        'NODE_ENV: ${self:custom.stageToNodeEnvMapping.${self:provider.stage}}\n  deploymentBucket',
+        'NODE_ENV: ${self:custom.stageToNodeEnvMapping.${self:provider.stage}}\n    AWS_NODEJS_CONNECTION_REUSE_ENABLED: true # https://docs.aws.amazon.com/sdk-for-javascript/v2/developer-guide/node-reusing-connections.html\n  deploymentBucket',
+      ), // add this env flag // TODO: when we have special support for yml, do this better (i.e., instead of string replace, just add to the yml object after parsing it)
+  };
+};

package/dist/practices/terraform/bad-practices/terraform-dir/.declapract.readme.md

@@ -0,0 +1,8 @@
+having a `/provision/terraform` dir is bad practice since we use the tool terraform to provision multiple things:
+- aws resources
+- github resources
+- etc
+
+instead, those should be separately identified and managed with separate tf projects
+
+we used to just manage aws-resources in each repo, so thats why this pattern originally existed

package/dist/practices/terraform/bad-practices/terraform-dir/provision/terraform/<star><star>/.gitignore.declapract.ts

@@ -0,0 +1,12 @@
+import { FileCheckType, FileFixFunction } from 'declapract';
+
+export const check = FileCheckType.EXISTS;
+
+export const fix: FileFixFunction = (_, context) => {
+  return {
+    relativeFilePath: context.relativeFilePath.replace(
+      'provision/terraform/',
+      'provision/aws/',
+    ),
+  };
+};

package/dist/practices/terraform/bad-practices/terraform-dir/provision/terraform/<star><star>/<star>.hcl.declapract.ts

@@ -0,0 +1,12 @@
+import { FileCheckType, FileFixFunction } from 'declapract';
+
+export const check = FileCheckType.EXISTS;
+
+export const fix: FileFixFunction = (_, context) => {
+  return {
+    relativeFilePath: context.relativeFilePath.replace(
+      'provision/terraform/',
+      'provision/aws/',
+    ),
+  };
+};

package/dist/practices/terraform/bad-practices/terraform-dir/provision/terraform/<star><star>/<star>.tf.declapract.ts

@@ -0,0 +1,12 @@
+import { FileCheckType, FileFixFunction } from 'declapract';
+
+export const check = FileCheckType.EXISTS;
+
+export const fix: FileFixFunction = (_, context) => {
+  return {
+    relativeFilePath: context.relativeFilePath.replace(
+      'provision/terraform/',
+      'provision/aws/',
+    ),
+  };
+};

package/dist/practices/terraform/best-practice/.terraform-version

@@ -0,0 +1 @@
+0.14.11

package/dist/practices/terraform/best-practice/package.json

@@ -0,0 +1,6 @@
+{
+  "scripts": {
+    "fix:format:terraform": "terraform fmt -recursive provision",
+    "test:format:terraform": "terraform fmt -recursive -check provision"
+  }
+}

package/dist/practices/terraform/best-practice/package.json.declapract.ts

@@ -0,0 +1,3 @@
+import { FileCheckType } from 'declapract';
+
+export const check = FileCheckType.CONTAINS;

package/dist/practices/terraform/best-practice/provision/aws/environments/dev/main.tf

@@ -0,0 +1,18 @@
+provider "aws" {
+  region              = "us-east-1"
+  allowed_account_ids = ["@declapract{variable.awsAccountId.dev}"] # `dev` environment resources are provisioned in the `dev` aws account
+}
+
+terraform {
+  backend "s3" {
+    bucket  = "terraform-state-@declapract{variable.infrastructureNamespaceId}-dev"
+    key     = "@declapract{variable.projectName}-dev"
+    region  = "us-east-1"
+    encrypt = true
+  }
+}
+
+module "product" {
+  source      = "../../product"
+  environment = "dev"
+}

package/dist/practices/terraform/best-practice/provision/aws/environments/dev/versions.tf

@@ -0,0 +1,9 @@
+terraform {
+  required_version = ">= 0.14"
+  required_providers {
+    aws = {
+      source  = "hashicorp/aws"
+      version = "3.74.3"
+    }
+  }
+}

package/dist/practices/terraform/best-practice/provision/aws/environments/prod/main.tf

@@ -0,0 +1,18 @@
+provider "aws" {
+  region              = "us-east-1"
+  allowed_account_ids = ["@declapract{variable.awsAccountId.prod}"] # `prod` environment resources are provisioned in the `prod` aws account
+}
+
+terraform {
+  backend "s3" {
+    bucket  = "terraform-state-@declapract{variable.infrastructureNamespaceId}-prod"
+    key     = "@declapract{variable.projectName}-prod"
+    region  = "us-east-1"
+    encrypt = true
+  }
+}
+
+module "product" {
+  source      = "../../product"
+  environment = "prod"
+}

package/dist/practices/terraform/best-practice/provision/aws/environments/prod/versions.tf

@@ -0,0 +1,9 @@
+terraform {
+  required_version = ">= 0.14"
+  required_providers {
+    aws = {
+      source  = "hashicorp/aws"
+      version = "3.74.3"
+    }
+  }
+}

package/dist/practices/terraform/best-practice/provision/aws/environments/test/main.tf

@@ -0,0 +1,18 @@
+provider "aws" {
+  region              = "us-east-1"
+  allowed_account_ids = ["@declapract{variable.awsAccountId.dev}"] # `test` environment resources are provisioned in the `dev` aws account
+}
+
+terraform {
+  backend "s3" {
+    bucket  = "terraform-state-@declapract{variable.infrastructureNamespaceId}-test"
+    key     = "@declapract{variable.projectName}-test"
+    region  = "us-east-1"
+    encrypt = true
+  }
+}
+
+module "product" {
+  source      = "../../product"
+  environment = "test"
+}

package/dist/practices/terraform/best-practice/provision/aws/environments/test/versions.tf

@@ -0,0 +1,9 @@
+terraform {
+  required_version = ">= 0.14"
+  required_providers {
+    aws = {
+      source  = "hashicorp/aws"
+      version = "3.74.3"
+    }
+  }
+}

package/dist/practices/terraform/best-practice/provision/aws/product/variables.tf

@@ -0,0 +1,14 @@
+variable "environment" {}
+
+locals {
+  service      = "@declapract{variable.projectName}"
+  s3_namespace = "@declapract{variable.infrastructureNamespaceId}"
+}
+
+locals {
+  tags = {
+    app         = "@declapract{variable.organizationName}"
+    environment = var.environment
+    product     = local.service
+  }
+}

package/dist/practices/terraform/best-practice/provision/github/environment/import-existing-repo.sh

@@ -0,0 +1,12 @@
+# terraform... why cant you just look this stuff up given that you already have the inputs....
+REPO_NAME='@declapract{variable.projectName}';
+
+# import the repo itself
+terraform import module.product.github_repository.this $REPO_NAME;
+
+# import the default branch setting
+terraform import module.product.github_branch_default.default_branch_is_main $REPO_NAME;
+
+# import the branch protection rules
+terraform import "module.product.github_branch_protection.main_branch[0]" $REPO_NAME:main
+

package/dist/practices/terraform/best-practice/provision/github/environment/main.tf

@@ -0,0 +1,20 @@
+provider "github" {
+  owner = "@declapract{variable.organizationName}" # for the @declapract{variable.organizationName} organization
+  # note: `token` is sourced from env var GITHUB_TOKEN (e.g,. `use.github`)
+}
+
+terraform {
+  backend "s3" {
+    bucket  = "terraform-state-@declapract{variable.infrastructureNamespaceId}-prod" # tracked in the prod aws account's s3 bucket, so `use.@declapract{variable.organizationName}.prod`
+    key     = "@declapract{variable.projectName}-github"
+    region  = "us-east-1"
+    encrypt = true
+  }
+}
+
+module "product" {
+  source     = "../product"
+  name       = "@declapract{variable.projectName}"
+  visibility = "private"
+  protected  = true
+}

package/dist/practices/terraform/best-practice/provision/github/environment/versions.tf

@@ -0,0 +1,9 @@
+terraform {
+  required_version = ">= 0.14"
+  required_providers {
+    github = {
+      source  = "hashicorp/github"
+      version = "~> 4.19.1"
+    }
+  }
+}

package/dist/practices/terraform/best-practice/provision/github/product/repository.tf

@@ -0,0 +1,70 @@
+variable "name" {
+  type = string
+}
+variable "visibility" {
+  default = "private"
+}
+variable "protected" {
+  default = true
+}
+
+# the repository itself
+resource "github_repository" "this" {
+  name       = var.name
+  visibility = var.visibility
+
+  # standard settings
+  has_issues             = true # only issues are allowed... todo: decide if there's too much overlap between linear and issues
+  has_projects           = false
+  has_wiki               = false
+  has_downloads          = false
+  is_template            = false
+  allow_squash_merge     = true  # only squash merges are allowed
+  allow_merge_commit     = false # especially not merge merges. never merge merges
+  allow_rebase_merge     = false
+  allow_auto_merge       = true
+  delete_branch_on_merge = true # always delete branch on merging
+
+  lifecycle {
+    ignore_changes = [
+      description # dont worry about the description in these checks; the purpose of these checks is to check developer experience, not repo contents
+    ]
+  }
+}
+
+# the main branch (its "data" instead of a "resource" because 1. you cant create it w/ terraform anyway, 2. there's a constant "etag" out of sync with terraform)
+data "github_branch" "main" {
+  repository = github_repository.this.name
+  branch     = "main"
+}
+
+# set it as the default for this org
+resource "github_branch_default" "default_branch_is_main" {
+  repository = github_repository.this.name
+  branch     = data.github_branch.main.branch
+}
+
+# the branch protection rule on the main branch
+resource "github_branch_protection" "main_branch" {
+  repository_id = github_repository.this.node_id
+  pattern       = data.github_branch.main.branch
+
+  enforce_admins      = true  # yes, even admins need to follow this (note: they can still take the time to go and change the settings temporarily for the exceptions)
+  allows_deletions    = false # dont allow the `main` branch to be deleted
+  allows_force_pushes = false # dont allow `main` branch to be force pushed to
+
+  required_status_checks {
+    strict = true # branch must be up to date. otherwise, we dont know if it will really pass once it is merged
+    contexts = [
+      "test-commits",
+      "test-types",
+      "test-format",
+      "test-lint",
+      "test-unit",
+      "test-integration",
+      "test-acceptance-locally"
+    ]
+  }
+
+  count = var.protected == false ? 0 : 1 # only define protection rules if this repository is marked as protected (default = true)
+}

package/dist/practices/testing/bad-practices/mocks/.declapract.readme.md

@@ -0,0 +1,8 @@
+should not have any `__mock__` directories
+
+these are hard to track down and are too "magical" and affect _all_ tests (i.e., inter test dependency)
+
+just explicitly mocks manually in each test OR root level in setup env with:
+```ts
+jest.mock("...");
+```

package/dist/practices/testing/bad-practices/mocks/<star><star>/__mock__/<star>.js

@@ -0,0 +1,3 @@
+/**
+ * bad practice: file exists
+ */

package/dist/practices/testing/bad-practices/mocks/<star><star>/__mock__/<star>.ts

@@ -0,0 +1,3 @@
+/**
+ * bad practice: file exists
+ */

package/dist/practices/testing/bad-practices/old-acceptance-test-utils/acceptance/_utils/getDataFromLastOpenBracketAtStartOfLine.ts.declapract.ts

@@ -0,0 +1,3 @@
+import { FileCheckType } from 'declapract';
+
+export const check = FileCheckType.EXISTS;

package/dist/practices/testing/bad-practices/old-acceptance-test-utils/acceptance/_utils/invokeLambda.ts.declapract.ts

@@ -0,0 +1,3 @@
+import { FileCheckType } from 'declapract';
+
+export const check = FileCheckType.EXISTS;

package/dist/practices/testing/bad-practices/old-acceptance-test-utils/acceptance/lambdas/<star>.ts.declapract.ts

@@ -0,0 +1,44 @@
+import { FileCheckFunction, FileFixFunction } from 'declapract';
+
+import { getProjectVariables } from '../../../../../../getVariables';
+
+export const check: FileCheckFunction = (contents) => {
+  if (
+    contents?.includes(`import { invokeLambda } from '../_utils/invokeLambda';`)
+  )
+    return; // then it matches bad practice
+  throw new Error('does not match bad practice');
+};
+
+export const fix: FileFixFunction = async (contents, context) => {
+  if (!contents) return {}; // do nothing if no contents. really, this shouldn't have been called
+  const { projectName } = await getProjectVariables(context);
+  return {
+    contents: contents
+      .replace(
+        // replace the bad import
+        `import { invokeLambda } from '../_utils/invokeLambda';`,
+        [
+          `import { invokeLambdaForTesting } from 'simple-lambda-testing-methods';`,
+          `import { stage } from '../../src/utils/environment';`,
+          `import { locally } from '../environment';`,
+        ].join('\n'),
+      )
+      .replace(
+        // replace calls to the fn
+        /invokeLambda\(\{/g,
+        'invokeLambdaForTesting({',
+      )
+      .replace(
+        // replace the inputs
+        /name\: '(\w+)',/g,
+        [
+          `service: '${projectName}',`,
+          `function: '$1',`,
+          `stage,`,
+          `locally,`,
+        ].join('\n'),
+      )
+      .replace(/data\: /g, 'event: '),
+  };
+};

package/dist/practices/testing/bad-practices/old-acceptance-test-utils-2/acceptance/__test_utils__/environment.ts.declapract.ts

@@ -0,0 +1,3 @@
+import { FileCheckType } from 'declapract';
+
+export const check = FileCheckType.EXISTS;

package/dist/practices/testing/bad-practices/old-acceptance-test-utils-2/acceptance/lambdas/<star>.ts.declapract.ts

@@ -0,0 +1,22 @@
+import { FileCheckFunction, FileFixFunction } from 'declapract';
+
+export const check: FileCheckFunction = (contents) => {
+  if (
+    contents?.includes(
+      `import { locally } from '../__test_utils__/environment';`,
+    )
+  )
+    return; // then it matches bad practice
+  throw new Error('does not match bad practice');
+};
+
+export const fix: FileFixFunction = async (contents) => {
+  if (!contents) return {}; // do nothing if no contents. really, this shouldn't have been called
+  return {
+    contents: contents.replace(
+      // replace the bad import
+      `import { locally } from '../__test_utils__/environment';`,
+      `import { locally } from '../environment';`,
+    ),
+  };
+};