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
Copy link
Copy Markdown

vercel Bot commented Jun 6, 2026

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

Project Deployment Actions Updated (UTC)
spec Building Building Preview, Comment Jun 6, 2026 10:03pm

Request Review

@github-actions github-actions Bot 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 main Jun 6, 2026
8 of 9 checks passed
@xuyushun441-sys xuyushun441-sys deleted the refactor/rename-mcp branch June 6, 2026 22:03
@github-actions github-actions Bot added the size/m label Jun 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file documentation Improvements or additions to documentation protocol:ai size/m tests tooling

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants