Since async-await was introduced into the code base a lot has changed. This proposes to lift the restriction that flushSync cannot be called in effects. No tests fail, and in general the whole logic is a lot more robust now (and, to be fair, the stack traces when debugging are confusing either way).

Closes #17131 (though I still wonder why Skeleton does that)

Before submitting the PR, please make sure you do the following

• It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
• Prefix your PR title with feat:, fix:, chore:, or docs:.
• This message body should clearly illustrate what problems it solves.
• Ideally, include a test that fails without this PR but passes with it.
• If this PR changes code within packages/svelte/src, add a changeset (npx changeset).

Tests and linting

• Run the tests with pnpm test and lint the project with pnpm lint