Skip to content

docs(readme): llmAudit 配置表补齐全部触发源与 operation_type(#286 遗留) - #293

Merged
modusensus merged 2 commits into
slow-stack:mainfrom
z2Ace0107:codex/fix-286-readme-llmaudit-config-row
Sep 23, 2026
Merged

modusensus merged 2 commits into
slow-stack:mainfrom
z2Ace0107:codex/fix-286-readme-llmaudit-config-row

Conversation

@z2Ace0107

@z2Ace0107 z2Ace0107 commented Sep 23, 2026 •

Copy link
Copy Markdown
Contributor

修补 #286 的遗漏。

#286 给实体抽取与 Sleep Mode 接上了 llm_audit_logs 记账,并改了 README 的「LLM 消耗审计」专节,但漏了同一份 README 里配置索引表中的 llmAudit 行——那一行至今仍写「每次后台 LLM 调用(autoDream/autoSummarize)」,与专节矛盾。CodeRabbit 在 #286 提了这条 minor,合并时未处理。

改动只有一行,把配置表也同步到实际覆盖面:

  • 四个 trigger_source:autoDream / autoSummarize / sleep / entityExtract
  • 六个 operation_type:dream_consolidate / dream_summarize / dream_narrative、summarize_compress、sleep_conflict / sleep_pattern、entity_extract

dream_narrative 需 dreamNarrativeEnabled(默认关)才跑,与专节口径一致。

验证:npm test 1285 passed / 0 failed,check-sync 平价,git diff --check 干净。无 src/ 改动,不需要 npm run sync。

Refs #286

Summary by CodeRabbit

  • 文档
    • 更新配置说明,补充 llmAudit 覆盖的后台 LLM 调用来源及对应的操作类型,包括睡眠处理和实体提取相关调用。
    • 保留原有说明:审计失败会记录错误但不阻塞流程,相关 API 端点为只读。

slow-stack#286 只改了 README 的「LLM 消耗审计」专节,漏了同一份 README 里配置索引表
中的 llmAudit 行——两处现在互相矛盾。CodeRabbit 在 slow-stack#286 提了这条 minor,
合并时未处理。

顺带把 autoDream 的 operation_type 补全(dream_consolidate /
dream_summarize / dream_narrative,后者需 dreamNarrativeEnabled)。

Refs slow-stack#286
@coderabbitai

coderabbitai Bot commented Sep 23, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Warning

Review limit reached

Next included review available in 5 minutes.

Check out review usage here.

View limit details

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

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

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Repository: slow-stack/dsh-mneme/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: c218a16a-6ba9-4943-9b9e-4f62a7d0275a

📥 Commits

Reviewing files that changed from the base of the PR and between 3c839f4 and 843d1b0.

📒 Files selected for processing (1)
  • dsh-mneme/README.md

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: slow-stack/dsh-mneme/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: fe708feb-3ea0-4e7b-8b05-2077fd29db06

📥 Commits

Reviewing files that changed from the base of the PR and between 7412400 and 3c839f4.

📒 Files selected for processing (1)
  • dsh-mneme/README.md

Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review.


📝 Walkthrough

Walkthrough

README 中的 llmAudit 配置说明将审计覆盖范围扩展至 sleep 和 entityExtract,并列出各调用来源对应的 operation_type。

Changes

llmAudit 配置说明

层 / 文件 摘要
更新审计来源说明
dsh-mneme/README.md
配置说明新增 sleep 和 entityExtract 来源及其对应的 operation_type。失败记 error 不阻塞、只读 API 端点等描述保持不变。

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~2 minutes

Change: Other

Suggested reviewers: modusensus

Merge Risk: ⚪ Minimal · up to 3c839

No actionable merge-blocking risk remains: the README already explains that dream_narrative is disabled by default and runs only when enabled.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 标题准确概括了 README 配置表中补全 llmAudit 触发源和 operation_type 的主要变更。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR

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

Comment @coderabbitai help to get the list of available commands.

@codecov

codecov Bot commented Sep 23, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@modusensus
modusensus merged commit d0d621d into slow-stack:main Sep 23, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants