199 Commits

This Branch

This Branch

All Branches

Author	SHA1	Message	Date
yuetsh	457df1ef3c	fix(api): collab 房间越权与几处时序/静默丢帧缺口 ... Critical：学生排队断线分支补归属+状态校验，避免同账号双标签页 把 active 请求错删、被另一个学生请求顶替后又被别的老师抢建出 第二间房；handleAccept 补 getRoom(studentId) 兜底同一学生 id 下已有房间的情况。 Important：handleAccept 的库查询是个 await 点，之后补上 teacherSockets().has(ws) 判活，教师在等待期间断线不会再对着 死 socket 建房间。 Minor：handleReject 补上和 handleAccept 一致的库复核，堵住 连接存活期间被降级/禁用的教师继续掐请求的口子； handleCollabBinary 检查 peer.send() 返回值，转发失败时走既有 teardownRoom 拆房间，不再静默丢帧、悄悄分叉两边的代码。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K1d8B3f4SXJwDvUY625eQd	2026-08-28 02:48:35 -06:00
yuetsh	b14639d890	feat(api): collab 房间管理与 Yjs 帧转发 ... accept 时按库里的 adminType 复核身份，房间以学生为键。 服务端只按房间转发二进制帧，不解析内容。 老师掉线请求退回排队，学生掉线请求随人清除。 顺带处理三项 Task 2 评审遗留： - TEACHER_ROLES 去重，handler.ts 改为从 routes/helpers.ts 导入，不再自留一份 - 补全学生排队中断线（尚未进房间）的清理分支，避免陈旧请求卡死在列表里 - 修正 websocket.ts 里一处过期注释：username/adminType 是三种 kind 都会填， 不是只有 collab 才填 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K1d8B3f4SXJwDvUY625eQd	2026-08-28 02:36:12 -06:00
yuetsh	1db2c49b87	feat(api): 新增 /ws/collab 通道与课堂求助控制面 ... 学生发起/撤销求助，在线教师收到全量列表。求助只在内存，不落库。 二进制帧的限流单独一档，避免协作输入把连接踢掉。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K1d8B3f4SXJwDvUY625eQd	2026-08-28 02:24:44 -06:00
yuetsh	c04570b4ad	feat(web): BaseWebSocket 支持二进制帧收发 ... onmessage 原来无条件 JSON.parse，收到 Yjs 二进制帧会落进 catch。 加 onBinary 钩子与 sendRaw，让 collab 通道能复用基类的重连与心跳。	2026-08-28 02:06:13 -06:00
yuetsh	59191c9433	docs: 课堂求助与协作编辑实现计划 ... 8 个任务：前端 WS 基类支持二进制 → 后端 collab 通道控制面 → 房间与帧转发 → 前端 store → 学生求助按钮 → 教师顶栏列表 → 协作编辑 → 清理旧实现。 每个任务收尾是手动验证步骤（项目约定不写测试）。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K1d8B3f4SXJwDvUY625eQd	2026-08-28 02:00:37 -06:00
yuetsh	275e70e23a	docs: 课堂求助与协作编辑设计文档 ... 学生发起求助、老师从全局列表接单进入协作。传输从 y-webrtc P2P 改为 后端 WebSocket 通道，房间归属与权限由服务端判定。 替换掉现有 sync.ts 的四个结构性问题：房间按题号分导致老师无法指定帮谁、 初始内容源竞态、权限靠客户端自报且三人以上失效、信令服务器无部署源。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K1d8B3f4SXJwDvUY625eQd	2026-08-28 01:50:00 -06:00
yuetsh	f38444c97a	fix(代码规则): 给 C 题配的规则一半是哑弹，C++ 题根本配不了 ... 后台的节点下拉是一张 C/Python 混在一起的 15 条表，整份铺给每种语言。给 C 题选到 只有 Python 有的 list_comprehension、f-string，规则存得进去，判题时拿裸名去比节点 类型，C 的语法树里永远不存在它——「必须使用列表推导式」永远失败、「不能使用 f-string」永远通过，两头都不报错，只有学生受着。反过来 mappings 支持的 do_while、 switch、struct、include 在表里没有，编辑器根本选不到。 标签表改成按语言分组，和判题机 mappings 的键集逐条对齐；保存时校验规则与语言是否 匹配，不匹配给中文提示。C 的 target 从 8 个可用变成 14 个。 C++ 接上了 tree-sitter-cpp，386 道 C++ 题从此能配规则。它继承 tree-sitter-c 的语法， C 那 14 个 target 实测全部通用，另加范围 for、类定义、try-catch、throw、namespace、 模板、lambda、using 共 22 个。调用形态和 C/Python 都不同，一并处理：a.push_back() 和 p->push_back() 是 call_expression + field_expression，不是 Python 的 attribute； std::sort(...) 的 function 是 qualified_identifier 而不是 identifier，所以额外比一次 :: 末段，否则学生写没写 using namespace std 会得到不同判定。 一起收掉的几处： - Java/Golang/JavaScript 配的规则一条都不会跑，题目页却照常把它们渲染成「要求」 挂给学生看。现在后台不给这些语言开 tab，下发给学生的要求也按语言过滤。 - 「出现次数」不填数字存下来是一条恒真规则，描述还退化成光秃秃一个「for 循环」。 切换引擎时给默认值，保存时拦下，读取时整条丢弃。 - 次数规则失败只说「if 条件 出现 2 次 ✗」，学生不知道自己写了几次，补上「当前 N 次」。 旧栈的引擎其实算了这个数，但 checker 只取 describe，算完就扔。 - must_have_nesting 的文案没走标签表，学生看到的是「必须使用 for_loop 嵌套」。 - 运算符文案给的是逻辑名，C 题的学生看到「必须使用 and 运算符」，而 C 里写的是 &&。 语义校验放在 astRulesError() 而不是 zod 的 refine 上：astRulesSchema 同时用于读后台 题目详情，在读路径上抛错会让历史脏数据把整个题目详情打不开。保存前先 pickAstRules() 剔除够不着的分组再校验，否则早年配过 C++ 规则的题会把老师锁死——tab 里看不到那组 规则，保存却被拦下。 生产库那 17 条规则（全是 Python3 的 must_exist_node / count_node）行为不变，逐条实跑 核对过。C++ 的 22 个节点 target、25 个运算符也逐个跑了，没有恒假的哑弹。改了带 wasm 内嵌的 ast.ts，dev 和编译两种形态都验过。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-27 11:53:37 -06:00
yuetsh	280443c892	fix	2026-08-27 11:26:00 -06:00
yuetsh	77717e439d	fix(主题切换): 圆环从视口正中扩散，跟点击的地方对不上 ... 深浅色切换的圆环扩散原本把圆心写死在 window.innerWidth / 2，不管点哪儿都从屏幕 正中冒出来。改成取指针落点。 半径也跟着修了。原来是 hypot(x, y)，只量到左上角——圆心居中时恰好是对的，一挪到 按钮（在右上角）就不够长了，右下角会留一大块旧配色等圆环扩过去，看着像没刷新。 现在取到最远那个角的距离。 键盘触发（Enter / 空格）时 clientX/clientY 是 0，照用会让圆环从屏幕左上角冒出来， 这种情况退回按钮自身中心，用 event.detail === 0 判断。 startViewTransition 的降级判断原样保留，机房那批 Chrome 低于 94 仍然是直接切。 实测（钩住 documentElement.animate 看真实关键帧，视口 1280x633、按钮中心 1255,29）： 点正中得 circle at 1255px 29px / r=1392.78，点按钮左上角 (1246,22) 得 at 1246px 22px， 说明跟的是指针不是按钮；键盘 Enter 得 at 1255px 29px，没跑到 (0,0)。三次主题都正确 翻转，无报错。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-27 11:18:12 -06:00
yuetsh	6c2381ad2c	perf(提交列表): 越早的页越慢，最后一页要等 1.2 秒 ... 翻页用的是 LIMIT n OFFSET m，而 Postgres 对 OFFSET 没有捷径：前 m 行必须真的 产出再丢掉，丢弃又发生在 join 之后，每一行都白回了一次表。生产快照（10.4 万条 公开提交）实测最后一页 1258ms、碰了 95347 个 buffer。最早那几页平时没人翻， 数据页从来不在缓存里，全是冷读，所以感受上比新的几页慢得多。 改成两步：先只 select create_time / id 数到第 m 行拿游标——这两列正好是部分索引 的全部内容，跳过 m 行走 Index Only Scan，Heap Fetches 为 0，纯在索引页里数数； 再拿这一行做 keyset 回查，只回表取 limit 行。同一页降到约 9ms、885 个 buffer， 端到端 HTTP 10.7ms。代价变成 O(m) 个索引条目而不是堆页，按快照密度外推， 涨到 100 万条时最深一页仍在几十毫秒量级。 接口签名和前端都没动，页码跳转照旧。offset 为 0、以及按题号筛选时（条件在 problem 表上，第一步得跟着 join，index-only 就没了）退回普通 offset。 部分索引从 (create_time) 换成 (create_time, id)：create_time 由 new Date().toISOString() 生成，只有毫秒精度，不是全序，游标用 <= 回查时同毫秒的 上一页末行会重复出现在下一页页首。加 id 之后两步走同一个顺序。索引 2.3MB → 6.9MB。 索引两列都建成默认 ASC，靠 Index Only Scan Backward 反着扫。别照着 ORDER BY 写成 (create_time DESC, id DESC)：ORDER BY 的 DESC 默认 NULLS FIRST，索引的 DESC 默认 NULLS LAST，规划器认为出不了序，会退化成 external merge sort（5.2MB 落盘）， 比不加索引还糟。这一条已写进 schema.ts 和迁移文件的注释。 正确性：在快照上把新旧写法返回的 id 序列逐页比对，14 个 offset × 4 个 limit 共 56 组全部一致，含末尾残页与越界。 比赛提交列表暂不改：单场比赛撑死几千条，offset 不构成问题。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-27 11:13:38 -06:00
yuetsh	8861393529	fix(流程图): 评分明细的顺序是乱的，40 分的那项排在最后 ... AI 是按分值从高到低返回的：`逻辑正确性(40) → 完整性(30) → 规范性(20) → 清晰度(10)`。 但 `ai_criteria_details` 存在 jsonb 列里，而 Postgres 的 jsonb **不保留键序** —— 它按「键长度 + 字节序」重排，读出来变成 `完整性 → 清晰度 → 规范性 → 逻辑正确性`， 权重最高的那项被排到了最后。 评分弹框和教师端的评分详情都是直接 `v-for` 遍历这个对象，所以两处都乱。 统计面板因为用的是写死的 `CRITERIA_ORDER`，一直是对的 —— 于是同一份数据在两个 地方的顺序还不一致。 把顺序抽到 `utils/constants` 共享，三处统一走 `sortFlowchartCriteria()`； 表里没有的键排到后面，AI 万一返回别的评分项也不会丢。 顺带把限流的提示改得能看懂：撞上 429 时原来只显示「流程图提交失败」，学生会以为 是自己的图有问题然后反复点，越点等得越久。现在按错误码分支，提示「提交太频繁了， 缓一会儿再交」。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-27 10:53:29 -06:00
yuetsh	e921506f40	fix(流程图): 「重新判题」从来没成功过，而且会把原来的评分清掉 ... `bullmq` 对自定义 jobId 有一条兼容老式可重复任务的校验：一旦包含 `:`，就必须 正好切成三段（`job.js` 里的 `split(':').length !== 3`），否则抛 `Custom Id cannot contain :`。 - 代码提交的重判用的是 `${id}:rejudge:${时间戳}` —— 三段，能过； - 流程图的重判用的是 `${id}:${时间戳}` —— **两段，必抛**。 所以这个接口从上线起就没有成功过一次。更糟的是清空评分发生在入队**之前**： await db.update(...).set({ status: 0, aiScore: null, ... }) await flowchartQueue.add(...) // ← 在这里抛，500 每点一次「重新判题」，原来的分数、等级、反馈、评分明细就永久丢一次，提交卡在 PENDING，队列里没有任何任务会来救它，老师看到的只是「重新评分失败」。 改成三段式 jobId，并把入队失败落成 FAILED（而不是留在 PENDING）—— 与 `POST /flowcharts` 的处理保持一致。 之前几轮验证没发现，是因为当时手上只有一条 PENDING 的提交，被 409（状态不允许 重判）挡在了入队之前，正好绕开了这个 bug。这次完整走教师流程才撞上。 实测：修复前点重判 → 前端「重新评分失败」、库里 status 变 0 且分数清空、 api 日志抛 `Custom Id cannot contain :`；修复后 → 「重新评分已提交」， 70分B级 重评为 86分A级。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-27 10:51:00 -06:00
yuetsh	e00ab7b876	perf(流程图统计): 给词云的分词量封顶 ... 统计接口会把时间窗内所有已完成提交的 criteria / feedback / suggestions 全取回来， 逐条走一遍 jieba。而前端的「全部时段」是不带 start 的 —— 攒一学年就得把所有评语 重新 cut 一遍，而这是个同步阻塞的请求。 只给词云的分词条数封顶（3000），并按时间倒序取，留下的是最近的那批。 **数值不封顶**：总数、均分、等级分布、各项平均分、完成人数仍然按整个时间窗精确 计算 —— 那只是已取回行上的算术，不额外花钱。这些数一旦采样，老师看到的完成率和 均分就是错的，而且从界面上完全看不出来；词云是辅助性的，看的是高频问题，取最近 这些条足够。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-27 06:27:33 -06:00
yuetsh	74d3b97f05	feat(流程图列表): 补上状态列，重判按钮按状态置灰 ... 教师端的流程图提交列表一直没有状态列。「排队中」「评分中」「评分失败」三种情况 在界面上长得一模一样 —— 都只是分数栏空着，老师分不出是还没评完还是评失败了。 三处一起改： - 加状态列（排队中 / 评分中 / 已完成 / 评分失败）。 - 分数列只在已完成时渲染 Grade。原来无论什么状态都渲染，没评完会显示成 0 分， 看着像「评了但得了 0 分」。 - 重判按钮按状态置灰。后端只接受已完成 / 已失败的重判（其余返回 409）， 原来是点了才知道不行。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-27 06:27:33 -06:00
yuetsh	f5ad5318a0	fix(流程图编辑器): Ctrl+Y 没接、清空画布不问一声、存档里塞满 vue-flow 内部字段 ... ## 存档里的一大半是运行时内部状态 画布上的 node 是 vue-flow 的 GraphNode，除了我们自己塞的字段，还挂着 `dimensions` / `computedPosition` / `handleBounds` / `selected` / `dragging` / `resizing` / `initialized` / `isParent` / `events`（见 vue-flow 的 parseNode）。 这些会跟着一起写进 localStorage、进 20 份历史快照被反复深拷贝、还压缩后提交进 数据库**长期存着**。而重新挂载时它们全都会被重新算一遍 —— 存下来没有任何意义， 还把存档格式和 vue-flow 的内部实现绑死了，将来升级或迁移数据都得跟着动。 `handleBounds` 尤其占地方：一个循环节点有 4 个 handle，每个 6 个数字。 抽一个 `serialize.ts`，落盘/入历史/提交前统一裁成 id / type / position / data / style 五个字段。`style` 保留 —— 它是建节点时按类型算好的，丢了恢复出来的图会变样。 实测：一个两节点带连线的图，存档 600 字节、节点上只剩那五个字段，九个内部字段 一个不剩；提交上去压缩后 396 字节。 ## Ctrl+Y 是假的 工具栏按钮的 title 写着「重做 (Ctrl+Y)」，但只实现了 Ctrl+Shift+Z，按 Y 没反应。 顺手把 key 比较改成小写不敏感（按住 Shift 时 event.key 是大写的 "Z"）。 ## 清空画布点一下就没了 那个按钮不但清空画布，`clearCache()` 还会把这道题存着的草稿一起删掉，刷新也找 不回来。学生误点的代价太大，加一道确认；画布本来就是空的时候不弹。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-27 06:27:19 -06:00
yuetsh	9e41855610	fix(流程图): 点进去空白的 tab、以及「渲染成功」这个只进不出的开关 ... ## 两个开关同时打开会做出一个空 tab 后端在 `allowFlowchart` 为真时把 `mermaidCode` 置成 null（不能把标准答案下发给 正要自己画图的学生，见 routes/problem.ts）。而前端 `tabOptions` 只看 `showFlowchart` 就把 "flowchart" 加进选项，面板那边却要求 `showFlowchart && mermaidCode` —— 两个开关都打开时，选项存在、面板不存在， URL 里带 `?tab=flowchart` 就会选中一个渲染不出任何东西的页签。 三个断点条件还各不相同（两处要求两者都有，第三处只看 showFlowchart，那处会拿 null 去渲染 ProblemFlowchart）。统一成一个 `canShowFlowchart`。 后台那边把「显示标准流程图」在允许提交流程图时置灰并说明原因，再补一个 watch 把存量数据里两个都开着的情况纠正掉 —— 它们本来就是互斥的。 ## 「渲染成功」只进不出 保存前的校验靠 `mermaidRenderSuccess`，而 MermaidEditor 只在成功时 emit、 这个 ref 也就只会从 false 变 true，永不复位。**先写对、再改坏，照样能存进库。** 改成上报渲染结果本身（`render-state`），并在 modelValue 一变就立刻打回 「未验证」，等防抖后的渲染真跑完再报结论 —— 只挂防抖那一支的话，改完 300ms 内 点保存读到的还是上一次的结论，刚改坏的代码会被当成校验通过。宁可让出题人多等 一下，也不能放脏数据进库。 实测：改动后 50ms 读到 false（此时保存会被拦），渲染完成后回到 true； 贴一段坏语法则一直是 false。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-27 06:27:02 -06:00
yuetsh	d10172f017	fix(流程图): 节点名里带一个双引号，生成的 mermaid 就是坏的 ... 节点和连线的标签是学生自己敲的任意文字，而转换器是直接把它塞进 `"..."` 里： ${nodeId}(("${label}")) 出现一个双引号就把语法撑破了。后果不只是图渲染不出来 —— 这段坏掉的代码会**原样 提交给 AI 打分**，学生完全不知道自己被扣分是因为一个引号。把 `说"你好"` 当节点名 是很自然的写法。 改用 mermaid 的实体转义。`#` 必须先转，否则标签里本来就有的 `#quot;` 之类会被当 成实体解释；换行转成 `<br/>`，不然会截断整条语句。 实测：`说"你好" #1` 现在生成 `说#quot;你好#quot; #35;1`，渲染出来仍然显示 `说"你好" #1`；未转义的那版渲染报语法错误、一个 svg 都出不来。 顺带补上 `edges` 的空值保护 —— `nodes` 有，`edges` 一直没有，拿到 undefined 直接抛。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-27 06:26:45 -06:00
yuetsh	0392445dd2	fix(流程图): 补上几处静默失败 —— 点了没反应、弹框永久转圈 ... `utils/api.ts` 的拦截器只对 login-required / account-disabled / permission-denied 弹提示，其余业务错误一律静默 reject。而流程图这一块的调用点基本都没 catch， 于是失败时用户什么都看不到： - **重新判题**：后端会拒掉还在评分中的提交（409 retry-not-allowed），也可能撞上 限流。老师点下去完全没反应，连报错都没有。实测修复前 0 条提示，修复后弹出 「这条还在评分中，等出了结果再重新评分」。 提示按**错误码**分支而不是 match 文案（`utils/api.ts` 里写明的约定）—— 后端文案是英文的，直接弹给老师看不合适。 - **课堂统计**：请求失败后图表停在旧数据上，没有任何迹象表明这次没拉到。 同一批里 `SubmitFlowchart` 的三处（弹框翻页、打开评分详情、加载到编辑器）随 上一个提交一起改了，问题是一样的：前两处失败时 `rendering` 卡在 true，弹框 永久转圈。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-27 06:15:11 -06:00
yuetsh	5480abaaea	fix(流程图编辑器): 换题草稿串题、第一步撤销不了、历史存的是改动前的状态 ... ## 换题时画布不跟着换 storage key 是按题目 ID 算的 computed。`useStorage` 确实会 watch key，但它只把 新 key 的内容读进 `storedData`，**不会回填 `nodes`/`edges`**，而 `loadFromCache()` 只在 onMounted 调一次。于是同名路由换参数（题目 → 题目，组件不重新挂载）时， 画布上还留着上一题的图；学生一动，防抖保存就把上一题的内容写进**这一题的 key**， 把原本存着的草稿覆盖掉。 补一个 key 的 watch，重新载入并在载不到时清空画布。这里依赖 `useStorage` 内部 对 key 的 watch 先于本 watch 执行 —— 两者都是 pre flush，且 useStorage 在上方 先创建，pre 队列按创建顺序跑，此刻 `storedData` 已经是新 key 的数据。 实测（router.push 直接切题）：修复前 1003 的画布上挂着 1002 的节点，修复后 1003 是空的、1002 的草稿完好。 **需要说明**：今天的 UI 走不到这条路 —— 题目页没有「下一题」入口，题单和比赛 切题都要先回列表页（不同路由、组件会重新挂载）。所以这条目前是加固，一旦以后 加了题内切题入口就立刻变成必需品。 ## 撤销少一步、存的还是旧状态 `historyIndex` 从 -1 开始，而 `canUndo` 要求 `index > 0`，第一步操作永远撤销 不了。补 `resetHistory`，挂载时和换题后各播一次初始快照（换题不重建的话，一次 撤销会把上一题的图还原到这一题里）。 `addEdges` / `removeNodes` / `removeEdges` 之后紧接着 `saveState(nodes.value, edges.value)` —— 而 vue-flow 的 store → v-model 回写走的是 `watchPausable` （pre flush，异步），此刻读到的还是**改动前**的数组，存进历史整体错开一步。 画布上的 `handleDrop` 早就 `await nextTick()` 了，这几处一直漏了； `clearCanvas` 因为是直接赋值 model ref（同步）反而是对的 —— 所以这套行为一直 是「有时对有时错」，更难排查。 顺带：`handleNodeDelete` 里手动删相连边是多余的，`removeNodes` 的 `removeConnectedEdges` 默认就是 true；`deleteSelected` 在什么都没选中时不再 白记一条历史。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-27 06:14:59 -06:00
yuetsh	5f0fe713dc	fix(流程图渲染): 出错后再也画不出来、每失败一次往 body 漏一个 div ... ## 渲染失败后永久空白 四个渲染点都是同一个写法： <n-alert v-if="renderError" ... /> <div v-else ref="mermaidContainer"></div> 一旦渲染出错，容器被 `v-else` 卸载，`mermaidContainer.value` 变成 null。而 `renderFlowchart` 一进来就先清 `renderError`、再因为 `!container` 直接 return —— 于是**报错提示消失了，图也再画不出来**，只剩一块空白，只能刷新页面。翻历史提交 最容易踩到。改成容器常驻、用 `v-show` 隐藏，和 MermaidEditor 本来的写法一致。 `FlowchartScoreDetail` 那处 Teleport 上不能挂 v-show，把条件挪到内层容器。 实测：修复前切回合法代码后 0 个 svg（永久空白），修复后正常渲染。 ## 每次渲染失败都往 body 漏一个 div `m.render(id, code)` 没传容器，mermaid 会在 `document.body` 上建一个临时 `div#d{id}`。而 `suppressErrorRendering` 默认关着，看 mermaid 源码，解析出错时 是先 `errorRenderer.draw()` 再 `throw`，**清理临时容器的那行在 throw 之后**， 永远执行不到；每次 render 用的又是新的随机 id，`removeExistingElements` 也清不掉 旧的。于是渲染失败一次就留一个。 出题页是边敲边预览，且没有防抖，每个字符触发一次完整渲染，中间态几乎全是语法 错误 —— 实测逐字符敲 26 个字符，body 里留下 16 个残留 div。打开 `suppressErrorRendering`（该分支是先清理再抛）+ 预览防抖 300ms，实测降到 0， 预览功能不受影响。 ## 顺带 `loadMermaid` 缓存的是实例，两个组件同屏挂载时会双双落进 `if (!mermaidInstance)`， import 和 initialize 各跑两次。改成缓存 Promise，并在失败时清掉缓存，避免一次 网络抖动把后续所有渲染都钉死在这个失败结果上。 `SubmitFlowchart` 的 `updatePage` / `openDetailModal` / `loadToEditor` 一并补了 错误兜底（同文件，见下一个提交的说明）：前两个失败时 `rendering` 会卡在 true， 弹框永久转圈；`loadToEditor` 是裸 `JSON.parse`，老提交数据坏掉就点了没反应。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-27 06:14:40 -06:00
yuetsh	c27c9fdbf9	fix(流程图评分): 队列重试从没生效过、AI 调用不设超时、等级由模型自报 ... ## 重试是摆设 `flowchartQueue` 配了 `attempts: 3` + 指数退避，但任务开头有一道守卫： if (!row || ![0, 1].includes(row.flowchart.status)) return 而 catch 里第一件事就是把 status 写成 3(FAILED)。于是第 2、3 次尝试进来一看 状态是 3，直接 return、算作成功 —— **实际只跑了一次**。AI 侧的偶发失败（限流、 超时、网络抖动）永远等不到重试，学生看到「评分失败」只能自己重新提交。 改成只有最后一次尝试才落 FAILED，中间几次把状态留在 PROCESSING(1) 让守卫放行。 「是不是最后一次」由 worker 算好传进来：`attemptsMade` 是「此前已失败几次」， 当前这次还没计入，所以判据是 `attemptsMade + 1 >= attempts`。 实测（用没配 AI_KEY 这条必然失败的路径）：修复前 t+1s 就落 FAILED、只评一次； 修复后评满 3 次，状态到 t+7s 才落 FAILED。 ## fetch 不设超时 `completeChat` 直接 `fetch`，而 fetch 默认不超时。AI 侧一挂就把 worker 的并发位 （只有 2 个）一直占着，学生那边的按钮也就一直转。加 60 秒超时。 流式调用**不加**：那边超时会把正在推的长回答直接掐断，而客户端断开本来就能收尾。 ## 等级不该由模型说了算 提示词里写死了 S/A/B/C 四档分数区间，但模型偶尔会给出「88 分配 S 级」这种自相 矛盾的结果，甚至直接吐「优秀」。脏值会一路串到等级分布图、等级筛选，以及 「A/S 才把流程图展示给学生」的判断里。改成一律由分数推出等级，模型自报的 grade 不再采信；score 本来就已经 clamp 到 0-100。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-27 06:14:21 -06:00
yuetsh	8f08ed03a0	fix(流程图): 学生能翻出全班评分、AI 调用没有限流 ... ## 列表漏了一道门 代码提交列表在 `routes/submission.ts` 里有 `submission_list_show_all` 兜底： 关掉时非管理员一律返回空。流程图列表**从来没有这道门**，而它的过滤是 if (myself === "1" || (!username && 是普通用户)) 只看自己 else if (username) 按用户名模糊匹配 —— 只要带上 `username`，第二支就把第一支的限制绕过去了。学生在提交记录页把 语言切成「流程图」、用户名框随便填一个字，就能翻出全班同学的 AI 评分，不需要 动接口。补上和代码提交同一套口径。 ## 提交与重判没有限流 每一次流程图提交都会触发一次外部 AI 调用，是和判题沙箱同级的有限资源，而这两个 入口都没限流。`canView` 还允许**本人**重试自己的提交，等于学生可以对着自己的 提交反复点，无上限地刷 AI 调用。 限流桶不能直接用 `throttling:user:<id>` —— 那是代码提交在用的桶（capacity 20， 回填约 1.8 个/分钟），共用的话学生在机房连着交几次代码，流程图这边就会莫名其妙 交不上去。单独开 `throttling:user:flowchart:<id>`。 重判对教师放行：成批点几十行是他们的正常用法。 ## 提交编号的权限判断在前端自己算了一遍 契约里 `flowchartListItem.showLink` 是后端逐行下发的（与 `GET /flowcharts/:id` 的放行条件同源），前端却没用，自己按「超管或本人」重算了一次 —— 教师因此看得到 「重新判题」却打不开评分详情。 更要命的是无权限那一支渲染的 `n-text` **照样挂着 @click**，权限判断只改了外观。 学生点别人的编号，后端以 404 挡下，`loadSubmission` 只 console.error，于是弹出 一个 600px 高的空白面板，什么提示都没有。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-27 06:14:05 -06:00
yuetsh	64facc5701	fix(WebSocket): 断线不重连、评分结果会丢、登出后连接还活着 ... 排查 WS 这一块时发现的一批问题，多数是「机制写了但从没生效过」。 ## 重连 `disconnect()` 里 `enableAutoReconnect = false`，而 `connect()` 从不改回 true —— 登出再登录后，这条连接就永远失去了自动重连能力（configUpdate 那条 watch 上尤其 明显）。改成用 `closedByUser` 表达「用户主动断开」的意图，和 `enableAutoReconnect` 这个**配置**分开。 `scheduleDisconnect` 的回调里断完紧接着一句 `enableAutoReconnect = true`，而 close 是异步的 —— 等 onclose 跑到时标志已经翻回来了，1 秒后又自动连上。那个 「15 分钟空闲省资源」从来没真正断开过。现在只断开，不做事后翻转。 重连的 setTimeout 没存句柄，组件卸载后照样触发 `connect()`，在已销毁的组件上 又建一条连接。现在 `disconnect()` 里 clearTimeout。 退避从「线性 ×5 次」改成「指数 + 抖动、30 秒封顶、次数不封顶」。原来 1+2+3+4+5 只有 15 秒，后端 deploy 重启一次就超了，之后这条连接死到用户刷新为止。抖动是 为了避免一个班几十台机器在同一毫秒一起冲回刚起来的后端。另挂 online / visibilitychange，网络恢复或切回标签页立刻重连，不必等退避走完。 所有 socket 回调改成闭包住局部 ws 并在入口 `if (ws !== this.ws) return`，旧连接 迟到的 onclose 不再污染新连接的状态 —— 也是让 `disconnect()` 能被 onclose 识别 出来的关键。 ## 订阅重放 `pendingSubmissionId` 一发送成功就清空，它只解决了「还没连上就 subscribe」， **没解决断线重连**。而真正会丢结果的恰恰是后者：服务端收到 subscribe 会回一份 当前状态，掉线期间错过的推送就是靠这次重放补回来的；不重新订阅，重连后只收得到 「将来」的事件，可结果已经是过去式了。改成订阅意图保留到显式 `unsubscribe()`， 每次 onConnected 都重发。 这套逻辑原来只有 SubmissionWebSocket 有，FlowchartWebSocket 是 send 失败打一行 日志了事 —— socket 一掉，那次评分结果就再也回不来，按钮一直转圈。提成公共基类 SubscribingWebSocket，两条通道共用。 流程图另加轮询兜底：提交后 5 秒 WS 还没出结果就每 3 秒拉一次，读 status 2/3 结算，3 分钟上限。判题那边一直有兜底，流程图这边没有，而 Redis pub/sub 是发完 不管的，worker 推的那一刻连接不在就永远丢了。 `useSubmissionMonitor` 里 `watch(wsStatus, ..., { immediate: true })` 的回调在 watch() **返回之前**就同步跑了，已经连着时 `unwatch` 还是 null，if 不成立， watcher 永远停不掉：每提交一次泄漏一个，往后每次重连它们都会把各自那个早就判完 的旧 submissionId 重新订阅一遍。整块删掉，直接 subscribe —— 基类已经管了时序。 WS handler 原来不校验 submissionId。学生同时开着几道题的页面时，每条连接都订在 同一个用户 topic 上，别的页面的评分结果会被当成自己的。 ## 会话 握手时校验过一次会话就再也不管了，这条连接却能挂几个小时：用户在别的标签页 登出、或者会话本身到期，旧 socket 照样收推送。加一条 60 秒一轮的巡检，用 Redis EXPIRE 一条命令同时完成「判断存在」和「续期」（续期是必要的：只开着页面 挂 WS 的人一次 HTTP 请求都不发，不该被算成不活跃踢下线）。Redis 抛错时整轮 放弃，绝不因为一次抖动把全班踢下线。 禁用只改数据库的 isDisabled 列、不动 Redis 里的会话，巡检永远发现不了。加 `session:revoked` 频道主动通知，**两种作用域不能混**： { token } 用户登出。只断这一张会话 —— 同一个人在别的设备上是另一张 会话，按 userId 广播会把他手机上的登录一起踢掉 { userId } 账号被禁用。所有设备都得断 先发一帧 force_logout 再隔 100ms 断开。只断不发的话前端只看到一次普通掉线， 会照常重连、页面上还显示着登录态。token 不进帧里 —— 那是 httpOnly cookie 的值， 推到 WS 上就等于交给了 JS，匹配全在服务端做。 前端在协议层拦截 force_logout（和 pong 一样，不下发给业务 handler），并主动 disconnect —— 否则会一路 401 重连到退避上限，正是这机制要消掉的浪费。表现刻意 和 utils/api.ts 里 account-disabled / login-required 两支保持一致：同一件事从 HTTP 和 WS 两条路进来，学生看到的不该有两个样子。 ## 开销与安全 `void handleMessage(...)` 是裸的，里面有两次 DB 查询和一个会抛的 schema.parse， 库抖一下就是一个 unhandled rejection（隔壁 bridgeSubmissionEvents 两处都接住了， 只有这里漏了）。 ping 提到用户查询之前。原来的顺序是「先查 user 再看消息类型」，每个客户端每 30 秒都要为一次心跳打一趟数据库。禁用用户不会因此漏网：推送路径上 bridge 会查， subscribe 这条真正读数据的路径下面照样查。 bridge 两条 per-user 通道都先看 `server.subscriberCount(topic)`，没人订阅就别 查库了 —— 判题高峰期绝大多数事件的目标用户此刻并不在线。 flowchart 评分失败原来把 error.message 原样推给学生、前端直接弹出来，AI provider 的地址和内部报错就这么进了浏览器。改成真实原因写服务端日志。 加每连接令牌桶（20 突发 + 每秒回填 2）。一条 subscribe 在服务端是一到两次数据库 查询，一个学生开着一条 socket 狂发就能压住库。正常流量离阈值几十倍远。 升级时校验 Origin。会话 cookie 是 SameSite=Lax、WS 握手不是导航，跨站页面本来就 带不上 cookie，所以这是防御纵深不是唯一防线。同源放行；本机开发（Vite 5173 → API 3000）自动放行，且只在两边都是本机时成立 —— 生产环境 url.hostname 是正式 域名，这条永远不触发；跨域部署走 ALLOWED_WS_ORIGINS。不发 Origin 的一律放行： 真正的攻击面是带着受害者 cookie 的浏览器页面，而浏览器一定会带 Origin。 顺带：useConfigWebSocket 的 handler 从 onMounted 挪到同步注册（调用方在 setup 阶段就 connect() 了），删掉每条消息打完整内容的 console.log 和死字段 ws.data.username。 ## 没动的 题目页上并没有两条 /ws/submissions —— Form.vue 里 SubmitFlowchart 和 SubmitCode 是 v-if/v-else，互斥。学生实际是 2 条连接：全站一条 /ws/config + 一条 /ws/submissions，正常，不必合并。 ## 验证 55 个用例，分六组打桩跑（假 WebSocket + 假计时器；会话/限流/吊销三组对着真 Redis）： 重连语义 7 断开后不再自我复活、卸载后计时器已取消、退避封顶、 online 立即重连、旧 onclose 不污染新连接 订阅重放 9 未就绪时补发、重连后重新订阅、unsubscribe 后不再重放 会话巡检 8 EXPIRE 三态、只断失效的、同 token 只查一次、 Redis 抖动时一个都不踢 Origin/限流 13 跨站与跨端口拒绝、生产不因 localhost 开后门、 突发额度、按时间回填 强制登出 13 登出只断同 token（别的设备不受牵连）、禁用断所有设备、 巡检先通知再断 前端登出 5 两支表现、收到后不再重连、两条通道只处理一次 前两组做了改动前/后对比，老代码该挂的都挂了 —— 「重连后自动重新订阅」正是这么 跑出来的，此前我以为 pendingSubmissionId 已经覆盖了这种情况。 apps/api 的 tsc 和 apps/web 的 vue-tsc 都干净，仓库既有测试照常通过。 **SubmitFlowchart.vue 的轮询兜底未经运行时验证** —— 在 SFC 内，没搭组件挂载 环境，只过了类型检查和人工核对。要验的话，停掉 worker 提交一次流程图，看 5 秒 后是否转入轮询、3 分钟后是否给出超时提示。 这批改动动了 WS 的行为面（限流会断连接、Origin 会拒绝、巡检会踢会话），上线前 建议手测：提交代码看判题、切流程图看评分、后台改配置看全站生效、开两个标签页 在一个里登出、禁用一个在线学生看另一端反应。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-27 05:07:26 -06:00
yuetsh	47a43b9880	fix(提交列表): 转圈圈延迟 300ms 出现，快请求不再闪一下 ... 翻页、切筛选大多一百毫秒内就回来了，转圈圈闪一下再切数据比直接切更晃眼。 给 loading 加一层延迟：只有真的慢到 300ms 以上才显示。 是**单向**延迟 —— 只推迟「显示」，请求一结束立刻收掉，不留尾巴（用 refDebounced 之类的双向去抖会在数据到了之后还多转 300ms）。 没有用 n-spin 的 delay：n-data-table 压根不走 n-spin，它渲染的是内部的 _internal/loading，直接由 loading 控制、没有 delay 参数。要用内置的就得把表格 包进 <n-spin :show :delay> 再去掉 :loading —— 那会换掉加载样式，还会丢掉 loading 时锁住排序/分页交互（DataTable 里那个 `disabled: this.loading`）。 这几行就是 n-spin 内部同样的写法，理由写在注释里了。 验证：headless chromium 走 CDP，只给 /api/submissions 加延迟（给全站加延迟的话 dev 模式几百个模块请求全被拖慢，页面根本起不来，第一次就是这么测出假结果的）， 每 120ms 采一次 DOM： API 不加延迟 转圈圈全程没出现，表格直接 0 行 → 5 行 API +800ms 表格 291ms 挂上 → 654ms 转圈圈出现（挂载后约 300ms） → 1138ms 数据到位 → 1381ms 收掉 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-27 00:53:40 -06:00
yuetsh	6ac458e6f3	refactor(WebSocket): 判题/流程图推送改用驼峰，去掉没人读的三个字段 ... submission_id / time_cost / memory_cost / err_info 是 OJ2 两端自己定的线上格式， 没有第三方消费（/ws/submissions 只有 apps/web 一个客户端），没理由留着 snake。 flowchart 那条更别扭：同一个对象里 submission_id 是 snake、criteriaDetails 是驼峰。 三个字段直接删掉而不是改名 —— 它们是从 statistic_info 原样抄出来的一份，前端 一处都没读过（useSubmissionMonitor 只用 submissionId / result / status）。耗时和 错误信息在提交详情里本来就有，判完了去拉一次就是，不必让推送顺带背一份 JSONB 的形状。score 保留，它不涉及大小写。 **没动的都是有外部约束的**，别顺手一起改： - 发给判题沙箱的请求体（language_config / max_cpu_time / max_memory / test_case_id / io_mode）和它回的字段（cpu_time / memory / test_case）—— 那是沙箱的 API，不是我们的 - 测试点 info 文件的键，沙箱直接读那个文件 - submission.statistic_info 里的 time_cost / err_info / ast_results —— 判题机按这套写，12 万条历史提交就是这形状 - acm_problems_status、progress_detail 这些存量 JSONB - AST 规则键（for_loop）、成就指标（accepted_count）、reaction 语义键 —— 那是词汇表标识符不是字段名，for_loop 还要映射到 tree-sitter 的 while_statement 验证：起 dev 栈（api + worker + 沙箱），学生账号真提一次代码，抓 /ws/submissions 的帧： {"type":"submission_update","submissionId":"bf13b7f0…","result":6,"status":"pending"} {"type":"submission_update","submissionId":"bf13b7f0…","result":7,"status":"judging"} {"type":"submission_update","submissionId":"bf13b7f0…","result":-2,"status":"finished","score":0} subscribe 帧也换成 submissionId 并被接受（否则会回一个 error 帧，没有）。 流程图那条路径要 AI 评分才跑得起来，本机没配，只做了类型检查。 前后端同一个 docker 栈一起构建部署，没有版本错配窗口。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-27 00:48:54 -06:00
yuetsh	bc0b782443	update	2026-08-27 00:42:39 -06:00
yuetsh	8ae14f055a	refactor(站点配置): WS 直接推契约字段名，去掉前端那层 snake→驼峰胶水 ... 上一版是在前端加 toCamel 把 `enable_maxkb` 换成 `enableMaxkb`。但 snake_case 是 options 表从 Django 继承来的**存储格式**，只该活在库里；WS 这一跳两边都是 OJ2 自己写的，没理由让前端再维护一层换名。 后端改成广播 OPTION_KEYS 的 key（契约字段名）而不是 value（表列键），前端直接 按名字赋值。库里的列名一个字没动。 顺带清掉两处已经死掉的客户端→服务端配置推送： - admin/setting/config.vue 保存后调 updateConfig("enable_maxkb", ...) 和 ("submission_list_show_all", ...)。那条 ConfigWebSocket 从没 connect() 过， send() 直接返回 false；就算连上了，服务端的消息处理也只认 ping / subscribe， 会回一个 error 帧。广播本来就由后端在 POST /admin/website 里做，八个键一个 不落，这两行纯属多余，且用的正是刚废掉的 snake 键。 - ConfigWebSocket.updateConfig 一并删掉，免得再有人调一个静默失败的方法。 这条通道是单向的，原地留注释说明。 验证：dev 栈 + headless chromium 走 CDP 抓 Network.webSocketFrameReceived。 后台保存配置后，页面收到的八条帧的 key 全是驼峰（websiteName / enableMaxkb / submissionListShowAll ...），且页面不刷新，document.title 和页头站点名当场 跟着变。小助手那四条路径重跑一遍照旧：未登录不加载、登录后出现、后台关掉当场 消失、开回来当场重现。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-27 00:42:14 -06:00
yuetsh	0f34d67cc8	fix(站点配置): WebSocket 广播的 key 是 snake，store 字段是驼峰，实时生效一直空转 ... 后端推的是 options 表里的 snake_case 键： {"type":"config_update","key":"enable_maxkb","value":false} 前端 configUpdate.ts 收到后写的是 `if (data.key in configStore.config)`，而 store 里的字段是驼峰（enableMaxkb）。这个 `in` 判断**永远是 false**，一条配置 都写不进去 —— 站点名称、页脚、班级名单、允许注册、提交列表看全部、知识库挂件， 全部都要刷新才生效，且没有任何报错。`conf.ts` 里那句「前端 configStore.config 用的就是这套键名」是写错的，八成就是照着它写的这段代码。 加一层 snake → 驼峰的转换。八个键都符合通用规则，所以用正则转而不是列表， 将来加键不用改这里；转完认不出来的键直接忽略，别把 store 撑出野字段。 站点改名时顺便同步 document.title —— getConfig() 里本来就是这么设的， 只更新页面里那份、标签页还是旧名字说不过去。 顺带把 MaxKB 挂件那块理干净： - 加载条件改成「已登录 且 后台开关打开」。挂件本来就要登录才能用，原来对匿名 访客也照样注入。 - 删掉它自己那条 /ws/config 连接和 handleConfigUpdate。store 修好之后， 原有的 watch 自然就跟着动了；多开一条连接是重复的，而且没登录时会撞 401 （后端 /ws/config 要求会话），控制台常年一条「[WebSocket] 连接错误」就是它。 匿名访客不做实时生效：/ws/config 保持要求登录，不对外开不鉴权的 WS 端点。 没登录的人下次刷新页面时拿到新配置。 验证：起 dev 栈，用一个假的 MaxKB embed 端点（返回建出 #maxkb-chat-button 的 脚本）顶掉真实地址，headless chromium 走 CDP 看 DOM： ① 开关=开、未登录 → 挂件不在，script 标签 0 ② 开关=开、已登录 → 挂件在， script 标签 1 ③ 后台关掉（页面不刷新） → 挂件消失，script 标签 0 ④ 后台开回来（不刷新） → 挂件重现，script 标签 1 另外把其余三条 WS 通道的键名也对了一遍，没有同类问题：submission_update （submission_id / time_cost / memory_cost / err_info）两边都是 snake； flowchart 的 criteriaDetails 两边都是驼峰且 handler 没读它；成就通知的字段 全是单词，不涉及大小写。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-27 00:34:51 -06:00
yuetsh	8600f8143a	fix(MaxKB): 「启用 MaxKB」开关在生产构建下拦不住脚本 ... vite 的 inject-maxkb 插件把 `<script src>` 写进 index.html 的 head，于是构建 产物在 Vue 启动之前就把第三方挂件拉下来执行了。App.vue 的 useMaxKB() 那条运行时 路径只能事后把标签和 `[id^="maxkb-"]` 的 DOM 删掉 —— 代码早跑完了。表现就是后台 把开关关掉、刷新，挂件的副作用照旧（其中之一是它自带的性能上报在每个页面抛 `Cannot read properties of undefined (reading 'startTime')`，堆栈全是 <anonymous> 因为它 bootstrap 时又 eval 了一层），而且关不掉。 改成只留运行时那一条路： - 删掉 inject-maxkb 插件，index.html 不再带这个 script 标签。原地留注释说明 为什么不能加回去。 - config store 的 enableMaxkb 默认值从 true 改成 false。默认 true 的话 getConfig() 还没回来挂件就已经加载了，服务端说「关」依然晚一步。代价是配置 拿不到时挂件不出现 —— 第三方脚本 fail closed 是对的方向。 - URL 为空的兜底从 vite 插件挪进 loadMaxKBScript，否则会拿 undefined 当 src。 验证：headless chromium 走 CDP 记录网络请求。 - dev（5173）：enable_maxkb=false → 0 个请求、0 个 script 标签；改成 true → 发出 embed 请求、head 里 1 个标签。 - 生产构建（bun run build 后把 dist 静态跑起来、/api 转发到后端）：同样两种 配置下分别是 0 和 1。构建出来的 index.html 里已经没有 maxkb 的 script 标签， 只剩原本就在源文件里的那行 maxkbMaskTip。 MaxKB 脚本自身那个 startTime 报错是它自己的 bug，本次只保证「关得掉」，没去 动第三方代码。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-27 00:26:54 -06:00
yuetsh	d3757954cd	fix(AI 学情报告): pinnedOnly 那支返回裸数组，页面每次打开都白屏 ... `GET /admin/ai/reports` 同一个 URL 返回两种形状：带 limit/offset 时是 `{results, total}`，带 `pinnedOnly=true` 时是裸数组。前端 getPinnedAIReports 声明的返回类型是 AdminAiReportList、读的是 `res.results`，于是 `pinnedReports.value` 被赋成 undefined，模板里 `pinnedReports.length > 0` 在渲染时抛 `Cannot read properties of undefined (reading 'length')`， 整个页面白屏。空库也照抛 —— 裸数组同样没有 .results。 置顶那支改成同样的 `{results, total}` 信封，total 就是条数。「不分页」的意图 没变，只是别再让一个 URL 有两种形状：调用方没法照着一个类型写，类型标注也就 成了谎话。 验证：本机起 dev 栈，用 headless chromium 走 CDP 抓未捕获异常。修之前 /admin/ai/reports 稳定复现 `TypeError ... at Proxy._sfc_render (src/admin/ai/list.vue:182)`；修之后同一页面零异常，且置顶提示条正确渲染出 「以下 2 位用户的 AI 分析报告已被锁定」加两个用户标签、表格 3 行，PIN 切换 往返（取消 → total 1、再 PIN → total 2）也正常。 顺带把另外 19 个后台/学生端页面扫了一遍，只有这一处抛异常。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-27 00:18:58 -06:00
yuetsh	a75c70c82d	perf(后端): 干掉 14 处 N+1 查询 ... 列表接口按行发查询是从阶段 3 一路带过来的写法：`Promise.all(rows.map(...))` 看着是并发的，但每行都往库里打一次，行数一多就是几百上千次往返。同样的模式 也散在几个后台批处理和写入路径里。全部改成「先收集 id，一条 inArray/group by 查回来建 Map」——`routes/problem.ts` 的 getProblemTags 早就是这么写的，这次 只是把剩下的地方对齐。 用户可见的列表： - `GET /problem-sets` 每行 5 条（题目数/我的进度/奖章/已获奖章/创建者）， limit 上限 250 就是 1250 次往返。改成固定 5 条，与行数无关。 - `GET /contests` 每场比赛一条 creator 查询。后台的比赛列表本来就是 join 出来的，只有这条公开列表漏了。 - `GET /admin/problems`、`GET /admin/contests/:id/problems` 每题一条标签查询。 - `GET /admin/problem-sets` 每行 3 条；`.../badges` 每个奖章一条 count。 后台批处理： - `refreshContestJoinedForAll` 原来是每个用户 1 条 count 加一个独立事务里的 insert/select for update/update。改成一条 group by 出全部用户的场次，再分批 upsert，`metrics || excluded.metrics` 是 jsonb 浅合并，只覆盖 contest_joined 一个键，其余指标原样保留 —— 合并在一条语句里完成，for update 那把锁不再需要。 - `rescanAchievement` 补发循环、`unlockAchievements`：命中的一次插完， onConflictDoNothing 的 returning 就是真新解锁的那批，unlockCount 改成一次 +N。 - `resyncProgress` 逐行 UPDATE 改成一条，completed 用 least() 夹住。 写入路径： - 标签解析抽出 normalizeTagNames + findTagsByName（一条 lower(name) IN）， 新建题、改题、批量打标签三条路共用。 - 克隆比赛：题面一条 INSERT、标签一条 SELECT 加一条 INSERT。新旧题的对应 关系靠 _id 认，不依赖 returning 的行序。 - `POST /admin/website` 8 个键一条多行 upsert。 - 题单奖章判定一次插完。 `/ai/duration`：原来每个时间桶两条查询、桶之间还串行，一年 12 个桶 24 次往返。 改成先算桶、再一条查询把整段区间拉回来在内存里分桶。时间戳用 `extract(epoch) * 1000` 取毫秒回来比，别指望 Date.parse 认 pg 那个 `2026-08-12 00:00:00+00` 格式。**相邻桶首尾相接、两端闭区间**（落在边界上的 提交两个桶都算）这条旧语义是照搬的，不要顺手改成半开区间。 验证：本机起 dev 栈，造了覆盖各分支的种子数据（创建者重复的题单、零题目/零 奖章的题单、completed > total 的脏进度、除不尽的百分比、大小写混写的已有标签、 带/不带标签的比赛题、正好落在分桶边界上的提交），旧代码跑一遍、新代码跑一遍： - 51 个接口响应逐字节一致 - 12 张表的快照逐行一致（唯一差别是 progress_detail 里的 submit_time 墙钟值） - 打开 log_statement=all 数过条数，例如 `GET /admin/problems` 20 道题 24 → 5，`GET /problem-sets` 28 → 8，`/ai/duration` years:1 28 → 5， 202 个用户的成就补发 1828 → 614 一处可观察的行为变化：补发现在整批共用一个 unlockTime，原来是每人一个 new Date()。rescanAchievement 上方的注释本来就写着补发会给几百人盖同一个 时间戳、前端据此只显示「已获得」不显示日期，所以这个方向是对的。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-26 23:43:57 -06:00
yuetsh	f00c941ede	perf(构建): builder 分开拷源码，只改后端不再陪编一次前端 ... builder 阶段是 `COPY . .` 一把梭，而 docker 的层缓存是链式的：那一层的校验和 覆盖整个仓库，改任何一个文件都会把它作废，后面 vite build 和 bun compile 全部 重跑。前端那 160s 一次都躲不掉，改一篇 markdown 也要陪着编。 按各自的输入分开拷，慢的那条放前面（缓存链上越靠前越不容易被碰到）： 只改 apps/api 前端两层命中缓存，只重编后端 ~1s 只改 apps/web 前端重编，后端被链条带着 ~160s 改 packages/contract 两边都重编（本来就都依赖它，是对的） 改 docs / 根上的杂项 builder 根本看不见 0 代价是 builder 只看得见这里点名拷进来的四样，新加顶层目录要同步加 COPY。 顺带堵掉两个同源的洞： · apps/web/docs、tests、**/CLAUDE.md 进了 .dockerignore。它们在 apps/web 里面， 整个目录拷进去的话，改一篇文档也会让 vite build 那层失效。 · .dockerignore 末尾为 --prebuilt 开的例外（!dist/oj2-api、!apps/web/dist）让本地 产物照样进构建上下文 —— 这是比 `COPY . .` 更隐蔽的一条：在本机编一次后端， 80MB 的 dist/oj2-api 变了，rsync 上去，服务器就重编一次前端，源码一行没动。 dist/oj2-api 现在不再被 builder 拷贝；apps/web/dist 仍在 COPY apps/web 里面， 所以 deploy.sh 头上那条手动 rsync 补了 --exclude dist。CI 走 --prebuilt， builder 整个不进构建图，不受影响。 验证：用老 Dockerfile 和新 Dockerfile 各构建一次 --target artifacts 导出对比， 后端二进制 sha256 相同，前端 dist 500 个文件逐个 sha256 一致。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-26 10:25:15 -06:00
yuetsh	1d9686b929	feat(后台用户): 管理员密码默认打码，点「显示」才展开 ... 后台用户列表把所有账号的明文密码直接铺在表格里。学生密码是有意保留的运营需求 （老师要能查），但管理员账号的密码跟着一起明文显示，投屏或者旁人路过就看到了。 现在学生管理员／教师管理员／超管三类账号的密码默认渲染成 •••••• 加一个「显示」 按钮，点一下该行展开成明文（仍然是 TextCopy，点击复制照旧）。普通用户不变。 展开状态按 user id 记在 list.vue 里，listUsers() 里清空，所以翻页、搜索、换筛选、 换排序之后一律回到打码；不做定时自动隐藏，也不记进 localStorage。 只改前端。rawPassword 照旧随列表下发——/users* 每条路由本来就是 requireSuperAdmin， 非超管连列表都拿不到，要防的是屏幕被看到，不是拿到响应的人。改成「按需取单个密码」 反而要多开一个下发明文的端点，把面扩大。 rawPassword 为空时不打码：给一个点了什么也不显示的按钮没意义，直接走原来的 TextCopy。 顺带把角色标签缩短（学生管理员→生管、教师管理员→师管）。getUserRole 只有同一张表的 用户名列在用，密码列变宽之后那一列需要让出横向空间。 实跑验证（本机 dev 栈 + 超管登录）：管理员行打码、学生行明文照旧、点「显示」后展开 且仍是可复制的按钮、搜索触发重新拉列表后回到打码。vue-tsc 与 build 均通过。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-26 10:08:30 -06:00
yuetsh	bb4b5825c3	feat(迁移): 0000 改成可执行，空库能自举 ... 0000_crazy_gateway.sql 原本是 drizzle-kit pull 的产物，整份被 /* */ 包着、 可执行语句 0 条，所以任何新库的结构都只能先手工 psql 灌一遍 schema.sql 再打基线。 现在它的内容由 docs/specs/schema.sql（2026-08-07 的生产 pg_dump --schema-only） 机械转换而来：去掉 psql 专有指令 12 条、去掉 7 张 Django 遗留表及其索引外键 41 条， 保留 212 条语句、顺序不动。转换脚本入库在 docs/spikes/。 不含 Django 那 7 张表，是因为 meta/0000_snapshot.json 从来就没有它们（pull 当时 tablesFilter 滤掉了），不建它们才和快照一致；0002 那串 DROP ... IF EXISTS 在新库上 空转、在生产库上真删，两边跑同一串迁移落点相同。 改 0000 对生产库没有影响：migrator 只比 created_at、从不校验 hash，而生产库那行 baseline-0000-faked 早把它挡在门外了。 migrate.ts 配套：空库直接从 0000 建起（并自建 drizzle 记账表——原来这步由 drizzle 的 migrate() 顺手做掉）；自举时不触发破坏性闸门，因为空库上没有数据可丢，拦下来只会逼 每个新环境都带一次 OJ2_ALLOW_DESTRUCTIVE，把这道闸训练成习惯动作。「有表但没基线」 仍然 exit 3。 验证：空库自举出来的结构，和「灌 schema.sql + 打基线 + 跑迁移」这条老路子跑出来的 结构，pg_dump --schema-only 逐字节一致（734 行，零差异）。 转换时踩到两个坑，都写进注释了：注释里不能出现 statement-breakpoint 的字面量 （readMigrationFiles 纯文本切分，会把注释从中间切开）；过滤 Django 对象要看 public.X 形式的对象引用，不能扫语句字样——user 表有一列叫 auth_token。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-26 09:54:31 -06:00
yuetsh	2e871cb5b0	refactor(迁移): 换掉 drizzle 的执行器，一条迁移一个事务 ... drizzle 的 migrate()（pg-core/dialect.js）把所有待执行的迁移塞进同一个 session.transaction()，两个后果：第 3 条失败会把第 1、2 条一起回滚，和 Django migrate 的逐条提交语义不一样；以及 CREATE INDEX CONCURRENTLY 一律跑不了， 没有任何开关。 现在自己按 journal 逐条执行，一条一个事务。文件第一行写 `-- oj2:no-transaction` 的迁移走裸执行（简单查询协议——扩展协议会把语句包进隐式事务块，CONCURRENTLY 照样被拒），代价是没有回滚，所以这类迁移一个文件只放一条语句。 记账行的写法和 drizzle 完全一致（hash = 整文件 sha256，created_at = journal 的 when），migrator 又只比 created_at、不校验 hash，两套执行器可以互换。 顺带：日志和报错说得出迁移文件名了（readMigrationFiles 不返回 tag，自己读一遍 journal），失败时打印的是 Postgres 那句话而不是 postgres.js 的内部堆栈， 新增退出码 5。 实跑验证（本机 Docker，生产 schema 灌进探针库）：CONCURRENTLY 带标记成功、 indisvalid = t，去掉标记复现 "cannot run inside a transaction block"； 0003 成功 + 0004 失败时，0003 留下、0004 的首条合法语句没留下。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-26 09:54:01 -06:00
yuetsh	eabb61189c	fix(数据库): 索引方向被 .op() 吞掉，schema.ts 和真实库对不上 ... 根因不是 .desc()，是 opclass。drizzle-kit 的 CreatePgIndexConvertor 里那个 三元一旦走进 opclass 分支就回不到方向分支，而 pull 给每一列都挂了 .op(...)， 于是"写了 .desc() 却生成不出 DESC"每次都会重演。实测确认：不写 .op() 时 多列混合方向能正常生成，所以原先"多列混合方向的索引别指望 generate"这条 自我限制不成立。 announcement_list_idx 和 contest_create_time_idx 两条真实索引踩在这上面： 生产库里它们带 DESC（schema.sql:1636、:1685），schema.ts 却会生成成全 ASC。 contest_create_time_idx 的 opclass 还串了位（contest_id 标 timestamptz_ops、 create_time 标 int4_ops），那条 SQL 真拿去执行 Postgres 会直接拒绝。 改快照而不是生成迁移：generate 想 drop + recreate，但重建出来的和生产库里 那两条完全等价（DESC 默认就是 NULLS FIRST），执行它只是在 12.3 万行的表上 白挨一次锁。手法和当初处理 problem_tag_name_ci_unique 的 opclass 一致。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-26 09:53:05 -06:00
yuetsh	1f9f33aa3b	docs: 0002 迁移已在生产库执行完毕 ... `docker exec oj-api oj2-api migrate` 回「没有待执行的迁移」，确认 0002_drop_django_leftovers 已经跑过，7 张 Django 框架表已从生产库删除。 CLAUDE.md 顶部原本写「生产库尚未执行」，切换手册的「回滚保证」写「即将作废」， 两处都过期了。旧栈现在是真的起不来，回滚只剩从备份恢复这一条路，措辞相应改成 既成事实。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-26 09:18:53 -06:00
yuetsh	7793a2fe4c	清理旧后端下线后的残留 ... 旧 Django 后端 2026-08-26 下线、回滚路径作废，代码里还留着几处以它为前提的 死代码和过期注释。 - PUBLIC_WS_URL / PUBLIC_OJ_URL 全部删除。BaseWebSocket 的 `${PUBLIC_WS_URL}/${path}/` fallback 是 Channels 时代的写法，而三个子类 早就各自传 url，等于永不执行；config.vue 里 PUBLIC_OJ_URL 只是个会被 getWebsiteConfig() 立刻覆盖、且指着 :8000 的假初值。WebSocketConfig.path 随之去掉，url 改成必填。 - vite.config：删掉 /api2、/ws2 的迁移期注释，换成还成立的约束（代理这三段 要和 docker/Caddyfile 同步）；newBackend 改名 backend。 - 六处 “回滚时旧后端还要读” 换成真实理由：snake_case 保留的结论不变，但因为 判题机按这套键名写、存量 JSONB 就是这形状。 - CLAUDE.md：数据库一节开头「不写迁移 + 先想清楚回滚」与下一节的 drizzle migration 自相矛盾，改掉；判题状态码不再要求同步到 OnlineJudge，理由换成 历史 submission.result 和沙箱用的就是这套编码。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-26 09:09:26 -06:00
yuetsh	64cb7b62ef	ci(部署): 手动触发可放行破坏性迁移，push 触发永远不放行 ... deploy.sh 现在会在起栈前跑迁移，但 CI 没法传 OJ2_ALLOW_DESTRUCTIVE， 于是任何破坏性迁移都会让自动部署一直红着，且没有出口。 给 workflow_dispatch 加一个 allow_destructive 布尔 input，经 env: 传到 ssh 命令里。push 触发拿不到 inputs，值恒为空 —— 也就是说**自动部署永远 不会执行 DROP TABLE 之类的迁移**，只会停在闸门上把工作流判红。这是有意的： 那种改动得有人先确认备份，不该由一次 git push 顺带完成。 日常的加索引 / 加列属于非破坏性，CI 部署会照常自动执行，无需干预。 留了一条注释标明必须用 `inputs.` 而不是 `github.event.inputs.`：后者会把 布尔字符串化，没勾选时拿到字符串 "false"，那是非空字符串、在 `&&` 里为真， 等于每次手动部署都无条件放行。 验证：本机 YAML 解析通过，确认 inputs 定义挂在 workflow_dispatch 下。 表达式三种情况的语义（勾选 → '1'、未勾选 → ''、push 事件 inputs 为空 → ''） 是 GitHub 表达式的既定行为，要真的 push 到 github 才跑得起来。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-26 08:38:47 -06:00
yuetsh	5f04f3283f	fix(部署): 迁移那步的服务名写成了 api，实际是 oj-api ... `docker compose run ... api oj2-api migrate` 直接报 `no such service: api`。 compose 里六个服务全带 oj- 前缀（oj-api / oj-worker / oj-web / oj-judge / oj-postgres / oj-redis），没有叫 api 的。 顺带修掉失败提示：原来不管迁移因为什么失败，都说「破坏性迁移需要 OJ2_ALLOW_DESTRUCTIVE=1 显式放行」——而这次的真实原因是服务名不存在， 提示把人往完全错误的方向带。现在改成让人去看 migrate 自己的输出 （它对每种情况都打印了具体该做什么），只把三种常见原因列成线索。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-26 08:32:19 -06:00
yuetsh	19054718d5	fix(部署): 迁移那步去掉 --no-deps，否则自带数据形态下起不来库 ... 上一个提交让 deploy.sh 支持「自带数据」形态，但迁移那步还留着 --no-deps。 `run --rm --no-deps api oj2-api migrate` 不会拉起 oj-postgres，于是这一步 直接连不上库 —— 而它正好排在 `up -d` 之前，postgres 这时候还没起。 oj-api 的依赖恰好就是 oj-postgres / oj-redis，两种形态都是对的： - 自带数据：靠 run 把 postgres 起来，depends_on 的 condition: service_healthy 还顺带保证库真的就绪了才开始迁移 - 外接：那两个服务不在启用的 profile 里，depends_on 上的 required: false 让 compose 跳过，不会多起东西 worker / web 不是 api 的依赖，两种形态下都不会被带起来，去掉 --no-deps 不会有副作用。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-26 08:25:17 -06:00
yuetsh	e6c8c3c155	feat(部署): deploy.sh 识别「自带数据」形态，不再写死试跑假设 ... 旧 Django 后端下线后要把 postgres / redis 收回 OJ2 自己的 compose 管。 compose 那边本来就备好了（oj-postgres / oj-redis 挂在 local-data profile 下， 镜像和旧栈同版本），但 deploy.sh 三处都写死了「并行试跑」的假设，会直接挡住： - COMPOSE 没有 --profile local-data，自带的两个容器永远不会被启动 - 自检② 见到 DATABASE_URL 指向 oj-postgres 就 die —— 而合并后它必然指向那里 - 自检④ 要求已经有 oj-postgres 在跑 —— 旧的停了、新的还没起，正好卡死 现在按 DB_HOST 是否为空自动判形态。判据用 DB_HOST 而不是另加开关，是因为它本来 就决定了 DATABASE_URL 指向谁；两个变量各说各话迟早会出现「起了自带的库、却连到 别处」这种自相矛盾的状态。 自检② 改成两个方向都查：外接形态下不许指向 oj-postgres，自带形态下必须指向 `oj-postgres:5432`。连端口一起查是因为清了 DB_HOST 却忘了清 DB_PORT 会拼出 `oj-postgres:5445` —— 那是宿主机映射端口，容器网络里不通。 新增自检②b：自带形态下检查 $DATA_DIR/postgres/PG_VERSION 存在且是 16。 这是整个部署里唯一会**静默**走歪的地方 —— DATA_DIR 不对的话 postgres 当成全新 部署，在空目录上初始化一个空库，站点起得来、能注册能登录，就是一道题都没有。 验证：用模拟的 env 跑 `docker compose config` 确认 --profile local-data → 服务多出 oj-postgres / oj-redis DATABASE_URL → postgres://onlinejudge:...@oj-postgres:5432/onlinejudge REDIS_URL → redis://oj-redis:6379 卷 → /root/OJDeploy/data/{postgres,redis}（旧栈同一批目录） 并验证了「清了 DB_HOST 但忘清 DB_PORT」会被自检②拦下、正常配置会放行。 形态判定对空值 / 非空值 / 缺失三种情况都试过。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-26 08:19:28 -06:00
yuetsh	66d325d460	ci(部署): rsync 加 --no-owner --no-group，别把 runner 的 uid 搬到服务器 ... 服务器上 `git pull` 报「检测到可疑的仓库所有权」： drwxr-xr-x 9 1001 1001 /root/OJDeploy/OJ2 drwxr-xr-x 8 0 0 /root/OJDeploy/OJ2/.git 原因是 `rsync -a` 展开含 `-o -g`（保留属主/属组），接收端又是 root， 所以源端的 uid 会被真写上去 —— GitHub 托管 runner 的 runner 用户是 uid 1001。 而 `.git` 在 --exclude 里，一直是当初手动 clone 时的 root。两者对不上， git 2.35.2 起的 safe.directory 检查就把服务器上所有 git 操作挡下了。 加 --no-owner --no-group 之后文件落地归 ssh 登录用户（root）所有，不再错位。 注意这只防复发，**不修复已经错位的目录**：rsync 不带 -o/-g 时不会去改已存在 文件的属主。服务器上还要手动跑一次 chown root:root /root/OJDeploy/OJ2 只改这一个目录，**不要 -R** —— git 的属主检查不递归，而 OJ2/data 下可能是 「自带数据」形态的整个 postgres 数据目录和判题沙箱以别的 uid 建的运行目录， 递归 chown 会让它们起不来。 这个问题一直都在，只是以前没在服务器上跑过 git 命令所以没撞上。 验证：只做了 YAML 语法校验 —— 这条工作流要真的 push 到 github 才跑得起来， 本机验不了。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-26 08:08:06 -06:00
yuetsh	586c88f629	build(数据库): 改用 drizzle migration，加提交列表索引、清掉 Django 残留 ... 一条线上的三件事：让 drizzle 的迁移机制真正可用 → 用它加索引 → 用它清掉 不再需要的 Django 表，最后接进部署和 CI。 ## 1. 让 drizzle-kit generate 可用 原本以为不能用：只加一个索引，generate 却吐出一堆噪音，其中 5 条 `DROP SEQUENCE auth_*/django_*` 打到生产库上会直接搞坏旧后端。 逐个查下来全是 `pull` 出的基线自己不能 round-trip，都是可修的： - **快照里的 Django 序列**：tablesFilter 只过滤表、不过滤它们的序列。 已从 0000_snapshot.json 清掉。 - **bigint 上限精度**：pull 生成的 `maxValue: 9223372036854775807` 是 JS number 字面量，round-trip 成 ...776000，每次 generate 都多出 10 条 ALTER COLUMN。改成字符串。 - **表达式索引的 opclass**：problem_tag_name_ci_unique 在快照里带 opclass，drizzle 自己序列化不出来，导致每次 drop + recreate。已去掉。 改完 `generate` 是干净的 no-op。 两个改不掉、只能绕的写进了 CLAUDE.md：索引 `.desc()` 生成 SQL 时会被丢 （单列索引不写方向即可，Postgres 用 Index Scan Backward 服务 ORDER BY DESC，实测同样 0.08ms）；migrator 把所有语句包一个事务， CREATE INDEX CONCURRENTLY 跑不了。 最容易吃亏的是 drizzle 没有 --fake-initial：对已有数据的库直接 migrate 会从 0000 跑起、撞表回滚，**而且 exit 1 但一个错误都不打印**。 ## 2. 0001 提交列表索引 `WHERE contest_id IS NULL ORDER BY create_time DESC LIMIT n` 用不上现有的 contest_create_time_idx (contest_id, create_time DESC) —— Postgres 不把 `IS NULL` 当成能吃掉首列、从而继承第二列有序性的等值条件。把 enable_seqscan / enable_bitmapscan 全关掉逼它用也不肯，宁可走单列 contest_id 索引再全量排序。于是每翻一页都 Parallel Seq Scan 扫完整张表。 换成部分索引后谓词由索引自己保证，索引序就是查询要的排序序。生产快照 （12.3 万条提交）实测首页取 10 行：61.8ms / 读 18936 blocks → 0.22ms / 读 34 blocks。端到端 94ms → 6ms。 真正要命的不是单次 61ms，是每个请求都要把 169MB 的表刷一遍 shared_buffers —— 一节课几十个学生同时开提交列表，磁盘和缓存直接被打穿。 ## 3. 0002 删掉 Django 残留 确认旧 Django 后端不再使用、也不再作为回滚路径。删前核实过：没有任何 OJ2 保留的表引用这 7 张，3 条外键全在它们内部（所以不用 CASCADE，真有 漏网的会报错而不是被悄悄级联掉）；5 个序列都由各自的表 owned，随 DROP TABLE 一并消失；数据全是 Django 自身元数据。tablesFilter 随之移除。 **回滚路径就此作废** —— CLAUDE.md 开头和 runbook 的「回滚保证」「七、回滚」 都改了。这条迁移已在本机 dev 库和生产快照副本上跑通，**生产库尚未执行**。 ## 4. migrate 接进部署与 CI deploy.sh 在构建之后、起栈之前跑 `oj2-api migrate`，失败就中止部署（旧 容器原样还在跑）。CI 走的也是 deploy.sh，所以不用给 GitHub 配数据库凭据， 也不用把生产库对外开放。 迁移文件**不内嵌进二进制**，随镜像装在 /usr/local/share/oj2/migrations。 这样 drizzle 的 migrate() 能原样用 —— 靠 _journal.json 自动发现，新增迁移 不用改任何代码，和 Django 扫 migrations/ 是一回事。内嵌就得为每条迁移 手写一行 import，那是迟早会漏的账。（CLAUDE.md 里「单二进制不能读文件」 那条讲的是 node_modules 和 import.meta.dir 推路径，按显式绝对路径读一个 数据目录不在此列。） 三道闸门，都是写完测出来才补上的： - **破坏性迁移拦截**：DROP TABLE / DROP COLUMN / DROP SCHEMA / ALTER COLUMN ... TYPE / TRUNCATE 命中就退出 4，需要 `OJ2_ALLOW_DESTRUCTIVE=1` 显式放行。DROP INDEX / DROP CONSTRAINT 不算， 拦了只会让人习惯性带上放行开关。扫描前先剥注释，避免误报。 - **基线缺失**：退出 3 并直接打印该敲的 SQL。注意判的是 `max(created_at) < 0` 而不是「表不存在」—— 表存在而为空（上次迁移失败 留下的）同样是没基线。 - **迁移目录读不到**：这是最可能犯的错（Dockerfile 漏拷），原本是 drizzle 的堆栈，现在直接说该检查哪一行。 另外发现 0000_crazy_gateway.sql 是 pull 的产物，**整份被 /* */ 包着， 可执行语句 0 条**，所以这个库根本不能靠迁移自举建表。原先写的「空库就从 0000 建」跑起来会炸在一个和真实原因毫不相干的 unterminated /* comment 上。 现在如实说明：结构只能来自 docs/specs/schema.sql 或生产 dump。 验证：镜像内编译（ARTIFACTS=build）出的真实镜像跑完五种场景 —— 拦截、 放行、幂等、无基线、漏拷目录，全部符合预期；dev 形态同样五种场景全过。 tsc / 路由遮蔽 / deploy.sh 语法 / generate no-op 都通过。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-26 07:56:30 -06:00
yuetsh	2ee61756b8	perf(提交列表): count 去掉无谓 join、只取序列化用得到的列 ... 用户反馈「HEADER 点提交后页面空白一段时间才有数据」。拿生产快照 （12.3 万条提交 / submission 表 169MB）在本机实测，问题分两头。 **后端**（本次改的）： - count 无条件 `innerJoin(problem)`，但 problem 只有按题号筛选时才出现在 where 里。带 join 的 count 走 seq scan 78ms，去掉 join 走索引 7.5ms。 - 行查询 `select submission.* + problem.*` 把两张表所有列都拉回来，包括 submission.code（学生源码）、info、ip，以及 problem 的 description / hint / samples / answers / flowchart_data / sql_display —— 这些字段 map 的时候一个都没用上。改成只 select 需要的列。 canViewSubmission 的参数类型随之从整行 $inferSelect 收窄成实际用到的 字段，完整行结构上仍然满足，详情接口调用不受影响。 公开列表和比赛列表两处是同一份代码，一起改了。 **前端**： - list.vue 静态 import 了四个只在默认关闭的 n-modal 里用的组件，其中两个 统计面板还只有老师看得见。光 chart.js 就 197KB，进页面前必须先下完。 改成 defineAsyncComponent 后本路由增量下载 675KB / 59 个文件 → 375KB / 43 个文件。 - n-data-table 没传 :loading，等接口这段时间表格就是一片空白，连转圈都 没有 —— 这是「页面空白」最直接的观感来源。用 try/finally 包，接口抛错 不会把转圈卡死。 - isAuthed 变化时重复拉了一次今日提交数。它不看登录态，onMounted 那次 就够了。列表本身仍然重拉（要更新提交编号列的可点击状态），那次不是 浪费；本想用 userStore.isFinished 把首次请求延后，但 getProfile() 一旦 reject，isFinished 会永远停在 false，匿名用户就再也看不到列表了。 还有一个更大头的原因是索引用不上，导致每次翻页全表扫 169MB，那部分 需要加索引，走下一个提交。 验证：起真实 API 打生产快照，匿名 / 已登录 / myself / 题号筛选 / 语言+状态 / today / offset=5000 / 比赛列表全部 200，响应体大小前后一致 （2990 bytes），字段没丢。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-26 07:55:47 -06:00
yuetsh	a9332df6bd	update	2026-08-26 00:28:18 -06:00
yuetsh	7b6bfeb6fb	fix(标签): /problem-tags 计数漏过滤隐藏题和赛题 ... 标签列表按题目数 > 0 过滤，但计数没有像 /problems 那样限制 visible=true 且非赛题，导致标签在首页出现、点进去却一道题都没有。 同时把旧仓库冻结政策收紧写进 CLAUDE.md：从今天起旧仓库零改动， 不再有内部小修的例外，所有后续工作只落在 OJ2。 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-26 00:22:05 -06:00
yuetsh	d3365ecb32	ci(部署): actions/checkout v5→v7、actions/cache v4→v6 ... `oven-sh/setup-bun` 已经在最新大版本（v2.2.0 属于 v2 线），不动。 **actions/cache v4 → v6** 是这次的实际收益：v4 跑在 **node20** 上，GitHub 正在 退役这个 runtime；v6 是 node24，和另外两个 action 一致。v6.0.0 的改动是「迁到 ESM + 更新依赖」，没有输入输出上的破坏性变更。 ⚠️ cache v5 起要求 Actions Runner >= 2.327.1。这里 `runs-on: ubuntu-latest` 是 GitHub 托管的、始终满足；哪天换自建 runner 要留意这条（已写进 yml 注释）。 **actions/checkout v5 → v7**：v5 本来就跑在 node24，运行时没变。两条标了 BREAKING 的改动都打不到这个工作流： - `allow-unsafe-pr-checkout`（更安全的 pull_request_target 默认值）已经反向 移植到 v2 到 v6 所有线，v5 上本来就有； - v7.0.0 的「block checking out fork pr」只影响 `pull_request_target` 和 `workflow_run`。 本工作流的触发器只有 `push: main` 和 `workflow_dispatch`，两者都不涉及。 版本按仓库原有习惯钉大版本号，没钉到 patch。 验证：只做了 YAML 语法校验 —— **这条工作流要真的 push 到 github 才跑得起来， 本机验不了**。下次 `git push github main` 时留意一下 Actions 那边。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-26 00:01:28 -06:00
yuetsh	75dd9bfa42	fix(提交): 独立的 /submission/:id 页面上「复制回到题目」抛 problemID ... ## 复现与根因 在 `/submission/<提交id>` 这个独立页面点「复制回到题目」，必抛： Error: Missing required param "problemID" `detail.vue` 的 `problemID` 是**可选** prop，但 `copyToProblem()` 无条件把它塞 进 `router.push({ params: { problemID } })`。三个组件调用方（提交列表弹框、 题目页提交弹框、后台 ACM 助手）都显式传了这个 prop，所以弹框里一直是好的； 而路由 `submission/:submissionID` 配的是 `props: true`，只喂得进 `submissionID` —— 那条路上这个 prop 恒为 undefined。 组件想自己兜底也兜不了：`submissionDetailSchema` 只有 `problemId`（内部数字 id），没有拼路由要用的 display id（`problem._id`）。 ojnext 里是**一模一样**的代码，不是这次重写引入的。 ## 改法 后端补 `problemDisplayId`：`submissionDetail()` 本来就 join 了 problem 表， 不额外查库。前端 `props.problemID ?? submission.problemDisplayId`。 ## ⚠️ 还剩一条边没关 非管理员看到的 `contestId` 是被抹掉的（`full ? contestId : null`，对齐旧后端 `SubmissionSafeModelSerializer` 的 exclude，把关的是角色不是归属）。所以**学生 自己的比赛提交**从这个独立 URL 打开时，组件判断不出它属于比赛，会跳到公开题 路由 `/problem/<显示编号>`，那里查的是 `contestId is null`，落到「题目不存在」。 改之前这条路是抛异常，改之后是跳错地方 —— 都不对，但主路径（非比赛提交）现在 是对的。要彻底关掉得让后端把 `contestId` 也发给**提交本人**（不只是管理员）， 那是在动一条刻意对齐旧后端的决定，留给你定。另外这条路由**全站没有任何入口 链接**（查过了），只有直接输 URL 或外部链接才会到。 ## 验证 真起服务、真点按钮： - 修之前：点击不跳转，控制台 `Missing required param "problemID"`， Vue 警告里能看到 `<Detail submissionID="..." >` 确实没有 problemID。 - 修之后：无异常，跳到 `/problem/1004`（那条提交对应的题）， 编辑器里是这条提交的代码（`n = int(input())…`），语言也带过去了。 - 回归：走提交列表弹框（有传 prop 的那条路）照样无异常、照样跳 `/problem/1004`。 - tsc(apps/api) 0 error、check:routes 168 条无遮蔽、vue-tsc 0 error、build 通过。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-25 23:59:46 -06:00
yuetsh	dd6a6a04eb	refactor(密码): 删掉 PASSWORD_HASH_UPGRADE，无条件写 argon2 ... 上一个 commit 把五个写入点收进 hashPassword，顺手把开关默认值翻成了 true 并 留下 false 分支当退路。这个退路是假的，删掉。 **开关只能拦住将来，修不了已经发生的。** 等真到了要把旧站拉回来的那天，活跃 账号早就都被登录/重置密码升成 argon2 了，这时候设 `=false` 一个也救不回来。 正经退路是切换手册「万一已经改坏了」那节的脚本 —— 拿 `raw_password` 重算 Django 的 `make_password`，能修已经变成 argon2 的账号。留着一个永远轮不到它 上场的开关，只是给后来人多一件要读懂的东西。 删掉的：`config.passwordHashUpgrade`、`hashDjangoPbkdf2` 和它那套 Django 参数 （1200000 迭代 / 22 位 salt / RANDOM_STRING_CHARS）、两个 compose 里的 `PASSWORD_HASH_UPGRADE` env。auth.ts 的登录升级变成无条件。 保留的： - `hashPassword()` —— 虽然现在只是一行 argon2，但「只有一个地方写密码」正是上 一个 commit 的重点。五处各写各的才是当初那个 bug 的根。 - `verifyPassword` 的 **pbkdf2 分支** —— 生产库 1710 个账号全是 Django 写的 pbkdf2（迭代次数 120000～1200000），只会在各自下次登录时才迁移。删了就是 全站登不上。代码注释里写死了这句话。 - 切换手册里的 `raw_password` 恢复脚本，以及回滚流程里新加的「密码要额外处理」 那一步。 ## 验证 改完重跑了一遍关键路径（临时造一个 Django 生成的 120000 迭代老哈希）： - 存量账号登录 200 → 哈希变成 argon2 → 再登 200 → 错密码 401 - 后台重置密码 → argon2 → 新密码能登录 - 注册 → argon2 → 能登录 tsc(apps/api) 0 error、check:routes 168 条无遮蔽、vue-tsc 0 error、build 通过、 两个 compose `config -q` 通过。测试用户已删干净。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>	2026-08-25 23:50:55 -06:00