Typescript Typings To Use Conditional Logic about await-to-js HOT 9 CLOSED

@lucaspoffo, @bezreyhan

How about this #46?

from await-to-js.

use res!.data.everything
to get safe deeply access object, read more detail here https://codewithstyle.info/Deep-property-access-in-TypeScript/

from await-to-js.

@lucaspoffo I have not tried this myself but it looks like conditional types with arrays should be possible: https://www.typescriptlang.org/docs/handbook/release-notes/typescript-2-8.html#example-2

from await-to-js.

async function testA() {
  const result = await to(successAsync())
  if (result[0]) {
    console.log('Found error in successAsync')
    return // don't forget
  }
  console.log(result[1].value) // No type error
}

async function testB() {
  const [err, result] = await to(successAsync())
  if (err) {
    console.log('Found error in successAsync')
    return
  }
  console.log(result.value) // Type error
}

so is there any why to separate into two variables and show no err? the res[0] and res[1] is not quite clean for each time to evoke this function.

from await-to-js.

@Rizaldinurm using ! tells TS to ignore the type so when you use ! you lose type safety.

The ideal situation is that TS makes sure you have checked that error is undefined (or that res is defined) before using res.

from await-to-js.

@bezreyhan When the return type is an Array you cannot infer the values between each other,
but if the return type is an Object you can do it, like this:

async function to<T, U = Error>(promise: Promise<T>): Promise<{ error: U; data: undefined } | { error: null; data: T }> {
  return promise.then((data: T) => ({ error: null, data }))
    .catch((error: U) => ({ error, data: undefined }))
}


async function successAsync(): Promise<{ value: number }> { return { value: 5 } }

async function test() {
  const result = await to(successAsync())
  if (result.error) {
    console.log('Found error in successAsync')
    return // don't forget
  }
  console.log(result.data.value) // No type error
}

test()

from await-to-js.

@bezreyhan You're right, arrays works too, but you can't separate in 2 variables:

async function to<T, U = Error>(promise: Promise<T>): Promise<[U, undefined] | [null, T]> {
  return promise.then<[null, T]>((data) => ([null, data]))
    .catch<[U, undefined]>((error: U) => [error, undefined])
}

async function successAsync(): Promise<{ value: number }> { return { value: 5 } }

async function testA() {
  const result = await to(successAsync())
  if (result[0]) {
    console.log('Found error in successAsync')
    return // don't forget
  }
  console.log(result[1].value) // No type error
}

async function testB() {
  const [err, result] = await to(successAsync())
  if (err) {
    console.log('Found error in successAsync')
    return
  }
  console.log(result.value) // Type error
}

from await-to-js.

@JFGHT I haven't tested it but that look good to me!

from await-to-js.

I think this can now be closed because v3.0.0 with the fixes in #46 was released yesterday.

from await-to-js.