Uh oh!
There was an error while loading. Please reload this page.
docs(license): name the Console UI component and drop the retired Account UI - #114
Merged
Merged
Conversation
…ount UI Both "Console" occurrences on this page sit in licence-scope enumerations — lists of the artifacts Apache-2.0 covers. #79 retired "Console" as the name of the end-user surface but deliberately kept the names of things the product ships, so the fix is to disambiguate rather than rename: line 16's bare "Console" becomes "Console UI", the spelling architecture.mdx and configure/system-settings.mdx were kept at. Line 95 claimed two UIs ship. Since ADR-0048 only one does: the standalone account-portal SPA was removed and Account is now an app inside the Console UI (com.objectstack.account), which is what #108 recorded in the glossary. An MDX comment records the decision next to the lines, so the next corpus grep for the retired name finds the reasoning in the same output. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Yaqu7kmKZM3tRPd9Y4xivo
os-elon
marked this pull request as ready for review
August 18, 2026 16:33
Uh oh!
There was an error while loading. Please reload this page.
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 freeto 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#107
Both occurrences of "Console" on
content/docs/resources/license.mdxsit inlicence-scope enumerations — lists of the artifacts Apache-2.0 covers. #79
retired "Console" as the name of the end-user surface but deliberately kept the
names of things the product ships (a plugin id, the line the CLI prints, a box
in an architecture diagram). A licence grants rights over artifacts, so this page
is in the kept category: the fix is to disambiguate, not rename, and neither
line is rewritten to "the UI" or to "ObjectOS".
Line 16 — bare "Console" becomes "Console UI"
The card's real finding is that bare "Console", sitting between "production
runtime" and "the MCP server", reads as the surface even though its neighbours
are all components. Naming the component fixes that. The spelling is not
invented here — it is the one #79 kept, verified on
origin/main@89371c6:It is also what the framework calls its own artifact —
packages/cli/src/commands/serve.ts:224: "Mount the Console UI at/_console/when the package is installed".
Line 95 — "the Console and Account UIs" claims two UIs; one ships
This is where the diff departs from the card, because one of the card's inputs
moved after it was filed. The card reasoned that line 95 reads as component
names partly because "Account UIs" is "a name #79 kept everywhere". #108 has
since ruled that Account is not a surface, and the upstream source is
explicit that the separate UI is gone —
packages/apps/account/src/index.ts(objectstack@f6c71ea):This repository's own glossary already agrees, as landed by #111: account
self-service is "
/_console/apps/account— the Account app, alongside Setup at/_console/apps/setup".So the enumeration named a shipped artifact that no longer exists, in the one
document where the enumeration is the point. The line now reads "the Console
UI, including the Account app it serves" — the same two things, spelled the way
they actually ship, with no new nouns and no surface vocabulary. Deliberately
not done: anything outside this file. This is not an Account-vocabulary
sweep; the wider question of how the corpus talks about Account is reported, not
answered here.
The durable half: a note the next grep will find
The card asks for the decision to be recorded so the next corpus grep for the
retired name does not reopen it a third time. An MDX comment now sits above the
first occurrence — and because it contains the word "Console" itself, the same
grep -n Consolethat finds the two lines finds the reasoning interleaved withthem:
MDX comments had zero precedent in
content/docs/, so the construct itselfwas verified rather than assumed: the production build is green and the comment
is absent from the prerendered page
(
apps/docs/.next/server/app/en/docs/resources/license.html, 0 matches for thenote's text). Both prose lines render exactly as written.
Gates
All run in the task worktree at
2d00b32, which is byte-identical to theverified tree (
git status --porcelainempty). Derived from.github/workflows/ci.ymland.github/workflows/translations.yml, not from asupplied list.
pnpm install --frozen-lockfilepnpm turbo run type-check --continuepnpm turbo run buildpnpm turbo run test--forcere-run, not the replay)check-translations.mjscheck-translation-ownership.mjs --actor os-elon --files ...TRANSLATION_BOT_LOGINunset, per #68)check-translation-output.mjs --files ...check-translation-output.mjs --self-testtestcache-hit on the first run and was re-run with--force; the hit iscorrect by design —
turbo.jsonscopestestto.github/scripts/**andi18n.ts, not tocontent/docs/**, and the task is the validator self-test.type-checkandbuildboth missed cache, so no content-only green wasreplayed.
The output gate's report is unchanged by this PR except for five length ratios
on this page's locale siblings (for example
license.de.mdx0.52x → 0.48x);no finding was added or removed. That drift is itself a small gate defect,
filed as #115 — where a second, sharper leg of the same blindness is measured:
the blocking
unsaferule reads MDX comments too, so text no reader can see canfail it.
Scope notes
blocking, per
AGENTS.md. No page is retired or renamed, so no sibling isdeleted.
why.mdx,architecture.mdx,configure/system-settings.mdx,quickstart.mdx) are untouched.os startoutput does not match what CLI 17.0.0 prints, and the plugin count is wrong #94 already covers the staleos startbanner onquickstart.mdx(which still lists "Account UI", and"Console UI" twice) — nothing new filed for it.
check-translation-output.mjsmeasures MDX comments as if a reader saw them #115 is the one new finding.Generated by Claude Code