feat: cleanup integration api

[florian-lefebvre]

Changes

• Closes Remove routes on astro:build:done #14444
• Removes routes from astro:build:done
• Removes entryPoints from astro:build:ssr as it was unused since the removal of functionPerRoute
• Removes the IntegrationRouteData type

Testing

Updated

Docs

withastro/docs#12445

[changeset-bot]

🦋 Changeset detected

Latest commit: e894dd5

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

[github-actions]

This PR is blocked because it contains a major changeset. A reviewer will merge this at the next release if approved.

[yanthomasdev]

This line is unaligned with the rest

Suggested change

	+ let routes
	+ let routes

[sarah11918]

@florian-lefebvre I'm just curious about the changesets being for the specific integrations only. This removes some stuff (including the IntegrationRouteDataType) from the Integrations API entirely, right? Does this mean that some integration authors might need to know about this in the context of their own integrations? Are these things that are literally only used, one for sitemap and one for the Vercel integration? (Just checking because that seems really specific. But then again, maybe that's why it's safe to get rid of them!)

[florian-lefebvre]

All integration authors are affected. I chose to scope the changesets to astro + each official integration it affects because it made sense to me, but now that you're saying it maybe it's not that relevant! Technically, changes made to these integrations are not public facing

[sarah11918]

OK, yeah in that case this feels like a change to the Integrations API proper, and should be represented as an Astro change? (That just happens to affect some of our own existing official integrations)

I'd only have ONE changeset, listing ALL the things removed from the Integrations API. And then have the multiple examples in the "what should I do?"

[florian-lefebvre]

Updated!

[sarah11918]

lol, two changesets is fine! 😄 let me read them now!

[florian-lefebvre]

Yeah sorry, I kinda prefer 2 changesets for it

[sarah11918]

Just some tiny tiny bits of polishing suggestions, in part to make this extra super obvious that this affects Astro integrations only! But this is great!