Playwright bun-sqlite compatibility #85
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
This PR resolves issues preventing Playwright E2E tests from running by:
bun:sqliteand falling back tolru-cachewhen not running in Bun.entry.server.tsxfor environments whererenderToPipeableStreamis unavailable (e.g., Bun).bunxfor execution.Test Plan
Run
bun run test:e2e:runto verify end-to-end Playwright coverage for SSR and core flows.Checklist
Screenshots
Login page refresh renders cleanly with the Bun SSR fallback.
Note
Medium Risk
Touches SSR entrypoint wiring and the background text-sending cron runner (behavior intended to be the same but now lives in a new module), so regressions could impact page rendering or scheduled message delivery.
Overview
Updates server-side rendering to import
renderToPipeableStreamfromreact-dom/server.nodeand removes an unnecessaryasyncPromise executor inentry.server.tsx, while also switching cron initialization to a newcron-runner.server.tsmodule.Extracts the interval-based cron runner and
sendNextTextslogic out ofcron.server.ts(leaving only cron parsing/scheduling helpers), updates cron tests to target the new runner, and adjusts Playwright E2E scripts to run viabunx.Removes the test global setup’s import of
cache.server.ts, avoidingbun:sqlite-only code during Playwright/global setup execution.Written by Cursor Bugbot for commit 68f4b0f. This will update automatically on new commits. Configure here.