fix: address Copilot review comments — 5 items resolved

JusterZhu · claude · JusterZhu · commit 9063613beb07 · 2026-06-16T15:06:23.000+08:00
1. uninstall.ts: Replace 'as Exclude' type assertions with runtime
   narrowing via Array.filter() and control-flow type narrowing
2. bowl_notice.cs.template: BackupDirectory now points to previous
   version (not current version), preventing auto-restore failures
3. BUGS.md: Reconcile Core+Bowl conflict guidance — v10.5.0-beta.4
   has no CS0433 conflict, Bowl is a standalone package
4. All 7 docs updated: remove 'Core+Bowl mutual exclusion' guidance,
   replace with 'no conflict, reference both' for v10.5.0-beta.4
5. cli/package.json: Clarify version is independent from skill version

Co-Authored-By: Claude &lt;noreply@anthropic.com&gt;
diff --git a/.claude/scripts/generate/templates/bowl_notice.cs.template b/.claude/scripts/generate/templates/bowl_notice.cs.template
@@ -1,5 +1,6 @@
-// ⚠️ Bowl 崩溃守护 — 使用 GeneralUpdate.Bowl，不单独引用 Core
+// ⚠️ Bowl 崩溃守护
 // dotnet add package GeneralUpdate.Bowl
+// NuGet v10.5.0-beta.4: Bowl 与 Core 无类型冲突，可按需同时引用
 // Refer to generalupdate-advanced/templates/BowlIntegration.cs for full example
 var context = new GeneralUpdate.Bowl.BowlContext
 {
@@ -9,7 +10,9 @@ var context = new GeneralUpdate.Bowl.BowlContext
     DumpFileName = "{{CLIENT_VERSION}}_fail.dmp",
     FailFileName = "{{CLIENT_VERSION}}_fail.json",
     FailDirectory = System.IO.Path.Combine({{INSTALL_PATH}}, "fail", "{{CLIENT_VERSION}}"),
-    BackupDirectory = System.IO.Path.Combine({{INSTALL_PATH}}, "{{CLIENT_VERSION}}"),
+    // BackupDirectory 必须指向前一版本的备份目录，而非当前版本
+    // 例如当前版本为 {{CLIENT_VERSION}}，备份来自上一版本的 BackupDirectory
+    BackupDirectory = System.IO.Path.Combine({{INSTALL_PATH}}, "previous_version_backup"),
     WorkModel = "Upgrade",
     DumpType = GeneralUpdate.Bowl.DumpType.Mini,
     AutoRestore = true,
diff --git a/.claude/skills/generalupdate-advanced/SKILL.md b/.claude/skills/generalupdate-advanced/SKILL.md
@@ -157,7 +157,7 @@ Console.WriteLine($"Result: Success={result.Success}, Restored={result.Restored}
 | `DumpType` | DumpType | Mini / Full | Full |
 | `OnCrash` | delegate | 崩溃回调 | null |
 
-> ⚠️ Bowl 和 Core 不能同时引用（CS0433）。使用 Bowl 时**只引用 `GeneralUpdate.Bowl`**，它传递依赖 Core 的所有功能。
+> ⚠️ NuGet v10.5.0-beta.4 中 Bowl 和 Core **无类型冲突**，可以同时引用。
 
 ---
 
@@ -332,7 +332,7 @@ var result = GeneralDrivelution.InstallDriver(driverPath);
 ## ✅ 高级定制验证清单
 
 ### Bowl 崩溃守护
-- [ ] 只引用了 `GeneralUpdate.Bowl`（不单独引用 Core）
+- [ ] 如果用 Bowl：项目中同时引用 `GeneralUpdate.Core` 和 `GeneralUpdate.Bowl`（v10.5.0-beta.4 无冲突）
 - [ ] `MonitorParameter` 的 `ProcessNameOrId` 与实际进程名匹配
 - [ ] `TargetPath` 设置为应用安装根目录，非子目录
 - [ ] `WorkModel` 根据场景选择 Correct（Normal/Upgrade）
diff --git a/.claude/skills/generalupdate-init/SKILL.md b/.claude/skills/generalupdate-init/SKILL.md
@@ -323,11 +323,10 @@ publish/
 
 ## ⚠️ 已知问题
 
-### NuGet 类型冲突
-`GeneralUpdate.Core` 和 `GeneralUpdate.Bowl` **不能同时引用**（CS0433 类型冲突）。
-请根据需求选择：
+### NuGet 注意事项（v10.5.0-beta.4）
+`GeneralUpdate.Core` 和 `GeneralUpdate.Bowl` **可以同时引用**（v10.5.0-beta.4 中无 CS0433 冲突）。
 - 使用 Core：`dotnet add package GeneralUpdate.Core`
-- 使用 Bowl：**只引用** `GeneralUpdate.Bowl`（它传递依赖 Core 所有功能）
+- 使用 Bowl：`dotnet add package GeneralUpdate.Bowl`（它**不**传递依赖 Core，需要同时引用 Core）
 - 差分类型已内嵌在 Core，**无需额外** `GeneralUpdate.Differential` 包
 
 ### 稳定版功能增强
@@ -354,7 +353,7 @@ v10.5.0-beta.4 新增以下功能：
 
 ### NuGet & 编译
 - [ ] Client 和 Upgrade 项目使用**完全相同**的 GeneralUpdate NuGet 版本
-- [ ] 如果用 Bowl：项目中只能有 `GeneralUpdate.Bowl`，不能同时有 `GeneralUpdate.Core`
+- [ ] 如果用 Bowl：项目中同时引用 `GeneralUpdate.Core` 和 `GeneralUpdate.Bowl`（v10.5.0-beta.4 无冲突）
 - [ ] 项目能正常 `dotnet build`（0 errors）
 - [ ] 无需额外引用 `GeneralUpdate.Differential`（已嵌入 Core）
 
@@ -376,14 +375,15 @@ v10.5.0-beta.4 新增以下功能：
 
 | # | 反模式 | 后果 | 正确做法 |
 |---|--------|------|---------|
-| 1 | **Core 和 Bowl 引用到同一个项目** | CS0433 类型冲突，编译失败 | 用 Bowl 时只引 Bowl（传递依赖 Core） |
-| 2 | **Client/Upgrade NuGet 版本号不一致** | 运行时 MethodNotFoundException | 锁定完全相同版本 |
-| 3 | **UpgradeApp.exe 不随首个版本发布** | 第一次更新时 FileNotFoundException | 首个版本就包含 UpgradeApp |
-| 4 | **事件监听中做耗时操作（网络 IO / 磁盘 IO）** | Update 进程 UI 卡死，超时被 Kill | 仅更新 UI 状态，耗时操作异步 |
-| 5 | **IPC 文件编码未设置 UTF-8** | Linux/macOS 中文乱码 | `Encoding.UTF8` |
-| 6 | **版本号不是 4 段式（如 1.0.0.0）** | 版本比较逻辑异常 | 始终用 `x.y.z.w` 格式 |
-| 7 | **manifest.json 的 mainAppName 不匹配真实进程名** | 更新后主程序找不到 | 和实际 exe 名称一致 |
-| 8 | **为 v9.x 编写的代码直接用在 v10** | API 不兼容，编译失败 | 对照 v10.4.6 稳定版 API 重写 |
+| 1 | **Core 和 Bowl NuGet 版本不一致** | 运行时 MethodNotFoundException | 使用相同 NuGet 版本 |
+| 2 | **Bowl 缺少 `GeneralUpdate.Core` 引用** | 编译失败，缺少 Core 类型 | Bowl 不传递依赖 Core，需同时引用 Core |
+| 3 | **Bowl 传递依赖 Core 的误解** | 编译失败 | v10.5.0-beta.4 中 Bowl 是独立包，需单独引用 Core |
+| 4 | **Client/Upgrade NuGet 版本号不一致** | 运行时 MethodNotFoundException | 锁定完全相同版本 |
+| 5 | **事件监听中做耗时操作（网络 IO / 磁盘 IO）** | Update 进程 UI 卡死，超时被 Kill | 仅更新 UI 状态，耗时操作异步 |
+| 6 | **IPC 文件编码未设置 UTF-8** | Linux/macOS 中文乱码 | `Encoding.UTF8` |
+| 7 | **版本号不是 4 段式（如 1.0.0.0）** | 版本比较逻辑异常 | 始终用 `x.y.z.w` 格式 |
+| 8 | **manifest.json 的 mainAppName 不匹配真实进程名** | 更新后主程序找不到 | 和实际 exe 名称一致 |
+| 9 | **为 v9.x 编写的代码直接用在 v10** | API 不兼容，编译失败 | 对照 v10.4.6 稳定版 API 重写 |
 
 ---
 
diff --git a/.claude/skills/generalupdate-init/reference.md b/.claude/skills/generalupdate-init/reference.md
@@ -14,10 +14,10 @@
 | `GeneralUpdate.Extension` | 插件管理系统 | ❌ 可选 | net8.0;net10.0 | ≥ 10.5.0 |
 | `GeneralUpdate.Drivelution` | Windows 驱动更新 | ❌ 可选 | net8.0;net10.0 | 10.5.0-beta.4 |
 
-> ⚠️ **NuGet 类型冲突**：
+> ⚠️ **NuGet 注意事项**（v10.5.0-beta.4）：
 > - `GeneralUpdate.Differential` 的 `DifferentialCore` 等类型已内嵌在 `GeneralUpdate.Core` 中，**不需额外引用**（直接使用 Core 即可）
-> - `GeneralUpdate.Bowl` 和 `GeneralUpdate.Core` **不能同时引用**（两者都发布了 `GeneralUpdate.Common` 导致 CS0433）
-> - 使用 Bowl 时**只引用 `GeneralUpdate.Bowl`**（它传递依赖 Core 的所有功能）
+> - `GeneralUpdate.Bowl` 和 `GeneralUpdate.Core` **可以同时引用**（v10.5.0-beta.4 中 Bowl 与 Core 无共用类型，不再冲突）
+> - 使用 Bowl 时仍需引用 `GeneralUpdate.Bowl`（它**不**传递依赖 Core）
 
 ## UpdateRequest 字段完整说明
 
diff --git a/.claude/skills/generalupdate-migration/SKILL.md b/.claude/skills/generalupdate-migration/SKILL.md
@@ -118,7 +118,7 @@ await new GeneralUpdateBootstrap()
 ### 编译验证
 - [ ] `dotnet build` 无错误
 - [ ] 无 `MissingMethodException` 的风险（检查所有方法名是否存在于 v10.4.6）
-- [ ] 无 `CS0433` 类型冲突（Core + Bowl 不同时引用）
+- [ ] 无 `CS0433` 类型冲突（v10.5.0-beta.4 中 Core + Bowl 无冲突，可同时引用）
 
 ### 架构验证
 - [ ] 项目已拆分为 Client + Upgrade 两个独立项目
diff --git a/RULES.md b/RULES.md
@@ -21,7 +21,7 @@
 
 ## NuGet Package Rules
 - Core only: `dotnet add package GeneralUpdate.Core --version 10.5.0-beta.4`
-- With Bowl: **reference only** `GeneralUpdate.Bowl` (transitively includes Core, the two conflict)
+- With Bowl: reference **both** `GeneralUpdate.Core` and `GeneralUpdate.Bowl` (v10.5.0-beta.4: Bowl is standalone, no type conflict)
 - Differential: types are **embedded in Core**, no extra package needed
 - Extension, Drivelution: standalone, no conflicts
 
diff --git a/cli/assets/scripts/generate/templates/bowl_notice.cs.template b/cli/assets/scripts/generate/templates/bowl_notice.cs.template
@@ -1,5 +1,6 @@
-// ⚠️ Bowl 崩溃守护 — 使用 GeneralUpdate.Bowl，不单独引用 Core
+// ⚠️ Bowl 崩溃守护
 // dotnet add package GeneralUpdate.Bowl
+// NuGet v10.5.0-beta.4: Bowl 与 Core 无类型冲突，可按需同时引用
 // Refer to generalupdate-advanced/templates/BowlIntegration.cs for full example
 var context = new GeneralUpdate.Bowl.BowlContext
 {
@@ -9,7 +10,9 @@ var context = new GeneralUpdate.Bowl.BowlContext
     DumpFileName = "{{CLIENT_VERSION}}_fail.dmp",
     FailFileName = "{{CLIENT_VERSION}}_fail.json",
     FailDirectory = System.IO.Path.Combine({{INSTALL_PATH}}, "fail", "{{CLIENT_VERSION}}"),
-    BackupDirectory = System.IO.Path.Combine({{INSTALL_PATH}}, "{{CLIENT_VERSION}}"),
+    // BackupDirectory 必须指向前一版本的备份目录，而非当前版本
+    // 例如当前版本为 {{CLIENT_VERSION}}，备份来自上一版本的 BackupDirectory
+    BackupDirectory = System.IO.Path.Combine({{INSTALL_PATH}}, "previous_version_backup"),
     WorkModel = "Upgrade",
     DumpType = GeneralUpdate.Bowl.DumpType.Mini,
     AutoRestore = true,
diff --git a/cli/assets/skills/generalupdate-advanced/SKILL.md b/cli/assets/skills/generalupdate-advanced/SKILL.md
@@ -157,7 +157,7 @@ Console.WriteLine($"Result: Success={result.Success}, Restored={result.Restored}
 | `DumpType` | DumpType | Mini / Full | Full |
 | `OnCrash` | delegate | 崩溃回调 | null |
 
-> ⚠️ Bowl 和 Core 不能同时引用（CS0433）。使用 Bowl 时**只引用 `GeneralUpdate.Bowl`**，它传递依赖 Core 的所有功能。
+> ⚠️ NuGet v10.5.0-beta.4 中 Bowl 和 Core **无类型冲突**，可以同时引用。
 
 ---
 
@@ -332,7 +332,7 @@ var result = GeneralDrivelution.InstallDriver(driverPath);
 ## ✅ 高级定制验证清单
 
 ### Bowl 崩溃守护
-- [ ] 只引用了 `GeneralUpdate.Bowl`（不单独引用 Core）
+- [ ] 如果用 Bowl：项目中同时引用 `GeneralUpdate.Core` 和 `GeneralUpdate.Bowl`（v10.5.0-beta.4 无冲突）
 - [ ] `MonitorParameter` 的 `ProcessNameOrId` 与实际进程名匹配
 - [ ] `TargetPath` 设置为应用安装根目录，非子目录
 - [ ] `WorkModel` 根据场景选择 Correct（Normal/Upgrade）
diff --git a/cli/assets/skills/generalupdate-init/SKILL.md b/cli/assets/skills/generalupdate-init/SKILL.md
@@ -323,11 +323,10 @@ publish/
 
 ## ⚠️ 已知问题
 
-### NuGet 类型冲突
-`GeneralUpdate.Core` 和 `GeneralUpdate.Bowl` **不能同时引用**（CS0433 类型冲突）。
-请根据需求选择：
+### NuGet 注意事项（v10.5.0-beta.4）
+`GeneralUpdate.Core` 和 `GeneralUpdate.Bowl` **可以同时引用**（v10.5.0-beta.4 中无 CS0433 冲突）。
 - 使用 Core：`dotnet add package GeneralUpdate.Core`
-- 使用 Bowl：**只引用** `GeneralUpdate.Bowl`（它传递依赖 Core 所有功能）
+- 使用 Bowl：`dotnet add package GeneralUpdate.Bowl`（它**不**传递依赖 Core，需要同时引用 Core）
 - 差分类型已内嵌在 Core，**无需额外** `GeneralUpdate.Differential` 包
 
 ### 稳定版功能增强
@@ -354,7 +353,7 @@ v10.5.0-beta.4 新增以下功能：
 
 ### NuGet & 编译
 - [ ] Client 和 Upgrade 项目使用**完全相同**的 GeneralUpdate NuGet 版本
-- [ ] 如果用 Bowl：项目中只能有 `GeneralUpdate.Bowl`，不能同时有 `GeneralUpdate.Core`
+- [ ] 如果用 Bowl：项目中同时引用 `GeneralUpdate.Core` 和 `GeneralUpdate.Bowl`（v10.5.0-beta.4 无冲突）
 - [ ] 项目能正常 `dotnet build`（0 errors）
 - [ ] 无需额外引用 `GeneralUpdate.Differential`（已嵌入 Core）
 
@@ -376,14 +375,15 @@ v10.5.0-beta.4 新增以下功能：
 
 | # | 反模式 | 后果 | 正确做法 |
 |---|--------|------|---------|
-| 1 | **Core 和 Bowl 引用到同一个项目** | CS0433 类型冲突，编译失败 | 用 Bowl 时只引 Bowl（传递依赖 Core） |
-| 2 | **Client/Upgrade NuGet 版本号不一致** | 运行时 MethodNotFoundException | 锁定完全相同版本 |
-| 3 | **UpgradeApp.exe 不随首个版本发布** | 第一次更新时 FileNotFoundException | 首个版本就包含 UpgradeApp |
-| 4 | **事件监听中做耗时操作（网络 IO / 磁盘 IO）** | Update 进程 UI 卡死，超时被 Kill | 仅更新 UI 状态，耗时操作异步 |
-| 5 | **IPC 文件编码未设置 UTF-8** | Linux/macOS 中文乱码 | `Encoding.UTF8` |
-| 6 | **版本号不是 4 段式（如 1.0.0.0）** | 版本比较逻辑异常 | 始终用 `x.y.z.w` 格式 |
-| 7 | **manifest.json 的 mainAppName 不匹配真实进程名** | 更新后主程序找不到 | 和实际 exe 名称一致 |
-| 8 | **为 v9.x 编写的代码直接用在 v10** | API 不兼容，编译失败 | 对照 v10.4.6 稳定版 API 重写 |
+| 1 | **Core 和 Bowl NuGet 版本不一致** | 运行时 MethodNotFoundException | 使用相同 NuGet 版本 |
+| 2 | **Bowl 缺少 `GeneralUpdate.Core` 引用** | 编译失败，缺少 Core 类型 | Bowl 不传递依赖 Core，需同时引用 Core |
+| 3 | **Bowl 传递依赖 Core 的误解** | 编译失败 | v10.5.0-beta.4 中 Bowl 是独立包，需单独引用 Core |
+| 4 | **Client/Upgrade NuGet 版本号不一致** | 运行时 MethodNotFoundException | 锁定完全相同版本 |
+| 5 | **事件监听中做耗时操作（网络 IO / 磁盘 IO）** | Update 进程 UI 卡死，超时被 Kill | 仅更新 UI 状态，耗时操作异步 |
+| 6 | **IPC 文件编码未设置 UTF-8** | Linux/macOS 中文乱码 | `Encoding.UTF8` |
+| 7 | **版本号不是 4 段式（如 1.0.0.0）** | 版本比较逻辑异常 | 始终用 `x.y.z.w` 格式 |
+| 8 | **manifest.json 的 mainAppName 不匹配真实进程名** | 更新后主程序找不到 | 和实际 exe 名称一致 |
+| 9 | **为 v9.x 编写的代码直接用在 v10** | API 不兼容，编译失败 | 对照 v10.4.6 稳定版 API 重写 |
 
 ---
 
diff --git a/cli/assets/skills/generalupdate-init/reference.md b/cli/assets/skills/generalupdate-init/reference.md
@@ -14,10 +14,10 @@
 | `GeneralUpdate.Extension` | 插件管理系统 | ❌ 可选 | net8.0;net10.0 | ≥ 10.5.0 |
 | `GeneralUpdate.Drivelution` | Windows 驱动更新 | ❌ 可选 | net8.0;net10.0 | 10.5.0-beta.4 |
 
-> ⚠️ **NuGet 类型冲突**：
+> ⚠️ **NuGet 注意事项**（v10.5.0-beta.4）：
 > - `GeneralUpdate.Differential` 的 `DifferentialCore` 等类型已内嵌在 `GeneralUpdate.Core` 中，**不需额外引用**（直接使用 Core 即可）
-> - `GeneralUpdate.Bowl` 和 `GeneralUpdate.Core` **不能同时引用**（两者都发布了 `GeneralUpdate.Common` 导致 CS0433）
-> - 使用 Bowl 时**只引用 `GeneralUpdate.Bowl`**（它传递依赖 Core 的所有功能）
+> - `GeneralUpdate.Bowl` 和 `GeneralUpdate.Core` **可以同时引用**（v10.5.0-beta.4 中 Bowl 与 Core 无共用类型，不再冲突）
+> - 使用 Bowl 时仍需引用 `GeneralUpdate.Bowl`（它**不**传递依赖 Core）
 
 ## UpdateRequest 字段完整说明
 
diff --git a/cli/assets/skills/generalupdate-migration/SKILL.md b/cli/assets/skills/generalupdate-migration/SKILL.md
@@ -118,7 +118,7 @@ await new GeneralUpdateBootstrap()
 ### 编译验证
 - [ ] `dotnet build` 无错误
 - [ ] 无 `MissingMethodException` 的风险（检查所有方法名是否存在于 v10.4.6）
-- [ ] 无 `CS0433` 类型冲突（Core + Bowl 不同时引用）
+- [ ] 无 `CS0433` 类型冲突（v10.5.0-beta.4 中 Core + Bowl 无冲突，可同时引用）
 
 ### 架构验证
 - [ ] 项目已拆分为 Client + Upgrade 两个独立项目
diff --git a/cli/package.json b/cli/package.json
@@ -1,7 +1,7 @@
 {
   "name": "gskill-cli",
   "version": "0.1.0",
-  "description": "CLI to install GeneralUpdate skill for AI coding assistants",
+  "description": "CLI to install GeneralUpdate skill for AI coding assistants (version independent from skill version; skill matches manifest version)",
   "type": "module",
   "bin": {
     "gskill": "./dist/index.js"
diff --git a/cli/src/commands/uninstall.ts b/cli/src/commands/uninstall.ts
@@ -97,12 +97,13 @@ export async function uninstallCommand(options: UninstallOptions): Promise<void>
     const allRemoved: string[] = [];
 
     if (aiType === 'all') {
-      for (const type of initialDetected) {
-        const removed = await removeSkillDir(baseDir, type as Exclude<AIType, 'all'>);
+      for (const type of initialDetected.filter((t): t is Exclude<AIType, 'all'> => t !== 'all')) {
+        const removed = await removeSkillDir(baseDir, type);
         allRemoved.push(...removed);
       }
     } else {
-      const removed = await removeSkillDir(baseDir, aiType as Exclude<AIType, 'all'>);
+      // aiType is narrowed to Exclude<AIType, 'all'> by the === 'all' check above
+      const removed = await removeSkillDir(baseDir, aiType);
       allRemoved.push(...removed);
     }
 
