From 19eaa794cf2be6de4a79e8d72d151f3cbbadf068 Mon Sep 17 00:00:00 2001 From: chengma Date: Mon, 6 Jul 2026 11:32:02 +0800 Subject: [PATCH] feat: track settings changes and enforce Chinese UI copy --- AGENTS.md | 1 + app/gui/main_window.py | 47 ++++++++ app/gui/tabs/settings.py | 124 ++++++++++++++++++++- docs/05-coding-rules.md | 1 + docs/06-tasks.md | 2 +- docs/api.md | 4 +- docs/current-state.md | 15 +-- docs/routes.md | 4 +- progress.md | 21 +++- tests/test_gui.py | 230 +++++++++++++++++++++++++++++++++++++++ 10 files changed, 430 insertions(+), 19 deletions(-) diff --git a/AGENTS.md b/AGENTS.md index 381da63..8695b75 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -53,4 +53,5 @@ T-006 前若 `tests/` 不存在，不运行 `unittest discover`，只在回复 ## 风格 - 文档与 UI 文案用中文，标识符用英文。 +- 所有用户可见文本必须使用中文：包括弹窗标题/正文/按钮、窗口标题、按钮、菜单、label、placeholder、tooltip、状态栏、空状态、确认框、运行日志、错误提示和成功提示等。URL、配置键、API 字段、模型别名、第三方/Shopee 原始错误可保留原文，但外围必须给中文解释；不得直接把英文技术报错裸露给用户。 - 内容面向 agent 执行：能落到“读什么、改什么、验证什么”。 diff --git a/app/gui/main_window.py b/app/gui/main_window.py index f720680..a53d473 100644 --- a/app/gui/main_window.py +++ b/app/gui/main_window.py @@ -27,6 +27,9 @@ class MainWindow(QMainWindow): ) self.setWindowTitle("cmshopee") self.resize(1180, 760) + self._settings_tab_index = TAB_TITLES.index("⑤ 设置") + self._last_tab_index = 0 + self._reverting_tab_change = False self.tabs = QTabWidget() self.tabs.setObjectName("mainTabs") self.tabs.setStyleSheet(TAB_STYLE) @@ -82,8 +85,52 @@ class MainWindow(QMainWindow): widget.refresh_tasks() def _on_tab_changed(self, index): + if self._reverting_tab_change: + self._last_tab_index = index + self.statusBar().showMessage(f"当前：{self.tabs.tabText(index)}") + return + previous = self._last_tab_index + if previous == self._settings_tab_index and index != previous: + if not self._confirm_leave_settings_tab(): + self._reverting_tab_change = True + try: + self.tabs.setCurrentIndex(previous) + finally: + self._reverting_tab_change = False + self._last_tab_index = previous + self.statusBar().showMessage(f"当前：{self.tabs.tabText(previous)}") + return + self._last_tab_index = index self.statusBar().showMessage(f"当前：{self.tabs.tabText(index)}") + def _settings_tab(self): + return self.tabs.widget(self._settings_tab_index) + + def _confirm_leave_settings_tab(self): + settings_tab = self._settings_tab() + if not hasattr(settings_tab, "is_dirty") or not settings_tab.is_dirty(): + return True + box = QMessageBox(self) + box.setWindowTitle("未保存更改") + box.setText("⑤ 设置有未保存更改。要先保存再离开吗？") + save_button = box.addButton("保存", QMessageBox.AcceptRole) + discard_button = box.addButton("放弃", QMessageBox.DestructiveRole) + cancel_button = box.addButton("取消", QMessageBox.RejectRole) + box.setDefaultButton(save_button) + box.exec() + clicked = box.clickedButton() + if clicked is save_button: + return bool(settings_tab.save_app_settings()) + if clicked is discard_button: + return bool(settings_tab.discard_unsaved_changes()) + return False + + def closeEvent(self, event): + if self._confirm_leave_settings_tab(): + event.accept() + return + event.ignore() + def open_accounts_tab(self): self.tabs.setCurrentIndex(TAB_TITLES.index("④ 账号管理")) diff --git a/app/gui/tabs/settings.py b/app/gui/tabs/settings.py index b6e3589..5487351 100644 --- a/app/gui/tabs/settings.py +++ b/app/gui/tabs/settings.py @@ -2,6 +2,8 @@ from __future__ import annotations +from contextlib import contextmanager + from ..widgets import * from ..workers import AIModelTestWorker as _RealAIModelTestWorker from ..workers import CMHubSettingsWorker as _RealCMHubSettingsWorker @@ -63,6 +65,8 @@ class SettingsTab(QWidget): self._loaded_cmhub_api_key = "" self._cmhub_auto_refresh_done = False self._compat_test_item_id = "" + self._dirty = False + self._suspend_dirty = 0 self.backend_combo = QComboBox() self.backend_combo.setObjectName("aiBackendCombo") @@ -173,6 +177,10 @@ class SettingsTab(QWidget): self.cdp_ready_timeout_spin.setObjectName("cdpReadyTimeoutSpin") self.cdp_ready_timeout_spin.setRange(1, 3600) self.save_config_button = QPushButton("保存设置") + self.unsaved_changes_label = QLabel("● 未保存更改") + self.unsaved_changes_label.setObjectName("settingsUnsavedChangesLabel") + self.unsaved_changes_label.setStyleSheet("color: #bc4c00; font-weight: 600;") + self.unsaved_changes_label.setVisible(False) self.allow_real_submit_checkbox = QCheckBox("允许真实提交线上商品") self.allow_real_submit_checkbox.setObjectName("allowRealSubmitCheckbox") self.allow_cover_update_checkbox = QCheckBox("允许更新封面") @@ -351,7 +359,12 @@ class SettingsTab(QWidget): panel_layout.addSpacing(18) panel_layout.addWidget(self.infrastructure_section_title) panel_layout.addLayout(path_form) - panel_layout.addWidget(self.save_config_button) + save_settings_layout = QHBoxLayout() + save_settings_layout.setContentsMargins(0, 0, 0, 0) + save_settings_layout.addWidget(self.save_config_button) + save_settings_layout.addWidget(self.unsaved_changes_label) + save_settings_layout.addStretch(1) + panel_layout.addLayout(save_settings_layout) panel_layout.addStretch(1) scroll = QScrollArea() @@ -380,9 +393,12 @@ class SettingsTab(QWidget): self._update_response_timeout_label ) self.save_config_button.clicked.connect(self.save_app_settings) + self._connect_dirty_signals() - self.refresh_models() - self._populate_app_settings() + with self._dirty_tracking_suspended(): + self.refresh_models() + self._populate_app_settings() + self._set_dirty(False) def _three_column_form(self, fields): layout = QGridLayout() @@ -436,6 +452,92 @@ class SettingsTab(QWidget): if self.status_callback is not None: self.status_callback(message) + @contextmanager + def _dirty_tracking_suspended(self): + self._suspend_dirty += 1 + try: + yield + finally: + self._suspend_dirty -= 1 + + def _connect_dirty_signals(self): + line_edits = ( + self.cmhub_base_url_edit, + self.cmhub_api_key_edit, + self.name_edit, + self.model_id_edit, + self.url_edit, + self.api_key_edit, + self.chrome_path_edit, + self.user_data_root_edit, + self.image_dir_edit, + self.db_path_edit, + ) + combos = ( + self.backend_combo, + self.cmhub_title_alias_combo, + self.cmhub_image_alias_combo, + self.category_combo, + self.api_type_combo, + self.default_text_model_combo, + self.default_image_model_combo, + self.resolution_combo, + ) + spin_boxes = ( + self.cmhub_connect_timeout_spin, + self.connect_timeout_spin, + self.title_concurrency_spin, + self.image_concurrency_spin, + self.retry_spin, + self.jpg_quality_spin, + self.default_debug_port_spin, + self.debug_port_start_spin, + self.debug_port_end_spin, + self.cdp_ready_timeout_spin, + self.max_items_per_run_spin, + self.max_parallel_accounts_spin, + ) + checkboxes = ( + self.cmhub_check_balance_checkbox, + self.enabled_checkbox, + self.allow_real_submit_checkbox, + self.allow_cover_update_checkbox, + self.close_success_tab_checkbox, + self.parallel_accounts_checkbox, + ) + for widget in line_edits: + widget.textChanged.connect(self._mark_dirty) + for widget in combos: + widget.currentIndexChanged.connect(self._mark_dirty) + for widget in spin_boxes: + widget.valueChanged.connect(self._mark_dirty) + for widget in checkboxes: + widget.toggled.connect(self._mark_dirty) + + def _mark_dirty(self, *args): + if self._suspend_dirty > 0: + return + self._set_dirty(True) + + def _set_dirty(self, dirty): + self._dirty = bool(dirty) + self.unsaved_changes_label.setVisible(self._dirty) + + def is_dirty(self): + return self._dirty + + def discard_unsaved_changes(self): + try: + saved = appconfig.load_config(self.config_path) + except Exception as exc: + self._show_error(exc) + return False + self._replace_config(saved) + self._populate_app_settings() + self._set_dirty(False) + self._set_status("已放弃未保存更改") + return True + def _maybe_auto_refresh_cmhub_models(self): if self._cmhub_auto_refresh_done: return @@ -594,7 +696,7 @@ class SettingsTab(QWidget): def save_app_settings(self, checked=False): settings = self._app_settings_values() if settings is None: - return + return False cmhub_key = self.cmhub_api_key_edit.text() if self._should_warn_plaintext_cmhub_api_key(cmhub_key): self._show_plaintext_cmhub_api_key_warning() @@ -606,11 +708,13 @@ class SettingsTab(QWidget): saved = appconfig.save_config(settings, path=self.config_path) except Exception as exc: self._show_error(exc) - return + return False self._replace_config(saved) self._populate_app_settings() + self._set_dirty(False) self._set_status("设置已保存") QMessageBox.information(self, "保存设置", "设置已保存") + return True def _app_settings_values(self): start_port = self.debug_port_start_spin.value() @@ -700,6 +804,10 @@ class SettingsTab(QWidget): self.config.update(internal) def _populate_app_settings(self): + if self._suspend_dirty <= 0: + with self._dirty_tracking_suspended(): + self._populate_app_settings() + return self._populate_role_model_combos() ai_cfg = appconfig.ai_config(self.config) self._set_combo_by_data(self.backend_combo, "cmhub") @@ -1011,7 +1119,7 @@ class SettingsTab(QWidget): image_count = self._cmhub_alias_count("image") balance = payload.get("points_balance") balance_text = f"；余额 {balance}" if balance is not None else "" - message = f"cmhub 连接成功：生文别名 {title_count} 个，生图别名 {image_count} 个{balance_text}" + message = f"cmhub 连接成功：生文别名 {title_count} 个，生图别名 {image_count} 个{balance_text}；别名已拉取，记得点『保存设置』持久化" self.cmhub_result_label.setText(message) self._set_status(message) @@ -1021,6 +1129,10 @@ class SettingsTab(QWidget): self._set_status(message) def _populate_cmhub_alias_combos(self, models, title_selected="", image_selected=""): + if self._suspend_dirty <= 0: + with self._dirty_tracking_suspended(): + self._populate_cmhub_alias_combos(models, title_selected, image_selected) + return self._populate_cmhub_alias_combo( self.cmhub_title_alias_combo, models, diff --git a/docs/05-coding-rules.md b/docs/05-coding-rules.md index eb44eac..5ba29b3 100644 --- a/docs/05-coding-rules.md +++ b/docs/05-coding-rules.md @@ -43,6 +43,7 @@ ## 5. 代码规范 - 标识符使用英文；UI 文案、注释、文档保持中文，与现有脚本一致。 +- 所有用户可见文本必须使用中文：包括弹窗标题/正文/按钮、窗口标题、按钮、菜单、label、placeholder、tooltip、状态栏、空状态、确认框、运行日志、错误提示和成功提示等。不要直接使用 `Save` / `Discard` / `Cancel` 等英文标准按钮露出给用户；如需三选一弹窗，使用自定义中文按钮文案（如“保存”“放弃”“取消”）。URL、配置键、API 字段、模型别名、第三方/Shopee 原始错误可保留原文，但外围必须有中文解释。 - 错误必须处理：CDP 超时、tab 找不到、上传失败、按钮禁用都要给明确提示，不吞错。 - 注释解释“为什么”（尤其 CDP 的坑：代理、Origin、就绪判断、拖拽落点），不复述“做了什么”。 - 复用现有脚本里已验证的 JS 字符串，不重写出不一致的版本。 diff --git a/docs/06-tasks.md b/docs/06-tasks.md index b075c35..b3634ef 100644 --- a/docs/06-tasks.md +++ b/docs/06-tasks.md @@ -129,7 +129,7 @@ | T-528 | ② 计费错误提示 + 余额展示 | T-526, T-527, T-303 | 依据 `docs/cmhub-integration-design.md` v3.2。② AI生成页把 cmhub 计费失败态显式化：通过 `CMHubError.code` 识别 `insufficient_points`，弹明确提示「点数不足，请先充值」并引导去网页端充值，本轮未开始任务可提前中止，不靠中文字符串匹配、不淹没在失败计数里；用 T-526 成功响应事件里的 `points_balance` 刷新②页剩余点数显示，`/balance` 仅作手动刷新/可选批量前预检；`points_cost`/`call_id` 记入脱敏 run_logs。只改② UI、`GenerateWorker` 事件处理/文案及 GUI 单测；不改 AI HTTP 协议、DB schema、Excel、Shopee/CDP 流程 | DONE | | T-529 | 默认 cmhub 网关并隐藏 AI 后端选择 | T-528, T-527 | 产品收口：后续普通用户只使用 cmhub 网关，不再在⑤设置页暴露「AI 后端」label 与 direct/cmhub 下拉框。方案：`DEFAULT_CONFIG.ai.backend` 改为 `cmhub`；⑤设置页默认直接展示 cmhub 网关配置（Base URL、API Key、生文/生图别名、刷新别名、测试连接/查余额），隐藏 direct 模型选择、模型详情、标题/图片模型角色下拉和「AI 后端」下拉；保存设置固定写 `ai.backend=cmhub`，允许先保存不完整 cmhub 配置，②生成时仍由 `app/ai.py` 对缺 Base URL/Key/别名给出「请去⑤配置 cmhub」错误。direct 代码、`config/ai_models.json` helper 与 direct 单测保留为内部兼容/手工回滚路径，但普通 UI 不提供切换入口。同步 `appconfig`、⑤设置页、②余额默认显示和 GUI/appconfig 单测；不改 AI HTTP 协议、DB schema、Excel、Shopee/CDP 流程 | DONE | | T-530 | cmhub Base URL 规整 + 404 明确提示 | T-526, T-527 | 现象：手动填入 Base URL 后点「刷新别名」提示 notfound（HTTP 404）。核对确认 cmshopee 请求已达标（`GET /api/v1/models` + Bearer，对齐对接文档 §4.4 与 cmhub `ModelsView` 路由，见 `docs/troubleshooting.md`「cmhub 刷新别名 notfound」与 `docs/cmhub-integration-design.md` v3.4）；404 根因是 Base URL 带多余 `/api(/v1)` 路径导致双拼、或所连实例未部署 `/api/v1/models`。方案：保存/请求前规整 Base URL——去掉结尾的 `/`、`/api`、`/api/v1` 等多余路径段，只保留 scheme+host(+port)（`appconfig.cmhub_request_url()` 或保存时统一处理），并在⑤输入框旁给出「只填网关根，如 https://host」提示；`_cmhub_code_for_status` 给 404 一个明确码/中文提示（如「cmhub 接口不存在，请检查 Base URL 或该实例是否已部署 /api/v1/models」），不再透传生涩英文原文。补 `test_appconfig`（Base URL 规整：带 `/api/v1`、带尾斜杠、带路径均归一到根）与 `test_ai`（404 → 明确提示）单测；不改 AI HTTP 协议、DB schema、Excel、Shopee/CDP 流程 | DONE | -| T-531 | ⑤ 设置未保存状态追踪 + 离开确认 | T-530, T-529, T-527 | 现象：在⑤填写 cmhub Base URL/API Key（或改任何设置）后，没有任何"未保存"提示；切换到其它 Tab、关闭窗口都不拦截，输入静默丢失。且「刷新别名/测试连接」用输入框实时文本、不落盘，成功后易误以为已保存。采用**方式 B（显式保存 + 脏状态追踪 + 离开确认）**，不改自动保存（API Key/半份 cmhub 配置不宜自动写盘）。方案：① 脏追踪——⑤ 所有可编辑控件（cmhub Base URL/API Key/别名下拉、模型详情、生成参数、路径端口、Shopee 更新安全等）接 `textChanged`/`currentIndexChanged`/`toggled`/`valueChanged` 置 `self._dirty=True`；「保存设置」成功后清 `False`；`save_app_settings()` 或等价保存入口必须返回成功/失败，失败时保留 dirty 并阻止离开。② 程序化回填保护——加载配置、保存后重载、刷新别名写回下拉、`_populate_app_settings()` / `_populate_cmhub_alias_combos()` 等必须用 `_suspend_dirty` 或 `QSignalBlocker` 避免误置脏；用户手动输入后由刷新/测试触发的 Base URL 规整仍应保持 dirty，提醒其保存。③ 离开拦截——`main_window` 切 Tab 与 `closeEvent` 若发现⑤ `_dirty` 为真，弹三选一「保存 / 放弃 / 取消」：保存则调保存入口且成功后再切/关；放弃必须重新从 `config.json` + `config/cmhub.json` 载入并回填控件，再清 dirty，不能只清标志；取消则留在⑤不切不关。`QTabWidget.currentChanged` 是切换后信号，实现时需维护上一 Tab index，并用 `_reverting_tab_change`/类似 guard 防止 `setCurrentIndex()` 递归触发。④ 视觉提示——「保存设置」旁显示「● 未保存更改」，保存或放弃回填后消失。⑤ 「刷新别名/测试连接」成功后提示语补「别名已拉取，记得点『保存设置』持久化」，**不**顺带偷偷保存 url/key（保持显式保存一致性）。只改 ⑤ 设置页、`main_window` Tab 切换/关闭守卫和 GUI 单测（脏标志置位/清除、切 Tab 保存/放弃/取消/保存失败分支、程序化回填不置脏、放弃会还原输入、currentChanged 回退不递归）；不改配置 schema、AI HTTP 协议、DB schema、Excel、Shopee/CDP 流程 | TODO | +| T-531 | ⑤ 设置未保存状态追踪 + 离开确认 | T-530, T-529, T-527 | 现象：在⑤填写 cmhub Base URL/API Key（或改任何设置）后，没有任何"未保存"提示；切换到其它 Tab、关闭窗口都不拦截，输入静默丢失。且「刷新别名/测试连接」用输入框实时文本、不落盘，成功后易误以为已保存。采用**方式 B（显式保存 + 脏状态追踪 + 离开确认）**，不改自动保存（API Key/半份 cmhub 配置不宜自动写盘）。方案：① 脏追踪——⑤ 所有可编辑控件（cmhub Base URL/API Key/别名下拉、模型详情、生成参数、路径端口、Shopee 更新安全等）接 `textChanged`/`currentIndexChanged`/`toggled`/`valueChanged` 置 `self._dirty=True`；「保存设置」成功后清 `False`；`save_app_settings()` 或等价保存入口必须返回成功/失败，失败时保留 dirty 并阻止离开。② 程序化回填保护——加载配置、保存后重载、刷新别名写回下拉、`_populate_app_settings()` / `_populate_cmhub_alias_combos()` 等必须用 `_suspend_dirty` 或 `QSignalBlocker` 避免误置脏；用户手动输入后由刷新/测试触发的 Base URL 规整仍应保持 dirty，提醒其保存。③ 离开拦截——`main_window` 切 Tab 与 `closeEvent` 若发现⑤ `_dirty` 为真，弹三选一「保存 / 放弃 / 取消」：保存则调保存入口且成功后再切/关；放弃必须重新从 `config.json` + `config/cmhub.json` 载入并回填控件，再清 dirty，不能只清标志；取消则留在⑤不切不关。`QTabWidget.currentChanged` 是切换后信号，实现时需维护上一 Tab index，并用 `_reverting_tab_change`/类似 guard 防止 `setCurrentIndex()` 递归触发。④ 视觉提示——「保存设置」旁显示「● 未保存更改」，保存或放弃回填后消失。⑤ 「刷新别名/测试连接」成功后提示语补「别名已拉取，记得点『保存设置』持久化」，**不**顺带偷偷保存 url/key（保持显式保存一致性）。只改 ⑤ 设置页、`main_window` Tab 切换/关闭守卫和 GUI 单测（脏标志置位/清除、切 Tab 保存/放弃/取消/保存失败分支、程序化回填不置脏、放弃会还原输入、currentChanged 回退不递归）；不改配置 schema、AI HTTP 协议、DB schema、Excel、Shopee/CDP 流程 | DONE | ## Phase 8 · 工程基础设施后续（`docs/engineering-review.md`） diff --git a/docs/api.md b/docs/api.md index 35326e5..f2ae7b0 100644 --- a/docs/api.md +++ b/docs/api.md @@ -381,9 +381,9 @@ T-523 后 GUI 已从旧 `app/gui.py` 拆为 `app/gui/` 包：`__init__.py` 负 ⑤ 设置当前要点（T-501）： -- `SettingsTab` 使用居中内容区 + 适度左右留白布局，当前留白已从 T-506 初始实现缩短到约 40%；实现上使用最大内容宽度和自适应 margin，避免固定像素导致小屏挤压。各设置组默认响应式 3 列表单：短字段占 1 格，长字段（URL/API Key/路径）跨 2 格或 3 格，窄窗口降为 2 列/1 列。点击「保存设置」成功后，调用 `QMessageBox.information` 弹出“设置已保存”轻量提示框，同时保留状态栏提示。T-531 后 `save_app_settings()` 或包装保存入口应返回 bool：成功写 `config.json` + `config/cmhub.json` 后清 dirty，失败保留 dirty 并让调用方阻止离开。 +- `SettingsTab` 使用居中内容区 + 适度左右留白布局，当前留白已从 T-506 初始实现缩短到约 40%；实现上使用最大内容宽度和自适应 margin，避免固定像素导致小屏挤压。各设置组默认响应式 3 列表单：短字段占 1 格，长字段（URL/API Key/路径）跨 2 格或 3 格，窄窗口降为 2 列/1 列。点击「保存设置」成功后，调用 `QMessageBox.information` 弹出“设置已保存”轻量提示框，同时保留状态栏提示。T-531 已完成：`save_app_settings()` 返回 bool，成功写 `config.json` + `config/cmhub.json` 后清 dirty，失败保留 dirty 并让调用方阻止离开。 - `SettingsTab` 的 cmhub 网关配置：Base URL 保存/刷新前规整为网关根；API Key 单独读写 `config/cmhub.json`；别名下拉来自 `fetch_cmhub_models()`，按 `operation_type` 分生文/生图并过滤未计价别名；「测试连接/查余额」调用 cmhub models + balance helper；保存设置固定写 `ai.backend=cmhub`。`refresh_cmhub_models()` / `test_cmhub_connection()` 使用输入框实时值但不得自动保存，成功文案提醒用户保存。`is_dirty()` / `discard_unsaved_changes()` / `_suspend_dirty`（或等价机制）用于 T-531：用户编辑置脏，程序化回填不置脏，放弃时重新加载 `config.json` + `config/cmhub.json` 并回填控件。 -- `MainWindow` 在 T-531 后负责⑤设置页离开守卫：切 Tab 与 `closeEvent` 发现 `SettingsTab.is_dirty()` 时弹保存/放弃/取消；保存成功后继续，保存失败或取消时回到⑤。由于 `QTabWidget.currentChanged` 是切换后信号，需维护上一个 index，并用 `_reverting_tab_change` 或等价 guard 防止 `setCurrentIndex()` 递归。 +- `MainWindow` 已负责⑤设置页离开守卫：切 Tab 与 `closeEvent` 发现 `SettingsTab.is_dirty()` 时弹保存/放弃/取消；保存成功后继续，保存失败或取消时回到⑤。由于 `QTabWidget.currentChanged` 是切换后信号，需维护上一个 index，并用 `_reverting_tab_change` 或等价 guard 防止 `setCurrentIndex()` 递归。 - 模型详情字段按 3 个组件一组排列：启用、类别、api_type、连接超时等短字段一格；服务商名、模型 ID 视宽度占一格或两格；网址、密钥跨整行或跨 2/3 列。 - 模型数据读写复用 `appconfig.list_ai_models(..., reveal_api_key=True)`、`add_ai_model()`、`update_ai_model()`、`delete_ai_model()`；保存时保留现有 `extra_body` 与 `timeout_seconds`。 - 密钥字段使用 `QLineEdit.Password` 打码显示；首次保存/变更非空 Key 前弹窗提示“本地明文保存”；明文只写入已 gitignore 的 `config/ai_models.json`，不得进入日志/导出。 diff --git a/docs/current-state.md b/docs/current-state.md index 9ff7b38..85308bb 100644 --- a/docs/current-state.md +++ b/docs/current-state.md @@ -6,15 +6,16 @@ ## 当前快照 - 日期：2026-07-06 -- 阶段：V0 单账号 CDP 流程已验证；V1 已完成 T-000 正式代码包结构、T-001 `app/editor.py` 模块化、T-002 `app/appconfig.py` 应用配置、T-003 SQLite 持久化地基、T-004 本地数据忽略规则、T-005 AI 模型清单后端、T-006 单元测试基座、T-101 账号 user-data-dir 工具、T-102 Chrome 启动器、T-103 登录保活与检测、T-103b 登录检测补充识别 Shopee accounts 登录页、T-104 PySide6 主窗口骨架、T-104b PySide6 worker 基座、T-105 Tab④ 账号管理、T-106 账号快捷方式、T-201 Excel 导入入库、T-202 Tab① 任务列表与导入按钮、T-202b Tab① 导入汇总栏、T-203 采集旧标题旧封面、T-204 回写旧字段到原 Excel、T-204b 采集完成自动回写、T-205 首次未配账号/Chrome 未启动/未登录引导保护、T-205b 采集后关闭自动新建商品页 tab、T-206 Tab① 删除指定批次软删除、T-301 AI 生成接口、T-302 Tab② AI 生成布局与任务列表、T-302p 提示词管理、T-303 Tab② 开始生成/停止/进度、T-303b ②/③ 商品ID筛选、T-401 Tab③ 更新列表筛选与开始更新确认、T-402 Tab③ 确认后串行更新、T-403 Tab③ 结果回写与结束汇总、T-501 Tab⑤ AI 模型管理 UI、T-501b Tab⑤ 角色与生成参数、T-501c Tab⑤ Shopee 更新安全开关、T-502 换封面删第一张再上传、T-503 敏感信息本地明文保存提示与日志脱敏、T-504 多账号并行/dry-run/运行日志、T-207 ①采集诊断日志、② AI生成图片失败诊断日志补丁、T-404a ②/③ 选中记录重置、T-404 真实 Shopee 单条更新冒烟验收、T-506 正式使用批量更新体验、T-507 正式批量更新移除普通流程测试商品 ID 限制、T-508 ③ 更新shopee生产化操作区、T-509 ② 新标题人工微调、T-510 ③ 检查本轮更新文案统一、T-505 全流程诊断日志扩展、T-511 语义色板与任务状态列上色、T-512 ③高风险按钮上色与①导入校验数字标红、T-513 登录点/③Tab危险标识/破坏性按钮上色、T-514 ①②③ 首次空状态引导卡片、T-515 批次阶段进度总览、T-516 ①筛选对齐②③、T-517 ⑤设置分区与兼容字段清理、T-518 ②左栏提示词区组件密度优化、T-519 ②AI生成长任务进度条与用户可读滚动日志、T-520 ②AI生成封面可选生成开关、T-521 依赖清单（锁版本 requirements）、T-522 CI 自动跑语法 + 单元 / GUI 测试、T-523 拆分 `app/gui.py` 为 `app/gui/` 包、T-523a ②/③ 新一轮运行前清空界面日志显示、T-404b 商品详情页加载失败 toast 自动捕获、T-524 PyInstaller 打包为免安装 exe、T-526 `app/ai.py` + `appconfig` 接入 cmhub backend、T-527 ⑤设置 cmhub 网关面板、T-528 ②计费错误提示与余额展示、T-529 默认 cmhub 网关并隐藏 AI 后端选择、T-530 cmhub Base URL 规整与 404 明确提示。 +- 阶段：V0 单账号 CDP 流程已验证；V1 已完成 T-000 正式代码包结构、T-001 `app/editor.py` 模块化、T-002 `app/appconfig.py` 应用配置、T-003 SQLite 持久化地基、T-004 本地数据忽略规则、T-005 AI 模型清单后端、T-006 单元测试基座、T-101 账号 user-data-dir 工具、T-102 Chrome 启动器、T-103 登录保活与检测、T-103b 登录检测补充识别 Shopee accounts 登录页、T-104 PySide6 主窗口骨架、T-104b PySide6 worker 基座、T-105 Tab④ 账号管理、T-106 账号快捷方式、T-201 Excel 导入入库、T-202 Tab① 任务列表与导入按钮、T-202b Tab① 导入汇总栏、T-203 采集旧标题旧封面、T-204 回写旧字段到原 Excel、T-204b 采集完成自动回写、T-205 首次未配账号/Chrome 未启动/未登录引导保护、T-205b 采集后关闭自动新建商品页 tab、T-206 Tab① 删除指定批次软删除、T-301 AI 生成接口、T-302 Tab② AI 生成布局与任务列表、T-302p 提示词管理、T-303 Tab② 开始生成/停止/进度、T-303b ②/③ 商品ID筛选、T-401 Tab③ 更新列表筛选与开始更新确认、T-402 Tab③ 确认后串行更新、T-403 Tab③ 结果回写与结束汇总、T-501 Tab⑤ AI 模型管理 UI、T-501b Tab⑤ 角色与生成参数、T-501c Tab⑤ Shopee 更新安全开关、T-502 换封面删第一张再上传、T-503 敏感信息本地明文保存提示与日志脱敏、T-504 多账号并行/dry-run/运行日志、T-207 ①采集诊断日志、② AI生成图片失败诊断日志补丁、T-404a ②/③ 选中记录重置、T-404 真实 Shopee 单条更新冒烟验收、T-506 正式使用批量更新体验、T-507 正式批量更新移除普通流程测试商品 ID 限制、T-508 ③ 更新shopee生产化操作区、T-509 ② 新标题人工微调、T-510 ③ 检查本轮更新文案统一、T-505 全流程诊断日志扩展、T-511 语义色板与任务状态列上色、T-512 ③高风险按钮上色与①导入校验数字标红、T-513 登录点/③Tab危险标识/破坏性按钮上色、T-514 ①②③ 首次空状态引导卡片、T-515 批次阶段进度总览、T-516 ①筛选对齐②③、T-517 ⑤设置分区与兼容字段清理、T-518 ②左栏提示词区组件密度优化、T-519 ②AI生成长任务进度条与用户可读滚动日志、T-520 ②AI生成封面可选生成开关、T-521 依赖清单（锁版本 requirements）、T-522 CI 自动跑语法 + 单元 / GUI 测试、T-523 拆分 `app/gui.py` 为 `app/gui/` 包、T-523a ②/③ 新一轮运行前清空界面日志显示、T-404b 商品详情页加载失败 toast 自动捕获、T-524 PyInstaller 打包为免安装 exe、T-526 `app/ai.py` + `appconfig` 接入 cmhub backend、T-527 ⑤设置 cmhub 网关面板、T-528 ②计费错误提示与余额展示、T-529 默认 cmhub 网关并隐藏 AI 后端选择、T-530 cmhub Base URL 规整与 404 明确提示、T-531 ⑤设置未保存状态追踪与离开确认。 - 技术栈：Python 3.10+，根目录 `requirements.txt` 锁定运行依赖，GitHub Actions 使用 Windows + Python 3.11 自动跑语法和单元/GUI 测试，PyInstaller onedir 打包为 Windows 免安装 exe；自研 CDP（websocket-client + requests），SQLite（sqlite3）+ `config.json` + openpyxl + AI（默认 cmhub: `config.json` 网关配置 + `config/cmhub.json` Key；direct: `config/ai_models.json` 通用 HTTP 内部兼容），GUI PySide6 5 Tab（已定）。 - 生产代码：已建立 `app/` 包 + 根入口 `main.py`；`app/cdp.py` 为已验证 CDP 底座，已区分 `CDP.close()` 断开 WebSocket 与 `close_tab()` 关闭浏览器 target；`app/editor.py` 已封装登录状态检测、标题/封面/采集/更新按钮能力，打开商品详情页时会安装 toast 捕获并在详情页加载失败时上浮商品失效/无权限等错误原因，`open_product()` 内部失败会关闭本轮自动新建的失败 tab，复用用户已有 tab 不关闭，并在采集结束后只关闭本轮自动新建的商品编辑页 tab、保留用户已有 tab，更新提交成功且设置开启时可关闭本轮自动新建商品页，确认后跳回商品列表页时关闭前等待 2 秒；`click_update()` 已补 Shopee 站点侧确认框处理，页面主「更新」后若出现 `確定您要更新商品嗎?` modal（`.eds-modal__content` / `.eds-modal__box`），只点击弹窗主按钮「更新」并避开「立即優化」，且记录确认后是否跳回商品列表页；`replace_cover()` 已实现更新封面统一先校验本地旧封面备份，再点第一张删除、可见确认框、等待图片管理器稳定和上传 input 恢复，随后先点击上传块模拟人工入口、短暂等待、重新获取 input、注入文件上传新图并等待 Shopee CDN 后拖到第一位的代码路径；已修正 2026-06-30 稳定等待回归：删除第一张前不要求上传 input 可用，只等图片列表稳定；删除后再等上传入口恢复；`重複/重复/duplicate` 上传 toast 会立即判为新封面重复错误；`app/image_paths.py` 已统一新采集/新生成图片路径为 `images///__old/new.jpg`，历史 DB 路径继续按原路径读取；`app/appconfig.py` 已实现 `config.json` 默认值/读写/更新、AI 参数、端口读取、Shopee 更新安全与执行模式默认值，`ai.backend` 默认 `cmhub`，`config/ai_models.json` 模型清单 CRUD/过滤/打码/测试连接与 OpenAI-compatible base URL 自动补 endpoint，`config/cmhub.json` cmhub Key 读写/打码 helper、cmhub Base URL 规整与请求 URL 拼接 helper，以及 `mask_secret()`、`sanitize_for_log()`、`redact_secrets()` 敏感信息脱敏工具；`app/ai.py` 已实现 `gen_title()`/`gen_cover()`/`generate_batch()`，支持 direct 通用 HTTP 和 cmhub 网关 backend，direct 按默认文本/图片模型调用，cmhub 按 Base URL + 生文/生图别名调用 title/image/models/balance 接口，生图 `image_url` 安全下载后转本地 JPEG，计费 metadata 通过事件回调传出，HTTP 404 映射为 `not_found` 并给出中文排障提示，且保留重试、错误脱敏、图片 URL/base64 解析、resolution resize、jpg_quality 保存、按 `ai.generate_cover` 选择只生成标题或先并发标题再并发封面、逐条 `set_generated`、失败 `mark_failed`、步骤级事件/错误回调与停止取消未开始项；`app/prompts.py` 已实现标题提示词读写、封面模板 CRUD 与变量替换；`app/db.py` 已实现 SQLite schema、连接 PRAGMA、批次/账号/任务与阶段写库函数、T-206 批次软删除标记与默认业务查询过滤、T-404a 本地生成结果/更新状态重置函数、T-509 `update_generated_title()` 本地新标题微调函数，以及 `run_logs/run_log_events` 运行日志函数；`app/diagnostics.py` 已实现 gitignore 本地诊断日志、滚动写入、结构化 payload 和自由文本脱敏；T-505 已把 Excel 导入/回写、③更新shopee、④Chrome 启动/登录检测、⑤AI模型测试连接接入 `run_logs/run_log_events` 与本地脱敏诊断日志；`app/excel.py` 已实现多 Excel 输入列解析、整文件列校验、脏行统计跳过、导入批次与任务入库、别名匹配统计、旧标题/旧封面路径回写原 Excel、更新结果回写原 Excel 与另存副本；`app/config.py` 已实现账号 slug 与 user-data-dir 创建；`app/accounts.py` 已实现账号 CRUD 服务、端口默认分配、启动登录、检测登录、生成快捷方式；`app/chrome.py` 已实现 Chrome 参数拼装、启动、CDP 端口探测、PowerShell `.lnk` 快捷方式生成；`app/gui/` 已由 T-523 拆分为 PySide6 GUI 包，包入口 `__init__.py` 兼容旧导入，`main_window.py` 放 `MainWindow`，`models.py` 放 3 个 TableModel，`widgets.py` 放色板/空状态/批次总览/helper，`workers.py` 放具体 GUI worker，`tabs/` 放 ①~⑤ Tab；整体仍实现 PySide6 `MainWindow`、五 Tab、顶部 Tab 栏防误点样式、统一语义色板、①②③任务状态列前景色、③「开始更新」warning 描边/文字色和①导入校验数字标红、④登录状态点上色、③更新shopee Tab warning 小圆点和删除类按钮 danger 样式、①②③首次空状态引导卡片、①②③批次阶段进度总览、① 导入采集的 Excel 导入按钮/导入汇总栏/批次筛选与删除批次软删除入口/QTableView 任务列表/未匹配筛选与略过标记/采集旧标题旧封面 worker/采集前账号就绪预检与④引导/采集完成自动回写/旧数据回写重试按钮与 worker/采集运行日志视图、② AI生成左右布局/标题与封面提示词管理/批次/店铺/商品ID/状态筛选/任务列表/新标题列本地微调/变量预览/生成封面图片成本开关/开始生成/停止/标题与图片双进度条/双击新旧封面预览/用户可读自动滚动AI生成运行日志/cmhub余额显示/点数不足弹窗/计费日志/重置生成结果与 `GenerateWorker`、③ 更新shopee批次/店铺/商品ID/状态筛选栏/任务列表/开始更新主按钮/重置更新状态右键菜单/更新安全开关拦截与「前往设置」跳转/「检查本轮更新」按钮/开始更新确认弹窗/确认后 `ApplyWorker` 按每批最大更新条数分批执行当前筛选全部可更新记录/账号就绪和端口冲突预检/按账号并行可选/逐条 `set_applied`/运行日志/自动回写结果到 Excel/结束汇总弹窗/手动回写重试按钮、④ 账号管理表格/弹窗/按钮/快捷方式与状态栏、密码明文保存提示、⑤ 设置页 T-529 后默认展示 cmhub 网关 Base URL、API Key、动态生文/生图别名下拉、Base URL 网关根提示、刷新别名与测试连接/查余额，不再显示「AI 后端」label/dropdown、direct 模型选择、模型详情或标题/图片模型角色下拉；保存设置固定写 `ai.backend=cmhub`，保存/刷新前会把 Base URL 规整为网关根，允许先保存不完整 cmhub 配置，生成时再提示补齐；direct 模型配置、`AIModelTestWorker` 和 `config/ai_models.json` 仅作内部兼容/手工回滚；通用生成参数、路径/端口、Shopee 更新安全与多账号并行设置继续持久化 `config.json`，cmhub Key 单独写入 `config/cmhub.json`，保存成功后弹轻量提示框；⑤ 设置页已将「Shopee 更新安全 / 执行模式」前置、将「基础设施（路径与端口）」后置，`test_item_id` 与 `dry_run` 不再有用户可操作控件，保存时保留 `test_item_id` 兼容值并固定 `dry_run=false`；③ 普通正式更新不再用 `test_item_id` 阻断非测试商品，确认弹窗不再显示测试商品 ID；`app/workers.py` 已实现 `BaseWorker`、通用 signals、取消标记和 `QThread` 启动包装。 -- 测试：`tests/` 已建立；T-522 已新增 GitHub Actions 在 push / pull_request 自动运行语法检查与全量 unittest；T-006 后纯逻辑改动必须运行 `python -m unittest discover -s tests`，当前覆盖 appconfig/db/config/accounts/chrome 启动与快捷方式/editor 登录检测（含 Shopee accounts 登录页）与商品 tab 生命周期、商品详情页失效/隐藏 toast 捕获、更新成功后关闭本轮新开 tab、更新封面备份缺失阻断/8张与9张先删第一张/删除确认/上传入口初始不可用/重复图片 toast/物流错误不阻断上传/删图后稳定等待/上传状态诊断/拖首位 mock 路径/Shopee 更新确认框、成功跳回商品列表和残留错误 toast 不覆盖成功 mock 路径/excel 导入/旧字段与更新结果回写/ai 标题与封面 HTTP 解析/`generate_batch` 正常、失败与停止/prompts 读写与渲染/gui ① 导入采集、删除批次软删除与采集诊断日志/gui ② AI生成布局与批次/店铺/商品ID/状态筛选、提示词管理、生成 worker、图片失败诊断日志、双击预览、重置生成结果与新标题本地编辑/gui ③ 更新shopee批次/店铺/商品ID/状态筛选列表、确认弹窗、开始更新主按钮、重置更新状态右键菜单、Shopee 更新安全拦截与前往设置、`ApplyWorker` 串行/检查/分批/按账号并行/端口冲突预检、运行日志、结果回写与汇总/gui ④ 账号管理、密码打码与明文保存提示/gui ⑤ AI 模型管理、API Key 打码与明文保存提示、角色/生成参数设置、Shopee 更新安全设置、检查按钮/每批最大更新条数/多账号并行设置、测试商品 ID 限制移除/worker signal 与线程包装、T-505 全流程诊断日志（import/write_back/apply/chrome_launch/login_check/ai_model_test）与本地日志脱敏、gui T-511 语义色板和任务状态列前景色、T-512 高风险按钮与导入校验数字样式、T-513 登录状态/Tab 标识/删除按钮样式、T-514 首次空状态引导卡片、T-515 批次阶段进度总览、T-516 ①筛选对齐②③、T-517 ⑤设置分区与兼容字段清理、T-518 ②左栏提示词区组件密度优化、T-519 ②AI生成长任务进度条与用户可读滚动日志、T-520 ②AI生成封面可选生成开关、T-524 PyInstaller 打包入口/构建脚本/发布目录校验、T-526 cmhub backend mock、T-527 设置页 cmhub backend 切换/别名下拉/worker mock、T-528 ②cmhub余额显示/计费日志/点数不足中止提示、T-529 默认 cmhub/隐藏 AI 后端选择/保存固定 cmhub/direct 兼容、T-530 cmhub Base URL 规整/404 not_found 明确提示；2026-07-01 已完成 5 个真实商品的 T-404 更新验收，后续 CDP/Shopee 改动仍需测试商品手动验证。 +- 测试：`tests/` 已建立；T-522 已新增 GitHub Actions 在 push / pull_request 自动运行语法检查与全量 unittest；T-006 后纯逻辑改动必须运行 `python -m unittest discover -s tests`，当前覆盖 appconfig/db/config/accounts/chrome 启动与快捷方式/editor 登录检测（含 Shopee accounts 登录页）与商品 tab 生命周期、商品详情页失效/隐藏 toast 捕获、更新成功后关闭本轮新开 tab、更新封面备份缺失阻断/8张与9张先删第一张/删除确认/上传入口初始不可用/重复图片 toast/物流错误不阻断上传/删图后稳定等待/上传状态诊断/拖首位 mock 路径/Shopee 更新确认框、成功跳回商品列表和残留错误 toast 不覆盖成功 mock 路径/excel 导入/旧字段与更新结果回写/ai 标题与封面 HTTP 解析/`generate_batch` 正常、失败与停止/prompts 读写与渲染/gui ① 导入采集、删除批次软删除与采集诊断日志/gui ② AI生成布局与批次/店铺/商品ID/状态筛选、提示词管理、生成 worker、图片失败诊断日志、双击预览、重置生成结果与新标题本地编辑/gui ③ 更新shopee批次/店铺/商品ID/状态筛选列表、确认弹窗、开始更新主按钮、重置更新状态右键菜单、Shopee 更新安全拦截与前往设置、`ApplyWorker` 串行/检查/分批/按账号并行/端口冲突预检、运行日志、结果回写与汇总/gui ④ 账号管理、密码打码与明文保存提示/gui ⑤ AI 模型管理、API Key 打码与明文保存提示、角色/生成参数设置、Shopee 更新安全设置、检查按钮/每批最大更新条数/多账号并行设置、测试商品 ID 限制移除/worker signal 与线程包装、T-505 全流程诊断日志（import/write_back/apply/chrome_launch/login_check/ai_model_test）与本地日志脱敏、gui T-511 语义色板和任务状态列前景色、T-512 高风险按钮与导入校验数字样式、T-513 登录状态/Tab 标识/删除按钮样式、T-514 首次空状态引导卡片、T-515 批次阶段进度总览、T-516 ①筛选对齐②③、T-517 ⑤设置分区与兼容字段清理、T-518 ②左栏提示词区组件密度优化、T-519 ②AI生成长任务进度条与用户可读滚动日志、T-520 ②AI生成封面可选生成开关、T-524 PyInstaller 打包入口/构建脚本/发布目录校验、T-526 cmhub backend mock、T-527 设置页 cmhub backend 切换/别名下拉/worker mock、T-528 ②cmhub余额显示/计费日志/点数不足中止提示、T-529 默认 cmhub/隐藏 AI 后端选择/保存固定 cmhub/direct 兼容、T-530 cmhub Base URL 规整/404 not_found 明确提示、T-531 设置页未保存状态/离开确认/放弃还原；2026-07-01 已完成 5 个真实商品的 T-404 更新验收，后续 CDP/Shopee 改动仍需测试商品手动验证。 - 数据：`config.json`、`config/ai_models.json`、`config/cmhub.json`、`cmshopee.db`、`cmshopee.db-*`、`chrome_user_data_dir/`、`images/`、`logs/`、`prompts/`、`title_prompt.txt`、`build/`、`dist/` 已由 `.gitignore` 排除；密码与 API Key 本地明文保存但保存/变更时提示，UI 打码，日志/导出必须脱敏；运营填写后的 Excel 业务文件默认忽略，标准空模板 `shopee待处理任务模板.xlsx` 可提交；`app/appconfig.py` 首次读取缺失的 `config.json` 时会在本地写默认配置，`app/db.py` 调用 `init_db()` 时会在本地创建 SQLite DB。 ## 既定设计要点（文档已定） - GUI：**PySide6 5 Tab 流水线**，顺序 ① 导入采集 → ② AI生成 → ③ 更新shopee → ④ 账号管理 → ⑤ 设置；后台任务用 worker/QThread/signal。 +- 全局用户可见文案规则：弹窗标题/正文/按钮、窗口标题、按钮、菜单、label、placeholder、tooltip、状态栏、空状态、确认框、运行日志、错误提示和成功提示等都必须使用中文。URL、配置键、API 字段、模型别名、第三方/Shopee 原始错误可保留原文，但必须配中文解释，不得裸露英文技术报错。 - 流水线阶段：imported → collected（采集旧标题/旧封面+下载+回写）→ generated（AI 提示词生成新标题/新封面）→ applied（③ 弹窗批量确认后改 Shopee 并提交）。**无 confirmed、无常驻提交开关。** - 存储：`config.json`（应用设置和 AI backend 内部字段，普通默认 cmhub）+ `config/ai_models.json`（direct AI 模型清单与本地明文 Key）+ `config/cmhub.json`（cmhub 本地明文 Key）+ SQLite `cmshopee.db`（账号/任务/各阶段结果，密码本地明文仅参考）+ openpyxl（Excel）+ 本地 `images/`（旧/新封面）；密码/API Key 保存或变更时提示，展示和日志/导出必须脱敏。 - 多账号隔离：每账号独立 user-data-dir（非 profile）。 @@ -38,7 +39,7 @@ | `requirements-build.txt` | 已有 | T-524 产出：打包环境依赖，先安装运行依赖再安装 PyInstaller 5.13.2；不作为运行时依赖文件 | | `.github/workflows/tests.yml` | 已有 | T-522 产出：push / pull_request 自动在 `windows-latest` + Python 3.11 安装 `requirements.txt`，设置 `QT_QPA_PLATFORM=offscreen`，运行 `python -m compileall app main.py` 与 `python -m unittest discover -s tests` | | `cmshopee.spec` / `scripts/build_exe.ps1` | 已有 | T-524 产出：PyInstaller onedir 构建配置与发布目录校验脚本；`cmshopee.spec` 不声明本地数据 `datas`，脚本检查 `dist\\cmshopee` 不含配置、DB、图片、日志、Chrome 登录态或提示词 | -| `app/gui/` | 已有 | T-523 产出：由旧 `app/gui.py` 拆分的 PySide6 GUI 包，包入口 `__init__.py` 继续兼容 `from app import gui` / `from app.gui import MainWindow`；`main_window.py` 放 MainWindow，`models.py` 放 3 个 TableModel，`widgets.py` 放色板/空状态/批次总览/helper，`workers.py` 放具体 GUI workers，`tabs/` 放 ①~⑤ Tab； T-104/T-105/T-106/T-202/T-202b/T-203/T-204/T-204b/T-205/T-206/T-207/T-302/T-302p/T-303/T-303b/T-401/T-402/T-403/T-404a/T-501/T-501b/T-501c/T-503/T-504/T-506/T-507/T-508/T-509/T-510/T-505/T-511/T-512/T-513/T-514/T-515/T-516/T-517/T-518/T-519/T-520/T-527/T-528/T-529/T-530 产出：PySide6 `QMainWindow` + 五 Tab；顶部 Tab 栏防误点样式；统一语义色板、①②③状态列前景色、③开始更新 warning 样式、①导入校验数字标红、④登录状态点、③Tab warning 小圆点、删除按钮 danger 样式、①②③首次空状态引导卡片和批次阶段进度总览；① 导入采集导入按钮、导入汇总栏、批次/店铺/商品ID/状态筛选、删除批次软删除按钮、`QTableView` 任务列表、未匹配筛选与略过标记、采集旧标题旧封面 worker（旧封面路径按批次/店铺/任务细分）、采集前账号就绪预检与④引导、采集完成自动回写与手动重试、采集运行日志视图；② AI生成左右布局、提示词管理、批次/店铺/商品ID/状态筛选栏、任务列表、新标题列本地编辑、开始生成/停止/标题与图片双进度条、双击新旧封面预览、用户可读自动滚动AI生成运行日志、重置生成结果与 `GenerateWorker`；③ 更新shopee批次/店铺/商品ID/状态筛选栏、任务列表、开始更新主按钮、重置更新状态右键菜单、Shopee 更新安全拦截与「前往设置」跳转、「检查本轮更新」按钮、开始更新确认弹窗、`ApplyWorker` 检查/分批串行/按账号并行、账号与端口预检、运行日志、逐条 `set_applied`、自动回写结果到 Excel、结束汇总弹窗与手动回写重试；④ 账号管理表格、账号弹窗、密码本地明文保存提示、启动登录、检测登录、快捷方式；⑤ 设置普通默认展示 cmhub 网关 Base URL、API Key、生文/生图动态别名、Base URL 网关根提示、刷新别名、测试连接/查余额并保存 `config/cmhub.json` Key，不再显示 AI 后端选择、direct 模型选择/详情/角色下拉；direct 配置仅内部兼容/手工回滚；通用生成参数、路径端口配置、三列 Shopee 更新安全与执行模式设置仍保留，高频安全/执行模式分区前置、基础设施分区后置，`test_item_id`/`dry_run` 兼容字段无用户入口且普通更新不再阻断正式更新；②封面模板「另存为/重命名/删除」低频操作已收敛进「模板操作」菜单；②AI生成底部已增加标题/图片双进度条，运行日志已改为用户可读、自动滚动、脱敏的长任务日志，且②本轮「生成封面图片（成本较高）」开关默认关闭并持久化到 `ai.generate_cover`，关闭时只生成标题并进入可更新状态；②cmhub模式会显示生成后的剩余点数，记录 points_cost/call_id 计费日志，并在点数不足时弹提示且中止未开始任务 | +| `app/gui/` | 已有 | T-523 产出：由旧 `app/gui.py` 拆分的 PySide6 GUI 包，包入口 `__init__.py` 继续兼容 `from app import gui` / `from app.gui import MainWindow`；`main_window.py` 放 MainWindow，`models.py` 放 3 个 TableModel，`widgets.py` 放色板/空状态/批次总览/helper，`workers.py` 放具体 GUI workers，`tabs/` 放 ①~⑤ Tab； T-104/T-105/T-106/T-202/T-202b/T-203/T-204/T-204b/T-205/T-206/T-207/T-302/T-302p/T-303/T-303b/T-401/T-402/T-403/T-404a/T-501/T-501b/T-501c/T-503/T-504/T-506/T-507/T-508/T-509/T-510/T-505/T-511/T-512/T-513/T-514/T-515/T-516/T-517/T-518/T-519/T-520/T-527/T-528/T-529/T-530/T-531 产出：PySide6 `QMainWindow` + 五 Tab；顶部 Tab 栏防误点样式；统一语义色板、①②③状态列前景色、③开始更新 warning 样式、①导入校验数字标红、④登录状态点、③Tab warning 小圆点、删除按钮 danger 样式、①②③首次空状态引导卡片和批次阶段进度总览；① 导入采集导入按钮、导入汇总栏、批次/店铺/商品ID/状态筛选、删除批次软删除按钮、`QTableView` 任务列表、未匹配筛选与略过标记、采集旧标题旧封面 worker（旧封面路径按批次/店铺/任务细分）、采集前账号就绪预检与④引导、采集完成自动回写与手动重试、采集运行日志视图；② AI生成左右布局、提示词管理、批次/店铺/商品ID/状态筛选栏、任务列表、新标题列本地编辑、开始生成/停止/标题与图片双进度条、双击新旧封面预览、用户可读自动滚动AI生成运行日志、重置生成结果与 `GenerateWorker`；③ 更新shopee批次/店铺/商品ID/状态筛选栏、任务列表、开始更新主按钮、重置更新状态右键菜单、Shopee 更新安全拦截与「前往设置」跳转、「检查本轮更新」按钮、开始更新确认弹窗、`ApplyWorker` 检查/分批串行/按账号并行、账号与端口预检、运行日志、逐条 `set_applied`、自动回写结果到 Excel、结束汇总弹窗与手动回写重试；④ 账号管理表格、账号弹窗、密码本地明文保存提示、启动登录、检测登录、快捷方式；⑤ 设置普通默认展示 cmhub 网关 Base URL、API Key、生文/生图动态别名、Base URL 网关根提示、刷新别名、测试连接/查余额并保存 `config/cmhub.json` Key，不再显示 AI 后端选择、direct 模型选择/详情/角色下拉；direct 配置仅内部兼容/手工回滚；通用生成参数、路径端口配置、三列 Shopee 更新安全与执行模式设置仍保留，高频安全/执行模式分区前置、基础设施分区后置，`test_item_id`/`dry_run` 兼容字段无用户入口且普通更新不再阻断正式更新；②封面模板「另存为/重命名/删除」低频操作已收敛进「模板操作」菜单；②AI生成底部已增加标题/图片双进度条，运行日志已改为用户可读、自动滚动、脱敏的长任务日志，且②本轮「生成封面图片（成本较高）」开关默认关闭并持久化到 `ai.generate_cover`，关闭时只生成标题并进入可更新状态；②cmhub模式会显示生成后的剩余点数，记录 points_cost/call_id 计费日志，并在点数不足时弹提示且中止未开始任务；⑤设置页已显示“● 未保存更改”，切 Tab/关闭窗口时拦截保存/放弃/取消，放弃会从 `config.json` 与 `config/cmhub.json` 重新回填，刷新别名/测试连接不自动保存并提醒点保存 | | `app/workers.py` | 已有 | T-104b 产出：`BaseWorker` + 通用 signals + 取消标记 + `run_worker()` QThread 包装 | | `app/accounts.py` | 已有 | T-105/T-106 产出：账号 CRUD 服务、目录创建、端口分配、启动登录、检测登录、快捷方式 | | `app/editor.py` | 已有 | T-001/T-103/T-205b/T-207/T-501c/T-502/T-404b + T-404 补丁产出：登录状态检测、打开商品页、详情页加载失败 toast 捕获与商品失效归因、读/写标题、读/下载封面、采集步骤回调、更新步骤回调、采集后关闭自动新建商品页 tab、上传前等待图片管理器稳定、点击上传块后注入文件并检测 CDN 后拖封面、更新封面统一先校验旧封面备份再删线上第一张、重复图片 toast 明确失败、页面主更新按钮、Shopee 站点侧确认框主按钮、apply_task；更新成功后可按设置关闭本轮自动新建商品页 | @@ -68,8 +69,8 @@ 任务状态以 [`06-tasks.md`](06-tasks.md) 为准，历史记录见 [`../progress.md`](../progress.md)。 -- 已完成：T-000（正式代码包结构）、T-001（`app/editor.py` 模块化）、T-002（`app/appconfig.py` + `config.json`）、T-003（SQLite 建表）、T-004（本地数据 gitignore）、T-005（AI 模型清单后端）、T-006（单元测试基座）、T-101（账号 slug/user-data-dir）、T-102（Chrome 启动器）、T-103（登录保活与检测）、T-103b（登录检测补充识别 Shopee accounts 登录页）、T-104（PySide6 五 Tab 主窗口骨架）、T-104b（PySide6 worker 基座）、T-105（Tab④ 账号管理）、T-106（账号快捷方式）、T-201（Excel 导入：解析多文件输入列入库）、T-202（Tab① 任务列表 + 导入按钮 + 别名匹配标记）、T-202b（Tab① 导入汇总栏）、T-203（采集旧标题+旧封面）、T-204（回写旧字段到原 Excel）、T-204b（采集完成自动回写旧字段）、T-205（首次未配账号 / Chrome 未启动 / 未登录引导保护）、T-205b（采集后关闭自动新建商品页 tab）、T-206（Tab① 删除指定批次软删除）、T-301（AI 生成接口）、T-302（Tab② 左右布局与任务列表）、T-302p（提示词管理）、T-303（Tab② 开始生成 + 停止 + 进度）、T-303b（②/③ 商品ID筛选）、T-401（Tab③ 更新列表筛选 + 开始更新确认弹窗）、T-402（Tab③ 确认后串行更新）、T-403（Tab③ 结果回写与结束汇总）、T-501（Tab⑤ AI 模型管理 UI）、T-501b（Tab⑤ 角色与生成参数）、T-501c（Tab⑤ Shopee 更新安全开关）、T-502（换封面删第一张再上传）、T-503（敏感信息本地明文保存提示与日志脱敏）、T-504（多账号并行 / dry-run / 运行日志）、T-207（①采集诊断日志）、② AI生成图片失败诊断日志补丁、T-404a（②/③ 选中记录重置）、T-404（真实 Shopee 单条更新冒烟验收）、T-506（正式使用批量更新体验）、T-507（正式批量更新：移除普通流程测试商品 ID 限制）、T-508（③ 更新shopee生产化操作区）、T-509（② 新标题人工微调）、T-510（③ 检查本轮更新文案统一）、T-505（全流程诊断日志扩展）、T-511（语义色板 + ①②③任务状态列上色）、T-512（③高风险按钮上色 + ①导入校验数字标红）、T-513（登录点 / ③Tab危险标识 / 破坏性按钮上色）、T-514（①②③ 首次空状态引导卡片）、T-515（批次阶段进度总览）、T-516（①筛选对齐②③）、T-517（⑤设置分区 + 清理兼容字段）、T-518（②左栏提示词区组件密度优化）、T-519（②AI生成长任务进度条 + 用户可读滚动日志）、T-520（②AI生成封面可选生成开关）、T-521（依赖清单：锁版本 requirements）、T-522（CI：自动跑语法 + 单元 / GUI 测试）、T-523（拆分 `app/gui.py` 为 `app/gui/` 包）、T-523a（②/③ 新一轮运行前清空界面日志显示）、T-404b（商品详情页加载失败 toast 自动捕获）、T-524（PyInstaller 打包为免安装 exe）、T-526（`app/ai.py` + `appconfig` 接入 cmhub backend）、T-527（⑤设置 cmhub 网关面板）、T-528（②计费错误提示 + 余额展示）、T-529（默认 cmhub 网关并隐藏 AI 后端选择）、T-530（cmhub Base URL 规整 + 404 明确提示）。 -- 下一个可领取任务：T-531（⑤ 设置未保存状态追踪 + 离开确认）；T-525 ruff 顺延到 T-531 后。 +- 已完成：T-000（正式代码包结构）、T-001（`app/editor.py` 模块化）、T-002（`app/appconfig.py` + `config.json`）、T-003（SQLite 建表）、T-004（本地数据 gitignore）、T-005（AI 模型清单后端）、T-006（单元测试基座）、T-101（账号 slug/user-data-dir）、T-102（Chrome 启动器）、T-103（登录保活与检测）、T-103b（登录检测补充识别 Shopee accounts 登录页）、T-104（PySide6 五 Tab 主窗口骨架）、T-104b（PySide6 worker 基座）、T-105（Tab④ 账号管理）、T-106（账号快捷方式）、T-201（Excel 导入：解析多文件输入列入库）、T-202（Tab① 任务列表 + 导入按钮 + 别名匹配标记）、T-202b（Tab① 导入汇总栏）、T-203（采集旧标题+旧封面）、T-204（回写旧字段到原 Excel）、T-204b（采集完成自动回写旧字段）、T-205（首次未配账号 / Chrome 未启动 / 未登录引导保护）、T-205b（采集后关闭自动新建商品页 tab）、T-206（Tab① 删除指定批次软删除）、T-301（AI 生成接口）、T-302（Tab② 左右布局与任务列表）、T-302p（提示词管理）、T-303（Tab② 开始生成 + 停止 + 进度）、T-303b（②/③ 商品ID筛选）、T-401（Tab③ 更新列表筛选 + 开始更新确认弹窗）、T-402（Tab③ 确认后串行更新）、T-403（Tab③ 结果回写与结束汇总）、T-501（Tab⑤ AI 模型管理 UI）、T-501b（Tab⑤ 角色与生成参数）、T-501c（Tab⑤ Shopee 更新安全开关）、T-502（换封面删第一张再上传）、T-503（敏感信息本地明文保存提示与日志脱敏）、T-504（多账号并行 / dry-run / 运行日志）、T-207（①采集诊断日志）、② AI生成图片失败诊断日志补丁、T-404a（②/③ 选中记录重置）、T-404（真实 Shopee 单条更新冒烟验收）、T-506（正式使用批量更新体验）、T-507（正式批量更新：移除普通流程测试商品 ID 限制）、T-508（③ 更新shopee生产化操作区）、T-509（② 新标题人工微调）、T-510（③ 检查本轮更新文案统一）、T-505（全流程诊断日志扩展）、T-511（语义色板 + ①②③任务状态列上色）、T-512（③高风险按钮上色 + ①导入校验数字标红）、T-513（登录点 / ③Tab危险标识 / 破坏性按钮上色）、T-514（①②③ 首次空状态引导卡片）、T-515（批次阶段进度总览）、T-516（①筛选对齐②③）、T-517（⑤设置分区 + 清理兼容字段）、T-518（②左栏提示词区组件密度优化）、T-519（②AI生成长任务进度条 + 用户可读滚动日志）、T-520（②AI生成封面可选生成开关）、T-521（依赖清单：锁版本 requirements）、T-522（CI：自动跑语法 + 单元 / GUI 测试）、T-523（拆分 `app/gui.py` 为 `app/gui/` 包）、T-523a（②/③ 新一轮运行前清空界面日志显示）、T-404b（商品详情页加载失败 toast 自动捕获）、T-524（PyInstaller 打包为免安装 exe）、T-526（`app/ai.py` + `appconfig` 接入 cmhub backend）、T-527（⑤设置 cmhub 网关面板）、T-528（②计费错误提示 + 余额展示）、T-529（默认 cmhub 网关并隐藏 AI 后端选择）、T-530（cmhub Base URL 规整 + 404 明确提示）、T-531（⑤设置未保存状态追踪 + 离开确认）。 +- 下一个可领取任务：T-525（引入 ruff lint + format）。 ## 当前已知限制 @@ -93,14 +94,14 @@ - T-528 已完成：②AI生成页在 cmhub backend 下显示生成后的剩余点数；GenerateWorker 读取计费 metadata，把 points_cost、points_balance、call_id 写入脱敏运行日志；遇到 CMHubError.code=insufficient_points 时弹出「点数不足，请先充值」并中止本轮未开始任务，不靠中文错误字符串匹配。未改 AI HTTP 协议、DB schema、Excel、Shopee/CDP 流程。 - T-529 已完成：普通默认 `ai.backend=cmhub`；⑤设置页不再显示「AI 后端」label/dropdown、direct 模型选择、模型详情或标题/图片模型角色下拉，直接展示 cmhub 网关配置；保存设置固定写 `ai.backend=cmhub` 且允许先保存不完整 cmhub 配置，②生成时仍由 `app/ai.py` 提示补齐 Base URL/API Key/别名。direct 代码和 `config/ai_models.json` 保留为内部兼容/手工回滚，不改 AI HTTP 协议、DB schema、Excel、Shopee/CDP 流程。 - T-530 已完成：`appconfig.normalize_cmhub_base_url()` / `cmhub_request_url()` 会把 cmhub Base URL 规整到 scheme+host(+port)，去掉 `/api`、`/api/v1`、其它路径、查询串和片段；⑤设置页输入框提示只填网关根，保存/刷新前同步规整；cmhub HTTP 404 统一映射为 `CMHubError(code="not_found")`，显示“cmhub 接口不存在，请检查 Base URL 或该实例是否已部署 /api/v1/models”。未改 AI HTTP 协议、DB schema、Excel、Shopee/CDP 流程。 -- T-531 待做：⑤设置页需要增加未保存状态追踪与离开确认。目标是保留显式保存，不自动保存 URL/API Key；用户修改任何设置后显示“● 未保存更改”，切换 Tab/关闭窗口时提供保存、放弃、取消三选一。放弃必须重新从 `config.json` 与 `config/cmhub.json` 回填控件；程序化回填和刷新别名写入下拉不得误置脏；`currentChanged` 回退需 guard 防递归。 +- T-531 已完成：⑤设置页增加未保存状态追踪与离开确认。用户修改设置后显示“● 未保存更改”；`save_app_settings()` 成功写入 `config.json` + `config/cmhub.json` 后清 dirty，失败保留 dirty；切换 Tab/关闭窗口时提供保存、放弃、取消三选一，且使用自定义中文按钮避免系统默认英文按钮；放弃会重新从 `config.json` 与 `config/cmhub.json` 回填控件；程序化回填和刷新别名写入下拉不置脏；`currentChanged` 回退使用 guard 防递归；刷新别名/测试连接成功只提醒保存，不自动落盘。 - T-521 已完成：根目录新增 `requirements.txt` 并锁定当前运行依赖版本；`docs/03-tech-stack.md` 和 `docs/README.md` 已改为 `python -m pip install -r requirements.txt` 安装；未新增运行时依赖、未改业务代码。 - T-522 已完成：新增 `.github/workflows/tests.yml`，在 push / pull_request 上用 Windows + Python 3.11 安装 `requirements.txt`，设置 `QT_QPA_PLATFORM=offscreen`，自动运行语法检查和全量单元/GUI 测试；CI 不连接真实 Shopee 或真实 AI。 - T-523 已完成：旧 `app/gui.py` 已拆为 `app/gui/` 包，公开导入路径保持兼容；本轮为纯结构重构，未改 Shopee/CDP、AI、DB、Excel 行为。 - T-523a 已完成：②点击「开始生成」、③点击「检查本轮更新」或「开始更新」时，先清空对应界面的旧日志显示并写入本轮开始摘要；运行中只追加本轮日志。历史 `run_logs/run_log_events` 和本地 `logs/` 不删除、不自动混入当前运行界面。 - T-404b 已完成：`open_product()` 在导航商品详情页前后安装 toast 捕获，等待详情页关键元素超时时会读取最近 `.eds-toasts`/toast/message 节点及页面缓存，把明确商品失效/商品不存在/无权限类 toast 上浮为 `商品失效：<原始toast>`；如果失败发生在 `open_product()` 内部，本轮自动新建的失败 tab 会关闭，复用用户已有 tab 不关闭；①列表“阶段”列只在这类明确失效错误时显示“商品失效”，底层仍保持 `stage=imported/status=failed`，其他打开失败仍显示“失败”。 - T-524 已完成：新增 PyInstaller onedir 打包配置、构建脚本和发布目录校验；打包版启动时工作目录切到 `cmshopee.exe` 所在目录，首次运行在 exe 同级生成/使用本地配置、DB、图片、日志、登录态和提示词；发布包不内置这些本地数据，用户后续更新采用关闭程序后覆盖程序文件、保留本地数据的方式。 -- Phase 7 cmhub 网关对接已完成并已做 T-529/T-530 收口：T-526 已接 `app/ai.py`/`appconfig` backend，T-527 已完成⑤设置页 cmhub 网关配置，T-528 已完成②cmhub余额展示、计费日志和点数不足中止提示，T-529 已把普通默认改为 cmhub 并隐藏后端选择，T-530 已完成 Base URL 规整和 404 中文提示。因 cmhub URL/API Key 保存体验仍有丢失风险，下一步先做 T-531；T-525 ruff 顺延。 +- Phase 7 cmhub 网关对接已完成并已做 T-529/T-530/T-531 收口：T-526 已接 `app/ai.py`/`appconfig` backend，T-527 已完成⑤设置页 cmhub 网关配置，T-528 已完成②cmhub余额展示、计费日志和点数不足中止提示，T-529 已把普通默认改为 cmhub 并隐藏后端选择，T-530 已完成 Base URL 规整和 404 中文提示，T-531 已完成设置页未保存状态与离开确认；下一步回到 T-525 ruff。 - T-301/T-303 已完成通用 HTTP AI 接口、批量生成编排和 GUI 接入 mock 单测；真实 AI 生成还需要在 `config/ai_models.json` 填入可用 url/model/api_key 后做一次成本可控的小样本实测；URL 可填完整 endpoint 或 OpenAI-compatible base URL。 - 本地 `config/ai_models.json` 若由旧版本或手工维护，可能缺少 `category`；启动报 “AI 模型 category 必须是 text 或 image” 时，按 [`troubleshooting.md`](troubleshooting.md) 只补 `category` / `enabled` 等非密钥字段，保留 API Key，且不要提交该文件。 diff --git a/docs/routes.md b/docs/routes.md index 4f5b2d5..46e3284 100644 --- a/docs/routes.md +++ b/docs/routes.md @@ -142,12 +142,12 @@ ## ⑤ 设置 -- 设置页整体布局：内容区居中，左右留白已从 T-506 初始实现缩短到约 40%；实现上用最大内容宽度 + 自适应 margin，而不是写死窗口像素。所有设置组默认响应式 3 列表单：短字段占 1 格，URL/API Key/路径等长字段跨 2 格或 3 格；窄窗口自动降为 2 列/1 列。点击「保存设置」成功后，状态栏显示“设置已保存”，并弹出轻量提示框。T-531 后保存成功还需清除未保存标记；保存失败时保留未保存标记并阻止离开。 +- 设置页整体布局：内容区居中，左右留白已从 T-506 初始实现缩短到约 40%；实现上用最大内容宽度 + 自适应 margin，而不是写死窗口像素。所有设置组默认响应式 3 列表单：短字段占 1 格，URL/API Key/路径等长字段跨 2 格或 3 格；窄窗口自动降为 2 列/1 列。点击「保存设置」成功后，状态栏显示“设置已保存”，并弹出轻量提示框。T-531 已完成：保存成功会清除未保存标记；保存失败时保留未保存标记并阻止离开。 - cmhub 网关配置（T-529 后普通用户唯一 AI 入口）：⑤设置页不再显示「AI 后端」label 或 direct/cmhub 下拉，直接展示 cmhub 网关 Base URL、API Key、生文别名、生图别名、连接超时、刷新别名、测试连接/查余额；Base URL 输入框旁提示“只填网关根，如 https://host”，保存/刷新前会规整掉 `/api`、`/api/v1` 或其它路径。刷新别名/测试连接使用当前输入框内容发起请求，但不自动保存 URL/API Key，成功文案需提醒“记得点保存设置持久化”。 - API Key 存在 `config/cmhub.json`，本地明文保存；保存/变更时提示；UI 使用密码框打码显示，不进入日志/导出。 - 生文/生图别名来自 `GET /api/v1/models` 动态下拉，过滤未定价别名并展示单价/需原图提示；网关临时不可达时保留已存别名。 - 保存设置固定写 `ai.backend=cmhub`。允许先保存不完整 cmhub 配置，②真正生成时如果缺 Base URL/API Key/别名，会提示去⑤补配置，不静默回退 direct。 - - T-531 后，⑤设置页任意可编辑控件变更都进入未保存状态，保存按钮旁显示“● 未保存更改”；切换到其它 Tab 或关闭窗口时弹出保存/放弃/取消。放弃必须重新从本地配置文件回填控件，避免未保存的 URL/API Key 留在界面上；程序化回填、保存后重载和刷新别名填充下拉不得误触发未保存状态。 + - T-531 已完成：⑤设置页任意可编辑控件变更都进入未保存状态，保存按钮旁显示“● 未保存更改”；切换到其它 Tab 或关闭窗口时弹出保存/放弃/取消。放弃会重新从本地配置文件回填控件，避免未保存的 URL/API Key 留在界面上；程序化回填、保存后重载和刷新别名填充下拉不会误触发未保存状态。 - direct 模型清单和 `config/ai_models.json` 代码路径保留为内部兼容/手工回滚，不在普通 UI 暴露。 - AI 生成参数：标题并发、图片并发、失败重试、分辨率、返回超时、jpg 质量等短字段按三列排列。 - 分辨率为 `512 / 1k / 2k / 4k`；返回超时只读展示 `resolution_timeouts[resolution]`，不单独编辑。 diff --git a/progress.md b/progress.md index 31e77b1..a59902c 100644 --- a/progress.md +++ b/progress.md @@ -1156,4 +1156,23 @@ - 背景：Claude Code 已在任务看板新增 T-531，用于解决⑤设置页填写 cmhub Base URL/API Key 后没有未保存提示、切 Tab/关闭窗口会静默丢输入的问题。 - 修正：认可“显式保存 + 脏状态追踪 + 离开确认”方案，不做自动保存；补充 `save_app_settings()` 成功/失败返回语义、放弃必须重新从 `config.json` 与 `config/cmhub.json` 回填、程序化回填不置脏、`currentChanged` 回退需 guard 防递归、刷新别名/测试连接成功只提醒保存不落盘。 - 文档：同步 `docs/06-tasks.md`、`docs/current-state.md`、`docs/routes.md`、`docs/api.md`；下一个可领取任务改为 T-531，T-525 ruff 顺延。 -- 验证：文档-only 更新，未运行单元测试。 \ No newline at end of file +- 验证：文档-only 更新，未运行单元测试。 + +## 【2026-07-06】T-531 完成 · ⑤设置未保存状态追踪 + 离开确认 + +- 状态：DONE +- 代码：⑤设置页新增 dirty 状态、保存按钮旁“● 未保存更改”提示、`is_dirty()`、`discard_unsaved_changes()`，`save_app_settings()` 改为返回成功/失败；保存成功清除 dirty，保存失败保留 dirty；放弃会重新读取 `config.json` 与 `config/cmhub.json` 并回填控件。 +- 主窗口：`MainWindow` 在切 Tab 和关闭窗口时检查⑤设置页未保存状态，弹保存/放弃/取消；保存失败或取消会回到⑤设置页；`currentChanged` 回退使用 guard 防止递归。 +- cmhub：刷新别名/测试连接继续使用输入框实时值，但不自动保存 URL/API Key；成功文案提醒“记得点『保存设置』持久化”。程序化回填和别名下拉刷新不触发 dirty。 +- 边界：未改配置 schema、AI HTTP 协议、DB schema、Excel、Shopee/CDP 流程。 +- 测试：新增 GUI 单测覆盖 dirty 置位/清除、程序化刷新不置脏、放弃还原 URL/Key、切 Tab 取消、保存失败、放弃分支、关闭窗口取消。 +- 验证：`python -m py_compile app\gui\tabs\settings.py app\gui\main_window.py tests\test_gui.py` 通过；`python -m unittest discover -s tests -p test_gui.py` 通过（86 tests）。 +- 下一步：T-525「引入 ruff（lint + format）+ 可选 pre-commit」。 + +## 【2026-07-06】文档规则 · 用户可见文本统一中文 + +- 规则：所有用户可见文本必须使用中文，覆盖弹窗标题/正文/按钮、窗口标题、按钮、菜单、label、placeholder、tooltip、状态栏、空状态、确认框、运行日志、错误提示和成功提示等。 +- 边界：URL、配置键、API 字段、模型别名、第三方/Shopee 原始错误可保留原文，但外围必须给中文解释；不得直接把英文技术报错裸露给用户。 +- 文档：同步 `AGENTS.md`、`docs/05-coding-rules.md`、`docs/current-state.md`。 +- 代码：同步修正 T-531 设置未保存离开确认框，改用自定义中文按钮“保存 / 放弃 / 取消”，避免系统默认按钮显示英文。 +- 验证：`python -m py_compile app\gui\main_window.py tests\test_gui.py` 通过；`python -m unittest discover -s tests -p test_gui.py` 通过（86 tests）；`python -m compileall app main.py` 通过；`python -m unittest discover -s tests` 通过（195 tests）；`git diff --check` 通过，仅有本机 LF/CRLF 提示。 \ No newline at end of file diff --git a/tests/test_gui.py b/tests/test_gui.py index ed4478b..808944c 100644 --- a/tests/test_gui.py +++ b/tests/test_gui.py @@ -84,6 +84,44 @@ class GuiTests(TempDirMixin, unittest.TestCase): self.assertIsNotNone(value) self.assertEqual(color, value.name()) + def make_fake_message_box(self, selected_label): + boxes = [] + + class FakeMessageBox: + AcceptRole = object() + DestructiveRole = object() + RejectRole = object() + + def __init__(self, parent=None): + self.parent = parent + self.title = "" + self.text = "" + self.buttons = {} + self.default_button = None + boxes.append(self) + + def setWindowTitle(self, title): + self.title = title + + def setText(self, text): + self.text = text + + def addButton(self, label, role): + button = object() + self.buttons[label] = button + return button + + def setDefaultButton(self, button): + self.default_button = button + + def exec(self): + return 0 + + def clickedButton(self): + return self.buttons[selected_label] + + return FakeMessageBox, boxes + def test_collect_table_shows_product_unavailable_only_for_explicit_error(self): account = SimpleNamespace(alias="papa", account_name="papa 店铺") invalid_task = SimpleNamespace( @@ -936,6 +974,198 @@ class GuiTests(TempDirMixin, unittest.TestCase): self.assert_removed(temp_dir) + def test_settings_tab_tracks_dirty_state_and_programmatic_cmhub_refresh_is_clean(self): + with self.make_temp_dir() as temp_dir: + cfg = self.make_config(temp_dir) + cfg["ai"] = appconfig.default_config()["ai"] + cfg["ai"]["backend"] = "cmhub" + cfg["ai"]["cmhub"] = { + "base_url": "https://cmhub.old", + "title_alias": "title-old", + "image_alias": "image-old", + "connect_timeout": 9, + "check_balance_before_batch": False, + } + cfg["cmhub_config_path"] = os.path.join(temp_dir, "config", "cmhub.json") + appconfig.save_cmhub_config({"api_key": "sk-old-secret"}, path=cfg["cmhub_config_path"]) + tab = SettingsTab(config=cfg, config_path=cfg["config_path"], ai_models_path=cfg["ai_models_path"]) + self.addCleanup(tab.close) + + self.assertFalse(tab.is_dirty()) + self.assertTrue(tab.unsaved_changes_label.isHidden()) + + tab._on_cmhub_finished( + { + "ok": True, + "models": [ + { + "alias": "title-fresh", + "operation_type": "title", + "pricing_status": "priced", + "prices": [{"resolution": "1K", "points_cost": 1}], + }, + { + "alias": "image-fresh", + "operation_type": "image", + "pricing_status": "priced", + "requires_image": True, + "prices": [{"resolution": "1K", "points_cost": 5}], + }, + ], + } + ) + self.assertFalse(tab.is_dirty()) + self.assertIn("记得点『保存设置』", tab.cmhub_result_label.text()) + + tab.cmhub_base_url_edit.setText("https://cmhub.example.com/api/v1/") + self.assertTrue(tab.is_dirty()) + self.assertFalse(tab.unsaved_changes_label.isHidden()) + + with mock.patch("app.gui.QMessageBox.information") as info: + self.assertTrue(tab.save_app_settings()) + + info.assert_called_once_with(tab, "保存设置", "设置已保存") + self.assertFalse(tab.is_dirty()) + self.assertTrue(tab.unsaved_changes_label.isHidden()) + + self.assert_removed(temp_dir) + + def test_settings_tab_discard_unsaved_changes_reloads_saved_config_and_key(self): + with self.make_temp_dir() as temp_dir: + cfg = self.make_config(temp_dir) + cfg["ai"] = appconfig.default_config()["ai"] + cfg["ai"]["backend"] = "cmhub" + cfg["ai"]["cmhub"] = { + "base_url": "https://cmhub.saved", + "title_alias": "title-saved", + "image_alias": "image-saved", + "connect_timeout": 11, + "check_balance_before_batch": False, + } + cfg["cmhub_config_path"] = os.path.join(temp_dir, "config", "cmhub.json") + appconfig.save_config( + {key: value for key, value in cfg.items() if key not in {"config_path", "ai_models_path", "cmhub_config_path"}}, + path=cfg["config_path"], + ) + appconfig.save_cmhub_config({"api_key": "sk-saved-secret"}, path=cfg["cmhub_config_path"]) + tab = SettingsTab(config=cfg, config_path=cfg["config_path"], ai_models_path=cfg["ai_models_path"]) + self.addCleanup(tab.close) + + tab.cmhub_base_url_edit.setText("https://cmhub.changed") + tab.cmhub_api_key_edit.setText("sk-changed-secret") + self.assertTrue(tab.is_dirty()) + + self.assertTrue(tab.discard_unsaved_changes()) + + self.assertFalse(tab.is_dirty()) + self.assertEqual("https://cmhub.saved", tab.cmhub_base_url_edit.text()) + self.assertEqual("sk-saved-secret", tab.cmhub_api_key_edit.text()) + self.assertTrue(tab.unsaved_changes_label.isHidden()) + + self.assert_removed(temp_dir) + + def test_main_window_cancelled_tab_change_keeps_user_on_dirty_settings(self): + with self.make_temp_dir() as temp_dir: + window = MainWindow(config=self.make_config(temp_dir)) + self.addCleanup(window.close) + settings_index = TAB_TITLES.index("⑤ 设置") + window.tabs.setCurrentIndex(settings_index) + settings_tab = window.tabs.widget(settings_index) + settings_tab.cmhub_base_url_edit.setText("https://cmhub.changed") + + message_box, boxes = self.make_fake_message_box("取消") + with mock.patch("app.gui.main_window.QMessageBox", message_box): + window.tabs.setCurrentIndex(0) + + self.assertEqual(1, len(boxes)) + self.assertEqual("未保存更改", boxes[0].title) + self.assertEqual(["保存", "放弃", "取消"], list(boxes[0].buttons)) + self.assertEqual(settings_index, window.tabs.currentIndex()) + self.assertTrue(settings_tab.is_dirty()) + settings_tab._set_dirty(False) + + self.assert_removed(temp_dir) + + def test_main_window_save_failure_keeps_user_on_dirty_settings(self): + with self.make_temp_dir() as temp_dir: + window = MainWindow(config=self.make_config(temp_dir)) + self.addCleanup(window.close) + settings_index = TAB_TITLES.index("⑤ 设置") + window.tabs.setCurrentIndex(settings_index) + settings_tab = window.tabs.widget(settings_index) + settings_tab.cmhub_base_url_edit.setText("https://cmhub.changed") + + message_box, _ = self.make_fake_message_box("保存") + with mock.patch("app.gui.main_window.QMessageBox", message_box), \ + mock.patch.object(settings_tab, "save_app_settings", return_value=False) as save_settings: + window.tabs.setCurrentIndex(0) + + save_settings.assert_called_once_with() + self.assertEqual(settings_index, window.tabs.currentIndex()) + self.assertTrue(settings_tab.is_dirty()) + settings_tab._set_dirty(False) + + self.assert_removed(temp_dir) + + def test_main_window_discard_tab_change_restores_saved_settings(self): + with self.make_temp_dir() as temp_dir: + cfg = self.make_config(temp_dir) + cfg["ai"] = appconfig.default_config()["ai"] + cfg["ai"]["backend"] = "cmhub" + cfg["ai"]["cmhub"] = { + "base_url": "https://cmhub.saved", + "title_alias": "title-saved", + "image_alias": "image-saved", + "connect_timeout": 10, + "check_balance_before_batch": False, + } + cfg["cmhub_config_path"] = os.path.join(temp_dir, "config", "cmhub.json") + appconfig.save_config( + {key: value for key, value in cfg.items() if key not in {"config_path", "ai_models_path", "cmhub_config_path"}}, + path=cfg["config_path"], + ) + appconfig.save_cmhub_config({"api_key": "sk-saved-secret"}, path=cfg["cmhub_config_path"]) + window = MainWindow(config=cfg, config_path=cfg["config_path"], ai_models_path=cfg["ai_models_path"]) + self.addCleanup(window.close) + settings_index = TAB_TITLES.index("⑤ 设置") + window.tabs.setCurrentIndex(settings_index) + settings_tab = window.tabs.widget(settings_index) + settings_tab.cmhub_base_url_edit.setText("https://cmhub.changed") + settings_tab.cmhub_api_key_edit.setText("sk-changed-secret") + + message_box, _ = self.make_fake_message_box("放弃") + with mock.patch("app.gui.main_window.QMessageBox", message_box): + window.tabs.setCurrentIndex(0) + + self.assertEqual(0, window.tabs.currentIndex()) + self.assertFalse(settings_tab.is_dirty()) + self.assertEqual("https://cmhub.saved", settings_tab.cmhub_base_url_edit.text()) + self.assertEqual("sk-saved-secret", settings_tab.cmhub_api_key_edit.text()) + + self.assert_removed(temp_dir) + + def test_main_window_close_cancel_ignores_dirty_settings_close(self): + with self.make_temp_dir() as temp_dir: + window = MainWindow(config=self.make_config(temp_dir)) + self.addCleanup(window.close) + settings_index = TAB_TITLES.index("⑤ 设置") + window.tabs.setCurrentIndex(settings_index) + settings_tab = window.tabs.widget(settings_index) + settings_tab.cmhub_base_url_edit.setText("https://cmhub.changed") + event = SimpleNamespace(accepted=False, ignored=False) + event.accept = lambda: setattr(event, "accepted", True) + event.ignore = lambda: setattr(event, "ignored", True) + + message_box, _ = self.make_fake_message_box("取消") + with mock.patch("app.gui.main_window.QMessageBox", message_box): + window.closeEvent(event) + + self.assertFalse(event.accepted) + self.assertTrue(event.ignored) + settings_tab._set_dirty(False) + + self.assert_removed(temp_dir) + def test_generate_tab_has_prompt_editors_and_task_table(self): with self.make_temp_dir() as temp_dir: title_prompt_path = os.path.join(temp_dir, "title_prompt.txt")