Unsubscribing last subscriber twice from a writable store runs null()

[pushkine]

https://svelte.dev/repl/fc33f2d0f07c43c0a90b5de49b267396?version=3.23.0

svelte/src/runtime/store/index.ts

Lines 100 to 109 in a4dadf8

	return () => {
	const index = subscribers.indexOf(subscriber);
	if (index !== -1) {
	subscribers.splice(index, 1);
	}
	if (subscribers.length === 0) {
	stop();
	stop = null;
	}
	};

[Conduitry]

I don't know whether you intend this as a bug report, but I don't know that I'd call this a bug. I also don't know that I'd call this a particularly worthwhile feature request. People should probably be careful to only unsubscribe from a subscription once.

edit: Eh. If we're already otherwise allowing unsubscribers to be called multiple times (are we?) then also handling that here might make sense.

[dimfeld]

Someone in the discord yesterday was seeing a problem that I’m pretty sure was this issue, and he said he was only using autosubscription. I asked if he could put together a REPL but haven’t heard back yet (unless I missed it overnight).

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[gtm-nayan]

fixed by #8186