[Refactor] context comes first

ljharb · ljharb · commit 0c804bb93a74 · 2024-05-16T16:57:45.000-07:00
diff --git a/lib/rules/button-has-type.js b/lib/rules/button-has-type.js
@@ -135,7 +135,7 @@ module.exports = {
         checkValue(node, propValue);
       },
       CallExpression(node) {
-        if (!isCreateElement(node, context) || node.arguments.length < 1) {
+        if (!isCreateElement(context, node) || node.arguments.length < 1) {
           return;
         }
 
diff --git a/lib/rules/checked-requires-onchange-or-readonly.js b/lib/rules/checked-requires-onchange-or-readonly.js
@@ -115,7 +115,7 @@ module.exports = {
         checkAttributesAndReport(node, propSet);
       },
       CallExpression(node) {
-        if (!isCreateElement(node, context)) {
+        if (!isCreateElement(context, node)) {
           return;
         }
 
diff --git a/lib/rules/forbid-elements.js b/lib/rules/forbid-elements.js
@@ -95,7 +95,7 @@ module.exports = {
       },
 
       CallExpression(node) {
-        if (!isCreateElement(node, context)) {
+        if (!isCreateElement(context, node)) {
           return;
         }
 
diff --git a/lib/rules/iframe-missing-sandbox.js b/lib/rules/iframe-missing-sandbox.js
@@ -131,7 +131,7 @@ module.exports = {
       },
 
       CallExpression(node) {
-        if (isCreateElement(node, context) && node.arguments && node.arguments.length > 0) {
+        if (isCreateElement(context, node) && node.arguments && node.arguments.length > 0) {
           const tag = node.arguments[0];
           if (tag.type === 'Literal' && tag.value === 'iframe') {
             checkProps(context, node);
diff --git a/lib/rules/jsx-indent.js b/lib/rules/jsx-indent.js
@@ -425,7 +425,7 @@ module.exports = {
         }
         if (
           !fn
-          || !jsxUtil.isReturningJSX(node, context, true)
+          || !jsxUtil.isReturningJSX(context, node, true)
         ) {
           return;
         }
diff --git a/lib/rules/jsx-no-comment-textnodes.js b/lib/rules/jsx-no-comment-textnodes.js
@@ -17,7 +17,12 @@ const messages = {
   putCommentInBraces: 'Comments inside children section of tag should be placed inside braces',
 };
 
-function checkText(node, context) {
+/**
+ * @param {Context} context
+ * @param {ASTNode} node
+ * @returns {void}
+ */
+function checkText(context, node) {
   // since babel-eslint has the wrong node.raw, we'll get the source text
   const rawValue = getSourceCode(context).getText(node);
   if (/^\s*\/(\/|\*)/m.test(rawValue)) {
@@ -56,10 +61,10 @@ module.exports = {
 
     return {
       Literal(node) {
-        checkText(node, context);
+        checkText(context, node);
       },
       JSXText(node) {
-        checkText(node, context);
+        checkText(context, node);
       },
     };
   },
diff --git a/lib/rules/jsx-sort-default-props.js b/lib/rules/jsx-sort-default-props.js
@@ -112,7 +112,7 @@ module.exports = {
      */
     function checkSorted(declarations) {
       // function fix(fixer) {
-      //   return propTypesSortUtil.fixPropTypesSort(fixer, context, declarations, ignoreCase);
+      //   return propTypesSortUtil.fixPropTypesSort(context, fixer, declarations, ignoreCase);
       // }
 
       declarations.reduce((prev, curr, idx, decls) => {
diff --git a/lib/rules/no-adjacent-inline-elements.js b/lib/rules/no-adjacent-inline-elements.js
@@ -111,7 +111,7 @@ module.exports = {
         validate(node, node.children);
       },
       CallExpression(node) {
-        if (!isCreateElement(node, context)) {
+        if (!isCreateElement(context, node)) {
           return;
         }
         if (node.arguments.length < 2 || !node.arguments[2]) {
diff --git a/lib/rules/no-children-prop.js b/lib/rules/no-children-prop.js
@@ -21,7 +21,7 @@ const report = require('../util/report');
  * object literal, False if not.
 */
 function isCreateElementWithProps(node, context) {
-  return isCreateElement(node, context)
+  return isCreateElement(context, node)
     && node.arguments.length > 1
     && node.arguments[1].type === 'ObjectExpression';
 }
diff --git a/lib/rules/no-namespace.js b/lib/rules/no-namespace.js
@@ -36,7 +36,7 @@ module.exports = {
   create(context) {
     return {
       CallExpression(node) {
-        if (isCreateElement(node, context) && node.arguments.length > 0 && node.arguments[0].type === 'Literal') {
+        if (isCreateElement(context, node) && node.arguments.length > 0 && node.arguments[0].type === 'Literal') {
           const name = node.arguments[0].value;
           if (typeof name !== 'string' || name.indexOf(':') === -1) return undefined;
           report(context, messages.noNamespace, 'noNamespace', {
diff --git a/lib/rules/no-unstable-nested-components.js b/lib/rules/no-unstable-nested-components.js
@@ -68,7 +68,7 @@ function isCreateElementMatcher(node, context) {
   return (
     node
     && node.type === 'CallExpression'
-    && isCreateElement(node, context)
+    && isCreateElement(context, node)
   );
 }
 
diff --git a/lib/rules/sort-default-props.js b/lib/rules/sort-default-props.js
@@ -107,7 +107,7 @@ module.exports = {
      */
     function checkSorted(declarations) {
       // function fix(fixer) {
-      //   return propTypesSortUtil.fixPropTypesSort(fixer, context, declarations, ignoreCase);
+      //   return propTypesSortUtil.fixPropTypesSort(context, fixer, declarations, ignoreCase);
       // }
 
       declarations.reduce((prev, curr, idx, decls) => {
diff --git a/lib/rules/sort-prop-types.js b/lib/rules/sort-prop-types.js
@@ -119,8 +119,8 @@ module.exports = {
 
       function fix(fixer) {
         return propTypesSortUtil.fixPropTypesSort(
-          fixer,
           context,
+          fixer,
           declarations,
           ignoreCase,
           requiredFirst,
diff --git a/lib/rules/style-prop-object.js b/lib/rules/style-prop-object.js
@@ -77,7 +77,7 @@ module.exports = {
     return {
       CallExpression(node) {
         if (
-          isCreateElement(node, context)
+          isCreateElement(context, node)
           && node.arguments.length > 1
         ) {
           if (node.arguments[0].name) {
diff --git a/lib/rules/void-dom-elements-no-children.js b/lib/rules/void-dom-elements-no-children.js
@@ -108,7 +108,7 @@ module.exports = {
         return;
       }
 
-      if (!isCreateElement(node, context)) {
+      if (!isCreateElement(context, node)) {
         return;
       }
 
diff --git a/lib/util/Components.js b/lib/util/Components.js
@@ -298,15 +298,20 @@ function componentRule(rule, context) {
      * @returns {Boolean} True if createElement is destructured from the pragma
      */
     isDestructuredFromPragmaImport(variable) {
-      return isDestructuredFromPragmaImport(variable, context);
+      return isDestructuredFromPragmaImport(context, variable);
     },
 
+    /**
+     * @param {ASTNode} ASTNode
+     * @param {boolean=} strict
+     * @returns {boolean}
+     */
     isReturningJSX(ASTNode, strict) {
-      return jsxUtil.isReturningJSX(ASTNode, context, strict, true);
+      return jsxUtil.isReturningJSX(context, ASTNode, strict, true);
     },
 
     isReturningJSXOrNull(ASTNode, strict) {
-      return jsxUtil.isReturningJSX(ASTNode, context, strict);
+      return jsxUtil.isReturningJSX(context, ASTNode, strict);
     },
 
     isReturningOnlyNull(ASTNode) {
diff --git a/lib/util/isCreateElement.js b/lib/util/isCreateElement.js
@@ -5,11 +5,11 @@ const isDestructuredFromPragmaImport = require('./isDestructuredFromPragmaImport
 
 /**
  * Checks if the node is a createElement call
- * @param {ASTNode} node - The AST node being checked.
  * @param {Context} context - The AST node being checked.
- * @returns {Boolean} - True if node is a createElement call object literal, False if not.
+ * @param {ASTNode} node - The AST node being checked.
+ * @returns {boolean} - True if node is a createElement call object literal, False if not.
 */
-module.exports = function isCreateElement(node, context) {
+module.exports = function isCreateElement(context, node) {
   if (
     node.callee
     && node.callee.type === 'MemberExpression'
@@ -24,7 +24,7 @@ module.exports = function isCreateElement(node, context) {
     node
     && node.callee
     && node.callee.name === 'createElement'
-    && isDestructuredFromPragmaImport('createElement', context)
+    && isDestructuredFromPragmaImport(context, 'createElement')
   ) {
     return true;
   }
diff --git a/lib/util/isDestructuredFromPragmaImport.js b/lib/util/isDestructuredFromPragmaImport.js
@@ -6,11 +6,11 @@ const variableUtil = require('./variable');
 /**
  * Check if variable is destructured from pragma import
  *
- * @param {string} variable The variable name to check
  * @param {Context} context eslint context
- * @returns {Boolean} True if createElement is destructured from the pragma
+ * @param {string} variable The variable name to check
+ * @returns {boolean} True if createElement is destructured from the pragma
  */
-module.exports = function isDestructuredFromPragmaImport(variable, context) {
+module.exports = function isDestructuredFromPragmaImport(context, variable) {
   const pragma = pragmaUtil.getFromContext(context);
   const variables = variableUtil.variablesInScope(context);
   const variableInScope = variableUtil.getVariable(variables, variable);
diff --git a/lib/util/jsx.js b/lib/util/jsx.js
@@ -86,13 +86,13 @@ function isWhiteSpaces(value) {
 /**
  * Check if the node is returning JSX or null
  *
- * @param {ASTNode} ASTnode The AST node being checked
  * @param {Context} context The context of `ASTNode`.
+ * @param {ASTNode} ASTnode The AST node being checked
  * @param {Boolean} [strict] If true, in a ternary condition the node must return JSX in both cases
  * @param {Boolean} [ignoreNull] If true, null return values will be ignored
  * @returns {Boolean} True if the node is returning JSX or null, false if not
  */
-function isReturningJSX(ASTnode, context, strict, ignoreNull) {
+function isReturningJSX(context, ASTnode, strict, ignoreNull) {
   const isJSXValue = (node) => {
     if (!node) {
       return false;
@@ -114,7 +114,7 @@ function isReturningJSX(ASTnode, context, strict, ignoreNull) {
       case 'JSXFragment':
         return true;
       case 'CallExpression':
-        return isCreateElement(node, context);
+        return isCreateElement(context, node);
       case 'Literal':
         if (!ignoreNull && node.value === null) {
           return true;
diff --git a/lib/util/propTypesSort.js b/lib/util/propTypesSort.js
@@ -115,8 +115,8 @@ const commentnodeMap = new WeakMap(); // all nodes reference WeakMap for start a
 /**
  * Fixes sort order of prop types.
  *
+ * @param {Context} context the second element to compare.
  * @param {Fixer} fixer the first element to compare.
- * @param {Object} context the second element to compare.
  * @param {Array} declarations The context of the two nodes.
  * @param {Boolean=} ignoreCase whether or not to ignore case when comparing the two elements.
  * @param {Boolean=} requiredFirst whether or not to sort required elements first.
@@ -126,8 +126,8 @@ const commentnodeMap = new WeakMap(); // all nodes reference WeakMap for start a
  * @returns {Object|*|{range, text}} the sort order of the two elements.
  */
 function fixPropTypesSort(
-  fixer,
   context,
+  fixer,
   declarations,
   ignoreCase,
   requiredFirst,
diff --git a/lib/util/usedPropTypes.js b/lib/util/usedPropTypes.js
@@ -253,13 +253,13 @@ function isPropTypesUsageByMemberExpression(node, context, utils, checkAsyncSafe
 
 /**
  * Retrieve the name of a property node
- * @param {ASTNode} node The AST node with the property.
  * @param {Context} context
+ * @param {ASTNode} node The AST node with the property.
  * @param {Object} utils
  * @param {boolean} checkAsyncSafeLifeCycles
  * @return {string|undefined} the name of the property or undefined if not found
  */
-function getPropertyName(node, context, utils, checkAsyncSafeLifeCycles) {
+function getPropertyName(context, node, utils, checkAsyncSafeLifeCycles) {
   const property = node.property;
   if (property) {
     switch (property.type) {
@@ -312,7 +312,7 @@ module.exports = function usedPropTypesInstructions(context, components, utils)
     switch (node.type) {
       case 'OptionalMemberExpression':
       case 'MemberExpression':
-        name = getPropertyName(node, context, utils, checkAsyncSafeLifeCycles);
+        name = getPropertyName(context, node, utils, checkAsyncSafeLifeCycles);
         if (name) {
           allNames = parentNames.concat(name);
           if (
diff --git a/tests/util/jsx.js b/tests/util/jsx.js
@@ -35,7 +35,7 @@ const mockContext = {
 describe('jsxUtil', () => {
   describe('isReturningJSX', () => {
     const assertValid = (codeStr) => assert(
-      isReturningJSX(parseCode(codeStr), mockContext)
+      isReturningJSX(mockContext, parseCode(codeStr))
     );
 
     it('Works when returning JSX', () => {

Original file line number	Diff line number	Diff line change
`@@ -135,7 +135,7 @@ module.exports = {`
`135`	`135`	`checkValue(node, propValue);`
`136`	`136`	`},`
`137`	`137`	`CallExpression(node) {`
`138`		`- if (!isCreateElement(node, context) \|\| node.arguments.length < 1) {`
	`138`	`+ if (!isCreateElement(context, node) \|\| node.arguments.length < 1) {`
`139`	`139`	`return;`
`140`	`140`	`}`
`141`	`141`
Original file line number	Diff line number	Diff line change
`@@ -115,7 +115,7 @@ module.exports = {`
`115`	`115`	`checkAttributesAndReport(node, propSet);`
`116`	`116`	`},`
`117`	`117`	`CallExpression(node) {`
`118`		`- if (!isCreateElement(node, context)) {`
	`118`	`+ if (!isCreateElement(context, node)) {`
`119`	`119`	`return;`
`120`	`120`	`}`
`121`	`121`
Original file line number	Diff line number	Diff line change
`@@ -95,7 +95,7 @@ module.exports = {`
`95`	`95`	`},`
`96`	`96`
`97`	`97`	`CallExpression(node) {`
`98`		`- if (!isCreateElement(node, context)) {`
	`98`	`+ if (!isCreateElement(context, node)) {`
`99`	`99`	`return;`
`100`	`100`	`}`
`101`	`101`
Original file line number	Diff line number	Diff line change
`@@ -425,7 +425,7 @@ module.exports = {`
`425`	`425`	`}`
`426`	`426`	`if (`
`427`	`427`	`!fn`
`428`		`- \|\| !jsxUtil.isReturningJSX(node, context, true)`
	`428`	`+ \|\| !jsxUtil.isReturningJSX(context, node, true)`
`429`	`429`	`) {`
`430`	`430`	`return;`
`431`	`431`	`}`
Original file line number	Diff line number	Diff line change
`@@ -111,7 +111,7 @@ module.exports = {`
`111`	`111`	`validate(node, node.children);`
`112`	`112`	`},`
`113`	`113`	`CallExpression(node) {`
`114`		`- if (!isCreateElement(node, context)) {`
	`114`	`+ if (!isCreateElement(context, node)) {`
`115`	`115`	`return;`
`116`	`116`	`}`
`117`	`117`	`if (node.arguments.length < 2 \|\| !node.arguments[2]) {`
Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,7 @@ const report = require('../util/report');`
`21`	`21`	`* object literal, False if not.`
`22`	`22`	`*/`
`23`	`23`	`function isCreateElementWithProps(node, context) {`
`24`		`- return isCreateElement(node, context)`
	`24`	`+ return isCreateElement(context, node)`
`25`	`25`	`&& node.arguments.length > 1`
`26`	`26`	`&& node.arguments[1].type === 'ObjectExpression';`
`27`	`27`	`}`