Skip to content

三处 docblock 在 #3660 落地后变成事实错误:仍写「setup 左导航 Datasources 指向 component/metadata/resource 别名」 #3666

Closed

Description

@yinlianghui

越界发现,记录于 #3660(4 处生产端改发规范拼法)实施期间。这三处不是既有陈旧,而是 #3660 的 PR 一旦合并就会当场制造出来的错误 —— 它们描述的正是该 PR 改掉的两个 sidebar 字面量。按纪律只报不改(#3660 的派发文件面只有两个 sidebar + SystemHubPage + 测试 + changeset,其中 console/AppContent.tsx 另挂 ⛔),未认领,留 PM 分诊。

观察类:纯注释,用户看不到,运行时行为无关。故打 finding、不打 pm:queue。

事实

#3660 把两个 sidebar 的 sys-datasources 由 /apps/setup/component/metadata/resource?type=datasource 改为规范的 /apps/setup/metadata/datasource。以下三段 docblock 以现在时断言该导航项仍指向别名:

# 位置 现有措辞(#3660 后为假)
1 packages/app-shell/src/views/metadata-admin/datasource/register.ts:11-13 「The setup left-nav "Datasources" item still points at the older …/component/metadata/resource?type=datasource spelling.」
2 packages/app-shell/src/views/metadata-admin/datasource/DatasourceResourcePage.tsx:9-11 「the setup left-nav "Datasources" item points at …/component/metadata/resource?type=datasource (rewritten by LegacyMetadataRedirect)」
3 packages/app-shell/src/console/AppContent.tsx:650-651 零应用分支注释:「sys-datasources points straight at …/component/metadata/resource?type=datasource」

1 和 2 是 PR #3658(修 #3639)昨天刚写下的 —— 当时完全正确,#3660 使其失效。3 是 PR #3636(修 #3610)写的,叙述的是 #3610 当时的成因,可辩护为历史陈述,但同样用现在时,读起来是当前事实。

为什么值得单独记一笔,而不是"注释洁癖"

这正是 #3639 立单的原因本身。#3658 的正文写得很清楚:被订正的四段 docblock 把别名说成 "the engine route",而「#3610 的派发单正是顺着这句话推出 component/... 才是规范拼法的,与路由表恰好相反」。同一个坑在同一条链上已经付过一次钱。留下三段新的现在时假陈述,下一个 agent grep component/metadata 找生产端时,会从 1 和 2 得到"sidebar 还在喂别名"的结论,而实际上已经没有任何生产端在喂了。

建议处置(两条路,PM 择一)

我(#3660 的执行 agent)倾向 A,但没有自作主张:派发单的文件面写了「越界即停」,而这两个文件不在其中。PM 若裁决 A,我可在 #3660 的分支上直接补一次提交。

别名路由本身不受影响

三段注释描述的别名路由仍然存在且仍然转发 —— #3660 明确不主张删别名(书签/外链)。要改的只是"谁在喂它"这一句。

关联

已就关键词(component/metadata / sys-datasources / docblock / stale comment / DatasourceResourcePage / register.ts)与上述文件路径搜过本仓开放 issue 与 PR,无同源单(命中的 #3655 / #3661 / #3561 均属不同问题)。


Generated by Claude Code

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions