Skip to content

Conversation

jackxu2011
Copy link
Contributor

feat(core): upgrade spring-ai to 1.0.1, spring-boot to 3.4.8 and fix test hang in mac with docker

Describe what this PR does / why we need it

Does this pull request fix one issue?

Describe how you did it

Describe how to verify it

Special notes for reviews

@github-actions github-actions bot added area/graph SAA Grpah module area/jmanus SAA Jmanus module area/core SAA Core module area/deepresearch SAA Deepresearch module area/nl2sql SAA nl2sql module labels Aug 6, 2025
@yuluo-yx yuluo-yx added king/pending and removed area/graph SAA Grpah module area/jmanus SAA Jmanus module area/deepresearch SAA Deepresearch module area/nl2sql SAA nl2sql module labels Aug 7, 2025
@jackxu2011 jackxu2011 force-pushed the feat-spring-ai-1.0.1 branch from c2659ae to f2e0aac Compare August 14, 2025 15:22
@github-actions github-actions bot added area/graph SAA Grpah module area/jmanus SAA Jmanus module area/deepresearch SAA Deepresearch module area/nl2sql SAA nl2sql module labels Aug 14, 2025
@jackxu2011 jackxu2011 force-pushed the feat-spring-ai-1.0.1 branch from f2e0aac to d8cb21a Compare August 14, 2025 15:38
@github-actions github-actions bot removed the area/graph SAA Grpah module label Aug 14, 2025
@jackxu2011 jackxu2011 force-pushed the feat-spring-ai-1.0.1 branch 2 times, most recently from 35299b8 to ec1529e Compare August 19, 2025 01:04
@jackxu2011 jackxu2011 force-pushed the feat-spring-ai-1.0.1 branch from ec1529e to 329c054 Compare August 25, 2025 02:28
@jackxu2011 jackxu2011 force-pushed the feat-spring-ai-1.0.1 branch from 329c054 to aa1c5a7 Compare August 25, 2025 06:07
@yuluo-yx
Copy link
Collaborator

yuluo-yx commented Sep 7, 2025

lgtm, Can you list what updates have been made?

@jackxu2011
Copy link
Contributor Author

jackxu2011 commented Sep 7, 2025

lgtm, Can you list what updates have been made?

  1. upgrade spring-ai to 1.0.1
  2. upgrade spring-boot to 3.4.8
  3. upgrade docker-java to 3.5.3 to fix the test blocked in macos
  4. in test cases change the AI_DASHSCOPE_API_KEY matches from .* to sk.+ @EnabledIfEnvironmentVariable(named = "AI_DASHSCOPE_API_KEY", matches = "sk.+") to avoid testing when export AI_DASHSCOPE_API_KEY=""
  5. fix deepsearch's ChatController used the SaverConstant.MEMORY that has been removed in 1.0.0.4-snapshot
  6. upgrade mysql-connector-java to mysql-connector-j
  7. using project.version replace spring-ai-alibaba.version in spring-ai-alibaba-nl2sql
  8. remove some useless properties in spring-ai-alibaba-nl2sql

Copy link
Collaborator

@yuluo-yx yuluo-yx left a comment

Choose a reason for hiding this comment

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

LGTM thx

@yuluo-yx yuluo-yx merged commit 9fbf03b into alibaba:main Sep 9, 2025
10 checks passed
github-actions bot pushed a commit to Hipple/spring-ai-alibaba that referenced this pull request Sep 9, 2025
zhouyou9505 pushed a commit to zhouyou9505/spring-ai-alibaba that referenced this pull request Sep 13, 2025
mcallzbl pushed a commit to mcallzbl/spring-ai-alibaba that referenced this pull request Sep 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/core SAA Core module area/deepresearch SAA Deepresearch module area/jmanus SAA Jmanus module area/nl2sql SAA nl2sql module king/pending
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants