AskaEth/Unlucky-Launcher
1
0 Fork 0
Code Issues Pull Requests Actions 5 Packages Projects Releases Wiki Activity
257 Commits 3 Branches 5 Tags
954a129e01cd19b5241037e12ff0071bea72ff56
Commit Graph
4 Commits
Author SHA1 Message Date
AskaEth 609bcb3044 fix(dlflow): M105 r1 —— 取消后不再逐个派发剩余文件；取消证据补强；取消提示进 i18n 
复审三条（p2-3items / fix-then-merge，其中两条按 p1 对待）：

**P2-1 取消后仍为每个剩余文件各发一次请求（p1）**
`select` 先轮询左支 ⇒ `install_one` 第一次 poll 就把 `engine.download(…)` 派出去了；
只把标志检查写在右支，等于取消后「每一个剩余 item 都创建并至少 poll 一次」
（一次整文件 sha1 + 一次请求），盘上已就绪的还会第一 poll 就 `Ready` 并回调 `sink`。
修：`async move` 第一句就 `if is_cancelled(cancel) { return None; }`，这些 item 连
`install_one` 都不进。守门用例 `cancelling_mid_run_stops_creating_remaining_files`
（三文件全在盘上、并发 1、sink 首次回调即置标志 ⇒ 数 sink 回调次数）：
修后 1，删掉那两行变异后 **3**（见报告，红/绿均实跑）。
模块文档同步改成与实现一致，并把复审点名的固有边界写进注释：取消瞬间最多有
`concurrency`（GUI 传 16）个 future 已经过了那道门，其中「盘上已就绪」的仍会跑完自己的
回调（靠 presenter 的安装代次丢掉这些进度事件），真在下载的会在同一次 poll 被取消。

**P2-3 真网络装置的断言比声称的弱，且有一条探的是恒不成立的路径（p1）**
① `.ullpart` 原来只断言「5 s 后不再增长」⇒「留下一个不增长的非 0 中间文件」会绿；
改成 `assert_eq!(parts_after, 0)`。
② 实例登记原来探 `<game>/instances`（根本不存在 ⇒ 恒 false，什么也没证明）；
改成读 `LauncherLayout::profiles_config()` 的 `read_profiles()` 并断言 0 条。
③ 模块文档里那条探针的路径一并订正。
变异自证：留一个 20 字节 `.ullpart` ⇒ 红（`left: 20 / right: 0`）；
走真 `instance::register()` 造一条 profile ⇒ 红（`profiles.json 里 1 条实例`）。

**P2-2 取消文案仍是硬编码中文，与新选项区同屏一中一英**
`presenter` 没有 i18n 依赖，所以不是搬常量能解决的。改成发稳定信号：
新增 `CoreEvent::InstallCancelled`（不带文本），`AppState::install_cancelled` 只计数；
app 层在 `refresh_view` 里比对计数、按 `Key::DownloadCancelled` 现求文本弹一次 Toast
（`pages::download::cancelled_toast`）。`dispatch.rs` 的中文 Toast 删掉。
语言入口收成 `app::view::ui_lang()` 一处（现恒为英文；M106 接上语言只改这里，
M106 合并前中文译文在产品路径不可达）。守门：`cancelling_announces_once_in_the_page_language`
（只弹一条、文案来自目录、不含中文、重复 refresh 不重复弹）。

门禁（真实退出码）：fmt=0、clippy(--workspace --all-targets -j 2 -D warnings)=0、
`cargo test -p unlucky-app-iced --lib`=0（440 passed）、`cargo test -p unlucky-core`=0、
`cargo test --workspace --exclude unlucky-app -j 2`=0（57 个 target 全 ok，0 failed）。
 2026-11-04 22:12:22 +08:00
AskaEth 30b1ff6f48 feat(dlflow): M105 安装选项区（四家加载器 + 官方版本列表 + 实例名）与取消安装真停 
用户实跑反馈两条，本轮各自定位到根因并修掉：

1. 「下载页没有前置加载器选项」= 功能未做：`Message::InstallVersion` 直接派
   `AppCommand::InstallVersion`（= 只装原版），core 早在 M32/M35/M45 就有四家加载器的
   安装管线。现在选中版本之后面板先给「安装选项」区（原版 / Forge / NeoForge / Fabric /
   Quilt + 该加载器的官方版本列表 + 实例名），点「安装」才派
   `AppCommand::CreateInstance`。
   - 加载器版本列表复用 core 的 `LoaderSource::loader_versions`，经新增的
     `AppCommand::ListLoaderVersions` / `CoreEvent::LoaderVersions` 回流；
     缓存键是 `(加载器, 原版版本)`，切版本不会串列表；取不到给具名原因并**禁掉确认**。
   - 不兼容组合用静态规则在**选中那一刻**说清（NeoForge 需 1.20.1+、Fabric/Quilt 需
     1.14+、远古版四家都不支持），不让用户「选完才失败」，也不为不可能的组合作无谓联网。
   - 选项草稿（选中 + 加载器 + 加载器版本 + 实例名）住在 `App`，`refresh_view` 带进页面
     模型（与分类筛选同一做法；页面自己存不住跨事件的东西）。

2. 「点取消没反应」= 真 bug，两跳各断一半：
   - 第一跳（UI 侧）：取消确实生效了，但**还在跑的后台线程继续发进度**，`InstallProgress(None)`
     刚把面板收回，下一条「下载中」立刻又点亮它 —— 用户看到的就是「点了没反应」。
     修法：`CoreHost` 加安装代次 `install_epoch`（开工与取消都 +1），`HostTask::Install` /
     `InstallDone` 带代次回来，代次不符一律丢。
   - 第二跳（core 侧）：`execute` 没有任何取消入参，取消只停在段边界，传输根本没停。
     新增 `execute_with_cancel(…, cancel: Option<&AtomicBool>)`（既有 `execute` 原样包装，
     CLI 一字未改）：取消后不再起新文件，在飞的那个与「取消 future」`select` 赛跑，取消赢就
     drop 掉下载 future（响应流关闭、`.ullpart` 由 `PartFile::Drop` 清掉），并且不写版本清单，
     返回新的 `InstallErrorKind::Cancelled`。
   - 另外：取消后**绝不登记实例**（文件段跑完但用户已取消的那一路也拦住）；取消文案把
     「哪些留着、哪些一定没发生」说清（D21）。

实测（真网络装置 `host/cancel_tests.rs`，`#[ignore]` 守着，跑法见文件头）：
取消前 files_done=2 / `.ullpart` 613 字节；取消后 20 s 进度事件 **0 条**、`.ullpart` **0 字节**
（-613）、版本清单未落盘、实例目录不存在。取消前是 +201466 字节（同一个装置量的）。

文案全部进 i18n（中英，9 条新 key + 金表同步）。
 2026-11-04 22:12:22 +08:00
AskaEth da2da4cd9b fix(app-iced,presenter): M93 r4 —— Float 键写不回 / prefs.rs 超限 / CPU 档判定 / 平铺退化 
复审（代码）r1 的两条 P1 + 两条必修。

P1-1 `SetSetting` 写不回 Float 键（整窗透明度重启回旧值）
- `presenter/src/host/settings.rs::parse_value` 改成**按 schema 的 `ValueKind` 取类型**，
  不再硬编码 Int/Bool 键名。历史：`appearance.window_opacity`（Float）被当 String 送进
  `store.set` → schema 的 `as_f64()` 返回 None → `InvalidValue` → **`save()` 根本没执行**，
  α 只在内存生效、重启回旧值、每次拖动弹一条 Error toast。
- 测试（presenter 层，`host/settings_tests.rs`）：
  ① `float_key_round_trips_through_set_setting`：set 0.8 → **新开 store（= 重启）** 读回 0.8；
  ② `every_float_key_in_the_schema_can_be_written_back`：遍历 schema 里**所有**
  `ValueKind::Float` 的键逐个写回并读回（集合级守门，下次新增 Float 键漏接会立刻红）。

P1-2 `presenter/src/prefs.rs` 404 行 → 拆测试
- `#[cfg(test)] mod tests` 整体挪到 `crates/presenter/src/prefs/tests.rs`，结构体与 re-export
  一字未动。`wc -l`：**404 → 315**（tests.rs 90 行）。

P2-2 CPU 档判定只在 `--renderer=cpu` 时成立
- 新增 `app/backend.rs`：`effective_backend()` 把 `probe::apply` 写下的 `ICED_BACKEND`
  候选列表翻回档位（`tiny-skia` / `tiny-skia,wgpu` → CPU；`wgpu,tiny-skia` → GPU；未设 → 请求值）。
  `App::with_presenter` 用它喂 `appearance::State`，于是**默认 Auto 档在没有独显的机器上**
  也会走「面板 α / 玻璃强制 1.0」。
- 测试：`every_iced_backend_list_maps_to_a_tier` + `auto_with_a_software_renderer_forces_the_panel_opaque`。
- 实跑：`ICED_BACKEND=tiny-skia`（= probe 在无独显机器上写下的值）→
  `[appearance] boot … panel_alpha=1.000 (requested 0.550) backend=cpu`。

P2-1 平铺退化成「一条竖条」——根因不是几何，是**渲染层没吃 scale**（r3 的编辑没落到 `tiled()`）
- `picture::tiled()` 原来仍用 `ContentFit::None` + 原图尺寸，所以 `tiles()` 算出的 `scale`
  完全没被消费：小图被按 32dp 画在 129dp 的格子里 ⇒ 到处是缝。
- 现在：每块 `源尺寸 × tiles.scale`、`ContentFit::Fill`、行高显式 `Fixed`、整列 `Fill`。
- 诊断：新增 `background::small_tile_note()`（短边 < 200dp 的图在 tile 档下给一句
  「scaled up so the window stays covered」），已接进设置页诊断块（首屏可见）。
- 测试：`tile_grid_always_covers_the_viewport_within_budget`（6 组尺寸，含 32×32 / 16×16 / 1×1，
  断言覆盖 ≥ 视口且块数 ≤ 预算）、`small_images_get_a_tile_diagnostic_instead_of_silently_degrading`。
- 出图：`r5-tile-tiny32.png`（32×32 铺满全窗、右缘无裸白、提示可见）、`r5-tile-120.png`、
  `r5-tile-big.png`。

顺手（复审 P2-3/P2-4/P2-5）
- P2-4：`MAX_SOURCE_EDGE` 16384 → **8192**，并给解码器加 `image::Limits`（宽/高 + `max_alloc`
  320MB）作第二道闸；补 `an_over_sized_image_reports_too_large_instead_of_trying_to_decode`
  （`TooLarge` 这条分支此前无测试）。
- P2-5：`transparency.rs` 补一段「为什么 `LWA_ALPHA` + `WS_EX_LAYERED` 不影响命中测试与
  自绘标题栏拖动」（穿透要 `WS_EX_TRANSPARENT` 或 `LWA_COLORKEY`，两者都没用；`window::drag`
  走 `WM_NCLBUTTONDOWN(HTCAPTION)`）。
- P2-3：断链 `[background::dimmed]` → `dim_alpha`；`iced/image` → `image-without-codecs`；
  测试文档 204 → 205；马赛克残留说明只保留 r1→r2 的历史一句。
- 解码在 UI 线程同步做：**登记欠账**（挪后台线程超出本 mission）。
 2026-11-02 18:46:43 +08:00
AskaEth f159529936 feat(presenter): 新建 unlucky-presenter，把 host/状态/DTO/设置桥从 app 迁出（M85 第 1 块） 
D26（iced 重写）的关键一次性动作：UI 框架无关的宿主层独立成 crate，
Slint 版（crates/app）与 iced 版（crates/app-iced）共用，Slint 退役后不再重写一遍。

- 新建 crates/presenter（package `unlucky-presenter` / lib `unlucky_presenter`），
  workspace 登记 + crate 文档头写清「这层为什么存在」、对外表面、UI 侧允许/禁止、
  iced 接线骨架、以及哪些东西留在了 crates/app 及为什么。
- `git mv`（保留文件历史）迁入：host/（25 文件）、dto/（11 文件）、state.rs(+tests)、
  prefs.rs、settings_bridge.rs、toast.rs、router.rs、dialogs.rs、wizard.rs、
  mock.rs、mock_data.rs、backend.rs。模块路径与公开 API 名称不变（`host::install` 等），
  内部实现零改动。
- 新增 `theme`：把原本混在 ui_bind/theme_runtime 里的**纯判据**（CONFIG_POLL_TICKS /
  SYSTEM_THEME_POLL_TICKS / TILE_REGENERATE_STEP / BackgroundKey / tile_viewport /
  system_theme_value / 两个 poll 判据）抽出来，附 5 个单测；app 侧改成调用它们。
- crates/app 改为消费者：删掉迁走的文件，`crate::host|state|dto|prefs|settings_bridge|
  toast|router|dialogs|wizard|mock|mock_data|backend` 全部改写为 `unlucky_presenter::*`
  （64 处），backend.rs 的 config_path/build_backend 提为 pub。
- crates/app-iced 只登记 unlucky-presenter 依赖，本 mission 不接线。
- prefs.rs 的 2 处 slint 文档引用改写；`grep -rn slint crates/presenter` 现在无输出。
 2026-11-02 09:29:31 +08:00