diff --git a/CHANGELOG.md b/CHANGELOG.md index a27085b1..88b23bba 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -10,8 +10,13 @@ All notable changes to vouch are documented here. Format follows - **sandbox docker argv on Windows** (#582): omit `--user uid:gid` when `os.getuid` / `os.getgid` are unavailable so sandboxed dual-solve no longer raises `AttributeError` while building the docker command. -- **`kb.session_transcript` handler test needs a KB** : `test_handler_returns_degraded_when_absent` now chdirs into a temp KB and points Claude/Codex search roots at empty dirs, so the handler can return the degraded payload instead of raising `KBNotFoundError`. -) +- **`kb.session_transcript`'s degraded-path test no longer depends on an + ambient KB**: it called `handle_request` against whatever `.vouch/` the + cwd happened to sit under, so it passed on a developer checkout and + failed in CI — where `.vouch/` is gitignored — with an `internal_error` + from `_store()`. it now chdirs into a fixture KB and points both + transcript locators at empty dirs, so the absence it asserts is the + raw transcript's. - **`kb.search` excludes retracted claims and archived pages** (#581): `search_kb` now drops `ARCHIVED` / `SUPERSEDED` / `REDACTED` claims and `ARCHIVED` pages the same way `kb.context` already does, so lifecycle diff --git a/tests/test_session_transcript.py b/tests/test_session_transcript.py index 5af1a244..0dba7d1d 100644 --- a/tests/test_session_transcript.py +++ b/tests/test_session_transcript.py @@ -211,7 +211,7 @@ def test_handler_returns_degraded_when_absent( "id": "3", "method": "kb.session_transcript", "params": {"session_id": "11111111-1111-1111-1111-111111111111"}, }) - assert resp["ok"] is True + assert resp["ok"] is True, resp assert resp["result"]["available"] is False