wehub-resource-sync
555e282cc4
pi-agent-plugin checks / lint (push) Has been cancelled
pi-agent-plugin checks / test (20) (push) Has been cancelled
pi-agent-plugin checks / test (22) (push) Has been cancelled
pi-agent-plugin checks / build (push) Has been cancelled
TypeScript SDK CI / check_changes (push) Has been cancelled
TypeScript SDK CI / changelog_check (push) Has been cancelled
ci / changelog_check (push) Has been cancelled
ci / check_changes (push) Has been cancelled
ci / build_mem0 (3.10) (push) Has been cancelled
ci / build_mem0 (3.11) (push) Has been cancelled
ci / build_mem0 (3.12) (push) Has been cancelled
CLI Node CI / lint (push) Has been cancelled
CLI Node CI / test (20) (push) Has been cancelled
CLI Node CI / test (22) (push) Has been cancelled
CLI Node CI / build (push) Has been cancelled
CLI Python CI / lint (push) Has been cancelled
CLI Python CI / test (3.10) (push) Has been cancelled
CLI Python CI / test (3.11) (push) Has been cancelled
CLI Python CI / test (3.12) (push) Has been cancelled
CLI Python CI / build (push) Has been cancelled
openclaw checks / lint (push) Has been cancelled
openclaw checks / test (20) (push) Has been cancelled
openclaw checks / test (22) (push) Has been cancelled
openclaw checks / build (push) Has been cancelled
opencode-plugin checks / build (push) Has been cancelled
TypeScript SDK CI / build_ts_sdk (20) (push) Has been cancelled
TypeScript SDK CI / build_ts_sdk (22) (push) Has been cancelled
TypeScript SDK CI / integration_ts_sdk (20) (push) Has been cancelled
TypeScript SDK CI / integration_ts_sdk (22) (push) Has been cancelled
80 lines
2.3 KiB
Python
80 lines
2.3 KiB
Python
"""Regression tests for capture_session_summary.py request body construction.
|
|
|
|
Guards against the double-JSON-encoding bug where ``files_touched`` was stored
|
|
as a pre-serialized JSON string and then encoded a second time with the rest of
|
|
the request body — surfacing as escaped, slash-heavy blobs in the memories shown
|
|
inside Claude Code / Cursor / Codex / Antigravity (all four editors share this
|
|
script).
|
|
"""
|
|
|
|
from __future__ import annotations
|
|
|
|
import json
|
|
|
|
|
|
class _FakeResp:
|
|
status = 200
|
|
|
|
def __enter__(self):
|
|
return self
|
|
|
|
def __exit__(self, *_):
|
|
return False
|
|
|
|
|
|
def _capture_request_body(monkeypatch):
|
|
"""Patch urlopen so store_summary posts nowhere; capture the request body."""
|
|
import capture_session_summary as css
|
|
|
|
captured: dict = {}
|
|
|
|
def fake_urlopen(req, timeout=0):
|
|
captured["raw"] = req.data.decode("utf-8")
|
|
captured["body"] = json.loads(captured["raw"])
|
|
return _FakeResp()
|
|
|
|
monkeypatch.setattr(css.urllib.request, "urlopen", fake_urlopen)
|
|
return captured, css
|
|
|
|
|
|
def test_files_touched_is_json_array_not_double_encoded(monkeypatch):
|
|
"""files_touched must be a real JSON array, encoded exactly once."""
|
|
captured, css = _capture_request_body(monkeypatch)
|
|
files = ["mem0/memory/main.py", "src/client/index.ts"]
|
|
|
|
css.store_summary(
|
|
api_key="test-key",
|
|
summary_prompt="did some work",
|
|
user_id="u1",
|
|
session_id="s1",
|
|
project_id="p1",
|
|
branch="main",
|
|
files=files,
|
|
)
|
|
|
|
files_touched = captured["body"]["metadata"]["files_touched"]
|
|
assert isinstance(files_touched, list), (
|
|
"files_touched must be a JSON array, not a double-encoded string; "
|
|
f"got {type(files_touched).__name__}: {files_touched!r}"
|
|
)
|
|
assert files_touched == files
|
|
# The file paths must not appear as an escaped JSON string inside the body.
|
|
assert '\\"' not in captured["raw"]
|
|
|
|
|
|
def test_files_touched_omitted_when_no_files(monkeypatch):
|
|
"""No files touched -> no files_touched key (unchanged behaviour)."""
|
|
captured, css = _capture_request_body(monkeypatch)
|
|
|
|
css.store_summary(
|
|
api_key="test-key",
|
|
summary_prompt="did some work",
|
|
user_id="u1",
|
|
session_id="s1",
|
|
project_id="p1",
|
|
branch="main",
|
|
files=[],
|
|
)
|
|
|
|
assert "files_touched" not in captured["body"]["metadata"]
|