Skip to content

fix: round-4 batch-2 — npm uninstall order, upgrade scope, decode leaks, bilingual matrix (F5-F7, F10) - #7

Open
voidvec wants to merge 1 commit into
masterfrom
fix/round4-batch2-f5-f7-f10
Open

voidvec wants to merge 1 commit into
masterfrom
fix/round4-batch2-f5-f7-f10

Conversation

@voidvec

@voidvec voidvec commented Sep 27, 2026

Copy link
Copy Markdown
Owner

Summary

第四轮整改批次②(报告 F5–F7、F10,每条先见 RED):

  • F5 npm uninstall 判定前置:旧顺序"先删 bundle 后读 v3 戳"在混合安装下留下"bundle 缺失但戳宣称 claude/zcode 已装"的说谎态。现在先判账本——无戳(npm 自装自卸)或纯 bundle → 全清;混合/损坏戳 → bundle、legacy、戳一概不动,提示交 PyPI CLI(损坏戳有独立正向对照)。
  • F6 py upgrade 认 v2 bundle 布局:npm 装的项目无 v3 戳,旧逻辑按"无戳退化全量"凭空落下 8 家产物;现在检出 bundle 布局 → 收敛 claude,zcode。"完全无产物 → 全量"旧语义保留(既有测试锁)。
  • F7 非 UTF-8 用户文件三条泄漏路径(上轮只修了卸载剥段):verify traceback / install 整批失败 / legacy 卸载崩溃,统一"读不动=不碰"(显 — / skipped+告警 / 保留文件)。
  • F10 zh-CN 宿主矩阵与 CLI 表行补 N4/N5 落点(双语"README 已标注"至此才属实);check-consistency 的"双语 README 结构对齐"扩为宿主矩阵路径单元格对拍——注入单行漂移实测见红,含两文件同错的假绿防线。
  • 测试基线变更:批次④ L1 混合对照断言由"bundle 被删"改为"bundle 保留"(F5 设计变更,CHANGELOG 已记)。

Test plan

  • F5/F6/F7 各 RED 先行(混合 bundle 被删 / upgrade 越界落 8 家 / verify·install·legacy 三处 UnicodeDecodeError),修复后全绿
  • 门禁方向验证:zh 矩阵注入漂移 → check-consistency rc=1,还原 → 14 项绿
  • pytest tests/ -q — 195 passed / 2 skipped(净增 5 例),输出无警告
  • sync-assets 双侧 --check、gen-host-artifacts --check 绿;node cli.js --help 语法过

基于 master(含批次①)。

… scope from v2 layout; decode-safe verify/install/legacy; bilingual host-matrix gate

第四轮批次②(F5/F6/F7/F10,每条先见 RED):
- F5 npm cmdUninstall 先读 v3 戳再决定删什么:混合/损坏戳下 bundle、
  legacy、戳一概不动(旧顺序先删 bundle 后判账本,留下戳与产物说谎态);
  损坏戳正向对照锁"什么都不删"
- F6 py upgrade 无 v3 戳但检出 bundle 布局(v2/legacy,即 npm 装的项目)
  → 范围收敛 claude,zcode;完全无产物才退化全量(旧行为保留,有测试锁)
- F7 非 UTF-8 用户文件三处泄漏统一"读不动=不碰":verify 显 — 不崩、
  install skipped+告警不拖死整批、legacy CLAUDE.md 保留不崩溃
- F10 zh-CN 宿主矩阵/CLI 表行同步 N4/N5 落点;check-consistency
  "双语结构对齐"项扩为宿主矩阵路径单元格对拍(注入漂移实测见红)
- 基线变更:批次④ L1 混合对照测试随 F5 语义改为断言 bundle 保留
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant