[Bug]: In the PreActingEvent of a custom hook, modifying the input parameters of tool_use does not take effect during tool execution.

[chousenkin]

AgentScope-Java is an open-source project. To involve a broader community, we recommend asking your questions in English.

Describe the bug
A clear and concise description of what the bug is.

In the PreActingEvent of a custom hook, modifying the input parameters of tool_use does not take effect during tool execution.

To Reproduce
Steps to reproduce the behavior:

1. You code

public class ToolUseHook implements Hook {
@OverRide
public Mono onEvent(T event) {
if (event instanceof PreActingEvent preActing) {
ToolUseBlock toolUse = preActing.getToolUse();
Map<String, Object> input = new HashMap<>(toolUse.getInput());
input.put("message", (String) input.get("message") + "123456");
ToolUseBlock newToolUse = ToolUseBlock.builder()
.id(toolUse.getId())
.name(toolUse.getName())
.input(input)
.content(toolUse.getContent())
.metadata(toolUse.getMetadata())
.build();
preActing.setToolUse(newToolUse);
}
// Return the event unchanged
return Mono.just(event);
}
}

2. How to execute
3. See error
   Expected behavior
   A clear and concise description of what you expected to happen.

Error messages
Detailed error messages.

Environment (please complete the following information):

• AgentScope-Java Version: [e.g. 1.0.4, check your pom.xml dependency version or run mvn dependency:tree | grep agentscope-parent:pom(only mac/linux)]
• Java Version: [e.g. 17]
• OS: [e.g. macos, windows]

Additional context
Add any other context about the problem here.

[AlbumenJ]

Pre-memory insertion occurs after PostReasoning and before PreActing. Therefore, it is expected behavior that modifications to ToolUse during PreActing are not reflected in Memory.

However, due to current implementation issues in PreActing, the modified ToolUse is not being applied in the current Acting phase. This logic needs to be adjusted so that it executes immediately before the actual Acting.