--- /dev/null
+/**
+ * @fileoverview Rule to suggest using "Reflect" api over Function/Object methods
+ * @author Keith Cirkel <http://keithcirkel.co.uk>
+ * @deprecated in ESLint v3.9.0
+ */
+"use strict";
+
+//------------------------------------------------------------------------------
+// Rule Definition
+//------------------------------------------------------------------------------
+
+module.exports = {
+ meta: {
+ type: "suggestion",
+
+ docs: {
+ description: "require `Reflect` methods where applicable",
+ category: "ECMAScript 6",
+ recommended: false,
+ url: "https://eslint.org/docs/rules/prefer-reflect"
+ },
+
+ deprecated: true,
+
+ replacedBy: [],
+
+ schema: [
+ {
+ type: "object",
+ properties: {
+ exceptions: {
+ type: "array",
+ items: {
+ enum: [
+ "apply",
+ "call",
+ "delete",
+ "defineProperty",
+ "getOwnPropertyDescriptor",
+ "getPrototypeOf",
+ "setPrototypeOf",
+ "isExtensible",
+ "getOwnPropertyNames",
+ "preventExtensions"
+ ]
+ },
+ uniqueItems: true
+ }
+ },
+ additionalProperties: false
+ }
+ ]
+ },
+
+ create(context) {
+ const existingNames = {
+ apply: "Function.prototype.apply",
+ call: "Function.prototype.call",
+ defineProperty: "Object.defineProperty",
+ getOwnPropertyDescriptor: "Object.getOwnPropertyDescriptor",
+ getPrototypeOf: "Object.getPrototypeOf",
+ setPrototypeOf: "Object.setPrototypeOf",
+ isExtensible: "Object.isExtensible",
+ getOwnPropertyNames: "Object.getOwnPropertyNames",
+ preventExtensions: "Object.preventExtensions"
+ };
+
+ const reflectSubsitutes = {
+ apply: "Reflect.apply",
+ call: "Reflect.apply",
+ defineProperty: "Reflect.defineProperty",
+ getOwnPropertyDescriptor: "Reflect.getOwnPropertyDescriptor",
+ getPrototypeOf: "Reflect.getPrototypeOf",
+ setPrototypeOf: "Reflect.setPrototypeOf",
+ isExtensible: "Reflect.isExtensible",
+ getOwnPropertyNames: "Reflect.getOwnPropertyNames",
+ preventExtensions: "Reflect.preventExtensions"
+ };
+
+ const exceptions = (context.options[0] || {}).exceptions || [];
+
+ /**
+ * Reports the Reflect violation based on the `existing` and `substitute`
+ * @param {Object} node The node that violates the rule.
+ * @param {string} existing The existing method name that has been used.
+ * @param {string} substitute The Reflect substitute that should be used.
+ * @returns {void}
+ */
+ function report(node, existing, substitute) {
+ context.report({
+ node,
+ message: "Avoid using {{existing}}, instead use {{substitute}}.",
+ data: {
+ existing,
+ substitute
+ }
+ });
+ }
+
+ return {
+ CallExpression(node) {
+ const methodName = (node.callee.property || {}).name;
+ const isReflectCall = (node.callee.object || {}).name === "Reflect";
+ const hasReflectSubsitute = Object.prototype.hasOwnProperty.call(reflectSubsitutes, methodName);
+ const userConfiguredException = exceptions.indexOf(methodName) !== -1;
+
+ if (hasReflectSubsitute && !isReflectCall && !userConfiguredException) {
+ report(node, existingNames[methodName], reflectSubsitutes[methodName]);
+ }
+ },
+ UnaryExpression(node) {
+ const isDeleteOperator = node.operator === "delete";
+ const targetsIdentifier = node.argument.type === "Identifier";
+ const userConfiguredException = exceptions.indexOf("delete") !== -1;
+
+ if (isDeleteOperator && !targetsIdentifier && !userConfiguredException) {
+ report(node, "the delete keyword", "Reflect.deleteProperty");
+ }
+ }
+ };
+
+ }
+};