fix: make adapter the last element of integrations instead of the first - #12870

🦋 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 Performance Report

Merging #12870 will not alter performance

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

Summary

✅ 6 untouched benchmarks

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?

@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.

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

The bug is the same as withastro/adapters#445

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

@ematipico Thank you so much!