Anyone know if there's a bug for this? I'm still s...

# javascript

[mike.holler]

Anyone know if there's a bug for this? I'm still seeing promises as non-exportable in 1.6.21.

[hfhbd]

It is exportable, but until 1.7.0 you need id("io.github.turansky.kfc.definitions") version "5.12.0" plugin: https://github.com/turansky/kfc-plugins
https://youtrack.jetbrains.com/issue/KT-51205/K-JS-IR-external-class-is-mapped-to-any

[mike.holler]

Ohhhh thank you

[mike.holler]

@hfhbd do I need to configure the plugin or just apply it?

[hfhbd]

Applying is enough 🙂

[mike.holler]

Very nice. Does it add any code to the bundle?

[hfhbd]

Nope, in fact, it is only a regex removing kotlin.js prefix from your d.ts file

[hfhbd]

https://github.com/turansky/kfc-plugins/blob/master/kfc/kfc-gradle-plugin/src/main/kotlin/io/github/turansky/kfc/gradle/plugin/DefinitionsPlugin.kt
Thats all, no magic 😄

[mike.holler]

Cool

[mike.holler]

Messy little hack though

[mike.holler]

@hfhbd compiler still produces a warning after applying the plugin -- is this expected?

[hfhbd]

What warning? JsExport is still experimental

[mike.holler]

Exported declaration uses non-exportable return type: Promise<InitechCloudResponse<OrganizationAccount>>
Both OrganizationAccount and InitechCloudResponse are marked with @JSExport

[mike.holler]

public fun createAccount(
        organizationIdOrAlias: String,
        body: CreateOrganizationAccount
    ): Promise<InitechCloudResponse<OrganizationAccount>> {
        return scope.promise {
            initechCloudClientDelegate
                .createAccount(organizationIdOrAlias, body)
        }
    }

Here is the function in question

[hfhbd]

It's JavaScript... You could ignore the error 😄 But are you sure, both classes uses @JsExport and are exportable? If not, the types are mapped to any. Just look into your d.ts file

[mike.holler]

Yeah I'd ignore the error but excessive warnings like this make it easy to overlook other warnings that actually matter

[mike.holler]

Both classes definitely have @JsExport on them but I'll try the other thing.