Add error codes to production invariants like React did in 15.2

[philippe-git]

Do you want to request a feature or report a bug?

Request a feature

What is the current behavior?

We're currently seeing rare invariant errors in production with the good old "Minified exception occurred; use the non-minified dev environment for the full error message and additional helpful warnings".

We're using React 15.4.2, so I was expecting nice error codes instead from reactProdInvariant.

Looking more closely, I realized DraftJS was also using invariant, but not reactProdInvariant; it feels pretty safe to assume that those production exceptions come from DraftJs.

What is the expected behavior?

Could DraftJS implement the same improvement to production invariants that React did in 15.2 with reactProdInvariant? See react/react#6948

[sophiebits]

Draft doesn't have that many invariants and is less size-constrained; we could just not strip them when building the packages.

[nburt]

@sophiebits Are you suggesting updating from
!selectionState.isCollapsed() ? process.env.NODE_ENV !== 'production' ? invariant(false, '`insertText` should only be called with a collapsed range.') : invariant(false) : void 0;
to
!selectionState.isCollapsed() ? invariant(false, '`insertText` should only be called with a collapsed range.') : void 0;?

We're getting several hundred of these errors every week so if you're open to a PR for this I'll find some time