Feat: strip unsupported sampling params for models flagged shouldSkipTemperature

[cosminacho]

Problem

Claude Opus 4.7 (and any future model that sets modelDetails.shouldSkipTemperature: true in discovery) rejects temperature / top_k / top_p with 400 Bad Request. Callers must not be forced to know which models are reasoning-only — the clients should strip these params transparently.

Solution

Single mechanism on UiPathBaseLLMClient / UiPathBaseChatModel:

• UiPathBaseLLMClient.model_details: dict | None — optional ctor field carrying the discovery modelDetails dict. If omitted, resolved_model_details lazy-fetches via get_model_info().
• UiPathBaseLLMClient._should_skip_sampling_params — reads modelDetails.shouldSkipTemperature, with a name-based fallback (is_claude_opus_4_or_above) for models not in discovery.
• UiPathBaseChatModel._strip_unsupported_sampling_params_at_init (model_validator(mode="after")) — nulls temperature / top_k / top_p AND discards them from __pydantic_fields_set__, so UiPathChat(model="...", temperature=0.6) doesn't leak the value to payload builders that key on model_fields_set.
• UiPathBaseChatModel._strip_sampling_from_kwargs — called inside _generate / _agenerate / _stream / _astream to pop the same keys from invocation-time kwargs. Handles llm.invoke("msg", temperature=0.7) and .bind(temperature=0.7).

Factory wiring (this branch's final commit):

• get_chat_model / get_embedding_model extract modelDetails from the model_info they already fetch and pass it through as model_details (via setdefault, so caller override wins). Skips the redundant lazy-fetch inside resolved_model_details.

All five per-client sampling-param overrides removed.

UiPathChat._default_params still drops temperature when thinking is set (Anthropic extended thinking requires temperature=1, its default).

UiPathChatBedrockConverse._converse_params defensively filters None temperature / topP from inferenceConfig so boto3 doesn't serialize them as explicit nulls.

Core additions (uipath.llm_client.utils.model_family):

• is_claude_opus_4_or_above(model_name) — name-based fallback for models missing from discovery
• CLAUDE_OPUS_4_UNSUPPORTED_SAMPLING_PARAMS = frozenset({"temperature", "top_k", "top_p"})

Which packages

• Core → 1.9.10 (model_family.py helpers)
• Langchain → 1.9.9 (base-client refactor + factory piping)

Test plan

• ruff check / ruff format --check / pyright
• pytest tests — 1645 passed, 769 skipped, 9 xpassed
• TestSamplingParamStripping in tests/langchain/clients/bedrock/test_unit.py covers ctor-time stripping, invocation-time stripping, and supported-model passthrough
• New factory tests in tests/langchain/features/test_factory_function.py assert modelDetails propagates from discovery to client ctor and that caller overrides win

🤖 Generated with Claude Code