forkless/strix
1
0
Fork 0
mirror of https://github.com/usestrix/strix.git synced 2026-08-24 11:52:38 +02:00
Code Issues Packages Projects Releases Wiki Activity

fix(llm): keep tool-call ids unique so a recycled id can't erase history (#976)

Browse Source 
Co-authored-by: Ahmed Allam <ahmed39652003@gmail.com>
This commit is contained in:
devin-ai-integration[bot]
2026-08-06 00:06:46 +03:00
committed by GitHub
co-authored by Ahmed Allam
parent 657aa5cbe6
commit 68ea6fca65
5 changed files with 482 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/test_disable_streaming.py
+6 -4
View File
@@ -31,7 +31,7 @@ from openai.types.responses import (
from strix.config import codex, loader
from strix.config.loader import load_settings
from strix.config.models import StrixProvider, _NonStreamingModel
from strix.config.models import StrixProvider, _NonStreamingModel, _UniqueToolCallIdModel
if TYPE_CHECKING:
@@ -299,10 +299,11 @@ def test_get_model_wraps_when_disabled(
load_settings()
model = StrixProvider().get_model("openai/gpt-4o-mini")
assert isinstance(model, _NonStreamingModel)
assert isinstance(model, _UniqueToolCallIdModel)
assert isinstance(model._inner, _NonStreamingModel)
def test_get_model_unwrapped_by_default(
def test_get_model_keeps_streaming_by_default(
monkeypatch: pytest.MonkeyPatch, _reset_settings: None
) -> None:
inner = _DummyModel()
@@ -310,7 +311,8 @@ def test_get_model_unwrapped_by_default(
load_settings()
model = StrixProvider().get_model("openai/gpt-4o-mini")
assert model is inner
assert isinstance(model, _UniqueToolCallIdModel)
assert model._inner is inner
def test_get_model_does_not_wrap_subscription_model(