Fix lint errors

Author: drewcorlin1

packages/esbuild-plugin-node/src/config/main.ts

@@ -17,7 +17,6 @@
 import {
   Instrumentation,
   InstrumentationBase,
-  InstrumentationModuleDefinition,
 } from '@opentelemetry/instrumentation';
 
 import { EsbuildInstrumentationConfigMap } from '../types';
@@ -31,9 +30,8 @@ function getModuleDefinitions(instrumentation: Instrumentation) {
   return [];
 }
 
-export const instrumentations: InstrumentationModuleDefinition<any>[] =
+export const instrumentations =
   getNodeAutoInstrumentations().flatMap(getModuleDefinitions);
-// console.log({ instrumentations });
 
 function configGenerator<T extends { enabled?: boolean }>(
   config?: T

packages/esbuild-plugin-node/src/plugin.test.ts

@@ -131,7 +131,7 @@ function dotFriendlyResolve(path: string, directory: string): string {
  *   input: '/foo/node_modules/@co/stuff/foo/bar/baz.js'
  *   output: { package: '@co/stuff', path: 'foo/bar/baz.js' }
  */
-export function extractPackageAndModulePath(
+function extractPackageAndModulePath(
   path: string,
   resolveDir: string
 ): { path: string; extractedModule: ExtractedModule } {

packages/esbuild-plugin-node/src/plugin.ts

@@ -17,13 +17,6 @@
 import type { OnLoadArgs as EsbuildOnLoadArgs } from 'esbuild';
 import type { InstrumentationConfigMap } from '@opentelemetry/auto-instrumentations-node';
 
-// TODO: Get all names
-export type InstrumentedPackage =
-  | 'fastify'
-  | 'pino'
-  | '@smithy/smithy-client'
-  | '@smithy/middleware-stack';
-
 export interface ExtractedModule {
   package: string | null;
   path: string | null;
@@ -32,9 +25,9 @@ export interface ExtractedModule {
 export type OnLoadArgs = Omit<EsbuildOnLoadArgs, 'pluginData'> & {
   pluginData?: {
     shouldPatchPackage: boolean;
-    package: InstrumentedPackage;
+    package: string;
     instrumentation: {
-      name: InstrumentedPackage;
+      name: string;
     };
     extractedModule: ExtractedModule;
   };

packages/esbuild-plugin-node/src/types.ts

@@ -17,7 +17,8 @@
 import * as assert from 'assert';
 
 import { exec as execCb, spawnSync } from 'child_process';
-import { inspect, promisify } from 'util';
+
+import { promisify } from 'util';
 
 const exec = promisify(execCb);
 

packages/esbuild-plugin-node/test/plugin.test.ts

@@ -40,7 +40,10 @@ interface GraphQLArgs {
   typeResolver?: Maybe<GraphQLTypeResolver<any, any>>;
 }
 
-const executeGraphqlQuery = (queryFunc: Function, args: GraphQLArgs) => {
+const executeGraphqlQuery = (
+  queryFunc: (...args: unknown[]) => unknown,
+  args: GraphQLArgs
+) => {
   const pre16Version =
     !version || version.startsWith('14.') || version.startsWith('15.');
   if (pre16Version) {