Skip to content

test: add coverage for issue #91 runtime modules#101

Open
knqiufan wants to merge 3 commits into
stepfun-ai:mainfrom
knqiufan:codex/issue-91-test-coverage
Open

test: add coverage for issue #91 runtime modules#101
knqiufan wants to merge 3 commits into
stepfun-ai:mainfrom
knqiufan:codex/issue-91-test-coverage

Conversation

@knqiufan

@knqiufan knqiufan commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

关联 Issue

Closes #91

变更类型(勾选所有适用项)

  • fix bug 修复
  • test 仅测试

影响范围

  • 仅涉及测试与一处内部的跨平台回退一致性修复

变更说明(why,而非 what)

Issue #91 指出的七个关键运行时模块此前没有进入覆盖率统计,导致全局覆盖率无法反映这些模块的实际测试情况。本 PR 将它们纳入覆盖率报告,并为 AgentTeam、Harness、ToolRuntime、原生文件与 Shell 工具、CodingBridge 及 ConversationMemory 的关键路径补充单元测试。

本次 CI 失败修复:Grep 在没有 rg 的环境会使用 JS 回退。该回退此前将 include: *.ts 仅匹配根目录文件,和 ripgrep 的任意目录深度文件名匹配行为不一致;现已修正并由新增用例覆盖。

测试计划

  • 单元测试:pnpm test
  • 覆盖率:pnpm test:coverage
  • 手工验证:检查 GitHub Actions 失败日志,并在 Windows 本地复现 Shell 工具测试

自检清单

  • 本地 pnpm check 已通过(lint / dep-guard / deadcode / tsc / format / test)
  • 新加文件配套加入了 .test.ts
  • 平台相关用例使用了 vi.skipIf,未硬编码平台判断
  • 新增被覆盖统计的模块已同步加入 vitest.config.tscoverage.include
  • 不包含二进制、构建产物、密钥
  • 未使用 --no-verify 绕过钩子

@github-actions github-actions Bot added area/build scripts, .github, build/config files area/core packages/core area/realtime packages/realtime + extensions/realtime-* labels Jul 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/build scripts, .github, build/config files area/core packages/core area/realtime packages/realtime + extensions/realtime-*

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[bug] 大量关键模块无测试覆盖

1 participant