fix(service): degrade readonly recall probe against mnemon CLI 0.2.6 OOM regression - #146
Closed
HHHEEEWWW wants to merge 2 commits into
Closed
fix(service): degrade readonly recall probe against mnemon CLI 0.2.6 OOM regression#146HHHEEEWWW wants to merge 2 commits into
HHHEEEWWW wants to merge 2 commits into
Conversation
…OOM regression --readonly recall fails on every query with SQLite 'out of memory (1)' on mnemon CLI 0.2.6 (mnemon-dev/mnemon#123), making memory-body browse and graph detail panels report 'connection unavailable' in DSH. Keep the readonly snapshot as the preferred probe contract; when the CLI reports the known failure, retry once without the snapshot so browse and metadata probes keep working on affected CLI releases. The readonly path is used again automatically once an upstream fix lands. Adds a regression test asserting the first attempt still uses --readonly and the degraded retry drops it.
5 tasks
Collaborator
|
感谢提供复现和补丁。该问题已在 mnemon-dev/mnemon#125 修复，并随 v0.2.7 发布。 移除 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
摘要 / Summary
DSH 记忆体详情页在安装 mnemon CLI v0.2.6 时显示"连接不可用"（真实关系图 / 可枚举内容面板）。根因是 v0.2.6 的
--readonly recall在任意查询下触发 SQLiteout of memory (1)（与库大小无关，Windows amd64 实测必现，上游：mnemon-dev/mnemon#123）。本 PR 保留
--readonly快照作为首选探测契约（不增加访问计数），仅在 CLI 报告该已知缺陷时降级重试一次（无快照）；上游修复后自动恢复 readonly 路径，无需配置或版本钉死。关联 Issue 或背景 / Related Issue or Context
--readonly recall在任何查询/limit/库大小下失败于SQL logic error: out of memory (1)；去掉--readonly后同一命令立即成功。allNativeInsights/nativeMetadataSample/ graph 元数据）在受影响 CLI 上全部失败。涉及区域 / Affected Areas
PR 类型 / PR Type
最新代码确认 / Latest Codebase Confirmation
main分支开发，或在提交前已 rebase 或合并最新main。 / I developed from the latestmain, or rebased or merged the latestmainbefore submitting.同步命令 / Sync command：fork 与分支均基于最新
main（v0.4.3，aa446cf）创建，无分叉；git fetch origin && git rebase origin/main结果：Already up to date.AI 编码披露 / AI Coding Disclosure
使用的 AI 模型 / AI model used：deepseek-v4-flash
使用的编码 Agent 工具 / Coding Agent tool used：DeepSeek Harness（DSH）桌面 Agent
仓库规范检查 / Repository Rules
兼容性与数据安全 / Compatibility and Data Safety
N/A —— 本次变更仅在
--readonly探测命中已知 CLI OOM 错误特征时降级重试一次；成功路径（含 readonly 快照契约）与所有写入/数据路径完全不变，无数据安全影响。本地验证 / Local Validation
执行的命令 / Commands run：
pnpm build；pnpm vitest run tests/service.spec.ts -t readonly；pnpm vitest run tests/service.spec.ts结果摘要 / Result summary：build OK；readonly 相关 3/3 通过（含新增降级回归测试，首次调用仍带
--readonly）；service 全套 39/40 —— 唯一失败为 Windows 平台 stat-mode 差异（draft 权限位 438 vs 384）的既有问题，与本次改动无关（见 PR 内说明）。用户可见变更证据 / Local Feature Evidence
Fork 分支上的图片原址：https://github.com/HHHEEEWWW/dsh-mnemon/raw/fix/readonly-recall-oom-degrade/pr-assets/readonly-oom-evidence.png
图片内容：修复前后对比 —— BEFORE（readonly 探测 OOM、面板"连接不可用"）与 AFTER（降级重试返回记忆、readonly 保持首选），及本地验证摘要。