mirror of
https://github.com/LHY0125/PathEditor.git
synced 2026-06-29 01:45:54 +08:00
Serendipity
21da3b2930
- 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>
36 lines
955 B
Markdown
36 lines
955 B
Markdown
# 未修复问题清单
|
|
|
|
> 从 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 代码审查*
|