mirror of
https://github.com/pewdiepie-archdaemon/odysseus.git
synced 2026-07-08 11:56:59 +00:00
Tal.Yuan
6f6cb6ea88
Slice 2d of the route-domain reorganization (#4082/#4071, per specs/architecture-runtime-inventory.md §6.3). Moves history_routes.py into routes/history/, leaving a backward-compat sys.modules shim at the old path. Pure file reorganization, no behavior change. The shim uses sys.modules replacement (same pattern as the merged gallery #4903, research #4975, and memory #5007 slices) so that `import routes.history_routes`, `from routes.history_routes import X`, `importlib.import_module(...)`, and the `import ... as history_routes` + `monkeypatch.setattr(history_routes, ...)` pattern used by test_history_compact_tool_calls.py / test_fork_session_metadata.py all operate on the same module object the application uses. The canonical module does NOT depend on the shim — routes/history/ history_routes.py imports only from core/, src/, and routes.session_routes (a sibling route module whose old import path stays valid via its own shim when session is migrated later). Three source-introspection test sites repointed to the new canonical path: - test_history_db_fallback_hidden.py - test_history_order_by_timestamp_regression.py - test_model_helper_owner_scope.py Adds tests/test_history_routes_shim.py to pin the sys.modules shim contract (legacy and canonical paths resolve to the same module object; monkeypatch via legacy alias reaches the canonical module). Verified: compileall clean; full suite 4351 passed, 3 skipped.
39 lines
1.6 KiB
Python
39 lines
1.6 KiB
Python
"""Regression: the DB fallback in get_session_history must hide the same
|
|
messages the in-memory path hides.
|
|
|
|
The in-memory branch skips messages whose metadata has ``hidden`` (e.g.
|
|
compaction summaries that are kept for AI context but not shown to the user).
|
|
The DB fallback (taken when the in-memory history is empty, e.g. after a
|
|
restart) built the client response from every DB row with no such filter, so
|
|
hidden messages leaked to the client on DB-served sessions. The rebuilt
|
|
in-memory ``session.history`` must still keep them, though, so only the response
|
|
is filtered.
|
|
|
|
get_session_history depends on the DB, the session manager and a FastAPI
|
|
request, so this pins the regression at the source level (as other route tests
|
|
in this repo do).
|
|
"""
|
|
import ast
|
|
from pathlib import Path
|
|
|
|
SRC = Path(__file__).resolve().parent.parent / "routes" / "history" / "history_routes.py"
|
|
|
|
|
|
def _function_source(src_text, name):
|
|
tree = ast.parse(src_text)
|
|
for node in ast.walk(tree):
|
|
if isinstance(node, (ast.FunctionDef, ast.AsyncFunctionDef)) and node.name == name:
|
|
return ast.get_source_segment(src_text, node)
|
|
raise AssertionError(f"{name} not found in {SRC}")
|
|
|
|
|
|
def test_db_fallback_filters_hidden_from_response():
|
|
src = _function_source(SRC.read_text(), "get_session_history")
|
|
marker = "load from DB"
|
|
assert marker in src, "expected the DB fallback block in get_session_history"
|
|
db_section = src.split(marker, 1)[1]
|
|
assert "hidden" in db_section, (
|
|
"the DB-fallback path must filter `hidden` messages from the response "
|
|
"to match the in-memory path"
|
|
)
|