Skip to content

fix: align parser span kind with OpenInference - #1026

Merged
radu-mocanu merged 1 commit into
mainfrom
fix/openinference-parser-span-kind
Aug 7, 2026
Merged

fix: align parser span kind with OpenInference#1026
radu-mocanu merged 1 commit into
mainfrom
fix/openinference-parser-span-kind

Conversation

@radu-mocanu

@radu-mocanu radu-mocanu commented Aug 6, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • ensure parser spans use the OpenInference CHAIN classification consistently
  • keep OpenInference within the 0.1.x compatibility range
  • apply the two-day package quarantine consistently across integration testcases

Why

OpenInference 0.1.69 changed parser spans from UNKNOWN to CHAIN in Arize-ai/openinference#3486. Requiring that version keeps runtime traces and test expectations deterministic.

Copilot AI review requested due to automatic review settings August 6, 2026 12:10

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the project’s OpenInference LangChain instrumentation dependency to ensure parser spans are classified as CHAIN (per the OpenInference contract change), and aligns the ticket-classification trace expectations accordingly.

Changes:

  • Bump openinference-instrumentation-langchain to >=0.1.69 (and lock to 0.1.69) to match updated span-kind behavior.
  • Update the expected trace fixture to assert PydanticOutputParser spans are openinference.span.kind = CHAIN.
  • Increment package version to 0.15.4 and keep uv “exclude-newer” settings aligned.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
pyproject.toml Raises the minimum OpenInference LangChain instrumentation version and bumps the package version to 0.15.4.
uv.lock Locks the updated dependency version and reflects the project version bump in the editable package entry.
testcases/ticket-classification/expected_traces.json Updates span-kind expectation from UNKNOWN to CHAIN for parser spans to match OpenInference behavior.

@radu-mocanu
radu-mocanu force-pushed the fix/openinference-parser-span-kind branch from efb18dd to a249d99 Compare August 6, 2026 12:26
@radu-mocanu
radu-mocanu force-pushed the fix/openinference-parser-span-kind branch from a249d99 to ebc3f0c Compare August 6, 2026 16:55
@radu-mocanu
radu-mocanu enabled auto-merge (squash) August 7, 2026 07:10
@sonarqubecloud

sonarqubecloud Bot commented Aug 7, 2026

Copy link
Copy Markdown

@radu-mocanu
radu-mocanu merged commit 241b5ad into main Aug 7, 2026
19 of 29 checks passed
@radu-mocanu
radu-mocanu deleted the fix/openinference-parser-span-kind branch August 7, 2026 07:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants