fix: ensure derived signals properly capture consumers

[trueadm]

This is a follow up to #10209. Essentially there were a few bugs that came into play with unowned signals that made this a bit more complex to resolve.

• When dealing with unowned signals (they were created without an owner effect), we were skipping consumers incorrectly. We should always add consumers when we're part of effect flushing, so now we use a dedicated flag is_flushing_effect.
• The removal of consumers was broken and we were traversing through the current dependencies rather than the actual dependencies. Cleaned up the code to make it far clearer what is what.
• When removing consumers we had a flag that was completely redundant - remove_unowned. We should always be removing unowned now that we have fix old issues around incorrectly removing consumers that didn't need to be removed.
• We should only be marking effect signals inert, not derived signals.

[changeset-bot]

🦋 Changeset detected

Latest commit: ea63691

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
svelte	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
svelte-5-preview	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 17, 2024 9:12pm