fix: make adapter the last element of integrations instead of the first

[Kang-Jh]

Change

• Make adapter the last element of integrations instead of the first
  • This ensures that adapter is called after all integrations are called so that adapter can their job correctly
  • It fixes partytown integration doesn't work with vercel adapter in production #12832
• I hope it's not a breaking change, but it might be

Testing

I tested locally using reproduction repo in #12832

Docs

/cc @withastro/maintainers-docs for feedback!

[changeset-bot]

🦋 Changeset detected

Latest commit: d2d1802

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

[codspeed-hq]

CodSpeed Performance Report

Merging #12870 will not alter performance

_{Comparing Kang-Jh:main (d2d1802) with main (70a9f0b)}

Summary

✅ 4 untouched benchmarks

[ematipico]

We can't accept this change, because it reverts a change we did while back to fix another bug.

Now, what's the actual issue? Why is the order important for the partytown integration?

[Kang-Jh]

@ematipico
The Partytown integration copies the Partytown JavaScript file to the output directory (/dist), and the Vercel adapter copies the files from the /dist directory to the .vercel/output directory. However, since the Vercel adapter is executed first, the Partytown JavaScript file is not copied into the .vercel/output directory because the Vercel adapter process terminates beforehand.

[Kang-Jh]

Hmm, if the current change isn't acceptable, I have to dig into it to fix this without the current change.

[ematipico]

The bug is the same as withastro/adapters#445

Different output directories. As a current workaround, you need to move the files yourself

[Kang-Jh]

@ematipico Thank you so much!