Tags: gregkonush/CopilotKit
Tags
fix(showcase): LGT remaining delta — 16 test fixes + component extrac… …tion + cleanup (CopilotKit#4906) ## Summary Closes the LGT-vs-LGP delta from PR CopilotKit#4879. Fixes 16 of 17 LGT-only test failures, extracts inlined components to match LGP's factored pattern, removes orphan demos, and ports infrastructure. ## Changes ### Test fixes (16 of 17 LGT-only failures) - **headless-complete**: `normalizeResponse()` for additional_kwargs.tool_calls + missing `get_revenue_chart` tool - **readonly-state**: Removed aimock `systemMessage: "Atai"` gate (gpt-5.4 sends role:developer) - **reasoning-custom**: Created missing demo page + wired route to existing reasoning graph - **reasoning-default**: Renamed from `reasoning-default-render` + added `suggestions.ts` - **declarative-gen-ui**: Fixed A2UI tool name (`render_a2ui` → `_design_a2ui_surface`) + state-aware ToolNode wrapper - **beautiful-chat**: A2UI v0.9 operation format + ExampleCanvas agentId + manage_todos Command + copilotkitEmitState ### Component extraction (inlining → LGP pattern) - **hitl**: 7 inlined components extracted to separate files (step-selector, steps-feedback, suggestions, 4 shadcn components) - **declarative-hashbrown**: All files aligned to LGP - **declarative-json-render**: All files aligned to LGP - **shared-state-read**: Extracted recipe-card.tsx + types.ts from 460-line page.tsx - **reasoning-default**: Extracted suggestions.ts - **beautiful-chat**: Extracted home-page.tsx ### Cleanup - Removed `agentic-chat-reasoning` (deprecated, untested, covered by reasoning-custom) - Removed `shared-state-write` (unimplemented stub, superseded by shared-state-read-write) - Ported `layout.tsx` from LGP (per-demo title metadata) ## Variance after this PR - Frontend files: 240 identical (96%), 10 justified diffs, 0 functional gaps - Tests: 37 identical (100%) - API routes: 15 identical (100%) ## 1 remaining LGT-only failure - reasoning-default "Show reasoning pill" — fixture doesn't produce reasoning tokens for LGT pipeline ## Test plan - [ ] Full LGP suite on production-parity local cluster - [ ] Full LGT suite on production-parity local cluster - [ ] CI checks
fix(showcase): complete open-gen-ui scrub + fix smoke-test filter reg… …ression (CopilotKit#4029) ## Summary Prior PR removed the open-gen-ui feature but left several loose ends. This PR completes the scrub: 1. **Source YAML** — removed `open` profile + `open-gen-ui` entries from `showcase/shared/constraints.yaml` (was missed before; would have re-introduced `open-gen-ui` on next generator run) 2. **Schema enum** — dropped `"open"` from `generative_ui` enum in `showcase/shared/manifest.schema.json` 3. **Test fixture** — `invalid-genui-manifest.yaml` now uses `[unknown-profile]` instead of `[open]`; tests still pass (validator rejects unknown profiles) 4. **Manifest descriptions** — all 17 `showcase/packages/*/manifest.yaml` files: "5 GenUI rendering strategies" → "4" (open-gen-ui was the 5th; now removed) 5. **Derived JSON regen** — `registry.json` regenerated cleanly via `generate-registry.ts` 6. **Smoke-test filter fix** — `integration-smoke.spec.ts:383` now reads top-level `i.deployed` instead of stale `i.starter?.deployed`. The old filter returned 0 starters post-regen (silent CI skip every 6h); new filter correctly gates on canonical top-level field. ## Context - `showcase/shell/src/data/demo-content.json` is also regenerated but NOT committed because it exceeds the lefthook 1MB binary-size cap (pre-existing repo condition, separate from this PR). - 3 starters (mastra, crewai-crews, claude-sdk-typescript) that had manually-patched `starter.deployed: false` are now covered by smoke — verified all 3 starter URLs respond HTTP 200 live. ## Test plan - [ ] CI green - [ ] Post-merge `starter-smoke` workflow picks up all 17 starters (not zero, as was the silent broken state) - [ ] No `open-gen-ui` references remain anywhere in showcase/
PreviousNext