fix: memory_recall includeFullText now returns L2 content instead of L0

[lamkacheong]

Closes #333

Problem

memory_recall with includeFullText: true was returning L0 (one-line abstract) instead of the full L2 content. This happened because the text column in LanceDB stores the L0 abstract, and the tool read directly from that column for both includeFullText: true and false — making the parameter a no-op.

// Before (broken)
const base = includeFullText
  ? r.entry.text            // ← r.entry.text is L0
  : metadata.l0_abstract || r.entry.text;

Fix

Resolve full content through the L2 → L1 → L0 fallback chain when includeFullText: true:

// After
const base = includeFullText
  ? (metadata.l2_content || metadata.l1_overview || r.entry.text)
  : (metadata.l0_abstract || r.entry.text);

Also add fullText to sanitizeMemoryForSerialization so the details.memories structured payload exposes L2 content for callers that inspect the tool response directly.

Why this matters

When an agent explicitly calls memory_recall, it signals active intent to retrieve memory detail. Auto-recall (passive injection) returning L0 hints is fine — it keeps context concise. But an explicit tool call should return the full narrative, especially for cases and patterns categories where L2 contains actionable step-by-step content that L0 cannot capture in one line.

Changes

• src/tools.ts — 2 hunks, no new dependencies

[AliceLJY]

确认了这个 bug 在 beta.10 上依然存在，tools.ts 里 includeFullText 确实还是返回 L0 的 text。fix 方向正确，保持 text 不变 + 加 fullText 字段，向后兼容。

麻烦 rebase 到最新 master，然后补一个 tool 级回归测试：验证 rendered output 和 details.memories[].fullText 都拿到的是 L2 内容。

rebase + 补测试后可以合 👍

[lamkacheong]

已完成！rebase 到最新 master，并在 test/recall-text-cleanup.test.mjs 补了回归测试。

新增测试（includeFullText=true renders L2 content in output, not L0 abstract）覆盖了你要求的两个验证点：

• 验证 rendered text 行包含 L2 内容，而非 L0 摘要
• 验证 details.memories[0].fullText 等于 L2 内容
• 顺带断言 details.memories[0].text 仍保留 L0，向后兼容

16 个测试全部通过，可以合了 🙏

[AliceLJY]

感谢补测试和 rebase，修复方向没问题。代码层面有两个点需要调整：

1. fullText 暴露范围过大

sanitizeMemoryForSerialization 是共享 helper，memory_forget、memory_update、memory_explain_rank 等工具都在调。把 fullText 加在这里意味着所有工具都会无条件返回 L2 全文，包括 includeFullText=false 的场景——违背了 opt-in 的设计意图，也会让不需要全文的响应体膨胀。

建议：fullText 不要动 sanitizeMemoryForSerialization，改为仅在 memory_recall handler 内、includeFullText=true 时注入到 details.memories 里。

2. 补两个测试

• includeFullText=false 时 details.memories[].fullText 应该是 undefined（验证 opt-in 边界）
• legacy 记忆（metadata 里没有 l2_content / l1_overview）的 fallback 行为——确认最终拿到的是 r.entry.text

这两个改完再看 🙏

[lamkacheong]

两个问题都改好了：

1. fullText 收回到 memory_recall handler 内
sanitizeMemoryForSerialization 已恢复原样，fullText 只在 memory_recall execute handler 里、includeFullText=true 时才注入到 details.memories，不影响其他工具。

2. 补了两个测试

• includeFullText=false 时 details.memories[].fullText 为 undefined（验证 opt-in 边界）
• 没有 smart metadata 的 legacy 记忆，fullText fallback 到 entry.text

18 个测试全部通过 🙏

[AliceLJY]

@rwmjhb 恒哥看一下这个 PR，作者已经改了两轮，我这边 review 过了觉得可以，你确认下能不能合 👀