refactor(reactivity): don't convert Set into array

[Alfred-Skyblue]

close #8714

[sxzz]

/ecosystem-ci run

[vue-bot]

📝 Ran ecosystem CI: Open

suite	result
nuxt	✅ success
pinia	✅ success
quasar	❌ failure
router	✅ success
test-utils	✅ success
vant	✅ success
vite-plugin-vue	✅ success
vitepress	❌ failure
vue-i18n	✅ success
vue-macros	❌ failure
vuetify	✅ success
vueuse	✅ success
vue-simple-compiler	✅ success

[skirtles-code]

Note for reviewers: I think there's some overlap with other PRs: #6185, #7827 and #8947.

[skirtles-code]

I believe the reason for copying the dep into an array is to ensure that the subsequent iterations are stable. The calls to triggerEffect() inside those loops could modify dep while we're trying to iterate over it.

The PR description and linked issue don't seem to mention this, so I'm wondering whether there's been a misunderstanding about the purpose of the array?

[johnsoncodehk]

Also resolved by #5912. Note that dep could change during iterations, but in #5912 it will only change after iterations, so the spread can be safely removed.