Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 0 additions & 9 deletions .claude/launch.json
Original file line number Diff line number Diff line change
Expand Up @@ -29,15 +29,6 @@
"port": 3000,
"autoPort": true
},
{
"name": "cockpit",
"runtimeExecutable": "/bin/bash",
"runtimeArgs": [
"-c",
"export PATH=/Users/blove/.nvm/versions/node/v22.14.0/bin:$PATH && npx nx serve cockpit --port 4201"
],
"port": 4201
},
{
"name": "streaming",
"runtimeExecutable": "/bin/bash",
Expand Down
192 changes: 16 additions & 176 deletions .github/workflows/ci.yml

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion .github/workflows/deploy-ag-ui.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ on:
paths:
- 'cockpit/ag-ui/**/python/**'
- 'cockpit/runtimes/**/python/**'
- 'apps/cockpit/scripts/capability-registry.ts'
- 'libs/cockpit-registry/src/lib/capability-registry.ts'
- 'scripts/generate-ag-ui-deployment-config.ts'
- 'deployments/ag-ui-dev/**'
workflow_dispatch:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/deploy-langgraph.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ on:
- 'cockpit/langgraph/**/python/**'
- 'cockpit/deep-agents/**/python/**'
- 'examples/chat/python/**'
- 'apps/cockpit/scripts/capability-registry.ts'
- 'libs/cockpit-registry/src/lib/capability-registry.ts'
- 'scripts/generate-shared-deployment-config.ts'
- 'deployments/shared-dev/langgraph.json'
workflow_dispatch:
Expand Down
38 changes: 18 additions & 20 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ cause, and each is fixed by the install above:
| --- | --- |
| `Cannot find module 'posthog-node'` when building `website` | Installed only into `apps/website/node_modules`, never hoisted. The walk up from the worktree's `apps/website/` never passes through the main checkout's. |
| `Could not resolve "node_modules/katex/dist/katex.min.css"` | Referenced by literal path from the workspace root, not by module resolution — so there is no upward walk to inherit through. |
| `Next.js inferred your workspace root, but it may not be correct` when building `cockpit` | Turbopack resolves its own workspace root and will not compile outside it. |
| `Next.js inferred your workspace root, but it may not be correct` when building `website` | Turbopack resolves its own workspace root and will not compile outside it. |

Do not fix these by copying individual packages from the main checkout. The list
keeps growing, a partial copy pulls in a package without its transitive
Expand All @@ -48,17 +48,18 @@ Do not run `npm install` in a worktree on macOS either: it rewrites
`package-lock.json` and drops the Linux `@next/swc-*` bindings, which breaks CI.
`npm ci` is the safe command.

One more worktree-only trap, after install: `nx build cockpit` can die with a
One more worktree-only trap, after install: `nx build website` can die with a
Turbopack panic — `FileSystemPath(...).join(...) leaves the filesystem root` —
whenever a website `.next` cache exists anywhere in the workspace
(`apps/website/.next` from `next dev`/e2e, or `dist/apps/website` from a build).
Those caches contain a `node_modules` symlink whose relative target resolves
through the *main* checkout's root; the path is valid on disk, but Turbopack's
virtual filesystem refuses to traverse above its project root and panics.
Pinning `turbopack.root` does not help. Clear the caches instead:
whenever a stale website `.next` cache exists anywhere in the workspace
(`apps/website/.next` from `next dev`/e2e, or `dist/apps/website` from a
previous build). Those caches contain a `node_modules` symlink whose relative
target resolves through the *main* checkout's root; the path is valid on
disk, but Turbopack's virtual filesystem refuses to traverse above its
project root and panics. Pinning `turbopack.root` does not help. Clear the
caches instead:

```bash
rm -rf apps/website/.next dist/apps/website && npx nx build cockpit
rm -rf apps/website/.next dist/apps/website && npx nx build website
```

CI never hits this — its jobs build from fresh checkouts that are not nested
Expand Down Expand Up @@ -197,8 +198,8 @@ not an alternative.

### PR-side deploy verification

Two lanes run the deploy job's verification on pull requests against real
Vercel previews, so deploy-only failures surface before merge:
One lane runs the deploy job's verification on pull requests against a real
Vercel preview, so deploy-only failures surface before merge:

- **Website — e2e (deployed preview)** builds and deploys the Website and
the examples as previews under deterministic aliases
Expand All @@ -209,15 +210,12 @@ Vercel previews, so deploy-only failures surface before merge:
the Website alias in their parent-origin policy and Playwright seeds the
examples origin's bypass cookie (`apps/website/e2e/runtime-bypass-setup.ts`).
A later push re-points both aliases; the deployments behind them are kept.
- **Cockpit — immutable preview smoke** deploys a throwaway cockpit preview,
runs the exhaustive redirect smoke against it, and removes it.

Both need repository secrets and therefore skip on fork PRs; the required
gate only demands them when they were eligible to run. Each Vercel project
has its own Protection Bypass for Automation secret:
`VERCEL_AUTOMATION_BYPASS_SECRET` (Website),
`VERCEL_EXAMPLES_AUTOMATION_BYPASS_SECRET`, and
`VERCEL_COCKPIT_AUTOMATION_BYPASS_SECRET`. A secret added while a run is in

It needs repository secrets and therefore skips on fork PRs; the required
gate only demands it when it was eligible to run. Each Vercel project has its
own Protection Bypass for Automation secret:
`VERCEL_AUTOMATION_BYPASS_SECRET` (Website) and
`VERCEL_EXAMPLES_AUTOMATION_BYPASS_SECRET`. A secret added while a run is in
flight does not reach that run; re-run after provisioning. Never pass
`--skip-domain` to a preview deploy; Vercel requires it to accompany `--prod`.

Expand Down
84 changes: 0 additions & 84 deletions apps/cockpit/ag-ui-agent-url.spec.ts

This file was deleted.

14 changes: 0 additions & 14 deletions apps/cockpit/next.config.spec.ts

This file was deleted.

16 changes: 0 additions & 16 deletions apps/cockpit/next.config.ts

This file was deleted.

11 changes: 0 additions & 11 deletions apps/cockpit/package.json

This file was deleted.

Loading
Loading