fix(nuxt): don't joinURL with remote sources on NuxtIsland

[huang-julien]

🔗 Linked issue

fix #23092

❓ 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

Hi 👋

When doing a previous fix on adding baseURL for nuxt islands, and merging the remote sources PR after it, a condition hasn't been updated causing remote islands to not work

📝 Checklist

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

[stackblitz]

Run & review this pull request in StackBlitz Codeflow.

[huang-julien]

@danielroe do you know if there's a way to test remote island on the repo ? I don't think we can start two seperate servers on fixtures tests and this would cost a lot of time on the CI.
Maybe starting a small h3 server which would listen to 3001 when running a specific test ?

[danielroe]

I think it might be sufficient to provide a mock endpoint, which we could create in the test suite?

[huang-julien]

Do you mean something like <NuxtIsland source="/mock/island" > ?

[danielroe]

Or even fire up a mock server with listhen + h3 with a URL....

[huang-julien]

A mock server may be the best solution atm, I don't think having /mock/island would work as a second argument of new URL(), it will probably throw

[huang-julien]

Setting this to draft, i need to verify something on URL
I think we can set up theses test on anoither PR ? Or shall we do it on this one ?
Edit: all's good