Skip to content

Commit f8f4095

Browse files
docs(ui-skills): the console guide names apps/console as the one host (#25922) (#26484)
<!-- claude-projects-attribution: {"github_login":"os-steve"} --> _Requested by **Steve** · [project thread](https://claude.ai/code/project/chan_01FdL1XeUyvVNxLreh3F4Qi9?thread=cmsg_01FdL1XeUyvVNxLreh3F4Qi97Wh4Pf8UgMzHvahNM6Rh28)_ Part of #25922 (the governed half of the unpublish item: one guide bullet. The unpublish itself is #26483, which lands after this.) ## 维护者速读 - 改了什么:`ui/.claude/skills/objectui-contributor/guides/console-development.md` 里的一条要点。原文让第三方去 fork `examples/console-starter`,改成「`apps/console` 是唯一宿主」:`@object-ui/app-shell` 已是私有包,没有可 fork 的宿主模板。 - 为什么改:#26483 按裁决 6431234244 删除 `examples/console-starter`,这一行随之成为死路径,`check-skills-paths`(Test UI)会红。`ui/.claude/**` 是 Tier H 受管面,所以单独成 PR、先落地。 - 风险与回滚:只改文档一处(+5/-4),无代码;回滚 = revert。 - 先看:`ui/.claude/skills/objectui-contributor/guides/console-development.md` 第 39 行起的那一条。 - 两分钟验证:`node ui/scripts/check-skills-paths.mjs`,预期 exit 0(本分支与 #26483 合并后都应为 0)。 - 需要你做的:批准本 PR(Tier H,GOVERNED_APPROVERS)。 ## What changes One bullet in the contributor guide's "where symbols live" note. It said third parties fork `examples/console-starter`, which wires `ConsoleShell` and host routes. It now says `apps/console` is the one host: `@object-ui/app-shell` is a private workspace package, and there is no host template to fork. A surface belongs in `app-shell` (or an app's `navigation[]` metadata) when it is shell or view code, and in `apps/console` when it is this deployment's own wiring. ## Why #26483 deletes `ui/examples/console-starter` under ruling 6431234244 on #25922 (「本项目的组件就是为 objectos console服务的,可定制console暂时没有业务需求,可以废弃」「相关例子也可以删除」). Once it does, this line names a dead path and `check-skills-paths` reds. The file is a Tier H surface, so it lands on its own, first, and #26483 then merges `main`. ## Tests None: a prose edit. No ablation (docs only). Publishes nothing, so this PR carries `skip-changeset` (`ui/.claude/**` ships in no package). ## Verification At `54ecffdf`, with `pnpm install --frozen-lockfile` in this branch's own worktree: - `node ui/scripts/check-skills-paths.mjs`: exit 0 on this branch. The same patch applied to #26483's head gave `check-skills-paths.test.ts` 35/35 and the gate exit 0. - `node scripts/pm/dispatch-gates.mjs --commands --repo objectstack-ai/objectstack`: 12 families, each run after the last commit with its exit code captured, all exit 0; `--ran`: `12 derived famil(ies) accounted for — 12 run, 0 NOT-MEASURED`. ## Acceptance notes - Governed (Tier H): no ready, no queue, no auto-merge before a GOVERNED_APPROVERS approval. 🤖 Generated with [Claude Code](https://claude.com/claude-code) https://claude.ai/code/session_01ADHkmQmtHYkx1Ve9ob2ikN --- _Generated by [Claude Code](https://claude.ai/code/session_01ADHkmQmtHYkx1Ve9ob2ikN)_ Co-authored-by: Claude <noreply@anthropic.com>
1 parent c91544a commit f8f4095

1 file changed

Lines changed: 5 additions & 4 deletions

File tree

‎ui/.claude/skills/objectui-contributor/guides/console-development.md‎

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -36,10 +36,11 @@ Working on the Console application (`apps/console/`), this repo's reference admi
3636
>
3737
> - **Look in `packages/app-shell` first.** Most symbols you want to change are
3838
> there, not under `apps/console/src/`. Grep before you assume a path.
39-
> - **Third parties fork the template, not this app** — `examples/console-starter`
40-
> wires `ConsoleShell` + host routes in ~100 LOC. Adding a surface to
41-
> `apps/console` only benefits this repo's own deployment; adding it to
42-
> `app-shell` (or to an app's `navigation[]` metadata) benefits every host.
39+
> - **`apps/console` is the one host.** `@object-ui/app-shell` is a private
40+
> workspace package (no longer published to npm) and there is no host
41+
> template to fork, so a surface belongs in `app-shell` (or in an app's
42+
> `navigation[]` metadata) when it is shell or view code, and in
43+
> `apps/console` when it is this deployment's own wiring.
4344
4445
## Console architecture overview
4546

0 commit comments

Comments
 (0)