feat(StoreDevtools): Add support for ActionSanitizer and StateSanitizer

[kgkma]

Continuing the work from #544 and requested by #604, this PR actually adds the sanitization functionality to @ngrx/store-devtools.

Unfortunately, simply passing actionSanitizer and stateSanitizer to the Redux Devtools Extension does not work, since the send(action, state) function exposed by its API does not sanitize anything if the action passed in is null, which is the current behaviour. As a result, it is necessary to sanitize within ngrx Devtools itself, and then pass that into the extension.

This implementation deliberately avoids mutating the lifted state's actionsByIds and computedStates properties, but instead creates a sanitized copy of them. This prevents the possibility of a sanitized action from changing the store state if, for example, its sanitized type coincided with a real action's type within the application.

Testing

In app.module.ts:

StoreDevtoolsModule.instrument({
  actionSanitizer: (action: Action, id: number) => {
    return { type: 'SANITIZED_ACTION' };
  },
  stateSanitizer: (state: any, index: number) => {
    return { ...state, extra: 'SANITIZED_STATE' };
  }
})

Open Chrome, confirm that it appears, while app and time travel seem to work fine.

[coveralls]

Coverage increased (+0.1%) to 92.827% when pulling c6d6110 on kgkma:master into 0429276 on ngrx:master.

[brandonroberts]

Pass the stateSanitizer as an argument instead of the config.

[brandonroberts]

One minor change

[kgkma]

@brandonroberts Thank you! Fixed as requested.

[brandonroberts]

Thanks!

[aboyton]

Any chance of a patch release so I can use this?