Repository navigation
docs(concepts): the architecture page's sixth protocol card becomes Marketplace Protocol - #19178
Merged
os-try-charles merged 1 commit intoSep 19, 2026
Merged
Conversation
…arketplace Protocol The six cards on the protocol-stack section map one-to-one onto the namespaces `packages/spec` publishes, and `./cloud` is no longer one of them: PR #17372 moved the cloud control plane's contracts to the cloud repo while the package format stayed as `@objectstack/spec/marketplace`. The card and the mermaid node under "Why Separated Layers?" were two expressions of that stale namespace. Rename the sixth card to `Marketplace Protocol` with the description "Packaging: package format, versions, listings, templates.", swap its `Cloud` icon for `Package`, and rewrite the mermaid node `System · Automation · Cloud Protocol` / `… Multi-tenancy` to match. Multi-tenancy and Licensing leave the page with the card they described. Claude-Session: https://claude.ai/code/session_017ef78bLdybu3AffehKkhfk Co-authored-by: Claude <noreply@anthropic.com>
os-try-charles
marked this pull request as ready for review
September 19, 2026 07:34
os-try-charles
deleted the
claude/issue-17387-marketplace-protocol-card
branch
September 19, 2026 07:47
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.
Fixes #17387
Clause-②: no
What changed
content/docs/concepts/architecture.mdxonly — 1 file, +5 / -5. No code, no published surface, no workflow, nothing underpackages/spec.The six cards under "The Protocol Stack" map one-to-one onto the protocol namespaces
packages/specpublishes, and./cloudis no longer one of them: PR #17372 moved the cloud control plane's contracts into the cloud repo while the package format stayed as@objectstack/spec/marketplace. The sixth card and the mermaid node under "Why Separated Layers?" were two expressions of that one stale namespace, so both moved together — changing only one would have left the page self-contradictory.Marketplace Protocol, described as "Packaging: package format, versions, listings, templates."Multi-tenancyandLicensingleave the page with the card that described them.System · Automation · Cloud Protocol/Auth · Permissions · Orchestration · Multi-tenancynow readsSystem · Automation · Marketplace Protocol/Auth · Permissions · Orchestration · Packaging.Two deviations from the literal ruling text, both declared rather than silent:
Cloudglyph fromlucide-react; a card titledMarketplace Protocolcarrying a cloud icon is the same drift in a third expression, so the import and theicon=prop now namePackageinstead.Packageis already the icon other pages use for packaging (content/docs/protocol/kernel/plugin-spec.mdx,content/docs/protocol/objectql/schema.mdx) and it is exported by the installedlucide-react@1.31.0.Ruling executed, not re-weighed
The card body listed three options and called the choice a framing decision. The director seat ruled A on 2026-09-11T11:53Z (issue comment
5634035531, class-one self-adjudication recorded in the summon #22 ledger for the maintainer's later ratification), on the standing ruling 「文档应该以实际实现为准」. Triage graded itpriority:p3/ typeBug(5690506939). This PR executes ruling A; it does not reopen the options.Evidence
Premise re-verified on this branch's base
30288da04(the PM measured on83649b870; the reading is unchanged):The card's own predicate, re-measured after the edit, with its same-shape control:
A zero with no control is not a reading; the control still hits, and it grew by exactly this page.
Gates
node scripts/pm/dispatch-gates.mjs --commandsre-derived against the real changed set (not a hand-written list) at30288da04: 39 commands, all run, all exit 0, reconciled with--rancarrying each exit code:Five of the 39 first exited
3/1withPREREQUISITE NOT MET(unbuilt@objectstack/lint,@objectstack/formula,@objectstack/spec,@objectstack/client-react). Those are not findings; the packages were built under the shared verify lock and all five then exited 0. Notablycheck:docs-section-name— the gate the card named — passes both plainly and with--self-test, so the card's claim that no gate reads this page's card titles holds as measured, not as quoted.Outside the derived 39 and not measured here: the 52 artifact-roster families, the 11 declared wide-population families, the 14 families that would apply once a changeset existed, and the 2 path-scheduled CI jobs (
ci.yml · Test Core,ci.yml · Build Docs). CI owns those.skip-changesetis correct and measured, not assumed: no package'sfiles[]array names anything undercontent/, andapps/docsis"private": true. Nothing published moves. The ruling namesskip-changesetas well.Acceptance notes
content/docs/concepts/architecture.mdx:147carries a third occurrence of the word on the page —- Multi-tenancy and data isolation, a bullet in the "Layer 2: Kernel (Control Protocol)" responsibilities list. It is not part of the card or the mermaid node, it predates this drift, and it states something about the Kernel rather than about a spec namespace. The ruling's Execution paragraph scopes the work to "the card and the mermaid node", so this PR leaves:147untouched and hands the question up rather than deciding it. If the intended reading of "drop Multi-tenancy from the page" includes that bullet, it is a one-line follow-up.ArrowRightandCheckCircleimports are unrelated to this change and were left alone.Generated by Claude Code