Hi everyone, I’m trying to use okio for kotlinJS b...

# javascript

[François]

Hi everyone, I’m trying to use okio for kotlinJS browser, and it doesn’t work, did I miss something?
okio: 3.9.1
kotlin: 2.1.0

Module not found: Error: Can't resolve 'fs' in '.../build/js/packages/ktorfilecaching-test/kotlin'
Module not found: Error: Can't resolve 'path' in '.../build/js/packages/ktorfilecaching-test/kotlin'

BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.

If you want to include a polyfill, you need to:
	- add a fallback 'resolve.fallback: { "path": require.resolve("path-browserify") }'
	- install 'path-browserify'
If you don't want to include a polyfill, you can use an empty module like this:
	resolve.fallback: { "path": false }
Module not found: Error: Can't resolve 'os' in '/Users/francoisdabonot/DEV/KtorKMPFileCaching/build/js/packages/ktorfilecaching-test/kotlin'

BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.

If you want to include a polyfill, you need to:
	- add a fallback 'resolve.fallback: { "os": require.resolve("os-browserify/browser") }'
	- install 'os-browserify'
If you don't want to include a polyfill, you can use an empty module like this:
	resolve.fallback: { "os": false }

[Chrimaeon]

it does not work on browsers - there is no files api on browsers. it only work on node.js
https://square.github.io/okio/multiplatform/#file-system

[François]

Oh, they didn’t mention it https://square.github.io/okio/file_system/#kotlinjs

[François]

So, how to separate Browser and NodeJS implementation?

[Chrimaeon]

create separate modules, on for browser and one for node.

[François]

I see, the hard way 😄
An issue discussing the problem but no solution 😞 https://youtrack.jetbrains.com/issue/KT-47038/KJS-MPP-Split-JS-target-into-JsBrowser-and-JsNode

[Edoardo Luppi]

@François if you haven't already, upvote it. Sooner or later we might get it.

[François]

Yes, I did it 😄

[François]

https://youtrack.jetbrains.com/issue/KT-47038/KJS-MPP-Split-JS-target-into-JsBrowser-and-JsNode#focus=Comments-27-7501103.0-0
I kind of working on my project but have weird effect

[Edoardo Luppi]

Can't use that anymore.
See https://youtrack.jetbrains.com/issue/KT-59316

[François]

Yeah , I got that message

w: Kotlin Target 'js()' is already declared.

Declaring multiple Kotlin Targets of the same type is not supported.

But it’s still working on kotlin 2.1.0

[François]

I’m building a library, so I guess it just works 😄

[François]

My project https://github.com/frankois944/KtorKMPFileCaching

[Edoardo Luppi]

https://github.com/JetBrains/kotlin/blob/7b6219e875ee553b9860afc1e6d07b097dd59841/[…]ugin/diagnostics/checkers/KotlinTargetAlreadyDeclaredChecker.kt

For JS it's still a warning temporarily.

[Edoardo Luppi]

Which references my comment
https://youtrack.jetbrains.com/issue/KT-59316/Deprecate-multiple-same-targets#focus=Comments-27-9992405.0-0

Might mean you're safe to use it for now.

[François]

Right, I guess you’re looking for the same solution as me, longer time than me.
I’m sticking with my current solution until there is a real one.

[Edoardo Luppi]

@hfhbd just FYI, it's still a warning, and not an error as we had discussed. See comments above.

[Edoardo Luppi]

I’m sticking with my current solution until there is a real one

I will probably do the same once I get to the point of targeting the browser.

[Edoardo Luppi]

I have been extremely careful in importing libraries and using externals, for the sake of not having to go crazy at that point.

[Edoardo Luppi]

For example, I'm now using Node.js externals without @JsModule, as that creates an AOT import of the module, and immediately breaks on browsers.

[François]

My project is not so complex, okio + localstorage, that’s all and a lot of copy/paste.