Skip to content

revenue/index 的《Where to find things》(:48-:60) 四处与 crm.app.ts 不符:虚构 **Products** 分组、Approvals 分组里两个不存在的条目 + 一个错 label #943

Closed

Description

@yinlianghui

发现自 #938 / PR #942 的实施过程(为 marketing/index 补 Products 条目时,需要确认产品目录在文档 IA 里的归属,顺手读了 content/docs/revenue/index.mdx 的同名小节)。

基线 origin/main = 11f62429(平台 17.0.0-rc.3)。本单不与 #938 / PR #942 重叠:那单只动 content/docs/marketing/index*.mdx 的《Where to find things》一节,完全不碰 revenue/。

事实

content/docs/revenue/index.mdx:48-:60(三语同址同形,zh-Hans/zh-Hant 见 :50/:52/:56-:60):

## Where to find things

In the Enterprise CRM app, the **Products** group contains:

- **Products** — the catalog

And the **Sales** group contains **Contracts** (because they belong to the sales process).

Approvals live in the **Approvals** group:

- **Approval Requests** — what's pending
- **Action History** — who approved what, when
- **Processes** — admin view of the approval rules

逐条对 src/apps/crm.app.ts:

文档断言 源码实况
存在 Products 分组 不存在。 crm.app.ts 里没有 group_products。nav_product 是 group_marketing 的 child(:126)——参见 #938
Sales 分组含 Contracts ✅ 成立,nav_contract(:60)
Approvals 分组含 Approval Requests label 不对。group_approvals(:160)只有一个 child nav_approval_requests,其 label 是 Inbox(:165)
Approvals 分组含 Action History 不存在。 该分组无此条目,全仓也无此 label
Approvals 分组含 Processes 不存在,且是被刻意删掉的:crm.app.ts:166-:169 的注释写明「no sys_approval_process object exists in any installed plugin, so the old item's requiresObject guard hid it on every install, forever」

即:group_approvals 真实只有 1 项(label Inbox),文档写了 3 项且无一 label 正确。

为什么要紧

与 #927 / #932、#938 / #942 同一族,但两个方向同时踩:既虚构了一个分组(Products),又在 Approvals 下列了两个永远点不到的条目。后果是照文档找不到路——尤其 Products 分组这一条会把读者送去一个不存在的地方,而真正的入口在 Marketing 分组下、且该分组默认折叠(GroupNavItemSchema.expanded 默认 false,group_marketing 未声明该键)。

边界与口径建议

Refs #927 #932 #938 #942

Activity

  1. added
    documentationImprovements or additions to documentation
    pm:queueReady for the PM dispatch loop
    pm:dispatchedDispatched to a dev agent by /pm-dispatch
    and removed
    pm:queueReady for the PM dispatch loop
    on Aug 6, 2026
  2. self-assigned this
    on Aug 6, 2026
  3. yinlianghui commented on Aug 6, 2026

    @yinlianghui
    CollaboratorAuthor

    🔒 认领(/pm-dispatch R31)— session session_01VHrPAGEgFDoHjphqYG4BMa,分支 claude/issue-943-revenue-nav。

    文件面:content/docs/revenue/index{,.zh-Hans,.zh-Hant}.mdx 的《Where to find things》(基线 :48-:60)——fresh main 重定位。

    裁定:①stale-premise 复核 crm.app.ts(虚构 Products 分组的真身在 group_marketing:126、Approvals 组的 Action History/Processes 不存在且 :166-169 注释刻意删除、真实条目 label 是 Inbox);②沿 #932/#942 先例:分组 children 全量核对写实 + 不静默删名(Processes 被刻意删的原因如实写:无 sys_approval_process 对象);③#595/#596 不预判;src/ 零改动;三语同步;zh 内链不带锚。


    Generated by Claude Code

  4. yinlianghui commented on Aug 6, 2026

    @yinlianghui
    CollaboratorAuthor

    ✅ 验收通过 —— PR #953 已转 ready 并挂 auto-merge(CI 8/8 全绿实核)。

    验收要点:四处断言全成立且行号未漂;Action History 的处理最见功力——从已装 plugin-approvals 枚举六个 sys_approval_* 对象证明「数据是真的但无导航入口」,按此实况写而非简单说不存在;Processes 保留名字 + :166-169 刻意删除的原因(requiresObject 守卫)如实写入;Inbox label 纠正并补 zh 语言包译名「待我审批」;两分组默认折叠(expanded 默认 false)的失效机制一并写实,与 #942 同手法。#595/#596 未预判,落地行零回退。

    越界发现处置:#960(quick-tour 导航表整表失实——8 行里 4 个分组不存在、真实 3 组零出现)与 #963(revenue/approvals 复刻假条目 + 三个双面零命中的视图名,与 #953 写实后的 index 页自相矛盾)均已复核成立,标 pm:queue,R32 候选。


    Generated by Claude Code

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

documentationImprovements or additions to documentationpm:dispatchedDispatched to a dev agent by /pm-dispatch

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions