🐛 BUG: fsonathan... Vite error when using fs in external files

[aFuzzyBear]

What version of astro are you using?

latest

What package manager are you using?

yarn

What operating system are you using?

wsl2; ubuntu 20.04, reproduced on stackblitz

Describe the Bug

Hi team,

I would like to report an issue raised in the support thread of Discord: https://discord.com/channels/830184174198718474/931342087725645834

Here I reported a vite error where when using nodes fs package in external *.js files, Vite throws an error, saying it cannot resolve dependencies,

This occurs with the following import statements

import fs from 'fs'
import fs from 'fs/promises'
import fs from 'node:fs'
import fs from 'node:fs/promises'

The present work around is to disable the import statement when launching the dev server and enabling them shortly after launch, it does throw in build also prohibiting the final build since Its one wee file using node.fs.writeFile()

Thanks to Muhymin for producing a reproduction,

Thank you guys for looking into this,

Link to Minimal Reproducible Example

https://stackblitz.com/edit/github-xs2jbp?file=src%2Fpages%2Findex.astro&on=stackblitz

[jonathantneal]

Verified. If this is an intended limitation from Vite or Astro, we should document this.

[zadeviggers]

I ran into this while trying to do some stuff with @astropub/codecs, if that's useful to anyone working on this.

[FredKSchott]

I also just ran into this, in a file that was only ever run on the server and never meant for the client bundle. As @aFuzzyBear said, the current solution is to remove the import on dev server start and then add back after start, which feels weiiiiiiiird.

[FredKSchott]

Ah, I believe that this is a Vite bug. fixed in vitejs/vite@4a05a6e but not yet in Astro.

[zadeviggers]

Any ideas on when the fix might land in Astro?

[aFuzzyBear]

Thanks Fred, for a while there I thought I was losing the plot there 😅

[FredKSchott]

I had the same problem! @zadeviggers asap, you can track progress in #2424

[tony-sull]

Fixed with the with the Vite upgrade in #2424 🥳

[aFuzzyBear]

@tony-sull legend thank you...