fix: pass custom entry filename when resolving prerenderer

[brandonroberts]

🔗 Linked issue

Closes #2444

❓ Type of change

• 📖 Documentation (updates to the documentation, readme, or JSdoc annotations)
• 🐞 Bug fix (a non-breaking change that fixes an issue)
• 👌 Enhancement (improving an existing functionality like performance)
• ✨ New feature (a non-breaking change that adds functionality)
• 🧹 Chore (updates to the build process or auxiliary tools and libraries)
• ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

This allows the rollupConfig.options.output.entryFileNames to be passed to the nitro prerenderer if its a string, with the fallback to index.mjs. This fixes an issue where prerendering fails if a custom entry filename is used.

📝 Checklist

• I have linked an issue or discussion.
• I have updated the documentation accordingly.

[brandonroberts]

I'll check further into the failing tests

[brandonroberts]

@danielroe I took a different approach and just override the entry filename explicitly because the prerenderer is being built internally. Let me know if you have another recommended approach.

[pi0]

Nice find! I think we can move the default rollup config to the pretender preset.

[brandonroberts]

Nice find! I think we can move the default rollup config to the pretender preset.

Cool, done!

[pi0]

@brandonroberts sorry my bad just tried locally moving to preset won't work because explicit config overrides entry name still.

I took a different approach and just override the entry filename explicitly because the prerenderer is being built internally. Let me know if you have another recommended approach.

May i ask there was a reason to not use original approach? While since now we have only one chunk entry for pretender, if in the future we opt pretender bundle into multi chunks for less mem footprint, we might need to change solution (and since no regression test might forget). If original approach was working well, i would suggest to use that and dynamically compute serverFilename

[brandonroberts]

That approach broke 3 test suites tests where the preset is already overriding the entryFileNames property in combination with custom serverDir and output options.

I'll take another look

[pi0]

Thank you! Please let me know if could help on investing failed tests.

[brandonroberts]

Should be sorted out now. I needed to use the produced nitrorenderer config to build the server filename dynamically.

[pi0]

💚

[brandonroberts]

Thanks @pi0 and @danielroe!