Back

langchain #34848

fix(deepseek): Tool Choice to `required` for Azure Deployment

by keenborder786 · Feb 06, 2026 at 19:30 UTC · scan-b0e7f300d6075bb4

View on Github · Download JSON
Low Risk (20%)

Get this automatically on every PR

Install the Axiomo GitHub App to get Signals as check runs and PR comments on every pull request.

Install App

Risk Assessment

Risk level: Low (20%)

Risk Drivers

  • api_surface_change: API surface changed in 1 file(s)

Intent

2/2 criteria met

Make Tool Choice a required parameter for Azure Deepseek deployment.

Acceptance Criteria

  • ✓
    Tool Choice is set to required for Azure.

    langchain_deepseek/chat_models.py adds logic to bind tools and check base URL relevant to Azure.

  • ✓
    Tests cover Azure integration changes.

    test_chat_models.py adds tests related to Azure OpenAI integration.

Confidence: 90.0% Source: diff analysis AI: openai

Contributors

keenborder786 PR Author 10 commits ~ Established
Account Age: 2634 days
Prior PRs: 163
Merged: 89

Established contributor with 89 merged PRs. maintains 101 public repositories. unfamiliar with 2 files.

Evidence

Evidence Completeness: 60.0%
ci_passing Failing
lint_passing Passing
tests_passing Unavailable
build_successful Passing
Missing: security_scan_clean, coverage_maintained

Supply Chain

None Risk
Modifies dependencies
Modifies lockfile
Modifies CI config
Modifies build scripts

Focus Files

Review 1 high-priority file(s)

libs/partners/deepseek/tests/unit_tests/test_chat_models.py +113

113 lines changed; Source code

high
libs/partners/deepseek/langchain_deepseek/chat_models.py +6

Source code

medium

Triage

20

minutes to review

medium

effort level

none

staleness risk

Standard review process

Recommendation

COMMENT 68.0% readiness

Some concerns to address before approval

Next Steps

Question

Why is security_scan_clean missing? Consider adding this check.

Question

Why is coverage_maintained missing? Consider adding this check.