fix(types): import opentracing from the consumer's installed package

`opentracing.initGlobalTracer(tracer)` fails to type-check because dd-trace's
public types reach into `./vendor/dist/opentracing` while consumers reach into
their own `node_modules/opentracing`. `Reference` / `Tracer` / `Span` declare
`protected` members, so the two copies are nominally distinct classes and the
dd-trace `Tracer` is not a subtype of the consumer's `opentracing.Tracer`.

Type imports reference `opentracing` directly again and the package becomes an
optional peer dependency so consumers using the integration share a single
class identity with dd-trace's types without forcing every install to pull in
opentracing. The vendored type copy and its rspack build step are dropped;
dd-trace's runtime never required the package.

Fixes: #8523
Refs: #6958

Author: BridgeAR

LICENSE-3rdparty.csv

@@ -83,7 +83,6 @@
 "mutexify","https://github.com/mafintosh/mutexify","['MIT']","['Mathias Buus']"
 "node-addon-api","https://github.com/nodejs/node-addon-api","['MIT']","['nodejs']"
 "node-gyp-build","https://github.com/prebuild/node-gyp-build","['MIT']","['Mathias Buus']"
-"opentracing","https://github.com/opentracing/opentracing-javascript","['Apache-2.0']","['opentracing']"
 "oxc-parser","https://github.com/oxc-project/oxc","['MIT']","['Boshen and oxc contributors']"
 "pprof-format","https://github.com/DataDog/pprof-format","['MIT']","['Datadog Inc.']"
 "protobufjs","https://github.com/protobufjs/protobuf.js","['BSD-3-Clause']","['Daniel Wirtz']"

docs/API.md

@@ -379,6 +379,8 @@ const opentracing = require('opentracing')
 opentracing.initGlobalTracer(tracer)
 ```
 
+`opentracing` is declared as an optional peer dependency and is not installed with `dd-trace`. Add it to your own dependencies (`npm install --save opentracing`) when using the OpenTracing API so the runtime and TypeScript types line up with the same package instance.
+
 The following tags are available to override Datadog-specific options:
 
 * `service.name`: The service name to be used for this span. The service name from the tracer will be used if this is not provided.

docs/test.ts

@@ -25,7 +25,7 @@ import {
   SPAN_TYPE,
 } from '../ext/tags'
 import { HTTP, WEB } from '../ext/types'
-import * as opentracing from '../vendor/dist/opentracing';
+import * as opentracing from 'opentracing';
 import { IncomingMessage, OutgoingMessage } from 'http';
 
 opentracing.initGlobalTracer(tracer);

index.d.ts

@@ -1,6 +1,6 @@
 import { ClientRequest, IncomingMessage, OutgoingMessage, ServerResponse } from "http";
 import { LookupFunction } from 'net';
-import * as opentracing from "./vendor/dist/opentracing";
+import * as opentracing from "opentracing";
 import * as otel from "@opentelemetry/api";
 
 /**

index.d.v5.ts

@@ -1,6 +1,6 @@
 import { ClientRequest, IncomingMessage, OutgoingMessage, ServerResponse } from "http";
 import { LookupFunction } from 'net';
-import * as opentracing from "./vendor/dist/opentracing";
+import * as opentracing from "opentracing";
 import * as otel from "@opentelemetry/api";
 
 /**

package.json

@@ -163,6 +163,14 @@
     "dc-polyfill": "^0.1.11",
     "import-in-the-middle": "^3.0.1"
   },
+  "peerDependencies": {
+    "opentracing": ">=0.14.7"
+  },
+  "peerDependenciesMeta": {
+    "opentracing": {
+      "optional": true
+    }
+  },
   "optionalDependencies": {
     "@datadog/libdatadog": "0.9.3",
     "@datadog/native-appsec": "11.0.1",

packages/dd-trace/test/opentracing/tracer.spec.js

@@ -7,7 +7,7 @@ const { describe, it, beforeEach } = require('mocha')
 const sinon = require('sinon')
 const proxyquire = require('proxyquire')
 
-const opentracing = require('../../../../vendor/dist/opentracing')
+const opentracing = require('opentracing')
 require('../setup/core')
 const SpanContext = require('../../src/opentracing/span_context')
 const formats = require('../../../../ext/formats')

packages/dd-trace/test/scope.spec.js

@@ -5,7 +5,7 @@ const assert = require('node:assert/strict')
 const { describe, it, beforeEach } = require('mocha')
 const sinon = require('sinon')
 
-const { Span } = require('../../../vendor/dist/opentracing')
+const { Span } = require('opentracing')
 require('./setup/core')
 const Scope = require('../src/scope')
 

vendor/package-lock.json

@@ -22,7 +22,6 @@
     "meriyah": "^6.1.4",
     "module-details-from-path": "^1.0.4",
     "mutexify": "^1.4.0",
-    "opentracing": ">=0.14.7",
     "pprof-format": "^2.1.1",
     "protobufjs": "^8.0.1",
     "retry": "^0.13.1",