Skip to content

fix: snapshot watch results before rendering#3215

Open
skt-shinyruo wants to merge 1 commit into
alibaba:masterfrom
skt-shinyruo:fix-watch-snapshot-rendering
Open

fix: snapshot watch results before rendering#3215
skt-shinyruo wants to merge 1 commit into
alibaba:masterfrom
skt-shinyruo:fix-watch-snapshot-rendering

Conversation

@skt-shinyruo

Copy link
Copy Markdown
Contributor

Summary

  • render watch expression results at advice time and store the rendered snapshot on ObjectVO
  • make terminal, HTTP API, and MCP ObjectVO rendering prefer the stored watch snapshot
  • add regression coverage for mutable AtEnter arguments and MCP renderedValue handling

Test Plan

  • ./mvnw -ntp -pl core -am -Dtest=WatchAdviceListenerTest,McpObjectVOFilterTest -Dsurefire.failIfNoSpecifiedTests=false test
  • ./mvnw -ntp -pl core -am test
  • git diff --check

Notes

  • Full local reactor check passed with 263 tests, 0 failures, 0 errors, 7 skipped.

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.

1 participant