Skip to content

docs: 记忆闭环示意图(浅/深两版)+ 界面截图移入 docs/PANEL.md - #691

Merged
modusensus merged 6 commits into
mainfrom
docs/panel-gallery
Nov 13, 2026
Merged

modusensus merged 6 commits into
mainfrom
docs/panel-gallery

Conversation

@modusensus

@modusensus modusensus commented Nov 13, 2026 •

Copy link
Copy Markdown
Member

做了什么

三件互相配套的事,都在文档层,不动代码:

  1. 界面截图搬进 docs/PANEL.md(12bf11b1):12 张截图从仓库根 images/ 移到 docs/images/,README 的「面板」段压成一句 + 一个链接。README 首屏不再是一堵截图画廊,版面留给含义。
  2. 新增记忆闭环示意图(d18118c5):docs/memory-loop.svg 与 docs/memory-loop-dark.svg,纯手写 SVG(各 15.6 KB,1600×980),README:33–40 用 <picture> + prefers-color-scheme 挂两版。图回答的是「记住之后发生了什么」,不是「面板长什么样」:写入 → 质量闸 → 原子记忆 → SQLite 正本 / Markdown 镜像(双箭头标「手改优先」)→ 空闲巩固(autoDream / Sleep,批注「取代而非覆盖,判错能回退」)→ 四路召回 → 融合 → 注入配额 → 三个出口,外框虚线写 no server。
    • 图里不写任何数量(没有「4 路」「10 个工具」),版本迭代不用重画。
    • 不画还没接的机制。
  3. 修图中文字体回退 + npm 页补链接(b4778254 / 8766294c):正本/镜像 原与英文共用一个 mono 栈,mono 无中文字形会回退宋体,16px/600 下「镜」糊成一片;改为中文走无衬线、英文用 <tspan> 保留等宽。npm 页渲染的是 dsh-mneme/README.md,根 docs/ 不在 tarball 内,相对路径必然 404,故用绝对 raw URL 挂一行(与那份 README 第 2 行 banner 同一写法)。

验证

  • 两版 SVG 用无头 Edge 真实渲染(--headless=new --window-size=1600,980)逐处看,400% 放大核过 正本 SQLite / 镜像 Markdown / 记忆库(本机) / pin 排块头不占额,深浅两版字面一致。
  • test/readme-claims.test.js 3 pass(根 README 的 ## 隐私承诺 / ## Privacy 精确标题未破)。
  • 无代码路径改动,src/lib 未动。

已知取舍

  • npm 那句链接指向 main,合并前打不开;图也不随包走,靠 GitHub 托管。
  • images/ 里另有 4 张没人引用的旧截图(settings/help/features 的英文版),未删,留待定。

Summary by CodeRabbit

  • 文档
    • README 新增记忆闭环示意图及说明,展示记忆写入、存储、巩固与召回等流程。
    • 新增中英文面板文档页面,提供记忆浏览、实体与关系图谱、状态与审计的截图及说明。
    • README 的面板章节改为功能介绍和文档链接,替代原有截图;英文版也新增记忆闭环图的文档链接。

面板每改一次就要重截一遍,README 首屏跟着一起过期。截图搬到 docs/PANEL.md
单独维护,README 只留一处入口与说明;图片路径随之从 images/ 移到 docs/images/。
Copilot AI lite review requested due to automatic review settings November 13, 2026 07:59

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@coderabbitai

coderabbitai Bot commented Nov 13, 2026 •

Copy link
Copy Markdown

Explain Change →

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 31 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

Commits

Reviewing files that changed from the base of the PR and between 8766294 and 2e14e18.

Files ignored due to path filters: 13

  • docs/images/screenshot-entities-en.png is excluded by !**/*.png
  • docs/images/screenshot-entities.png is excluded by !**/*.png
  • docs/images/screenshot-features-all-settings.png is excluded by !**/*.png
  • docs/images/screenshot-features-first-run.png is excluded by !**/*.png
  • docs/images/screenshot-help-en.png is excluded by !**/*.png
  • docs/images/screenshot-help.png is excluded by !**/*.png
  • docs/images/screenshot-memories-en.png is excluded by !**/*.png
  • docs/images/screenshot-memories.png is excluded by !**/*.png
  • docs/images/screenshot-settings-en.png is excluded by !**/*.png
  • docs/images/screenshot-settings.png is excluded by !**/*.png
  • docs/images/screenshot-status-en.png is excluded by !**/*.png
  • docs/images/screenshot-status.png is excluded by !**/*.png
  • docs/memory-loop.svg is excluded by !**/*.svg
Files selected for processing: 3

  • README.md
  • docs/PANEL.md
  • dsh-mneme/README.md

No actionable comments were generated in the recent review.

Walkthrough

README 新增记忆闭环图及说明,并将面板截图介绍改为功能概述和文档链接。新增中英文面板截图文档,介绍记忆浏览、实体与关系图谱、状态与审计。

Changes: 项目文档

层 / 文件 摘要
记忆闭环图与说明
README.md, dsh-mneme/README.md
README 新增记忆闭环图说明及英文版图示链接;dsh-mneme/README.md 新增记忆闭环 SVG。
面板截图文档与 README 链接
docs/PANEL.md, README.md
新增中英文面板截图文档,并将 README 中的面板截图介绍替换为功能概述和文档链接。

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

Change: Other

Suggested reviewers: anans-amarre

Merge Risk: Minimal · previous review 87662

This change reorganizes documentation and adds diagrams, so it carries no behavioral risk. It can be merged after normal checks.

Pre-merge checks: Passed 4

Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed 标题「docs: 记忆闭环示意图(浅/深两版)+ 界面截图移入 docs/PANEL.md」与变更内容相符。变更包括新增浅色与深色两版记忆闭环图、将截图移入 docs/images/、新增 docs/PANEL.md,并将 README 的截图区替换为说明与链接。标题涵盖两项主要变更,格式符合 Conventional Commits 的 docs 前缀。
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)

  • Commit to this branch
  • Create a new PR
  • 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

Recent review info

Run configuration

Commits

Reviewing files that changed from the base of the PR and between 9beec0a and 8766294.

Files ignored due to path filters: 14

  • docs/images/screenshot-entities-en.png is excluded by !**/*.png
  • docs/images/screenshot-entities.png is excluded by !**/*.png
  • docs/images/screenshot-features-all-settings.png is excluded by !**/*.png
  • docs/images/screenshot-features-first-run.png is excluded by !**/*.png
  • docs/images/screenshot-help-en.png is excluded by !**/*.png
  • docs/images/screenshot-help.png is excluded by !**/*.png
  • docs/images/screenshot-memories-en.png is excluded by !**/*.png
  • docs/images/screenshot-memories.png is excluded by !**/*.png
  • docs/images/screenshot-settings-en.png is excluded by !**/*.png
  • docs/images/screenshot-settings.png is excluded by !**/*.png
  • docs/images/screenshot-status-en.png is excluded by !**/*.png
  • docs/images/screenshot-status.png is excluded by !**/*.png
  • docs/memory-loop-dark.svg is excluded by !**/*.svg
  • docs/memory-loop.svg is excluded by !**/*.svg
Files selected for processing: 3

  • README.md
  • docs/PANEL.md
  • dsh-mneme/README.md

Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 0 remain after this review.

@codecov

codecov Bot commented Nov 13, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

同一张图存两份没必要:深色版几何完全相同、只是配色不同,维护要改两处。
README 改成单张 <img> 直接引 docs/memory-loop.svg,英文段同步去掉 dark 链接。
Copilot AI lite review requested due to automatic review settings November 13, 2026 08:12

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Copilot AI lite review requested due to automatic review settings November 13, 2026 08:17

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@modusensus
modusensus merged commit ce38fac into main Nov 13, 2026
13 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