fix: do not mark already resolved ids as external (optimize deps)

[patricklx]

Description

our custom esbuild resolver sometimes resolves an id to a full path and calls build.resolve with it. this causes those paths to be marked as external.

Additional context

i'm not sure if there is a better solution. Ideally i would like to skip all plugins on build.resolve in esbuild, but that option is not available. Is there any alternative?

---

What is the purpose of this pull request?

• [X] Bug fix
• [ ] New Feature
• [ ] Documentation update
• [ ] Other

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

• [X] Read the Contributing Guidelines, especially the Pull Request Guidelines.
• [x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
• [x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
• [ ] Update the corresponding documentation if needed.
• [ ] Ideally, include relevant tests that fail without this PR but pass with it.

[bolt-new-by-stackblitz[bot]]

Run & review this pull request in StackBlitz Codeflow.

[patricklx]

Resolved by not passing absolute paths to build.resolve. I still think this is relevant