@checkdigit/eslint-plugin 7.2.0 → 7.3.0-PR.75-aa6d

package/src/agent/response-reference.ts

@@ -0,0 +1,109 @@
+// agent/response-reference.ts
+
+/*
+ * Copyright (c) 2021-2024 Check Digit, LLC
+ *
+ * This code is licensed under the MIT license (see LICENSE.txt for details).
+ */
+
+import { strict as assert } from 'node:assert';
+import type { MemberExpression, VariableDeclaration } from 'estree';
+import { type Scope } from 'eslint';
+
+import { getParent } from '../library/tree';
+
+/**
+ * analyze response related variables and their references
+ * the implementation is for fixture API, but it can be used for fetch API as well since the tree structure is similar
+ * @param variableDeclaration - variable declaration node
+ */
+export function analyzeResponseReferences(
+  variableDeclaration: VariableDeclaration | undefined,
+  scopeManager: Scope.ScopeManager,
+): {
+  variable?: Scope.Variable;
+  bodyReferences: MemberExpression[];
+  headersReferences: MemberExpression[];
+  statusReferences: MemberExpression[];
+  destructuringBodyVariable?: Scope.Variable;
+  destructuringHeadersVariable?: Scope.Variable;
+  destructuringHeadersReferences?: MemberExpression[] | undefined;
+} {
+  const results: {
+    variable?: Scope.Variable;
+    bodyReferences: MemberExpression[];
+    headersReferences: MemberExpression[];
+    statusReferences: MemberExpression[];
+    destructuringBodyVariable?: Scope.Variable;
+    destructuringHeadersVariable?: Scope.Variable;
+    destructuringHeadersReferences?: MemberExpression[] | undefined;
+  } = {
+    bodyReferences: [],
+    headersReferences: [],
+    statusReferences: [],
+  };
+  if (!variableDeclaration) {
+    return results;
+  }
+
+  const responseVariables = scopeManager.getDeclaredVariables(variableDeclaration);
+  for (const responseVariable of responseVariables) {
+    const identifier = responseVariable.identifiers[0];
+    assert.ok(identifier);
+    const identifierParent = getParent(identifier);
+    assert.ok(identifierParent);
+    if (identifierParent.type === 'VariableDeclarator') {
+      // e.g. const response = ...
+      results.variable = responseVariable;
+      const responseReferences = responseVariable.references.map((responseReference) =>
+        getParent(responseReference.identifier),
+      );
+      // e.g. response.body
+      results.bodyReferences = responseReferences.filter(
+        (node): node is MemberExpression =>
+          node?.type === 'MemberExpression' && node.property.type === 'Identifier' && node.property.name === 'body',
+      );
+      // e.g. response.headers / response.header / response.get()
+      results.headersReferences = responseReferences.filter(
+        (node): node is MemberExpression =>
+          node?.type === 'MemberExpression' &&
+          node.property.type === 'Identifier' &&
+          (node.property.name === 'header' || node.property.name === 'headers' || node.property.name === 'get'),
+      );
+      // e.g. response.status / response.statusCode
+      results.statusReferences = responseReferences.filter(
+        (node): node is MemberExpression =>
+          node?.type === 'MemberExpression' &&
+          node.property.type === 'Identifier' &&
+          (node.property.name === 'status' || node.property.name === 'statusCode'),
+      );
+    } else if (
+      // body reference through destruction/renaming, e.g. "const { body } = ..."
+      identifierParent.type === 'Property' &&
+      identifierParent.key.type === 'Identifier' &&
+      identifierParent.key.name === 'body'
+    ) {
+      results.destructuringBodyVariable = responseVariable;
+    } else if (
+      // header reference through destruction/renaming, e.g. "const { headers } = ..."
+      identifierParent.type === 'Property' &&
+      identifierParent.key.type === 'Identifier' &&
+      identifierParent.key.name === 'headers'
+    ) {
+      results.destructuringHeadersVariable = responseVariable;
+      results.destructuringHeadersReferences = responseVariable.references
+        .map((reference) => reference.identifier)
+        .map(getParent)
+        .filter(
+          (parent): parent is MemberExpression =>
+            parent?.type === 'MemberExpression' &&
+            parent.property.type === 'Identifier' &&
+            parent.property.name !== 'get' &&
+            getParent(parent)?.type !== 'CallExpression',
+        );
+    } else {
+      throw new Error(`Unknown response variable reference: ${responseVariable.name}`);
+    }
+  }
+  return results;
+}

package/src/agent/url.ts

@@ -0,0 +1,25 @@
+// agent/url.ts
+
+// eslint-disable-next-line @typescript-eslint/no-inferrable-types
+export const PLAIN_URL_REGEXP: RegExp = /^[`']\/\w+(?<serviceNamePart>-\w+)*\/v\d+\/(?<any>.|\r|\n)+[`']$/u;
+// eslint-disable-next-line @typescript-eslint/no-inferrable-types
+export const TOKENIZED_URL_REGEXP: RegExp = /^`\$\{(?<serviceNamePart>[A-Z]+_)*BASE_PATH\}\/(?<any>.|\r|\n)+`$/u;
+
+export function replaceEndpointUrlPrefixWithBasePath(url: string): string {
+  // eslint-disable-next-line no-template-curly-in-string
+  return url.replace(/^`\/\w+(?<parts>-\w+)*\/v\d+\//u, '`${BASE_PATH}/');
+}
+
+export function replaceEndpointUrlPrefixWithDomain(url: string): string {
+  return url.replace(
+    /^(?<quotStart>[`'])\/(?<servicename>\w+(?<parts>-\w+)*)(?<path>\/v\d+\/(?<any>.|\r|\n)+(?<quotEnd>[`'])$)/u,
+    '$1https://$2.checkdigit/$2$4',
+  );
+}
+
+export function addBasePathUrlDomain(url: string): string {
+  return url.replace(
+    /^(?<quotStart>[`'])\/(?<servicename>\w+(?<parts>-\w+)*)(?<path>\/v\d+(?<quotEnd>[`'])$)/u,
+    '$1https://$2.checkdigit/$2$4',
+  );
+}

package/src/index.ts

@@ -8,10 +8,29 @@
 
 import type { TSESLint } from '@typescript-eslint/utils';
 
+import addUrlDomain, { ruleId as addUrlDomainRuleId } from './agent/add-url-domain';
+import agentTestWiring, { ruleId as agentTestWiringRuleId } from './agent/agent-test-wiring';
+import fetchResponseBodyJson, { ruleId as fetchResponseBodyJsonRuleId } from './agent/fetch-response-body-json';
+import fetchResponseHeaderGetter, {
+  ruleId as fetchResponseHeaderGetterRuleId,
+} from './agent/fetch-response-header-getter';
+import fetchThen, { ruleId as fetchThenRuleId } from './agent/fetch-then';
+import fixFunctionCallArguments, {
+  ruleId as fixFunctionCallArgumentsRuleId,
+} from './agent/fix-function-call-arguments';
 import invalidJsonStringify, { ruleId as invalidJsonStringifyRuleId } from './invalid-json-stringify';
 import noDuplicatedImports, { ruleId as noDuplicatedImportsRuleId } from './no-duplicated-imports';
+import noFixture, { ruleId as noFixtureRuleId } from './agent/no-fixture';
 import noFullResponse, { ruleId as noFullResponseRuleId } from './agent/no-full-response';
+import noMappedResponse, { ruleId as noMappedResponseRuleId } from './agent/no-mapped-response';
 import noPromiseInstanceMethod, { ruleId as noPromiseInstanceMethodRuleId } from './no-promise-instance-method';
+import noServiceWrapper, { ruleId as noServiceWrapperRuleId } from './agent/no-service-wrapper';
+import noStatusCode, { ruleId as noStatusCodeRuleId } from './agent/no-status-code';
+import noUnusedFunctionArguments, {
+  ruleId as noUnusedFunctionArgumentsRuleId,
+} from './agent/no-unused-function-argument';
+import noUnusedImports, { ruleId as noUnusedImportsRuleId } from './agent/no-unused-imports';
+import noUnusedServiceVariables, { ruleId as noUnusedServiceVariablesRuleId } from './agent/no-unused-service-variable';
 import requireFixedServicesImport, {
   ruleId as requireFixedServicesImportRuleId,
 } from './require-fixed-services-import';
@@ -44,19 +63,32 @@ const rules: Record<string, TSESLint.LooseRuleDefinition> = {
   'object-literal-response': objectLiteralResponse,
   [invalidJsonStringifyRuleId]: invalidJsonStringify,
   [noPromiseInstanceMethodRuleId]: noPromiseInstanceMethod,
+  [noFixtureRuleId]: noFixture,
+  [fetchThenRuleId]: fetchThen,
+  [noServiceWrapperRuleId]: noServiceWrapper,
+  [noStatusCodeRuleId]: noStatusCode,
+  [fetchResponseBodyJsonRuleId]: fetchResponseBodyJson,
+  [fetchResponseHeaderGetterRuleId]: fetchResponseHeaderGetter,
+  [addUrlDomainRuleId]: addUrlDomain,
   [noFullResponseRuleId]: noFullResponse,
+  [noMappedResponseRuleId]: noMappedResponse,
   [requireResolveFullResponseRuleId]: requireResolveFullResponse,
-  [requireTypeOutOfTypeOnlyImportsRuleId]: requireTypeOutOfTypeOnlyImports,
   [noDuplicatedImportsRuleId]: noDuplicatedImports,
-  [requireFixedServicesImportRuleId]: requireFixedServicesImport,
   [noServeRuntimeRuleId]: noServeRuntime,
+  [requireFixedServicesImportRuleId]: requireFixedServicesImport,
+  [requireTypeOutOfTypeOnlyImportsRuleId]: requireTypeOutOfTypeOnlyImports,
+  [noUnusedFunctionArgumentsRuleId]: noUnusedFunctionArguments,
+  [noUnusedServiceVariablesRuleId]: noUnusedServiceVariables,
+  [noUnusedImportsRuleId]: noUnusedImports,
+  [fixFunctionCallArgumentsRuleId]: fixFunctionCallArguments,
+  [agentTestWiringRuleId]: agentTestWiring,
 };
 
 const plugin: TSESLint.FlatConfig.Plugin = {
   rules,
 };
 
-const configs: Record<string, TSESLint.FlatConfig.Config> = {
+const configs: Record<string, TSESLint.FlatConfig.Config | TSESLint.FlatConfig.Config[]> = {
   all: {
     plugins: {
       '@checkdigit': plugin,
@@ -75,10 +107,25 @@ const configs: Record<string, TSESLint.FlatConfig.Config> = {
       [`@checkdigit/${noPromiseInstanceMethodRuleId}`]: 'error',
       [`@checkdigit/${noFullResponseRuleId}`]: 'error',
       [`@checkdigit/${requireResolveFullResponseRuleId}`]: 'error',
-      [`@checkdigit/${requireTypeOutOfTypeOnlyImportsRuleId}`]: 'error',
       [`@checkdigit/${noDuplicatedImportsRuleId}`]: 'error',
       [`@checkdigit/${requireFixedServicesImportRuleId}`]: 'error',
+      [`@checkdigit/${requireTypeOutOfTypeOnlyImportsRuleId}`]: 'error',
       [`@checkdigit/${noServeRuntimeRuleId}`]: 'error',
+      // --- agent rules BEGIN ---
+      [`@checkdigit/${noMappedResponseRuleId}`]: 'off',
+      [`@checkdigit/${addUrlDomainRuleId}`]: 'off',
+      [`@checkdigit/${noFixtureRuleId}`]: 'off',
+      [`@checkdigit/${noServiceWrapperRuleId}`]: 'off',
+      [`@checkdigit/${noStatusCodeRuleId}`]: 'off',
+      [`@checkdigit/${fetchResponseBodyJsonRuleId}`]: 'off',
+      [`@checkdigit/${fetchResponseHeaderGetterRuleId}`]: 'off',
+      [`@checkdigit/${fetchThenRuleId}`]: 'off',
+      [`@checkdigit/${noUnusedFunctionArgumentsRuleId}`]: 'off',
+      [`@checkdigit/${noUnusedServiceVariablesRuleId}`]: 'off',
+      [`@checkdigit/${noUnusedImportsRuleId}`]: 'off',
+      [`@checkdigit/${fixFunctionCallArgumentsRuleId}`]: 'off',
+      [`@checkdigit/${agentTestWiringRuleId}`]: 'off',
+      // --- agent rules END ---
     },
   },
   recommended: {
@@ -99,10 +146,62 @@ const configs: Record<string, TSESLint.FlatConfig.Config> = {
       [`@checkdigit/${noPromiseInstanceMethodRuleId}`]: 'error',
     },
   },
+  'agent-phase-1-test': [
+    {
+      files: ['*.spec.ts', '*.test.ts', 'src/api/v*/index.ts'],
+      plugins: {
+        '@checkdigit': plugin,
+      },
+      rules: {
+        [`@checkdigit/${noMappedResponseRuleId}`]: 'error',
+        [`@checkdigit/${addUrlDomainRuleId}`]: 'error',
+        [`@checkdigit/${noFixtureRuleId}`]: 'error',
+        [`@checkdigit/${noServiceWrapperRuleId}`]: 'error',
+        [`@checkdigit/${noStatusCodeRuleId}`]: 'error',
+        [`@checkdigit/${fetchResponseBodyJsonRuleId}`]: 'error',
+        [`@checkdigit/${fetchResponseHeaderGetterRuleId}`]: 'error',
+        [`@checkdigit/${fetchThenRuleId}`]: 'error',
+        [`@checkdigit/${noUnusedFunctionArgumentsRuleId}`]: 'error',
+        [`@checkdigit/${noUnusedServiceVariablesRuleId}`]: 'error',
+        [`@checkdigit/${noUnusedImportsRuleId}`]: 'error',
+        [`@checkdigit/${fixFunctionCallArgumentsRuleId}`]: 'error',
+      },
+    },
+    {
+      files: ['*.spec.ts'],
+      plugins: {
+        '@checkdigit': plugin,
+      },
+      rules: {
+        [`@checkdigit/${agentTestWiringRuleId}`]: 'error',
+      },
+    },
+  ],
+  'agent-phase-2-production': {
+    plugins: {
+      '@checkdigit': plugin,
+    },
+    rules: {
+      [`@checkdigit/${noMappedResponseRuleId}`]: 'error',
+      [`@checkdigit/${addUrlDomainRuleId}`]: 'error',
+      [`@checkdigit/${noFixtureRuleId}`]: 'off',
+      [`@checkdigit/${noServiceWrapperRuleId}`]: 'error',
+      [`@checkdigit/${noStatusCodeRuleId}`]: 'error',
+      [`@checkdigit/${fetchResponseBodyJsonRuleId}`]: 'error',
+      [`@checkdigit/${fetchResponseHeaderGetterRuleId}`]: 'error',
+      [`@checkdigit/${fetchThenRuleId}`]: 'error',
+      [`@checkdigit/${noUnusedFunctionArgumentsRuleId}`]: 'error',
+      [`@checkdigit/${noUnusedServiceVariablesRuleId}`]: 'error',
+      [`@checkdigit/${noUnusedImportsRuleId}`]: 'error',
+      [`@checkdigit/${fixFunctionCallArgumentsRuleId}`]: 'error',
+    },
+  },
 };
 
-const pluginToExport: TSESLint.FlatConfig.Plugin = {
+const defaultToExport: Exclude<TSESLint.FlatConfig.Plugin, 'config'> & {
+  configs: Record<string, TSESLint.FlatConfig.Config | TSESLint.FlatConfig.Config[]>;
+} = {
   ...plugin,
   configs,
 };
-export default pluginToExport;
+export default defaultToExport;