Skip to content

fix(infra): skip serdes tests in min-version release step#36730

Merged
ccurme (ccurme) merged 1 commit intomasterfrom
cc/update_release_workflow
Apr 14, 2026
Merged

fix(infra): skip serdes tests in min-version release step#36730
ccurme (ccurme) merged 1 commit intomasterfrom
cc/update_release_workflow

Conversation

@ccurme
Copy link
Copy Markdown
Collaborator

test_serdes snapshot tests fail when langchain-core is installed from PyPI rather than as a local editable package. Pydantic's FieldInfo.get_default() behaves differently between editable and wheel installs for fields using default_factory, causing extra fields (openai_proxy, output_version, azure_ad_token) to appear in serialization output.

This only surfaced now because langchain-core==1.2.29 was released off the v1.2 branch, so the min-version step resolves to the PyPI package rather than the local editable source.

The test_serdes tests still run in the normal test step. This skips them only in the min-version step. This change should be reverted once the root cause is fixed in core.

Authored using deepagents-cli.

@github-actions github-actions Bot added fix For PRs that implement a fix github_actions Pull requests that update GitHub Actions code infra PRs made that include chores, devops, repo meta changes internal size: XS < 50 LOC labels Apr 14, 2026
@ccurme ccurme (ccurme) merged commit d647311 into master Apr 14, 2026
93 checks passed
@ccurme ccurme (ccurme) deleted the cc/update_release_workflow branch April 14, 2026 22:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix For PRs that implement a fix github_actions Pull requests that update GitHub Actions code infra PRs made that include chores, devops, repo meta changes internal size: XS < 50 LOC

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant