Skip to content

fix: pin opentelemetry-instrumentation-langchain and add telemetry integration tests#143

Open
LucasAlvesSoares wants to merge 22 commits into
mainfrom
telemetry-integration-tests
Open

fix: pin opentelemetry-instrumentation-langchain and add telemetry integration tests#143
LucasAlvesSoares wants to merge 22 commits into
mainfrom
telemetry-integration-tests

Conversation

@LucasAlvesSoares
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Member

@cassiofariasmachado cassiofariasmachado left a comment

Choose a reason for hiding this comment

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

Nice work so far!

Comment thread tests/core/integration/telemetry/telemetry.feature
@LucasAlvesSoares LucasAlvesSoares marked this pull request as ready for review June 5, 2026 17:37
@LucasAlvesSoares LucasAlvesSoares requested a review from a team as a code owner June 5, 2026 17:37
The old lock file had a corrupted importlib-metadata entry that caused
uv to fail with "missing source field but has more than one matching package".
Lift State TypedDict to module level so ty can see it satisfies
StateGraph's type bound. Remove now-stale ignore comments in DMS tests.
ty cannot verify that TypedDict subclasses satisfy the TypedDictLikeV1
protocol, causing a false positive. Switching to a dataclass satisfies
the DataclassLike protocol bound that ty can check statically.
@LucasAlvesSoares LucasAlvesSoares changed the title test: telemetry integration tests fix: pin opentelemetry-instrumentation-langchain and add telemetry integration tests Jun 5, 2026
Copy link
Copy Markdown
Member

@cassiofariasmachado cassiofariasmachado left a comment

Choose a reason for hiding this comment

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

LGTM. Just a friendly reminder: descriptions help reviewers 😅

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.

2 participants