fix: cannot assign readonly properties primitives

[alvarosabu]

No description provided.

[netlify]

✅ Deploy Preview for tresjs-docs ready!

Name	Link
🔨 Latest commit	6506921
🔍 Latest deploy log	https://app.netlify.com/sites/tresjs-docs/deploys/665998749a49d2000868477d
😎 Deploy Preview	https://deploy-preview-698--tresjs-docs.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[andretchen0]

Hey @alvarosabu ,

I'm not sure I understand the issue here, so I don't know what to do to approve or request changes.

I tried to find an open issue for more context, but didn't find one. So, I'm just going by what's changed here.

There seems to be conflicting info:

• the issue is called fix: cannot assign readonly properties primitives
• in nodeOps, the modified code only applies to arguments
• the added test only tests new arguments, but doesn't test a primitive, as far as I can tell

---

I ran this test to see if read-only properties were handled.

      it('handles read-only properties 2', () => {
        const obj = Object.freeze({ foo: "bar" });
        nodeOps.patchProp(obj as any, 'foo', 'bar', 'baz')
        expect(obj.foo).toBe('bar')
      })

I'm not sure that this case is what the PR is trying to fix, but nodeOps throws an error, fwiw.