Skip to content

Conversation

@loopingz
Copy link

@loopingz loopingz commented Jan 22, 2026

openai sdk is import for more than just types as it is used in a instanceof instruction

Therefore it will fail with strict dependencies management like pnpm or bazel


Note

Scope: @arizeai/openinference-instrumentation-openai

  • Add openai as a peerDependency (kept in devDependencies for local development) to ensure the SDK is present at runtime for instanceof checks under strict managers like pnpm/Bazel.
  • Include a changeset marking a patch release.

Written by Cursor Bugbot for commit aa00c37. This will update automatically on new commits. Configure here.

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jan 22, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Jan 22, 2026

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@loopingz
Copy link
Author

I have read the CLA Document and I hereby sign the CLA

github-actions bot added a commit that referenced this pull request Jan 22, 2026
@pkg-pr-new
Copy link

pkg-pr-new bot commented Jan 22, 2026

Open in StackBlitz

@arizeai/openinference-core

npm i https://pkg.pr.new/Arize-ai/openinference/@arizeai/openinference-core@2632

@arizeai/openinference-genai

npm i https://pkg.pr.new/Arize-ai/openinference/@arizeai/openinference-genai@2632

@arizeai/openinference-instrumentation-anthropic

npm i https://pkg.pr.new/Arize-ai/openinference/@arizeai/openinference-instrumentation-anthropic@2632

@arizeai/openinference-instrumentation-bedrock

npm i https://pkg.pr.new/Arize-ai/openinference/@arizeai/openinference-instrumentation-bedrock@2632

@arizeai/openinference-instrumentation-bedrock-agent-runtime

npm i https://pkg.pr.new/Arize-ai/openinference/@arizeai/openinference-instrumentation-bedrock-agent-runtime@2632

@arizeai/openinference-instrumentation-beeai

npm i https://pkg.pr.new/Arize-ai/openinference/@arizeai/openinference-instrumentation-beeai@2632

@arizeai/openinference-instrumentation-langchain

npm i https://pkg.pr.new/Arize-ai/openinference/@arizeai/openinference-instrumentation-langchain@2632

@arizeai/openinference-instrumentation-langchain-v0

npm i https://pkg.pr.new/Arize-ai/openinference/@arizeai/openinference-instrumentation-langchain-v0@2632

@arizeai/openinference-instrumentation-mcp

npm i https://pkg.pr.new/Arize-ai/openinference/@arizeai/openinference-instrumentation-mcp@2632

@arizeai/openinference-instrumentation-openai

npm i https://pkg.pr.new/Arize-ai/openinference/@arizeai/openinference-instrumentation-openai@2632

@arizeai/openinference-mastra

npm i https://pkg.pr.new/Arize-ai/openinference/@arizeai/openinference-mastra@2632

@arizeai/openinference-semantic-conventions

npm i https://pkg.pr.new/Arize-ai/openinference/@arizeai/openinference-semantic-conventions@2632

@arizeai/openinference-vercel

npm i https://pkg.pr.new/Arize-ai/openinference/@arizeai/openinference-vercel@2632

commit: aa00c37

@mikeldking
Copy link
Collaborator

the general rule for auto-instrumentors is to be graceful if the package it's supposed to instrument is not installed. However given that we are for some reason using instanceof (maybe this is the case here), I think it's safe to add this for now until we can maybe use some level of duck typing.

@mikeldking
Copy link
Collaborator

@claude add a changeset to this PR with the changes so that it can be released.

@claude
Copy link

claude bot commented Jan 22, 2026

Claude finished @mikeldking's task in 1m 20s —— View job


Adding changeset for OpenAI peer dependency fix ✅

I've successfully added a changeset for this PR.

Tasks:

  • Examine existing changesets to understand format
  • Create changeset for @arizeai/openinference-instrumentation-openai
  • Commit and push the changeset

What was done:

Created a patch-level changeset at js/.changeset/openai-peer-dependency.md that will:

  • Bump @arizeai/openinference-instrumentation-openai to the next patch version
  • Include a changelog entry explaining that the openai SDK is now a peer dependency to fix strict dependency management issues with pnpm and bazel

The changeset has been committed and pushed to the PR branch. When this PR is merged, the release-please bot will pick up this changeset and include it in the next release.

--- |

Co-authored-by: Mikyo King <mikeldking@users.noreply.github.com>
@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. and removed size:XS This PR changes 0-9 lines, ignoring generated files. labels Jan 22, 2026
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jan 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer size:S This PR changes 10-29 lines, ignoring generated files.

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants