Phase 4 Testing: 收尾清理 #97

Closed
opened 2026-05-07 16:23:06 +00:00 by xingyue · 0 comments
Owner

验证目标

清理残留问题:重复 pathExists、首行解析去重、argv 统一、废弃参数清理、deprecate help 命令、模板分离、颜色工具提取。

测试步骤

  • Step 1: bundle-store.ts 不再有私有 pathExists

    grep 'async function pathExists' packages/cli-workflow/src/bundle-store.ts
    

    预期: 无匹配(改为 import from fs-utils)

  • Step 2: thread-scan.ts 无重复首行解析

    grep -c 'split.*\\n.*\[0\]' packages/cli-workflow/src/thread-scan.ts
    

    预期: ≤ 1(提取为通用函数)

  • Step 3: argv 解析独立文件

    grep 'parseAddArgv\|parseForkArgv' packages/cli-workflow/src/commands/workflow/add.ts packages/cli-workflow/src/commands/thread/fork.ts 2>&1
    

    预期: 函数定义不在命令文件内(已移到独立 argv 模块)

  • Step 4: cmd-cas 无 _threadId 残留

    grep '_threadId' packages/cli-workflow/src/commands/cas/*.ts
    

    预期: 无匹配

  • Step 5: help 命令标记 deprecated

    bun run packages/cli-workflow/src/cli.ts help 2>&1
    

    预期: 输出 deprecation warning,指向 skill

  • Step 6: init/templates.ts 独立存在

    ls packages/cli-workflow/src/commands/init/templates.ts
    

    预期: 文件存在,包含模板字符串函数

  • Step 7: cli-color.ts 存在

    ls packages/cli-workflow/src/cli-color.ts
    

    预期: 文件存在,包含 shouldUseColor 等

  • Step 8: 全量验证

    cd packages/cli-workflow && bun run check && bun run build && bun test
    

    预期: 全部 pass,无 lint warning

验证完成标准

所有 checkbox 打勾
代码整洁,无已知技术债

Ref

RFC #93, Phase 4

## 验证目标 清理残留问题:重复 pathExists、首行解析去重、argv 统一、废弃参数清理、deprecate help 命令、模板分离、颜色工具提取。 ## 测试步骤 - [ ] **Step 1: bundle-store.ts 不再有私有 pathExists** ```bash grep 'async function pathExists' packages/cli-workflow/src/bundle-store.ts ``` **预期:** 无匹配(改为 import from fs-utils) - [ ] **Step 2: thread-scan.ts 无重复首行解析** ```bash grep -c 'split.*\\n.*\[0\]' packages/cli-workflow/src/thread-scan.ts ``` **预期:** ≤ 1(提取为通用函数) - [ ] **Step 3: argv 解析独立文件** ```bash grep 'parseAddArgv\|parseForkArgv' packages/cli-workflow/src/commands/workflow/add.ts packages/cli-workflow/src/commands/thread/fork.ts 2>&1 ``` **预期:** 函数定义不在命令文件内(已移到独立 argv 模块) - [ ] **Step 4: cmd-cas 无 _threadId 残留** ```bash grep '_threadId' packages/cli-workflow/src/commands/cas/*.ts ``` **预期:** 无匹配 - [ ] **Step 5: help 命令标记 deprecated** ```bash bun run packages/cli-workflow/src/cli.ts help 2>&1 ``` **预期:** 输出 deprecation warning,指向 skill - [ ] **Step 6: init/templates.ts 独立存在** ```bash ls packages/cli-workflow/src/commands/init/templates.ts ``` **预期:** 文件存在,包含模板字符串函数 - [ ] **Step 7: cli-color.ts 存在** ```bash ls packages/cli-workflow/src/cli-color.ts ``` **预期:** 文件存在,包含 shouldUseColor 等 - [ ] **Step 8: 全量验证** ```bash cd packages/cli-workflow && bun run check && bun run build && bun test ``` **预期:** 全部 pass,无 lint warning ## 验证完成标准 ✅ 所有 checkbox 打勾 ✅ 代码整洁,无已知技术债 ## Ref RFC #93, Phase 4
Sign in to join this conversation.
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: uncaged/workflow#97