fix: only import types in all instrumentations (#150)

traceloop · Mar 15, 2024 · 7304563 · 7304563
1 parent ddfc3a2
commit 7304563
Show file tree

Hide file tree

Showing 16 changed files with 126 additions and 253 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/packages/instrumentation-azure/package.json b/packages/instrumentation-azure/package.json
@@ -46,7 +46,6 @@
     "@pollyjs/adapter-node-http": "^6.0.6",
     "@pollyjs/core": "^6.0.6",
     "@pollyjs/persister-fs": "^6.0.6",
-    "@types/expect": "^24.3.0",
     "@types/mocha": "^10.0.6",
     "llamaindex": "^0.0.40",
     "ts-mocha": "^10.0.0"

diff --git a/packages/instrumentation-azure/src/instrumentation.ts b/packages/instrumentation-azure/src/instrumentation.ts
@@ -33,7 +33,7 @@ import {
   SpanAttributes,
 } from "@traceloop/ai-semantic-conventions";
 import { AzureOpenAIInstrumentationConfig } from "./types";
-import {
+import type {
   ChatCompletions,
   ChatRequestMessage,
   Completions,

diff --git a/packages/instrumentation-bedrock/src/instrumentation.ts b/packages/instrumentation-bedrock/src/instrumentation.ts
@@ -28,7 +28,7 @@ import {
   safeExecuteInTheMiddle,
 } from "@opentelemetry/instrumentation";
 import { BedrockInstrumentationConfig } from "./types";
-import * as bedrock from "@aws-sdk/client-bedrock-runtime";
+import type * as bedrock from "@aws-sdk/client-bedrock-runtime";
 import {
   CONTEXT_KEY_ALLOW_TRACE_CONTENT,
   LLMRequestTypeValues,