test(log): cover search result summary redaction

[WeiminLee]

Summary

• add regression coverage for search result summary logging so memory payloads, preference notes, embeddings, and secrets are not rendered
• keep the replacement for Fix #2103: fix: Search_memories在默认dedup=mmr下调试用 info 日志打印完整 embedding 向量 #2110 scoped to fix: Search_memories在默认dedup=mmr下调试用 info 日志打印完整 embedding 向量 #2103, without the unrelated API/client/local-plugin changes
• rely on the existing dev-v2.0.29 behavior that logs summarize_search_results(memories_result) instead of serializing result payloads

Refs #2103.
Supersedes #2110.

Verification

• make format: failed because poetry is not installed in this environment
• uv run --with ruff ruff check --fix tests/test_log.py: All checks passed
• uv run --with ruff ruff format tests/test_log.py: 1 file left unchanged
• uv run --with pytest pytest tests/test_log.py -q: 8 passed, 1 warning

[Memtensor-AI]

🤖 Open Code Review

Target: PR #2223
Task: d401551f78cdcd5e
Base: dev-v2.0.29
Head: fix/2103-search-summary

🔍 OpenCodeReview found 1 issue(s) in this PR.

---

1. tests/test_log.py (L161-L163)

The assertions assert "embedding" not in rendered, assert "0.123456" not in rendered, and assert "do-not-log" not in rendered are vacuously true and provide no meaningful test coverage. The summarize_search_results function returns a fixed-shape dict ({"bucket_counts": ..., "item_counts": ..., "total_items": ...}) that never contains the input keys or values, so these assertions would pass even if the function were completely broken and leaked payload. To actually validate that private payload is excluded from the logged output, the test should assert on the absence of the raw input dict's string representation, or use a mock/spy to verify the argument passed to the logger. Alternatively, if a buggy implementation were introduced that passed results directly to the logger instead of log.summarize_search_results(results), the assertion assert "private memory payload" not in rendered and assert "private preference note" not in rendered would catch it, but the "embedding", "0.123456", and "do-not-log" assertions add nothing on top of those.

💡 Suggested Change

Before:

    assert "embedding" not in rendered
    assert "0.123456" not in rendered
    assert "do-not-log" not in rendered

After:

    # These values are private inputs that must not appear in the logged summary.
    # The assertions below already cover the key privacy guarantees:
    assert "private memory payload" not in rendered
    assert "private preference note" not in rendered
    # Note: "embedding", "0.123456", "do-not-log" would never appear in the summary
    # dict regardless of input, so those assertions are redundant and not meaningful.

Generated by cloud-assistant via Open Code Review.

[Memtensor-AI]

✅ Automated Test Results: PASSED

All tests passed (8/8 executed). memos_python_core/changed-repo-python: 8/8. Duration: 5s [advisory, non-gating] AI-generated tests on branch test/auto-gen-d401551f78cdcd5e-20260805163841: 43/43 passed — these do NOT affect the PR verdict; review the branch manually.

Branch: fix/2103-search-summary

[github-actions]

This PR has been automatically marked as stale due to inactivity.