fix(reactivity): respect readonly during ref unwrapping (#13905)

edison1105 · web-flow · commit aba7feda1703 · 2025-09-24T17:10:49.000+08:00
close #13903
diff --git a/packages/reactivity/__tests__/readonly.spec.ts b/packages/reactivity/__tests__/readonly.spec.ts
@@ -522,6 +522,16 @@ describe('reactivity/readonly', () => {
     expect(obj.r).toBe(ro)
     expect(r.value).toBe(ro)
   })
+
+  test('should keep nested ref readonly', () => {
+    const items = ref(['one', 'two', 'three'])
+    const obj = {
+      o: readonly({
+        items,
+      }),
+    }
+    expect(isReadonly(obj.o.items)).toBe(true)
+  })
 })
 
 test('should be able to trigger with triggerRef', () => {
diff --git a/packages/reactivity/src/baseHandlers.ts b/packages/reactivity/src/baseHandlers.ts
@@ -119,7 +119,8 @@ class BaseReactiveHandler implements ProxyHandler<Target> {
 
     if (isRef(res)) {
       // ref unwrapping - skip unwrap for Array + integer key.
-      return targetIsArray && isIntegerKey(key) ? res : res.value
+      const value = targetIsArray && isIntegerKey(key) ? res : res.value
+      return isReadonly && isObject(value) ? readonly(value) : value
     }
 
     if (isObject(res)) {

Original file line number	Diff line number	Diff line change
`@@ -119,7 +119,8 @@ class BaseReactiveHandler implements ProxyHandler<Target> {`
`119`	`119`
`120`	`120`	`if (isRef(res)) {`
`121`	`121`	`// ref unwrapping - skip unwrap for Array + integer key.`
`122`		`- return targetIsArray && isIntegerKey(key) ? res : res.value`
	`122`	`+ const value = targetIsArray && isIntegerKey(key) ? res : res.value`
	`123`	`+ return isReadonly && isObject(value) ? readonly(value) : value`
`123`	`124`	`}`
`124`	`125`
`125`	`126`	`if (isObject(res)) {`