E2E: temporary time limit bump

[phyllis-sy-wu]

WHY are these changes introduced?

With per-test store creation (next PR in stack), each test that uses app dev now creates and tears down a dev store. This adds ~30s per test, pushing the total suite time over the previous 20-minute Playwright globalTimeout and the 30-minute GitHub Actions job timeout on CI with 1 worker.

WHAT is this pull request doing?

• Bumps Playwright globalTimeout from 20 min to 35 min in playwright.config.ts
• Bumps GitHub Actions E2E job timeout-minutes from 30 to 40 in tests-pr.yml

Both are temporary — will be reduced when parallel workers (5 workers) are enabled in #7309, which cuts total suite time significantly.

How to test your changes?

pnpm test:e2e
# Should complete within 35 min on CI without hitting globalTimeout or job timeout

Checklist

• I've considered possible cross-platform impacts (Mac, Linux, Windows)
• I've considered possible documentation changes
• I've considered analytics changes to measure impact
• The change is user-facing, so I've added a changelog entry with pnpm changeset add

[phyllis-sy-wu]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• Trial branch for more workers #7343
• E2E: parallel workers #7309
• E2E: per-test store isolation with automatic teardown #7279
• E2E: temporary time limit bump #7282 👈 (View in Graphite)
• E2E: improve error context in assertions #7281
• E2E: one-time global auth before test start #7272
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[gonzaloriestra]

Aren't you planning to merge the whole stack together? Maybe you can just skip this PR if it's going to be solved in the next one.

[github-actions]

Differences in type declarations

We detected differences in the type declarations generated by Typescript for this branch compared to the baseline ('main' branch). Please, review them to ensure they are backward-compatible. Here are some important things to keep in mind:

• Some seemingly private modules might be re-exported through public modules.
• If the branch is behind main you might see odd diffs, rebase main into this branch.

New type declarations

We found no new type declarations in this PR

Existing type declarations

packages/cli-kit/dist/public/node/upgrade.d.ts

@@ -1,5 +1,3 @@
-import { getAutoUpgradeEnabled, setAutoUpgradeEnabled } from '../../private/node/conf-store.js';
-export { getAutoUpgradeEnabled, setAutoUpgradeEnabled };
 /**
  * Utility function for generating an install command for the user to run
  * to install an updated version of Shopify CLI.

[phyllis-sy-wu]

@gonzaloriestra Thanks for the suggestion! I added this PR as a safety net since the isolation PR (#7279) times out on CI without it and in case I ever need to merge partially. Also, I wasn't sure if the stack would always be merged atomically or sequentially. If it's merged one PR at a time, there'd be a window where CI breaks. But if we can merge the whole stack together, happy to drop this one!