tryCatch should work with nAry arguments

Currently, tryCatch always returns unary functions even if the input passed to it is nAry.

We can change this behaviour with the proposed implementation, if this doesn’t cause any breaking problems.

const tryCatch =  function tryCatch(fn) {
  if(!isFunction(fn)) {
    throw new TypeError('tryCatch: Function required for first argument')
  }

  const safe = function() {
    try { return Ok(fn.apply(this, arguments)) }
    catch(e) { return Err(e) }
  }

  Object.defineProperty(safe, "length", { value: fn.length })
  
  return safe
}


const add = tryCatch((a, b) => {
  return a + b
})

add(1, 2) => 3
curry(add)(1)(2) => 3

[karthikiyengar]

I don’t see problems with the defineProperty itself, but the spread operator is going to destroy context. Edited to use the old-school way.

[evilsoft]

Shipped with 0.11.0