test(harness): add regression test for memory consolidation glob#1479
test(harness): add regression test for memory consolidation glob#1479guslegend0510 wants to merge 3 commits into
Conversation
|
|
guslegend0510
force-pushed
the
fix/localfilesystem-glob-direct-files
branch
from
2dceb35 to
505d6cb
Compare
|
Thanks for addressing the MemoryConsolidator path issue. I think the same class of bug also affects MemoryMaintenanceHook, which is mentioned in the issue scope. |
AgentScopeJavaBot
added
area/harness
AgentScopeJavaBot
left a comment
AgentScopeJavaBot
left a comment
There was a problem hiding this comment.
🤖 AI Review
This PR fixes a real bug where LocalFilesystem.glob() returns absolute paths that downstream code was treating as relative paths. The fix centralizes path normalization into a new WorkspaceManager.toWorkspaceRelativePath() method and migrates the affected call sites. Two regression tests exercise the end-to-end flow with LocalFilesystem, confirming the glob-returned absolute paths are now correctly relativized.
(inline comments could not be attached — line numbers fell outside PR hunks. See archived report.)
5 participants
AgentScope-Java Version
1.1.0-SNAPSHOT
Description
本 PR 新增一个回归测试，覆盖
LocalFilesystem.glob("**/*.md")在memory/目录直下文件场景的匹配行为。之前这个模式可能漏掉目录直接子文件，导致
MemoryConsolidator在汇总memory/下的日报时读不到新增内容，进而影响MEMORY.md的正常更新。这次补上测试后，可以验证：
memory/目录直下的.md文件能够被正确扫描到；MemoryConsolidator能正常读取日报并完成合并；关联 issue：#1458
Checklist
mvn spotless:apply格式化代码mvn test