Skip to content

Plan: merge Electron smoke tests into the shared mobile harness - #2

Draft
ashleydavis wants to merge 2 commits into
mainfrom
cursor/electron-mobile-smoke-merge-plan-f470
Draft

Plan: merge Electron smoke tests into the shared mobile harness#2
ashleydavis wants to merge 2 commits into
mainfrom
cursor/electron-mobile-smoke-merge-plan-f470

Conversation

@ashleydavis

@ashleydavis ashleydavis commented Jul 18, 2026

Copy link
Copy Markdown
Owner

Folds the older host-bridge draft into a single plan of record for merging Electron smoke tests into the shared mobile harness.

Plan of record

docs/plans/new/plan-merge-electron-into-mobile-smoke-tests.md

What changed in this PR

Scope of the plan

  • Electron joins apps/smoke-tests as PLATFORM=electron
  • Retire in-process TestControlServer
  • Preserve parallel batches, two-app LAN (.sequential), and test:electron in test:all
  • Migrate story-player / UX screenshots / CLI↔Desktop LAN share off desktop common.sh

Implementation must land on a branch based on origin/mobile (or on main after mobile merges). This PR only adds/updates plan docs.

Open in Web Open in Cursor 

cursoragent and others added 2 commits July 18, 2026 06:33
Documents folding apps/desktop/smoke-tests into apps/smoke-tests behind
PLATFORM=electron, retiring the in-process TestControlServer, and covering
gaps the earlier mobile-branch draft missed (two-app LAN, parallel runner,
platform skips, and other common.sh consumers).

Co-authored-by: Ashley Davis <ashley@codecapers.com.au>
Absorbs the earlier plan-electron-host-bridge-smoke-tests draft (constraints,
driver/bridge steps, verify notes) into the merge plan and leaves a done/
pointer so there is a single plan of record.

Co-authored-by: Ashley Davis <ashley@codecapers.com.au>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants