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.

QUICK_REFERENCE.md 与 skills/objectui/guides/project-setup.md 仍在教「pnpm --filter PKG test」——正是 #3378 的假绿跑法 #3442

Closed

Description

@yinlianghui

(占位符写作 PKG,不用尖括号:GitHub 正文消毒会把「尖括号+字母」当 HTML 标签吃掉。)

现象

两处面向人和 agent 的入门文档,把 #3378 里那条会静默假绿的调用当作推荐用法写着:

  • QUICK_REFERENCE.md:26-27
    pnpm --filter @object-ui/console test     # Run just the console tests
    pnpm --filter @object-ui/core test        # Run a single package's tests
    
  • skills/objectui/guides/project-setup.md:309
    pnpm --filter @object-ui/core test        # Test single package
    

按 #3378 的机制,这条命令让 vitest 的 root 落在包目录,根级 projects 的 include
(packages/** 等)相对它匹配不到任何文件,只有以绝对路径引进来的 apps/console project
仍解析成功 —— 于是跑的是 @object-ui/console 的 22 个文件、报 Test Files 22 passed (22),
而文档里点名的那个包(core / console 自己的包级语义)一个都没按预期跑。注释写的
"Run a single package's tests" 与实际行为不符。

为什么现在必须改

PR #3437(fixes #3378 / #3288)给 vitest.config.mts 加了 invocation guard:vitest root
不是仓根即拒绝。该 PR 合并后,上面这三行文档命令会直接以非零退出,并打印指向正确跑法的
报错。也就是说:

  • 合并前:文档教的命令假绿(更糟);
  • 合并后:文档教的命令报错(不再骗人,但文档仍然是错的)。

两种情况下这几行都该换成仓根的正确跑法:

pnpm exec vitest run packages/PKG/          # 只跑一个包
pnpm exec vitest run packages/PKG/src/FILE.test.ts
pnpm test                                   # 全量

范围

我这次的 dispatch 文件围栏只包含 AGENTS.md 的测试章节(已写入正确跑法与两条陷阱),
QUICK_REFERENCE.md 与 skills/** 不在围栏内,因此不在 PR #3437 里顺手改 —— 单开此条。

顺带:content/docs/guide/plugin-development.md:422 的 pnpm vitest run packages/plugin-board
是正确形状(仓根 + 路径),无需改。

相关

(说明:提交前尝试搜索重复 issue 时 GitHub API 多次返回 rate limit,只跑成功了两次关键词检索,
均无命中。若已有孪生条目,请直接关掉这条。)

Activity

  1. yinlianghui commented on Aug 6, 2026

    @yinlianghui
    ContributorAuthor

    分诊:pm:queue(objectui 分片 PM,会话 session_01GTRjn8xBqp75dk7kFupVRt)。

    判级理由:PR #3437 落地后,QUICK_REFERENCE.md:26-27 与 skills/objectui/guides/project-setup.md:309 教的 pnpm --filter PKG test 从「静默假绿」变成「响亮报错」——文档必须跟上,否则每个照文档操作的人先撞守卫再来查为什么。纯文档小修,content/docs/guide/plugin-development.md:422 已核为正确形状无需动。

    派发前置:等 PR #3437 MERGED 后从新 main 切分支(改法以守卫打印的规范命令为准:仓根 pnpm exec vitest run <paths>);文件面 QUICK_REFERENCE.md + skills/objectui/guides/,与业务包互不相交。


    Generated by Claude Code

  2. self-assigned this
    on Aug 6, 2026
  3. yinlianghui commented on Aug 6, 2026

    @yinlianghui
    ContributorAuthor

    认领:objectui 分片 PM 并行批第 36 单
    会话:session_01GTRjn8xBqp75dk7kFupVRt
    分支:claude/issue-3442-docs-vitest-invocation
    Worktree:objectui-issue-3442
    文件面:QUICK_REFERENCE.md、skills/objectui/guides/project-setup.md(仅测试命令行及其注释)。前置已满足:PR #3437 已 MERGED(main@34a3badc9),守卫在岗,改法以其打印的规范命令为准。
    与同批第 34 单(app-shell/e2e)、35 单(collaboration+语言包)互不相交。


    Generated by Claude Code

  4. yinlianghui commented on Aug 6, 2026

    @yinlianghui
    ContributorAuthor

    验收:ACCEPT → PR #3443(并行批第 36 单,会话 session_01GTRjn8xBqp75dk7kFupVRt),已转 ready + auto-merge(docs-only,ci/lint 对 *.md path-ignore,已跑的检查零失败)。

    复核要点:两处文档共 3 个违规点全改,且文档命令对着真树验证过(pnpm exec vitest run packages/core/ 收集 64/64 本包文件、vitest list 零外包混入;反向验证守卫对旧写法 exit 1);正确用法的兄弟行(--filter build/dev)未误伤;残留的 3 个 --filter.*test grep 命中全部是「点名禁用形式」的说明性散文,符合 AGENTS.md 先例——如实报告而非声称零命中。dev 还自纠了一处会被守卫反咬的字面示例路径。docs:check-links 的既有红已查明系 #3292/#3213 在追踪,未重复立单。


    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

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions