fix: when re-connecting unowned deriveds, remove their unowned flag

[trueadm]

Fixes #15229.
Fixes #15180.

Where we know that an unowned that was temporarily made unowned, and has definitely become owned again, we can remove the unowned flag entirely. This is the inverse of the logic that makes the derived temporarily unowned to begin with.

[changeset-bot]

🦋 Changeset detected

Latest commit: 8a952ba

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

[github-actions]

Playground

pnpm add https://pkg.pr.new/svelte@15255

[techniq]

Thanks @trueadm, you're amazing! This resolves the 4 LayerChart regressions identified in techniq/layerchart#384 (tested locally with pnpm add https://pkg.pr.new/svelte@15255)

[Leonidaz]

this pr also fixes #15180

[trueadm]

@Leonidaz @techniq, I changed the PR to opt for a slightly simpler strategy, can you both please re-test and let me know if things still are resolved?

[Leonidaz]

just tested, @trueadm as of 2957e6c, #15180 again appears to be broken.

[techniq]

@trueadm all regressions appear to still be fixed after re-running pnpm add https://pkg.pr.new/svelte@15255

[Leonidaz]

just tested, @trueadm as of 2957e6c, #15180 again appears to be broken.

if it helps this is the last commit that made it work: b054746

[trueadm]

The simplification on props actually introduced other issues, so the previous fix was indeed correct. Thanks for you help!