## Work Log - 当前环境将线路管理菜单置顶（2026-06-08）

- 背景：
  - 仓库代码已支持 `admin.power_lines` 菜单，但默认排序仍落在其他后台一级菜单之后。
  - 用户要求在当前环境中让“线路管理”展示在后台菜单最前面。

- 本次处理：
  - 当前环境数据库：
    - 将 `menus.code='admin.power_lines'` 的 `sort_order` 调整为 `0`。
  - 仓库代码：
    - `api/app/services/seed_service.py`
      - 将默认种子菜单中 `admin.power_lines` 的 `sort_order` 同步改为 `0`，避免后续补种回退。
    - `MEMORY.md`
      - 补充“线路管理默认置顶”的长期口径。

- 验证：
  - 复核后台菜单排序后，`线路管理` 已位于一级菜单首位。

- 风险与关注点：
  - 已登录后台的浏览器会话需要刷新页面后才会看到最新菜单顺序。

## Work Log - 放出后台剩余目录菜单并补齐保护（2026-06-08）

- 背景：
  - 当前仓库默认菜单集合缺少 `admin.basic_data`、`admin.system_monitor`、`admin.system` 3 个目录菜单。
  - `admin.fl_analysis` 已有完整页面，但旧权限映射与菜单保护集合仍未完全覆盖。

- 本次处理：
  - `api/app/services/seed_service.py`
    - 补齐上述 3 个目录菜单，并将其加入 `admin` 默认菜单绑定。
  - `api/app/services/admin_service.py`
    - 将 `admin.system`、`admin.system_monitor`、`admin.basic_data`、`admin.fl_analysis`、`admin.fault_recurrence` 加入现代菜单删除保护集合。
  - `api/app/services/legacy_admin_rbac_service.py`
    - 将 `admin.system`、`admin.system_monitor`、`admin.basic_data`、`admin.fl_analysis` 加入 legacy 受保护菜单集合。
  - `api/app/services/legacy_authz_service.py`
    - 补齐 `admin.fl_analysis` 的 legacy 权限映射。

- 验证：
  - `git diff --check` 通过。
  - `npm --workspace web exec tsc --noEmit` 通过。
  - `npm --workspace web exec eslint src/app/admin/fl-analysis/page.tsx src/app/admin/fault-recurrence/page.tsx` 通过。
  - Python 变更文件 AST 语法解析通过。

- 风险与关注点：
  - `admin.basic_data`、`admin.system_monitor`、`admin.system` 仍为无 `path` 的目录菜单，只承担菜单树占位与分组作用。

## Work Log - 在共享后端镜像内置 Wine（2026-06-08）

- 背景：
  - 新需求要求把 Wine 安装到 `fquiz-celery-worker`。
  - 当前仓库中 `celery-worker`、`api`、`celery-beat`、`flower` 复用同一个 `api/Dockerfile` / `${API_IMAGE}`，且后端已有 `/api/v1/wine/*` 直接探测与执行 Wine 的能力。

- 本次处理：
  - `api/Dockerfile`
    - 在系统依赖安装阶段加入 `wine`，并用 `command -v wine` 做镜像构建期存在性校验。
  - `MEMORY.md`
    - 将 Wine 口径从“运行环境自行安装”更新为“共享后端镜像已内置 Wine”。

- 验证：
  - `python3 -c "import yaml; ..."` 解析 `deploy/dev-deploy/compose.yml`、`deploy/pro-deploy/compose.yml`、`.github/workflows/main.yml` 通过。
  - `git diff --check` 通过。
  - `python3 -m py_compile api/app/services/wine_service.py api/app/services/fl_analysis_external.py api/app/tasks/fl_analysis_tasks.py api/app/core/config.py` 通过。
  - 当前执行环境无 `docker` 命令，未能本地实际跑 `docker compose build` / `docker compose config`。

- 风险与关注点：
  - 由于部署链路复用同一个后端镜像，本次会同时影响 `api/celery-worker/celery-beat/flower` 的基础镜像体积，而不只是 `celery-worker`。

## Work Log - 修正 Wine 缺少 wine32 时的误报（2026-06-08）

- 背景：
  - 当前环境里 `wine --version` 会输出 `it looks like wine32 is missing`，但后端 `/api/v1/wine/status` 之前只按返回码判定，可能把半可用环境误报成“可用”。
  - `ATP` 的 Wine 引擎状态也复用同一 Wine 二进制，存在同类误报风险。

- 本次处理：
  - `api/app/services/wine_probe.py`
    - 新增共享 Wine 探测 helper，统一将 `wine32 is missing` 识别为不可用状态。
  - `api/app/services/wine_service.py`
    - Wine 状态接口与测试执行前校验改为复用共享探测 helper。
  - `api/app/services/atp_model_service.py`
    - ATP Wine 引擎状态与执行前校验同步复用共享探测 helper。
  - `api/Dockerfile`
    - 构建阶段显式开启 `i386` 并安装 `wine32:i386`，避免镜像只装到半套 Wine 运行时。
  - `api/tests/test_wine_probe.py`
    - 新增针对 `wine32 is missing` 误报场景的最小单测。

- 验证：
  - 变更前基线：`python3 -m pytest api/tests/test_fl_analysis_external.py`、`python3 -m pytest api/tests/test_fl_analysis_service.py` 因本地运行环境缺少 `pytest/sqlalchemy` 无法执行。
  - 变更后执行：
    - `python3 -m py_compile api/app/services/wine_probe.py api/app/services/wine_service.py api/app/services/atp_model_service.py api/tests/test_wine_probe.py` 通过。
    - `./.venv/bin/python -m pytest api/tests/test_wine_probe.py` 通过（3 passed）。
    - `git diff --check` 通过。

- 风险与关注点：
  - 当前本地环境仍不具备完整后端依赖，无法直接回归所有 FastAPI/SQLAlchemy 相关测试；本次验证聚焦在 Wine 探测逻辑和语法层面。

## Work Log - 修复 ATP 文本上传中文乱码（2026-06-08）

- 背景：
  - `ATP文本转换与预览` 页面上传 `.atp/.txt` 文件时直接使用 `file.text()`，浏览器会按 UTF-8 解码。
  - 现场 ATP 文件常见为 Windows 导出的 GBK/GB18030 文本，导致导入后编辑区和转换预览中的中文注释/名称乱码。

- 本次处理：
  - `web/src/lib/text-file.ts`
    - 新增前端文本解码工具，优先识别 BOM、兼容无 BOM 的 UTF-16，并在 UTF-8 严格解码失败时回退到 `GB18030`。
  - `web/src/app/admin/power-lines/atp-viewer/page.tsx`
    - 上传 ATP 文本时改为基于 `arrayBuffer + TextDecoder` 自动判定编码，不再固定走 UTF-8。
  - `web/src/lib/text-file.test.js`
    - 补充最小测试，覆盖 `UTF-8`、`GB18030(兼容 GBK)`、无 BOM `UTF-16LE` 三类输入。

- 验证：
  - 基线：`npm_config_cache=/tmp/npm-cache npm --workspace web exec tsc --noEmit` 通过；`npm_config_cache=/tmp/npm-cache npm --workspace web exec eslint src/app/admin/power-lines/atp-viewer/page.tsx` 通过。
  - 修改后：
    - `npm_config_cache=/tmp/npm-cache npm --workspace web exec tsc --noEmit`
    - `npm_config_cache=/tmp/npm-cache npm --workspace web exec eslint src/app/admin/power-lines/atp-viewer/page.tsx src/lib/text-file.ts src/lib/text-file.test.js`
    - `node --test web/src/lib/text-file.test.js`

- 风险与关注点：
  - 已经以错误编码写入数据库的历史 ATP 文本不会被自动修复；本次修复只覆盖后续上传与预览入口。

## Work Log - 全局成功失败提示切换为右上角弹消息（2026-06-08）

- 背景：
  - 后台多个页面仍把普通 CRUD 成功/失败提示渲染在页面顶部，占用列表与表单空间。
  - 现有页面已经混用 Ant Design `message` 与顶部 `Alert`，交互不统一。

- 本次处理：
  - `web/src/components/ui-antd.tsx`
    - 为全局 Ant Design `message` 统一配置顶部偏移与默认停留时长。
  - `web/src/app/globals.css`
    - 将 `message` 容器样式调整为右上角浮层展示。
  - `web/src/hooks/use-toast-feedback.ts`
    - 新增轻量 hook，用于把页面本地 `error/success` 状态统一转成右上角自动消失的提示。
  - 后台页面：
    - `users`、`system-params`、`wine-runner`、`lightning-currents`、`power-lines/atp-viewer`
    - `roles`、`menus`、`files`、`tower-models`、`elevation`、`power-lines`、`fault-recurrence`
    - 去掉顶部“操作成功/失败”提示条，改为右上角弹消息；保留权限不足、加载失败、解析提醒等需要常驻占位的 `Alert`。

- 验证：
  - 基线：
    - 初次 `npm --workspace web exec tsc --noEmit --pretty false` 因当前环境缺少 `web/node_modules` 且默认 npm cache 不可写未能执行。
    - 补装依赖后，基线 `NPM_CONFIG_CACHE=/tmp/fquiz-npm-cache npm --workspace web exec tsc --noEmit --pretty false` 通过。
  - 修改后：
    - `NPM_CONFIG_CACHE=/tmp/fquiz-npm-cache npm --workspace web exec tsc --noEmit --pretty false` 通过。
    - `NPM_CONFIG_CACHE=/tmp/fquiz-npm-cache npm --workspace web exec eslint src/components/ui-antd.tsx src/hooks/use-toast-feedback.ts src/app/admin/users/page.tsx src/app/admin/system-params/page.tsx src/app/admin/wine-runner/page.tsx src/app/admin/lightning-currents/page.tsx src/app/admin/power-lines/atp-viewer/page.tsx src/app/admin/roles/page.tsx src/app/admin/menus/page.tsx src/app/admin/files/page.tsx src/app/admin/tower-models/page.tsx src/app/admin/elevation/page.tsx src/app/admin/power-lines/page.tsx src/app/admin/fault-recurrence/page.tsx`
      - 仅剩仓库原有 warning（`users`/`tower-models` 的 hooks 依赖与 `img` 提示），无新增 error。
    - `git diff --check` 通过。

- 风险与关注点：
  - 本次只统一“瞬时成功/失败反馈”；权限态、加载态、解析告警等长驻提示仍保留 `Alert`，属于有意设计，不是遗漏。