--- /dev/null
+/**
+ * @fileoverview Reports useless `catch` clauses that just rethrow their error.
+ * @author Teddy Katz
+ */
+
+"use strict";
+
+//------------------------------------------------------------------------------
+// Rule Definition
+//------------------------------------------------------------------------------
+
+module.exports = {
+ meta: {
+ type: "suggestion",
+
+ docs: {
+ description: "disallow unnecessary `catch` clauses",
+ category: "Best Practices",
+ recommended: true,
+ url: "https://eslint.org/docs/rules/no-useless-catch"
+ },
+
+ schema: []
+ },
+
+ create(context) {
+ return {
+ CatchClause(node) {
+ if (
+ node.param &&
+ node.param.type === "Identifier" &&
+ node.body.body.length &&
+ node.body.body[0].type === "ThrowStatement" &&
+ node.body.body[0].argument.type === "Identifier" &&
+ node.body.body[0].argument.name === node.param.name
+ ) {
+ if (node.parent.finalizer) {
+ context.report({
+ node,
+ message: "Unnecessary catch clause."
+ });
+ } else {
+ context.report({
+ node: node.parent,
+ message: "Unnecessary try/catch wrapper."
+ });
+ }
+ }
+ }
+ };
+ }
+};