A modern programming language that makes developers happier.

kotlinlang

Hi, I have just updated to the latest wrappers and I can't understand how to use `LoaderFunction` now. Lambdas are no longer compatible with the type. E.g. this used to work:  `loader = { scope.promise { fetchX() } }` but now it doesn't compile because `Type mismatch: inferred type is () -&gt; Promise&lt;List&lt;WebNativeServerInfo&gt;&gt; but LoaderFunction&lt;Any?&gt;? was expected`

And I do not understand how to possibly create an instance

What I see in MUI showcase is
```private val PageLoader: LoaderFunction&lt;Any?&gt; = {
    resolve(MATERIAL_SHOWCASES)
}.unsafeCast&lt;LoaderFunction&lt;Any?&gt;&gt;()```

This, honestly, feels so wrong I think it's time to switch to TS

<https://github.com/JetBrains/kotlin-wrappers/blob/e00c7692a7e68e6ccc5e077bcf1d986c9a02901b/kotlin-remix-run-router/src/jsMain/generated/remix/run/router/LoaderFunction.kt#L4|https://github.com/JetBrains/kotlin-wrappers/blob/e00c7692a7e68e6ccc5e077bcf1d986c[…]-router/src/jsMain/generated/remix/run/router/LoaderFunction.kt>

Looks like you need to return a PromiseResult and it may infer the lambda to a LoaderFunction.

It doesn't even if I create a dummy lambda that returns PromiseResult

But even if it did, it's still wrong, as it should handle a promise

Yeah from <https://remix.run/docs/en/main/route/loader#loader|this> it looks like it should be returning a Promise, not sure why they changed it. In the meantime you can cast your loader function returning a Promise to dynamic. Prob a good idea to open a youtrack ticket also.

<@U05LUM3Q85N> modern Promise was introduced <https://github.com/JetBrains/kotlin-wrappers/blob/master/CHANGELOG.md#pre580|here>.

To fix problem, which you described we added:
1. Missed `promise` extension for `CoroutineScope` - it returns required <https://github.com/JetBrains/kotlin-wrappers/commit/c21f589a7b6ec4b854c2be465d9887f1cbe58d79|Promise>
2. <https://github.com/JetBrains/kotlin-wrappers/commit/dd1c0d008f4072f94ce85a9d62fdd66e4578f7f4|Adapter> for `LoaderFunction` 
Changes will be available in next release `pre.732` (probably today)

> Prob a good idea to open a youtrack ticket also.
It's definitely good idea to <https://github.com/JetBrains/kotlin-wrappers/issues|report issue> if you have problems like this.
If you can't solve problem without `asDynamic` , `unsafeCast`  - please report issue.

I updated and now the `LoaderFunction` adapter is not there anymore and I get the error `Interface LoaderFunction does not have constructors`

`LoaderFunction` factory <https://github.com/JetBrains/kotlin-wrappers/blob/3ece24442774a3f50cdd442e5ef5823368162e00/kotlin-remix-run-router/src/jsMain/generated/remix/run/router/LoaderFunction.ext.kt#L7|is here>.
It's signature was changed by authors.
Probably you missed second parameter

Thank you for your quick reply.
I can't really figure it out, for brevity a short loader that had some use before, but is now unused (other Loader have the same issue though):
```val deviceLoader = LoaderFunction<Any?> { args, context ->
    PromiseResult(Unit)
}```
Following the `LoaderFunction` reference leads me to:
```typealias LoaderFunction<Context> = (args: LoaderFunctionArgs<Context>) -> Any? /* Promise<DataFunctionValue> | DataFunctionValue */```

Screenshot 2024-06-04 at 11.43.58.png

Looks like outdated indices :disappointed:

I think we already used that factory, because we updated from pre690 to pre754

Oh, so invalidating caches and gradle clean should fix it?

"Refresh" in Gradle toolbox will do the job

Screenshot 2024-06-04 at 11.46.31.png

What a silly error. Yes it just didn't occur to me to do a gradle refresh.
I just clean and build it.