修复 harness 中 memory glob 路径归一化问题#1481
Open
guslegend0510 wants to merge 1 commit into
Open
Conversation
|
|
Codecov Report❌ Patch coverage is 📢 Thoughts on this report? Let us know! |
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
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.
概述
修复
MemoryConsolidator在处理memory/*.md时对路径归一化不一致的问题,避免本地文件系统返回绝对路径或 Windows 路径时,根目录下的日记文件被漏读、漏合并。变更内容
WorkspaceManager.toWorkspaceRelativePath(...),统一把文件系统返回的路径转换为 workspace 相对路径。MemoryConsolidator在读取 daily ledger 时,先将glob结果归一化,再进行过滤、排序和读取。fileName(...),兼容 Windows 路径分隔符。LocalFilesystem下根目录memory/2026-05-20.md的合并场景。验证
mvn -s codex-maven-settings.xml -pl agentscope-harness -am "-Dtest=MemoryConsolidatorFilesystemTest,FilesystemGlobTest" test