refactor: simplify the e2e and script configs

[vicb]

• On CI, e2e was rebuilding the apps (via preview), drop that as there is a build step before
• Factor the playwright configs
• Automatically assign a different port to each app - no need to hardcode ports in the package.json any more
• Do not run next dev tests on multiple browsers
• Remove dev:worker
• Rename preview:worker to preview - to be consistent with c3 templates

This should make the e2e CI job much faster (timeout is reduced)

[changeset-bot]

⚠️ No Changeset found

Latest commit: 094eb08

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[pkg-pr-new]

Open in Stackblitz

pnpm add https://pkg.pr.new/@opennextjs/cloudflare@342

commit: 094eb08

[dario-piotrowicz]

Very nice improvement 🫶

[dario-piotrowicz]

Very nice improvement 🫶

[dario-piotrowicz]

@vicb this is unrelated to this PR but I wanted to mention that here I notice that we use npm instead of pnpm for some of the examples, we even have some package-lock files: https://github.com/opennextjs/opennextjs-cloudflare/blob/simplify-configs/examples/bugs/gh-219/package-lock.json, that's (I hope we all agree) clearly wrong and we should just consolidate everything to use the pnpm workspace

[vicb]

@vicb this is unrelated to this PR but I wanted to mention that here I notice that we use npm instead of pnpm for some of the examples, we even have some package-lock files: https://github.com/opennextjs/opennextjs-cloudflare/blob/simplify-configs/examples/bugs/gh-219/package-lock.json, that's (I hope we all agree) clearly wrong and we should just consolidate everything to use the pnpm workspace

Good catch.

Yep, we should use pnpm everywhere. I didn't pay attention when importing the apps for e2e.
I'll handle that in a follow up PR.

[vicb]

Thanks for the review Dario!