Support for optional params in NodeMethod

[KishanBagaria]

const swift = require(`./binaries/${process.platform}-${process.arch}/swift.node`)

console.log(new swift.AwesomeClass().foo("a", undefined)) // throws Error: Parameter 1 should convertible to type String
console.log(new swift.AwesomeClass().foo("a", null)) // throws Error: Parameter 1 should convertible to type String

import NodeAPI

final class AwesomeClass: NodeClass {
  static let properties: NodeClassPropertyList = [
    "foo": NodeMethod(foo),
  ]

  init(_ args: NodeFunction.Arguments) throws {}

  func foo(bar: String, baz: String?) throws -> NodeValueConvertible {
    print("\(bar) \(baz)")
    return try NodeUndefined()
  }
}

@main struct AwesomeMod: NodeModule {
  let exports: NodeValueConvertible

  init() throws {
    exports = try NodeObject([
      "string": "asd",
      "AwesomeClass": AwesomeClass.constructor(),
    ])
  }
}

[KishanBagaria]

This is now fixed:

console.log(new swift.AwesomeClass().foo("a", null))

But this still throws:

console.log(new swift.AwesomeClass().foo("a", undefined)) // throws Error: Parameter 1 should convertible to type String

[kabiroberai]

Could you please confirm that a1e0935 fixes this?

[KishanBagaria]

Confirmed, fixed!