5.4.1 Non observed properties are no longer included

So 5.4.1 fixed a previous error for https://github.com/mobxjs/mobx-utils/issues/191 but is causing another. Any properties that aren’t marked as observable simply don’t appear anymore.

If you cut and paste the below code into the demo sandbox, you’ll see what happens. the notObservedField doesn’t exist… and in 5.2 it did.

https://codesandbox.io/s/wp88p

import React from "react";
import { render } from "react-dom";
import { observable, action, toJS } from "mobx";
import { observer } from "mobx-react";
import DevTools from "mobx-react-devtools";
import { createViewModel } from 'mobx-utils';

class MyClass {
  @observable firstname;
  @observable lastname;
  notObservedField = 'im not included!';
  someFunction(someParam) {
  }
}

class AppState {
  constructor() {
    const myClass = new MyClass();
    const observableMyClass = createViewModel(myClass);
    console.log(toJS(observableMyClass));
    // is now undefined in 5.4.1
    console.log(observableMyClass.notObservedField);
  }

  @action.bound
  reset() {
    this.timer = 0;
  }
}

const TimerView = observer(({ appState }) => (
  <button onClick={appState.reset}>Seconds passed: {appState.timer}</button>
));


render(
  <div>
    <TimerView appState={new AppState()} />
    <DevTools />
  </div>,
  document.getElementById("root")
);

Edit: I should also mention that this only happened in production builds for me? Dev builds worked fine. Was a bit of a panic after rollover.

[ItamarShDev]

notObservedField was not logged also in 5.4.0 and 5.3.0 Am i missing the bugs?

https://codesandbox.io/embed/minimal-mobx-react-project-jh2lz The latest version I can log the property is in 5.2.0

[mweststrate]

Published as 5.5.0