fix(tool): correct sessionId parameter description#2195
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
AgentScopeJavaBot
left a comment
There was a problem hiding this comment.
🤖 AI Review
This is a clean, minimal documentation fix that corrects a misleading @ToolParam description for the sessionId parameter in SessionSearchTool.sessionHistory(). The old description "AgentStateStore ID" was inaccurate — the parameter is semantically a session identifier, not an "AgentStateStore" identifier, as confirmed by the parameter name, usage context (workspaceManager.resolveSessionContextFile), and the surrounding code. The fix to "Session ID" is correct and consistent.
(inline comments could not be attached — line numbers fell outside PR hunks. See archived report.)
AgentScopeJavaBot
left a comment
There was a problem hiding this comment.
🤖 AI Review
This is a clean, minimal documentation fix that corrects a misleading @ToolParam description for the sessionId parameter in SessionSearchTool.sessionHistory(). The old description "AgentStateStore ID" was inaccurate — the parameter is semantically a session identifier, not an "AgentStateStore" identifier, as confirmed by the parameter name, usage context (workspaceManager.resolveSessionContextFile), and the surrounding code. The fix to "Session ID" is correct and consistent.
(inline comments could not be attached — line numbers fell outside PR hunks. See archived report.)
Summary
SessionSearchTool.sessionHistory():"AgentStateStore ID"→"Session ID"Test plan
mvn compile— passed