Skip to content

[codex] Remove chat protocol e2e target#423

Merged
blove merged 1 commit into
mainfrom
codex/remove-chat-protocol-e2e
May 18, 2026
Merged

[codex] Remove chat protocol e2e target#423
blove merged 1 commit into
mainfrom
codex/remove-chat-protocol-e2e

Conversation

@blove
Copy link
Copy Markdown
Contributor

@blove blove commented May 18, 2026

Summary

  • Delete the standalone examples/chat/e2e protocol test project.
  • Remove the examples-chat-protocol-e2e CI job and deploy prerequisites.
  • Clean stale project references from root TypeScript config, CI scope tests, workflow tests, and contributor guidance.

Validation

  • node --test scripts/ci-scope.spec.mjs scripts/ci-workflow.spec.mjs
  • npx nx show projects --with-target e2e
  • npx nx e2e examples-chat-angular --skip-nx-cache
  • git diff --check origin/main..HEAD

Note: the first browser e2e attempt hit a local port conflict from an existing server on :2024. After clearing that process and resetting Nx, examples-chat-angular passed all 10 tests.

@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 5:49pm

Request Review

@blove blove enabled auto-merge (squash) May 18, 2026 17:46
@blove blove merged commit bc2b8b3 into main May 18, 2026
15 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.

1 participant