diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml index 6d33a08..6f7e648 100644 --- a/.github/workflows/tests.yml +++ b/.github/workflows/tests.yml @@ -20,15 +20,21 @@ jobs: with: python-version: "3.11" cache: pip - cache-dependency-path: requirements.txt + cache-dependency-path: | + requirements.txt + requirements-dev.txt - name: Install dependencies run: | python -m pip install --upgrade pip python -m pip install -r requirements.txt + python -m pip install -r requirements-dev.txt + + - name: Ruff lint + run: python -m ruff check app tests main.py - name: Syntax check run: python -m compileall app main.py - name: Unit and GUI tests - run: python -m unittest discover -s tests \ No newline at end of file + run: python -m unittest discover -s tests diff --git a/app/gui/main_window.py b/app/gui/main_window.py index 8301ccb..ad5186a 100644 --- a/app/gui/main_window.py +++ b/app/gui/main_window.py @@ -188,7 +188,7 @@ class MainWindow(QMainWindow): box.setText("⑤ 设置有未保存更改。要先保存再离开吗？") save_button = box.addButton("保存", QMessageBox.AcceptRole) discard_button = box.addButton("放弃", QMessageBox.DestructiveRole) - cancel_button = box.addButton("取消", QMessageBox.RejectRole) + box.addButton("取消", QMessageBox.RejectRole) box.setDefaultButton(save_button) box.exec() clicked = box.clickedButton() diff --git a/docs/03-tech-stack.md b/docs/03-tech-stack.md index 007d76a..b2f08c6 100644 --- a/docs/03-tech-stack.md +++ b/docs/03-tech-stack.md @@ -24,6 +24,7 @@ | 运行日志 | SQLite `run_logs` / `run_log_events` | 已定 | ③ dry-run 与真实更新都留痕；结构化内容走脱敏 | | 图片处理 | `requests`（下载）+ `Pillow`（按分辨率/jpg质量存盘） | 部分待定 | 下载旧封面；新封面按 resolution 生成、jpg_quality 存盘 | | 测试 | `python -m compileall app main.py` + `unittest` + 手动 CDP/AI 验证 | 已定（分层） | 配置/DB/Excel/prompts 用单测；CDP/Shopee 与真实 AI 属集成验证或 mock | +| 代码质量 | `ruff`（lint + format 可用） | 已定 | T-525 引入开发检查依赖；CI 先强制安全类 lint，不做全仓格式化重排 | | 打包分发 | PyInstaller onedir（`cmshopee.spec`） | 已定 | 产出 Windows 免安装文件夹；不内置配置、DB、图片、日志、Chrome 登录态或提示词等本地数据；运行时统一写入程序同级 `data/` | ## 二、决策记录与演进 @@ -48,7 +49,10 @@ | 用途 | 命令 | | --- | --- | | 安装依赖 | `py -3.10 -m pip install -r requirements.txt` | +| 安装开发检查依赖 | `py -3.10 -m pip install -r requirements-dev.txt` | | 检查 PySide6 | `python -c "import PySide6; print(PySide6.__version__)"` | +| ruff 安全检查 | `py -3.10 -m ruff check app tests main.py` | +| ruff 格式化（按需） | `py -3.10 -m ruff format app tests main.py` | | 语法检查 | `py -3.10 -m compileall app main.py` | | 启动 GUI | `py -3.10 main.py` / `py -3.10 -m app` | | 单元测试（T-006 后） | `py -3.10 -m unittest discover -s tests` | @@ -75,6 +79,7 @@ set AUTO=1 && python prototypes/demo.py ## 四、依赖纪律 - 运行时第三方依赖统一写入根目录 `requirements.txt` 并锁定版本；换机或 CI 使用 `python -m pip install -r requirements.txt` 安装。 +- 开发检查依赖写入 `requirements-dev.txt`；当前只包含 ruff，不属于运行时或打包依赖。 - 打包依赖只写入 `requirements-build.txt`；PyInstaller 不属于运行时依赖，不写进 `requirements.txt`。 - 当前直接依赖锁定：PySide6 6.5.3、openpyxl 3.1.3、requests 2.31.0、websocket-client 1.6.1、Pillow 9.5.0。 - 新增第三方依赖前，先在本文说明用途、替代方案和维护成本，并同步更新 `requirements.txt`。 @@ -87,8 +92,9 @@ set AUTO=1 && python prototypes/demo.py | 层级 | 覆盖对象 | 验证方式 | | --- | --- | --- | | 语法 | 正式代码包与入口 | `python -m compileall app main.py` | +| lint | 正式代码、测试与入口 | `python -m ruff check app tests main.py`；当前只开启安全类规则，格式化命令可用但不强制全仓重排 | | 单元 | `appconfig/db/excel/prompts/config/chrome` 的纯逻辑 | T-006 建立 `tests/` 后运行 `python -m unittest discover -s tests`，使用临时目录/临时 SQLite/样例 Excel | | GUI 轻测 | PySide6 主窗口可创建、Tab 数量、worker signal 基本行为 | 可用 unittest 构造 `QApplication`，不连真实 Shopee | -| CI | push / pull_request 自动跑语法 + 全量 unittest | `.github/workflows/tests.yml` 安装 `requirements.txt`，运行 `python -m compileall app main.py` 与 `python -m unittest discover -s tests`；不连真实 Shopee/AI | +| CI | push / pull_request 自动跑 ruff + 语法 + 全量 unittest | `.github/workflows/tests.yml` 安装 `requirements.txt` 与 `requirements-dev.txt`，运行 `python -m ruff check app tests main.py`、`python -m compileall app main.py` 与 `python -m unittest discover -s tests`；不连真实 Shopee/AI | | 集成 | CDP/editor 操作 Shopee 测试商品 | 手动跑 `prototypes/demo.py` 或后续专用集成脚本 | | 外部 AI | 模型连接、文本/图像生成 | 默认 mock；真实调用只在手动验证时跑，避免成本和限流 | diff --git a/docs/05-coding-rules.md b/docs/05-coding-rules.md index efe8a7c..90a56d9 100644 --- a/docs/05-coding-rules.md +++ b/docs/05-coding-rules.md @@ -53,6 +53,7 @@ 完成前至少检查： - [ ] `python -m compileall app main.py` 通过（T-000 前仅文档改动不要求）。 +- [ ] `python -m ruff check app tests main.py` 通过（T-525 后；当前只开启安全类规则）。 - [ ] T-006 完成后，纯逻辑改动有对应 `unittest`，至少覆盖正常路径和一个失败路径。 - [ ] 涉及 CDP 的改动，在测试商品（ITEM_ID 51100639510）上实跑验证。 - [ ] 涉及更新封面删除线上第一张图时，必须先验证该任务 `old_cover_path` 本地备份存在；备份缺失不得删除线上图片。 @@ -64,6 +65,7 @@ - [ ] 回复里如实说明跑了什么命令、结果如何。 ```bash +python -m ruff check app tests main.py python -m compileall app main.py python -m unittest discover -s tests # T-006 完成且 tests/ 存在后 python prototypes/demo.py # 单账号闭环验证（不提交） @@ -72,6 +74,7 @@ python prototypes/demo.py # 单账号闭环验证（不提交） 测试命令时机： - T-006 前：`tests/` 尚未建立时，只要求运行与当前任务匹配的可用验证；不要因为 `python -m unittest discover -s tests` 缺目录而判失败。 +- T-525 后：`python -m ruff check app tests main.py` 成为代码改动的必跑项；当前只启用安全类 lint，不要求全仓格式化。 - T-006 后：`python -m unittest discover -s tests` 成为纯逻辑改动的必跑项；新增/修改 appconfig、db、excel、prompts、config、chrome 纯逻辑时同步补测。 ## 7. 绝不 diff --git a/docs/06-tasks.md b/docs/06-tasks.md index af0f9cb..9bd85e6 100644 --- a/docs/06-tasks.md +++ b/docs/06-tasks.md @@ -143,7 +143,7 @@ | --- | --- | --- | --- | --- | | T-541 | 打包版主窗口初始位置与小屏适配 | T-523, T-537, T-540 | Bug：在 Windows 10 虚拟机/小分辨率环境运行打包后的 `dist/cmshopee/cmshopee.exe` 或 release 包时，主窗口可能贴到屏幕左上角且左边/标题栏显示不全，看不到「蝦皮圈優化助手 v」标题，用户无法拖动窗口到中间。根因：`MainWindow.__init__()` 只调用 `self.resize(1180, 760)`，没有根据 `QApplication.primaryScreen().availableGeometry()` 限制初始尺寸、没有居中、没有保证窗口 frame/title bar 在可用屏幕范围内；PyInstaller 本身不是根因，小屏、高 DPI、任务栏占用、多显示器或 VM 分辨率更容易触发。方案：① 抽一个主窗口初始布局 helper（如 `_fit_and_center_window(window, preferred_size=(1180,760), margin=40)`），读取当前屏幕 `availableGeometry()`；② 初始宽高不超过可用区域减安全边距，保留合理最小值（如 960x640，若屏幕更小则继续适配）；③ `resize()` 后按可用区域中心 `move()`，确保 `frameGeometry()` 左上角不小于可用区域左上角，标题栏完整可见；④ offscreen/headless 测试环境不依赖真实屏幕时可跳过或用 mock geometry；⑤ 不保存/恢复历史窗口位置，避免把坏坐标持久化。同步 `docs/packaging.md` 打包验收：在目标 VM/小分辨率环境启动 release exe，窗口标题栏必须完整可见且可拖动。GUI 单测覆盖小屏 mock 下尺寸被限制、窗口被居中/夹进可用区域；不改业务流程、DB、AI、Excel、CDP/Shopee、打包产物结构 | DONE | | T-542 | GUI 用户可见 `shopee/Shopee` 文案统一改为「蝦皮」 | T-523, T-537 | 需求：运行程序后，GUI 上用户能看到的 `shopee` / `Shopee` 字符统一改成中文「蝦皮」，例如主 Tab「③ 更新shopee」改为「③ 更新蝦皮」，设置分区「Shopee 更新安全 / 执行模式」改为「蝦皮更新安全 / 执行模式」，引导、弹窗、状态栏、运行日志、空状态、右键/确认文案里的“更新 Shopee / 未打开 Shopee / 不回滚 Shopee”等也改为「蝦皮」。范围：只改**用户可见文案与 GUI 测试断言**，包括 `app/gui/widgets.py` 的 `TAB_TITLES`、`MainWindow` 里依赖 Tab 文案的判断、`tabs/apply.py`、`tabs/settings.py`、`tabs/collect.py`、`tabs/generate.py`、`gui/workers.py` 中的可见提示和相关 `tests/test_gui.py`；同时修正 `app/editor.py` 里会冒到 GUI 的自定义错误文案。边界：不得改内部标识符、配置键、DB 字段、文件名、URL、域名、CDP 选择器和第三方原始错误，例如 `shopee_update`、`cmshopee`、`shopee待处理任务模板.xlsx`、`seller.shopee.tw`、`.shopee-image-manager`、`window.__cmshopee_*`、`cmshopee.db` 必须保持原样；代码注释/技术文档中指平台事实可继续写 `Shopee`。验收：`rg -n "更新shopee|Shopee 更新安全|更新 Shopee|未打开 Shopee|不回滚 Shopee" app tests` 不命中；`python -m compileall app main.py` 与 `python -m unittest discover -s tests` 通过；不涉及 CDP 流程实跑 | DONE | -| T-525 | 引入 ruff（lint + format）+ 可选 pre-commit | T-006 | 依据 `docs/engineering-review.md` P1。加 `ruff` 配置（lint + format），先以现状为基线不做大规模风格重排，只开启安全规则（未用 import/变量、明显错误）；可选 `.pre-commit-config.yaml`；不改业务逻辑；CI（T-522）可串入 ruff 检查。数据模型渐进上 mypy 作为后续可选。当前因业务优先级调整顺延到 T-526~T-528 之后 | TODO | +| T-525 | 引入 ruff（lint + format）+ 可选 pre-commit | T-006 | 依据 `docs/engineering-review.md` P1。加 `ruff` 配置（lint + format），先以现状为基线不做大规模风格重排，只开启安全规则（未用 import/变量、明显错误）；可选 `.pre-commit-config.yaml`；不改业务逻辑；CI（T-522）可串入 ruff 检查。数据模型渐进上 mypy 作为后续可选。当前因业务优先级调整顺延到 T-526~T-528 之后 | DONE | | T-537 | 品牌名「蝦皮圈優化助手」（显示层中文品牌，cmshopee 保留为代号/标识符） | T-523 | 需求：项目/软件正式定名「蝦皮圈優化助手」。**原则**：产品**显示/品牌名** = 「蝦皮圈優化助手」；`cmshopee` 保留为**代码库/仓库/技术代号**，一切**标识符不改**——遵项目「文档与 UI 文案用中文、标识符用英文」规则，且避免非 ASCII 文件名破坏现有用户数据。**改（显示/品牌层）**：① `app/gui/main_window.py` 窗口标题 `setWindowTitle("cmshopee")` → `"蝦皮圈優化助手"`（任务栏/标题栏显示中文品牌）；② 文档产品名散文 `docs/00-ai-start-here.md`、`docs/README.md`、`AGENTS.md`（及 vision/brief 若有）把「cmshopee 是一个…」改为「蝦皮圈優化助手（代号 cmshopee）是一个…」，明确品牌名并保留代号；③ 空的 `README.md` 补品牌标题行；④ 可选：`app/gui/__init__.py` PySide6 缺失启动失败提示带上品牌名。**不改（技术标识符，保留 cmshopee）**：`app` 包名、`cmshopee.exe` 打包产物名、`cmshopee.db`(+`-wal/-shm`)/`db_path` 默认值、`cmshopee.spec`、`cmshopee.log`、`config.json`/`config/*.json` 文件名、slug/`chrome_user_data_dir`、仓库目录；`app/editor.py` 注入页面的 JS 全局 `window.__cmshopee_*`（JS 标识符须 ASCII）、`app/ai.py` multipart boundary `----cmshopee-*`、`app/workers.py` `thread._cmshopee_worker`、`app/chrome.py` 快捷方式内部前缀 `cmshopee-`。**决策点**：`app/excel.py` 回写文件名后缀 `_cmshopee回写` 为用户可见产物名——本任务**默认保留**（改动会影响已生成副本命名习惯），如需改「_優化助手回写」另行确认。**影响**：不改 db/exe/spec/包名 → 打包、更新流程、现有用户数据零影响。验收：窗口标题栏/任务栏/README/相关文档显示「蝦皮圈優化助手」；`cmshopee.db`/exe/包名不变；GUI 单测若断言窗口标题须同步更新；`python -m compileall app main.py` + `python -m unittest discover -s tests` 全绿。边界：纯显示/文档层，不改任何文件名、schema、流程、CDP/Shopee 逻辑 | DONE | | T-538 | 打包产物用户数据收进 `data/` 子文件夹（便携方案 1） | T-524 | 问题：当前 onedir 产物把用户数据（`config.json`、`config/ai_models.json`、`config/cmhub.json`、`cmshopee.db(+-wal/-shm)`、`chrome_user_data_dir/`、`images/`、`logs/`、`prompts/`、`title_prompt.txt`）生成在 exe 顶层，和程序文件/依赖文件混放：更新易误删数据、放进 Program Files 只读目录会写失败/被 VirtualStore 重定向、程序与数据关注点混杂。方案（保持便携）：① **统一数据根**——所有用户数据落到 exe 同级的单个 `data/` 子文件夹；`app/appconfig.py` 用显式 `data_dir`（默认 `/data`，源码运行时可为项目根/`data`）解析并返回绝对路径，**替代现在 `main.py` 的 chdir**（chdir 影响全进程相对路径/文件对话框默认目录，脆弱）；文件名保持不变（仍是 `cmshopee.db` 等），只是移入 `data/`。② **旧布局迁移**——新版首次运行若在 exe 顶层发现旧的 `config.json`/`cmshopee.db` 等，自动迁移进 `data/`（或明确提示手动迁移），不得对老用户生成空 `data/` 丢失原数据。③ **可写性检测**——启动时若 `data_dir` 不可写（如放在 Program Files），弹清晰错误「请把程序放到可写目录，勿放 Program Files」并优雅退出，不静默失败。④ **发布只含程序**——`cmshopee.spec`/`scripts/build_exe.ps1` 构建检查确保不打包 `data/` 或任何用户数据；发布 zip 不含 `data/`，从而「整包解压覆盖」也安全（覆盖程序文件、不碰 `data/`）；T-540 后当前打包脚本固定 PyInstaller 6.11.1，产物为 `cmshopee.exe` + `_internal/` onedir，具体版本和 release 命名由 T-540 维护。⑤ **更新流程前提**——原地更新=换程序留 `data/`，依赖：DB 迁移幂等且只向前（现 `init_db`+`ALTER TABLE ADD COLUMN`）、`config` 向前兼容（`load_config` 补默认值）。同步更新 `docs/packaging.md`（目录结构、首次运行数据位置、§5 更新步骤简化为「覆盖新版程序文件和依赖文件，`data/` 不动」）。测试：`test_appconfig` 断言各路径解析进 `data_dir`、可写性检测行为、旧布局迁移；`test_packaging` 断言产物不含 `data/`/用户数据。边界：只改路径解析层（`appconfig` 路径 + `main.py` 数据根 + 迁移/可写检测）、spec/构建脚本、打包文档；不改文件名、DB schema、业务逻辑、AI/Excel、CDP/Shopee 流程 | DONE | | T-539 | ⑤ 隐藏数据路径设置（设置简化，非防逆向） | T-538, T-517 | 背景：T-538 后用户数据固定收进 exe 同级 `data/`，⑤「路径与端口」里的 DB 路径（`dbPathEdit`）、图片目录（`imageDirEdit`）、账号数据根目录（`userDataRootEdit`）已不该由用户改动——改了会指向 `data/` 之外导致数据分裂/丢失。**定性说明**：本任务是**设置简化**，与 T-517/T-529 隐藏 `test_item_id`/`dry_run` 同一思路；**不是防逆向手段**——`cmshopee.db` 在磁盘上任何 SQLite 浏览器可直接打开，隐藏 UI 字段对逆向无意义，防逆向由另行任务（发布包加固/价值上移 cmhub）承担。方案：① ⑤普通设置页**隐藏** `db_path`/`image_dir`/账号数据根三个输入框（不再占表单位置），`config.json` 对应字段保留为内部兼容——手工编辑配置仍生效（内部回滚路径），加载/保存不丢字段；② **保留可见**：Chrome 路径（每台机器安装位置不同，真实配置需求）与默认端口/端口区间/CDP 就绪超时（端口冲突需可调）；③ 首次运行/迁移后各路径默认解析到 `data/` 下（沿用 T-538 的 `data_dir` 规则），无需用户感知。同步 `docs/routes.md` ⑤ 说明与 GUI 设置单测（隐藏字段不再可见、config 字段仍读写、手工配置值仍被尊重）。边界：只改⑤设置页展示层与文档；不改 `config.json` schema、路径解析逻辑（T-538 范围）、DB、业务流程、CDP/Shopee | TODO | diff --git a/docs/README.md b/docs/README.md index 227e181..f85b196 100644 --- a/docs/README.md +++ b/docs/README.md @@ -32,15 +32,17 @@ ```powershell python -m pip install -r requirements.txt +python -m pip install -r requirements-dev.txt +python -m ruff check app tests main.py python -m compileall app main.py python -m unittest discover -s tests python -m pip install -r requirements-build.txt powershell -ExecutionPolicy Bypass -File scripts\build_exe.ps1 ``` -`requirements.txt` 只锁定运行所需第三方依赖；打包环境使用 `requirements-build.txt` 安装 PyInstaller；T-538 后 `data/` 统一保存配置、SQLite、图片、登录态、日志、提示词和密钥，运营 Excel 仍是本地数据，不提交版本库。 +`requirements.txt` 只锁定运行所需第三方依赖；`requirements-dev.txt` 锁定 ruff 等开发检查依赖；打包环境使用 `requirements-build.txt` 安装 PyInstaller；T-538 后 `data/` 统一保存配置、SQLite、图片、登录态、日志、提示词和密钥，运营 Excel 仍是本地数据，不提交版本库。 -GitHub Actions 工作流见 `.github/workflows/tests.yml`，在 push / pull_request 上使用 Windows + Python 3.11 安装 `requirements.txt`，并自动运行语法检查和全量单元/GUI 测试。 +GitHub Actions 工作流见 `.github/workflows/tests.yml`，在 push / pull_request 上使用 Windows + Python 3.11 安装 `requirements.txt` 与 `requirements-dev.txt`，并自动运行 ruff、语法检查和全量单元/GUI 测试。 ## 任务 / 进度 / 当前状态 diff --git a/docs/cmhub-integration-design.md b/docs/cmhub-integration-design.md index 044976f..09e4c4e 100644 --- a/docs/cmhub-integration-design.md +++ b/docs/cmhub-integration-design.md @@ -187,4 +187,4 @@ cmhub 返回结构化 `{error:{code}}`。映射层**按 `code` 优先分支**（ - **第五步（T-530）**：Base URL 规整到网关根，404 给出明确中文提示。 - 文档随每步同步。 -> 当前已在 `docs/06-tasks.md` 落成 T-526~T-530；下一步回到 T-525 工程基础设施，仍遵守 `docs/05-coding-rules.md` 验证清单。 +> 当前已在 `docs/06-tasks.md` 落成 T-526~T-535，且 T-525 工程基础设施已完成；下一步按看板进入 T-539 设置简化，仍遵守 `docs/05-coding-rules.md` 验证清单。 diff --git a/docs/current-state.md b/docs/current-state.md index 32f13ea..8d8937d 100644 --- a/docs/current-state.md +++ b/docs/current-state.md @@ -8,10 +8,11 @@ - 日期：2026-07-07 - 阶段：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-105b ④启动登录复用已打开 Chrome、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⑤ 蝦皮更新安全开关、T-502 换封面删第一张再上传、T-503 敏感信息本地明文保存提示与日志脱敏、T-504 多账号并行/dry-run/运行日志、T-207 ①采集诊断日志、② AI生成图片失败诊断日志补丁、T-404a ②/③ 选中记录重置、T-404 真实 Shopee 单条更新冒烟验收、T-506 正式使用批量更新体验、T-507 正式批量更新移除普通流程测试商品 ID 限制、T-508 ③ 更新蝦皮生产化操作区、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-532 ⑤cmhub连接成功提示显示账号名、T-533 ②增量生成：按缺失组件补生成、T-534 ②重置增强：多选/筛选范围 + 按组件重置、T-535 ②cmhub生成标题读取等待固定600秒、T-536 GUI按钮圆角全局统一、T-537 品牌名显示为蝦皮圈優化助手、T-538 打包产物用户数据收进 `data/`、T-540 打包版本号与 release 产物命名统一、T-541 打包版主窗口初始位置与小屏适配、T-542 GUI 用户可见 shopee/Shopee 文案统一改为蝦皮。 - T-542 已完成：运行程序后 GUI 用户可见 `shopee/Shopee` 平台名统一显示为「蝦皮」（例如「③ 更新蝦皮」「蝦皮更新安全」）；内部标识符、配置键、URL、域名、CDP 选择器和第三方原始错误仍保留原名。 +- T-525 已完成：根目录新增 `pyproject.toml` 与 `requirements-dev.txt`，引入 ruff 0.5.7；当前只开启安全类规则（`E4/E7/E9/F`），CI 已在语法检查和 unittest 前运行 `python -m ruff check app tests main.py`；`ruff format` 配置可用，但本轮不做全仓格式化重排。 - 测试卫生修正：`WriteBackWorker` 的 GUI 单测不再使用相对路径 `db.sqlite`，改为临时目录内 SQLite，避免跑测试后在项目根目录重新生成空 `db.sqlite`。 -- 技术栈：正式打包固定 Python 3.10（脚本调用 `py -3.10`），CI 使用 Windows + Python 3.11 自动跑语法和单元/GUI 测试，根目录 `requirements.txt` 锁定运行依赖，PyInstaller onedir 打包为 Windows 免安装 exe；自研 CDP（websocket-client + requests），SQLite（sqlite3）+ `data/config.json` + openpyxl + AI（默认 cmhub: `data/config.json` 网关配置 + `data/config/cmhub.json` Key；direct: `data/config/ai_models.json` 通用 HTTP 内部兼容），GUI PySide6 5 Tab（已定）。 +- 技术栈：正式打包固定 Python 3.10（脚本调用 `py -3.10`），CI 使用 Windows + Python 3.11 自动跑 ruff lint、语法和单元/GUI 测试，根目录 `requirements.txt` 锁定运行依赖，`requirements-dev.txt` 锁定开发检查依赖（ruff），PyInstaller onedir 打包为 Windows 免安装 exe；自研 CDP（websocket-client + requests），SQLite（sqlite3）+ `data/config.json` + openpyxl + AI（默认 cmhub: `data/config.json` 网关配置 + `data/config/cmhub.json` Key；direct: `data/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、注入文件上传新图并等待 蝦皮 CDN 后拖到第一位的代码路径；已修正 2026-06-30 稳定等待回归：删除第一张前不要求上传 input 可用，只等图片列表稳定；删除后再等上传入口恢复；`重複/重复/duplicate` 上传 toast 会立即判为新封面重复错误；`app/image_paths.py` 已统一新采集/新生成图片路径为 `images///__old/new.jpg`，历史 DB 路径继续按原路径读取；`app/appconfig.py` 已实现 `config.json` 默认值/读写/更新、AI 参数、端口读取、蝦皮更新安全与执行模式默认值，`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-534 本地生成结果/更新状态重置函数（②生成结果可按标题/封面组件重置并保持 generated）、T-509 `update_generated_title()` 本地新标题微调函数，以及 `run_logs/run_log_events` 运行日志函数；`app/diagnostics.py` 已实现 gitignore 本地诊断日志、滚动写入、结构化 payload 和自由文本脱敏；T-505 已把 Excel 导入/回写、③更新蝦皮、④Chrome 启动/登录检测、⑤AI模型测试连接接入 `run_logs/run_log_events` 与本地脱敏诊断日志；`app/excel.py` 已实现多 Excel 输入列解析、整文件列校验、脏行统计跳过、导入批次与任务入库、别名匹配统计、旧标题/旧封面路径回写原 Excel、更新结果回写原 Excel 与另存副本；`app/config.py` 已实现账号 slug 与 user-data-dir 创建；`app/accounts.py` 已实现账号 CRUD 服务、端口默认分配、启动登录幂等复用已打开 Chrome、检测登录、生成快捷方式；`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 描边/文字色和①导入校验数字标红、④登录状态点上色、③更新蝦皮 Tab warning 小圆点和删除类按钮 danger 样式、①②③首次空状态引导卡片、①②③批次阶段进度总览、① 导入采集的 Excel 导入按钮/导入汇总栏/批次筛选与删除批次软删除入口/QTableView 任务列表/未匹配筛选与略过标记/采集旧标题旧封面 worker/采集前账号就绪预检与④引导/采集完成自动回写/旧数据回写重试按钮与 worker/采集运行日志视图、② AI生成左右布局/标题与封面提示词管理/批次/店铺/商品ID/状态筛选/任务列表/新标题列本地微调/变量预览/生成封面图片成本开关/开始生成/停止/标题与图片双进度条/双击新旧封面预览/用户可读自动滚动AI生成运行日志/cmhub余额显示/点数不足弹窗/计费日志/重置生成结果与 `GenerateWorker`，且②重置支持多选/当前筛选结果并可只重置标题、只重置封面或重置全部；③ 更新蝦皮批次/店铺/商品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` 仅作内部兼容/手工回滚；通用生成参数、路径/端口、蝦皮更新安全与多账号并行设置继续持久化 `config.json`，cmhub Key 单独写入 `config/cmhub.json`，保存成功后弹轻量提示框；⑤ 设置页已将「蝦皮更新安全 / 执行模式」前置、将「基础设施（路径与端口）」后置，`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（含 T-105b 启动登录复用已运行 Chrome）/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 ③ 更新蝦皮批次/店铺/商品ID/状态筛选列表、确认弹窗、开始更新主按钮、重置更新状态右键菜单、蝦皮更新安全拦截与前往设置、`ApplyWorker` 串行/检查/分批/按账号并行/端口冲突预检、运行日志、结果回写与汇总/gui ④ 账号管理、密码打码与明文保存提示/gui ⑤ AI 模型管理、API Key 打码与明文保存提示、角色/生成参数设置、蝦皮更新安全设置、检查按钮/每批最大更新条数/多账号并行设置、测试商品 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 设置页未保存状态/离开确认/放弃还原、T-532 cmhub连接成功账号名提示、T-533 增量生成组件口径、T-534 ②按标题/封面组件重置与多选/筛选范围重置、T-535 cmhub标题请求读取等待固定600秒、T-536 GUI按钮全局基础样式与语义色按钮叠加样式、T-537 主窗口标题显示中文品牌；2026-07-01 已完成 5 个真实商品的 T-404 更新验收，后续 CDP/Shopee 改动仍需测试商品手动验证。 +- 测试：`tests/` 已建立；T-522 已新增 GitHub Actions 在 push / pull_request 自动运行语法检查与全量 unittest，T-525 后 CI 先运行 ruff 安全类 lint；T-006 后纯逻辑改动必须运行 `python -m unittest discover -s tests`，当前覆盖 appconfig/db/config/accounts（含 T-105b 启动登录复用已运行 Chrome）/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 ③ 更新蝦皮批次/店铺/商品ID/状态筛选列表、确认弹窗、开始更新主按钮、重置更新状态右键菜单、蝦皮更新安全拦截与前往设置、`ApplyWorker` 串行/检查/分批/按账号并行/端口冲突预检、运行日志、结果回写与汇总/gui ④ 账号管理、密码打码与明文保存提示/gui ⑤ AI 模型管理、API Key 打码与明文保存提示、角色/生成参数设置、蝦皮更新安全设置、检查按钮/每批最大更新条数/多账号并行设置、测试商品 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 设置页未保存状态/离开确认/放弃还原、T-532 cmhub连接成功账号名提示、T-533 增量生成组件口径、T-534 ②按标题/封面组件重置与多选/筛选范围重置、T-535 cmhub标题请求读取等待固定600秒、T-536 GUI按钮全局基础样式与语义色按钮叠加样式、T-537 主窗口标题显示中文品牌；2026-07-01 已完成 5 个真实商品的 T-404 更新验收，后续 CDP/Shopee 改动仍需测试商品手动验证。 - 数据：T-538 后默认用户数据根为 `data/`（整体 gitignore）；旧布局的 `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` 首次读取缺失的 `data/config.json` 时会在本地写默认配置，`app/db.py` 调用 `init_db()` 时会在本地创建 SQLite DB。 ## 既定设计要点（文档已定） @@ -39,8 +40,10 @@ | `docs/troubleshooting.md` | 已有 | 常见问题排查；已记录 `data/config/ai_models.json` 缺失 `category`、cmhub 404、②无可生成任务、④启动登录重复开 Chrome，以及打包版窗口左上角显示不全/标题栏不可拖动的原因、临时处理和修复方案 | | `app/__init__.py` / `app/__main__.py` / `main.py` | 已有 | 正式包与启动入口；`python main.py` / `python -m app` 可运行占位入口 | | `requirements.txt` | 已有 | T-521 产出：锁定运行时直接依赖 PySide6 6.5.3、openpyxl 3.1.3、requests 2.31.0、websocket-client 1.6.1、Pillow 9.5.0；换机/CI 使用 `python -m pip install -r requirements.txt` | +| `requirements-dev.txt` | 已有 | T-525 产出：锁定开发检查依赖 ruff 0.5.7；不属于运行时或打包依赖 | +| `pyproject.toml` | 已有 | T-525 产出：ruff 配置；当前 lint 只开启 `E4/E7/E9/F` 安全规则，格式化配置可用但不强制全仓重排 | | `requirements-build.txt` | 已有 | T-524/T-540 产出：打包环境依赖，先安装运行依赖再安装 PyInstaller 6.11.1；不作为运行时依赖文件 | -| `.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` | +| `.github/workflows/tests.yml` | 已有 | T-522/T-525 产出：push / pull_request 自动在 `windows-latest` + Python 3.11 安装 `requirements.txt` 与 `requirements-dev.txt`，设置 `QT_QPA_PLATFORM=offscreen`，运行 ruff lint、`python -m compileall app main.py` 与 `python -m unittest discover -s tests` | | `app/version.py` | 已有 | T-540 产出：统一维护 `APP_NAME`、`APP_CODE_NAME`、`APP_VERSION` 和窗口显示名；GUI 标题、打包脚本、release 目录、portable zip 和 `version.txt` 共用该版本源 | | `cmshopee.spec` / `scripts/build_exe.ps1` | 已有 | T-524/T-540 产出：PyInstaller onedir 构建配置与发布目录校验脚本；`cmshopee.spec` 不声明本地数据 `datas`，脚本固定使用 `py -3.10`，先检查 `dist\\cmshopee` 不含配置、DB、图片、日志、Chrome 登录态或提示词，再组装 `release\\cmshopee-\\` 与 `release\\cmshopee--portable.zip` | | `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/T-532/T-536/T-537/T-540/T-541/T-542 产出：PySide6 `QMainWindow` + 五 Tab；窗口标题显示「蝦皮圈優化助手 v」，启动时按屏幕可用区域限制尺寸、居中并保证标题栏可见；顶部 Tab 栏防误点样式；全局 `QPushButton` 基础样式统一 4px 圆角、hover/pressed/disabled/focus 状态；统一语义色板、①②③状态列前景色、③开始更新 warning 语义色、①导入校验数字标红、④登录状态点、③Tab warning 小圆点、删除/未匹配按钮 danger 语义色样式、①②③首次空状态引导卡片和批次阶段进度总览；① 导入采集导入按钮、导入汇总栏、批次/店铺/商品ID/状态筛选、删除批次软删除按钮、`QTableView` 任务列表、未匹配筛选与略过标记、采集旧标题旧封面 worker（旧封面路径按批次/店铺/任务细分）、采集前账号就绪预检与④引导、采集完成自动回写与手动重试、采集运行日志视图；② AI生成左右布局、提示词管理、批次/店铺/商品ID/状态筛选栏、任务列表、新标题列本地编辑、开始生成/停止/标题与图片双进度条、双击新旧封面预览、用户可读自动滚动AI生成运行日志、重置生成结果与 `GenerateWorker`；③ 更新蝦皮批次/店铺/商品ID/状态筛选栏、任务列表、开始更新主按钮、重置更新状态右键菜单、蝦皮更新安全拦截与「前往设置」跳转、「检查本轮更新」按钮、开始更新确认弹窗、`ApplyWorker` 检查/分批串行/按账号并行、账号与端口预检、运行日志、逐条 `set_applied`、自动回写结果到 Excel、结束汇总弹窗与手动回写重试；④ 账号管理表格、账号弹窗、密码本地明文保存提示、启动登录、检测登录、快捷方式；⑤ 设置普通默认展示 cmhub 网关 Base URL、API Key、生文/生图动态别名、Base URL 网关根提示、刷新别名、测试连接/查余额并保存 `config/cmhub.json` Key，不再显示 AI 后端选择、direct 模型选择/详情/角色下拉；direct 配置仅内部兼容/手工回滚；通用生成参数、路径端口配置、三列 蝦皮更新安全与执行模式设置仍保留，高频安全/执行模式分区前置、基础设施分区后置，`test_item_id`/`dry_run` 兼容字段无用户入口且普通更新不再阻断正式更新；②封面模板「另存为/重命名/删除」低频操作已收敛进「模板操作」菜单；②AI生成底部已增加标题/图片双进度条，运行日志已改为用户可读、自动滚动、脱敏的长任务日志，且②本轮「生成封面图片（成本较高）」开关默认关闭并持久化到 `ai.generate_cover`，关闭时只生成标题并进入可更新状态；②cmhub模式会显示生成后的剩余点数，记录 points_cost/call_id 计费日志，并在点数不足时弹提示且中止未开始任务；⑤设置页已显示“● 未保存更改”，切 Tab/关闭窗口时拦截保存/放弃/取消，放弃会从 `config.json` 与 `config/cmhub.json` 重新回填，刷新别名/测试连接不自动保存并提醒点保存；cmhub 测试连接/查余额成功时会优先显示 cmhub 账号名或脱敏邮箱，缺账号信息时保留普通成功提示 | @@ -75,8 +78,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-105b（④启动登录复用已打开 Chrome）、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⑤ 蝦皮更新安全开关）、T-502（换封面删第一张再上传）、T-503（敏感信息本地明文保存提示与日志脱敏）、T-504（多账号并行 / dry-run / 运行日志）、T-207（①采集诊断日志）、② AI生成图片失败诊断日志补丁、T-404a（②/③ 选中记录重置）、T-404（真实 Shopee 单条更新冒烟验收）、T-506（正式使用批量更新体验）、T-507（正式批量更新：移除普通流程测试商品 ID 限制）、T-508（③ 更新蝦皮生产化操作区）、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-532（⑤cmhub连接成功提示显示账号名）。 -- 最近完成补充：T-533（②增量生成：按缺失组件补生成）、T-534（②重置增强：多选/筛选范围 + 按组件重置）、T-535（②cmhub生成标题读取等待固定600秒）、T-536（GUI按钮圆角全局统一）、T-537（品牌名显示为蝦皮圈優化助手）、T-538（打包产物用户数据收进 `data/`）、T-540（打包版本号与 release 产物命名统一）、T-541（打包版主窗口初始位置与小屏适配）、T-542（GUI 用户可见 shopee/Shopee 文案统一改为蝦皮）。 -- 下一个可领取任务：T-525（引入 ruff lint/format，先以安全规则为主）。 +- 最近完成补充：T-533（②增量生成：按缺失组件补生成）、T-534（②重置增强：多选/筛选范围 + 按组件重置）、T-535（②cmhub生成标题读取等待固定600秒）、T-536（GUI按钮圆角全局统一）、T-537（品牌名显示为蝦皮圈優化助手）、T-538（打包产物用户数据收进 `data/`）、T-540（打包版本号与 release 产物命名统一）、T-541（打包版主窗口初始位置与小屏适配）、T-542（GUI 用户可见 shopee/Shopee 文案统一改为蝦皮）、T-525（引入 ruff lint/format 工程检查）。 +- 下一个可领取任务：T-539（⑤ 隐藏数据路径设置）。 ## 当前已知限制 @@ -113,12 +116,12 @@ - T-541 已完成：`MainWindow` 启动时通过 `_fit_and_center_window()` 读取 `QApplication.primaryScreen().availableGeometry()`，按可用屏幕限制初始尺寸、居中并夹进屏幕；首次 `showEvent` 后再应用一次，保证打包版在 Windows 10 VM/小分辨率环境标题栏完整可见、可拖动，不保存历史坏坐标。 - 主窗口标题当前由 `app.version.display_name()` 统一显示为「蝦皮圈優化助手 v」；T-541 不改品牌/版本来源，只改启动尺寸与位置。 - 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-522/T-525 已完成：`.github/workflows/tests.yml` 在 push / pull_request 上用 Windows + Python 3.11 安装 `requirements.txt` 与 `requirements-dev.txt`，设置 `QT_QPA_PLATFORM=offscreen`，自动运行 ruff 安全类 lint、语法检查和全量单元/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 打包配置、构建脚本和发布目录校验；T-538 后打包版不再切换全进程工作目录，首次运行在 exe 同级 `data/` 生成/使用本地配置、DB、图片、日志、登录态和提示词；发布包不内置 `data/`，用户后续更新采用关闭程序后覆盖程序文件、保留 `data/` 的方式。 -- Phase 7 cmhub 网关对接已完成 T-526~T-535；②生成阶段已支持按标题/封面缺失组件增量生成，②重置生成结果已支持多选/筛选范围和按标题/封面组件重置，cmhub 标题请求读取等待固定 600 秒。T-541 与 T-542 已完成，下一步按看板回到 T-525 ruff / T-539 设置简化。 +- Phase 7 cmhub 网关对接已完成 T-526~T-535；②生成阶段已支持按标题/封面缺失组件增量生成，②重置生成结果已支持多选/筛选范围和按标题/封面组件重置，cmhub 标题请求读取等待固定 600 秒。T-541、T-542 与 T-525 已完成，下一步按看板进入 T-539 设置简化。 - T-301/T-303 已完成通用 HTTP AI 接口、批量生成编排和 GUI 接入 mock 单测；真实 AI 生成还需要在 `data/config/ai_models.json` 填入可用 url/model/api_key 后做一次成本可控的小样本实测；URL 可填完整 endpoint 或 OpenAI-compatible base URL。 - 本地 `data/config/ai_models.json` 若由旧版本或手工维护，可能缺少 `category`；启动报 “AI 模型 category 必须是 text 或 image” 时，按 [`troubleshooting.md`](troubleshooting.md) 只补 `category` / `enabled` 等非密钥字段，保留 API Key，且不要提交该文件。 @@ -132,6 +135,10 @@ # 安装锁定依赖（T-521 后） python -m pip install -r requirements.txt +# 安装开发检查依赖并运行 ruff（T-525 后） +python -m pip install -r requirements-dev.txt +python -m ruff check app tests main.py + # 安装打包依赖并构建免安装 exe（T-524 后） py -3.10 -m pip install -r requirements-build.txt powershell -ExecutionPolicy Bypass -File scripts\build_exe.ps1 diff --git a/docs/engineering-review.md b/docs/engineering-review.md index be07457..2e2bc64 100644 --- a/docs/engineering-review.md +++ b/docs/engineering-review.md @@ -44,6 +44,8 @@ ### 5. 没有 lint/format/type 配置（→ T-525） +> 落地状态：T-525 已完成。当前已加入 `pyproject.toml`、`requirements-dev.txt` 和 CI ruff 检查；先只强制安全类 lint，不做全仓格式化重排，mypy 仍作为后续可选。 + - **现状**：无 ruff/black/mypy/pre-commit；类型注解部分覆盖。 - **影响**：大代码库 + AI agent 持续改，缺一致性门禁会慢慢腐化（未用 import、风格漂移）。 - **方案**：引入 ruff（lint+format 一把梭）+ 可选 pre-commit hook；数据模型可渐进上 mypy。 diff --git a/progress.md b/progress.md index 21a8797..2c526ee 100644 --- a/progress.md +++ b/progress.md @@ -1370,3 +1370,13 @@ - 原因：`tests/test_gui.py` 中两个 `WriteBackWorker` 单测把 `db_path` 写成相对路径 `db.sqlite`；测试创建 run_log 时会打开 SQLite，导致在项目根目录生成一个空 `db.sqlite`。 - 修复：这两个测试改为使用 `self.make_temp_dir()` 下的 `db.sqlite`，并断言临时目录被清理；根目录空 `db.sqlite` 已删除。 - 验证：`python -m unittest discover -s tests -p test_gui.py` 通过（97 tests）；`python -m unittest discover -s tests` 通过（221 tests），跑完后根目录没有 `db.sqlite`。 + +## 【2026-07-07】T-525 完成 · 引入 ruff lint/format 工程检查 + +- 状态：DONE +- 代码：新增 `pyproject.toml` 配置 ruff，当前只开启 `E4/E7/E9/F` 安全类规则；保留 `ruff format` 配置但不做全仓格式化重排。新增 `requirements-dev.txt` 锁定 `ruff==0.5.7`，不写入运行依赖或打包依赖。 +- CI：`.github/workflows/tests.yml` 安装 `requirements-dev.txt`，并在语法检查和 unittest 前执行 `python -m ruff check app tests main.py`。 +- 基线：对 `app/cdp.py` 先清代理再导入、`main.py` 先补 `sys.path` 再导入 GUI、`app/gui/widgets.py` 的 GUI 代理初始化顺序配置了明确 `E402` 豁免；修正两个真实未用变量。 +- 文档：同步 `docs/03-tech-stack.md`、`docs/05-coding-rules.md`、`docs/README.md`、`docs/engineering-review.md`、`docs/cmhub-integration-design.md`、`docs/06-tasks.md` 与 `docs/current-state.md`；下一个可领取任务为 T-539。 +- 验证：`python -m pip install -r requirements-dev.txt` 成功；`python -m ruff check app tests main.py` 通过；`python -m compileall app main.py` 通过；`python -m unittest discover -s tests` 通过（221 tests）；`git diff --check` 无空白错误；根目录未生成 `db.sqlite`。 +- 边界：未改 AI、DB、Excel、Shopee/CDP 流程；未新增 pre-commit 或 mypy。 diff --git a/pyproject.toml b/pyproject.toml new file mode 100644 index 0000000..f75e3b3 --- /dev/null +++ b/pyproject.toml @@ -0,0 +1,29 @@ +[tool.ruff] +target-version = "py310" +line-length = 120 +extend-exclude = [ + "build", + "chrome_user_data_dir", + "data", + "dist", + "images", + "logs", + "release", +] + +[tool.ruff.lint] +select = ["E4", "E7", "E9", "F"] + +[tool.ruff.lint.per-file-ignores] +"app/cdp.py" = ["E402"] +"main.py" = ["E402"] +"app/gui/*.py" = ["F403", "F405"] +"app/gui/tabs/*.py" = ["F403", "F405"] +"app/gui/__init__.py" = ["F401", "F403", "F405"] +"app/gui/widgets.py" = ["E402", "F401"] + +[tool.ruff.format] +quote-style = "double" +indent-style = "space" +line-ending = "auto" +docstring-code-format = false diff --git a/requirements-dev.txt b/requirements-dev.txt new file mode 100644 index 0000000..3af089a --- /dev/null +++ b/requirements-dev.txt @@ -0,0 +1 @@ +ruff==0.5.7 diff --git a/tests/test_gui.py b/tests/test_gui.py index 0a11fb4..5152b32 100644 --- a/tests/test_gui.py +++ b/tests/test_gui.py @@ -2963,7 +2963,6 @@ class GuiTests(TempDirMixin, unittest.TestCase): close_flags = [] progress = [] rows = [] - logs = [] def fake_apply(account, task, close_success_tab=False, on_step=None): applied_aliases.append(account.alias)