Skip to content

[codex] align chat e2e with smoke checklist#433

Merged
blove merged 3 commits into
mainfrom
codex/chat-e2e-checklist-alignment
May 18, 2026
Merged

[codex] align chat e2e with smoke checklist#433
blove merged 3 commits into
mainfrom
codex/chat-e2e-checklist-alignment

Conversation

@blove
Copy link
Copy Markdown
Contributor

@blove blove commented May 18, 2026

Summary

  • expands canonical chat example e2e coverage into checklist-aligned specs for initial render, routing, model picker, devtools, palette, theme, lifecycle, errors, keyboard/accessibility, markdown, browser hygiene, and visual overflow
  • updates shared e2e helpers for deterministic browser hygiene, devtools state handling, and send/wait flows
  • fixes chat devtools default-open hydration and sidebar Escape handling surfaced by the new coverage
  • keeps stop-streaming as an explicit skipped harness pilot until in-flight stream state is deterministic enough for CI

Verification

  • npx nx test chat --skip-nx-cache
  • npx nx test examples-chat-angular --skip-nx-cache
  • npx nx lint chat --skip-nx-cache
  • npx nx lint examples-chat-angular --skip-nx-cache
  • npx playwright test -c examples/chat/angular/e2e/playwright.config.ts examples/chat/angular/e2e/keyboard-accessibility.spec.ts
  • npx nx e2e examples-chat-angular --skip-nx-cache (36 passed, 1 skipped)

Note: lint commands exit successfully with existing warnings only.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 18, 2026

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

Project Deployment Actions Updated (UTC)
cacheplane Ready Ready Preview, Comment May 18, 2026 7:30pm

Request Review

@blove blove force-pushed the codex/chat-e2e-checklist-alignment branch from 7797fd7 to a9f8215 Compare May 18, 2026 19:28
@blove blove merged commit fe2a85a into main May 18, 2026
18 checks passed
@blove blove deleted the codex/chat-e2e-checklist-alignment branch May 18, 2026 19:35
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.

1 participant