Add unregister token for stream finalization registry

simolus3 · simolus3 · commit 94c7c9f8e594 · 2026-05-05T12:36:19.000+02:00
diff --git a/.changeset/cuddly-bobcats-hang.md b/.changeset/cuddly-bobcats-hang.md
@@ -0,0 +1,5 @@
+---
+'@powersync/common': patch
+---
+
+Fix warnings about leaked subscriptions even though `unsubscribe()` was called.
diff --git a/packages/common/src/client/ConnectionManager.ts b/packages/common/src/client/ConnectionManager.ts
@@ -368,7 +368,7 @@ class SyncStreamSubscriptionHandle implements SyncStreamSubscription {
 
   constructor(readonly subscription: ActiveSubscription) {
     subscription.refcount++;
-    _finalizer?.register(this, subscription);
+    _finalizer?.register(this, subscription, this);
   }
 
   get name() {

-Original file line number
+Diff line change
@@ @@ -0,0 +1,5 @@ @@
 +---
 +'@powersync/common': patch
 +---
++
 +Fix warnings about leaked subscriptions even though `unsubscribe()` was called.
Original file line number	Diff line number	Diff line change
`@@ -368,7 +368,7 @@ class SyncStreamSubscriptionHandle implements SyncStreamSubscription {`
`368`	`368`
`369`	`369`	`constructor(readonly subscription: ActiveSubscription) {`
`370`	`370`	`subscription.refcount++;`
`371`		`- _finalizer?.register(this, subscription);`
	`371`	`+ _finalizer?.register(this, subscription, this);`
`372`	`372`	`}`
`373`	`373`
`374`	`374`	`get name() {`