Refactor some more code to use JSDoc

Author: wooorm

.gitignore

@@ -5,4 +5,3 @@ node_modules/
 *.log
 yarn.lock
 !/index.d.ts
-!/lib/callable-instance.d.ts

lib/callable-instance.d.ts

@@ -1,30 +1,39 @@
-/**
- * @param {string} property
- */
-export function CallableInstance(property) {
-  /** @type {Function} */
-  const self = this
-  const constr = self.constructor
-  // Prototypes do exist.
-  // type-coverage:ignore-next-line
-  const proto = /** @type {Record<string, Function>} */ (constr.prototype)
-  const func = proto[property]
-  const apply = function () {
-    return func.apply(apply, arguments)
-  }
+export const CallableInstance =
+  /**
+   * @type {new <Parameters extends Array<unknown>, Result>(property: string | symbol) => (...parameters: Parameters) => Result}
+   */
+  (
+    /** @type {unknown} */
+    (
+      /**
+       * @this {Function}
+       * @param {string | symbol} property
+       * @returns {(...parameters: Array<unknown>) => unknown}
+       */
+      function (property) {
+        const self = this
+        const constr = self.constructor
+        const proto = /** @type {Record<string | symbol, Function>} */ (
+          // Prototypes do exist.
+          // type-coverage:ignore-next-line
+          constr.prototype
+        )
+        const func = proto[property]
+        /** @type {(...parameters: Array<unknown>) => unknown} */
+        const apply = function () {
+          return func.apply(apply, arguments)
+        }
 
-  Object.setPrototypeOf(apply, proto)
+        Object.setPrototypeOf(apply, proto)
 
-  const names = Object.getOwnPropertyNames(func)
+        const names = Object.getOwnPropertyNames(func)
 
-  for (const p of names) {
-    const descriptor = Object.getOwnPropertyDescriptor(func, p)
-    if (descriptor) Object.defineProperty(apply, p, descriptor)
-  }
+        for (const p of names) {
+          const descriptor = Object.getOwnPropertyDescriptor(func, p)
+          if (descriptor) Object.defineProperty(apply, p, descriptor)
+        }
 
-  return apply
-}
-
-// Prototypes do exist.
-// type-coverage:ignore-next-line
-CallableInstance.prototype = Object.create(Function.prototype)
+        return apply
+      }
+    )
+  )

lib/callable-instance.js

@@ -13,19 +13,25 @@ try {
 const result = file
   .replace(/declare const Processor_base: [^\n]+/, function () {
     console.log('Fixed `CallableInstance` import')
-    return "declare const CallableInstance: import('./callable-instance.js').ICallableInstance"
+    return "import {CallableInstance} from './callable-instance.js'"
   })
   .replace(/extends Processor_base/, function () {
     console.log('Fixed `CallableInstance` use')
     return 'extends CallableInstance<[], Processor<ParseTree, HeadTree, TailTree, CompileTree, CompileResult>>'
   })
   .replace(
-    /\.\.\.parameters: Parameters_1 \| \[boolean] \| undefined/,
-    function () {
+    /\.\.\.parameters: (Parameters_\d) \| \[boolean] \| undefined/,
+    /**
+     *
+     * @param {string} $0
+     * @param {string} $1
+     * @returns {string}
+     */
+    function ($0, $1) {
       console.log(
         'Fixed `use` overload with plugin, and *non-optional* parameters'
       )
-      return '...parameters: Parameters_1 | [boolean]'
+      return '...parameters: ' + $1 + ' | [boolean]'
     }
   )
 

script/fix-types.js

@@ -11,5 +11,5 @@
     "target": "es2020"
   },
   "exclude": ["coverage/", "node_modules/"],
-  "include": ["**/*.js", "lib/callable-instance.d.ts", "index.d.ts"]
+  "include": ["**/*.js", "index.d.ts"]
 }