Skip to content

fix(ff-effect): memory leak in Inngest handler#66

Merged
fdarian merged 2 commits intomainfrom
memory-leak
Apr 2, 2026
Merged

fix(ff-effect): memory leak in Inngest handler#66
fdarian merged 2 commits intomainfrom
memory-leak

Conversation

@fdarian
Copy link
Copy Markdown
Owner

@fdarian fdarian commented Apr 2, 2026

Summary

  • Removes extract() function call that was keeping references alive
  • Replaces with direct Effect.scoped() to properly manage resource cleanup
  • Fixes memory leak in Inngest createFunction handler

Test plan

  • Verify Inngest functions work correctly with the updated handler
  • Monitor memory usage in production to confirm leak is resolved
  • Run existing tests to ensure no regressions

The extract function was keeping references alive, causing memory leak.
Now directly using Effect.scoped() to properly manage resource cleanup.
@fdarian fdarian changed the title Fix memory leak in Inngest handler fix(ff-effect): memory leak in Inngest handler Apr 2, 2026
@fdarian fdarian merged commit 43d04c5 into main Apr 2, 2026
3 checks passed
@fdarian fdarian deleted the memory-leak branch April 2, 2026 05:34
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