Skip to content

fix(app): reduce session cache memory - #46120

Closed
Nittarab wants to merge 1 commit into
anomalyco:devfrom
Nittarab:bound-session-cache
Closed

Nittarab wants to merge 1 commit into
anomalyco:devfrom
Nittarab:bound-session-cache

Conversation

@Nittarab

Copy link
Copy Markdown

Issue for this PR

Closes #46119

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

Reduces the app session cache from 40 transcripts to 10, matching the nearby-session prefetch working set. In a 50-session CDP run, retained JS heap after GC fell from 134.4 MB to 72.5 MB.

How did you verify your code works?

  • App cache unit test: 3 passed
  • App typecheck passed
  • Repository pre-push typecheck: 30 packages passed
  • Repeated the 50-session CDP memory run against the changed app

Screenshots / recordings

Not applicable. This change has no visual effect.

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

If you do not follow this template your PR will be automatically rejected.

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.

Web UI retains 40 full session transcripts

1 participant