Fix next in types of transformer signature

wooorm · web-flow · commit 350ed9dfe109 · 2021-07-06T22:24:14.000+02:00
Closes GH-149. Reviewed-by: Christian Murphy <christian.murphy.42@gmail.com>
diff --git a/types/index.d.ts b/types/index.d.ts
@@ -313,11 +313,7 @@ export type Attacher<S extends any[] = [Settings?], P = Settings> = (
 export type Transformer = (
   node: Node,
   file: VFile,
-  next: (
-    error: Error | null,
-    tree?: Node,
-    file?: VFile
-  ) => Record<string, unknown>
+  next: (error?: Error | null, tree?: Node, file?: VFile) => void
 ) => Error | Node | Promise<Node> | void | Promise<void>
 
 /**
diff --git a/types/unified-tests.ts b/types/unified-tests.ts
@@ -165,24 +165,27 @@ processor.use(
     (
       tree: Node,
       file: VFile,
-      next: (error: Error | null, tree: Node, file: VFile) => {}
+      next: (error: Error | null, tree: Node, file: VFile) => void
     ) => {}
 )
 processor.use(
   () =>
     (
       tree: Node,
       file: VFile,
-      next: (error: Error | null, tree: Node) => {}
+      next: (error: Error | null, tree: Node) => void
     ) => {}
 )
 processor.use(
-  () => (tree: Node, file: VFile, next: (error: Error | null) => {}) => {}
+  () => (tree: Node, file: VFile, next: (error: Error | null) => void) => {}
 )
 processor.use(
   // $ExpectError
   () => (tree: Node, file: VFile, next: () => {}) => {}
 )
+processor.use(() => (tree: Node, file: VFile, next: () => void) => {
+  next()
+})
 
 processor.use(pluginWithTwoSettings)
 processor.use(pluginWithTwoSettings).use(pluginWithTwoSettings)

Original file line number	Diff line number	Diff line change
`@@ -165,24 +165,27 @@ processor.use(`
`165`	`165`	`(`
`166`	`166`	`tree: Node,`
`167`	`167`	`file: VFile,`
`168`		`- next: (error: Error \| null, tree: Node, file: VFile) => {}`
	`168`	`+ next: (error: Error \| null, tree: Node, file: VFile) => void`
`169`	`169`	`) => {}`
`170`	`170`	`)`
`171`	`171`	`processor.use(`
`172`	`172`	`() =>`
`173`	`173`	`(`
`174`	`174`	`tree: Node,`
`175`	`175`	`file: VFile,`
`176`		`- next: (error: Error \| null, tree: Node) => {}`
	`176`	`+ next: (error: Error \| null, tree: Node) => void`
`177`	`177`	`) => {}`
`178`	`178`	`)`
`179`	`179`	`processor.use(`
`180`		`- () => (tree: Node, file: VFile, next: (error: Error \| null) => {}) => {}`
	`180`	`+ () => (tree: Node, file: VFile, next: (error: Error \| null) => void) => {}`
`181`	`181`	`)`
`182`	`182`	`processor.use(`
`183`	`183`	`// $ExpectError`
`184`	`184`	`() => (tree: Node, file: VFile, next: () => {}) => {}`
`185`	`185`	`)`
	`186`	`+processor.use(() => (tree: Node, file: VFile, next: () => void) => {`
	`187`	`+ next()`
	`188`	`+})`
`186`	`189`
`187`	`190`	`processor.use(pluginWithTwoSettings)`
`188`	`191`	`processor.use(pluginWithTwoSettings).use(pluginWithTwoSettings)`