Decorator for onReject function for async methods about proposal-decorators

[littledan]

I think you can implement the @catch decorator in the current proposal by using a decorator which replaces the value of the element's property descriptor with another function which does the catching. You could also use try/catch in your async function, of course. For bound methods, this repository contains source for an @bound decorator, or you could use a field declaration with an arrow function.

[ljharb]

Could your "catch" decorator not return function (...args) { return original.call(this, ...args).catch(this.errorHandler); }, without any need for ::?

iow, it'd be like:

class {
  @catch(this.errorHandler)
  async handler(e) {
    …
  }

  errorHandler(e) {
    this.e = e;
    console.error(e);
  }

[ljharb]

ah, after updating my comment to include a code sample, you're asking primarily about the this.errorHandler - and it's a bit awkward to do @catch(function (...args) { return this.errorHandler(...args); }).

[haraldrudell]

If you’re an async function or a promise and your invoker is not, ie. the invocation does not have await, then you are often going to need an onReject function

The onReject functions are typically bound or static, and a class method or a function value provided to the constructor

error handler functions are typically shared by multiple or even all async functions and promises

[littledan]

Please reopen this issue if you have any concerns with this response. As far as I can tell, your requirements are met by the current proposal.