AskaEth/Unlucky-Launcher
1
0 Fork 0
Code Issues Pull Requests Actions 5 Packages Projects Releases Wiki Activity
489 Commits 3 Branches 5 Tags
e386cfbeee722eb1f2044c95a7e07a0d623e200d
Commit Graph
7 Commits
Author SHA1 Message Date
AskaEth ff2f229b80 fix(addon): 复审 r2 —— 下载阶段抽出可测边界并补守门 
- 阻塞项：下载循环从 `install::run` 抽到新文件 `addon/downloads.rs`
  （`download_failure()` + `download_stage()`），假下载器就能驱动它；三条用例分别钉
  ①失败→Unreachable+叶子成因、②第 2 个失败不中断且已装不回滚、③解析完就取消一份都不下。
  实测删掉 `record_failure` 那行会红（failures 0 vs 1），把 `cause` 丢掉也会红；
- nit① 补 core 用例：夹具里 `{"version_id":"vid-only","dependency_type":"required"}` ⇒
  `plan.unresolved_required == [UnresolvedRequired{parent:"MIXED001",…}]`，同版本的
  `project_id` required 仍被跟；删掉收集那段即红（left: [] vs 一条）；
- nit② 取消检查加回解析之后（`download_stage` 入口），并用例钉「解析后取消 ⇒ 不下任何文件」；
- nit③ `fail()` 文档写明去重按**请求键**而非规范 id 的边界（先 slug 后 project_id 仍会两条），
  不消掉但不再让人误以为全覆盖；
- `install.rs` 只留解析 + 收口（275 行），模块表登记 `downloads`。
 2026-11-08 20:23:57 +08:00
AskaEth 011d525e8d feat(addon): Modrinth 前置依赖 BFS 解析与两档失败名单 
- model 纳入 `dependencies`（project_id / version_id / file_name + required /
  optional / incompatible / embedded / unknown 五值），只跟 required 递归；
- 新增 `online::dependency`（请求/计划/跳过的形状）与 `online::resolve`
  （BFS：规范项目 id 去重防环、每项目只解一次版本、同层按项目键字典序、
  默认深度 6 且截断写进计划、父子关系进 `required_by`）；
- 失败从类型上分两档：`Incompatible`（所选版本/加载器没有匹配项）与
  `Unreachable`（并保留叶子成因 NotFound/NotConnected/…），`retry_request()`
  只含失败项，解析永不返回 Err（失败是数据，不连坐）；
- 盘上第二道去重：`present_key` 归一化文件名（小写、砍路径、去 `.disabled`），
  命中即跳过下载，但依赖图仍然展开；
- presenter 装入路径改成「解析计划 → 逐个下载」，文案把两档分开写并列出
  「可只重试这 N 个」；`KNOWN_LOADERS` 补 iris/optifine（光影的加载器）；
- 离线夹具覆盖：嵌套两层、环、扇入、深度截断、两档失败、已存在跳过、光影加载器。
 2026-11-08 20:23:55 +08:00
AskaEth b394316df4 fix(m162-r1): 叶子成因穿出引擎 + 安装路径切 core 在线源 + 详情迟到结果闸门 
P1-1（p1-1items 的根因）：
- download/error.rs：`DownloadError` 增 `leaf: Option<DownloadErrorKind>` 与
  `effective_kind()`；`leaf_of()` 汇总规则只此一处：全 404 ⇒ NotFound、有
  Network/Timeout ⇒ Network、全 5xx ⇒ ServerError、混合按「网络错 > 5xx > 404」
  取最可诊断的那个（**不从 suppressed 文本猜状态码**）。
- download/bytes.rs：候选源循环收集逐源 kind，收尾时把叶子挂到 AllSourcesFailed 上。
- online/transport.rs：`From<DownloadError>` 改用 `effective_kind()`，于是
  NotFound/Server/Integrity 在生产路径可达（项目被删/id 拼错/限速不再画成「未联网」）。
- import/fetch.rs：新增 `FetchErrorKind::Offline`（真正的「没联网」）并与 Network 分开；
  `From<DownloadError>`/`From<OnlineError>` 都按叶子/大类映射。
- import/complete.rs：补全只在 `Offline` 时早停；404（真实 mrpack 大多托管在 CF CDN，
  Modrinth 必然 404）与 5xx 只影响那一条，不再掐掉整轮；注记「因未联网提前停止」因此
  只会在真断网时出现。

P2-1：presenter 安装路径不再自己拼 Modrinth URL / 自己解析 JSON——改用
`core::addon::online::Modrinth::versions` + `primary_file()` + `VersionFile::sha1/sha512`；
`HostTask::InstallDone.result` 改成 `Result<String, ContentFailure>`，
`InstallProgressView` 增 `failure_kind`（离线 ≠ 查无此项）。

P2-2：`app/update/content.rs` 把被详情方法抢走的「装入闸门」注释还回
`content_install_request`，详情方法只留自己那一行。

P2-3：`ContentState::apply_detail/fail_detail` 带上 project（事件与 HostTask 都携带请求时的
项目 id），迟到结果一律丢弃；`request_detail` 记下在飞项目，`detail_pending` 不再被迟到
结果清掉。

顺手项：`usable_hash` 让形状非法的那一档摘要被**丢弃**（sha512 仍校验，不再整条跳过；
两档都不可用才拒绝）；`digest_file` 的 IO 失败先删落盘文件再返回。i18n 的
ContentOfflineBody 改成不暗示「有缓存」的措辞（golden 同步）。

用例：download/error.rs 叶子四档 + effective_kind；online/tests/classify.rs 5 条
（AllSourcesFailed+leaf 404 ⇒ NotFound、无叶子 ⇒ NotConnected 保守、叶子一一映射、
单源自身 kind、导入侧 404≠Offline）；import_online 增「404 不停」「真离线才停」
（共 7 条）；download 增「sha1 形状非法仍用 sha512 校验成功」；presenter 增详情迟到闸门。
 2026-11-07 17:28:36 +08:00
AskaEth 81278f55af feat(m162): Modrinth 在线内容源（匿名）+ 远端条目按 sha512 补全 + 离线态 
core：新增 addon::online —— 按 Modrinth v2 真实形状建模（search facets / project /
project versions / version_file by hash），网络出口只有 Transport 一个 trait，生产实现
EngineTransport 走既有 DownloadEngine::download_bytes（零新依赖）；错误分大类，网络类
失败映射成 NotConnected（= 未联网），不再用空列表冒充「没有结果」。

core：online::download::download_verified —— sha1 交给引擎流式校验、sha512 落盘后复核
（download::HashAlgo 没有 Sha512，改它超出 scope，已登记 finding），不符/失败删文件重试，
最多 3 次尝试且不留半截文件。

core：远端条目补全 —— PackFetcher::modrinth_file_by_sha512（默认 Unsupported = 补不出），
import 管线与新增的 scan_pack_online 都会在「有 path 但缺 hashes.sha1」时按 sha512 反查；
补出的 sha1 走 M160 同一套 judge 与合并判定，补不出仍进 unknown_remote 计数（口径未改）。
DownloadEngineFetcher 改用 download_verified，FetchRequest 增 sha512。

presenter：Modrinth 检索/详情改由 core 的 online 客户端提供（不再自己拼 URL、不再读错
date_modified）；ContentFailure{kind} 取代裸字符串，无网 = Offline；新增
ContentDetailRequest/ContentDetail + LoadContentDetail 命令与 CoreEvent::ContentDetail 通道；
预检改走 scan_pack_online；内容库安装也走 download_verified（sha1+sha512）。

app-iced：Status::Offline 独立一态（未联网 ≠ 空列表 ≠ 普通失败），详情面板消费新 DTO
显示可用版本；选中项目即拉详情。i18n 新增 6 个键并同步金表。

测试：online 模块 24 个离线用例（录制 JSON + 假字节：搜索/详情/版本/按摘要反查/校验失败
即删/重试/离线分类/补全），import 侧 6 个（离线计数不变、补全后判冲突、补错摘要 ⇒ Failed），
presenter/app-iced 各补失败分类与详情状态机用例。
 2026-11-07 17:28:36 +08:00
AskaEth 99538e45b9 fix(packui): M113 r1 复审整改 —— 2 处「界面说假话」+ 7 条必修 P2 
复审 r1（p1-2items / fix-then-merge）逐条整改。**P2-6（core 的本地化契约）按塔台指示本轮不动**，
已登记为独立 mission 的债。

## P1

1. **「Save to」是死输入框**：`ExportRequestView` 的唯一构造点硬编码 `target: None`，用户敲的路径
   从不进请求。现在构造收成纯函数 `packui::model::export::request_of`（预检与确认共用同一份），
   `target` 由字段去空白后给出（空 ⇒ `None`，交给 host 给缺省）。
2. **取消导入的 toast 撒谎**：下载页那条说「安装已取消、实例未创建」，而导入会在后台跑完并
   **登记实例**。现在 `cancel_pack()` 回报「刚才有没有导入在跑」，`dispatch` 据此改发
   `PackEvent::Cancelled`（不再发 `InstallCancelled`），界面与提示改成
   「导入会在后台继续跑完 —— 本次结果作废；已写下的文件会留下」，并把这句话**留在屏上**
   （`ImportState.cancelled` + 面板位）。

## P2 必修（7 条）

3. **`Dismiss` 误伤下载页安装**（跨功能污染，最高优先）：pack 原来共用 `CoreHost::install_epoch`，
   关一次导入面板会把下载页正在跑的安装静默丢掉。现在 pack 有自己的
   `PACK_EPOCH`（模块级 `AtomicU64`），`handle` 的 pack 分支只比它，`Dismiss`/取消只动它。
4. **中文实例名塌成 `pack`**：新增 `convert::dir_name` —— 名字全 ASCII 安全字符时保持原样
   （`Better MC` → `Better-MC`），一旦折过东西（中文/空格/符号）就补 `-<hash8>`（FNV-1a），
   保证两个中文名得到不同目录。
5. **失败回滚后空目录挡住重试**：`target_of` 对「已存在但空」的目录**复用**（非空才拒），
   并在失败回滚后清掉自己建的空目录；`failed_report` 的 `target` 语义改正
   （以前把包路径塞进「目标实例目录」）。
6. **新实例流程下「以包为准」必然失败**：`can_overwrite` 追加 `page.instance.is_some()` ——
   新建实例那条路上不再显示一个必然失败的按钮。
7. **阶段词打架且文档说修好了**：`progress.rs` 的模块文档改成与屏幕一致（粗/细两层 + 已知重叠），
   并写清彻底解决要靠「安装面板提成共享组件」那条 finding。
8. **导入侧标题计数与行数对不上**：被拒 component 有了自己的标题（`The client does not support
   these: {count}`）；冲突标题改成中性的 `Conflicts: {count}`（逐条自带成因与处置）。
9. **预检「没进包」列表没有有界滚动**：13 条是真实形态。列表改成有界滚动（110px），
   并且**把动作按钮提到摘要下面**（`Write the pack` / `Back`）—— 明细再长也顶不掉按钮。
10. **导入成功后不刷新实例列表**：`handle` 的 pack 分支在收到 `Report` 时调 `refresh_instances()`
    （与 `InstallDone` 同一条），新实例不用等下一次无关的 `RefreshAll`。

## 证据

- 新增测试（都跑真管线/真函数）：`the_save_to_field_really_reaches_the_request`、
  `pack_epoch_is_its_own_counter`、`chinese_instance_names_get_distinct_directories`、
  `a_failed_new_instance_directory_can_be_reused_for_retry`、
  `overwrite_is_offered_only_when_importing_into_an_existing_instance`、
  `cancelled_state_is_visible_and_says_the_import_keeps_running`。
- 出图重出 12 张（6 态 × 浅深），新增 `06-cancelled-*`；`04-export-*` 用 13 条 skipped 的真实形态。

门禁：`cargo fmt --all --check` 通过；`cargo clippy -p unlucky-app-iced -p unlucky-presenter
-p unlucky-i18n --all-targets -- -D warnings` EXIT=0；`cargo test -p unlucky-app-iced --lib`
507 passed；`cargo test -p unlucky-presenter --lib` 184 passed；`cargo test -p unlucky-i18n --lib`
24 passed；全部改动文件 ≤400 行。
 2026-11-05 04:06:42 +08:00
AskaEth 5f09b8113a feat(packui): M113 phase 2 —— 导入导出真正接线（三个共享枚举各只多一个变体） 
搬到 M109 rebase 后的新 main（26e764e）之后按**新 main 的形状**写一次，不写两遍。

## 接线

- `dto/pack/event.rs`（新）：`PackCommand` / `PackEvent` / `PackState::apply`。
  整条流程在三个最挤的共享枚举里各只占**一个**变体（`AppCommand::Pack` / `CoreEvent::Pack` /
  `HostTask::Pack{event, epoch}`），子枚举承载细节 —— rebase 时不用替别人定序。
- `events.rs` / `job.rs` / `handle.rs` / `dispatch.rs` / `state.rs`：变体与分支，全部带
  `// M113: import/export` 锚点。`state.rs` 只多 1 个字段 + 1 条 apply 分支（399 行）。
- `host/addon/**`（新目录，父文件只加 `mod pack;` + 再导出）：
  - `pack.rs`：`CoreHost::pack` 落点 + 取消标记 + 实例目录/快照/格式/缺省路径解析
  - `pack/import.rs`：探测 → 目标目录 → `import_pack` → 报告（默认 KeepUser，显式才 Overwrite）
  - `pack/export.rs`：预检（`scan_instance`，**不写盘**）与写盘（`export_instance`）
  - `pack/convert.rs`：core 报告 → DTO，逐字段照搬（三态/五阶段/四处置/三成因/十一原因）
  - `pack/diagnose.rs`：探测诊断「像 X 缺 Y」（按标记文件的有无，不看文件名）
- `app/update/content.rs`：导入 / 以包为准 / 预检 / 确认写盘 / 关面板真的派指令；
  预检时先记下那份请求，确认时**原样发回**（防「预检 A、写 B」）。
- `pages/content/model.rs`：读 `state.pack`（不再是默认空态）。
- 面板接口按新形状改：`install_panel_model(&InstallProgressView, catalog) -> Option<Download>`
  （`InstallView` 不再从 `pages/download` 导出）。

## 取消（如实）

`CancelInstall` 现在同时置导入的取消标记 + 代次 +1：面板立刻收回、旧事件按代次丢弃。
但 core 的 `import_pack` 没有取消令牌、进度回调也拿不到控制流 ⇒ **后台那次导入会跑完**。
所以「取消」的真实含义是「不再显示 + 结果作废」，已在模块文档与报告里写清，并单开 finding。
取消标记用 `Arc<AtomicBool>` 而不是裸静态量：裸静态量会在并行测试间泄漏取消状态（实测到过）。

## 图看出来的三处（塔台要求）

1. `no download source (CurseForge file API not wired)` → `no download source available yet`，
   API 名只留在 DTO 的 detail 与日志里；顺带把 `sha512` / `projectID/fileID` 也改成用户话。
2. 右栏 `Installing modpacks is not available yet` → `Installing modpacks from the library into
   an instance is not available yet`（那句在 i18n 词表里，不在 M109 页面代码里，所以我自己改）。
3. 结果面板接线后可以关：`Dismiss` / `AbortExport` 发 `PackCommand::Dismiss` → host 回
   `PackEvent::Cleared` → 回到入口/浏览态，不再常驻。

## 证据（三条都是真跑，不是假数据）

- `bad_packs_get_a_named_diagnosis`：把文本改名成 .mrpack → 有具名诊断（不是静默、不 panic）；
  目录形态有 overrides/ 无 manifest.json → 说得出「缺哪个标记」。
- `a_real_pack_runs_through_the_pipeline_end_to_end`：造一个真 MultiMC 实例目录当包，
  走 `import::run` 真路径，断言事件流里有进度 + 结果、格式=MutliMC、报告有写盘文件、文件真落地、
  代次原样带回。
- `preview_counts_match_the_written_report`：同一实例上跑预检与 `export_instance`，
  inline / skipped / remote **三项计数逐项相等**，且逐条路径的归类一致。

门禁：cargo check ✓ / clippy -D warnings（三 crate）EXIT=0 /
cargo test（app-iced 504 + 6 守卫，i18n 24，presenter 181）全绿 / rustfmt --check ✓ /
所有改动文件 ≤400 行（`state.rs` 399、`addon.rs` 106，pack 子目录最大 338）。
 2026-11-05 04:06:39 +08:00
AskaEth 6b161227d8 feat(app-iced): M109 CONTENTDL —— 内容库页（整合包/Mod/资源包/光影） 
用户令「下载页只负责下载整合包和Mod/光影」：新开一页做**内容发现与装入**，
检索走 Modrinth（优先，无需 key）/ CurseForge（`ULL_CF_API_KEY` 环境变量，日志只打
set/unset），装入走 core 的下载引擎真下到实例目录，进度/取消/失败复用下载页那块安装面板。

- **入口（B1）**：`Route::Content` 是新顶层路由，但 `Route::nav()` 复用 `NavSection::Download`
  ——导航轨仍五项，页头右上是「版本管理 | 内容库」一级切换（版本管理页也加了同一个切换，
  否则内容库只有 `ULL_START_ROUTE` 能到）
- **页内四条一级侧栏**（`pages/content/layout.rs` 四档几何：992/856/648 三个门槛都是
  「Σ栏宽 + 间距」算出来的）：整合包 / Mod / 资源包 / 光影；标签栏 200（`Resource packs`
  比 M107 的版本类型名长，140 会切字，已登记偏离）
- **筛选**：关键词（草稿 + 显式「搜索」按钮，不逐字打网络）、来源、游戏版本、加载器
  （Fabric/Forge/NeoForge/Quilt）、排序；**分页**（「加载更多」）而不是无限滚动——快照只有一个槽、
  翻页要看得见、状态要可截图复现
- **四态**：加载 / 空 / 失败 / 有数据 + 「没有更多了」；筛选条在任何状态下都在（加载中藏筛选条
  等于把入口收走）
- **一键装入**：`Content::blockers()` 是纯函数——版本不匹配、加载器不匹配、没有目标实例、
  整合包缺管线**四条原因逐条**给出，按钮点之前就禁用；判定与画法分开（model/tests 直接钉）
- **整合包今天不可装**：`ContentKind::Modpack.dir()` 是 `None`，UI 禁用 + 说明，host 侧再挡
  一次（绕过 UI 也只会回一条 Failed）。真流程是 M111 的导入管线
- **零 UI 网络**：`presenter/host/addon.rs` + `addon/{sources,http,install}.rs`；无 header 的请求走
  core 的 `download_bytes`（信号量限流 + 健康度 + 退避全现成），带 `X-API-KEY` 的只有 CF 一条，
  自带 429/5xx 退避（优先吃 `Retry-After`，封顶 30s）；密钥不进 DTO/日志/仓库
- 文案 41 条中英进 `crates/i18n`（新域接在装配链末尾 ⇒ `Key::ALL` 与金表只追加）
- 演示数据 `ContentHit → Hit::of` 与产品路径同源（格式化不会两套），`ULL_CONTENT_KIND` /
  `ULL_CONTENT_STATE` 供出图矩阵选类型与状态

取证：`DevDocs/artifacts/m109/`（10 张：四类内容 × 浅/深 + 搜索无结果 + 加载器不兼容禁用）

门禁：fmt ✓ / clippy `-D warnings`（三 crate）✓ / app-iced 450+6、presenter 170、i18n 24 全绿；
`cargo tree` 两条零 UI 门禁仍空；全部改动文件 ≤400 行。
 2026-11-05 03:20:21 +08:00