Bug: "hostname" is not exported by "__vite-browser-external", about tslog

[drewbitt]

Let's re-open this.

[terehov]

Can you try v4.8.7 please? thx

[drewbitt]

Resolved in 4.8.7.

[terehov]

Thank you. There is already a MR for that, I'll release an update in a few min.
Sorry about that.

#253

[drewbitt]

Great! I'd like an issue for the PR in the future so I don't waste time opening an issue, but that's alright :)

[terehov]

Can you try V4.8.6 please? thx

[drewbitt]

I am getting the same error in 4.8.6.
RollupError: "hostname" is not exported by "__vite-browser-external", imported by "node_modules/.pnpm/[email protected]/node_modules/tslog/dist/esm/runtime/nodejs/index.js".

Also seeing
[plugin:vite:resolve] Module "os" has been externalized for browser compatibility, imported by "C:/Users/andys-pc/GitHub/MyStuff/Someplice/node_modules/.pnpm/[email protected]/node_modules/tslog/dist/esm/runtime/nodejs/index.js". See http://vitejs.dev/guide/troubleshooting.html#module-externalized-for-browser-compatibility for more details.

There's no reason os should be externalized when it wasn't in 4.8.2

[BCaspari]

We also get this issue with 4.8.6, the problem persists...

[terehov]

I agree. What are the exact steps to reproduce it?

[drewbitt]

Reproduction

pnpm create vite-extra tslog-hostname-bug --template ssr-vanilla-ts
cd tslog-hostname-bug
pnpm install
pnpm install tslog
# Add tslog command from README to counter
echo -e "import { Logger, ILogObj } from \"tslog\";\n\nconst log: Logger<ILogObj> = new Logger();\nlog.silly(\"I am a silly log.\");\n$(cat src/counter.ts)" > src/counter.ts
pnpm run build

Errors

[terehov]

Thank you. I found the problem in the package.json I fixed it. Releasing shortly