[xerrors]

变更说明

修复已合并 PR 的测试接线和说明缺口：部门 HTTP 回归进入 CI，补充文件工具不暴露 delete 的断言，完善 PostgreSQL 配额生效步骤，纠正 OCR 说明并记录模型别名浏览器证据。

• 任务类型：testing。
• 目标与非目标：补齐现有修复的验证与文档；不改变业务实现，不处理搁置的 fix(web): 入库领取间隙内自动刷新保持，避免入库完成后列表过期 #1121、fix(knowledge): .xls 按文件头嗅探分派兼容 WPS 伪 xls 格式 #1128。
• 判断：更新现有 decision，保留已有行为与架构。

工程主张与 Owner

• 部门人数排除已删除用户：Owner 为部门 repository；真实 HTTP 回读验证计数，system-tests.yml 运行既有回归。
• 模型工具不暴露 delete：Owner 为文件系统 middleware；测试检查真实注册结果。
• PostgreSQL 配额的生效条件：Owner 为 Compose，部署文档说明容器重建与检查步骤。
• OCR 模型与别名展示：事实由安装包、锁文件及真实组件拥有。
• 更新既有 decision：部门计数、OCR、模型别名；不引入新的业务决策。

验证情况

部门 HTTP 计数

• 失败面：已删除用户被计入人数。
• Owner：DepartmentRepository.list_with_user_count。
• 证据：独占 PostgreSQL、真实 TCP、JWT 与正式路由执行现有三个部门测试，全部通过；清理结果经数据库回读确认。运行 PYTHONPATH=backend backend/.venv/bin/python /tmp/yuxi-fix-pr-review/verify_department_http.py，使用仓库 isolated_audit_store、serve_http 与 test_department_router.py 三个既有函数；新增完整拓扑 CI 步骤直接运行该测试文件。
• 负向案例：移除删除过滤后，相同 HTTP 用例在最终计数断言失败。
• 结果：Passed。

文件工具暴露

• 失败面：delete 被注册为模型工具。
• Owner：文件系统 middleware。
• 证据：uv run --project backend --no-sync --group test pytest backend/test/unit/sandbox/test_user_sandbox_backend.py -q，35 项通过，断言真实工具名称。
• 负向案例：临时恢复 delete 注册后，新断言失败。
• 结果：Passed。

回归、文档与浏览器

• 失败面：组合合并回归、说明失真、别名展示或错误回退异常。
• Owner：现有测试、Compose、安装包与真实组件。
• 证据：工程契约和 66 项契约单测通过；后端 3436 项通过、1 项模型目录未配置跳过，配置真实目录后 OCR 三项通过；前端 879 项、ESLint、tsc、Vite build、VitePress build、Ruff、uv lock --check、git diff --check 通过。实际命令如下。
• 浏览器：真实组件的深浅主题、768/1440 宽度、长别名和未命中回退通过；明确等待实际 503 响应后，两个组件均回退原名。所有业务请求使用浏览器合成响应，截图已核对。
• 负向案例：部门及工具暴露负控如上；浏览器主动返回 503。
• 结果：Passed。

# 仓库根目录
python3 scripts/verify_engineering_contracts.py
python3 -m unittest scripts.test_verify_engineering_contracts
uv run --project backend --no-sync --group test pytest backend/test/unit -m "not slow" -q
uv run --project backend --no-sync ruff check backend/test/unit/sandbox/test_user_sandbox_backend.py
uv run --project backend --no-sync ruff format backend/test/unit/sandbox/test_user_sandbox_backend.py --check
uv lock --project backend --check
git diff --check

# frontend 目录：运行 package.json 脚本对应工具，避免 pnpm 自动重装共享依赖
node node_modules/eslint/bin/eslint.js . --max-warnings=0
node --test --test-concurrency=1 'test/**/*.test.js' 'test/**/*.spec.js'
node node_modules/typescript/bin/tsc --project tsconfig.json
node node_modules/vite/bin/vite.js build

# docs 目录
node node_modules/vitepress/bin/vitepress.js build

OCR 补跑由独立 Python 进程将 RAPIDOCR_MODEL_DIR 设置为当前 Python 的 rapidocr/models 包内目录，再运行 pytest.main(['backend/test/unit/knowledge/test_rapid_ocr_models.py', '-q'])，3 passed。浏览器使用独立 Playwright 会话和临时合成页面，通过 playwright-cli -s=yuxi-fix-review run-code --filename=/tmp/yuxi-fix-pr-review/browser-alias.js 执行；页面及测试 PostgreSQL 已清理，截图仅保留本地证据，不作为业务链路通过证明。

简化 / 删除验收

不涉及。

独立语义 Review

全新上下文 Reviewer 审查需求、完整 diff、规范、实际日志、HTTP 验证与浏览器证据。浏览器错误场景提前通过的问题已修正并重跑，最终批准，无剩余阻塞项。

未验证范围与风险

完整 Compose 新增部门步骤及全部远程检查已通过（Runtime System Tests）；真实登录后的后端模型目录链路未运行。共享 API 测试命令被自动审批拒绝，采用独立环境验证。pnpm 自动安装与共享依赖链接不兼容，前端使用脚本对应工具直接执行。

事故反馈

未达到高影响逃逸事故门槛；属于测试接线与说明补修。

界面变更

不改变界面实现；补充 #1116 的浏览器验证证据。

关联事项

补充 #1115、#1116、#1117、#1123、#1124、#1127 的审查交付。

补充说明

无数据迁移或依赖变更，未重建现有业务服务。

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

[chatgpt-codex-connector]

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review	Status	Commit	Review trigger
📝 Code Review	✅ Completed 2026-10-28T03:18:56.298956Z	b4b4188	PR opened

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.