feat: support args for input

[mbtools]

Some input processes require additional arguments. For example, password inputs depend on the { silent: true }. This change passes these arguments to the related input processes while keeping compatibility with existing API (i.e. callback for input.start).

cc: @wraithgar, I hope this is what you had in mind re: spinner fix.

References

Related to npm/cli#8322

[wraithgar]

I actually have a local branch w/ changes for this PR and was picking it up this week. The differences are minor and I'll add comments here.

[wraithgar]

Here's my diff:

-    start: function (fn) {
-      process.emit('input', 'start')
+    start: function (fn, ...args) {
+      let endArgs = args
       function end () {
-        return process.emit('input', 'end')
+        return process.emit('input', 'end', endArgs)
       }
       if (typeof fn === 'function') {
+        process.emit('input', 'start', ...args)
         const res = fn()
         if (res && res.finally) {
           return res.finally(end)
         }
-        end()
+        end(...args)
         return res
       }
+      process.emit('input', 'start', fn, ...args)
       return end
     },
-    end: function () {
-      return process.emit('input', 'end')
+    end: function (...args) {
+      return process.emit('input', 'end', ...args)
     },

[wraithgar]

TLDR; this looks fine, please don't muck about with the fn parameter like my suggestion started with.

[mbtools]

I was going for consistency with the other methods. Good to see that it was close. I will rework the spinner PR once this lands.