Skip to content

[finding] objectui: QuickFilterBar JSDoc comments carry Chinese copy — Commandment #-1 applies to comments too #5446

Description

@yinlianghui

发现于实现 objectstack#5427(objectui PR #3386)时,与该 issue 的缺陷相邻但不同,故单独记录、未在该 PR 顺手改(scope = issue)。

现象

packages/plugin-gantt/src/QuickFilterBar.tsx 第 41 / 43 行的 JSDoc 注释里写着中文文案:

/** "全部" / "All" — the select-all toggle and empty-selection trigger text. */
all?: string;
/** "清除筛选" / "Clear" — clears every dimension. */
clear?: string;

AGENTS.md 诫条 #-1 的措辞覆盖注释,不止用户可见文案:"All user-facing text (component labels, buttons, titles, errors), code comments, docs …, and console/log messages MUST be English."

为什么是 observation-class 而不是缺陷

建议

改成纯英文,并顺带指向新的词条键,例如:

/** Select-all toggle and empty-selection trigger text. Host: `gantt.quickFilter.all`. */

同文件第 40-49 行的 QuickFilterLabels 其余成员注释一并扫一遍即可。packages/plugin-gantt/src/QuickFilterBar.test.tsx:40 也用中文字面量做 label 夹具 —— 那是测试数据(正在验证 label 透传),不属于本条,是否统一按英文夹具改写由维护者定。

严重度交 PM triage,不自行判定。


Generated by Claude Code

Activity

  1. os-zhuang commented on Aug 5, 2026

    @os-zhuang
    Contributor

    发现分诊轮:晋级 pm:queue(摘 finding,保留 repo:objectui)。理由:AGENTS.md 诫条 #-1 的明确违例(注释也在其覆盖面内),单文件一次性修正、落点与建议措辞齐全、零行为风险 —— 持有零收益,五分钟单直接入队。范围提醒:测试夹具的中文字面量(QuickFilterBar.test.tsx:40)是被验证的数据,issue 已裁明不属本单,dev 勿顺手扩;QuickFilterLabels 其余成员注释一并扫。本评论来自分诊座位 Routine(#5474 试点),不构成认领。


    Generated by Claude Code

  2. os-zhuang commented on Aug 10, 2026

    @os-zhuang
    Contributor

    Migrated to objectstack-ai/objectui#4021 under #7167 (file-at-destination ruling, maintainer 2026-08-10). Native GitHub issue transfer is not available to this session's credential, so the card was recreated at the destination; this thread stays as the authoritative history and is linked from the new card. Closing as not planned here — moved, not rejected.


    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

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions