chore: add 'lint:deps' script to check for unused and unlisted deps (#2477)

This uses knip (https://knip.dev) to check dependencies, in hopes of avoiding
issues like #2473. This adds 'npm run lint:deps' and adds that to the
existing 'npm run lint'.

This change includes fixes for a handful of unused and unlisted deps.
For now knip is configured to only check 'production' deps. Checking non-prod
deps results in way too many false positives.

Note that knip is being run via `npx` rather than installing as a devDep because
there is a conflict: knip deps on typescript@5 and all packaegs in this repo
currently use [email protected]. Installing knip at the top-level results in a
ballooning of the node_modules install size as [email protected] is installed
40+ types in all separate workspaces.

Author: trentm

knip.jsonc

@@ -0,0 +1,11 @@
+// https://knip.dev/reference/configuration
+{
+  "$schema": "https://unpkg.com/knip@5/schema-jsonc.json",
+  "workspaces": {
+    "metapackages/auto-instrumentations-node": {
+      // Ensure register.ts is considered an entry-point (the `!` is to be
+      // included with the `--production` flag).
+      "entry": "src/register.ts!"
+    }
+  }
+}

package-lock.json

@@ -23,8 +23,9 @@
     "test-all-versions": "npm run --if-present --workspaces test-all-versions",
     "bump": "lerna publish",
     "changelog": "lerna-changelog",
-    "lint": "lerna run lint && npm run lint:readme && npm run lint:markdown",
+    "lint": "lerna run lint && npm run lint:deps && npm run lint:readme && npm run lint:markdown",
     "lint:fix": "lerna run lint:fix && npm run lint:markdown:fix",
+    "lint:deps": "npx --yes [email protected] --dependencies --production --tags=-knipignore",
     "lint:examples": "eslint ./examples/**/*.js",
     "lint:examples:fix": "eslint ./examples/**/*.js --fix",
     "lint:markdown": "markdownlint-cli2 $(git ls-files '*.md')",

package.json

@@ -44,6 +44,7 @@
   },
   "devDependencies": {
     "@opentelemetry/api": "^1.3.0",
+    "@opentelemetry/sdk-metrics": "^1.8.0",
     "@types/mocha": "8.2.3",
     "@types/node": "18.18.14",
     "@types/sinon": "10.0.20",
@@ -53,7 +54,6 @@
     "typescript": "4.4.4"
   },
   "dependencies": {
-    "@opentelemetry/sdk-metrics": "^1.8.0",
     "systeminformation": "5.22.9"
   },
   "homepage": "https://github.com/open-telemetry/opentelemetry-js-contrib/tree/main/packages/opentelemetry-host-metrics#readme"

packages/opentelemetry-host-metrics/package.json

@@ -16,6 +16,7 @@
 
 import { Meter, MeterProvider, diag, metrics } from '@opentelemetry/api';
 
+/** @knipignore */
 import { PACKAGE_NAME, PACKAGE_VERSION } from './version';
 
 /**

packages/opentelemetry-host-metrics/src/BaseMetrics.ts

@@ -47,7 +47,9 @@
     "@opentelemetry/core": "^1.0.0",
     "@opentelemetry/exporter-jaeger": "^1.3.1",
     "@opentelemetry/instrumentation": "^0.54.0",
+    "@opentelemetry/otlp-transformer": "^0.54.0",
     "@opentelemetry/resources": "^1.8.0",
+    "@opentelemetry/sdk-metrics": "^1.27.0",
     "@opentelemetry/sdk-node": "^0.54.0",
     "@opentelemetry/sdk-trace-base": "^1.8.0",
     "@opentelemetry/sdk-trace-node": "^1.8.0",

packages/opentelemetry-test-utils/package.json

@@ -16,6 +16,7 @@
 
 import { Logger, logs } from '@opentelemetry/api-logs';
 import TransportStream = require('winston-transport');
+/** @knipignore */
 import { PACKAGE_NAME, PACKAGE_VERSION } from './version';
 import { emitLogRecord } from './utils';
 

packages/winston-transport/src/OpenTelemetryTransportV3.ts

@@ -74,6 +74,7 @@ import {
   normalizeExchange,
   unmarkConfirmChannelTracing,
 } from './utils';
+/** @knipignore */
 import { PACKAGE_NAME, PACKAGE_VERSION } from './version';
 
 const supportedVersions = ['>=0.5.5 <1'];

plugins/node/instrumentation-amqplib/src/amqplib.ts

@@ -45,6 +45,7 @@
   },
   "devDependencies": {
     "@cucumber/cucumber": "^9.0.0",
+    "@cucumber/messages": "^22.0.0",
     "@opentelemetry/api": "^1.0.0",
     "@opentelemetry/core": "^1.3.1",
     "@opentelemetry/sdk-trace-base": "^1.3.1",

plugins/node/instrumentation-cucumber/package.json

@@ -38,6 +38,7 @@ import type {
 } from '@cucumber/cucumber/lib/support_code_library_builder/types';
 
 import { AttributeNames, CucumberInstrumentationConfig } from './types';
+/** @knipignore */
 import { PACKAGE_NAME, PACKAGE_VERSION } from './version';
 
 const hooks = ['Before', 'BeforeStep', 'AfterStep', 'After'] as const;