mirror of
https://github.com/facebook/react.git
synced 2026-02-23 20:23:02 +00:00
* Hoist error codes import to module scope When this code was written, the error codes map (`codes.json`) was created on-the-fly, so we had to lazily require from inside the visitor. Because `codes.json` is now checked into source, we can import it a single time in module scope. * Minify error constructors in production We use a script to minify our error messages in production. Each message is assigned an error code, defined in `scripts/error-codes/codes.json`. Then our build script replaces the messages with a link to our error decoder page, e.g. https://reactjs.org/docs/error-decoder.html/?invariant=92 This enables us to write helpful error messages without increasing the bundle size. Right now, the script only works for `invariant` calls. It does not work if you throw an Error object. This is an old Facebookism that we don't really need, other than the fact that our error minification script relies on it. So, I've updated the script to minify error constructors, too: Input: Error(`A ${adj} message that contains ${noun}`); Output: Error(formatProdErrorMessage(ERR_CODE, adj, noun)); It only works for constructors that are literally named Error, though we could add support for other names, too. As a next step, I will add a lint rule to enforce that errors written this way must have a corresponding error code. * Minify "no fallback UI specified" error in prod This error message wasn't being minified because it doesn't use invariant. The reason it didn't use invariant is because this particular error is created without begin thrown — it doesn't need to be thrown because it's located inside the error handling part of the runtime. Now that the error minification script supports Error constructors, we can minify it by assigning it a production error code in `scripts/error-codes/codes.json`. To support the use of Error constructors more generally, I will add a lint rule that enforces each message has a corresponding error code. * Lint rule to detect unminified errors Adds a lint rule that detects when an Error constructor is used without a corresponding production error code. We already have this for `invariant`, but not for regular errors, i.e. `throw new Error(msg)`. There's also nothing that enforces the use of `invariant` besides convention. There are some packages where we don't care to minify errors. These are packages that run in environments where bundle size is not a concern, like react-pg. I added an override in the ESLint config to ignore these. * Temporarily add invariant codemod script I'm adding this codemod to the repo temporarily, but I'll revert it in the same PR. That way we don't have to check it in but it's still accessible (via the PR) if we need it later. * [Automated] Codemod invariant -> Error This commit contains only automated changes: npx jscodeshift -t scripts/codemod-invariant.js packages --ignore-pattern="node_modules/**/*" yarn linc --fix yarn prettier I will do any manual touch ups in separate commits so they're easier to review. * Remove temporary codemod script This reverts the codemod script and ESLint config I added temporarily in order to perform the invariant codemod. * Manual touch ups A few manual changes I made after the codemod ran. * Enable error code transform per package Currently we're not consistent about which packages should have their errors minified in production and which ones should. This adds a field to the bundle configuration to control whether to apply the transform. We should decide what the criteria is going forward. I think it's probably a good idea to minify any package that gets sent over the network. So yes to modules that run in the browser, and no to modules that run on the server and during development only.
253 lines
7.9 KiB
JavaScript
253 lines
7.9 KiB
JavaScript
/**
|
|
* Copyright (c) Facebook, Inc. and its affiliates.
|
|
*
|
|
* This source code is licensed under the MIT license found in the
|
|
* LICENSE file in the root directory of this source tree.
|
|
*/
|
|
'use strict';
|
|
|
|
const fs = require('fs');
|
|
const {
|
|
evalStringConcat,
|
|
evalStringAndTemplateConcat,
|
|
} = require('../shared/evalToString');
|
|
const invertObject = require('./invertObject');
|
|
const helperModuleImports = require('@babel/helper-module-imports');
|
|
|
|
const errorMap = invertObject(
|
|
JSON.parse(fs.readFileSync(__dirname + '/codes.json', 'utf-8'))
|
|
);
|
|
|
|
const SEEN_SYMBOL = Symbol('transform-error-messages.seen');
|
|
|
|
module.exports = function(babel) {
|
|
const t = babel.types;
|
|
|
|
// TODO: Instead of outputting __DEV__ conditions, only apply this transform
|
|
// in production.
|
|
const DEV_EXPRESSION = t.identifier('__DEV__');
|
|
|
|
function CallOrNewExpression(path, file) {
|
|
// Turns this code:
|
|
//
|
|
// new Error(`A ${adj} message that contains ${noun}`);
|
|
//
|
|
// or this code (no constructor):
|
|
//
|
|
// Error(`A ${adj} message that contains ${noun}`);
|
|
//
|
|
// into this:
|
|
//
|
|
// Error(
|
|
// __DEV__
|
|
// ? `A ${adj} message that contains ${noun}`
|
|
// : formatProdErrorMessage(ERR_CODE, adj, noun)
|
|
// );
|
|
const node = path.node;
|
|
if (node[SEEN_SYMBOL]) {
|
|
return;
|
|
}
|
|
node[SEEN_SYMBOL] = true;
|
|
|
|
const errorMsgNode = node.arguments[0];
|
|
if (errorMsgNode === undefined) {
|
|
return;
|
|
}
|
|
|
|
const errorMsgExpressions = [];
|
|
const errorMsgLiteral = evalStringAndTemplateConcat(
|
|
errorMsgNode,
|
|
errorMsgExpressions
|
|
);
|
|
|
|
let prodErrorId = errorMap[errorMsgLiteral];
|
|
if (prodErrorId === undefined) {
|
|
// There is no error code for this message. We use a lint rule to
|
|
// enforce that messages can be minified, so assume this is
|
|
// intentional and exit gracefully.
|
|
return;
|
|
}
|
|
prodErrorId = parseInt(prodErrorId, 10);
|
|
|
|
// Import formatProdErrorMessage
|
|
const formatProdErrorMessageIdentifier = helperModuleImports.addDefault(
|
|
path,
|
|
'shared/formatProdErrorMessage',
|
|
{nameHint: 'formatProdErrorMessage'}
|
|
);
|
|
|
|
// Outputs:
|
|
// formatProdErrorMessage(ERR_CODE, adj, noun);
|
|
const prodMessage = t.callExpression(formatProdErrorMessageIdentifier, [
|
|
t.numericLiteral(prodErrorId),
|
|
...errorMsgExpressions,
|
|
]);
|
|
|
|
// Outputs:
|
|
// Error(
|
|
// __DEV__
|
|
// ? `A ${adj} message that contains ${noun}`
|
|
// : formatProdErrorMessage(ERR_CODE, adj, noun)
|
|
// );
|
|
path.replaceWith(t.callExpression(t.identifier('Error'), [prodMessage]));
|
|
path.replaceWith(
|
|
t.callExpression(t.identifier('Error'), [
|
|
t.conditionalExpression(DEV_EXPRESSION, errorMsgNode, prodMessage),
|
|
])
|
|
);
|
|
}
|
|
|
|
return {
|
|
visitor: {
|
|
NewExpression(path, file) {
|
|
const noMinify = file.opts.noMinify;
|
|
if (!noMinify && path.get('callee').isIdentifier({name: 'Error'})) {
|
|
CallOrNewExpression(path, file);
|
|
}
|
|
},
|
|
|
|
CallExpression(path, file) {
|
|
const node = path.node;
|
|
const noMinify = file.opts.noMinify;
|
|
|
|
if (!noMinify && path.get('callee').isIdentifier({name: 'Error'})) {
|
|
CallOrNewExpression(path, file);
|
|
return;
|
|
}
|
|
|
|
if (path.get('callee').isIdentifier({name: 'invariant'})) {
|
|
// Turns this code:
|
|
//
|
|
// invariant(condition, 'A %s message that contains %s', adj, noun);
|
|
//
|
|
// into this:
|
|
//
|
|
// if (!condition) {
|
|
// throw Error(
|
|
// __DEV__
|
|
// ? `A ${adj} message that contains ${noun}`
|
|
// : formatProdErrorMessage(ERR_CODE, adj, noun)
|
|
// );
|
|
// }
|
|
//
|
|
// where ERR_CODE is an error code: a unique identifier (a number
|
|
// string) that references a verbose error message. The mapping is
|
|
// stored in `scripts/error-codes/codes.json`.
|
|
const condition = node.arguments[0];
|
|
const errorMsgLiteral = evalStringConcat(node.arguments[1]);
|
|
const errorMsgExpressions = Array.from(node.arguments.slice(2));
|
|
const errorMsgQuasis = errorMsgLiteral
|
|
.split('%s')
|
|
.map(raw => t.templateElement({raw, cooked: String.raw({raw})}));
|
|
|
|
// Outputs:
|
|
// `A ${adj} message that contains ${noun}`;
|
|
const devMessage = t.templateLiteral(
|
|
errorMsgQuasis,
|
|
errorMsgExpressions
|
|
);
|
|
|
|
const parentStatementPath = path.parentPath;
|
|
if (parentStatementPath.type !== 'ExpressionStatement') {
|
|
throw path.buildCodeFrameError(
|
|
'invariant() cannot be called from expression context. Move ' +
|
|
'the call to its own statement.'
|
|
);
|
|
}
|
|
|
|
if (noMinify) {
|
|
// Error minification is disabled for this build.
|
|
//
|
|
// Outputs:
|
|
// if (!condition) {
|
|
// throw Error(`A ${adj} message that contains ${noun}`);
|
|
// }
|
|
parentStatementPath.replaceWith(
|
|
t.ifStatement(
|
|
t.unaryExpression('!', condition),
|
|
t.blockStatement([
|
|
t.throwStatement(
|
|
t.callExpression(t.identifier('Error'), [devMessage])
|
|
),
|
|
])
|
|
)
|
|
);
|
|
return;
|
|
}
|
|
|
|
let prodErrorId = errorMap[errorMsgLiteral];
|
|
|
|
if (prodErrorId === undefined) {
|
|
// There is no error code for this message. Add an inline comment
|
|
// that flags this as an unminified error. This allows the build
|
|
// to proceed, while also allowing a post-build linter to detect it.
|
|
//
|
|
// Outputs:
|
|
// /* FIXME (minify-errors-in-prod): Unminified error message in production build! */
|
|
// if (!condition) {
|
|
// throw Error(`A ${adj} message that contains ${noun}`);
|
|
// }
|
|
parentStatementPath.replaceWith(
|
|
t.ifStatement(
|
|
t.unaryExpression('!', condition),
|
|
t.blockStatement([
|
|
t.throwStatement(
|
|
t.callExpression(t.identifier('Error'), [devMessage])
|
|
),
|
|
])
|
|
)
|
|
);
|
|
parentStatementPath.addComment(
|
|
'leading',
|
|
'FIXME (minify-errors-in-prod): Unminified error message in production build!'
|
|
);
|
|
return;
|
|
}
|
|
prodErrorId = parseInt(prodErrorId, 10);
|
|
|
|
// Import formatProdErrorMessage
|
|
const formatProdErrorMessageIdentifier = helperModuleImports.addDefault(
|
|
path,
|
|
'shared/formatProdErrorMessage',
|
|
{nameHint: 'formatProdErrorMessage'}
|
|
);
|
|
|
|
// Outputs:
|
|
// formatProdErrorMessage(ERR_CODE, adj, noun);
|
|
const prodMessage = t.callExpression(
|
|
formatProdErrorMessageIdentifier,
|
|
[t.numericLiteral(prodErrorId), ...errorMsgExpressions]
|
|
);
|
|
|
|
// Outputs:
|
|
// if (!condition) {
|
|
// throw Error(
|
|
// __DEV__
|
|
// ? `A ${adj} message that contains ${noun}`
|
|
// : formatProdErrorMessage(ERR_CODE, adj, noun)
|
|
// );
|
|
// }
|
|
parentStatementPath.replaceWith(
|
|
t.ifStatement(
|
|
t.unaryExpression('!', condition),
|
|
t.blockStatement([
|
|
t.blockStatement([
|
|
t.throwStatement(
|
|
t.callExpression(t.identifier('Error'), [
|
|
t.conditionalExpression(
|
|
DEV_EXPRESSION,
|
|
devMessage,
|
|
prodMessage
|
|
),
|
|
])
|
|
),
|
|
]),
|
|
])
|
|
)
|
|
);
|
|
}
|
|
},
|
|
},
|
|
};
|
|
};
|