Skip to content

refactor(mcp)!: rename @objectstack/plugin-mcp-server → @objectstack/mcp + ADR-0036 amendment - #1627

Merged
xuyushun441-sys merged 1 commit into
mainfrom
refactor/rename-mcp
Jun 6, 2026
Merged

refactor(mcp)!: rename @objectstack/plugin-mcp-server → @objectstack/mcp + ADR-0036 amendment#1627
xuyushun441-sys merged 1 commit into
mainfrom
refactor/rename-mcp

Conversation

@xuyushun441-sys

Copy link
Copy Markdown
Contributor

What

Renames the outbound MCP-server package @objectstack/plugin-mcp-server@objectstack/mcp and records the related decisions in ADR-0036. This is PR① of the agreed sequence (rename + ADR, then Phase 2b surfacing).

Why

plugin- is legacy naming. The package is the outbound twin of @objectstack/rest — "your app exposed over a protocol" — and the repo already names that kind of capability as a top-level, no-prefix package (packages/rest). So rest (old surface) + mcp (new surface) sit as peers; inbound MCP (consuming external servers) stays @objectstack/connector-mcp.

Changes

  • packages/plugins/plugin-mcp-serverpackages/mcp; package name → @objectstack/mcp; internal plugin id → com.objectstack.mcp; build/tsconfig relative paths fixed for the new depth. Exported API unchanged (MCPServerPlugin, MCPServerRuntime, registerObjectTools, McpDataBridge, …).
  • @objectstack/cli: dependency + dynamic-loader pkg id updated.
  • Inbound refs updated: runtime test comment, spec/ai/mcp.zod.ts comment, .changeset/config.json fixed-group, the feat(mcp): Streamable HTTP transport — apps as MCP servers (ADR-0036 Phase 2) #1626 changeset, README/docs.
  • Pre-launch clean break — no compat shim (only @objectstack/cli depended on it internally).

ADR-0036 amendment (2026-06-07)

Adds an amendment block capturing three decisions that refine/supersede the original text:

  • A. Naming@objectstack/mcp.
  • B. Granularity = environment, not app. One MCP server per env covers all that env's apps (apps are metadata groupings; the data surface is objects). Dozens of apps → one connection. Dynamic apps are a non-issue: discovery is live (list_objects/describe_object), new app → zero reinstall. Narrowing via sys_api_key.scopes, not separate servers.
  • C. Distribution = "skills + MCP", not vendor config snippets (supersedes §3/§4's primary artifact). Verified 2026-06 that Agent Skills (SKILL.md) is now an open cross-platform standard (Claude Code, OpenAI Codex, Gemini CLI, Copilot, Cursor, …). MCP = live governed access (the moat); a single generic ObjectStack Skill = the portable how-to layer on top. Status updated: Phase 1a (feat(runtime): harden hand-rolled API-key auth (ADR-0036 Phase 1a) #1624) + Phase 2 (feat(mcp): Streamable HTTP transport — apps as MCP servers (ADR-0036 Phase 2) #1626) shipped; Phase 2b next.

Verification (local)

  • @objectstack/mcp build incl. DTS ✓; @objectstack/cli + @objectstack/runtime build ✓ (51 tasks).
  • tsc --noEmit clean (mcp, cli).
  • Tests: mcp 36, runtime mcp 5 — all green. Lockfile regenerated (no plugin-mcp-server left).

🤖 Generated with Claude Code

…mcp + ADR-0036 amendment
Drops the legacy `plugin-` prefix and moves the outbound MCP-server package to
the top level (`packages/mcp`), parallel to `@objectstack/rest` — both are "your
app exposed over a protocol". Inbound MCP stays `@objectstack/connector-mcp`.
- packages/plugins/plugin-mcp-server → packages/mcp; name → @objectstack/mcp;
internal plugin id → com.objectstack.mcp; build/tsconfig relative paths fixed
for the new depth. Exported API unchanged (MCPServerPlugin, MCPServerRuntime,
registerObjectTools, McpDataBridge, …).
- @objectstack/cli: dependency + dynamic-loader pkg id updated.
- Inbound refs (runtime test, spec comment, changeset config, docs) updated.
Pre-launch clean break — no compat shim (only cli depended on it internally).
- ADR-0036 amendment (2026-06-07): records (A) the rename, (B) granularity =
per-environment not per-app (one MCP server per env covers all apps; dynamic
apps via live discovery; key-scope for narrowing), (C) distribution = skills +
MCP (one generic portable Skill + live MCP, not hand-maintained vendor config
snippets) — verified Agent Skills is now an open cross-platform standard.
- Status updated: Phase 1a (#1624) + Phase 2 (#1626) shipped; Phase 2b next.
Build + typecheck + tests green (mcp 36, runtime mcp 5); lockfile regenerated.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@vercel

vercelBot commented Jun 6, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

ProjectDeploymentActionsUpdated (UTC)
specBuildingBuildingPreview, CommentJun 6, 2026 10:03pm

Request Review

@github-actionsgithub-actionsBot added documentation Improvements or additions to documentation dependencies Pull requests that update a dependency file tests protocol:ai tooling labels Jun 6, 2026
@xuyushun441-sys
xuyushun441-sys merged commit d9f72fe into mainJun 6, 2026
8 of 9 checks passed
@xuyushun441-sys
xuyushun441-sys deleted the refactor/rename-mcp branch June 6, 2026 22:03
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependenciesPull requests that update a dependency filedocumentationImprovements or additions to documentationprotocol:aisize/mteststooling

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@xuyushun441-sys@os-zhuang