Remove the ssr external for vue

[kalepail]

Changes

• What does this change?
  Solves Deploying to CF Pages with a Vue Island is broken adapters#207

Testing

Tested the CF Pages deploy and it works now.

Docs

[changeset-bot]

🦋 Changeset detected

Latest commit: 248b1ea

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

[alexanderniebuhr]

I'm not 100% sure where this should be landed. Either here or in the adapters PR 🤔
Any other opinions?

[ematipico]

I'm not 100% sure where this should be landed. Either here or in the adapters PR 🤔 Any other opinions?

Why was '@vue/server-renderer' there in the first place? That's the question we should answer first. For the time being, I would land the PR in the clouldflare adapter, and investigate other adapters before landing this PR.

[matthewp]

npm packages are external by default, so this is probably a remnant from before that was the case. Seems safe to do, to me.

[kalepail]

fwiw this package is deprecated anyway as it's now included natively in the Vue package itself. (See https://www.npmjs.com/package/@vue/server-renderer)

[ematipico]

Looks good. @kalepail can you please create a changeset?

[kalepail]

@ematipico done...I think...first time committing a changeset