Skip to content

test(lana): cover VS Code web host - #4

Merged
peternhale merged 4 commits into
mainfrom
ph/W-23939830-web-e2e
Aug 21, 2026
Merged

test(lana): cover VS Code web host#4
peternhale merged 4 commits into
mainfrom
ph/W-23939830-web-e2e

Conversation

@peternhale

@peternhale peternhale commented Aug 21, 2026

Copy link
Copy Markdown
Collaborator

PR overview

Stack 4 of 4. Depends on #3. Adds automated VS Code Web coverage and local browser-host tooling.

Changes made

  • Add Playwright coverage that opens a sample log in VS Code Web.
  • Verify the flame chart and Call Tree render in the webview.
  • Add the local VS Code Web server and serve:web workflow.
  • Keep Playwright output isolated from Jest, ESLint, and source control.

Type of change

  • Test
  • Chore

Related issues

related W-23939830

Validation

  • pnpm typecheck
  • pnpm test:ci: 137 suites, 1,830 tests
  • pnpm lint
  • pnpm build
  • pnpm test:e2e:web: 1 passed

@peternhale
peternhale marked this pull request as ready for review August 21, 2026 13:33
@peternhale
peternhale requested a review from mshanemc August 21, 2026 13:33
@peternhale
peternhale force-pushed the ph/W-23939830-web-e2e branch from ecb68cd to c098623 Compare August 21, 2026 18:42
@peternhale
peternhale force-pushed the ph/W-23939830-web-e2e branch from c098623 to 3bd51a0 Compare August 21, 2026 18:48
Base automatically changed from ph/W-23939830-web-host to main August 21, 2026 18:51
@peternhale
peternhale force-pushed the ph/W-23939830-web-e2e branch from 3bd51a0 to ae4b89b Compare August 21, 2026 18:51
@peternhale
peternhale merged commit 29e9e92 into main Aug 21, 2026
8 checks passed
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