chore: adjust warning

Author: yyx990803

packages/reactivity/__tests__/computed.spec.ts

@@ -16,8 +16,6 @@ import {
 import { DirtyLevels } from '../src/constants'
 import { COMPUTED_SIDE_EFFECT_WARN } from '../src/computed'
 
-const normalizedSideEffectWarn = `[Vue warn] ${COMPUTED_SIDE_EFFECT_WARN}`
-
 describe('reactivity/computed', () => {
   it('should return updated value', () => {
     const value = reactive<{ foo?: number }>({})
@@ -491,7 +489,7 @@ describe('reactivity/computed', () => {
     expect(c3.effect._dirtyLevel).toBe(
       DirtyLevels.MaybeDirty_ComputedSideEffect,
     )
-    expect(normalizedSideEffectWarn).toHaveBeenWarned()
+    expect(COMPUTED_SIDE_EFFECT_WARN).toHaveBeenWarned()
   })
 
   it('should work when chained(ref+computed)', () => {
@@ -506,7 +504,7 @@ describe('reactivity/computed', () => {
     expect(c2.value).toBe('0foo')
     expect(c2.effect._dirtyLevel).toBe(DirtyLevels.Dirty)
     expect(c2.value).toBe('1foo')
-    expect(normalizedSideEffectWarn).toHaveBeenWarned()
+    expect(COMPUTED_SIDE_EFFECT_WARN).toHaveBeenWarned()
   })
 
   it('should trigger effect even computed already dirty', () => {
@@ -529,7 +527,7 @@ describe('reactivity/computed', () => {
     expect(c2.effect._dirtyLevel).toBe(DirtyLevels.Dirty)
     v.value = 2
     expect(fnSpy).toBeCalledTimes(2)
-    expect(normalizedSideEffectWarn).toHaveBeenWarned()
+    expect(COMPUTED_SIDE_EFFECT_WARN).toHaveBeenWarned()
   })
 
   // #10185
@@ -573,7 +571,7 @@ describe('reactivity/computed', () => {
     expect(c3.effect._dirtyLevel).toBe(DirtyLevels.MaybeDirty)
 
     expect(c3.value).toBe('yes')
-    expect(normalizedSideEffectWarn).toHaveBeenWarned()
+    expect(COMPUTED_SIDE_EFFECT_WARN).toHaveBeenWarned()
   })
 
   it('should be not dirty after deps mutate (mutate deps in computed)', async () => {
@@ -595,7 +593,7 @@ describe('reactivity/computed', () => {
     await nextTick()
     await nextTick()
     expect(serializeInner(root)).toBe(`2`)
-    expect(normalizedSideEffectWarn).toHaveBeenWarned()
+    expect(COMPUTED_SIDE_EFFECT_WARN).toHaveBeenWarned()
   })
 
   it('should not trigger effect scheduler by recurse computed effect', async () => {
@@ -618,6 +616,6 @@ describe('reactivity/computed', () => {
     v.value += ' World'
     await nextTick()
     expect(serializeInner(root)).toBe('Hello World World World World')
-    expect(normalizedSideEffectWarn).toHaveBeenWarned()
+    expect(COMPUTED_SIDE_EFFECT_WARN).toHaveBeenWarned()
   })
 })

packages/reactivity/src/computed.ts

@@ -26,8 +26,9 @@ export interface WritableComputedOptions<T> {
 }
 
 export const COMPUTED_SIDE_EFFECT_WARN =
-  `Computed keep dirty after evaluation.` +
-  ` This might happen when you mutate the deps of computed in the getter.` +
+  `Computed is still dirty after getter evaluation,` +
+  ` likely because a computed is mutating its own dependency in its getter.` +
+  ` State mutations in computed getters should be avoided. ` +
   ` Check the docs for more details: https://vuejs.org/guide/essentials/computed.html#getters-should-be-side-effect-free`
 
 export class ComputedRefImpl<T> {