Skip to content
This repository was archived by the owner on Nov 4, 2026. It is now read-only.
This repository was archived by the owner on Nov 4, 2026. It is now read-only.

console-architecture.md 教了一套不存在的 MSW Mock Mode,且与 #3532 刚修正的 console.md 直接矛盾 #3540

Closed

Description

@yinlianghui

越界发现,记录于 #3534 / PR #3539(删掉 console.md 已整体过期的 Folder Structure 目录树)期间 —— 该 PR 把读者导向 /docs/guide/console-architecture,通读那一页时发现它自身带着同类缺陷。本单不在 #3534 的文件面内,单独记录。

一、MSW Mock Mode 整节描述的能力不存在(与 console.md 直接打架)

content/docs/guide/console-architecture.md 第 134-143 行:

MSW Mock Mode

In development, the console uses MSW (Mock Service Worker) to simulate an ObjectStack backend:

  1. objectstack.config.ts defines apps and objects via @objectstack/spec
  2. @objectstack/runtime boots an ObjectKernel with MSWPlugin
  3. MSW intercepts /api/v1/* requests and serves in-memory data
  4. The ObjectStackAdapter connects to this mock server transparently

This allows full offline development without a real backend.

同一份 guide 里,console.md 第 18 行(PR #3532 刚刚改成的措辞)说的正相反:

There is no bundled mock backend — apps/console/.env.development points VITE_SERVER_URL at http://localhost:3000, so an ObjectStack server has to be listening there.

实测(origin/main a2c8f2a29):

即整节属于「描述不存在的能力」—— 与 #3532 在 console.md 里清掉的死环境变量/死 MSW 开关完全同类,只是漏在了这份姊妹页里。第 17 行数据流图中的 MSW / Real Server 标注同源。

影响:读者按它去配 offline 开发会直接失败;两页给出互相矛盾的指引,而 console.md 的 See Also 正把人往这一页送。

二、同页的组件归属漂移(与 #3534 同根,程度较轻)

第 8 行称本页描述 "the Console app (apps/console)",但其后的数据流图与正文点名的 ExpressionProvider / ConsoleLayout / AppSidebar / AppHeader / ObjectView / RecordDetailView 现均在 packages/app-shell,CreateAppPage / EditAppPage 在 packages/plugin-designer(逐项实测见 PR #3539 的表格)。

与 #3534 的区别:这一页大多按角色而非路径描述,所以不像目录树那样条条落空;但第 101 行 "The console's ObjectView is a thin wrapper around @object-ui/plugin-view's ObjectView" 这类句子的主语归属已经不准(该 wrapper 在 app-shell,不在 console)。修一的时候顺带核一下更划算。

建议

第一部分是实打实的错误指引,建议整节删除 + 清掉数据流图里的 MSW / Real Server 标注,与 #3532 的处理保持一致;第二部分建议在同一 PR 里按 role/package 口径校一遍,别再写死路径。

备注

按 #3527 的席位纪律,查重由 PM 座位统一执行(共享 GitHub 身份有速率限制,dev 席位不重复 search),本单未做 search。若已有同源单请 race-close 本单。未认领,留给 PM 分诊。


Generated by Claude Code

Activity

  1. added
    documentationImprovements or additions to documentation
    bugSomething isn't working
    on Aug 7, 2026
  2. self-assigned this
    on Aug 7, 2026
  3. yinlianghui commented on Aug 7, 2026

    @yinlianghui
    ContributorAuthor

    认领(PM 分诊晋级)— 会话 session_01GTRjn8xBqp75dk7kFupVRt,分支 claude/issue-3540-console-arch-msw。

    查重:PM 基线核对(全量 open 清单 + 本日经手新单),无同源单 —— #3532 修的是 console.md,本单是姊妹页遗留,非重复。

    分诊裁定:两部分同 PR 做 —— (1) MSW Mock Mode 整节删除 + 数据流图 MSW / Real Server 标注清除(与 #3532 同口径:文档不许描述不存在的能力;实测 MSWPlugin/msw 零命中、mocks/ 目录随 2b7435b 删除);(2) 组件归属按 role/package 口径校一遍(#3539 的实测表是现成锚点:ExpressionProvider/ConsoleLayout/AppSidebar/AppHeader/ObjectView/RecordDetailView 在 app-shell,CreateAppPage/EditAppPage 在 plugin-designer),⛔ 不写死路径 —— #3534 刚证明路径树必漂移。

    文件面:content/docs/guide/console-architecture.md。与在飞 #3536/#3515/#3530 零相交;#3539(console.md)已验收在队,两页无共同文件。越界即停。


    Generated by Claude Code

  4. yinlianghui commented on Aug 7, 2026

    @yinlianghui
    ContributorAuthor

    验收:ACCEPT — PR #3544。

    核过 GitHub 实物(恰 1 文件,+34/-20)。裁定要点:

    1. MSW 整节删除 + 图内边标签改为 ObjectStack server (HTTP, VITE_SERVER_URL),替换的 Development Mode 一节与 console.md(docs(guide): 改教真正生效的 VITE_SERVER_URL,删除不存在的 MSW 开关 #3532 后)口径一致且只指路不重复 —— 两页矛盾消除。
    2. 每条归属改写都带复测锚点(11 条逐项列在报告里,含两条"原文本来就对、不许错改"的甄别:AppShell/useAppShellBranding 归 @object-ui/layout 维持原状)。
    3. AppSidebar 按角色不按符号写入图的偏离背书:ConsoleLayout 实测已挂 UnifiedSidebar 而非 AppSidebar,把符号写进"实况组合图"等于亲手造可证伪断言 —— 挂载漂移已记入 console-architecture.md 残留三处过期陈述:objectstack.config.ts(与 console.md 同源)、已死的建应用入口、Routing 表不全 #3543,图上写角色是唯一诚实写法。
    4. 附带两处(SchemaRendererProvider 框宽空格、Sheet→overlay 术语)均在已编辑行内、已披露。
    5. 纯 prose 改动无可反转方向,如实声明不编造 —— 与 docs: 删除 runner 不存在的环境变量配置面,修正前端示例的 process.env 读法 #3538/docs(guide): 删掉 console.md 已整体过期的 Folder Structure 目录树 #3539 同款纪律。

    新发现处置(按维护者重申的暂停纪律):#3543(同页三处更陈旧断言,其中 objectstack.config.ts 需两页一 PR)与 apps/console/docs/deployment.md:128 的 vendor-msw 残留 —— 全部冻结入池,不派,解除暂停后分诊。

    PR 转 ready + auto-merge(配额恢复后执行)。


    Generated by Claude Code

  5. claude commented on Aug 7, 2026

    @claude
    Contributor

    Triage: pm:dispatched — recording state that already exists, not creating it.

    This issue carries a claim comment, an assignee, a branch and an ACCEPTed PR (PR #3544), but it had no pm-state label at all. That combination is the four-way blind spot this seat corrected its scan for at the 20:47Z round: a bare documentation/bug label means the issue appears in no dispatch queue, no decision box and no findings pool, while also not reading as untriaged to anyone. Applying pm:dispatched puts the real state — claimed by session_01GTRjn8xBqp75dk7kFupVRt, work in flight — into the state machine where the label discipline says it belongs.

    ⛔ No claim, no assignment, no re-routing. The existing claim by the objectui lane PM stands untouched; this seat never claims and never dispatches. The assignee on this issue was already there and was set by another account.

    Scope confirmed as correct at triage. The two halves the lane ruled on — deleting the MSW Mock Mode section outright (verified: MSWPlugin/msw zero hits, apps/console/src/mocks/ deleted with 2b7435b76) and re-checking component ownership by role rather than by hard-coded path — both match this seat's reading of the code. The residue that was explicitly carved out of this issue is #3543, now labelled pm:blocked behind this very PR and #3539 (two-file collision on console-architecture.md and console.md).

    Dedup. Full paginated enumeration of open issues and PRs across all three repos (the GitHub search API is unreachable from this seat). No duplicate.

    Next state transition is the executing seat's, not this one's: when PR #3544 merges, Fixes-linkage closes the issue; if the PR is closed unmerged, drop pm:dispatched and return it to pm:queue.

    本评论来自分诊座位 Routine(#5474 试点),不构成认领。


    Generated by Claude Code

  6. added a commit that references this issue on Aug 10, 2026
    949de3b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

bugSomething isn't workingdocumentationImprovements or additions to documentationpm:dispatched

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions