Skip to content

fix: compare session suspend cutoff as datetime - #8129

Closed
ccqqlo wants to merge 4 commits into
NousResearch:mainfrom
ccqqlo:fix/session-suspend-datetime-cutoff
Closed

fix: compare session suspend cutoff as datetime#8129
ccqqlo wants to merge 4 commits into
NousResearch:mainfrom
ccqqlo:fix/session-suspend-datetime-cutoff

Conversation

@ccqqlo

@ccqqlo ccqqlo commented Apr 12, 2026

Copy link
Copy Markdown

Summary

  • fix gateway session startup suspension to compare datetime values instead of mixing datetime with float timestamps
  • skip entries with missing updated_at values
  • add a regression test covering recent vs old session suspension behavior

Testing

  • source venv/bin/activate && pytest -q tests/gateway/test_session_suspend_recently_active.py
  • source venv/bin/activate && pytest -q tests/tools/test_send_message_tool.py tests/gateway/test_session_suspend_recently_active.py

@alt-glitch alt-glitch added type/bug Something isn't working P1 High — major feature broken, no workaround comp/gateway Gateway runner, session dispatch, delivery labels Apr 28, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

Multiple prior PRs for the same datetime-vs-float bug in suspend_recently_active: #7995, #10397, #7973, #8336 (all closed). This appears to be the surviving fix attempt.

@ccqqlo ccqqlo closed this by deleting the head repository Jun 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/gateway Gateway runner, session dispatch, delivery P1 High — major feature broken, no workaround type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants