[engine] [object:vm undefined (0)] is set to a non-trackable object - confuses developers

[apapko]

Description

The error message for the non-trackable object confuses developers. We need to enhance this message to ensure that the intent of the warning is clear. Here is a primitive repro:
https://playground.lwcjs.org/projects/Cw4rnCPsc/3/edit

I think the intent of the message is good in a way that it lets our users know that the received value will never change and that they may want to ensure that the passed value is should indeed not be tracked or be public.

[apapko]

Talked to @caridy and we've decided to remove the warning to eliminate the confusion. The object mutation will result in an error in dev mode, however, in production it will be a noop.