Skip to content

fix: The issue of the file name being stored as part of the memory as well #1222

Open
whipser030 wants to merge 2 commits intoMemTensor:dev-20260309-v2.0.9from
whipser030:dev-20260309-v2.0.9_test
Open

fix: The issue of the file name being stored as part of the memory as well #1222
whipser030 wants to merge 2 commits intoMemTensor:dev-20260309-v2.0.9from
whipser030:dev-20260309-v2.0.9_test

Conversation

@whipser030
Copy link
Contributor

@whipser030 whipser030 commented Mar 13, 2026

Description

Please include a summary of the change, the problem it solves, the implementation approach, and relevant context. List any dependencies required for this change.

Related Issue (Required): Fixes #1221 #1223

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Refactor (does not change functionality, e.g. code style improvements, linting)
  • Documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Unit Test
  • Test Script Or Test Steps (please provide)

#1221
curl --location --request POST 'http://127.0.0.1:8002/product/add'
--header 'Content-Type: application/json'
--data-raw '{
"user_id": "test_0313",
"writable_cube_ids": ["test_0313_cube"],
"async_mode": "async",
"messages": [
{
"file": {
"file_data": "https://cdn.memtensor.com.cn/file/1773129771959_4lbzs9.txt",
"file_id": "0313file",
"filename": "0313file.txt"
},
"type": "file"
}
],
"allow_public":"false"
}'
After the above file is uploaded, the memory addition operation only records the file content, but does not record the "uploading of XX file" action.

#1223
curl --location --request POST 'http://127.0.0.1:8002/product/add'
--header 'Content-Type: application/json'
--data-raw '{
"user_id": "test_0314",
"writable_cube_ids": [
"test_0314_cube"
],
"async_mode": "async",
"messages": [
{
"role": "user",
"content": [
{
"type": "text",
"text": "本次 Beta 上线以“验证核心价值”为首要目标,阶段性关注用户真实使用场景与反馈质量,而非用户规模增长。上线后两周内将重点跟踪留存率、核心功能使用频次及关键负反馈类型,并在每周例会上进行数据复盘。如核心指标未达预期,将优先调整产品体验与定位策略,而非加大推广投入。"
},
{
"type": "file",
"file": {
"file_data": "https://cdn.memtensor.com.cn/file/1772452920110_z9w5j6.docx"
}
}
]
},
{
"role": "assistant",
"content": "好的,需要我为您解答吗?"
}
],
"allow_public": "false"
}'

After the asynchronous process has completed, by querying the memory, it can be found that the document information in the content list has also been stored as memory.

  • Pipeline Automated API Test (please provide)

Checklist

  • I have performed a self-review of my own code | 我已自行检查了自己的代码
  • I have commented my code in hard-to-understand areas | 我已在难以理解的地方对代码进行了注释
  • I have added tests that prove my fix is effective or that my feature works | 我已添加测试以证明我的修复有效或功能正常
  • I have created related documentation issue/PR in MemOS-Docs (if applicable) | 我已在 MemOS-Docs 中创建了相关的文档 issue/PR(如果适用)
  • I have linked the issue to this PR (if applicable) | 我已将 issue 链接到此 PR(如果适用)
  • I have mentioned the person who will review this PR | 我已提及将审查此 PR 的人

Reviewer Checklist

  • closes #xxxx (Replace xxxx with the GitHub issue number)
  • Made sure Checks passed
  • Tests have been provided

@whipser030 whipser030 changed the title fix: fix: The issue of the file name being stored as part of the memory as well Mar 13, 2026
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.

1 participant