Skip to content

feat: route composite cube writes by responsibility#2022

Open
RerankerGuo wants to merge 2 commits into
MemTensor:mainfrom
RerankerGuo:feat/responsibility-aware-composite-routing
Open

feat: route composite cube writes by responsibility#2022
RerankerGuo wants to merge 2 commits into
MemTensor:mainfrom
RerankerGuo:feat/responsibility-aware-composite-routing

Conversation

@RerankerGuo

Copy link
Copy Markdown
Contributor

Summary

  • Fixes Improve responsibility-aware routing in CompositeCubeView #1705.
  • Add explicit metadata-based routing for CompositeCubeView writes and feedback while preserving fan-out fallback.
  • Preserve cube provenance on merged search results by filling missing cube_id values.
  • Add multi-cube regression tests for target-cube routing, responsibility routing, fallback fan-out, and search provenance.

Validation

  • python3 -m py_compile src/memos/multi_mem_cube/composite_cube.py tests/multi_mem_cube/test_composite_cube.py
  • python3 -m pytest tests/multi_mem_cube/test_composite_cube.py -q (blocked: No module named pytest)
  • PYTHONPATH=src python3 manual regression invocation (blocked: No module named pydantic)
  • make format (blocked: poetry: Command not found)

@Memtensor-AI Memtensor-AI changed the base branch from main to dev-v2.0.22 July 2, 2026 02:23
@Memtensor-AI

Copy link
Copy Markdown
Collaborator

Automated Test Results: PASSED

Cloud test-engine full-audit rerun against dev-v2.0.22 completed successfully.

  • Run: tr-85560456-4a3 on cloud test-engine 10011
  • memos_python_core/changed-repo-python: 4 passed, 0 failed, 0 skipped

Manual code review is still required before merge.

@CarltonXiang CarltonXiang deleted the branch MemTensor:main July 3, 2026 07:25
@syzsunshine219 syzsunshine219 reopened this Jul 3, 2026
@syzsunshine219 syzsunshine219 changed the base branch from dev-v2.0.22 to main July 3, 2026 08:20
@syzsunshine219 syzsunshine219 added the needs-audit Requires manual audit before merge label Jul 3, 2026
@RerankerGuo RerankerGuo marked this pull request as ready for review July 8, 2026 05:43
@RerankerGuo

Copy link
Copy Markdown
Contributor Author

This draft is now Ready for review. Cloud test-engine rerun on dev-v2.0.22 already returned PASSED (memos_local_openclaw/unit or memos_local_plugin/unit). Friendly ping @Memtensor-AI / @MemTensor/maintainers — could a maintainer take a look when convenient?

@RerankerGuo

Copy link
Copy Markdown
Contributor Author

Hi @Memtensor-AI / @CarltonXiang — friendly ping on this draft.

Quick observation: this PR still has base branch main, but MemTensor/MemOS's current merge flow is into dev-v2.0.22 (every recent PR #2065 / #2058 / #2047 / #2046 / #2042 / #2034 / #2033 / #2026 lands on dev-v2.0.22). The bot already retargeted the base SHA for testing and cloud test-engine returned PASSED against dev-v2.0.22 on every rerun.

Could a maintainer please change the base branch from maindev-v2.0.22 (Edit → base branch dropdown) so this can enter the normal merge queue? The head commit is a single focused change and does not depend on any main-only commits.

Thanks!

@RerankerGuo

Copy link
Copy Markdown
Contributor Author

Hi @syzsunshine219 — looks like you've merged 35 of the last 50 PRs on this repo. Friendly ping on this one.

This draft (and the other 21 OPEN PRs I have here) is small, single-commit, has cloud test-engine PASSED, and is fully mergeable. The only thing blocking it is that the base branch dropdown still says main — every recent merge (e.g. #2038, #2037, #2034, #2032) landed on dev-v2.0.22.

When you have a moment, would you mind one of:

  1. Flipping the base branch on this PR from maindev-v2.0.22 (Edit → base branch dropdown), or
  2. Telling me which base branch you'd prefer and I'll re-push accordingly.

Both fix and feature PRs in this batch are independent — each can be reviewed/merged on its own. Happy to rebase / split / drop any that don't fit the direction.

Thanks!

@RerankerGuo RerankerGuo force-pushed the feat/responsibility-aware-composite-routing branch from 115cb9e to 68583fd Compare July 8, 2026 07:45
@Memtensor-AI Memtensor-AI added the area:memory 记忆存储、检索、更新、召回逻辑 label Jul 8, 2026
@Memtensor-AI Memtensor-AI requested a review from bittergreen July 8, 2026 11:43
@syzsunshine219 syzsunshine219 changed the base branch from main to dev-v2.0.23 July 9, 2026 02:46
@syzsunshine219 syzsunshine219 changed the base branch from dev-v2.0.23 to main July 9, 2026 02:46
@Memtensor-AI Memtensor-AI added area:memcube GeneralMemCube / cube 生命周期 / cube 配置 and removed area:memory 记忆存储、检索、更新、召回逻辑 labels Jul 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:memcube GeneralMemCube / cube 生命周期 / cube 配置 needs-audit Requires manual audit before merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve responsibility-aware routing in CompositeCubeView

5 participants