测试结构：npm run test:e2e 从不被执行，而无测试的恰是仓库自己认定最脆弱的那几个模块

[SongshGeo]

测试套件形状是好的（12 个 vitest 文件、0.4s、pre-push 跑），但它覆盖的和最该覆盖
的不是同一批文件，而 e2e 套件根本没有人跑。

一、npm run test:e2e 在任何地方都不会被执行

package.json 里有：

"test:e2e": "node test/e2e/background-flash.mjs"

• .github/workflows/ci.yml：跑 lint / test / check:content /
  i18n:manifest / setup:content / astro sync / check / build。没有 e2e。
• lefthook.yml：pre-commit 跑 lint + format，pre-push 跑 npm test。没有 e2e。
• vitest.config.ts 的注释明确说 e2e 不归它管，「belongs in an e2e suite」——
  但那个 suite 没有被接到任何地方。

它测的是 #33 / #34 / #36 那一组背景闪烁的回归。也就是说：三个还开着的视觉 bug，
有一个现成的自动化验收脚本，而它只在有人手敲命令时才运行。

要么把它接进 CI（Playwright 已经是 devDependency，需要 npx playwright install chromium 一步），要么就承认它是手动工具，在 test/e2e/README.md 和
package.json 里写清楚它不是 CI 的一部分。现在是第三种状态：看起来像 CI 的一
部分，其实不是。

二、没有测试的，恰好是仓库自己认定最脆弱的那几个

eslint.config.js 的注释点名了本仓最需要盯的代码：「path resolution, remark AST
surgery, recursive deletes」。三样东西对应的文件，全部没有单元测试：

无测试的模块	行数	是什么
src/utils/contentLayout.js	170	path resolution，23 个文件 import 它，全仓被依赖最多
src/utils/obsidian-links.js	275	remark AST surgery，wiki link 解析（曾经全部 404）
src/utils/rehype-callouts.js	147	rehype AST surgery
src/utils/remark-obsidian-callouts.js	94	remark AST surgery
src/utils/remark-revive-directives.js	57	remark AST surgery（19:15 时间戳那个坑）
scripts/setup-content.mjs	751	recursive deletes + 重试 + 复制兜底
src/content/loaders/newsLoader.ts	175	自定义 loader
src/utils/siteConfig.ts	84	站点身份 + 关于页
src/utils/libraryEntries.ts	28
src/utils/pageRelations.ts	49

render-markdown.test.ts 确实会间接走过这四个 markdown 插件，但它的定位是
sanitize 的安全边界测试（文件头注释写得很清楚），不是插件行为测试。插件各自的
边界条件——嵌套 callout、带别名的 wiki link、指向不存在条目的链接——没有直接
断言。

建议

按风险排序补三块，其余顺其自然：

1. contentLayout.js 的 findContentRoot —— 它要在三种 vault 形状里探测，
   是「每个新贡献者第一个撞到的报错」（它自己的注释这么写的）。三种形状各一个
   fixture 目录，三个断言。
2. obsidian-links.js —— 给定一个 index，[[名字]] / [[目录/名字|别名]] /
   指向不存在条目的链接，各断言一次 href。
3. newsLoader.ts —— 已经有 publicationsLoader.test.ts 做样板，照抄结构。

setup-content.mjs 的那块建议不补测试，直接按另一个 issue 删掉（六个软链是
content.config.ts 自己证明不需要的）。

[SongshGeo]

更正：第一部分我说重了

动手时读了 test/e2e/README.md，里面有一整节 “Not in CI, deliberately”，写清了取舍（要一个浏览器镜像 + 一份 served build，CI 时长大约翻倍，为一个测试不值），也点了出处（issue #33 的 PR 讨论）。

所以这不是「看起来像 CI 的一部分，其实不是」——它是一个记录在案的决定。我原文里那句判断是错的，撤回。

真正剩下的小问题只有一个：package.json 里 "test:e2e" 和别的 script 长得一样，而这条判断藏在 test/e2e/README.md 里。已做的两件事：

1. README 的命令表里点明，npm run test:e2e 是 manual, not in CI，并链到 test/e2e/README.md 看原因。
2. 没有服务时给一句人话。之前直接抛 Playwright 的 net::ERR_CONNECTION_REFUSED 堆栈；现在按 exit 2（和文件里既有的 INCONCLUSIVE 同一个语义——服务没起不是断言失败）退出，并打印怎么起服务、端口被占用时怎么传 BASE。

验证过：起 npm run preview 之后 npm run test:e2e 是 PASS（min luma 49.6）。

第二部分（单测缺口）成立，已补

新增 41 个测试，npm test 从 207 → 248，仍是 0.7 秒。

test/contentLayout.test.ts（18 个） —— 三种 vault 形态各一，加上探测优先级（已同步的 .content 必须赢过并排放着的 vault）、哨兵目录（一个空的 .content 不算 vault）、getContentRoot 的报错里必须写着 npm run setup:content，以及 BUNDLE_COLLECTION_DIRS 确实是从 COLLECTION_VAULT_PATHS 派生的。

顺带修了一处：findContentRoot / collectionBase 现在接受 cwd 参数而不是直接读 process.cwd()——不然测三种形态就得 chdir 整个进程。这也修了我在 #45 里引入的一个回归：buildContentIndex(cwd) 的 cwd 参数在 library base 变成绝对路径之后已经不起作用了。

test/obsidianLinks.test.ts（8 个） —— buildContentIndex 本身。目录和大小写要保住（词条/Git 不能变成 git，这正是当初全站 wiki link 404 的原因）、basename 索引、同名冲突要被记下来、publish: false 的条目不进索引、frontmatter 解析失败要保留条目（坏链接可恢复，页面从索引里消失不可恢复）、别的集合的目录要跳过、附件按 basename 索引但保留路径、没有 vault 时降级成空索引而不是崩在 config 加载期。

test/newsLoader.test.ts（15 个） —— 为此把 loader 里的纯逻辑抽成了三个导出函数（newsDateOf / newsBulletsOf / classifyMarkers），load() 剩下的是 Astro adapter。覆盖：发布是严格 opt-in（默认必须是「不」——这是别人的日记）、文件名按 UTC 解析、#P/ 走别名表、解析不到的 #P/ 要被报告而不是丢掉（就是这条当初被注释掉，导致全站 #P/ 无一命中却没人发现）、人名不能被当成 tag、句末标点要截断 marker。

setup-content.mjs 那块按建议没有补测试——#45 已经把它删了。