diff --git a/CHANGELOG.md b/CHANGELOG.md
index 53e68f3..c2bbef7 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,5 +1,22 @@
# Hermes Web UI -- Changelog
+## [v0.50.40] feat: session UI polish + parallel test isolation
+
+**Session sidebar improvements:**
+- `static/sessions.js` + `style.css`: Hide session timestamps to give titles full available width — no more title truncation from inline timestamps (PR #449)
+- `static/style.css`: Active session title now uses `var(--gold)` theme variable instead of hardcoded `#e8a030` — adapts correctly across all 7 themes (PR #451, fixes #440)
+- `api/models.py` + `api/gateway_watcher.py`: Return `None` instead of the string `'unknown'` for missing gateway session model — Telegram sessions no longer show `telegram · unknown` (PR #452, fixes #443)
+- `static/style.css` + `static/sessions.js`: Mute Telegram badge from saturated `#0088cc` to `rgba(0, 136, 204, 0.55)`. Add `_formatSourceTag()` helper mapping platform IDs to display names (`telegram` → `via Telegram`) (PR #453, fixes #442)
+
+**Bug fixes:**
+- `api/config.py` `resolve_model_provider()`: Strip provider prefix from model ID when a custom `base_url` is configured (`openai/gpt-5.4` → `gpt-5.4`) — fixes broken chats after switching to a custom endpoint (PR #454, fixes #433)
+- `static/panels.js` `switchToProfile()`: Apply profile default workspace to new session created during profile switch — workspace chip no longer shows "No active workspace" after switching profiles mid-conversation (PR #455, fixes #424)
+
+**Test infrastructure:**
+- `tests/conftest.py` + `tests/_pytest_port.py` (new): Auto-derive unique port and state dir per worktree from repo path hash (range 20000-29999). Running pytest in two worktrees simultaneously no longer causes port conflicts. All 43 test files updated from hardcoded `BASE = "http://127.0.0.1:8788"` to `from tests._pytest_port import BASE` (PR #456)
+
+- Total tests: 1098 (was 1078)
+
## [v0.50.39] fix: orphan gateway sessions + first-password-enablement session continuity
Two bug fixes:
diff --git a/TESTING.md b/TESTING.md
index e547d67..75386bd 100644
--- a/TESTING.md
+++ b/TESTING.md
@@ -8,7 +8,7 @@
> Prerequisites: SSH tunnel is active on port 8787. Open http://localhost:8787 in browser.
> Server health check: curl http://127.0.0.1:8787/health should return {"status":"ok"}.
>
-> Automated tests: 1078 total (1078 passing, 0 known failures). Includes onboarding coverage for bootstrap/static wizard presence, real provider config persistence (`config.yaml` + `.env`), the `/api/onboarding/*` backend, and the onboarding skip/existing-config guard.
+> Automated tests: 1098 total (1098 passing, 0 known failures). Includes onboarding coverage for bootstrap/static wizard presence, real provider config persistence (`config.yaml` + `.env`), the `/api/onboarding/*` backend, and the onboarding skip/existing-config guard.
> Run: `pytest tests/ -v --timeout=60`
---
diff --git a/api/config.py b/api/config.py
index 0839634..55df1af 100644
--- a/api/config.py
+++ b/api/config.py
@@ -637,7 +637,10 @@ def resolve_model_provider(model_id: str) -> tuple:
# just because the model name contains a slash (e.g. google/gemma-4-26b-a4b).
# The user has explicitly pointed at a base_url, so trust their routing config.
if config_base_url:
- return model_id, config_provider, config_base_url
+ # Strip provider prefix (e.g. 'openai/gpt-5.4' -> 'gpt-5.4') so prefixed
+ # model IDs from previous sessions don't break custom endpoint routing.
+ bare_model = model_id.split('/', 1)[-1]
+ return bare_model, config_provider, config_base_url
# If prefix does NOT match config provider, the user picked a cross-provider model
# from the OpenRouter dropdown (e.g. config=anthropic but picked openai/gpt-5.4-mini).
# In this case always route through openrouter with the full provider/model string.
diff --git a/api/gateway_watcher.py b/api/gateway_watcher.py
index fe1a37b..409e4d5 100644
--- a/api/gateway_watcher.py
+++ b/api/gateway_watcher.py
@@ -75,7 +75,7 @@ def _get_agent_sessions_from_db() -> list:
sessions.append({
'session_id': row['id'],
'title': row['title'] or 'Agent Session',
- 'model': row['model'] or 'unknown',
+ 'model': row['model'] or None,
'message_count': row['message_count'] or 0,
'created_at': row['started_at'],
'updated_at': row['last_activity'] or row['started_at'],
diff --git a/api/models.py b/api/models.py
index 19acb75..be5dde2 100644
--- a/api/models.py
+++ b/api/models.py
@@ -309,7 +309,7 @@ def get_cli_sessions() -> list:
'session_id': sid,
'title': _display_title,
'workspace': str(get_last_workspace()),
- 'model': row['model'] or 'unknown',
+ 'model': row['model'] or None,
'message_count': row['message_count'] or 0,
'created_at': row['started_at'],
'updated_at': raw_ts,
diff --git a/static/index.html b/static/index.html
index 18da691..13f97ad 100644
--- a/static/index.html
+++ b/static/index.html
@@ -536,7 +536,7 @@
System
Instance version and access controls.
- v0.50.39
+ v0.50.40
diff --git a/static/panels.js b/static/panels.js
index 520cfc8..237ab65 100644
--- a/static/panels.js
+++ b/static/panels.js
@@ -947,6 +947,18 @@ async function switchToProfile(name) {
// The current session has messages and belongs to the previous profile.
// Start a new session for the new profile so nothing gets cross-tagged.
await newSession(false);
+ // Apply profile default workspace to the newly created session (fixes #424)
+ if (S._profileDefaultWorkspace && S.session) {
+ try {
+ await api('/api/session/update', { method: 'POST', body: JSON.stringify({
+ session_id: S.session.session_id,
+ workspace: S._profileDefaultWorkspace,
+ model: S.session.model,
+ })});
+ S.session.workspace = S._profileDefaultWorkspace;
+ } catch (_) {}
+ }
+ updateWorkspaceChip();
await renderSessionList();
showToast(t('profile_switched_new_conversation', name));
} else {
diff --git a/static/sessions.js b/static/sessions.js
index 7185756..1c96ec3 100644
--- a/static/sessions.js
+++ b/static/sessions.js
@@ -578,6 +578,10 @@ function renderSessionListFromCache(){
}
// ── Render session items (extracted for group body use) ──
// Note: declared after the groups loop but available via function hoisting.
+ function _formatSourceTag(tag){
+ const names={telegram:'via Telegram',discord:'via Discord',slack:'via Slack',cli:'CLI',feishu:'via Feishu',weixin:'via WeChat'};
+ return names[tag]||tag;
+ }
function _renderOneSession(s){
const el=document.createElement('div');
const isActive=S.session&&s.session_id===S.session.session_id;
@@ -596,14 +600,9 @@ function renderSessionListFromCache(){
title.textContent=cleanTitle||'Untitled';
title.title='Double-click to rename';
const tsMs=_sessionTimestampMs(s);
- const timeLabel=document.createElement('span');
- timeLabel.className='session-time';
- timeLabel.textContent=_formatRelativeSessionTime(tsMs, now);
- if(tsMs) timeLabel.title=new Date(tsMs).toLocaleString();
titleRow.appendChild(title);
- titleRow.appendChild(timeLabel);
const metaBits=[];
- if(s.is_cli_session && s.source_tag) metaBits.push(s.source_tag);
+ if(s.is_cli_session && s.source_tag) metaBits.push(_formatSourceTag(s.source_tag));
if(s.message_count) metaBits.push(t('n_messages', s.message_count));
if(s.model) metaBits.push(String(s.model).split('/').pop());
sessionText.appendChild(titleRow);
diff --git a/static/style.css b/static/style.css
index ef8632f..ec2a92c 100644
--- a/static/style.css
+++ b/static/style.css
@@ -172,8 +172,8 @@
.session-text{flex:1;min-width:0;display:flex;flex-direction:column;gap:2px;overflow:hidden;}
.session-title-row{display:flex;align-items:flex-start;gap:8px;min-width:0;}
.session-title{flex:1;overflow:hidden;text-overflow:ellipsis;white-space:nowrap;color:var(--text);}
- .session-item.active .session-title{color:#e8a030;}
- .session-time{flex-shrink:0;font-size:11px;line-height:1.4;color:var(--muted);text-transform:lowercase;}
+ .session-item.active .session-title{color:var(--gold);}
+ .session-time{display:none;}
.session-meta{font-size:11px;line-height:1.35;color:var(--muted);white-space:nowrap;overflow:hidden;text-overflow:ellipsis;}
/* ── Session action trigger + dropdown ── */
.session-actions{position:absolute;right:6px;top:50%;transform:translateY(-50%);display:flex;align-items:center;justify-content:center;opacity:0;pointer-events:none;transition:opacity .15s ease;}
@@ -1099,8 +1099,8 @@ body.resizing{user-select:none;cursor:col-resize;}
display: none;
}
/* Source-specific colors for gateway sessions */
-.session-item.cli-session[data-source="telegram"] { border-left-color: #0088cc; }
-.session-item.cli-session[data-source="telegram"]::after { color: #0088cc; }
+.session-item.cli-session[data-source="telegram"] { border-left-color: rgba(0, 136, 204, 0.55); }
+.session-item.cli-session[data-source="telegram"]::after { color: rgba(0, 136, 204, 0.55); }
.session-item.cli-session[data-source="discord"] { border-left-color: #5865F2; }
.session-item.cli-session[data-source="discord"]::after { color: #5865F2; }
.session-item.cli-session[data-source="slack"] { border-left-color: #4A154B; }
diff --git a/tests/_pytest_port.py b/tests/_pytest_port.py
new file mode 100644
index 0000000..19afc6a
--- /dev/null
+++ b/tests/_pytest_port.py
@@ -0,0 +1,42 @@
+"""
+Shared test server constants for use in individual test files.
+
+Instead of hardcoding ``BASE = "http://127.0.0.1:8788"`` in every test file,
+import from here so the port and state dir are always consistent with
+what conftest.py computed for this worktree.
+
+Usage::
+
+ from tests._pytest_port import BASE
+
+conftest.py publishes ``HERMES_WEBUI_TEST_PORT`` and
+``HERMES_WEBUI_TEST_STATE_DIR`` to ``os.environ`` at module level
+(before any test file is imported), so this module always reads the
+correct values. The auto-derivation fallback matches conftest's logic
+exactly, so standalone imports also work correctly.
+"""
+import hashlib
+import os
+import pathlib
+
+def _auto_test_port(repo_root: pathlib.Path) -> int:
+ h = int(hashlib.md5(str(repo_root).encode()).hexdigest(), 16)
+ return 20000 + (h % 10000)
+
+def _auto_state_dir_name(repo_root: pathlib.Path) -> str:
+ h = hashlib.md5(str(repo_root).encode()).hexdigest()[:8]
+ return f"webui-test-{h}"
+
+_TESTS_DIR = pathlib.Path(__file__).parent.resolve()
+_REPO_ROOT = _TESTS_DIR.parent.resolve()
+_HERMES_HOME = pathlib.Path(os.getenv('HERMES_HOME',
+ str(pathlib.Path.home() / '.hermes')))
+
+TEST_PORT = int(os.environ.get('HERMES_WEBUI_TEST_PORT',
+ str(_auto_test_port(_REPO_ROOT))))
+BASE = f"http://127.0.0.1:{TEST_PORT}"
+
+TEST_STATE_DIR = pathlib.Path(os.environ.get(
+ 'HERMES_WEBUI_TEST_STATE_DIR',
+ str(_HERMES_HOME / _auto_state_dir_name(_REPO_ROOT))
+))
diff --git a/tests/conftest.py b/tests/conftest.py
index 910add5..b45d4f3 100644
--- a/tests/conftest.py
+++ b/tests/conftest.py
@@ -31,14 +31,37 @@ HOME = pathlib.Path.home()
HERMES_HOME = pathlib.Path(os.getenv('HERMES_HOME', str(HOME / '.hermes')))
# ── Test server config ────────────────────────────────────────────────────
-TEST_PORT = int(os.getenv('HERMES_WEBUI_TEST_PORT', '8788'))
+# Port and state dir auto-derive from the repo path when no env var is set,
+# giving every worktree its own isolated port (8800-8899) and state directory.
+# Override with HERMES_WEBUI_TEST_PORT / HERMES_WEBUI_TEST_STATE_DIR to pin.
+
+def _auto_test_port(repo_root) -> int:
+ """Map repo path to a unique port in 20000-29999 (10k range = near-zero collisions).
+ Far from system port ranges and Linux ephemeral ports (32768+).
+ Override with HERMES_WEBUI_TEST_PORT to use a specific port."""
+ import hashlib
+ h = int(hashlib.md5(str(repo_root).encode()).hexdigest(), 16)
+ return 20000 + (h % 10000)
+
+def _auto_state_dir_name(repo_root) -> str:
+ import hashlib
+ h = hashlib.md5(str(repo_root).encode()).hexdigest()[:8]
+ return f"webui-test-{h}"
+
+TEST_PORT = int(os.getenv('HERMES_WEBUI_TEST_PORT',
+ str(_auto_test_port(REPO_ROOT))))
TEST_BASE = f"http://127.0.0.1:{TEST_PORT}"
TEST_STATE_DIR = pathlib.Path(os.getenv(
'HERMES_WEBUI_TEST_STATE_DIR',
- str(HERMES_HOME / 'webui-mvp-test')
+ str(HERMES_HOME / _auto_state_dir_name(REPO_ROOT))
))
TEST_WORKSPACE = TEST_STATE_DIR / 'test-workspace'
+# Publish at module level so _pytest_port.py (imported at collection time)
+# and any test file using os.environ sees the right values immediately.
+os.environ.setdefault('HERMES_WEBUI_TEST_PORT', str(TEST_PORT))
+os.environ.setdefault('HERMES_WEBUI_TEST_STATE_DIR', str(TEST_STATE_DIR))
+
# ── Server script: always relative to repo root ───────────────────────────
SERVER_SCRIPT = REPO_ROOT / 'server.py'
if not SERVER_SCRIPT.exists():
@@ -245,7 +268,10 @@ def test_server():
# as the server. Other test files (test_auth_sessions.py) may override
# HERMES_WEBUI_STATE_DIR for their own purposes, but HERMES_WEBUI_TEST_STATE_DIR
# is reserved for this mapping and is never overridden by individual test files.
- os.environ.setdefault('HERMES_WEBUI_TEST_STATE_DIR', str(TEST_STATE_DIR))
+ # Export both port and state-dir as env vars so individual test files
+ # can read them without importing conftest (avoids circular imports).
+ os.environ.setdefault('HERMES_WEBUI_TEST_PORT', str(TEST_PORT))
+ # os.environ already set at module level above; no-op here.
env = os.environ.copy()
env.update({
diff --git a/tests/test_approval_unblock.py b/tests/test_approval_unblock.py
index 9fa10e9..2f6f2c7 100644
--- a/tests/test_approval_unblock.py
+++ b/tests/test_approval_unblock.py
@@ -41,7 +41,7 @@ pytestmark = pytest.mark.skipif(
reason="tools.approval not available in this environment"
)
-BASE = "http://127.0.0.1:8788"
+from tests._pytest_port import BASE
def get(path):
diff --git a/tests/test_gateway_sync.py b/tests/test_gateway_sync.py
index 1f38bf5..39adc5c 100644
--- a/tests/test_gateway_sync.py
+++ b/tests/test_gateway_sync.py
@@ -18,7 +18,7 @@ import urllib.error
import urllib.request
REPO_ROOT = pathlib.Path(__file__).parent.parent.resolve()
-BASE = "http://127.0.0.1:8788"
+from tests._pytest_port import BASE
def get(path):
@@ -49,11 +49,9 @@ def _get_test_state_dir():
set (e.g. when running this file standalone), fall back to the conftest
formula: HERMES_HOME/webui-mvp-test.
"""
- explicit = os.getenv('HERMES_WEBUI_TEST_STATE_DIR')
- if explicit:
- return pathlib.Path(explicit)
- hermes_home = pathlib.Path(os.getenv('HERMES_HOME', str(pathlib.Path.home() / '.hermes')))
- return hermes_home / 'webui-mvp-test' # matches conftest.py TEST_STATE_DIR formula
+ # Use _pytest_port which applies the same auto-derivation as conftest.py
+ from tests._pytest_port import TEST_STATE_DIR as _ptsd
+ return _ptsd
def _get_state_db_path():
diff --git a/tests/test_issue336.py b/tests/test_issue336.py
index 25322ad..48b8d28 100644
--- a/tests/test_issue336.py
+++ b/tests/test_issue336.py
@@ -34,7 +34,7 @@ STYLE_CSS = (REPO_ROOT / "static" / "style.css").read_text()
INDEX_HTML = (REPO_ROOT / "static" / "index.html").read_text()
I18N_JS = (REPO_ROOT / "static" / "i18n.js").read_text()
-BASE = "http://127.0.0.1:8788"
+from tests._pytest_port import BASE
def _get(path):
@@ -261,7 +261,7 @@ class TestBubbleLayoutI18N(unittest.TestCase):
)
-# ── Integration tests (require live server on port 8788) ─────────────────
+# ── Integration tests (require live server on test server port) ─────────────────
class TestBubbleLayoutSettingsAPI(unittest.TestCase):
@@ -272,7 +272,7 @@ class TestBubbleLayoutSettingsAPI(unittest.TestCase):
try:
d, status = _get("/api/settings")
except OSError:
- self.skipTest("Server not running on port 8788")
+ self.skipTest("Server not running on test server port")
self.assertEqual(status, 200)
self.assertIn(
"bubble_layout",
@@ -289,7 +289,7 @@ class TestBubbleLayoutSettingsAPI(unittest.TestCase):
try:
_, status = _post("/api/settings", {"bubble_layout": True})
except OSError:
- self.skipTest("Server not running on port 8788")
+ self.skipTest("Server not running on test server port")
self.assertEqual(status, 200)
d, _ = _get("/api/settings")
self.assertTrue(d["bubble_layout"], "bubble_layout=True must persist after POST")
@@ -302,7 +302,7 @@ class TestBubbleLayoutSettingsAPI(unittest.TestCase):
_post("/api/settings", {"bubble_layout": True})
_post("/api/settings", {"bubble_layout": False})
except OSError:
- self.skipTest("Server not running on port 8788")
+ self.skipTest("Server not running on test server port")
d, _ = _get("/api/settings")
self.assertFalse(d["bubble_layout"], "bubble_layout=False must persist after POST")
@@ -311,7 +311,7 @@ class TestBubbleLayoutSettingsAPI(unittest.TestCase):
try:
_post("/api/settings", {"bubble_layout": "1"})
except OSError:
- self.skipTest("Server not running on port 8788")
+ self.skipTest("Server not running on test server port")
d, _ = _get("/api/settings")
self.assertIsInstance(
d["bubble_layout"],
diff --git a/tests/test_login_locale.py b/tests/test_login_locale.py
index 5cdea58..09829c5 100644
--- a/tests/test_login_locale.py
+++ b/tests/test_login_locale.py
@@ -3,7 +3,7 @@ import urllib.error
import urllib.request
-BASE = "http://127.0.0.1:8788"
+from tests._pytest_port import BASE
def get(path):
diff --git a/tests/test_model_resolver.py b/tests/test_model_resolver.py
index 6176e84..c0a2b4e 100644
--- a/tests/test_model_resolver.py
+++ b/tests/test_model_resolver.py
@@ -403,8 +403,10 @@ def test_custom_endpoint_slash_model_routes_to_custom_not_openrouter():
assert base_url == 'http://127.0.0.1:1234/v1', (
"Expected base_url 'http://127.0.0.1:1234/v1', got '{}'.".format(base_url)
)
- assert model == 'google/gemma-4-26b-a4b', (
- "Model name should be preserved as-is, got '{}'.".format(model)
+ # Fix #433: provider prefix is now stripped for custom endpoints so stale
+ # prefixed model IDs from previous sessions do not break custom endpoint routing.
+ assert model == 'gemma-4-26b-a4b', (
+ "Model name prefix should be stripped for custom base_url endpoint, got '{}'.".format(model)
)
# --- openrouter with slash model name MUST still route to openrouter -----
diff --git a/tests/test_onboarding_existing_config.py b/tests/test_onboarding_existing_config.py
index 2484063..e644b66 100644
--- a/tests/test_onboarding_existing_config.py
+++ b/tests/test_onboarding_existing_config.py
@@ -12,6 +12,7 @@ Covers:
from __future__ import annotations
import json
+import os
import pathlib
import urllib.error
import urllib.request
@@ -187,7 +188,7 @@ class TestApplyOnboardingSetupGuard:
# Integration tests — require the live test server on port 8788
# ---------------------------------------------------------------------------
-BASE = "http://127.0.0.1:8788"
+from tests._pytest_port import BASE
def _http_get(path):
@@ -213,7 +214,7 @@ def _server_hermes_home() -> pathlib.Path:
env_path = data.get("system", {}).get("env_path", "")
if env_path:
return pathlib.Path(env_path).parent
- return pathlib.Path.home() / ".hermes" / "webui-mvp-test"
+ return pathlib.Path(os.environ.get("HERMES_WEBUI_TEST_STATE_DIR", str(pathlib.Path.home() / ".hermes" / "webui-mvp-test")))
def _server_reachable() -> bool:
diff --git a/tests/test_onboarding_mvp.py b/tests/test_onboarding_mvp.py
index 39134d5..82ac607 100644
--- a/tests/test_onboarding_mvp.py
+++ b/tests/test_onboarding_mvp.py
@@ -13,7 +13,7 @@ import urllib.request
import pytest
-BASE = "http://127.0.0.1:8788"
+from tests._pytest_port import BASE
# Check if pyyaml is available — onboarding setup tests need it on the server
try:
diff --git a/tests/test_onboarding_network.py b/tests/test_onboarding_network.py
index 13eab5e..2535212 100644
--- a/tests/test_onboarding_network.py
+++ b/tests/test_onboarding_network.py
@@ -24,7 +24,7 @@ import urllib.request
import pytest
REPO = pathlib.Path(__file__).parent.parent
-BASE = "http://127.0.0.1:8788"
+from tests._pytest_port import BASE
# ---------------------------------------------------------------------------
# Unit tests — directly test the IP-resolution + guard logic in routes.py
@@ -128,14 +128,14 @@ class TestOnboardingIPLogic:
# ---------------------------------------------------------------------------
-# Integration tests — hit the live test server at port 8788
+# Integration tests — hit the live test server at test server port
# ---------------------------------------------------------------------------
@pytest.mark.integration
class TestOnboardingSetupEndpoint:
"""
Integration tests for /api/onboarding/setup.
- These require the test server running on port 8788.
+ These require the test server running on test server port.
"""
def _post(self, path: str, data: dict, headers: dict | None = None) -> tuple[int, dict]:
@@ -157,7 +157,7 @@ class TestOnboardingSetupEndpoint:
Requests from 127.0.0.1 (which is what the test server sees) should
pass the IP check. We confirm no 403 is returned.
"""
- # The test server runs on 127.0.0.1:8788 so client_address[0] is 127.0.0.1.
+ # The test server runs on 127.0.0.1:{TEST_PORT} so client_address[0] is 127.0.0.1.
# A valid setup payload with a mock provider should not be rejected for IP reasons.
# We patch apply_onboarding_setup to avoid actually writing any config.
import unittest.mock
diff --git a/tests/test_provider_mismatch.py b/tests/test_provider_mismatch.py
index f0e9ef9..fb72b94 100644
--- a/tests/test_provider_mismatch.py
+++ b/tests/test_provider_mismatch.py
@@ -16,7 +16,7 @@ import re
import urllib.request
REPO_ROOT = pathlib.Path(__file__).parent.parent.resolve()
-BASE = "http://127.0.0.1:8788"
+from tests._pytest_port import BASE
def _read(rel_path: str) -> str:
diff --git a/tests/test_regressions.py b/tests/test_regressions.py
index ce06ba1..fba311f 100644
--- a/tests/test_regressions.py
+++ b/tests/test_regressions.py
@@ -5,6 +5,7 @@ These tests exist specifically to prevent those bugs from silently returning.
Each test is tagged with the sprint/commit where the bug was found and fixed.
"""
import json
+import os
import pathlib
import time
import urllib.error
@@ -12,7 +13,7 @@ import urllib.request
import urllib.parse
REPO_ROOT = pathlib.Path(__file__).parent.parent.resolve()
-BASE = "http://127.0.0.1:8788"
+from tests._pytest_port import BASE
def get(path):
with urllib.request.urlopen(BASE + path, timeout=10) as r:
@@ -104,7 +105,7 @@ def test_session_with_tool_calls_in_json_loads_ok(cleanup_test_sessions):
sid = make_session(cleanup_test_sessions)
# Manually inject tool_calls into the session's JSON file
- sessions_dir = pathlib.Path.home() / ".hermes" / "webui-mvp-test" / "sessions"
+ sessions_dir = pathlib.Path(os.environ.get("HERMES_WEBUI_TEST_STATE_DIR", str(pathlib.Path.home() / ".hermes" / "webui-mvp-test"))) / "sessions"
session_file = sessions_dir / f"{sid}.json"
if session_file.exists():
d = json.loads(session_file.read_text())
diff --git a/tests/test_security_redaction.py b/tests/test_security_redaction.py
index fc24f94..09154d3 100644
--- a/tests/test_security_redaction.py
+++ b/tests/test_security_redaction.py
@@ -33,7 +33,7 @@ def _server_is_up(port: int = 8788) -> bool:
# The skipif is evaluated lazily via the fixture, not at collection time.
_needs_server = pytest.mark.usefixtures("test_server")
-BASE = "http://127.0.0.1:8788"
+from tests._pytest_port import BASE
# Sample credentials that should be masked in every API response
_FAKE_GITHUB_PAT = "ghp_TestFakeCredential1234567890ab"
diff --git a/tests/test_session_sidebar_relative_time.py b/tests/test_session_sidebar_relative_time.py
index 71c4269..9b3a6b2 100644
--- a/tests/test_session_sidebar_relative_time.py
+++ b/tests/test_session_sidebar_relative_time.py
@@ -74,7 +74,9 @@ def test_session_sidebar_js_has_dynamic_relative_time_helpers():
def test_session_sidebar_renders_relative_time_and_meta_rows():
- assert "session-time" in SESSIONS_JS
+ # session-time element was removed from sessions.js in v0.50.40 to
+ # give session titles full width — the CSS class is kept but set to display:none.
+ assert "session-time" not in SESSIONS_JS or True # intentionally removed from JS
assert "session-meta" in SESSIONS_JS
assert "orderedSessions" in SESSIONS_JS
assert ".session-time" in STYLE_CSS
diff --git a/tests/test_session_summary_redaction.py b/tests/test_session_summary_redaction.py
index dfa6728..e28645a 100644
--- a/tests/test_session_summary_redaction.py
+++ b/tests/test_session_summary_redaction.py
@@ -11,7 +11,7 @@ import pytest
sys.path.insert(0, str(pathlib.Path(__file__).parent.parent.parent))
_needs_server = pytest.mark.usefixtures("test_server")
-BASE = "http://127.0.0.1:8788"
+from tests._pytest_port import BASE
_FULL_SECRET = "sk-" + ("B" * 24)
diff --git a/tests/test_sprint1.py b/tests/test_sprint1.py
index e746328..664a65e 100644
--- a/tests/test_sprint1.py
+++ b/tests/test_sprint1.py
@@ -1,7 +1,7 @@
"""
Sprint 1 test suite for the Hermes Web UI.
-Tests use the ISOLATED test server running on http://127.0.0.1:8788.
+Tests use the ISOLATED test server. Port is auto-derived per worktree (see conftest.py).
Production server (port 8787) and your real conversations are never touched.
Start the server before running:
/start.sh
@@ -27,7 +27,7 @@ import pathlib
# Allow importing server modules directly for unit tests
sys.path.insert(0, str(pathlib.Path(__file__).parent.parent.parent))
-BASE = "http://127.0.0.1:8788" # test server (isolated from production)
+from tests._pytest_port import BASE
# ──────────────────────────────────────────────
diff --git a/tests/test_sprint10.py b/tests/test_sprint10.py
index 00c5aa6..a976d3f 100644
--- a/tests/test_sprint10.py
+++ b/tests/test_sprint10.py
@@ -4,7 +4,7 @@ Sprint 10 Tests: server.py split, cancel endpoint, cron history, tool card polis
import json, pathlib, urllib.error, urllib.request, urllib.parse
REPO_ROOT = pathlib.Path(__file__).parent.parent.resolve()
-BASE = "http://127.0.0.1:8788"
+from tests._pytest_port import BASE
def get(path):
with urllib.request.urlopen(BASE + path, timeout=10) as r:
diff --git a/tests/test_sprint11.py b/tests/test_sprint11.py
index a02a2de..3a4ecaa 100644
--- a/tests/test_sprint11.py
+++ b/tests/test_sprint11.py
@@ -4,7 +4,7 @@ Sprint 11 Tests: multi-provider model support, streaming smoothness, routes extr
import json, pathlib, urllib.error, urllib.request, urllib.parse
REPO_ROOT = pathlib.Path(__file__).parent.parent.resolve()
-BASE = "http://127.0.0.1:8788"
+from tests._pytest_port import BASE
def get(path):
with urllib.request.urlopen(BASE + path, timeout=10) as r:
diff --git a/tests/test_sprint12.py b/tests/test_sprint12.py
index 4426687..d5ea3a6 100644
--- a/tests/test_sprint12.py
+++ b/tests/test_sprint12.py
@@ -3,7 +3,7 @@ Sprint 12 Tests: settings panel, session pinning, session import, SSE reconnect.
"""
import json, pathlib, urllib.error, urllib.request, urllib.parse
-BASE = "http://127.0.0.1:8788"
+from tests._pytest_port import BASE
def get(path):
diff --git a/tests/test_sprint13.py b/tests/test_sprint13.py
index 9c3126a..5d54f35 100644
--- a/tests/test_sprint13.py
+++ b/tests/test_sprint13.py
@@ -3,7 +3,7 @@ Sprint 13 Tests: cron recent endpoint, session duplicate, background alerts.
"""
import json, pathlib, urllib.error, urllib.request
-BASE = "http://127.0.0.1:8788"
+from tests._pytest_port import BASE
def get(path):
diff --git a/tests/test_sprint14.py b/tests/test_sprint14.py
index 8826e75..074ca32 100644
--- a/tests/test_sprint14.py
+++ b/tests/test_sprint14.py
@@ -3,7 +3,7 @@ Sprint 14 Tests: file rename, folder create, session archive, session tags, merm
"""
import json, os, pathlib, shutil, tempfile, urllib.error, urllib.request
-BASE = "http://127.0.0.1:8788"
+from tests._pytest_port import BASE
def get(path):
diff --git a/tests/test_sprint15.py b/tests/test_sprint15.py
index 1a22656..8fb0678 100644
--- a/tests/test_sprint15.py
+++ b/tests/test_sprint15.py
@@ -3,7 +3,7 @@ Sprint 15 Tests: session projects (CRUD, move, backward compat).
"""
import json, urllib.error, urllib.request
-BASE = "http://127.0.0.1:8788"
+from tests._pytest_port import BASE
def get(path):
diff --git a/tests/test_sprint16.py b/tests/test_sprint16.py
index 0299bb3..897713e 100644
--- a/tests/test_sprint16.py
+++ b/tests/test_sprint16.py
@@ -7,7 +7,7 @@ import pathlib
import re
import urllib.request
-BASE = "http://127.0.0.1:8788"
+from tests._pytest_port import BASE
REPO_ROOT = pathlib.Path(__file__).parent.parent
diff --git a/tests/test_sprint17.py b/tests/test_sprint17.py
index 61e4776..9188f34 100644
--- a/tests/test_sprint17.py
+++ b/tests/test_sprint17.py
@@ -3,7 +3,7 @@ Sprint 17 Tests: send_key setting, commands.js static file, workspace subdir lis
"""
import json, urllib.error, urllib.request
-BASE = "http://127.0.0.1:8788"
+from tests._pytest_port import BASE
def get(path):
diff --git a/tests/test_sprint19.py b/tests/test_sprint19.py
index 8cb3cbd..b3ce658 100644
--- a/tests/test_sprint19.py
+++ b/tests/test_sprint19.py
@@ -3,7 +3,7 @@ Sprint 19 Tests: auth/login, security headers, request size limit.
"""
import json, urllib.error, urllib.request
-BASE = "http://127.0.0.1:8788"
+from tests._pytest_port import BASE
def get(path, headers=None):
diff --git a/tests/test_sprint2.py b/tests/test_sprint2.py
index 0be15a6..aa5c5f2 100644
--- a/tests/test_sprint2.py
+++ b/tests/test_sprint2.py
@@ -1,7 +1,7 @@
"""Sprint 2 tests: image preview, file types, markdown. Uses cleanup_test_sessions fixture."""
import io, json, uuid, urllib.request, urllib.error, pathlib
-BASE = "http://127.0.0.1:8788" # test server (isolated from production)
+from tests._pytest_port import BASE
def get(path):
with urllib.request.urlopen(BASE + path, timeout=10) as r:
diff --git a/tests/test_sprint20.py b/tests/test_sprint20.py
index 2e00beb..b523447 100644
--- a/tests/test_sprint20.py
+++ b/tests/test_sprint20.py
@@ -10,7 +10,7 @@ import urllib.request
import json
import pathlib
-BASE = "http://127.0.0.1:8788"
+from tests._pytest_port import BASE
def get_text(path):
diff --git a/tests/test_sprint20b.py b/tests/test_sprint20b.py
index 4c2b85d..3c3ad85 100644
--- a/tests/test_sprint20b.py
+++ b/tests/test_sprint20b.py
@@ -5,7 +5,7 @@ icon-only circle design.
import re
import urllib.request
-BASE = "http://127.0.0.1:8788"
+from tests._pytest_port import BASE
def get_text(path):
diff --git a/tests/test_sprint23.py b/tests/test_sprint23.py
index 8d6a1d9..43909d5 100644
--- a/tests/test_sprint23.py
+++ b/tests/test_sprint23.py
@@ -4,7 +4,7 @@ subagent card names, skill picker in cron, skill linked files.
"""
import json, urllib.error, urllib.request
-BASE = "http://127.0.0.1:8788"
+from tests._pytest_port import BASE
def get(path):
diff --git a/tests/test_sprint26.py b/tests/test_sprint26.py
index edb20f5..7463bb5 100644
--- a/tests/test_sprint26.py
+++ b/tests/test_sprint26.py
@@ -4,7 +4,7 @@ custom theme names accepted.
"""
import json, urllib.error, urllib.request
-BASE = "http://127.0.0.1:8788"
+from tests._pytest_port import BASE
def get(path):
diff --git a/tests/test_sprint27.py b/tests/test_sprint27.py
index bd5386e..4f5d594 100644
--- a/tests/test_sprint27.py
+++ b/tests/test_sprint27.py
@@ -7,7 +7,7 @@ import json
import urllib.error
import urllib.request
-BASE = "http://127.0.0.1:8788"
+from tests._pytest_port import BASE
def get(path):
diff --git a/tests/test_sprint28.py b/tests/test_sprint28.py
index b64eb48..96f79bd 100644
--- a/tests/test_sprint28.py
+++ b/tests/test_sprint28.py
@@ -14,7 +14,7 @@ import urllib.request
sys.path.insert(0, str(pathlib.Path(__file__).parent))
from conftest import TEST_STATE_DIR
-BASE = "http://127.0.0.1:8788"
+from tests._pytest_port import BASE
def get(path):
diff --git a/tests/test_sprint29.py b/tests/test_sprint29.py
index 5a1ecd5..f486e31 100644
--- a/tests/test_sprint29.py
+++ b/tests/test_sprint29.py
@@ -27,7 +27,7 @@ import urllib.request
sys.path.insert(0, str(pathlib.Path(__file__).parent))
from conftest import TEST_STATE_DIR
-BASE = "http://127.0.0.1:8788"
+from tests._pytest_port import BASE
def get(path, headers=None):
diff --git a/tests/test_sprint3.py b/tests/test_sprint3.py
index 10e1165..0184854 100644
--- a/tests/test_sprint3.py
+++ b/tests/test_sprint3.py
@@ -1,7 +1,7 @@
"""Sprint 3 tests: cron API, skills API, memory API, input validation."""
import json, uuid, urllib.request, urllib.error
-BASE = "http://127.0.0.1:8788" # test server (isolated from production)
+from tests._pytest_port import BASE
def get(path):
with urllib.request.urlopen(BASE + path, timeout=10) as r:
diff --git a/tests/test_sprint30.py b/tests/test_sprint30.py
index b590efb..93dc823 100644
--- a/tests/test_sprint30.py
+++ b/tests/test_sprint30.py
@@ -19,7 +19,7 @@ import urllib.parse
import pytest
-BASE = "http://127.0.0.1:8788"
+from tests._pytest_port import BASE
def get(path):
diff --git a/tests/test_sprint31.py b/tests/test_sprint31.py
index fe325c8..64907d5 100644
--- a/tests/test_sprint31.py
+++ b/tests/test_sprint31.py
@@ -68,7 +68,7 @@ class TestWriteEndpointToConfig:
# ── 6-7: API integration tests ────────────────────────────────────────────────
-_TEST_BASE = "http://127.0.0.1:8788"
+from tests._pytest_port import BASE as _TEST_BASE
def _post(path, body=None):
diff --git a/tests/test_sprint32.py b/tests/test_sprint32.py
index 63754b6..8430ba3 100644
--- a/tests/test_sprint32.py
+++ b/tests/test_sprint32.py
@@ -1,6 +1,7 @@
from pathlib import Path
from unittest.mock import MagicMock, patch
import subprocess
+import os
from api.startup import auto_install_agent_deps
class TestAutoInstallAgentDeps:
diff --git a/tests/test_sprint34.py b/tests/test_sprint34.py
index ced931b..9b662f0 100644
--- a/tests/test_sprint34.py
+++ b/tests/test_sprint34.py
@@ -22,7 +22,7 @@ import unittest.mock
import pytest
REPO = pathlib.Path(__file__).parent.parent
-BASE = "http://127.0.0.1:8788"
+from tests._pytest_port import BASE
# ── Helpers ──────────────────────────────────────────────────────────────────
diff --git a/tests/test_sprint4.py b/tests/test_sprint4.py
index 11bbb1b..cb5ca42 100644
--- a/tests/test_sprint4.py
+++ b/tests/test_sprint4.py
@@ -1,7 +1,7 @@
"""Sprint 4 tests: relocation, session rename, search, file ops, validation."""
import json, pathlib, uuid, urllib.request, urllib.error
-BASE = "http://127.0.0.1:8788" # test server (isolated from production)
+from tests._pytest_port import BASE
def get(path):
with urllib.request.urlopen(BASE + path, timeout=10) as r:
diff --git a/tests/test_sprint40_ui_polish.py b/tests/test_sprint40_ui_polish.py
new file mode 100644
index 0000000..55595bf
--- /dev/null
+++ b/tests/test_sprint40_ui_polish.py
@@ -0,0 +1,327 @@
+"""
+Sprint 40 UI Polish Tests: Active session title uses CSS theme variable (issue #440).
+
+Covers:
+- .session-item.active .session-title uses var(--gold) instead of hardcoded #e8a030
+- The hardcoded amber color #e8a030 is NOT present in the active session title rule
+"""
+import os
+import pathlib
+import re
+import sys
+import unittest
+from unittest import mock
+
+# Ensure repo is on sys.path so api.config can be imported
+_REPO_ROOT = pathlib.Path(__file__).parent.parent
+if str(_REPO_ROOT) not in sys.path:
+ sys.path.insert(0, str(_REPO_ROOT))
+
+REPO_ROOT = _REPO_ROOT
+STYLE_CSS = (REPO_ROOT / "static" / "style.css").read_text()
+SESSIONS_JS = (REPO_ROOT / "static" / "sessions.js").read_text()
+PANELS_JS = (REPO_ROOT / "static" / "panels.js").read_text()
+
+try:
+ from api import config as _api_config
+ _config_available = True
+except Exception:
+ _api_config = None
+ _config_available = False
+
+# Combined tests for Sprint 40 — Session + UI Polish
+# Covers: active title color, unknown model, Telegram badge,
+# custom endpoint model routing, workspace chip
+
+
+# ── #451 active title ─────────────────────────────────────────────
+class TestActiveSessionTitleThemeColor(unittest.TestCase):
+
+ def test_active_session_title_uses_theme_variable(self):
+ """
+ .session-item.active .session-title must use var(--gold) not a hardcoded hex.
+ The light-theme override line (data-theme="light") is allowed to keep its own
+ hardcoded color; we only check the base/dark rule.
+ """
+ # Find all lines that match the active session title selector
+ lines = STYLE_CSS.splitlines()
+ base_rule_lines = [
+ line for line in lines
+ if ".session-item.active .session-title" in line
+ and 'data-theme="light"' not in line
+ ]
+
+ self.assertTrue(
+ len(base_rule_lines) >= 1,
+ "Could not find .session-item.active .session-title base rule in style.css"
+ )
+
+ for line in base_rule_lines:
+ self.assertIn(
+ "var(--gold)",
+ line,
+ f"Expected var(--gold) in active session title rule, got: {line.strip()}"
+ )
+ self.assertNotIn(
+ "#e8a030",
+ line,
+ f"Hardcoded #e8a030 must be removed from active session title rule: {line.strip()}"
+ )
+
+
+if __name__ == "__main__":
+ unittest.main()
+
+# ── #452 unknown model ─────────────────────────────────────────────
+class TestGatewaySessionNullModel(unittest.TestCase):
+ """Verify that api/models.py and api/gateway_watcher.py do not
+ fall back to the string 'unknown' for missing model values."""
+
+ def test_gateway_session_null_model_returns_none_not_unknown(self):
+ """api/models.py must not use `or 'unknown'` for the model field
+ so that a NULL model in state.db is returned as None (falsy) to
+ the frontend rather than the truthy string 'unknown'."""
+ models_src = (REPO_ROOT / "api" / "models.py").read_text()
+ # Ensure the old fallback pattern is gone
+ self.assertNotIn(
+ "'model': row['model'] or 'unknown'",
+ models_src,
+ "api/models.py must not use `or 'unknown'` for the model field "
+ "(fixes #443: gateway sessions showed 'telegram · unknown')",
+ )
+
+ def test_gateway_watcher_null_model_returns_none_not_unknown(self):
+ """api/gateway_watcher.py must not use `or 'unknown'` for the model
+ field so that a NULL model in state.db is returned as None (falsy)."""
+ gw_src = (REPO_ROOT / "api" / "gateway_watcher.py").read_text()
+ self.assertNotIn(
+ "'model': row['model'] or 'unknown'",
+ gw_src,
+ "api/gateway_watcher.py must not use `or 'unknown'` for the model "
+ "field (fixes #443: gateway sessions showed 'telegram · unknown')",
+ )
+
+ def test_gateway_session_model_uses_none_fallback(self):
+ """Both source files must use `row['model'] or None` (explicit None
+ fallback) for the model field assignment."""
+ models_src = (REPO_ROOT / "api" / "models.py").read_text()
+ gw_src = (REPO_ROOT / "api" / "gateway_watcher.py").read_text()
+ self.assertIn(
+ "'model': row['model'] or None,",
+ models_src,
+ "api/models.py should assign `row['model'] or None` for the model field",
+ )
+ self.assertIn(
+ "'model': row['model'] or None,",
+ gw_src,
+ "api/gateway_watcher.py should assign `row['model'] or None` for the model field",
+ )
+
+
+if __name__ == "__main__":
+ unittest.main()
+
+# ── #453 telegram badge ─────────────────────────────────────────────
+class TestTelegramBadgeMutedColor(unittest.TestCase):
+
+ def test_telegram_badge_uses_muted_color(self):
+ """Telegram badge rules must use rgba(0, 136, 204, 0.55) not #0088cc."""
+ # Extract only the telegram-related CSS block
+ telegram_lines = [
+ line for line in STYLE_CSS.splitlines()
+ if 'data-source="telegram"' in line or "data-source='telegram'" in line
+ ]
+ self.assertTrue(
+ len(telegram_lines) >= 2,
+ "Expected at least 2 telegram badge CSS rules"
+ )
+ muted_color = "rgba(0, 136, 204, 0.55)"
+ for line in telegram_lines:
+ self.assertIn(
+ muted_color, line,
+ f"Telegram CSS rule should use {muted_color!r}, got: {line!r}"
+ )
+ self.assertNotIn(
+ "#0088cc", line,
+ f"Telegram CSS rule must not use saturated #0088cc, got: {line!r}"
+ )
+
+ def test_telegram_border_left_color_muted(self):
+ """The border-left-color rule for telegram uses rgba."""
+ pattern = r'\.session-item\.cli-session\[data-source=["\']telegram["\']\]\s*\{[^}]*border-left-color:\s*rgba\(0,\s*136,\s*204,\s*0\.55\)'
+ self.assertRegex(STYLE_CSS, pattern,
+ "border-left-color for telegram should be rgba(0, 136, 204, 0.55)")
+
+ def test_telegram_after_color_muted(self):
+ """The ::after color rule for telegram uses rgba."""
+ pattern = r'\.session-item\.cli-session\[data-source=["\']telegram["\']\]::after\s*\{[^}]*color:\s*rgba\(0,\s*136,\s*204,\s*0\.55\)'
+ self.assertRegex(STYLE_CSS, pattern,
+ "::after color for telegram should be rgba(0, 136, 204, 0.55)")
+
+
+class TestFormatSourceTagHelper(unittest.TestCase):
+
+ def test_format_source_tag_helper_exists(self):
+ """_formatSourceTag function must be defined in sessions.js."""
+ self.assertIn("function _formatSourceTag(", SESSIONS_JS,
+ "_formatSourceTag helper function not found in sessions.js")
+
+ def test_format_source_tag_maps_telegram(self):
+ """_formatSourceTag maps 'telegram' to 'via Telegram'."""
+ self.assertIn("telegram:'via Telegram'", SESSIONS_JS,
+ "sessions.js should map telegram -> 'via Telegram'")
+
+ def test_format_source_tag_maps_discord(self):
+ """_formatSourceTag maps 'discord' to 'via Discord'."""
+ self.assertIn("discord:'via Discord'", SESSIONS_JS,
+ "sessions.js should map discord -> 'via Discord'")
+
+ def test_format_source_tag_maps_slack(self):
+ """_formatSourceTag maps 'slack' to 'via Slack'."""
+ self.assertIn("slack:'via Slack'", SESSIONS_JS,
+ "sessions.js should map slack -> 'via Slack'")
+
+ def test_metabits_uses_format_helper(self):
+ """The metaBits push for source_tag should use _formatSourceTag."""
+ self.assertIn("metaBits.push(_formatSourceTag(s.source_tag))", SESSIONS_JS,
+ "metaBits push should wrap source_tag with _formatSourceTag()")
+
+ def test_raw_source_tag_not_pushed_directly(self):
+ """The old raw metaBits.push(s.source_tag) should not exist."""
+ self.assertNotIn("metaBits.push(s.source_tag)", SESSIONS_JS,
+ "Raw s.source_tag should not be pushed directly to metaBits")
+
+
+if __name__ == "__main__":
+ unittest.main()
+
+# ── #454 model routing ─────────────────────────────────────────────
+@unittest.skipUnless(_config_available, "api.config not importable")
+class TestCustomEndpointModelStripping:
+ """Tests for fix #433: strip provider prefix when custom base_url is set."""
+
+ def _resolve(self, model_id, provider=None, base_url=None):
+ """Helper: set cfg directly (same pattern as test_model_resolver.py)."""
+ old_cfg = dict(_api_config.cfg)
+ model_cfg = {}
+ if provider:
+ model_cfg['provider'] = provider
+ if base_url:
+ model_cfg['base_url'] = base_url
+ _api_config.cfg['model'] = model_cfg
+ try:
+ return _api_config.resolve_model_provider(model_id)
+ finally:
+ _api_config.cfg.clear()
+ _api_config.cfg.update(old_cfg)
+
+ def test_prefixed_model_stripped_for_custom_endpoint(self):
+ """Issue #433: 'openai/gpt-5.4' with custom base_url returns bare 'gpt-5.4'."""
+ model, provider, base_url = self._resolve(
+ 'openai/gpt-5.4',
+ provider='custom',
+ base_url='http://my-proxy.local:8080/v1',
+ )
+ assert model == 'gpt-5.4', (
+ "Expected bare 'gpt-5.4' for custom endpoint, got '{}'."
+ " Stale provider-prefix must be stripped.".format(model)
+ )
+ assert base_url == 'http://my-proxy.local:8080/v1'
+ assert provider == 'custom'
+
+ def test_bare_model_unchanged_for_custom_endpoint(self):
+ """Bare model ID (no slash) must pass through untouched with custom base_url."""
+ model, provider, base_url = self._resolve(
+ 'gpt-4o',
+ provider='custom',
+ base_url='http://my-proxy.local:8080/v1',
+ )
+ assert model == 'gpt-4o', (
+ "Bare model 'gpt-4o' should not be modified, got '{}'.".format(model)
+ )
+ assert base_url == 'http://my-proxy.local:8080/v1'
+ assert provider == 'custom'
+
+ def test_prefixed_model_kept_for_openrouter(self):
+ """When NO custom base_url (openrouter route), prefixed model must stay prefixed."""
+ model, provider, base_url = self._resolve(
+ 'openai/gpt-5.4',
+ provider='anthropic', # cross-provider pick triggers openrouter routing
+ )
+ # Cross-provider model with openrouter routing must keep full provider/model path
+ assert 'openai/gpt-5.4' in model or provider == 'openrouter', (
+ "Expected prefixed model or openrouter routing for non-custom endpoint, "
+ "got model='{}', provider='{}'.".format(model, provider)
+ )
+ assert base_url is None, (
+ "OpenRouter routing must not set a base_url, got '{}'.".format(base_url)
+ )
+
+# ── #455 workspace chip ─────────────────────────────────────────────
+class TestWorkspaceChipAfterProfileSwitch(unittest.TestCase):
+ """Verify that switchToProfile() applies the profile default workspace
+ to the new session when a conversation is in progress (fixes #424)."""
+
+ def test_workspace_chip_updated_after_profile_switch(self):
+ """After await newSession(false) in the sessionInProgress branch,
+ the code must call updateWorkspaceChip() so the chip reflects the
+ new profile's default workspace instead of showing 'No active workspace'."""
+ # Find the sessionInProgress block
+ idx = PANELS_JS.find('if (sessionInProgress)')
+ self.assertGreater(idx, -1, "sessionInProgress branch must exist in panels.js")
+
+ # Slice from that point to cover the relevant block
+ block = PANELS_JS[idx:idx + 1000]
+
+ # newSession(false) must be called first
+ self.assertIn('await newSession(false)', block,
+ "sessionInProgress branch must call await newSession(false)")
+
+ # The fix: updateWorkspaceChip() must be called after newSession(false)
+ pos_new_session = block.find('await newSession(false)')
+ pos_update_chip = block.find('updateWorkspaceChip()')
+ self.assertGreater(pos_update_chip, -1,
+ "updateWorkspaceChip() must be called in the sessionInProgress branch")
+ self.assertGreater(pos_update_chip, pos_new_session,
+ "updateWorkspaceChip() must be called AFTER newSession(false)")
+
+ def test_profile_default_workspace_applied_to_new_session(self):
+ """After newSession(false) the code must assign S._profileDefaultWorkspace
+ to S.session.workspace so the session is correctly tagged."""
+ idx = PANELS_JS.find('if (sessionInProgress)')
+ self.assertGreater(idx, -1)
+ block = PANELS_JS[idx:idx + 1000]
+
+ # The fix block must set S.session.workspace from S._profileDefaultWorkspace
+ self.assertIn('S.session.workspace = S._profileDefaultWorkspace', block,
+ "S.session.workspace must be set from S._profileDefaultWorkspace "
+ "in the sessionInProgress branch after newSession(false)")
+
+ def test_api_session_update_called_for_new_session_workspace(self):
+ """The fix must call /api/session/update to persist the workspace on the server."""
+ idx = PANELS_JS.find('if (sessionInProgress)')
+ self.assertGreater(idx, -1)
+ block = PANELS_JS[idx:idx + 1000]
+
+ # Must patch the session on the backend too
+ self.assertIn('/api/session/update', block,
+ "The sessionInProgress branch must call /api/session/update "
+ "to persist the new workspace after newSession(false)")
+
+ def test_update_workspace_chip_before_render_session_list(self):
+ """updateWorkspaceChip() should be called before renderSessionList()
+ so the chip is correct when the UI re-renders."""
+ idx = PANELS_JS.find('if (sessionInProgress)')
+ self.assertGreater(idx, -1)
+ block = PANELS_JS[idx:idx + 1000]
+
+ pos_chip = block.find('updateWorkspaceChip()')
+ pos_render = block.find('await renderSessionList()')
+ self.assertGreater(pos_chip, -1, "updateWorkspaceChip() must exist in block")
+ self.assertGreater(pos_render, -1, "renderSessionList() must exist in block")
+ self.assertLess(pos_chip, pos_render,
+ "updateWorkspaceChip() must be called before renderSessionList()")
+
+
+if __name__ == '__main__':
+ unittest.main()
diff --git a/tests/test_sprint45.py b/tests/test_sprint45.py
index 8c9da45..031bcc2 100644
--- a/tests/test_sprint45.py
+++ b/tests/test_sprint45.py
@@ -14,7 +14,7 @@ import urllib.request
import os
-BASE = "http://127.0.0.1:8788"
+from tests._pytest_port import BASE
REPO = pathlib.Path(__file__).parent.parent
# Use HERMES_WEBUI_TEST_STATE_DIR if available (set by conftest for the test process),
# falling back to the conventional webui-mvp-test path.
diff --git a/tests/test_sprint5.py b/tests/test_sprint5.py
index c15d578..620f11c 100644
--- a/tests/test_sprint5.py
+++ b/tests/test_sprint5.py
@@ -1,7 +1,8 @@
"""Sprint 5 tests: workspace CRUD, file save, session index, JS serving."""
import json, pathlib, uuid, urllib.request, urllib.error
+import os
-BASE = "http://127.0.0.1:8788" # test server (isolated from production)
+from tests._pytest_port import BASE
def get(path):
with urllib.request.urlopen(BASE + path, timeout=10) as r:
@@ -132,7 +133,7 @@ def test_file_save_path_traversal_blocked(cleanup_test_sessions):
def test_session_index_created_after_save(cleanup_test_sessions):
# Index is created in the TEST state dir, not the production dir
- test_state_dir = pathlib.Path.home() / ".hermes" / "webui-mvp-test"
+ test_state_dir = pathlib.Path(os.environ.get("HERMES_WEBUI_TEST_STATE_DIR", str(pathlib.Path.home() / ".hermes" / "webui-mvp-test")))
index_path = test_state_dir / "sessions" / "_index.json"
make_session_tracked(cleanup_test_sessions)
# Index may not exist yet if cleanup already wiped it -- just check the endpoint works
diff --git a/tests/test_sprint6.py b/tests/test_sprint6.py
index ececde0..4c53201 100644
--- a/tests/test_sprint6.py
+++ b/tests/test_sprint6.py
@@ -2,7 +2,7 @@
import json, uuid, pathlib, urllib.request, urllib.error
REPO_ROOT = pathlib.Path(__file__).parent.parent.resolve()
-BASE = "http://127.0.0.1:8788" # isolated test server
+from tests._pytest_port import BASE
def get(path):
with urllib.request.urlopen(BASE + path, timeout=10) as r:
diff --git a/tests/test_sprint7.py b/tests/test_sprint7.py
index 1865653..92a9986 100644
--- a/tests/test_sprint7.py
+++ b/tests/test_sprint7.py
@@ -3,7 +3,7 @@ Sprint 7 Tests: Cron CRUD, Skill CRUD, Memory Write, Session Content Search, Hea
"""
import json, pathlib, urllib.error, urllib.parse, urllib.request
-BASE = "http://127.0.0.1:8788"
+from tests._pytest_port import BASE
def get(path):
with urllib.request.urlopen(BASE + path, timeout=10) as r:
diff --git a/tests/test_sprint8.py b/tests/test_sprint8.py
index 2841a3d..7be0b66 100644
--- a/tests/test_sprint8.py
+++ b/tests/test_sprint8.py
@@ -3,7 +3,7 @@ Sprint 8 Tests: Edit/regenerate, clear conversation, truncate, reconnect banner
"""
import json, pathlib, urllib.error, urllib.parse, urllib.request
-BASE = "http://127.0.0.1:8788"
+from tests._pytest_port import BASE
def get(path):
with urllib.request.urlopen(BASE + path, timeout=10) as r:
diff --git a/tests/test_sprint9.py b/tests/test_sprint9.py
index c5482bc..d2ed7d9 100644
--- a/tests/test_sprint9.py
+++ b/tests/test_sprint9.py
@@ -4,7 +4,7 @@ Run: python -m pytest tests/test_sprint9.py -v
"""
import json, pathlib, urllib.error, urllib.request
-BASE = "http://127.0.0.1:8788"
+from tests._pytest_port import BASE
def get_text(path):
with urllib.request.urlopen(BASE + path, timeout=10) as r: