feat(setting): 添加分类与目录路由预览 - #678
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
PR-Agent Code Review
该 PR 新增目录匹配模式、分类草稿联动和后端路由预览，并补充了相应类型、界面与多语言支持。当前异步加载与草稿状态处理存在一处可覆盖既有分类配置的数据丢失风险，以及两处可能产生错误或过期预览结果的状态同步回归，建议修复后再合并。
审查提交：5ba1006
| openSharedDialog( | ||
| CategoryEditDialog, | ||
| {}, | ||
| { initialConfig: cloneDeep(categoryConfig.value) }, |
There was a problem hiding this comment.
配置覆写
categoryConfig 在请求完成前就以空配置初始化，并始终作为 initialConfig 传入，导致 CategoryEditDialog 跳过自身的数据加载。首次请求较慢或失败时，用户可以打开一个看似已加载但实际为空的编辑器并保存，从而用空配置覆盖服务端已有分类，违反编辑器必须基于已加载配置保存的约定。修正边界是区分“尚未加载”与“已加载为空”，不能将前者的占位值作为可保存的初始配置。
| try { | ||
| const result = await api.get<{ value?: unknown }>(`system/setting/${directoryMatchModeKey}`) | ||
| directoryMatchMode.value = result.value === 'specificity' ? 'specificity' : 'sequential' | ||
| } catch (error) { |
There was a problem hiding this comment.
错误回退
loadDirectoryMatchMode 在任何读取异常时都会把当前值改为 sequential。当已加载或已保存为 specificity 的页面遇到一次静默刷新失败时，预览会立即使用错误模式，随后保存目录还会将该回退值写回服务端；这违反了仅在设置缺失或值无效时回退的兼容约定。修正边界是读取失败时保留最后一次成功值，并避免持久化未成功加载的回退状态。
| } | ||
| } | ||
|
|
||
| watch([movieList, tvList], () => emit('draft-change', buildPayload()), { deep: true }) |
There was a problem hiding this comment.
伪草稿
parseConfig 在挂载时会修改两个受深度监听的列表，因此仅打开对话框也会发出 draft-change，而父组件会将任意此类事件标记为未保存草稿。此后静默刷新会拒绝同步实际未修改的分类配置，使其他会话或服务端更新无法进入后续预览，违反 dirty 状态只表示真实用户修改的不变式。修正边界是过滤初始化产生的事件，或根据初始快照判断配置是否确实发生变化。
|
已完成上一轮审查后的状态重构与回归：
本次自动 PR-Agent 工作流的红灯是三个回退模型均返回 |
1 participant
关联
变更
useDirectoryRouteSettings，以单一设置契约管理目录、匹配模式、远端快照、本地草稿、保存锁和静默刷新代次。null兜底规则；保存期间若继续编辑，则保留最新草稿并提示再次保存。saved / outdated / failed显式保存结果，避免旧快照保存成功后误报最新草稿已保存。依赖与兼容性
sequential，保持现有默认行为。验证
localStorage故障注入的既有基线用例subscribe page > keeps page state usable when sort storage reads or writes fail。git diff --check与敏感信息扫描通过。