feat: add workspace MCP configuration - #3293
Conversation
|
Too many files changed for review (152 files, 100 file limit). Bypass the limit by tagging |
|
Important Review skippedToo many files! This PR contains 165 files, which is 15 over the limit of 150. To get a review, reduce the PR to 150 files or fewer by splitting it into smaller PRs or changing its base branch. Upgrade to a paid plan to raise the limit. This review couldn't start because sufficient usage credits or metered capacity aren't available. Add credits or update usage-based reviews in the billing tab, then retry. ⚙️ Run configurationConfiguration used: Organization UI Review profile: QUIET Plan: Team Run ID: 📒 Files selected for processing (165)
You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 954b41ba10
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
All reported issues were addressed across 152 files
Reply with feedback, questions, or to request a fix.
Re-trigger cubic
…e-mcp-c-dcc # Conflicts: # apps/backend/internal/backendapp/agents.go # docs/public/security.md
Cloudflare Pages docs previewBuilt from docs commit Stable PR alias: https://docs-pr-3293.landing-87j.pages.dev/docs |
…e-mcp-c-dcc # Conflicts: # docs/specs/agents/README.md
…e-mcp-c-dcc # Conflicts: # docs/public/feature-status.md
Tip
PR walkthrough: Open the visual walkthrough
Workspace MCP configuration was previously scattered across legacy profile JSON and launch-time assumptions, which made reusable workspace definitions, safe registry installs, and scoped task/session selection impossible. This adds a workspace-owned catalog with additive selection resolution, explicit marketplace review, lazy materialization, legacy migration, idle ACP reconfiguration, and responsive settings/selectors.
Important Changes
session/resumeand falling back tosession/loadwith durable status.Validation
env -u KANDEV_INTERNAL_CONFIG_FILE -u KANDEV_INTERNAL_CONFIG_HOME_FILE make -C apps/backend test(full Go suite)make -C apps/backend buildmake -C apps/backend e2e-plugin-packagepnpm --filter @kandev/web run lintpnpm --filter @kandev/web run typecheckpnpm run i18n:checkpnpm run i18n:ratchetpnpm run e2e:sleep-ratchetpnpm run build:e2enode --test scripts/validate-public-docs.test.mjsandnode scripts/validate-public-docs.mjspython3 scripts/lint-spec-files.py --allgit diff --checkThe full frontend Vitest suite was attempted twice, but its worker exited unexpectedly after happy-dom stylesheet warnings under Node 24. The focused changed-file suite passed.
Diagram
flowchart LR C[Workspace catalog] --> S[Scope selections] S --> R[Effective resolver] R --> L[Launch with resolved servers] R --> I[Idle session reconfiguration] M[Registry cache] --> C P[Legacy profile JSON] --> X[One-time workspace migration] X --> CPossible Improvements
Medium risk: the full frontend Vitest suite still needs a stable Node 24 happy-dom worker run; focused changed-file tests are green.
Checklist
apps/web/), I have added or updated Playwright e2e tests inapps/web/e2e/and verified them withmake test-e2e.docs/public/**and updated them or noted why no docs change is needed.Screenshots
Preview Environment
6f0cc94