Skip to content

Commit bc2b8b3

Browse files
authored
Remove chat protocol e2e target (#423)
1 parent d73934f commit bc2b8b3

11 files changed

Lines changed: 2 additions & 287 deletions

File tree

.github/workflows/ci.yml

Lines changed: 1 addition & 45 deletions
Original file line numberDiff line numberDiff line change
@@ -253,45 +253,6 @@ jobs:
253253
path: examples/chat/angular/e2e/test-results/
254254
retention-days: 7
255255

256-
examples-chat-protocol-e2e:
257-
name: examples/chat — protocol e2e
258-
needs: ci-scope
259-
if: github.event_name == 'push' || needs.ci-scope.outputs.examples_chat == 'true'
260-
runs-on: ubuntu-latest
261-
steps:
262-
- uses: actions/checkout@v6.0.2
263-
- uses: actions/setup-node@v6.3.0
264-
with:
265-
node-version: 22
266-
cache: npm
267-
- name: Install uv
268-
uses: astral-sh/setup-uv@v8.0.0
269-
with:
270-
python-version: '3.12'
271-
- run: npm ci
272-
- working-directory: examples/chat/python
273-
run: uv sync
274-
- name: Start LangGraph dev server
275-
working-directory: examples/chat/python
276-
run: uv run langgraph dev --no-browser &
277-
env:
278-
OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}
279-
LANGSMITH_API_KEY: ${{ secrets.LANGSMITH_API_KEY }}
280-
LANGSMITH_TRACING: 'true'
281-
LANGSMITH_PROJECT: examples-chat-protocol-e2e-ci
282-
- name: Wait for server to be ready
283-
run: |
284-
echo "Waiting for LangGraph server..."
285-
for i in {1..30}; do
286-
curl -sf http://localhost:2024/ok && echo "Server ready" && break
287-
echo "Attempt $i/30..."
288-
sleep 2
289-
done
290-
curl -sf http://localhost:2024/ok || (echo "Server failed to start after 60s" && exit 1)
291-
- run: npx nx e2e examples-chat-protocol-e2e --skip-nx-cache
292-
env:
293-
LANGGRAPH_URL: http://localhost:2024
294-
295256
cockpit-e2e:
296257
name: Cockpit — e2e
297258
needs: ci-scope
@@ -374,7 +335,6 @@ jobs:
374335
cockpit-deploy-smoke,
375336
examples-chat-smoke,
376337
examples-chat-e2e,
377-
examples-chat-protocol-e2e,
378338
cockpit-e2e,
379339
website-e2e,
380340
]
@@ -544,7 +504,7 @@ jobs:
544504
545505
demo-deploy:
546506
name: Canonical demo → Vercel
547-
needs: [examples-chat-smoke, examples-chat-e2e, examples-chat-protocol-e2e]
507+
needs: [examples-chat-smoke, examples-chat-e2e]
548508
runs-on: ubuntu-latest
549509
if: ${{ always() && !cancelled() && github.ref == 'refs/heads/main' && github.event_name == 'push' }}
550510
steps:
@@ -558,10 +518,6 @@ jobs:
558518
echo "::error::examples/chat — e2e finished with ${{ needs.examples-chat-e2e.result }}; refusing to deploy the canonical demo."
559519
exit 1
560520
fi
561-
if [ "${{ needs.examples-chat-protocol-e2e.result }}" != "success" ]; then
562-
echo "::error::examples/chat — protocol e2e finished with ${{ needs.examples-chat-protocol-e2e.result }}; refusing to deploy the canonical demo."
563-
exit 1
564-
fi
565521
- uses: actions/checkout@v6.0.2
566522
- uses: actions/setup-node@v6.3.0
567523
with:

AGENTS.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ This file is for agents working in this repository. It is contributor-facing, no
3838

3939
- `libs/langgraph`: main Angular library (`@ngaf/langgraph`).
4040
- `apps/website`: docs and marketing site.
41-
- `examples/chat/e2e`: protocol end-to-end coverage for the canonical chat example.
41+
- `examples/chat/angular/e2e`: browser end-to-end coverage for the canonical chat example.
4242

4343
## Working in This Repo
4444

examples/chat/e2e/project.json

Lines changed: 0 additions & 14 deletions
This file was deleted.

examples/chat/e2e/src/.gitkeep

Whitespace-only changes.

examples/chat/e2e/src/protocol.e2e.spec.ts

Lines changed: 0 additions & 157 deletions
This file was deleted.

examples/chat/e2e/tsconfig.json

Lines changed: 0 additions & 7 deletions
This file was deleted.

examples/chat/e2e/vite.config.mts

Lines changed: 0 additions & 16 deletions
This file was deleted.

scripts/ci-scope.mjs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -221,7 +221,6 @@ function applyFallbackPathScope(scope, changedFile) {
221221
function isGlobalCiFile(changedFile) {
222222
return (
223223
changedFile === '.github/workflows/ci.yml' ||
224-
changedFile === '.github/workflows/e2e.yml' ||
225224
changedFile === 'package.json' ||
226225
changedFile === 'package-lock.json' ||
227226
changedFile === 'nx.json' ||

scripts/ci-scope.spec.mjs

Lines changed: 0 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -52,14 +52,6 @@ const projects = [
5252
tags: [],
5353
targets: { build: {} },
5454
},
55-
{
56-
name: 'examples-chat-protocol-e2e',
57-
root: 'examples/chat/e2e',
58-
sourceRoot: 'examples/chat/e2e/src',
59-
projectType: 'application',
60-
tags: [],
61-
targets: { e2e: {} },
62-
},
6355
];
6456

6557
const workspace = {
@@ -122,18 +114,6 @@ test('global CI config changes keep full PR coverage', () => {
122114
assert.deepEqual(Object.values(scope), Object.values(scope).map(() => true));
123115
});
124116

125-
test('standalone E2E workflow changes keep full PR coverage', () => {
126-
const scope = classifyChangedFiles(['.github/workflows/e2e.yml'], workspace);
127-
128-
assert.deepEqual(Object.values(scope), Object.values(scope).map(() => true));
129-
});
130-
131-
test('examples chat protocol e2e changes drive examples chat scope', () => {
132-
const scope = classifyChangedFiles(['examples/chat/e2e/src/protocol.e2e.spec.ts'], workspace);
133-
134-
assert.equal(scope.examples_chat, true);
135-
});
136-
137117
test('unowned docs changes do not trigger heavy CI jobs', () => {
138118
const scope = classifyChangedFiles(['docs/notes.md'], workspace);
139119

scripts/ci-workflow.spec.mjs

Lines changed: 0 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -15,14 +15,6 @@ describe('CI workflow', () => {
1515
);
1616
}
1717

18-
async function readDemoDeployJob() {
19-
const workflow = await readWorkflow();
20-
return workflow.slice(
21-
workflow.indexOf(' demo-deploy:'),
22-
workflow.indexOf(' production-smoke:')
23-
);
24-
}
25-
2618
async function readProductionSmokeJob() {
2719
const workflow = await readWorkflow();
2820
return workflow.slice(
@@ -70,19 +62,4 @@ describe('CI workflow', () => {
7062
);
7163
});
7264

73-
it('runs examples chat protocol e2e before deploying the canonical demo', async () => {
74-
const workflow = await readWorkflow();
75-
const demoDeployJob = await readDemoDeployJob();
76-
77-
assert.match(workflow, /examples-chat-protocol-e2e:/);
78-
assert.match(workflow, /npx nx e2e examples-chat-protocol-e2e --skip-nx-cache/);
79-
assert.match(
80-
demoDeployJob,
81-
/needs:\s*\[examples-chat-smoke,\s*examples-chat-e2e,\s*examples-chat-protocol-e2e\]/
82-
);
83-
assert.match(
84-
demoDeployJob,
85-
/examples\/chat protocol e2e finished with \$\{\{ needs\.examples-chat-protocol-e2e\.result \}\}/
86-
);
87-
});
8865
});

0 commit comments

Comments
 (0)