fix: "chunks" directory appears in build output, if custom modules are imported in Astro files

[rishi-raj-jain]

fixes #4435

Attempts to maintain a list of directories from which the server side assets are found, and then after removing the files, loop over to remove only empty directories.

[changeset-bot]

🦋 Changeset detected

Latest commit: 3f75956

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[bluwy]

Thanks for tackling this! This looks good to me. I think a simple directory check is enough to capture most cases.

[rishi-raj-jain]

I think a simple directory check is enough to capture most cases.

Yeah, I was thinking of a recursive check, but can implement that if a case like that comes up?

[rishi-raj-jain]

Found https://gist.github.com/fixpunkt/fe32afe14fbab99d9feb4e8da7268445, which can be called on all the directories created from the .mjs files.

[bluwy]

Yeah I think this PR is good for now, you can implement a recursive one if you're interested too, maybe we should avoid the glob too and manually iterate through the directories to avoid too many fs calls. But I'll merge this one first. Thanks for your help around Astro lately!