Refactor code-style

Author: wooorm

index.d.ts

@@ -10,7 +10,6 @@ export {
 /**
  * MDX expression node, occurring in flow (block).
  */
-// eslint-disable-next-line @typescript-eslint/consistent-type-definitions
 export interface MdxFlowExpression extends MdastLiteral {
   /**
    * Node type.
@@ -24,15 +23,13 @@ export interface MdxFlowExpression extends MdastLiteral {
     /**
      * Program node from estree.
      */
-    // eslint-disable-next-line @typescript-eslint/ban-types
     estree?: Program | null | undefined
   } & MdastLiteral['data']
 }
 
 /**
  * MDX expression node, occurring in text (phrasing).
  */
-// eslint-disable-next-line @typescript-eslint/consistent-type-definitions
 export interface MdxTextExpression extends MdastLiteral {
   /**
    * Node type.
@@ -46,27 +43,25 @@ export interface MdxTextExpression extends MdastLiteral {
     /**
      * Program node from estree.
      */
-    // eslint-disable-next-line @typescript-eslint/ban-types
     estree?: Program | null | undefined
   } & MdastLiteral['data']
 }
 
+// To do: next major: remove.
 /**
  * Deprecated: use `MdxFlowExpression`.
  */
-// eslint-disable-next-line @typescript-eslint/naming-convention
 export type MDXFlowExpression = MdxFlowExpression
 
+// To do: next major: remove.
 /**
  * Deprecated: use `MdxTextExpression`.
  */
-// eslint-disable-next-line @typescript-eslint/naming-convention
 export type MDXTextExpression = MdxTextExpression
 
 /**
  * MDX expression node, occurring in flow (block), for hast.
  */
-// eslint-disable-next-line @typescript-eslint/consistent-type-definitions
 export interface MdxFlowExpressionHast extends HastLiteral {
   /**
    * Node type.
@@ -80,15 +75,13 @@ export interface MdxFlowExpressionHast extends HastLiteral {
     /**
      * Program node from estree.
      */
-    // eslint-disable-next-line @typescript-eslint/ban-types
     estree?: Program | null | undefined
   } & HastLiteral['data']
 }
 
 /**
  * MDX expression node, occurring in text (phrasing), for hast.
  */
-// eslint-disable-next-line @typescript-eslint/consistent-type-definitions
 export interface MdxTextExpressionHast extends HastLiteral {
   /**
    * Node type.
@@ -102,14 +95,12 @@ export interface MdxTextExpressionHast extends HastLiteral {
     /**
      * Program node from estree.
      */
-    // eslint-disable-next-line @typescript-eslint/ban-types
     estree?: Program | null | undefined
   } & HastLiteral['data']
 }
 
 // Add nodes to mdast content.
 declare module 'mdast' {
-  // eslint-disable-next-line @typescript-eslint/consistent-type-definitions
   interface RootContentMap {
     /**
      * MDX expression node, occurring in text (phrasing).
@@ -121,15 +112,13 @@ declare module 'mdast' {
     mdxFlowExpression: MdxFlowExpression
   }
 
-  // eslint-disable-next-line @typescript-eslint/consistent-type-definitions
   interface PhrasingContentMap {
     /**
      * MDX expression node, occurring in text (phrasing).
      */
     mdxTextExpression: MdxTextExpression
   }
 
-  // eslint-disable-next-line @typescript-eslint/consistent-type-definitions
   interface BlockContentMap {
     /**
      * MDX expression node, occurring in flow (block).
@@ -140,7 +129,6 @@ declare module 'mdast' {
 
 // Add nodes to hast content.
 declare module 'hast' {
-  // eslint-disable-next-line @typescript-eslint/consistent-type-definitions
   interface RootContentMap {
     /**
      * MDX expression node, occurring in flow (block).
@@ -152,7 +140,6 @@ declare module 'hast' {
     mdxTextExpression: MdxTextExpressionHast
   }
 
-  // eslint-disable-next-line @typescript-eslint/consistent-type-definitions
   interface ElementContentMap {
     /**
      * MDX expression node, occurring in flow (block).

lib/index.js

@@ -3,13 +3,15 @@
  * @typedef {import('mdast-util-from-markdown').Extension} FromMarkdownExtension
  * @typedef {import('mdast-util-from-markdown').Handle} FromMarkdownHandle
  *
- * @typedef {import('mdast-util-to-markdown').Options} ToMarkdownExtension
  * @typedef {import('mdast-util-to-markdown').Handle} ToMarkdownHandle
+ * @typedef {import('mdast-util-to-markdown').Options} ToMarkdownExtension
  *
  * @typedef {import('../index.js').MdxFlowExpression} MdxFlowExpression
  * @typedef {import('../index.js').MdxTextExpression} MdxTextExpression
  */
 
+import {ok as assert} from 'devlop'
+
 /**
  * Extension for `mdast-util-from-markdown` to enable MDX expressions.
  *
@@ -72,9 +74,8 @@ function enterMdxTextExpression(token) {
 function exitMdxExpression(token) {
   const value = this.resume()
   const estree = token.estree
-  const node = /** @type {MdxFlowExpression | MdxTextExpression} */ (
-    this.stack[this.stack.length - 1]
-  )
+  const node = this.stack[this.stack.length - 1]
+  assert(node.type === 'mdxFlowExpression' || node.type === 'mdxTextExpression')
   this.exit(token)
   node.value = value
 

package.json

@@ -40,6 +40,7 @@
     "@types/estree-jsx": "^1.0.0",
     "@types/hast": "^3.0.0",
     "@types/mdast": "^4.0.0",
+    "devlop": "^1.0.0",
     "mdast-util-from-markdown": "^2.0.0",
     "mdast-util-to-markdown": "^2.0.0"
   },
@@ -85,6 +86,18 @@
     "strict": true
   },
   "xo": {
+    "overrides": [
+      {
+        "files": [
+          "**/*.ts"
+        ],
+        "rules": {
+          "@typescript-eslint/ban-types": "off",
+          "@typescript-eslint/consistent-type-definitions": "off",
+          "@typescript-eslint/naming-convention": "off"
+        }
+      }
+    ],
     "prettier": true
   }
 }

readme.md

@@ -372,7 +372,7 @@ import {visit} from 'unist-util-visit'
 /** @type {import('mdast').Root} */
 const tree = getMdastNodeSomeHow()
 
-visit(tree, (node) => {
+visit(tree, function (node) {
   // `node` can now be an expression node.
 })
 ```