Uncurried function in record converted incorectly from *.re syntax

[JasoonS]

I tried to make a smaller reproduction of this error, but it didn't work. So apologies for that.

Steps to reproduce:

git clone git@github.com:JasoonS/ui.git
git checkout d8c29c4
yarn
./node_modules/.bin/bsc -format "./src/harberger-lib/Web3.re" -> "./src/harberger-lib/Web3.res"
rm -f ./src/harberger-lib/Web3.re

You get the code on the left after this:

Which needs to be corrected to the code in green on the right (here is the original reason code: https://github.com/JasoonS/ui/blob/d8c29c48704ccf32e492cfab1fd0f2b77738ef83/src/harberger-lib/RootProvider.re#L229)

And here is the corresponding error:

(this is using the latest bs-platform - 8.4.2)

[IwanKaramazow]

Thank you very much for the detailed report and code examples. I managed to shrink it to:

let _ =
  library.getBalance(. account)
  ->Promise.Js.catch(_ => Promise.resolved(None))

It seems that the conversion is hitting a strange path here. I'll look into this tomorrow.
You can manually add the . right now to the ReScript code. Everything should be fine then.

[JasoonS]

Thanks, and keep up the great work :)

[IwanKaramazow]

Hi @JasoonS.

Fixed the problem in master. You can manually add the . back in the ReScript version now. This way you can already get the advantages and goodies of the latest syntax, without having to wait for a compiler release.

If you have any questions or problems, don't hesitate to reach out :)
Happy to help!