fix: OCR 审查驱动的全库修复（安全/正确性/部署/前端） #64

2026-08-09T13:32:05Z

wangziqi commented

2026-08-09 13:32:05 +00:00

概述

由 OCR（open-codereview.ai，deepseek-v4-flash）对 gongxue-base 全库审查后驱动的修复，分 6 个批次提交。

批次

fix(ops)：部署/代理/迁移脚本加固（serve-proxy、migrate.sh、deploy.sh/CI、PM2、oxlint）
fix(security)：认证/越权/注入/上传/凭据加固（JWT、越权 IDOR、LIKE 转义、上传 XSS、集成密钥加密）
fix(correctness)：并发/事务/实体/时区/状态一致性（double-spend、幂等、校验矛盾、china-time）
fix(admin)：前端功能与安全修复（角色勾选、月视图新建、附件预览、考勤 late 等）
docs(ocr)：OCR 审查报告与修复记录
fix(security)：classrooms 补充

验证

server：typecheck ✅ / lint 0 error ✅ / jest 152 套件 1178 用例 ✅
admin：typecheck ✅ / vitest 175 ✅
多轮 OCR 定向复查：high/critical 归零

备注

生产部署需在 .env 配置 JWT_SECRET（必填）、AI_CONFIG_ENCRYPTION_KEY（用过集成配置时）
实体改动（class_student 外键 CASCADE 等）需接库后生成迁移：npm run migration:generate -- FixEntityMappings
存量集成配置可运行：npm run encrypt:integration-secrets（一次性回填）

Reviewed-by: OCR (open-codereview.ai)

## 概述由 OCR（open-codereview.ai，deepseek-v4-flash）对 gongxue-base 全库审查后驱动的修复，分 6 个批次提交。 ## 批次 1. fix(ops)：部署/代理/迁移脚本加固（serve-proxy、migrate.sh、deploy.sh/CI、PM2、oxlint） 2. fix(security)：认证/越权/注入/上传/凭据加固（JWT、越权 IDOR、LIKE 转义、上传 XSS、集成密钥加密） 3. fix(correctness)：并发/事务/实体/时区/状态一致性（double-spend、幂等、校验矛盾、china-time） 4. fix(admin)：前端功能与安全修复（角色勾选、月视图新建、附件预览、考勤 late 等） 5. docs(ocr)：OCR 审查报告与修复记录 6. fix(security)：classrooms 补充 ## 验证 - server：typecheck ✅ / lint 0 error ✅ / jest 152 套件 1178 用例 ✅ - admin：typecheck ✅ / vitest 175 ✅ - 多轮 OCR 定向复查：high/critical 归零 ## 备注 - 生产部署需在 .env 配置 JWT_SECRET（必填）、AI_CONFIG_ENCRYPTION_KEY（用过集成配置时） - 实体改动（class_student 外键 CASCADE 等）需接库后生成迁移：npm run migration:generate -- FixEntityMappings - 存量集成配置可运行：npm run encrypt:integration-secrets（一次性回填） Reviewed-by: OCR (open-codereview.ai)

wangziqi added 43 commits 2026-08-09 13:32:06 +00:00

feat(admin): 用户体验体系化提升与高危缺陷修复 67435e46ca

UX 缺陷修复:
- 校验失败不再卡死弹窗按钮(Users/Roles/Bills)
- 押金收取/批量收取/添加分期防重复提交;切换房型重置勾选
- AI 表单/批量确认不再出现"假成功"
- Dashboard 各数据模块独立加载,单接口失败不再整页清零
- 房间可视化加载失败显示错误态而非永久转圈
- 学生编辑表单回填前重置,避免字段残留污染
- 覆盖式导入增加二次确认;恢复默认考勤时段确认并同步表单
- 金数据匹配关闭前确认,同步中禁止误关

体验提升:
- 新增统一 QueryErrorState/QueryEmpty,20+ 页面加载失败显示错误态与重试
- 全局 ErrorBoundary + RouteKeeper 逐页兜底
- 新增 usePageVisible/useVisibleRefetch,保活页面切回自动刷新数据
- 新增首次登录角色引导 RoleTour 与业务闭环 NextStepHint 引导卡
- 重构 A2UI:useSubmissionState/useXCardSurface 收敛状态与命令生命周期,
  ArtifactErrorBoundary 渲染降级,图表空数据占位
- AI 助手欢迎语与建议话术按角色定制,会话列表空态引导
- 更新 a2ui-contract.md 契约文档说明实现现状

fix(admin): 通知分页、登录过期提示、附件打开反馈与导出统一 loading 00e2bc5acf

- 通知中心改为游标分页 + 加载更多,历史通知不再被 50 条上限截断;
  加载失败显示错误态与重试
- AI 流式请求 401 被登出时,登录页提示"登录已过期",不再无声踢出
- AI 附件/引用来源打开失败时给出明确错误提示,不再"点了没反应"
- 新增 useDownload hook:统一导出/下载的防重复、loading 与成功/失败反馈;
  接入学生/账单/房间/入住/费用五个页面的模板下载与导出按钮
- 学生页移除不检查响应状态的私有下载实现,统一走 downloadBlob

fix(admin): 编辑消息/切换会话保护、行内编辑撤销与上传进度反馈 8dc72d6e1b

- 编辑旧消息会删除其后全部消息时,先弹确认告知影响范围
- 有待发送附件时切换会话,先确认再丢弃(避免静默删除已上传文件)
- EditableCell 保存成功后提供 6 秒内"撤销"入口(把旧值重新保存),
  重新编辑时自动清除
- 导入向导/重新上传显示上传进度条与百分比,不再"假死"
- AI 附件上传透传进度给附件列表
- 租赁合同上传显示按钮 loading 与上传百分比

style(admin): 清理 AI slop 痕迹 ff3b6cdfde

- 移除教师端 hero 的英文眉标(TEACHING DAY),日期并入正文描述
- 删除不再使用的 attendance-eyebrow 样式
- Dashboard 的 ══ 装饰注释改为普通注释

kill-ai-slop 扫描: 18 处命中 → 7 处(剩余均为合理用法:
链接 hover 下划线、头像圆形角)

refactor(admin): 按 aislop 质量门禁修复两个代码质量警告 bcbf3c8b2e

- RoleTour: buildSteps 拆分为各角色的声明式步骤常量,函数从 91 行降至组装逻辑
- ImportWizardModal: 抽取 uploadRun 共享 helper,消除 handleUpload/handleReupload
  之间重复的上传进度与 loading 状态处理

aislop scan --changes: 99/100 → 100/100 (5 引擎 0 issues)

feat(admin): 弹窗未保存内容保护与空状态引导扩展 c10476d203

- 新增 useDirtyGuard hook:关闭弹窗时若表单被修改,先确认再关闭
- 接入 17 个弹窗:账号/密码/档案、角色、教室、宿舍/床位/柜子、
  考勤机、机构、班级、教师、考试、排课、入住/退宿/换房、费用
  (子组件弹窗用 useEffect 在回填后快照,时序正确)
- 学生列表空状态带「添加学生」引导动作
- 排课创建成功提示「同步钉钉」,考试创建成功提示「去详情录成绩」

aislop scan: 5 引擎 0 issues

feat: 空状态引导全面应用与考勤批量标记 1b4ba893fd

admin:
- 17 个列表页空态统一为 QueryEmpty 引导:学生/账单/入住/费用/押金/
  教室/房间/班级/考试/排课/机构/考勤机/钱包/租赁/通知/角色等,
  有创建权限的页面附带主操作按钮,无权限时纯展示
- 教师端课堂点名新增「全部已打卡/全部未打卡」批量按钮:
  仅作用于状态不一致的记录,确认后调用批量接口,展示成功/失败数量

server:
- 新增 PUT /attendance-records/batch-status 批量改状态接口
  (ids ≤200,逐条权限校验与会话锁,部分失败返回 failedIds,
  审计日志记录批量结果;路由声明在 :id 之前避免被捕获)

aislop scan: 5 引擎 0 issues

feat(admin): 管理端考勤明细批量纠错 8b5fa98028

- 原始考勤明细表支持行选择(跨页保留),选中后出现
  「标记正常/标记请假/标记缺勤」批量操作条
- 批量提交前确认,成功后按 updated/failed 数量反馈,
  失败记录(如已结算)提示原因;筛选条件变化自动清空选择
- 复用 PUT /attendance-records/batch-status 接口

aislop scan: 5 引擎 0 issues

refactor(server): 清理全量 any 类型安全警告 (692 → 0) a644a8de42

- 全模块类型化:controller 的 req: any → AuthenticatedRequest/RequestUser,
  聚合查询 getRawMany 泛型标注、导入行/响应体定义具体 interface、
  catch (e: any) → unknown + 收窄、no-base-to-string 用 String() 显式转换
- 第三方无类型库边界(pdfkit/exceljs)文件级或单行 disable 并注明理由
- 顺带修复:get-business-context.tool 两个 require-await error、
  bills.controller 参数顺序隐患、main.ts compression 调用
- 运行时逻辑零改动;测试 142 套件 / 1065 用例全部通过

refactor(admin): A2UI 双轨收敛,统一 artifact 协议 bcd2d1a559

- mergeArtifactIntoMessage 不再派发 legacy 列表,只维护 uiArtifacts
- 渲染层从 uiArtifacts 派生 forms/reviews/charts,为空时回退 legacy
  (历史消息兼容,老 metadata 仅 a2uiForm/a2uiReview/a2uiChart)
- sseReducer 删除 ui.form/ui.review/ui.chart 旧事件分支,只消费
  ui.artifact(后端过渡期双发,旧事件将被忽略)
- types.ts legacy 字段标注 deprecated;契约文档同步现状
- 测试更新:旧事件忽略 + uiArtifacts 合并/恢复断言

aislop scan: 5 引擎 0 issues

fix: code-review 审查问题修复 + A2UI 测试补齐 4b60e0c018

Standards 轴:
- 移除 uiArtifacts.ts 的 payloadOf 死代码残留
- AttendanceDevices 残留 any 类型化(补 ClassroomOption.status 字段)
- 批量考勤纠错区分业务失败(已结算/无权限)与系统错误,
  前端提示精确到两类数量
- Dashboard queryFn 六段重复校验块收敛为 safeValidate 助手

Spec 轴:
- 补齐阶段 3.4 A2UI 测试:图表空数据占位、ArtifactErrorBoundary
  降级隔离、useSubmissionState/useXCardSurface 单测(7 用例)
- 阶段 2.2 补两处引导:教师工作台区分「今日无课」与「未分配班级」、
  班级花名册空态带「添加学员」动作
- 契约文档修正 DynamicReview 状态管理描述(多提交点如实说明)

aislop 剩余 16 警告均为必要豁免(类型边界/声明式 SQL 配置/既有文件规模)

refactor(server): 收敛类型边界工具,aislop AI Slop 10 → 1 1e3d6ee20f

- 新增 common/buffer.ts bufferToArrayBuffer:9 处
  'as unknown as ArrayBuffer' 收敛为精确切片(含 byteOffset),
  消除潜在 Buffer 池偏移隐患,类型断言集中到单一实现
- 新增 common/stringify.ts:4 处重复的 stringify 助手收敛为共享实现
- aislop: AI Slop 10→1(仅剩 1 处有理由的 stringify 薄包装,
  eslint no-base-to-string 绕过所需);Code Quality 剩余
  4 重复块(声明式 SQL 配置)+ 2 文件过大(既有规模)均保留
- 测试 142 套件/1065 用例通过

refactor(server): A2UI 收敛——移除 legacy SSE 双发与 render_review 工具，提取公共校验 b37ae9471e

refactor(admin): AI 对话收敛为统一 ui.artifact 协议，移除 legacy 事件处理 57639a3869

feat(admin): 管理端 UI/UX 深度优化——错误/加载/空状态、表单快捷键、移动端适配与无障碍 a0829f17ce

fix(server): 考勤自动结算支持空班级直接完成，补充自动匹配测试 f96c1c26c3

chore(server): 移除 e2e 测试配置与 supertest 依赖 f160256865

ci: 新增 aislop 质量门禁工作流 b73ac5e3e4

docs: 更新项目结构与部署说明（apps/* 布局、monorepo 命令） f03f22c605

docs: 同步 A2UI 契约与 AI 工作流文档至统一 artifact/向导流程 bb8228bd0f

chore: 移除 aislop 质量门禁工作流 e50f660811

refactor(server): 待办统计日期懒计算，aislop 豁免注释与文档纠错 6c11fd7e16

fix: 修复 QA 巡检发现的 console 告警——useForm 未连接、RangePicker 空值禁用、dashboard 占用率类型 00cdb322d2

chore: admin 构建稳定 vendor 分包，消除 chunk 告警；更新 .env.example 品牌文案与端口 d27f10eb84

feat(admin): RouteDock 页签增强——pathname 唯一 key、20 上限 LRU、关闭其他/左/右/全部、登出清理、持久化校验收紧 260a7517d2

refactor(admin): EditableCell 跨单元格协调状态迁入 zustand，替代模块级单例并补单测 9cc9ed09df

refactor(admin): 钉钉集成配置缓存迁入 zustand 页面级 store，移除模块级单例 01cceea237

refactor(admin): 用已安装的 file-saver/fast-deep-equal 替代手写下载与 JSON 比较 dab28f85e8

refactor(admin): 用 usehooks-ts useEventListener/useTimeout 与 dayjs 替代手写监听、撤销计时与相对时间 74d5b90faa

refactor(server): 日期工具统一迁入 dayjs（utc 插件），移除手写 Intl/padStart 重复实现 d324b2fb0a

refactor(admin): LiteMermaid 用 usehooks-ts useIsMounted 替代手写 cancelled 标志 8ddc3ea690

refactor(server): 用已装库替换手写工具 92ba2e779f

- uuid v7 替换手写 RFC9562 实现（uuid@11.1.1 转为直接依赖）
- node:timers/promises setTimeout 替换两处手写 sleep（钉钉限流、AI 上游重试退避）
- dayjs 替换 4 处零散 Date 格式化（imports/expenses/occupancies 导入模板、档案报告日期）

refactor(server): 日期/月份格式化统一改用 dayjs 8936453e84

- 替换散落的 toISOString().slice(0,10) / split('T')[0] / replace('T',' ')（UTC 语义用 dayjs(...).utc() 保持完全一致）
- Asia/Shanghai 固定时区日期（Intl.DateTimeFormat en-CA）改用 dayjs().utcOffset(8)
- 月份边界 first/last、daysInMonth、nextMonth、shiftDate/addDays 等改 dayjs 简洁实现
- 涉及 attendance/classes/classrooms/room/dashboard/bills/expenses/occupancies/sync/ai-chat/rental 等 28 个文件

fix(attendance): 默认周范围改为中国日期周一起点 782b43ef0a

generateFromSchedules / getCalendar 原来用本地周一零点转 UTC 日期，
在上海时区会得到前一天的日期，导致默认范围变成周日~周日（8 天）。
改为 dayjs().utcOffset(8) 按中国日期计算，与 getTodayDateOnly 语义一致。

新增测试锁定：默认周 = 本周一~本周日，周日晚上仍归本周，显式传参原样透传。

fix(server): 业务日期'今天'统一为中国日期，避免凌晨 UTC 偏一天 7905f1ee23

joinDate/leaveDate/checkInDate/sync 起始日等业务日期默认值原来用
dayjs().utc()（UTC 今天），在国内 00:00-07:59 会取到昨天。
统一改为 dayjs().utcOffset(8)（固定 UTC+8 中国日期），与考勤周边界
修复保持一致；pending-tasks/controller-base 的本地 dayjs() 也显式化。

fix(server): 时间戳→业务日期/时间全部统一为中国时区（UTC+8） 4dd7b5a0b1

此前仅'今天'默认值统一为中国日期，Date→字符串的转换仍是 UTC：
- sync 考勤同步窗口 lastSyncAt 时间戳按 UTC 取日期，凌晨会偏一天
- 考勤导出 punchTime/createdAt 显示 UTC 时间
- normalizeDateOnly 把 ISO datetime 按 UTC 归一化，业务日期少一天
- getCourseClock/getChinaDateParts 用 dayjs.utc(date).add(8h) 隐式转换

统一为 dayjs(date).utcOffset(8)，纯日期字符串运算（shiftDate/addDays/
daysInMonth/nextMonth）保留 dayjs.utc；date-normalization 测试断言
同步更新为北京时间语义。

feat(admin): 深化 TanStack Query 数据层 9565a0f23c

- 新增 queryKeys.ts 统一 QueryKey 工厂（覆盖全部模块，invalidate/refetch/prefetch 同源）
- 新增 queryClient.ts 全局配置：retry 1、staleTime 30s、gcTime 5min、refetchOnWindowFocus false
- 新增 useApiQuery：zod schema 校验 + 类型收敛 + select 转换，消除页面重复 validateResponse 样板
- useApiMutation 支持 onMutate/onSettled/context（乐观更新）
- 示范迁移：Organizations（useApiQuery + 乐观更新归档/恢复）、Students（useApiQuery + queryKeys + 打开抽屉前 prefetch 档案聚合）
- StudentProfileContent queryKey 统一为 queryKeys.archive.detail，prefetch 可命中缓存

fix(ops): 加固部署/代理/迁移脚本，生产环境安全默认 3bcad138a1

由 OCR（open-codereview.ai，deepseek-v4-flash）审查驱动修复：
- serve-proxy：API_TARGET 生效、SPA 404 语义、流式静态文件、hop-by-hop/超时/断连/穿越防护
- migrate.sh：密码不进 argv/不泄漏 pm2、原子锁防重入、DML-only 事务说明、就绪诊断
- deploy.sh/CI：保护 .env、总是 npm ci（迁移依赖 devDeps）、健康检查、并发锁
- ecosystem/oxlint/.env.example：优雅停机、React 版本对齐、TRUST_PROXY 说明

Reviewed-by: OCR (open-codereview.ai)

fix(security): 认证/越权/注入/上传/凭据全链路加固 99ea931409

由 OCR（open-codereview.ai，deepseek-v4-flash）审查驱动修复：
- JWT 生产必填、密码 8-72 字节、防枚举；全局 ValidationPipe
- classes/dashboard/schedules/students/attendance/archive/exams 越权与 IDOR 修复
- LIKE 通配符转义（14 处）；上传 10MB 上限 + MIME 白名单 + 附件 XSS
- 集成配置 appSecret AES 加密 + 回填脚本；审计 best-effort；IP 来源防伪造

Reviewed-by: OCR (open-codereview.ai)

fix(correctness): 并发/事务/实体/时区/状态一致性修复 f50301148d

由 OCR（open-codereview.ai，deepseek-v4-flash）审查驱动修复：
- wallets 原子扣款防 double-spend；refund 条件更新幂等；findTransactions 分页
- financial/imports/occupancies/attendance 事务与 advisory lock；重复生成/提交幂等
- 矛盾校验器、日期区间、实体双映射/DECIMAL/nullable、时区统一（china-time）
- rbac-seed 防重激活、exam 权限恢复、状态一致性、路由顺序、N+1/IN 分块等性能项

Reviewed-by: OCR (open-codereview.ai)

fix(admin): 前端功能与安全修复（角色勾选/月视图新建/附件预览/考勤 late 等） a32c0a0731

由 OCR（open-codereview.ai，deepseek-v4-flash）审查驱动修复：
- Roles 多组勾选、Schedules 月视图新建、useDirtyGuard 稳定引用
- JinshujuMatchModal success:false、Attendance store 订阅、EditableCell Enter、DynamicChart 空数据、渲染期 ref
- AttachmentsTab 附件预览不再 window.open（防存储型 XSS）+ 请求乱序防护
- 考勤 late 计入出勤率与主状态

Reviewed-by: OCR (open-codereview.ai)

docs(ocr): 全库 OCR 审查与修复记录 1289aea7ce

由 OCR（open-codereview.ai，deepseek-v4-flash）审查产出：
- 审查汇总.md / high-severity-comments.md / all-comments.jsonl / 修复记录.md 等

Reviewed-by: OCR (open-codereview.ai)

fix(security): classrooms 上传限制/路由顺序/LIKE 转义补充

CI / check (pull_request) Failing after 3m26s

Details

db11a57e98

由 OCR（open-codereview.ai，deepseek-v4-flash）审查驱动修复：
- classrooms.controller：report 路由移到 :id 之前、上传 10MB + 判空
- classrooms.service：搜索关键词 LIKE 通配符转义

Reviewed-by: OCR (open-codereview.ai)

wangziqi merged commit fb9927e712 into main

2026-08-09 13:32:19 +00:00

wangziqi referenced this issue from a commit

2026-08-09 13:32:21 +00:00

Merge pull request 'fix: OCR 审查驱动的全库修复（安全/正确性/部署/前端）' (#64) from feat/ocr-review-fixes into main

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: wangziqi/gongxue-base#64