07646a7f72
The messaging gateway's /model wrote only model + provider to the session row. A row the Desktop/TUI had persisted with the Nous Portal route kept that base_url and api_mode, so resuming a chat switched to openai-codex built provider=openai-codex on the Portal URL and posted gpt-6-luna-900k to inference-api.nousresearch.com/v1/chat/completions: "Model 'gpt-6-luna-900k' isn't available on ChatGPT or Codex Subscription". - SessionDB.update_session_model writes the whole route (provider, base_url, api_mode) in both shapes resume reads (top-level for TUI/Desktop, gateway_runtime for the CLI) whenever a provider is given; the gateway /model passes the switch result's endpoint. - Resume readers (TUI/Desktop, CLI, gateway rehydrate) drop a persisted base_url that is another built-in provider's canonical endpoint, so rows already written by older builds heal. - A gateway session override whose credentials failed to re-resolve is resolved for its own provider on the turn instead of being layered over the default provider's runtime (which produced openai-codex + Nous key + Nous URL). If it still cannot resolve, that turn runs on the whole default route with the existing one-shot "Provider fallback" notice; the override is kept and retried next turn.
36 lines
1.8 KiB
Python
36 lines
1.8 KiB
Python
"""A chat switched to another provider must resume on THAT provider's endpoint.
|
|
|
|
The messaging gateway's /model wrote only ``model``/``provider`` over a row the Desktop had persisted
|
|
with the Nous Portal route, so resuming a chat switched to openai-codex sent the Codex slug to the
|
|
Portal's chat/completions ("Model 'gpt-6-luna-900k' isn't available on ChatGPT or Codex").
|
|
"""
|
|
|
|
import json
|
|
|
|
import pytest
|
|
|
|
from hermes_cli.cli_model_switch_mixin import stored_session_route
|
|
from hermes_state import SessionDB
|
|
from tui_gateway.server import _stored_session_runtime_overrides
|
|
|
|
NOUS_ROUTE = {"base_url": "https://inference-api.nousresearch.com/v1", "api_mode": "chat_completions"}
|
|
|
|
|
|
@pytest.mark.parametrize("row_origin", ["gateway_model_switch", "row_written_by_older_build"])
|
|
def test_resume_drops_previous_providers_endpoint(tmp_path, row_origin):
|
|
db = SessionDB(db_path=tmp_path / "state.db")
|
|
db.create_session("s1", source="telegram", model="openai/gpt-6-luna")
|
|
if row_origin == "gateway_model_switch":
|
|
db.update_session_meta("s1", json.dumps({"model": "openai/gpt-6-luna", "provider": "nous", **NOUS_ROUTE}),
|
|
"openai/gpt-6-luna")
|
|
db.update_session_model("s1", "gpt-6-luna-900k", provider="openai-codex")
|
|
else:
|
|
db.update_session_meta("s1", json.dumps({"model": "gpt-6-luna-900k", "provider": "openai-codex", **NOUS_ROUTE}),
|
|
"gpt-6-luna-900k")
|
|
row = db.get_session("s1")
|
|
|
|
desktop = _stored_session_runtime_overrides(row)["model_override"]
|
|
assert (desktop["provider"], desktop["base_url"], desktop["api_mode"]) == ("openai-codex", None, None)
|
|
assert stored_session_route(row, current_model="openai/gpt-6-luna", current_provider="nous") == (
|
|
"gpt-6-luna-900k", "openai-codex", None, None, True)
|