mirror of
https://github.com/unslothai/unsloth.git
synced 2026-08-13 02:44:21 +00:00
6d8c18cd1a
* Replace standalone Studio wording with Unsloth Replace the single word Studio with Unsloth wherever it is used as shorthand for Unsloth Studio in docs, CLI output, UI strings, i18n locales, workflow display names, comments and docstrings. Kept unchanged: the full name Unsloth Studio, third party product names (LM Studio, Visual Studio, Mac Studio), feature names (Recipe Studio, Fine-tuning Studio and its translations), and all identifiers such as env vars, commands, paths and filenames. * Address review feedback on the Studio wording rename Use "an" before Unsloth where the rename left the article as "a". Restore the split brand where Unsloth and Studio render as two halves of the full product name: the onboarding sidebar subtitle and the IPv6 localhost warning. Scope two messages to the full name Unsloth Studio where plain Unsloth was misleading: the AMD README bullet and the CLI studio setup error.
53 lines
1.7 KiB
Python
53 lines
1.7 KiB
Python
# SPDX-License-Identifier: AGPL-3.0-only
|
|
# Copyright 2026-present the Unsloth AI Inc. team. All rights reserved. See /studio/LICENSE.AGPL-3.0
|
|
|
|
"""Empty ``role="tool"`` content must be accepted on the OpenAI-compat surface.
|
|
|
|
Agentic clients send ``content: ""`` when a command produced no output;
|
|
OpenAI and llama-server both accept it. Unsloth used to 400, which standard
|
|
clients treat as non-retryable and kill the session. The validator must
|
|
normalize empty/missing tool content to ``""`` instead of raising.
|
|
"""
|
|
|
|
from __future__ import annotations
|
|
|
|
import sys
|
|
from pathlib import Path
|
|
|
|
import pytest
|
|
|
|
_BACKEND_DIR = str(Path(__file__).resolve().parent.parent)
|
|
if _BACKEND_DIR not in sys.path:
|
|
sys.path.insert(0, _BACKEND_DIR)
|
|
|
|
from models.inference import ChatMessage
|
|
|
|
|
|
def test_tool_message_empty_string_content_is_accepted():
|
|
msg = ChatMessage(role = "tool", content = "", tool_call_id = "call_1")
|
|
assert msg.content == ""
|
|
|
|
|
|
def test_tool_message_none_content_normalizes_to_empty_string():
|
|
msg = ChatMessage(role = "tool", content = None, tool_call_id = "call_1")
|
|
assert msg.content == ""
|
|
|
|
|
|
def test_tool_message_empty_list_content_normalizes_to_empty_string():
|
|
msg = ChatMessage(role = "tool", content = [], tool_call_id = "call_1")
|
|
assert msg.content == ""
|
|
|
|
|
|
def test_tool_message_real_content_is_preserved():
|
|
msg = ChatMessage(role = "tool", content = "ok", tool_call_id = "call_1")
|
|
assert msg.content == "ok"
|
|
|
|
|
|
def test_user_message_still_requires_content():
|
|
with pytest.raises(ValueError):
|
|
ChatMessage(role = "user", content = None)
|
|
|
|
|
|
def test_assistant_empty_content_still_collapses_to_none():
|
|
msg = ChatMessage(role = "assistant", content = "")
|
|
assert msg.content is None
|