Skip to content

test: cover SharedWorker BFCache restoration - #487

Merged
electrohyun merged 1 commit into
mainfrom
test-shared-worker-bfcache-lifecycle
Sep 3, 2026
Merged

test: cover SharedWorker BFCache restoration#487
electrohyun merged 1 commit into
mainfrom
test-shared-worker-bfcache-lifecycle

Conversation

@electrohyun

Copy link
Copy Markdown
Owner

Summary

Extend the production SharedWorker lifecycle workflow with a same-origin history navigation that enters and restores from Chromium's back-forward cache. The workflow proves pagehide cleanup, the restored facade's disconnected state, marker-based non-receipt, and a usable fresh generation after explicit reconnect and room admission.

Run the lifecycle checks in headed Chromium under Xvfb because Playwright disables BFCache in its default launch configuration. Document that the page JavaScript instance survives restoration while callers remain responsible for reconnecting and restoring application admission state.

Related issue

Closes #484

Checklist

  • Tests added or updated
  • Behavior verified against real socket.io — not applicable; BFCache lifecycle is specific to the SharedWorker facade and delivery behavior is unchanged

@vercel

vercel Bot commented Sep 3, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
smocket Building Building Preview Sep 3, 2026 3:23am UTC

@coderabbitai

coderabbitai Bot commented Sep 3, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 27 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Team

Run ID: ebee170b-245c-43f3-88ae-c3546158f4b7

📥 Commits

Reviewing files that changed from the base of the PR and between 3d85c4b and 924ac8b.

📒 Files selected for processing (4)
  • .github/workflows/ci.yml
  • browser-tests/shared-worker-lifecycle/page.ts
  • docs/shared-worker.md
  • scripts/shared-worker-lifecycle.mjs

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added test Test coverage and fixtures 📏 m Pull request changes 50–199 lines labels Sep 3, 2026
@electrohyun
electrohyun merged commit 5c0e750 into main Sep 3, 2026
19 checks passed
@electrohyun
electrohyun deleted the test-shared-worker-bfcache-lifecycle branch September 3, 2026 03:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

📏 m Pull request changes 50–199 lines test Test coverage and fixtures

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cover SharedWorker facade state after BFCache restoration

1 participant