test(cockpit-langgraph): aimock e2e for 7 remaining langgraph caps (Task #4)#482
Merged
Conversation
Brings langgraph product line to FULL aimock coverage (8/8 caps total with streaming pre-existing). Caps shipped: - persistence, interrupts, memory, durable-execution, subgraphs, time-travel, deployment-runtime. All 7 use composed <chat> in the main slot. Fixtures recorded via a new generic recorder at scripts/record-aimock-cap.sh (single shell script taking cap id + prompts; reads registry for ports/paths). Each spec sends "Hello" and asserts the assistant bubble renders — proves backend boots, langgraph routes, aimock replays, UI finalizes. Registry change: added pythonPort to all 8 langgraph cap entries (5300-5307, matching the +1000-from-angular convention used by chat caps). Required by the scaffold generator's all-or-nothing precondition check. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
…p with new per-cap pythonPort This PR added pythonPort to all 8 langgraph cap registry entries (5300-5307), but their proxy.conf.json files still pointed at the umbrella port 8123. The cockpit-e2e-wiring spec cross-checks that registry pythonPort matches the global-setup-impl's langgraphPort matches the proxy.conf.json target — mismatch caught by build/test. Fix: update all 8 langgraph proxy.conf.json files to per-cap ports, matching the chat-cap convention (angular port + 1000). Also add the missing langgraphPort to streaming's global-setup-impl (it previously relied on the proxy-fallback in createGlobalSetup; explicit now). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Brings the langgraph product line to FULL aimock coverage (8/8 caps; `streaming` was pre-existing).
Caps shipped: persistence, interrupts, memory, durable-execution, subgraphs, time-travel, deployment-runtime.
Approach
Registry change
Added `pythonPort` to all 8 langgraph cap entries (5300-5307, matching the `+1000-from-angular` convention used by chat caps). The generator's all-or-nothing precondition check requires this field.
New shared recorder
`scripts/record-aimock-cap.sh "" ["" ...]` — generic helper that:
Replaces the per-cap shell scripts shipped earlier (c-generative-ui, c-a2ui, c-interrupts). Those will be consolidated in a follow-up cleanup.
Coverage after this PR
Test plan
🤖 Generated with Claude Code