feat(mcp): add project management tools - #8677
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
There was a problem hiding this comment.
Effect service conventions: one violation found in apps/server/src/mcp/ProjectMcpService.ts (standalone service shape interface). The rest of the new MCP service module, toolkit, handlers, and contracts follow the repository's Effect conventions.
Posted via Macroscope — Effect Service Conventions
Thread transfer impact
This comment will update automatically after the next completed run. |
2cb24f5 to
c9605e0
Compare
c9605e0 to
db8d3a6
Compare
db8d3a6 to
e443710
Compare
e443710 to
56cf046
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.
Bugbot Autofix is ON, but a cloud agent failed to start.
Reviewed by Cursor Bugbot for commit 56cf046. Configure here.
ApprovabilityVerdict: Not approved Macroscope's review found this PR not approvable — This PR adds production MCP tools that can create, clone, update, and delete projects and cascade-delete their thread records, with changes across shared orchestration and source-control paths. It also changes persisted project defaults and effective workspace-mode behavior, making the scope and impact unsuitable for automatic approval. You can add or adjust custom eligibility rules. Learn more. |
There was a problem hiding this comment.
Effect service conventions: one minor import-hygiene issue in the new test module. The service definition itself (inline Context.Service interface, ProjectMcpService["Service"][...] annotations, namespaced module imports, make + layer order) follows the sibling WorktreeMcpService.ts conventions.
Posted via Macroscope — Effect Service Conventions
There was a problem hiding this comment.
One convention issue found: an aliased layer import in apps/server/src/mcp/ProjectMcpService.test.ts erases the service module namespace (and is shadowed later in the file). The service definition itself (inline Context.Service interface, make, typed layer, Foo["Service"] annotations, dependencies acquired from the environment) follows the conventions.
Posted via Macroscope — Effect Service Conventions

Problem
Agents can work inside projects but cannot discover, register, clone, configure, or safely remove project records through T3's MCP boundary. Existing project mutations also drop
defaultThreadEnvModeandfaviconPathalong some client/server paths.Change
t3_project_list,t3_project_read,t3_project_create,t3_project_update, andt3_project_deletetools backed byProjectService.t3_project_read.Behavior
Project reads expose roots, titles, model defaults, scripts, icons, and project/file/global/effective workspace modes. Clone creation never publishes or adopts unrelated existing contents. Project removal never deletes repository, worktree, or workspace files. Delete counts are per attempt; a retry after deletion returns
alreadyDeleted: trueand a zero count.Validation
@t3tools/contracts,@t3tools/client-runtime, and server targeted typechecks.Dependency
Targets the immutable rollout base
agents/mcp-controls/base-490318aat490318afa505d3d033295eca12d7e62b4b922725. This is the bottom layer of native stack #8710; #8678 is the dependent upper layer.Implemented by GPT-5.6-Sol via Codex in T3 Code.