fix(m168): 复审 r1 五项 —— 修红的 integration target、独立失败域、整体时限
复审 r1(p1-1/fix-then-merge)五项全改:
1. **p1-1 CI 会红的 4 条 integration test**(`crates/core/tests/setting/`,我上一轮只跑了 `--lib`,
scope 漏掉了覆盖 schema 版本号的目录式 target):
- `java_install_keys` / `isolation_keys` / `appearance_keys`:版本断言 1.6.0 → 1.7.0(M168 的
`ui.font_cdn`),并写明「每加一批键推一次 minor 是纪律」。
- `update_keys::a_read_only_launcher_config_never_retires_the_legacy_file`:夹具从**写死
1.7.0** 改成 `launcher_version().minor + 1` 经 `SchemaRef::url()` 生成。写死数字的版本一推
这条守卫就变成「同版本」⇒ 文件可写 ⇒ 迁移真的执行 ⇒ 旧文件被退役,而断言还绿着(用户数据
守卫静默失效)。相对推导不会过期。
2. **p2-1 备用镜像不备用**:`cdn` 与 `fastly` 同属 Fastly(都 CNAME 到 jsdelivr.map.fastly.net),
换掉。现在首选 `gcore.jsdelivr.net`(Cloudflare,独立失败域),二选 `cdn.jsdelivr.net`
(Fastly)。注释里放了实测表(本机 + 复审机两份数据、字节数、sha256、IPv6 慢路径),
并写清排序依据(Fastly 侧 IPv6 实测 67 KB/s ⇒ 丢掉 4 分钟;Cloudflare 侧两次都完整下完)。
3. **p2-2 整体时限**:`core::download::font` 新增 `ATTEMPT_DEADLINE = 90 s` /
`TOTAL_DEADLINE = 180 s`,落在本模块自己的候选循环里(不动引擎全体语义):每次尝试
`min(单次上限, 剩余总预算)`,超时归 `Network` 档。新增守卫
`a_trickling_connection_is_cut_by_the_deadline`(回环服务端每 100 ms 挤 1 字节、预算 300 ms)。
顺带修掉一个真 bug:`tokio::time::timeout` 在运行时外构造会 panic(5 条既有用例当场抓到),
现在整段包进 `block_on(async { … })`。UI 文案改成「网络不通或太慢(超时)」,不校验失败混。
4. **p2-3 说 `Source::PendingPin` 的两处**:`fonts/cjk.rs` 的模块文档已改;`ui/lang.rs::log_cjk`
那句「D28 的下载通道还是 PendingPin」(在下载已发起之后打印,等于当面否定本功能)改成
「已按 D28 发起下载,见 [font] cjk-download … 与设置页那一行」。全仓再扫,只剩注释里的历史说明。
5. **p2-4 假守卫**:判据抽成纯函数 `install::plan(strategy, current) -> Step`(`Skip` / `Apply` /
`Start`),`after_probe` 只执行。旧的空转用例换成打真判据的打表用例;变异证明:删掉
`Skip` 那一臂 ⇒ `a_started_or_finished_install_is_not_restarted` 红(已实跑)。
次要项也一并处理:`from_cache` 改成**本次是否真的命中缓存**(`CdnFetcher::was_cached()`,
`has_cached()` 降级为诊断字段 `cache_present=`);`FontSpec` 的 URL 口径收紧为「非回环必须
https」(`is_acceptable_url` + 守卫);`ui.font_cdn` 写进状态行 hint;文档里的 17.0 MB → 16.9 MB。
`download/font.rs` 拆出 `font/spec.rs` 守 400 行。
验证(这台机器上单个 800 MB 测试 exe 链接要 10 分钟,链接期间文件被 Defender 扫描,故耗时很长):
- `cargo test -p unlucky-core --all-targets` → **1188 passed / 0 failed**(16 个 target,含
`setting` 那个 115 条、上一轮红 4 条的 integration target)
- `cargo test -p unlucky-i18n` → 31 passed / 0 failed
- `cargo test -p unlucky-app-iced --lib` → 685 passed / 0 failed
- `cargo clippy -p unlucky-i18n -p unlucky-core -p unlucky-app-iced --all-targets -- -D warnings` → 干净
- `cargo fmt --all -- --check` → 干净;全仓受检文件行数最大 388(<400)
This commit is contained in:
1 parent
449ded2cb7
commit
875b792206
17 files changed
+512
-170
No files matched your search
@@ -27,6 +27,7 @@
|
||||
//! `ui.font_cdn` 允许换成国内镜像(整条 URL)。**哈希与字节数仍是内置那一份**:镜像是同一份内容
|
||||
//! 的不同出口,给了别的内容就该被拒——宁可不下载,也不把没校验过的字节注册成字体。
|
||||
|
||||
use std::cell::Cell;
|
||||
use std::path::PathBuf;
|
||||
|
||||
use unlucky_core::download::font::{self as core_font, FontFetchError, FontSpec};
|
||||
@@ -39,6 +40,12 @@ use crate::platform::data;
|
||||
pub struct CdnFetcher {
|
||||
cache_root: PathBuf,
|
||||
spec: FontSpec,
|
||||
/// 最近一次 `fetch` 是否**真的**命中缓存(`None` = 还没取过)。
|
||||
///
|
||||
/// 为什么不在调用方「取之前看一眼文件在不在」:那个判据会说谎——缓存文件若已损坏,
|
||||
/// `core::download::font` 会删掉重下(零网络不成立),而「取之前在不在」仍然是 true。
|
||||
/// 复审 r1 次要项点名的就是这个:日志 `from_cache=true` 必须等于「这次没联网」。
|
||||
last_from_cache: Cell<Option<bool>>,
|
||||
}
|
||||
|
||||
impl CdnFetcher {
|
||||
@@ -50,18 +57,28 @@ impl CdnFetcher {
|
||||
Ok(Self {
|
||||
cache_root,
|
||||
spec: spec_of(font)?,
|
||||
last_from_cache: Cell::new(None),
|
||||
})
|
||||
}
|
||||
|
||||
/// 缓存是不是已经在盘上(只 stat,不做校验)。
|
||||
/// 缓存文件是不是已经在盘上(只 stat,不做校验)。
|
||||
///
|
||||
/// 用途只有两个:UI 那行「已就绪」的字样、日志里 `from_cache=true` 的标注。**它不承担守卫**:
|
||||
/// 真正的命中判定在 `core::download::font`(读出来 + 复验魔数后才算命中)。
|
||||
/// 用途只有**诊断**(`[font] cjk-download start … cache_present=`)。它**不是**命中判据:
|
||||
/// 真正的命中判定在 `core::download::font`(读出来 + 复验哈希与魔数),而「这次有没有联网」
|
||||
/// 只看 [`Self::was_cached`]——坏缓存会被删掉重下,那时这个函数仍然会返回 `true`。
|
||||
#[must_use]
|
||||
pub fn has_cached(&self) -> bool {
|
||||
core_font::cache_path(&self.cache_root, &self.spec).is_file()
|
||||
}
|
||||
|
||||
/// 最近一次 [`Fetcher::fetch`] 是否零网络命中缓存(`None` = 还没取过)。
|
||||
///
|
||||
/// 状态行与日志的 `from_cache` 只认这个值;`has_cached()` 只是「文件在不在」,两者不可互换。
|
||||
#[must_use]
|
||||
pub fn was_cached(&self) -> Option<bool> {
|
||||
self.last_from_cache.get()
|
||||
}
|
||||
|
||||
/// 缓存文件落点(诊断日志用)。
|
||||
#[must_use]
|
||||
pub fn cache_file(&self) -> PathBuf {
|
||||
@@ -71,9 +88,10 @@ impl CdnFetcher {
|
||||
|
||||
impl unlucky_i18n::font::Fetcher for CdnFetcher {
|
||||
fn fetch(&self, _font: &'static CjkFont) -> Result<Vec<u8>, FetchError> {
|
||||
core_font::fetch_blocking(&self.cache_root, &self.spec)
|
||||
.map(|fetch| fetch.bytes)
|
||||
.map_err(|error| translate(error))
|
||||
let outcome = core_font::fetch_blocking(&self.cache_root, &self.spec).map_err(translate);
|
||||
self.last_from_cache
|
||||
.set(outcome.as_ref().ok().map(|fetch| fetch.from_cache));
|
||||
outcome.map(|fetch| fetch.bytes)
|
||||
}
|
||||
}
|
||||
|
||||
@@ -147,6 +165,7 @@ mod tests {
|
||||
let _ = std::fs::remove_dir_all(&dir);
|
||||
let fetcher = CdnFetcher::new(&NOTO_SANS_SC, dir.clone()).expect("下载器");
|
||||
assert!(!fetcher.has_cached());
|
||||
assert_eq!(fetcher.was_cached(), None, "还没取过 ⇒ 不知道");
|
||||
assert!(fetcher
|
||||
.cache_file()
|
||||
.ends_with("cjk-noto-sans-sc-a3041811a78c361b.ttf"));
|
||||
|
||||
Reference in new issue
Block a user