refactor(computed): deprecated computedRunners (#1458)

diff --git a/packages/reactivity/src/computed.ts b/packages/reactivity/src/computed.ts
index d6f89fe284ecdb24db4c1b20c02c38c073607944..611ecaefb566cb1378ca9b936e24df1ce93d81f0 100644 (file)
--- a/packages/reactivity/src/computed.ts
+++ b/packages/reactivity/src/computed.ts
@@ -48,7 +48,6 @@ export function computed<T>(
   const runner = effect(getter, {
     lazy: true,
     // mark effect as computed so that it gets priority during trigger
-    computed: true,
     scheduler: () => {
       if (!dirty) {
         dirty = true

diff --git a/packages/reactivity/src/effect.ts b/packages/reactivity/src/effect.ts
index c6187d296ee2046c94de030ab8d0d3aaa6976be7..98aff15ddbd3756081b90ec45d4dbc28b601f007 100644 (file)
--- a/packages/reactivity/src/effect.ts
+++ b/packages/reactivity/src/effect.ts
@@ -177,16 +177,11 @@ export function trigger(
   }
 
   const effects = new Set<ReactiveEffect>()
-  const computedRunners = new Set<ReactiveEffect>()
   const add = (effectsToAdd: Set<ReactiveEffect> | undefined) => {
     if (effectsToAdd) {
       effectsToAdd.forEach(effect => {
         if (effect !== activeEffect || !shouldTrack) {
-          if (effect.options.computed) {
-            computedRunners.add(effect)
-          } else {
-            effects.add(effect)
-          }
+          effects.add(effect)
         } else {
           // the effect mutated its own dependency during its execution.
           // this can be caused by operations like foo.value++
@@ -245,8 +240,5 @@ export function trigger(
     }
   }
 
-  // Important: computed effects must be run first so that computed getters
-  // can be invalidated before any normal effects that depend on them are run.
-  computedRunners.forEach(run)
   effects.forEach(run)
 }