fix(app-iced): M90 r3 —— 空闲 1Hz 心跳恢复配置热切 + 导航轨口径订正 + 文档收口
复审 P2-2（空闲态掐时钟 ⇒ config_poll_due 再也不跑）： - 空闲态改用 1 Hz 自续心跳（Task::perform(sleep(1s)) → Message::Heartbeat），忙碌态仍用 window::frames() + 33ms 闸门；两条时钟不并行（心跳在 needs_frames() 为真时只续排不推进）。 - ticks 改成「33 ms 一拍」记账（1 s 心跳记 30 拍），config 轮询判据从「整除」改成「跨越」—— 实测整除版在 ticks=137 之后每拍 +30 永远不命中，配置轮询静默死掉。 - 顺手修一个真 bug：Message::SystemTheme 原来无条件 set_flavor，会把配置文件里的显式主题盖掉 （优先级现在是 --theme > 配置文件 > 系统），否则「改 launcher.json 主题 1 秒内生效」验收根本看不到。 - 实测：14 s 交互跑，空闲段 [tick] clock=idle-1hz 每秒一行、frames 冻结在 2947、refreshes 冻结在 6； t≈7s / t≈11s 两次改 launcher.json，各自 ≤1 s 内打出 [settings] 配置文件变了 + [theme] 翻转。 复审 P2-1（导航轨口径）： - pages/mod.rs 交付表改成事实（产品路径的 rail 由 shell/nav.rs 画，组件版只给 harness/单测， 像素等价是复审判的），补偏离表第 10 条（两套 rail 各测各的，漂移没人拦）； - pages/tests.rs 的 the_real_rail_agrees_with_the_layout_system 注释同步订正。 P3 文档五条：shell 的「8px 拖动热区」改成事实（拖动面=整条 48px 标题栏到第一个 caption； 缩放热区 4px）、state/update 的「每拍重建视图模型」改成 view_dirty 门控、message 的 Frame 注释 改成「只在忙碌时订阅」、pages/model 的 From<&AppState> 改成自由函数 model(..)、pages/tests 的守门 目录补 src/app。 r3 浅/深产品图与 r2 逐字节相同（视觉零变化）。
This commit is contained in:
AskaEth committed
1 parent
022778a5a7
commit
6b66adff44
9 files changed
+254
-42
No files matched your search
@@ -5,7 +5,7 @@
|
||||
//! ```text
|
||||
//! 用户动作 ──dispatch(AppCommand)──► backend ──► drain_events() ──► AppState::apply ──► 视图模型
|
||||
//! ▲ ▲
|
||||
//! └── pages::Message（导航/启动/停止）──────────────────┘ Frame + 33 ms 时间闸门
|
||||
//! └── pages::Message（导航/启动/停止）──────────────────┘ 忙碌 33 ms / 空闲 1 Hz
|
||||
//! ```
|
||||
//!
|
||||
//! 三条纪律：
|
||||
@@ -13,7 +13,17 @@
|
||||
//! 1. **不做乐观回显**：点了「Play」不改本地状态，等 `InstanceRuntimeChanged` 事件回来才变
|
||||
//! （否则事件与本地状态打架时，磁贴与进度条会各说一套）；
|
||||
//! 2. **状态只有一个写入口**：`AppState::apply`（事件）与 `AppState::navigate`（路由）；
|
||||
//! 3. **每拍重建视图模型**：`refresh_view()` 是快照 → 页面的唯一转换点。
|
||||
//! 3. **只在状态变了才重建视图模型**：`refresh_view()` 是快照 → 页面的唯一转换点，
|
||||
//! 由 `view_dirty` 门控（M90 r2/P2-1 起；此前是每拍无条件重建）。
|
||||
//!
|
||||
//! ## 两个时钟（M90 r3 / P2-2）
|
||||
//!
|
||||
//! | 场景 | 时钟 | 周期 | 谁触发 |
|
||||
//! |---|---|---|---|
|
||||
//! | 截图 / 有 Busy-Running 实例 / 有 toast / 派发后 3 s 观察窗 | `window::frames()` + 闸门 | 33 ms | [`App::needs_frames`] |
|
||||
//! | 其余（空闲） | `Heartbeat` 自续链 | 1 s | [`App::heartbeat`] |
|
||||
//!
|
||||
//! 两条时钟不会同时干活：心跳在 `needs_frames()` 为真时**只续排、不推进**，忙碌结束自动接回。
|
||||
|
||||
use std::time::{Duration, Instant};
|
||||
|
||||
@@ -21,13 +31,14 @@ use iced::{window, Task};
|
||||
|
||||
use unlucky_design::color::SchemeFlavor;
|
||||
use unlucky_presenter::dto::AppCommand;
|
||||
use unlucky_presenter::prefs::ThemeMode;
|
||||
use unlucky_presenter::router::Route;
|
||||
use unlucky_presenter::theme::config_poll_due;
|
||||
use unlucky_presenter::theme::CONFIG_POLL_TICKS;
|
||||
|
||||
use crate::pages;
|
||||
|
||||
use super::message::{route_of, Message};
|
||||
use super::state::{App, TICK, WATCH_WINDOW};
|
||||
use super::state::{App, IDLE_TICK, TICK, TICK_MS, WATCH_WINDOW};
|
||||
use super::CAPTURE_AT_FRAME;
|
||||
|
||||
impl App {
|
||||
@@ -45,10 +56,16 @@ impl App {
|
||||
iced::theme::Mode::None | iced::theme::Mode::Light => SchemeFlavor::Light,
|
||||
};
|
||||
eprintln!("[theme] 系统主题 = {mode:?}");
|
||||
self.set_flavor(flavor);
|
||||
// 优先级：`--theme` > 配置文件里的 `appearance.theme_mode` > 系统。
|
||||
// 早先这里无条件盖掉，于是「配置文件写 dark、系统是浅色」会出现
|
||||
// `light -> dark -> light` 的一次抖动，配置永远不生效（M90 r3 顺手修）。
|
||||
if self.options.theme.is_none() && self.theme_mode == ThemeMode::Auto {
|
||||
self.set_flavor(flavor);
|
||||
}
|
||||
Task::none()
|
||||
}
|
||||
Message::Frame => self.on_frame(),
|
||||
Message::Heartbeat => self.on_heartbeat(),
|
||||
Message::Window(event) => {
|
||||
if let window::Event::Resized(size) = event {
|
||||
self.width = size.width;
|
||||
@@ -116,9 +133,26 @@ impl App {
|
||||
self.view_dirty = true;
|
||||
self.refresh_view();
|
||||
|
||||
// 空闲心跳从这里开始自续（它是唯一能在「没有帧事件」时戳醒 app 的时钟）
|
||||
tasks.push(heartbeat());
|
||||
|
||||
Task::batch(tasks)
|
||||
}
|
||||
|
||||
/// 空闲心跳：只有「不需要帧节拍」时才推进一拍，然后**无条件续排**下一拍。
|
||||
///
|
||||
/// 为什么无条件续排：忙碌结束（游戏退出、toast 过期、观察窗关闭）之后没有别的东西会叫醒
|
||||
/// app，心跳链必须一直在，才能把低频时钟接回去。忙碌期间它只是每秒醒一次、什么都不做。
|
||||
fn on_heartbeat(&mut self) -> Task<Message> {
|
||||
if !self.needs_frames() {
|
||||
let now = Instant::now();
|
||||
let elapsed = now.saturating_duration_since(self.last_tick).max(IDLE_TICK);
|
||||
self.last_tick = now;
|
||||
self.advance(elapsed);
|
||||
}
|
||||
heartbeat()
|
||||
}
|
||||
|
||||
/// 一帧画完：①够 33 ms 就推进一拍 presenter ②截图模式在第 [`CAPTURE_AT_FRAME`] 帧出图。
|
||||
///
|
||||
/// 时间闸门按**真实流逝时间**判（不是数帧）：掉帧不会把游戏启动进度拖慢，
|
||||
@@ -153,13 +187,27 @@ impl App {
|
||||
self.state
|
||||
.toasts
|
||||
.expire(u64::try_from(elapsed.as_millis()).unwrap_or(u64::MAX));
|
||||
self.ticks += 1;
|
||||
// `ticks` 的单位是**33 ms 的一拍**，不是「advance 被调了几次」：空闲心跳一次跨 1 s，
|
||||
// 要记 30 拍，这样 `theme::config_poll_due`（30 拍 ≈ 1 秒）在两种时钟下语义一致。
|
||||
let ticks = elapsed.as_millis().div_euclid(u128::from(TICK_MS));
|
||||
let ticks = u64::try_from(ticks).unwrap_or(u64::MAX).max(1);
|
||||
let bucket_before = self.ticks / 30;
|
||||
self.ticks += ticks;
|
||||
let bucket_after = self.ticks / 30;
|
||||
|
||||
// 外部（AI / CLI）改了配置文件：主题跟着换，其余外观项等玻璃/背景 mission。
|
||||
// 截图通道不轮询（D30 可复现），所以整条判断短路在 `has_screenshot` 上。
|
||||
let reloaded = !self.has_screenshot()
|
||||
&& config_poll_due(self.ticks)
|
||||
&& self.bridge.reload_if_changed().is_some();
|
||||
//
|
||||
// 判据用**跨越**而不是 presenter 的 `config_poll_due(ticks)`（整除）：两个时钟的相位不同
|
||||
// （帧节拍按 33 ms 累加、空闲心跳按 30 拍累加），整除判定一旦错过一次就**再也命中不了**
|
||||
// ——实测 ticks=137 之后每拍 +30 永远不整除，配置轮询静默死掉。间隔真源仍是
|
||||
// `CONFIG_POLL_TICKS`（30 拍 × 33 ms ≈ 1 秒）。截图通道不轮询（D30 可复现）。
|
||||
let poll_due = self.ticks >= self.config_due_at;
|
||||
if poll_due {
|
||||
// 追到「当前 ticks 之后的第一个间隔点」，避免心跳把相位拖飞
|
||||
self.config_due_at = self.ticks + CONFIG_POLL_TICKS;
|
||||
}
|
||||
let reloaded =
|
||||
!self.has_screenshot() && poll_due && self.bridge.reload_if_changed().is_some();
|
||||
if reloaded {
|
||||
eprintln!("[settings] 配置文件变了，重读主题");
|
||||
self.apply_config_theme();
|
||||
@@ -169,15 +217,20 @@ impl App {
|
||||
self.refresh_view();
|
||||
}
|
||||
|
||||
// 取证开关（`ULL_TICK_TRACE=1`）：每 30 拍打一行。**空闲态应当打完一阵就彻底安静**
|
||||
// —— 那正是「不再订阅帧事件」的可观测形式（没有 Frame ⇒ 这里根本不会被调用）。
|
||||
if self.ticks.is_multiple_of(30) && std::env::var_os("ULL_TICK_TRACE").is_some() {
|
||||
// 取证开关（`ULL_TICK_TRACE=1`）：每跨过 30 拍（≈1 秒）打一行。
|
||||
// 忙碌态由帧节拍跨桶（≈1 行/秒），空闲态由心跳一次 +30 跨桶（**也是 1 行/秒**）——
|
||||
// 两种时钟都用同一把尺子，所以 trace 的密度本身就是「时钟在跑」的证据。
|
||||
if bucket_after != bucket_before && std::env::var_os("ULL_TICK_TRACE").is_some() {
|
||||
eprintln!(
|
||||
"[tick] n={} frames={} refreshes={} frames_wanted={}",
|
||||
"[tick] n={} frames={} refreshes={} clock={}",
|
||||
self.ticks,
|
||||
self.frames,
|
||||
self.refreshes,
|
||||
self.needs_frames()
|
||||
if self.needs_frames() {
|
||||
"frames"
|
||||
} else {
|
||||
"idle-1hz"
|
||||
}
|
||||
);
|
||||
}
|
||||
}
|
||||
@@ -266,3 +319,14 @@ impl App {
|
||||
iced::exit()
|
||||
}
|
||||
}
|
||||
|
||||
/// 空闲心跳的一拍：睡 1 秒再发 [`Message::Heartbeat`]。
|
||||
///
|
||||
/// 用 `Task::perform` + `std::thread::sleep` 而不是 `iced::time::every`：后者只在 tokio/smol
|
||||
/// 后端提供（本 crate 的执行器是 thread-pool，Cargo.toml 不在 scope）。代价是每拍让线程池里
|
||||
/// 一个线程阻塞 1 秒——按 1 Hz 算，对线程池（默认 = CPU 数）是可接受的占用。
|
||||
fn heartbeat() -> Task<Message> {
|
||||
Task::perform(async { std::thread::sleep(IDLE_TICK) }, |()| {
|
||||
Message::Heartbeat
|
||||
})
|
||||
}
|
||||
Reference in new issue
Block a user