fix(dashboard): prevent superseded dashboard loads from mutating state (#301)

Adds two loadSeq guards in _loadDashboardDataInner: after the first await coreP and before the self-heal writeOpenclawConfig path. Prevents stale loads from polluting cache, popping toasts, or writing config after a newer load has been queued.
This commit is contained in:
cursor[bot]
2026-05-28 07:27:51 +08:00
committed by GitHub
parent 99b62ccc04
commit 8568dd0d2a

View File

@@ -219,7 +219,7 @@ function normalizeDefaultModelConfig(config) {
}
async function loadDashboardData(page, fullRefresh = false) {
// 并发保护如果上一次加载仍在进行跳过本次fullRefresh 除外)
// 将多次触发串行化到同一链上_dashboardLoadSeq 在排队时就会递增,供 inner 丢弃过期结果
const loadSeq = ++_dashboardLoadSeq
_dashboardLoadChain = _dashboardLoadChain.catch(() => {}).then(() => _loadDashboardDataInner(page, fullRefresh, loadSeq))
return _dashboardLoadChain
@@ -246,6 +246,8 @@ async function _loadDashboardDataInner(page, fullRefresh, loadSeq) {
// 第一波:服务状态 + 配置 + 版本 → 立即渲染统计卡片
const [servicesRes, configRes, panelConfigRes] = await coreP
// 排队期间可能已发起更新的加载:勿用淘汰中的结果改缓存、弹 toast 或写配置(避免与较新加载竞态)
if (loadSeq !== _dashboardLoadSeq || !page.isConnected) return
const services = servicesRes.status === 'fulfilled' ? servicesRes.value : []
let version = _dashboardVersionCache || {}
let config = configRes.status === 'fulfilled' ? configRes.value : null
@@ -273,6 +275,7 @@ async function _loadDashboardDataInner(page, fullRefresh, loadSeq) {
if (needsPatch) {
try {
const freshConfig = await api.readOpenclawConfig()
if (loadSeq !== _dashboardLoadSeq || !page.isConnected) return
let patched = false
if (!freshConfig.gateway) freshConfig.gateway = {}
if (!freshConfig.gateway.mode) { freshConfig.gateway.mode = 'local'; patched = true }