mirror of
https://github.com/CopilotKit/CopilotKit.git
synced 2026-09-14 16:26:20 +08:00
Maxim
f8e13e675c
## What Sunsets `examples/showcases/banking`. It is superseded by `examples/showcases/reskinnable-demo`, which ships the same banking experience as one of its runtime-swappable skins (alongside airline) on top of a shared shell. Keeping both means maintaining two copies of the same demo. 160 files deleted, plus the five places that pointed at the app: | File | Change | | --- | --- | | `pnpm-workspace.yaml` | Drops the workspace entry. Also fixes the adjacent NOTE, which attributed the canary AG-UI pin to "banking's agent" when it is reskinnable-demo's own Python deep agent that needs it. | | `pnpm-lock.yaml` | Regenerated (−919/+12). The 12 additions are peer-suffix re-keying caused by removing the importer — banking pinned a different `eslint`/`vitest` peer combination. No dependency version changes. | | `examples/README.md` | The banking row becomes a reskinnable-demo row, so the successor is listed and the showcase count is unchanged. | | `showcase/shell-docs/src/content/docs/faq.mdx` | The "Banking Assistant" link retargets to reskinnable-demo instead of 404ing. | | `.github/config-allowlist.txt` | Drops the deleted `next.config.mjs`. | Note that banking was a real pnpm workspace member using `workspace:*` deps, unlike reskinnable-demo, which sits deliberately outside the workspace with its own lockfile. That is why the root lockfile has to be regenerated here. ## Deliberately not changed - `scripts/migrate-demos.sh` and `scripts/archive-demo-repos.sh` still name `examples/showcases/banking`. Those are the already-executed one-shot manifests for the repo consolidation; the path is a historical record in them, not a live reference. - `reskinnable-demo`'s `.env.example` and `docker-compose.yml` still explain their +200 port offset in terms of banking's stack. The offset stays real, and "was cloned from banking" stays true. - The `banking` mentions in `test_reskinnable-demo.yml` refer to reskinnable-demo's **banking skin**, not this app. ## Verification - `.github/scripts/check-config-allowlist.sh` passes. - A full `pnpm install` agrees with the regenerated lockfile (no further diff). - The lockfile-only regen and the full install produce identical output. The `test-and-check-packages` pre-commit hook fires on any `pnpm-lock.yaml` change, so it ran `test,publint,attw` across all 25 packages. Four suites failed locally — `sqlite-runner`, `web-inspector`, `vue`, `react-core` — in a worktree installed with `--ignore-scripts`, which skips `better-sqlite3`'s native build. This change touches no package source, so CI is the gate on those; please confirm they are green here before merging. 🤖 Generated with [Claude Code](https://claude.com/claude-code)