Compare commits

...
Author SHA1 Message Date
ila f2fdf405a9 style: 放大章节阅读区与编辑区,使其随窗口宽度自适应
阅读区:正文列上限由固定 820px 改为 min(1040px, 92vw);打开查词面板时的容器由 1120px 改为
min(1240px, 96vw),大屏上正文列不再比不开面板时更窄。编辑区:章节编辑对话框由固定 620px 改为
min(1180px, 94vw),正文输入框最小高度改为 62vh(行数仅作初始值),书籍对话框 460px 改为 560px。

新增验证:reading 用例检查 768/1024/1280 三档宽度不横向溢出,并在 1280 下断言正文列宽 > 820px;
edit 用例断言章节对话框宽 > 900px、正文输入框高 > 400px 且不溢出。手机(≤760px)的底部抽屉样式未改动。

测试:学习端 159 单测、26 项 e2e、vue-tsc、build 全部通过。无长期文档影响(仅样式数值)。
2026-09-16 23:33:50 +08:00
ila 42945dec0a docs: 记录 #32、#40、#42 的验收结果与镜像 (#32 #40 #42)
- #40 与 #42 已由用户验收;PR #41 与 #43 以 rebase-ff 合入 main,#32 因分支携带仅改镜像的
  过程提交改以 squash 合入(f565c1c),三者均未改写分支历史。
- Wiki:Home 的两处"尚未验收"改为已验收文案,Project-Profile 追加三条已确认决策;
  会话规则数字、英汉词典与音标规则此前已发布。
- README 与 AGENTS 记录最终 schema v12、音标来源分层、30 天会话代价与 #32 的合并方式。
- docs/ 由 harness.py sync 导出,已通过 sync --check 与 harness check --strict。
2026-09-16 21:51:53 +08:00
ilaandila f565c1ca26 fix: 编辑回到曾经用过的正文版本复用该版本的任务 (#32)
编辑正文回到曾经用过的版本时,复用该内容版本已有的处理任务行(按 owner_id + chapter_id + content_sha256 匹配),避免重复插入触发唯一键错误。测试:Go 集成测试 A→B→A→B 全链路通过。

Co-authored-by: QiuSW <ila2002@qq.com>
2026-09-16 21:50:25 +08:00
ilaandila 4e4fb85d2f fix: 登录有效期由 8 小时改为 30 天 (#42)
- 有效期提取为具名常量 SessionLifetime(绝对有效期,不滑动续期),学习端与管理端共用
  同一登录接口,一处生效;数据库结构与接口字段不变,旧会话按各自 expires_at 自然过渡。
- 过期断言改为按常量计算(临期仍可用、超过后被拒),新增测试锁定 30 天常量、
  登录响应与库内 expires_at 一致、库内只存 SHA-256 摘要。
- 撤销规则不变:退出撤销当前会话,改密码/停用/重置撤销该账号全部会话。
- Wiki 已在线上更新(会话规则数字、验证说明、Home/Project-Profile 记录);镜像导出留到
  验收时执行,避免把尚未合入的 #40 段落带进本分支。
2026-09-16 21:42:38 +08:00
15 changed files with 304 additions and 26 deletions
+3
View File
@@ -300,3 +300,6 @@ MVP 内所有单元任务通过后才能做 MVP 集成验收;MVP 通过后才
- 2026-09-11 用户确认 #10 编辑与删除口径:可改名、改章节标题、编辑章节正文;**只有正文变化才重新处理**,重复保存或改回原内容不新建任务,只改标题不改状态。版本键是 `content_sha256`:任务只在与章节版本一致时才能影响章节,过期版本任务标为 `error_reason=superseded` 且**完全不触碰章节**(认领、恢复扫描、重试都按版本裁决);存储文本重算 SHA 与存储 SHA 不一致时按 `content_changed` 失败。删除为事务内硬删除 + 外键级联,删章后重排序号;个人词条、复习排期与作答记录**不随删除清理**。回收站/撤销、批量操作、章节跨书移动与语言变更不在范围内。编辑器行尾归一为 LF 是已知边界。
- 2026-09-11 用户确认 #9 TXT 导入口径:只接受 UTF-8(允许可选 BOM,解码时剥离且不进入原文),非法字节整体拒绝、不使用替换字符;UTF-16 按 BOM 识别后明确拒绝,GB18030 等按非法 UTF-8 拒绝。文件字节上限 2 MiB,之后仍套用单章 100000 码点上限;换行与空白不归一化。文件只在内存中解码、不创建临时文件,客户端文件名不参与任何路径也不入库。解码后交给既有 `PasteBook`/`PasteChapter`,分章(一次提交一章)、`requestId` 幂等与任务恢复与粘贴一致;不改 schema。EPUB/PDF/字幕、UTF-16 转码、按空行自动分章与断点续传不在范围内。
- 2026-09-11 用户确认 #8 到期单词复习决策表:固定间隔表 1/2/4/7/15/30/60 天,答对升级封顶 7、答错降级最低 1、再学一次不改等级,答错与再学立即回队;已知/忽略不入队,新保存的词立即到期,显式「学习中 level N」排 now+间隔[N];只有新建或状态/等级实际变化才移动复习时间,编辑释义或例句保留原排期,保存未提及等级时保留已获得等级。到期判定用 UTC 绝对时刻(`due_at ≤ now`),不引入本地日边界。作答按 `answerId` 去重并以 `expectedDueAt` 判定过期标签页,重复提交、网络重发与双标签页都不得重复更新次数与间隔(作答响应 `result` 只取 applied/stale,重放另用 `duplicate` 标记并返回首次结果);`correct_count` 只计答对,`wrong_count` 计答错与再学。短语复习归 #11,进度统计归 #13,不做策略配置 UI(X11)、练习模式(X08)与 FSRS。
- 2026-09-16 用户验收 #40 英汉词典与规范音标:ECDICT 常用子集(约 8.2 万条,MIT)提供中文释义、CMUdict(BSD-2)音标按固定 ARPAbet→IPA 映射转写,WordNet 词表用于裁剪子集;三者 sha256 pin 在 `server/zh-dictionary-resource.json`,`scripts/dict_prepare.py` 显式准备(产物不入库、运行时不联网,整包存 MySQL 因而包含在 #15 备份内)。**schema 最终为 v12**(放开 `lexgo_dictionaries` 单行 CHECK、增加 `provider` 列,两步带条件守卫、可重放;旧二进制见 v12 拒绝启动)。音标来源必须可追溯(`cmudict` 为 IPA 转写、`ecdict` 仅字符级规范化),不可靠就不显示(`grok` 属此类,`^` 丢失首音整条丢弃)。查词中文释义在前、英英释义随后,屈折形经 WordNet lemma 回查中文但不复制词目音标;两本词典可分别启停。许可文件随仓库保存,词典归档不提交。
- 2026-09-16 用户验收 #42 登录有效期:学习端与管理端共用同一登录接口,有效期由 8 小时改为 **30 天**(具名常量 `SessionLifetime`,绝对有效期、不滑动续期)。已知代价:令牌泄露窗口放大约 90 倍;设备/会话列表与「退出其他设备」未包含,如需另开工单。撤销规则不变(退出撤销当前会话,改密码/停用/重置撤销该账号全部会话,服务端只存 SHA-256 摘要)。
- 2026-09-16 用户验收 #32 缺陷修复:编辑正文回到曾用版本时复用该内容版本已有的处理任务行(按 `owner_id + chapter_id + content_sha256` 匹配),避免唯一键冲突;因分支带有仅改镜像的过程提交,PR #39 以 squash 方式合入 main(`f565c1c`),未改写分支历史。
+1 -1
View File
@@ -8,7 +8,7 @@
- [英语分词与离线词典验证小样](spikes/english/README.md)(#3 已验收,独立本机入口)
- [阅读选择验证小样](spikes/selection/README.md)(#4 已验收,真机详细测试证据缺口保留)
- [项目档案](docs/00-project-profile.md) · [需求总览](docs/09-product-requirements-overview.md)
- [工作量估算](docs/10-workload-estimate.md):#2、#3、#4、#5、#6、#7、#8、#9、#10、#11、#12、#13、#14、#18 已验收,原规划中的 #5 已完成;#15、#21 与 #37(章级音频、章节插图、章节/书籍作者)已验收;#24 待排期;缺陷 #32 待确认修复方案;后续结合集成结果重估,旧全量研究仅供参考。
- [工作量估算](docs/10-workload-estimate.md):#2、#3、#4、#5、#6、#7、#8、#9、#10、#11、#12、#13、#14、#18 已验收,原规划中的 #5 已完成;#15、#21 与 #37(章级音频、章节插图、章节/书籍作者)已验收;#32(编辑复用版本缺陷)、#40(英汉词典与规范音标)与 #42(登录有效期 30 天)已验收;#24 待排期;缺陷 #32 待确认修复方案;后续结合集成结果重估,旧全量研究仅供参考。
- [四阶段实施总览 #16](https://git.ilapage.cn/OPC/lexgo/issues/16):14 张单元工单,工程基础 → 技术验证 → 首条学习闭环 → 补齐 MVP;原型 v1 已获用户验收。两端使用账号(用户名)+密码登录,不要求邮箱。
- [原型工单 #1](https://git.ilapage.cn/OPC/lexgo/issues/1):Quant-UX 桌面/手机原型 v1,预览入口与审核记录见工单及需求总览。
- 运维命令:`./lexgo migrate|bootstrap|serve|backup|restore|verify`(纯二进制),开发便利与交叉验证走 `python scripts/server.py …` / `python scripts/ops.py …`
+7 -2
View File
@@ -2,8 +2,8 @@
generated: true (请先修改 Gitea Wiki,禁止直接编辑本文件)
wiki_page: Project-Profile
wiki_url: https://git.ilapage.cn/OPC/lexgo/wiki/Project-Profile.-
wiki_revision: e61817b949693c82e74ad3d1a8adb39983a90fab
synchronized_at: 2026-09-16T05:27:04Z
wiki_revision: a482d5ba1fa50d2b570042376134cc0d7a67ef36
synchronized_at: 2026-09-16T13:51:35Z
<!-- gitea-wiki-mirror:end -->
# LexGo 项目档案
@@ -196,3 +196,8 @@ server 是 go-admin 的选用模块接入:原样保留 SysUser、SysDept、必
至此 MVP 原范围 #2~#15 与增补 #21、#37 全部实施并验收;仍待处理 #24 书库列表优化与缺陷 #32。#16 的 #37 复选框已勾选。
- 2026-09-16 新增第二本英语词典(#40):ECDICT 常用子集提供中文释义、CMUdict 提供 IPA 转写音标,二者由 sha256 pin 的源显式准备;schema v12 放开 `lexgo_dictionaries` 单行约束并增加 `provider` 列,英英与英汉词典可分别启停。音标来源必须可追溯(IPA 转写 / ECDICT 记法),不确定时不显示。许可:ECDICT 仓库 MIT(数据由公开资料整理,再分发前需自行评估)、CMUdict BSD-2,许可证随仓库保存,词典归档不提交进 Git。
- 2026-09-16 登录有效期由 8 小时改为 **30 天**(#42):两端共用同一登录接口,有效期是具名常量 `SessionLifetime`,绝对有效期、不滑动续期;服务端只存令牌 SHA-256 摘要,改密码/停用/重置仍撤销该账号全部会话。设备与会话管理界面未包含,如需另建工单。
- 2026-09-16 用户验收 #40(英汉词典与规范音标):ECDICT 常用子集提供中文释义、CMUdict 提供 IPA 转写音标,两本英语词典可分别启停。**schema 最终为 v12**(放开 `lexgo_dictionaries` 单行 CHECK、增加 `provider` 列,两步带条件守卫、可重放)。音标来源必须可追溯(`cmudict` IPA / `ecdict` 记法),不可靠就不显示。资源由 `scripts/dict_prepare.py` 从 sha256 pin 的源显式准备,产物不入库;许可为 ECDICT MIT 与 CMUdict BSD-2。
- 2026-09-16 用户验收 #42(登录有效期):学习端与管理端共用同一登录接口,有效期由 8 小时改为 **30 天**(具名常量 `SessionLifetime`,绝对有效期、不滑动续期)。已知代价:令牌泄露窗口放大约 90 倍;设备/会话列表与"退出其他设备"未包含。
- 2026-09-16 用户验收 #32(缺陷修复):编辑正文回到曾经用过的版本时复用该内容版本已有的处理任务行;因分支携带仅改镜像的过程提交,PR #39 以 squash 方式合入 main(`f565c1c`),未改写分支历史。
+11 -2
View File
@@ -2,8 +2,8 @@
generated: true (请先修改 Gitea Wiki,禁止直接编辑本文件)
wiki_page: Architecture-and-Code-Map
wiki_url: https://git.ilapage.cn/OPC/lexgo/wiki/Architecture-and-Code-Map.-
wiki_revision: 8cc11b2fa972babeb4756b68c0b071282cdb1960
synchronized_at: 2026-09-16T05:27:05Z
wiki_revision: 64a83458a283469672e111f423d37d9478a3d0e3
synchronized_at: 2026-09-16T13:51:36Z
<!-- gitea-wiki-mirror:end -->
# 架构与代码地图
@@ -573,3 +573,12 @@ schema v8 新增两张表,都只用可重放的 `CREATE TABLE IF NOT EXISTS`
仍是 `resource_missing`。
- 屈折形:WordNet 的规则解析给出词目后,再用该词目回查中文词典,因此 `dogs` 也能看到"狗";
但音标属于词目,**不复制**到屈折形上(`dogs` 不显示 `/dɔɡ/`)。
## 登录会话有效期(#42)
- 会话有效期是单一具名常量 `SessionLifetime`(`server/app/lexgo/service.go`),当前为 **30 天**;
登录时写入 `lexgo_sessions.expires_at`,每次请求按 `expires_at > now` 校验。学习端与管理端共用
同一登录接口,因此这一个值同时决定两端的有效期。
- 有效期是**绝对**的,不随请求顺延;没有刷新令牌或轮换机制。撤销路径与有效期无关,保持独立:
退出删除当前会话行,改密码、停用、重置删除该账号全部会话行,登录时顺手清理该账号已过期的行。
- 服务端只保存令牌的 SHA-256 摘要(`token_hash` 主键),原始令牌只在客户端 sessionStorage。
+3 -3
View File
@@ -2,8 +2,8 @@
generated: true (请先修改 Gitea Wiki,禁止直接编辑本文件)
wiki_page: Business-Rules-and-Glossary
wiki_url: https://git.ilapage.cn/OPC/lexgo/wiki/Business-Rules-and-Glossary.-
wiki_revision: ba03226b60eee1c44062f4206b970a2047b8ddbc
synchronized_at: 2026-09-16T05:27:06Z
wiki_revision: 1f89692c0e24c3d2235a4cd7ad78b90512852f65
synchronized_at: 2026-09-16T13:51:36Z
<!-- gitea-wiki-mirror:end -->
# 业务规则与术语
@@ -54,7 +54,7 @@ M0 固定首发语言语料、词条身份规则、短语选择与重叠规则
- 账号去首尾空白并转小写,3~32 位、ASCII 字母开头,可含数字、点、下划线和连字符;唯一约束阻止大小写和并发重复。密码 10~72 字节,使用 bcrypt,不填写必需邮箱。
- 初始管理员只通过显式 bootstrap 建立;库中已有账号时拒绝再次 bootstrap,不覆盖密码。管理页面只创建学习者,不能将输入角色升级为管理员;本阶段不提供账号删除。
- 同一账号可以登录多个会话。随机令牌仅在客户端保存,服务端存 SHA-256 摘要,8 小时过期。退出仅撤销当前会话,启停/重置会撤销目标账号全部会话;重新启用不能恢复旧会话。
- 同一账号可以登录多个会话。随机令牌仅在客户端保存,服务端存 SHA-256 摘要,**30 天**过期(2026-09-16 由 8 小时改为 30 天,见 #42;单一常量 `SessionLifetime`,绝对有效期、不滑动续期)。退出仅撤销当前会话,启停/重置会撤销目标账号全部会话;重新启用不能恢复旧会话。
- 每次请求读取账号当前状态与会话有效期;学习数据由认证身份确定,管理员权限不能解除学习接口的本人过滤。
- 两端分别使用 lexgo-admin-token、lexgo-learner-token 的 sessionStorage;401、退出和账号切换清除私人状态,以代次检查阻止迟到响应恢复旧数据。管理端拒绝学习者并撤销其新会话;撤销服务异常会提示未确认,不冒充服务端退出成功。
- 新建账号拥有英语空空间。此处没有书籍、阅读、词典或复习功能,不把空页面视为学习业务已实现。
+18 -2
View File
@@ -2,8 +2,8 @@
generated: true (请先修改 Gitea Wiki,禁止直接编辑本文件)
wiki_page: Local-Development-and-Verification
wiki_url: https://git.ilapage.cn/OPC/lexgo/wiki/Local-Development-and-Verification.-
wiki_revision: afcd40ee530a24f924a39209c23a9dfd090114a2
synchronized_at: 2026-09-16T05:27:07Z
wiki_revision: 45306a5d50c5674f512ea206739c6b2d89e88a85
synchronized_at: 2026-09-16T13:51:36Z
<!-- gitea-wiki-mirror:end -->
# 本地开发与验证
@@ -1146,3 +1146,19 @@ python .local/verify-issue40-api.py # 28 项真实 API
读语句不受影响(新列有默认值),但旧二进制见到 v12 会拒绝启动并要求显式迁移,这是有意的保护。
- 排错:`provider` 不存在 → 未执行 migrate;导入 400 且提示版本不符 → 归档不是当前 pin 生成的
(重新运行 `dict_prepare.py`);查词只有英英释义 → 英汉词典未导入或被停用。
## 验证登录有效期(#42)
```powershell
# 固定值本身在测试里断言,改动会让测试立刻失败
cd server; go test ./app/lexgo/ -run TestMySQLSessionLifetimeIsThirtyDays -v
# 集成测试:到期前仍可用、到期后被拒绝(按 SessionLifetime 计算,不写死小时数)
LEXGO_TEST_DB_NAME=lexgo_test_issue42 python scripts/server.py test-integration
```
- `TestMySQLSessionLifetimeIsThirtyDays` 校验三件事:常量等于 30 天、登录响应 `expiresAt` 与库内
`expires_at` 一致、库内只存在摘要(原始令牌查不到);`TestMySQLAccountIsolationAndRevocation`
在 `SessionLifetime - 1h` 仍成功、再前进 2 小时被拒。
- 手工排查:`SELECT owner_id, expires_at FROM lexgo_sessions ORDER BY expires_at DESC;`。
若登录后立刻 401,先确认服务器时间与 `expires_at` 都以 UTC 存储(`DATETIME` 不存时区)。
- 旧的 8 小时会话不受影响:各自按自己的 `expires_at` 自然到期,不需要数据迁移。
+5 -3
View File
@@ -2,8 +2,8 @@
generated: true (请先修改 Gitea Wiki,禁止直接编辑本文件)
wiki_page: Home
wiki_url: https://git.ilapage.cn/OPC/lexgo/wiki/Home
wiki_revision: bbea927c73e51b6b5490de3cb52d42a88dfbb31d
synchronized_at: 2026-09-16T05:27:04Z
wiki_revision: e6e3b2a93c4d4f3a809872e8260c3776923c7623
synchronized_at: 2026-09-16T13:51:35Z
<!-- gitea-wiki-mirror:end -->
# LexGo 文档入口
@@ -101,4 +101,6 @@ Quant-UX 原型 v1 已通过用户验收。[桌面预览](https://qux.ilapage.cn
#37 章节级音频与章节插图已于 2026-09-15 实施完成,待用户验收:每章可以上传一段 MP3(≤20 MiB)与一张插图(JPG/PNG/WebP,≤2 MiB、≤4096×4096),封面仍按书;阅读页在正文上方显示本章插图的缩略图、点击后在弹窗里查看原图(书籍页章节列表不显示章节图片),播放器只在本章有音频时出现且不与文本同步,播放位置按「账号+章节」保存并在切换章节时上报;书籍页的封面压成一行紧凑控件(预览+上传/替换/移除),书名右侧显示书级作者,「编辑书籍」可改书名与作者,章节列表每行只有一个「编辑」,点开后在同一对话框里编辑标题、作者、正文与本章插图、音频(标题/作者/正文点保存后生效,文件选中后立即上传;阅读页在标题下显示作者)。类型按文件内容判定,读取需会话并支持 HTTP Range 与 ETag;附件存于数据库,包含在 #15 的备份恢复范围内(第三次恢复演练已验证按字节恢复)。schema v9 需显式迁移,书级音频接口已下线,其实测试期数据在迁移中清空。真机听感与浏览器真实解码播放仍需人工确认。
#40 英汉词典与规范音标已于 2026-09-16 实现并进入待验收:新增第二本英语词典——ECDICT 常用子集(约 8.2 万条中文释义,3.0 MB)与 CMUdict 转写的 IPA 音标。音标分三层来源:能由 CMUdict 音素按固定映射转写的用真 IPA(35,799 条),否则保留 ECDICT 记法仅做字符级规范化并标注来源(30,580 条),两者都不可靠就不显示(16,342 条,例如 `grok`);含 `^` 等丢失首音的记法整条丢弃,不做猜测。schema 升到 v12(放开 `lexgo_dictionaries` 单行 CHECK、增加 `provider` 列,两步都带条件守卫、可重放),英英与英汉词典可分别启停,查词时中文释义在前、英英释义随后,屈折形经 WordNet lemma 解析后也能看到中文释义但不会复制词目音标。归档由 `scripts/dict_prepare.py` 从 sha256 pin 的三个源显式准备(运行时不联网,产物不入库,仓库只保存 pin 与两份许可证),整包存进 MySQL 因而包含在 #15 的备份范围内。测试:Go 96 项、学习端 159 单测与 26 项 e2e(含 390×844 触摸视口)、管理端 32 项、真实 API 28 项检查;PR #41 待合入,尚未验收。发音音频、整句翻译与汉英方向不在本单范围。
#40 英汉词典与规范音标已于 2026-09-16 实现并进入待验收:新增第二本英语词典——ECDICT 常用子集(约 8.2 万条中文释义,3.0 MB)与 CMUdict 转写的 IPA 音标。音标分三层来源:能由 CMUdict 音素按固定映射转写的用真 IPA(35,799 条),否则保留 ECDICT 记法仅做字符级规范化并标注来源(30,580 条),两者都不可靠就不显示(16,342 条,例如 `grok`);含 `^` 等丢失首音的记法整条丢弃,不做猜测。schema 升到 v12(放开 `lexgo_dictionaries` 单行 CHECK、增加 `provider` 列,两步都带条件守卫、可重放),英英与英汉词典可分别启停,查词时中文释义在前、英英释义随后,屈折形经 WordNet lemma 解析后也能看到中文释义但不会复制词目音标。归档由 `scripts/dict_prepare.py` 从 sha256 pin 的三个源显式准备(运行时不联网,产物不入库,仓库只保存 pin 与两份许可证),整包存进 MySQL 因而包含在 #15 的备份范围内。测试:Go 96 项、学习端 159 单测与 26 项 e2e(含 390×844 触摸视口)、管理端 32 项、真实 API 28 项检查;PR #41 已合入 main,并由用户于 2026-09-16 验收通过。发音音频、整句翻译与汉英方向不在本单范围。
#42 登录有效期已于 2026-09-16 由 8 小时改为 **30 天**(学习端与管理端共用同一登录接口,一处生效):有效期提取为具名常量 `SessionLifetime`,仍然是**绝对**有效期、不做滑动续期,撤销规则不变(退出只撤销当前会话,改密码/停用/重置撤销该账号全部会话,服务端只存 SHA-256 摘要)。代价是令牌泄露后的可用窗口放大约 90 倍;设备/会话列表与"退出其他设备"未包含在本单,如需另开工单。无数据库结构变化,旧会话按各自到期时间自然过渡;测试:Go 92 项(新增 1 项锁定 30 天与摘要存储,并把过期断言改为按常量计算);PR #43 已合入 main,并由用户于 2026-09-16 验收通过。
+11
View File
@@ -80,9 +80,20 @@ test('rename the book, edit a chapter into a new version and delete both', async
await expect(page.getByRole('heading', { name: 'A long step' })).toBeVisible()
// Edit the chapter text: the new version re-processes and later becomes ready again.
// The chapter editor is the main writing surface, so on a wide window it must use that width and
// a tall body instead of a fixed 620px box.
await page.setViewportSize({ width: 1280, height: 900 })
await page.getByTestId('edit-chapter-9').click()
await expectModalOverlay(page, page.getByTestId('chapter-dialog'))
await expect(page.getByLabel('章节标题', { exact: true })).toHaveValue('First chapter')
await expect(page.getByLabel('正文', { exact: true })).toHaveValue(firstText)
const dialog = await page.getByTestId('chapter-dialog').boundingBox()
expect(dialog!.width, 'the chapter editor follows the window').toBeGreaterThan(900)
const body = await page.getByLabel('正文', { exact: true }).boundingBox()
expect(body!.height, 'the text area is tall enough to write in').toBeGreaterThan(400)
const noOverflow = await page.evaluate(() => ({ scroll: document.documentElement.scrollWidth, inner: window.innerWidth }))
expect(noOverflow.scroll, 'the wide editor must not overflow').toBeLessThanOrEqual(noOverflow.inner)
await page.setViewportSize({ width: 1280, height: 900 })
await page.getByLabel('正文', { exact: true }).fill('A replacement body.\n')
await page.getByTestId('save-chapter').click()
await expect(page.getByTestId('book-notice')).toContainText('已保存为新版本,正在重新处理')
+12
View File
@@ -137,6 +137,18 @@ test('paste English text, watch a chapter finish processing, then read it verbat
await expect(page.locator('.lookup-panel')).toContainText('Princeton WordNet')
expect(await readerText.evaluate(element => element.textContent)).toBe(pasted)
// The reading and editing surfaces must grow with the window instead of staying at a fixed
// 820px column, and no intermediate width may overflow sideways (the blind spot before this).
for (const width of [768, 1024, 1280]) {
await page.setViewportSize({ width, height: 900 })
await page.waitForTimeout(60)
const overflow = await page.evaluate(() => ({ scroll: document.documentElement.scrollWidth, inner: window.innerWidth }))
expect(overflow.scroll, `reader must not overflow at ${width}px`).toBeLessThanOrEqual(overflow.inner)
}
await page.setViewportSize({ width: 1280, height: 900 })
await page.waitForTimeout(60)
const wideBody = await page.locator('.reader-body').boundingBox()
expect(wideBody!.width, 'the text column follows the window').toBeGreaterThan(820)
// Browser narrow viewport check only; this is not real-device acceptance.
await page.setViewportSize({ width: 390, height: 844 })
await word.click()
+5 -2
View File
@@ -227,12 +227,12 @@ h1 { font-size: 30px; font-weight: 600; margin: 14px 0; letter-spacing: 1px; }
a.chapter-name:hover { color: var(--accent); text-decoration: underline; }
.chapter-meta { margin: 4px 0 0; color: var(--text-muted); font-size: 13px; overflow-wrap: anywhere; }
/* Reader keeps the pasted text exactly as it was, including line breaks and tabs. */
.reader-page { max-width: 820px; }
.reader-page { max-width: min(1040px, 92vw); }
.chapter-author { margin: -6px 0 0; color: var(--text-muted); font-size: 14px; }
.reader-text { white-space: pre-wrap; overflow-wrap: break-word; margin: 26px 0 0; font-family: Georgia, 'Times New Roman', 'Microsoft YaHei', serif; font-size: calc(17px * var(--reader-font-scale)); line-height: 2; }
.processing-hint { padding: 36px 0; color: var(--text-muted); }
.reader-nav { display: flex; align-items: center; justify-content: space-between; gap: 14px; flex-wrap: wrap; margin-top: 36px; padding-top: 22px; border-top: 1px solid var(--border); }
.reader-page.has-lookup { max-width: 1120px; }
.reader-page.has-lookup { max-width: min(1240px, 96vw); }
.reader-workspace { display: grid; grid-template-columns: minmax(0, 1fr); gap: 32px; align-items: start; }
.has-lookup .reader-workspace { grid-template-columns: minmax(0, 1fr) 320px; }
.reader-body { min-width: 0; }
@@ -270,6 +270,9 @@ a.chapter-name:hover { color: var(--accent); text-decoration: underline; }
.lookup-term .el-radio-group { margin-top: 8px; flex-wrap: wrap; gap: 4px 12px; }
.lookup-term .el-textarea { margin-top: 8px; }
.lookup-term .el-textarea textarea { font: inherit; line-height: 1.6; }
/* The chapter body is the main editing surface, so it grows with the window instead of staying
fixed at its rows attribute; the dialog is sized by viewport so a small laptop still fits. */
.chapter-text-input .el-textarea__inner { min-height: 62vh; line-height: 1.9; }
.lookup-saved { color: var(--saved); font-size: 14px; margin: 12px 0 0; }
.lookup-actions { display: flex; gap: 8px; margin-top: 14px; flex-wrap: wrap; }
.review-page { max-width: 680px; }
+3 -3
View File
@@ -398,7 +398,7 @@ onUnmounted(() => library.closeBook())
</section>
</template>
<ElDialog v-model="bookDialog" title="编辑书籍" width="460" data-testid="book-dialog">
<ElDialog v-model="bookDialog" title="编辑书籍" width="560" data-testid="book-dialog">
<div class="field-row">
<label for="book-title">书名</label>
<ElInput id="book-title" v-model="bookTitle" type="text" maxlength="200" :disabled="saving" />
@@ -414,7 +414,7 @@ onUnmounted(() => library.closeBook())
</template>
</ElDialog>
<ElDialog v-model="chapterDialog" title="编辑章节" width="620" data-testid="chapter-dialog">
<ElDialog v-model="chapterDialog" title="编辑章节" width="min(1180px, 94vw)" data-testid="chapter-dialog">
<p v-if="chapterLoading" role="status" class="loading">正在加载…</p>
<template v-else-if="chapterSource">
<div class="field-row">
@@ -426,7 +426,7 @@ onUnmounted(() => library.closeBook())
<ElInput id="chapter-author" v-model="chapterAuthor" type="text" maxlength="120" placeholder="可留空" :disabled="saving" data-testid="chapter-author" />
</div>
<label for="chapter-text">正文</label>
<ElInput id="chapter-text" v-model="chapterText" type="textarea" :rows="18" :disabled="saving" />
<ElInput id="chapter-text" v-model="chapterText" type="textarea" class="chapter-text-input" :rows="18" :disabled="saving" />
<p class="counter">{{ chapterLength }} / {{ TEXT_MAX_CODE_POINTS }} 字符 · 保存后重新处理这一章</p>
<div class="attachment-block">
+42 -6
View File
@@ -78,6 +78,45 @@ func editAuthor(raw string) (string, error) {
return author, nil
}
// stageEditJob returns the job that owns one content version of a chapter.
//
// One chapter and one content digest describe one version, so the row that already describes it is
// reused instead of adding another one. That matters because the derived request key of a version
// that has been edited before already exists: creating it again hit the unique key and failed the
// whole edit with a generic 500, so a learner could never return to a previous text. Reusing is also
// correct for the version a chapter was pasted with, whose row carries the paste request key.
func stageEditJob(tx *gorm.DB, owner int, chapter Chapter, sha string, ts time.Time) (IngestJob, error) {
var existing IngestJob
err := tx.Clauses(clause.Locking{Strength: "UPDATE"}).
Where("owner_id = ? AND chapter_id = ? AND content_sha256 = ?", owner, chapter.ID, sha).
Order("id ASC").First(&existing).Error
switch {
case errors.Is(err, gorm.ErrRecordNotFound):
key := contentSHA(fmt.Sprintf("edit:%d:%s", chapter.ID, sha))
created := IngestJob{OwnerID: owner, BookID: chapter.BookID, ChapterID: chapter.ID,
RequestKey: key, ContentSHA256: sha, Status: statusPending, CreatedAt: ts, UpdatedAt: ts}
if err := tx.Create(&created).Error; err != nil {
return IngestJob{}, err
}
return created, nil
case err != nil:
return IngestJob{}, err
}
// Publishing the same version again — edited back, or retried after it failed — reuses the row and
// restarts its attempt budget, so a person asking again is not blocked by the earlier failure.
if err = tx.Model(&IngestJob{}).Where("id = ?", existing.ID).
Updates(map[string]any{"status": statusPending, "attempts": 0, "error_reason": "",
"updated_at": ts, "finished_at": nil}).Error; err != nil {
return IngestJob{}, err
}
existing.Status = statusPending
existing.Attempts = 0
existing.ErrorReason = ""
existing.FinishedAt = nil
existing.UpdatedAt = ts
return existing, nil
}
// lockOwnedChapter returns the caller's chapter or reports it as missing, so another
// account's chapter id is never confirmed to exist.
func lockOwnedChapter(tx *gorm.DB, owner int, chapterID int64, chapter *Chapter) error {
@@ -176,14 +215,11 @@ func UpdateChapter(tx *gorm.DB, owner int, chapterID int64, input ChapterUpdateI
updates["char_count"] = count
updates["status"] = statusPending
updates["error_reason"] = ""
// The request key is derived from chapter and version, so one version has one job.
key := contentSHA(fmt.Sprintf("edit:%d:%s", chapter.ID, sha))
created := IngestJob{OwnerID: owner, BookID: chapter.BookID, ChapterID: chapter.ID,
RequestKey: key, ContentSHA256: sha, Status: statusPending, CreatedAt: ts, UpdatedAt: ts}
if err := tx.Create(&created).Error; err != nil {
staged, err := stageEditJob(tx, owner, chapter, sha, ts)
if err != nil {
return ChapterEdit{}, err
}
job = &created
job = &staged
}
}
if err := tx.Model(&Chapter{}).Where("id = ? AND owner_id = ?", chapter.ID, owner).Updates(updates).Error; err != nil {
+121
View File
@@ -719,3 +719,124 @@ func TestMySQLBookAuthorRoundTrip(t *testing.T) {
t.Fatal("another account must not edit the book")
}
}
// editChapterText edits the body of one owned chapter through the API and fails on any error.
func editChapterText(t *testing.T, r *gin.Engine, token string, chapterID int64, text string) ChapterEdit {
t.Helper()
code, msg, data := callRaw(t, r, "PATCH", fmt.Sprintf("/api/v1/chapters/%d", chapterID), token, map[string]string{"text": text})
if code != 200 {
t.Fatalf("editing back to a used version must succeed, got %d (%s)", code, msg)
}
var edited ChapterEdit
if err := json.Unmarshal(data, &edited); err != nil {
t.Fatal(err)
}
return edited
}
// TestMySQLChapterEditBackToAPreviousVersion covers the defect reported in #32: the request key of an
// edit job is derived from the content, so returning to a text used before must reuse that job instead
// of colliding with its unique key. Every step of A -> B -> A -> B has to succeed, the chapter has to
// end up holding the last text, and each version has to keep exactly one job row.
func TestMySQLChapterEditBackToAPreviousVersion(t *testing.T) {
db, r, owner := libraryFixture(t)
learner := newLearner(t, r, owner.Token)
first := "Curiosity opens the first door.\n"
second := "Curiosity opens a different door.\n"
code, pasted := pasteBook(t, r, learner.Token, map[string]string{
"requestId": "edit-version-0001", "title": "Fictional versions", "text": first, "language": "en"})
if code != 201 {
t.Fatalf("paste %d", code)
}
drainIngest(t, db)
chapterID := pasted.Chapter.ID
firstSHA, secondSHA := contentSHA(first), contentSHA(second)
// A -> B.
edited := editChapterText(t, r, learner.Token, chapterID, second)
if !edited.VersionChanged || edited.Chapter.Status != statusPending || edited.Job == nil {
t.Fatalf("the first edit must start a new version: %+v", edited)
}
// The response carries the client-facing job view; the version it describes is read back from the
// row itself.
var secondJob IngestJob
if err := db.Where("chapter_id = ? AND content_sha256 = ?", chapterID, secondSHA).First(&secondJob).Error; err != nil {
t.Fatalf("the new version needs its own job: %v", err)
}
if err := db.Model(&Chapter{}).Where("id = ?", chapterID).Update("status", statusReady).Error; err != nil {
t.Fatal(err)
}
// B -> A: this is the step that used to answer 500.
edited = editChapterText(t, r, learner.Token, chapterID, first)
if !edited.VersionChanged || edited.Job == nil {
t.Fatalf("editing back must publish the earlier version: %+v", edited)
}
// The row that already described the pasted version is reused, so no second row appears for it.
var rows int64
if err := db.Model(&IngestJob{}).Where("chapter_id = ?", chapterID).Count(&rows).Error; err != nil || rows != 2 {
t.Fatalf("one row per version is expected, got %d (%v)", rows, err)
}
var reused IngestJob
if err := db.Where("chapter_id = ? AND content_sha256 = ?", chapterID, firstSHA).First(&reused).Error; err != nil {
t.Fatal(err)
}
if reused.Status != statusPending || reused.Attempts != 0 || reused.FinishedAt != nil || reused.ErrorReason != "" {
t.Fatalf("a reused job must restart cleanly: %+v", reused)
}
if err := db.Model(&Chapter{}).Where("id = ?", chapterID).Update("status", statusReady).Error; err != nil {
t.Fatal(err)
}
// A -> B again, and this time the B job is the one being reused.
edited = editChapterText(t, r, learner.Token, chapterID, second)
if !edited.VersionChanged || edited.Job == nil {
t.Fatalf("switching between versions must keep working: %+v", edited)
}
if err := db.Model(&IngestJob{}).Where("chapter_id = ?", chapterID).Count(&rows).Error; err != nil || rows != 2 {
t.Fatalf("still one row per version, got %d (%v)", rows, err)
}
// A version whose job had failed is revived when it becomes current again.
if err := db.Model(&IngestJob{}).Where("chapter_id = ? AND content_sha256 = ?", chapterID, secondSHA).
Updates(map[string]any{"status": statusFailed, "attempts": maxIngestAttempts, "error_reason": reasonAttemptsExhausted}).Error; err != nil {
t.Fatal(err)
}
editChapterText(t, r, learner.Token, chapterID, first)
edited = editChapterText(t, r, learner.Token, chapterID, second)
if edited.Job == nil || edited.Job.Attempts != 0 || edited.Job.Status != statusPending {
t.Fatalf("a failed version must be revived when it becomes current again: %+v", edited.Job)
}
// The pipeline finishes the reused job and the chapter holds the last text.
drainIngest(t, db)
var chapter Chapter
if err := db.Where("id = ?", chapterID).First(&chapter).Error; err != nil {
t.Fatal(err)
}
if chapter.Status != statusReady || chapter.ContentSHA256 != secondSHA || chapter.OriginalText != second {
t.Fatalf("the chapter must hold the last edited text: %+v", chapter)
}
code, _, data := callRaw(t, r, "GET", fmt.Sprintf("/api/v1/chapters/%d", chapterID), learner.Token, nil)
if code != 200 {
t.Fatalf("reader %d", code)
}
var payload struct {
Chapter ChapterView
}
if err := json.Unmarshal(data, &payload); err != nil {
t.Fatal(err)
}
if payload.Chapter.OriginalText != second {
t.Fatalf("the reader must serve the last text: %q", payload.Chapter.OriginalText)
}
// An unchanged text is still not a new version.
code, _, _ = callRaw(t, r, "PATCH", fmt.Sprintf("/api/v1/chapters/%d", chapterID), learner.Token, map[string]string{"text": second})
if code != 200 {
t.Fatalf("saving the same text must succeed, got %d", code)
}
if err := db.Model(&IngestJob{}).Where("chapter_id = ?", chapterID).Count(&rows).Error; err != nil || rows != 2 {
t.Fatalf("an unchanged text must not add a job, got %d (%v)", rows, err)
}
}
+56 -1
View File
@@ -163,6 +163,55 @@ func TestMySQLLoginUsesStoredPasswordWithoutChangingCreationPolicy(t *testing.T)
}
}
// TestMySQLSessionLifetimeIsThirtyDays pins the login lifetime that both clients share. It checks
// the API response, the stored row and the privacy rule (only a digest is persisted) so changing the
// value without updating this test is impossible.
func TestMySQLSessionLifetimeIsThirtyDays(t *testing.T) {
if SessionLifetime != 30*24*time.Hour {
t.Fatalf("session lifetime %s, want 30 days", SessionLifetime)
}
db := testDB(t)
u := admin.SysUser{Username: randomName("sesslife"), Password: fixturePassword, RoleId: 2, Status: "2"}
if err := db.Create(&u).Error; err != nil {
t.Fatal(err)
}
r := Router(db, time.Now)
before := time.Now()
code, data := callAPI(t, r, "POST", "/api/v1/login", "", map[string]string{"username": u.Username, "password": fixturePassword})
if code != 200 {
t.Fatalf("login status %d", code)
}
var login struct {
Token string `json:"token"`
ExpiresAt time.Time `json:"expiresAt"`
}
if err := json.Unmarshal(data, &login); err != nil {
t.Fatal(err)
}
granted := login.ExpiresAt.Sub(before)
if granted < SessionLifetime-time.Minute || granted > SessionLifetime+time.Minute {
t.Fatalf("granted %s, want about %s", granted, SessionLifetime)
}
var stored Session
if err := db.Where("token_hash = ?", digest(login.Token)).First(&stored).Error; err != nil {
t.Fatal("session row missing")
}
if stored.ExpiresAt.Sub(stored.ExpiresAt.Truncate(time.Millisecond)) != 0 {
t.Fatal("stored expiry lost precision")
}
if delta := login.ExpiresAt.Sub(stored.ExpiresAt); delta > time.Millisecond || delta < -time.Millisecond {
t.Fatalf("stored expiry %s differs from the response by %s", stored.ExpiresAt, delta)
}
var raw int64
db.Table("lexgo_sessions").Where("token_hash = ?", login.Token).Count(&raw)
if raw != 0 {
t.Fatal("raw token persisted")
}
if code, _ = callAPI(t, r, "GET", "/api/v1/me", login.Token, nil); code != 200 {
t.Fatalf("fresh session rejected: %d", code)
}
}
func TestMySQLAccountIsolationAndRevocation(t *testing.T) {
db := testDB(t)
if err := Migrate(db); err != nil {
@@ -281,7 +330,13 @@ func TestMySQLAccountIsolationAndRevocation(t *testing.T) {
t.Fatal("old password accepted")
}
loginToken(t, r, users[0], "replacement-fixture-pass")
clock = clock.Add(9 * time.Hour)
// The lifetime is absolute, so a session is still valid just before it and gone right after;
// asserting against SessionLifetime keeps this test honest when the value changes.
clock = clock.Add(SessionLifetime - time.Hour)
if code, _ = callAPI(t, r, "GET", "/api/v1/me", tokenB, nil); code != 200 {
t.Fatal("session must still be valid before its lifetime ends", code)
}
clock = clock.Add(2 * time.Hour)
code, _ = callAPI(t, r, "GET", "/api/v1/me", tokenB, nil)
if code != 401 {
t.Fatal("expired session valid")
+6 -1
View File
@@ -38,6 +38,11 @@ type Space struct {
func (Space) TableName() string { return "lexgo_spaces" }
// SessionLifetime is how long a login lasts. It is absolute, not sliding: using the app does not
// extend it, so a stolen token cannot be kept alive forever. Both the learner and the admin client
// authenticate through the same login endpoint, so one value covers them.
const SessionLifetime = 30 * 24 * time.Hour
type Session struct {
TokenHash string `gorm:"primaryKey"`
OwnerID int
@@ -162,7 +167,7 @@ func login(db *gorm.DB, now time.Time, name, password string) (LoginResult, erro
return err
}
token := hex.EncodeToString(b)
expiry := now.Add(8 * time.Hour)
expiry := now.Add(SessionLifetime)
if err = tx.Where("owner_id = ? AND expires_at <= ?", u.UserId, now).Delete(&Session{}).Error; err != nil {
return err
}