Refactor code-style

Author: wooorm

lib/index.js

@@ -1,26 +1,17 @@
 /**
- * @typedef {import('mdast').Content} Content
  * @typedef {import('mdast').ListItem} ListItem
  * @typedef {import('mdast').Paragraph} Paragraph
- * @typedef {import('mdast').Parent} Parent
- * @typedef {import('mdast').Root} Root
  * @typedef {import('mdast-util-from-markdown').CompileContext} CompileContext
  * @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 {Extract<Root | Content, Parent>} Parents
- */
-
+import {ok as assert} from 'devlop'
 import {defaultHandlers} from 'mdast-util-to-markdown'
 
-// To do: next major: rename `context` -> `state`, `safeOptions` -> `info`, use
-// `track` from `state`.
 // To do: next major: replace exports with functions.
-// To do: next major: use `defaulthandlers.listItem`.
 
 /**
  * Extension for `mdast-util-from-markdown` to enable GFM task list items.
@@ -50,8 +41,9 @@ export const gfmTaskListItemToMarkdown = {
  * @type {FromMarkdownHandle}
  */
 function exitCheck(token) {
-  const node = /** @type {ListItem} */ (this.stack[this.stack.length - 2])
   // We’re always in a paragraph, in a list item.
+  const node = this.stack[this.stack.length - 2]
+  assert(node.type === 'listItem')
   node.checked = token.type === 'taskListCheckValueChecked'
 }
 
@@ -60,14 +52,15 @@ function exitCheck(token) {
  * @type {FromMarkdownHandle}
  */
 function exitParagraphWithTaskListItem(token) {
-  const parent = /** @type {Parents} */ (this.stack[this.stack.length - 2])
+  const parent = this.stack[this.stack.length - 2]
 
   if (
     parent &&
     parent.type === 'listItem' &&
     typeof parent.checked === 'boolean'
   ) {
-    const node = /** @type {Paragraph} */ (this.stack[this.stack.length - 1])
+    const node = this.stack[this.stack.length - 1]
+    assert(node.type === 'paragraph')
     const head = node.children[0]
 
     if (head && head.type === 'text') {
@@ -110,19 +103,19 @@ function exitParagraphWithTaskListItem(token) {
  * @type {ToMarkdownHandle}
  * @param {ListItem} node
  */
-function listItemWithTaskListItem(node, parent, context, safeOptions) {
+function listItemWithTaskListItem(node, parent, state, info) {
   const head = node.children[0]
   const checkable =
     typeof node.checked === 'boolean' && head && head.type === 'paragraph'
   const checkbox = '[' + (node.checked ? 'x' : ' ') + '] '
-  const tracker = context.createTracker(safeOptions)
+  const tracker = state.createTracker(info)
 
   if (checkable) {
     tracker.move(checkbox)
   }
 
-  let value = defaultHandlers.listItem(node, parent, context, {
-    ...safeOptions,
+  let value = defaultHandlers.listItem(node, parent, state, {
+    ...info,
     ...tracker.current()
   })
 

package.json

@@ -40,6 +40,7 @@
   ],
   "dependencies": {
     "@types/mdast": "^4.0.0",
+    "devlop": "^1.0.0",
     "mdast-util-from-markdown": "^2.0.0",
     "mdast-util-to-markdown": "^2.0.0"
   },

test.js

@@ -1,18 +1,17 @@
 import assert from 'node:assert/strict'
 import test from 'node:test'
+import {gfmTaskListItem} from 'micromark-extension-gfm-task-list-item'
 import {fromMarkdown} from 'mdast-util-from-markdown'
 import {toMarkdown} from 'mdast-util-to-markdown'
 import {removePosition} from 'unist-util-remove-position'
-import {gfmTaskListItem} from 'micromark-extension-gfm-task-list-item'
 import {
   gfmTaskListItemFromMarkdown,
   gfmTaskListItemToMarkdown
 } from './index.js'
-import * as mod from './index.js'
 
 test('core', async function (t) {
   await t.test('should expose the public api', async function () {
-    assert.deepEqual(Object.keys(mod).sort(), [
+    assert.deepEqual(Object.keys(await import('./index.js')).sort(), [
       'gfmTaskListItemFromMarkdown',
       'gfmTaskListItemToMarkdown'
     ])