Skip to content

fix modifications to PostActingEvent in Hook not taking effect as expected#154

Merged
AlbumenJ merged 1 commit intoagentscope-ai:mainfrom
LearningGp:develop-tool-hook
Dec 8, 2025
Merged

fix modifications to PostActingEvent in Hook not taking effect as expected#154
AlbumenJ merged 1 commit intoagentscope-ai:mainfrom
LearningGp:develop-tool-hook

Conversation

@LearningGp
Copy link
Contributor

fix modifications to PostActingEvent in Hook not taking effect as expected

AgentScope-Java Version

1.0.0

Description

Modified the ToolResult in the PostActingEvent hook, but changes aren't being reflected.

Checklist

Please check the following items before code is ready to be reviewed.

  • Code has been formatted with mvn spotless:apply
  • All tests are passing (mvn test)
  • Javadoc comments are complete and follow project conventions
  • Related documentation has been updated (e.g. links, examples, etc.)
  • Code is ready for review

…ing effect as expected

Change-Id: Id01842d7758e4df51096a17dc8cd8a7f9d942e3b
@LearningGp LearningGp requested a review from a team December 8, 2025 07:19
@cla-assistant
Copy link

cla-assistant bot commented Dec 8, 2025

CLA assistant check
All committers have signed the CLA.

@cla-assistant
Copy link

cla-assistant bot commented Dec 8, 2025

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@codecov
Copy link

codecov bot commented Dec 8, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@AlbumenJ AlbumenJ merged commit 6f17a0b into agentscope-ai:main Dec 8, 2025
4 checks passed
@LearningGp LearningGp deleted the develop-tool-hook branch December 8, 2025 08:11
JGoP-L pushed a commit to JGoP-L/agentscope-java that referenced this pull request Dec 29, 2025
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.

2 participants