Files
PathEditor/docs/REMAINING-ISSUES.md
Serendipity 21da3b2930 fix: v5.1 代码审查修复 — ESLint/CSV/测试隔离/CLI 去重
- ESLint: 迁移到 flat config ignores,删除已废弃的 .eslintignore
- CSV: Rust/TS 格式对齐,统一 type,path,enabled 3 列
- JSON: 导入导出统一为 {path, enabled} 对象格式
- scanner: 移除未使用的 max_threads 死代码 + TempDirGuard 测试清理
- profiles: rename_profile 添加目标存在检查
- CLI: 抽取 load_operate_save helper,简化 cmd_remove/cmd_edit
- PathTable: 抽取 usePathValidation hook,消除 set-state-in-effect
- 测试隔离: disabled/profiles 通过 #[cfg(test)] 重定向到 temp dir
- toolchain: 新增 rust-toolchain.toml 固定 stable-x86_64-pc-windows-gnu
- docs: 更新 CLAUDE.md/README.md 测试计数 + 架构树

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
2026-05-30 17:31:04 +08:00

955 B

未修复问题清单

从 v5.1 全面代码审查中筛选,暂不修复,留待后续评估。


1. CLI main.rs 单体文件 (639 行)

严重级别: LOW
文件: cli/src/main.rs

问题: 所有 18 条 CLI 命令集中在一个文件中。

建议: 当前规模尚可维护,等到命令数超过 25 条或文件超过 1000 行时再拆分为 commands/ 子模块。


2. GUI 命令层零测试

严重级别: LOW
文件: gui/src/commands/*.rs (8 个文件)

问题: GUI 命令层是纯薄包装,无独立测试。

建议: 不值得投入 — 命令正确性由编译器类型系统保证,运行期由 57 个 core 测试 + E2E 覆盖。


已修复(本批次)

  • disabled.rs 测试写入真实文件#[cfg(test)] 条件编译重定向到 std::env::temp_dir()
  • profiles.rs 同款问题 → 同上

更新于: 2026-05-30 | 审查批次: v5.1 代码审查