fix(ci): drop redundant rrweb worker map cp in playwright workflow

[arnohillen]

Problem

After #59518 dropped @posthog/rrweb as a direct frontend dependency, the E2E Playwright job started failing at the Collect static files step:

cp frontend/node_modules/@posthog/rrweb/dist/image-bitmap-data-url-worker-*.js.map frontend/dist/

@posthog/rrweb is no longer a hoisted top-level package — pnpm only exposes it under node_modules/.pnpm/, not node_modules/@posthog/rrweb/ — so the glob matches nothing and the cp errors out before collectstatic runs.

Competing fix in #59839 keeps the cp but re-points it at posthog-js/dist with 2>/dev/null || true. This PR proposes the leaner alternative.

Changes

Remove the cp entirely. frontend/build.mjs already calls copyRRWebWorkerFiles() from common/esbuilder/utils.mjs, which (post-#59518) mirrors node_modules/posthog-js/dist/image-bitmap-data-url-worker-*.js.map into frontend/dist/ as part of pnpm --filter=@posthog/frontend build — the step immediately preceding Collect static files. The workflow-level cp was always duplicating bundler logic; #59518 just made that duplication visible by breaking the stale path.

Why this over #59839's cp … posthog-js/dist … 2>/dev/null || true:

• Single source of truth in copyRRWebWorkerFiles(). The workflow shouldn't redo what the bundler already does.
• || true silently swallows real failures. If the rrweb files move again, we wouldn't notice until collectstatic fails much later (or, worse, ships a broken sourcemap reference).

How did you test this code?

I'm an agent. No manual testing performed beyond local verification that:

• frontend/node_modules/posthog-js/dist/image-bitmap-data-url-worker-*.js.map exists.
• frontend/node_modules/@posthog/rrweb/ no longer exists (confirming the stale cp is unreachable).
• copyRRWebWorkerFiles() is called from frontend/build.mjs and reads from node_modules/posthog-js/dist.

The real validation is CI on this PR.

Publish to changelog?

no

🤖 Agent context

Cursor agent (Claude Opus 4.7), kicked off after Mendral's alerts flagged the broken cp step.

Considered both approaches: (a) re-point the cp at posthog-js/dist and harden with || true (Mendral's PR #59839), or (b) drop the cp entirely and rely on the bundler-level mirror. Picked (b) for the reasons in the Changes section — keeping one canonical path-aware location (utils.mjs) is more robust than a defensive duplicate that masks failures.

If reviewer prefers Mendral's approach, close this PR and merge #59839 instead.

Made with Cursor

[greptile-apps]

_{Reviews (1): Last reviewed commit: "fix(ci): drop redundant rrweb worker map..." | Re-trigger Greptile}

[arnohillen]

@PostHog/team-devex Could I get a review on this PR please :) It's linked to an alert and the alert is spamming me on slack

[arnohillen]

E2E CI Playwright / Playwright tests pass (pull_request) and E2E CI Playwright / Playwright E2E tests (pull_request) fail (also after retry). any idea @rnegron?

[github-actions]

🎭 Playwright report · View test results →

⚠️ 1 flaky test:

• Redirect to appropriate place after login (chromium)

These issues are not necessarily caused by your changes.
Annoyed by this comment? Help fix flakies and failures and it'll disappear!

[deployment-status-posthog]

Deploy status

Environment	Status	Deployed At	Workflow
dev	✅ Deployed	2026-05-31 02:17 UTC	Run
prod-us	✅ Deployed	2026-05-31 02:30 UTC	Run
prod-eu	✅ Deployed	2026-05-31 02:30 UTC	Run