feat!: remove service-ai — open edition is MCP-only (ADR-0025 S2)#2325
Merged
Conversation
…025 S2) Completes the ADR-0025 migration on the framework side. @objectstack/service-ai (in-UI AI runtime + ask agent) now lives in cloud/packages/service-ai (closed). The open framework exposes AI only via @objectstack/mcp (BYO-AI). - delete packages/services/service-ai - drop dep from packages/cli + examples/app-todo (CLI keeps its dynamic-import + silent-skip path for cloud/EE hosts) - remove app-todo's 7 in-UI-AI tests; keep mcp-actions.test.ts (CE test that proves actions work over MCP with no service-ai) - drop service-ai from .changeset/config.json fixed[] - de-stale ai-capabilities + packages docs (open AI -> MCP) Verified: turbo build (CLI + example-todo closures) green; app-todo CE MCP-actions E2E passes with service-ai absent. BREAKING CHANGE: @objectstack/service-ai is no longer published from the open framework. Self-hosted open runtimes use @objectstack/mcp (BYO-AI). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
📓 Docs Drift CheckThis PR changes 2 package(s): 18 hand-written doc(s) reference the affected code and may need an implementation-accuracy re-verification:
|
os-zhuang
added a commit
that referenced
this pull request
Jun 25, 2026
Optional, neutral field carrying the latest user message — for a tool to detect explicit user intent (e.g. a confirm/approval) without re-deriving it from the transcript. Consumed by cloud's confirm-before-change gate. Scope reduced after #2325 (ADR-0025) removed framework service-ai: the agent route that POPULATES this now lives with the AI runtime (cloud-owned), so this PR adds only the contract field; cloud wires the value when it owns the route. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
os-zhuang
added a commit
that referenced
this pull request
Jun 25, 2026
…2328) Optional, neutral field carrying the latest user message — for a tool to detect explicit user intent (e.g. a confirm/approval) without re-deriving it from the transcript. Consumed by cloud's confirm-before-change gate. Scope reduced after #2325 (ADR-0025) removed framework service-ai: the agent route that POPULATES this now lives with the AI runtime (cloud-owned), so this PR adds only the contract field; cloud wires the value when it owns the route. Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
os-zhuang
added a commit
that referenced
this pull request
Jun 26, 2026
…25) (#2337) Propagate framework #2325 (service-ai removed from the open framework) across the remaining docs. Canonical decision: cloud ADR-0025 (service-ai -> cloud; open = MCP-only). Disambiguates the cross-repo "ADR-0025" reference, since framework ADR-0025 is plugin-distribution and unrelated. New model documented everywhere: - Open edition (cloud free tier / Docker / desktop / on-prem): no in-product ask/build chat. Data query via @objectstack/mcp (BYO-AI); metadata authoring via source mode + an AI coding agent. - Cloud / Enterprise: in-UI AI runtime (ask + build + in-product chat + /api/v1/ai/*), governed ask, managed/metered AI. User-facing: ai-capabilities (ask edition open->cloud, fixed the ambiguous ADR-0025 ref, added the open-edition two-path callout), plugin-chatbot-integration, environment-variables, knowledge, implementation-status, README (AI bullet -> MCP; drop the dead service-ai Services row), both objectstack-* skills. ADRs: cloud-owned banners on 0033/0038/0040/0063/0064 per cloud ADR-0025 (banners only; ADR bodies preserved as historical record). Left as historical/unaffected: docs/audits/2026-06-*, RELEASE_NOTES.md, ARCHITECTURE.md (spec/ai schemas stay open), apps/docs/app/llms* (generated route handlers), other ADRs' incidental service-ai mentions. Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
ADR-0025 migration — framework half (S2): remove service-ai, open edition is MCP-only.
Completes the boundary in ADR-0025 / ADR-0026 (cloud#513) and pairs with the cloud-side move (cloud#514).
@objectstack/service-ai(in-UI AI runtime +askagent) now lives incloud/packages/service-ai(closed); the open framework exposes AI only via@objectstack/mcp(BYO-AI).Changes
packages/services/service-aipackages/cli+examples/app-todo(CLI keeps its dynamic-import + silent-skip path so a cloud/EE host can still mount it)mcp-actions.test.ts(CE test proving actions work over MCP with no service-ai)@objectstack/service-aifrom.changeset/config.jsonfixed[]ai-capabilities.mdx+packages.mdx(open AI → MCP)Verified locally:
turbo build(CLI + example-todo closures) green; app-todo CE MCP-actions E2E passes with service-ai absent.BREAKING:
@objectstack/service-aiis no longer published from the open framework. (Already-published versions stay on npm; self-hosted open runtimes use@objectstack/mcp.)🤖 Generated with Claude Code