[modusensus]

做什么

#622 的可观测性补丁。parsePeakSpec 的失败语义（整串视为未配置）不动——排程是省钱手段，绝不因为写错格式把蒸馏/巩固停掉；补的是可见性：配置写错的唯一症状从「功能悄悄没生效」（#424 的排查只能靠『高峰期还在巩固』倒推）变成启动日志一条 warn + 面板输入行下一条红色提示。

实现

• 启动告警（src/index.js）：cfg 装配后对 summarizePeakHours / dreamPeakHours 各跑一次解析——非空且解析为 null 时 logger.warn（键名 + 原串 + 正确写法示例）。空串是合法的未配置，不告警（验收第 2 条）；行为零变化，只 log。
• 面板提示（src/api.js + lib/client.js）：GET/PUT /features 答复新增 invalidPeakHours——校验在服务端用同一份 parsePeakSpec 算好，面板只渲染（独立 bundle 复制解析器必然漂移；与 config-keys 卡「分类在服务端算好」同一纪律）。PUT 同走 featureSnapshot，保存后提示立即跟着换。
• 单一口径（src/peak-hours.js）：新增 PEAK_SPEC_KEYS + invalidPeakSpecs(cfg)。helper 只转述解析器：等价类归一（PR fix(peak-hours): 全角等价符号归一，错峰串不再静默失效（#424） #621，评审中）落地后救得了的写法自然不再告警，两批无需联动（本 PR 基线未含 fix(peak-hours): 全角等价符号归一，错峰串不再静默失效（#424） #621，全角串当前确实解析失败，告警是当前真话）。
• 不做的（issue 原文口径）：不改解析语义、不加配置键、不动 [Feature] 巩固/蒸馏的维护窗口：正向定点 + 债务上限 + 手动触发入口 #424 的排程增强五项。

验证

• npm test：3159 / 3158 通过 / 0 失败 / 1 跳过；check-sync 通过（118 文件）。
• 新增 test/peak-hours-observability.test.js 3 条（helper 只报「非空且解析失败」+ 全角现状断言 + 接线锁：告警读装配后的 cfg、不改写 cfg、排在 sessionGate 之前）；test/api.test.js +1（GET/PUT 答复字段与 helper 同源，合法/空串不报）；test/client.test.js +1（提示行走服务端字段、反拷贝锁匹配调用形态、i18n 双语键成对）。
• docs：CONFIGURATION.md 两个键行补可见性说明（全键闸通过）+ CHANGELOG。
• euthyna 审计已过：audit + history --pickaxe 两次测量，2 条删除行事实逐条阶段 C 裁定 0 TRUE POSITIVE / 2 FALSE POSITIVE（文档行就地扩展、pendingRestart 逐字保留）；euthyna gate 2/2 通过（exit 0）。报告在车道本地不入库。

Closes #622

[coderabbitai]

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 51 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used all 2 included reviews currently available.

Learn how review limits work.

Review configuration:

Run configuration

• Source mode: Auto
• Repository: slow-stack/mneme
• Branch: feat/622-peak-hours-observability
• File: .coderabbit.yaml
• Commit: 45a1325
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: f15b2fc8-11d2-49f7-a96d-3626de74c8fd

Commits

Reviewing files that changed from the base of the PR and between 514d238 and 45a1325.

Files selected for processing: 12

• dsh-mneme/CHANGELOG.md
• dsh-mneme/docs/CONFIGURATION.md
• dsh-mneme/lib/api.js
• dsh-mneme/lib/client.js
• dsh-mneme/lib/index.js
• dsh-mneme/lib/peak-hours.js
• dsh-mneme/src/api.js
• dsh-mneme/src/index.js
• dsh-mneme/src/peak-hours.js
• dsh-mneme/test/api.test.js
• dsh-mneme/test/client.test.js
• dsh-mneme/test/peak-hours-observability.test.js

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

• X
• Mastodon
• Reddit
• LinkedIn

[Copilot]

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

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.

[codecov]

Codecov Report

❌ Patch coverage is 91.37931% with 5 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
dsh-mneme/src/index.js	68.75%	5 Missing ⚠️

📢 Thoughts on this report? Let us know!