Skip to content

Commit 5696814

Browse files
committed
Simplify session response fields by removing redundant new_session in deprecated path
The deprecated behavior path was setting both new_session=True and unsaved=True, which was redundant. The unsaved field alone is sufficient to indicate that this is deprecated behavior returning an unpersisted session.
1 parent f17b5ce commit 5696814

File tree

2 files changed

+1
-3
lines changed

2 files changed

+1
-3
lines changed

agent_memory_server/api.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -386,7 +386,6 @@ async def get_working_memory(
386386
"GET /v1/working-memory/{session_id} will return 404 for missing sessions in version 1.0. "
387387
"Use get_or_create_working_memory client method instead."
388388
)
389-
new_session = True
390389
unsaved = True
391390
# Create empty working memory but DO NOT persist it
392391
working_mem = WorkingMemory(

tests/test_api.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -437,8 +437,7 @@ async def test_delete_memory(self, client, session):
437437
# Should return 200 with unsaved session (deprecated behavior for old clients)
438438
assert response.status_code == 200
439439
data = response.json()
440-
assert data["new_session"] is True # Session was created
441-
assert data["unsaved"] is True # But not persisted (deprecated behavior)
440+
assert data["unsaved"] is True # Not persisted (deprecated behavior)
442441
assert len(data["messages"]) == 0 # Empty session
443442
assert len(data["memories"]) == 0
444443

0 commit comments

Comments
 (0)