Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Memory leak: TUI and Slack bot event listeners not cleaned up #9155

Copy link
Copy link
@sauerdaniel

Description

@sauerdaniel
Issue body actions

Problem

Several event listeners in the TUI and Slack bot are never cleaned up:

TUI Sync Context (sync.tsx)

The sdk.event.listen() call returns an unsubscribe function that is never called:

// Listener never cleaned up
sdk.event.listen((e) => { ... })

Tooltip Component (tooltip.tsx)

Mouse event listeners (mouseenter, mouseleave) are added to elements but never removed on component unmount, causing accumulation.

Slack Bot (slack/index.ts)

  • Sessions Map grows unbounded (no cleanup of old sessions)
  • No graceful shutdown handling (SIGINT/SIGTERM)
  • No limit on maximum concurrent sessions

Impact

  • Event listeners accumulate over component lifecycles
  • Sessions accumulate in Slack bot
  • Memory grows continuously in long-running processes

Relates to #5363

Expected Behavior

  • TUI event listeners should be unsubscribed in onCleanup
  • Tooltip should remove event listeners on unmount
  • Slack bot should clean up old sessions and handle shutdown
Reactions are currently unavailable

Metadata

Metadata

Assignees

Labels

perfIndicates a performance issue or need for optimizationIndicates a performance issue or need for optimization

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions

    Morty Proxy This is a proxified and sanitized view of the page, visit original site.