Skip to content

[Refactor] Remove unused Session presence tracking - #2072

Open
roomote-roomote[bot] wants to merge 1 commit into
developfrom
refactor/remove-unused-session-presence-0okwthwca03ov
Open

[Refactor] Remove unused Session presence tracking#2072
roomote-roomote[bot] wants to merge 1 commit into
developfrom
refactor/remove-unused-session-presence-0okwthwca03ov

Conversation

@roomote-roomote

Copy link
Copy Markdown
Contributor

​Created by Roomote. View the task or mention @roomote-roomote for follow-up asks.

What changed

  • Removed the Session viewer heartbeat hook and its focus, visibility, page lifecycle, and timer handling.
  • Removed the authenticated presence API route and the Redis lease implementation and exports.
  • Removed the dedicated tests for the deleted lifecycle while preserving the shared Redis client structure used elsewhere.

Why this change was made

Session presence was continuously written from open browser tabs, but no production code consumed the presence query. Keeping the distributed browser, API, and Redis lifecycle imposed ongoing network and maintenance cost without affecting product behavior.

Impact

Opening a Session no longer starts background presence requests or creates Redis leases. There is no intended user-facing behavior change because notification routing and other production paths never read the presence signal.

@roomote-community

roomote-community Bot commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

No code issues found. See task

Reviewed 93ed1da

@roomote-roomote
roomote-roomote Bot marked this pull request as ready for review September 2, 2026 17:28
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