Skip to content

Allow empty session memory stdio route results#194

Merged
8Dionysus merged 1 commit into
mainfrom
codex/session-memory-stdio-empty-routes
Jun 16, 2026
Merged

Allow empty session memory stdio route results#194
8Dionysus merged 1 commit into
mainfrom
codex/session-memory-stdio-empty-routes

Conversation

@8Dionysus

Copy link
Copy Markdown
Owner

Summary

  • stop failing the session-memory MCP validator when stdio route tools return valid ok payloads with zero route results
  • keep the entity inventory non-empty assertion, but report route counts as health summary for sparse/in-progress archives
  • add a regression for zero result_count/window_count stdio route payloads

Validation

  • python -m pytest -q mcp/services/aoa-session-memory-mcp/tests/test_session_memory_mcp.py -k 'stdio_route_count_summary_allows_empty_route_results or agent_event_and_task_episode_routes_wrap_archive_cli'\n- python mcp/services/aoa-session-memory-mcp/scripts/validate_session_memory_mcp.py\n- python -m pytest -q mcp/services/aoa-session-memory-mcp/tests\n- python mcp/services/aoa-session-memory-mcp/scripts/release_check.py\n- python scripts/validate_stack.py\n- python scripts/validate_nested_agents.py\n- python scripts/ci_gate.py --mode source-fast\n- git diff --check\n\n## Notes\n- Decision review: no new decision record; this tightens validator semantics for an existing MCP smoke route.\n- Runtime exposure unchanged: stdio access-plane only, no maintenance/write authority added.\n- Merge method expected by repo settings: squash.

@8Dionysus 8Dionysus marked this pull request as ready for review June 16, 2026 04:45
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@8Dionysus 8Dionysus merged commit 2af5a7b into main Jun 16, 2026
4 checks passed
@8Dionysus 8Dionysus deleted the codex/session-memory-stdio-empty-routes branch June 16, 2026 04:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant