@lvce-editor/eslint-plugin-virtual-dom 18.5.0 → 18.7.0

package/dist/index.js

@@ -93,7 +93,7 @@ const controlsAllowingChildren = new Set(['A', 'Button']);
 const hasDirectName = node => {
   return Boolean(getProperty(node, 'ariaLabel') || getProperty(node, 'ariaLabelledBy') || getProperty(node, 'title'));
 };
-const meta$q = {
+const meta$t = {
   docs: {
     description: 'Require a statically determinable accessible name for virtual-dom controls'
   },
@@ -102,7 +102,7 @@ const meta$q = {
   },
   type: 'problem'
 };
-const create$q = context => {
+const create$t = context => {
   return {
     ObjectExpression(node) {
       if (!isVirtualDomNode(node) || hasDirectName(node)) {
@@ -144,11 +144,11 @@ const create$q = context => {
 
 const accessibleControlName = {
   __proto__: null,
-  create: create$q,
-  meta: meta$q
+  create: create$t,
+  meta: meta$t
 };
 
-const meta$p = {
+const meta$s = {
   docs: {
     description: 'Require an explicit role on clickable virtual-dom div nodes'
   },
@@ -157,7 +157,7 @@ const meta$p = {
   },
   type: 'problem'
 };
-const create$p = context => {
+const create$s = context => {
   return {
     ObjectExpression(node) {
       if (!isVirtualDomNode(node)) {
@@ -179,11 +179,11 @@ const create$p = context => {
 
 const clickableDivNeedsRole = {
   __proto__: null,
-  create: create$p,
-  meta: meta$p
+  create: create$s,
+  meta: meta$s
 };
 
-const meta$o = {
+const meta$r = {
   docs: {
     description: 'Require static virtual-dom nodes created inside functions to be hoisted to module scope'
   },
@@ -318,7 +318,7 @@ const isStaticExpression = (sourceCode, node) => {
       return false;
   }
 };
-const create$o = context => {
+const create$r = context => {
   return {
     ObjectExpression(node) {
       if (!isVirtualDomElement(node)) {
@@ -343,11 +343,11 @@ const create$o = context => {
 
 const hoistStaticNodes = {
   __proto__: null,
-  create: create$o,
-  meta: meta$o
+  create: create$r,
+  meta: meta$r
 };
 
-const meta$n = {
+const meta$q = {
   docs: {
     description: 'Disallow conditional spreads in virtual-dom arrays'
   },
@@ -365,7 +365,7 @@ const isVirtualDomArray$1 = node => {
 const addsVirtualDomChildren = spread => {
   return isVirtualDomArray$1(spread.argument.consequent) || isVirtualDomArray$1(spread.argument.alternate);
 };
-const create$n = context => {
+const create$q = context => {
   return {
     ArrayExpression(node) {
       const conditionalSpreads = node.elements.filter(isConditionalSpread);
@@ -384,11 +384,40 @@ const create$n = context => {
 
 const noConditionalSpread = {
   __proto__: null,
-  create: create$n,
-  meta: meta$n
+  create: create$q,
+  meta: meta$q
 };
 
-const meta$m = {
+const meta$p = {
+  docs: {
+    description: 'Disallow element helpers for creating virtual DOM nodes'
+  },
+  messages: {
+    noElementHelper: 'Create virtual DOM nodes directly as plain objects and arrays instead of using `element(...)` and flattening a nested tree.'
+  },
+  type: 'problem'
+};
+const create$p = context => {
+  return {
+    CallExpression(node) {
+      if (!isIdentifierNode(node.callee) || node.callee.name !== 'element' || !getVirtualDomElementName(node.arguments[0])) {
+        return;
+      }
+      context.report({
+        messageId: 'noElementHelper',
+        node
+      });
+    }
+  };
+};
+
+const noElementHelper = {
+  __proto__: null,
+  create: create$p,
+  meta: meta$p
+};
+
+const meta$o = {
   docs: {
     description: 'Disallow empty ARIA values in virtual-dom nodes'
   },
@@ -400,7 +429,7 @@ const meta$m = {
 const isAriaProperty = name => {
   return /^aria[A-Z]/.test(name);
 };
-const create$m = context => {
+const create$o = context => {
   return {
     ObjectExpression(node) {
       if (!isVirtualDomNode(node)) {
@@ -425,11 +454,11 @@ const create$m = context => {
 
 const noEmptyAria = {
   __proto__: null,
-  create: create$m,
-  meta: meta$m
+  create: create$o,
+  meta: meta$o
 };
 
-const meta$l = {
+const meta$n = {
   docs: {
     description: 'Disallow empty className values in virtual-dom nodes'
   },
@@ -438,7 +467,7 @@ const meta$l = {
   },
   type: 'problem'
 };
-const create$l = context => {
+const create$n = context => {
   return {
     ObjectExpression(node) {
       if (!isVirtualDomNode(node)) {
@@ -458,11 +487,11 @@ const create$l = context => {
 
 const noEmptyClassName = {
   __proto__: null,
-  create: create$l,
-  meta: meta$l
+  create: create$n,
+  meta: meta$n
 };
 
-const meta$k = {
+const meta$m = {
   docs: {
     description: 'Disallow inline event handlers in virtual-dom nodes'
   },
@@ -477,7 +506,7 @@ const isEventPropertyName = name => {
 const isInlineEventHandler = node => {
   return isStringLiteral(node) || isTemplateLiteralNode(node) || isArrowFunctionExpressionNode(node) || isFunctionExpressionNode(node);
 };
-const create$k = context => {
+const create$m = context => {
   return {
     ObjectExpression(node) {
       if (!isVirtualDomNode(node)) {
@@ -502,11 +531,11 @@ const create$k = context => {
 
 const noInlineEventHandlers = {
   __proto__: null,
-  create: create$k,
-  meta: meta$k
+  create: create$m,
+  meta: meta$m
 };
 
-const meta$j = {
+const meta$l = {
   docs: {
     description: 'Disallow inline style attributes in virtual-dom nodes'
   },
@@ -515,7 +544,7 @@ const meta$j = {
   },
   type: 'problem'
 };
-const create$j = context => {
+const create$l = context => {
   return {
     ObjectExpression(node) {
       if (!isVirtualDomNode(node)) {
@@ -535,8 +564,8 @@ const create$j = context => {
 
 const noInlineStyle = {
   __proto__: null,
-  create: create$j,
-  meta: meta$j
+  create: create$l,
+  meta: meta$l
 };
 
 const ignoredProperties$1 = new Set(['childCount', 'type']);
@@ -546,7 +575,7 @@ const isOptionalExpression = node => {
 const isNullish = node => {
   return isLiteralNode(node) && node.value === null || isIdentifierNode(node) && node.name === 'undefined' || isOptionalExpression(node);
 };
-const meta$i = {
+const meta$k = {
   docs: {
     description: 'Disallow nullish virtual-dom attribute values'
   },
@@ -555,7 +584,7 @@ const meta$i = {
   },
   type: 'problem'
 };
-const create$i = context => {
+const create$k = context => {
   return {
     ObjectExpression(node) {
       if (!isVirtualDomNode(node)) {
@@ -583,11 +612,11 @@ const create$i = context => {
 
 const noNullishAttributeValues = {
   __proto__: null,
-  create: create$i,
-  meta: meta$i
+  create: create$k,
+  meta: meta$k
 };
 
-const meta$h = {
+const meta$j = {
   docs: {
     description: 'Disallow object-like attribute values in virtual-dom nodes'
   },
@@ -600,7 +629,7 @@ const ignoredProperties = new Set(['childCount', 'type']);
 const isObjectLikeAttributeValue = node => {
   return isObjectExpressionNode(node) || isArrayExpressionNode(node) || isArrowFunctionExpressionNode(node) || isFunctionExpressionNode(node);
 };
-const create$h = context => {
+const create$j = context => {
   return {
     ObjectExpression(node) {
       if (!isVirtualDomNode(node)) {
@@ -625,11 +654,11 @@ const create$h = context => {
 
 const noObjectAttributeValues = {
   __proto__: null,
-  create: create$h,
-  meta: meta$h
+  create: create$j,
+  meta: meta$j
 };
 
-const meta$g = {
+const meta$i = {
   docs: {
     description: 'Disallow positive virtual-dom tabIndex values'
   },
@@ -638,7 +667,7 @@ const meta$g = {
   },
   type: 'problem'
 };
-const create$g = context => {
+const create$i = context => {
   return {
     ObjectExpression(node) {
       if (!isVirtualDomNode(node)) {
@@ -659,11 +688,11 @@ const create$g = context => {
 
 const noPositiveTabIndex = {
   __proto__: null,
-  create: create$g,
-  meta: meta$g
+  create: create$i,
+  meta: meta$i
 };
 
-const meta$f = {
+const meta$h = {
   docs: {
     description: 'Disallow raw string children in virtual-dom arrays'
   },
@@ -675,7 +704,7 @@ const meta$f = {
 const isVirtualDomArray = node => {
   return isArrayExpressionNode(node) && node.elements.some(element => isVirtualDomNode(element) || isTextCall(element));
 };
-const create$f = context => {
+const create$h = context => {
   return {
     ArrayExpression(node) {
       if (!isVirtualDomArray(node)) {
@@ -696,11 +725,11 @@ const create$f = context => {
 
 const noRawTextChildren = {
   __proto__: null,
-  create: create$f,
-  meta: meta$f
+  create: create$h,
+  meta: meta$h
 };
 
-const meta$e = {
+const meta$g = {
   docs: {
     description: 'Require each virtual-dom event listener to call its dedicated handler'
   },
@@ -709,7 +738,7 @@ const meta$e = {
   },
   type: 'problem'
 };
-const create$e = context => {
+const create$g = context => {
   return {
     ObjectExpression(node) {
       if (!isObjectExpressionNode(node)) {
@@ -739,11 +768,44 @@ const create$e = context => {
 
 const noSharedEventListenerHandlers = {
   __proto__: null,
-  create: create$e,
-  meta: meta$e
+  create: create$g,
+  meta: meta$g
 };
 
-const meta$d = {
+const meta$f = {
+  docs: {
+    description: 'Disallow tree helpers for creating virtual DOM nodes'
+  },
+  messages: {
+    noTreeHelper: 'Create virtual DOM nodes as plain objects and arrays instead of using `tree(...)`.'
+  },
+  type: 'problem'
+};
+const create$f = context => {
+  return {
+    CallExpression(node) {
+      if (!isIdentifierNode(node.callee) || node.callee.name !== 'tree') {
+        return;
+      }
+      const attributes = node.arguments[1];
+      if (!isObjectExpressionNode(attributes) || !hasProperty(attributes, 'className')) {
+        return;
+      }
+      context.report({
+        messageId: 'noTreeHelper',
+        node
+      });
+    }
+  };
+};
+
+const noTreeHelper = {
+  __proto__: null,
+  create: create$f,
+  meta: meta$f
+};
+
+const meta$e = {
   docs: {
     description: 'Prefer constants for single literal virtual-dom class names'
   },
@@ -752,7 +814,7 @@ const meta$d = {
   },
   type: 'suggestion'
 };
-const create$d = context => {
+const create$e = context => {
   return {
     ObjectExpression(node) {
       if (!isVirtualDomNode(node)) {
@@ -772,11 +834,11 @@ const create$d = context => {
 
 const preferClassNameConstants = {
   __proto__: null,
-  create: create$d,
-  meta: meta$d
+  create: create$e,
+  meta: meta$e
 };
 
-const meta$c = {
+const meta$d = {
   docs: {
     description: 'Prefer virtual-dom constants over raw type, role, aria boolean, and tabIndex values'
   },
@@ -792,7 +854,7 @@ const meta$c = {
 const isAriaBooleanProperty = name => {
   return /^aria[A-Z]/.test(name);
 };
-const create$c = context => {
+const create$d = context => {
   return {
     ObjectExpression(node) {
       if (!isVirtualDomNode(node)) {
@@ -847,11 +909,11 @@ const create$c = context => {
 
 const preferConstants = {
   __proto__: null,
-  create: create$c,
-  meta: meta$c
+  create: create$d,
+  meta: meta$d
 };
 
-const meta$b = {
+const meta$c = {
   docs: {
     description: 'Prefer destructuring layout properties before use'
   },
@@ -878,7 +940,7 @@ const isWriteExpression$1 = node => {
   }
   return parent.type === 'UnaryExpression' && parent.operator === 'delete' && parent.argument === node;
 };
-const create$b = context => {
+const create$c = context => {
   return {
     MemberExpression(node) {
       if (node.computed) {
@@ -906,11 +968,11 @@ const create$b = context => {
 
 const preferLayoutDestructuring = {
   __proto__: null,
-  create: create$b,
-  meta: meta$b
+  create: create$c,
+  meta: meta$c
 };
 
-const meta$a = {
+const meta$b = {
   docs: {
     description: 'Require Intl.DateTimeFormat instances to be created lazily'
   },
@@ -925,7 +987,7 @@ const isFunctionNode = node => {
 const isDateTimeFormat = node => {
   return node.type === 'MemberExpression' && !node.computed && node.object.type === 'Identifier' && node.object.name === 'Intl' && node.property.type === 'Identifier' && node.property.name === 'DateTimeFormat';
 };
-const create$a = context => {
+const create$b = context => {
   const isCreatedLazily = node => {
     return context.sourceCode.getAncestors(node).some(isFunctionNode);
   };
@@ -946,8 +1008,8 @@ const create$a = context => {
 
 const preferLazyDateTimeFormat = {
   __proto__: null,
-  create: create$a,
-  meta: meta$a
+  create: create$b,
+  meta: meta$b
 };
 
 const isClassNameKey = node => {
@@ -995,7 +1057,7 @@ const getParent = node => {
   }
   return undefined;
 };
-const meta$9 = {
+const meta$a = {
   docs: {
     description: 'Prefer mergeClassNames for composing virtual-dom className values'
   },
@@ -1004,7 +1066,7 @@ const meta$9 = {
   },
   type: 'suggestion'
 };
-const create$9 = context => {
+const create$a = context => {
   const reportNamespacedClassNameComposition = node => {
     if (!isManualClassNameConcatenation(node) || !hasClassNamesReference(node)) {
       return;
@@ -1035,11 +1097,11 @@ const create$9 = context => {
 
 const preferMergeClassNames = {
   __proto__: null,
-  create: create$9,
-  meta: meta$9
+  create: create$a,
+  meta: meta$a
 };
 
-const meta$8 = {
+const meta$9 = {
   docs: {
     description: 'Prefer destructuring state properties before use'
   },
@@ -1066,7 +1128,7 @@ const isWriteExpression = node => {
   }
   return parent.type === 'UnaryExpression' && parent.operator === 'delete' && parent.argument === node;
 };
-const create$8 = context => {
+const create$9 = context => {
   return {
     MemberExpression(node) {
       if (node.computed) {
@@ -1093,6 +1155,44 @@ const create$8 = context => {
 };
 
 const preferStateDestructuring = {
+  __proto__: null,
+  create: create$9,
+  meta: meta$9
+};
+
+const meta$8 = {
+  docs: {
+    description: 'Prefer URL.canParse over catching errors from the URL constructor'
+  },
+  messages: {
+    preferUrlCanParse: 'Use URL.canParse() to validate the URL before calling new URL().'
+  },
+  type: 'suggestion'
+};
+const isUrlConstructor = node => {
+  return node.callee.type === 'Identifier' && node.callee.name === 'URL';
+};
+const isInCaughtTryBlock = (context, node) => {
+  const ancestors = context.sourceCode.getAncestors(node);
+  return ancestors.some(ancestor => {
+    return ancestor.type === 'TryStatement' && ancestor.handler !== null && ancestors.includes(ancestor.block);
+  });
+};
+const create$8 = context => {
+  return {
+    NewExpression(node) {
+      if (!isUrlConstructor(node) || !isInCaughtTryBlock(context, node)) {
+        return;
+      }
+      context.report({
+        messageId: 'preferUrlCanParse',
+        node
+      });
+    }
+  };
+};
+
+const preferUrlCanParse = {
   __proto__: null,
   create: create$8,
   meta: meta$8
@@ -1522,6 +1622,7 @@ const plugin = {
     'clickable-div-needs-role': clickableDivNeedsRole,
     'hoist-static-nodes': hoistStaticNodes,
     'no-conditional-spread': noConditionalSpread,
+    'no-element-helper': noElementHelper,
     'no-empty-aria': noEmptyAria,
     'no-empty-class-name': noEmptyClassName,
     'no-inline-event-handlers': noInlineEventHandlers,
@@ -1531,12 +1632,14 @@ const plugin = {
     'no-positive-tab-index': noPositiveTabIndex,
     'no-raw-text-children': noRawTextChildren,
     'no-shared-event-listener-handlers': noSharedEventListenerHandlers,
+    'no-tree-helper': noTreeHelper,
     'prefer-class-name-constants': preferClassNameConstants,
     'prefer-constants': preferConstants,
     'prefer-layout-destructuring': preferLayoutDestructuring,
     'prefer-lazy-date-time-format': preferLazyDateTimeFormat,
     'prefer-merge-class-names': preferMergeClassNames,
     'prefer-state-destructuring': preferStateDestructuring,
+    'prefer-url-can-parse': preferUrlCanParse,
     'require-event-listener-options': requireEventListenerOptions,
     'require-image-alt': requireImageAlt,
     'secure-links': secureLinks,
@@ -1556,6 +1659,7 @@ const recommended = [{
     'virtual-dom/clickable-div-needs-role': 'error',
     'virtual-dom/hoist-static-nodes': 'error',
     'virtual-dom/no-conditional-spread': 'error',
+    'virtual-dom/no-element-helper': 'error',
     'virtual-dom/no-empty-aria': 'error',
     'virtual-dom/no-empty-class-name': 'error',
     'virtual-dom/no-inline-event-handlers': 'error',
@@ -1563,11 +1667,13 @@ const recommended = [{
     'virtual-dom/no-object-attribute-values': 'error',
     'virtual-dom/no-raw-text-children': 'error',
     'virtual-dom/no-shared-event-listener-handlers': 'error',
+    'virtual-dom/no-tree-helper': 'error',
     'virtual-dom/prefer-constants': 'error',
     'virtual-dom/prefer-layout-destructuring': 'error',
     'virtual-dom/prefer-lazy-date-time-format': 'error',
     'virtual-dom/prefer-merge-class-names': 'error',
     'virtual-dom/prefer-state-destructuring': 'error',
+    'virtual-dom/prefer-url-can-parse': 'error',
     'virtual-dom/require-event-listener-options': 'error',
     'virtual-dom/secure-links': 'error',
     'virtual-dom/valid-child-count': 'error'

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lvce-editor/eslint-plugin-virtual-dom",
-  "version": "18.5.0",
+  "version": "18.7.0",
   "main": "dist/index.js",
   "repository": {
     "type": "git",