fix: update transform error message

[btea]

Description

Related #13687

Additional context

---

What is the purpose of this pull request?

• Bug fix
• New Feature
• Documentation update
• Other

Before submitting the PR, please make sure you do the following

• Read the Contributing Guidelines.
• Read the Pull Request Guidelines and follow the PR Title Convention.
• Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
• Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
• Ideally, include relevant tests that fail without this PR but pass with it.

[stackblitz]

Run & review this pull request in StackBlitz Codeflow.

[btea]

@bluwy Please take a look when you are free, thank you!

[bluwy]

Sorry I meant prepending the 'Pre-transform error: ' string at these places instead. I think that helps clarify at which phase the error is coming from. Or am I missing how the error is logged?

vite/packages/vite/src/node/plugins/importAnalysis.ts

Line 646 in 27bffc4

config.logger.error(e.message, { error: e })

vite/packages/vite/src/node/server/middlewares/indexHtml.ts

Line 363 in 27bffc4

server.config.logger.error(e.message)

[btea]

I seem to have misunderstood what you meant. Do you want to concatenate the Pre-transform error: string in front of all error messages, not just EISDIR error messages?

[bluwy]

Yeah that's what I mean. Not sure if it's easily reproducible, but do you see 2 different error messages now with this tweak? I think that will hopefully clears up the error message if it ever happen twice again.

[btea]

Yeah, after I added the string to importAnalysis.ts, the error message became the following.

[bluwy]

Thanks! The differentiation between Pre-transform error and Internal server error is nice I think, so we know where the error is happening. IIUC they're separate processes (although the same request), so it's a little hard to deduplicate, but this could be safer for now.

The pre-transform error message could probably look better with the timestamp and [vite] prefix, but that can be improved later on.

[btea]

The pre-transform error message could probably look better with the timestamp and [vite] prefix, but that can be improved later on.

I've added timestamp option.