Redundant reactions?

I’m confused about the behavior of the following code:

const list = observable(['Hello'])
observe(() => console.log(list))
list.push('World!')

This logs ['Hello', 'World!'] twice. I expect it to log once. I get expected behavior from:

const list = observable(['Hello'])
observe(() => console.log(list.concat()))
list.push('World!')

Is this the correct behavior?

[lukeburns]

Interesting! I’m using node. Wonder why it would be platform specific. Different Proxy implementations?

[solkimicreb]

I fixed the first (infinite loop issue) in the 4.1.1 release. Thanks for catching the bug 🙂

The second issue is specific to the NodeJS console.log implementation. Normal for in loops and object enumeration works, but apparently NodeJS console.log is implemented in native code. It is not getting person.name under the hood (you can see this with the now working debugger).

I will see if this can or should be fixed. Is this a big issue for you? I imagine this would only be used for debugging purposes.

Edit: instead of getting each property key, Node gets a shady inspect prop for console.log. I guess Object.prototype has a custom inspect prop in Node which points to some native magic.