Skip to content

fix(demo): agui demo access return error Failed to parse request: Type definition error: [simple type, class io.agentscope.core.agui.model.MessageContent] - #2704

Open
yytv5 wants to merge 2 commits into
agentscope-ai:mainfrom
yytv5:hotfix_aguiExample
Open

fix(demo): agui demo access return error Failed to parse request: Type definition error: [simple type, class io.agentscope.core.agui.model.MessageContent]#2704
yytv5 wants to merge 2 commits into
agentscope-ai:mainfrom
yytv5:hotfix_aguiExample

Conversation

@yytv5

@yytv5 yytv5 commented Aug 14, 2026

Copy link
Copy Markdown
  • 新增MVC消息转换器和WebFlux解码器
  • 在自动配置类中注册自定义反序列化组件
  • 兼容Spring Boot 4默认Jackson3,仅针对RunAgentInput使用Jackson2
  • 新增对应单元测试验证功能正确性

AgentScope-Java Version

[The version of AgentScope-Java you are working on, e.g. 1.0.12, check your pom.xml dependency version or run mvn dependency:tree | grep agentscope-parent:pom(only mac/linux)]

Description

[Please describe the background, purpose, changes made, and how to test this PR]

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

- 新增MVC消息转换器和WebFlux解码器
- 在自动配置类中注册自定义反序列化组件
- 兼容Spring Boot 4默认Jackson3,仅针对RunAgentInput使用Jackson2
- 新增对应单元测试验证功能正确性
@CLAassistant

Copy link
Copy Markdown

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.


yangyongtai3 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@yytv5

yytv5 commented Aug 14, 2026

Copy link
Copy Markdown
Author

fix issue #2703

@yytv5 yytv5 changed the title fix(demo): agui Failed to parse request fix(demo): fix issue #2703 Aug 14, 2026
@yytv5 yytv5 changed the title fix(demo): fix issue #2703 fix(demo): agui demo access return error Failed to parse request: Type definition error: [simple type, class io.agentscope.core.agui.model.MessageContent] Aug 14, 2026
@codecov

codecov Bot commented Aug 14, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 73.68421% with 5 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...ot/agui/mvc/RunAgentInputHttpMessageConverter.java 57.14% 1 Missing and 2 partials ⚠️
...ng/boot/agui/webflux/RunAgentInputJsonDecoder.java 60.00% 0 Missing and 2 partials ⚠️

📢 Thoughts on this report? Let us know!

@AgentScopeJavaBot AgentScopeJavaBot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 AI Review

Adds type-scoped Jackson 2 converters for RunAgentInput in both MVC and WebFlux paths, solving Spring Boot 4 / Jackson 3 compatibility. Clean architecture, precise scoping, good tests. Recommendation: APPROVE.

@AgentScopeJavaBot AgentScopeJavaBot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 AI Review

Adds type-scoped Jackson 2 converters for RunAgentInput in both MVC and WebFlux paths, solving Spring Boot 4 / Jackson 3 compatibility. Clean architecture, precise scoping, good tests. Recommendation: APPROVE.

@yytv5

yytv5 commented Aug 14, 2026

Copy link
Copy Markdown
Author

@AgentScopeJavaBot please merge

@AgentScopeJavaBot AgentScopeJavaBot added bug Something isn't working area/ext/spring-boot Spring Boot starters labels Aug 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ext/spring-boot Spring Boot starters bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants