Production builds have error messages with much whitespace inside errors

[eweilow]

Intended outcome:

I'd expect to not have much whitespace inside errors for production, even though it's good for debugging 👍

Actual outcome:

For instance, this error:

apollo-client/packages/apollo-utilities/src/storeUtils.ts

Lines 128 to 131 in 6903c0e

	throw new Error(`The inline argument "${name.value}" of kind "${
	(value as any).kind
	}" is not supported.
	Use variables instead of inline arguments to overcome this limitation.`);

Cause the following code to be output:

throw new Error('The inline argument "'+t.value+'" of kind "'+r.kind+'" is not supported.\n                    Use variables instead of inline arguments to overcome this limitation.');

How to reproduce the issue:

Should be just a matter of building a project where apollo-client is used.

Version

• apollo-client@2.2.8

[reyhansofian]

hi, can I take this issue?

[hwillson]

The error message in the source was mangled a bit by prettier. We'd love a PR here that fixes the formatting of the error message, to get rid of the extra whitespace. This can just be as simple as removing the white space in the source, while keeping the message wrapped at 80 chars (in the source only - the 80 char limit doesn't need to be preserved at runtime). Thanks!

[hwillson]

Addressed in #3593 - thanks!