Skip to content

Conversation

Cirilla-zmh
Copy link
Collaborator

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

Cirilla-zmh and others added 17 commits September 2, 2025 21:25
- 在 BaseNl2SqlService 和 BaseSchemaService 中
- 添加对 agentId 进行 trim 并检查是否为空的逻辑
- 避免因空的 agentId 导致的潜在错误或异常
- 将数据库查询结果转换类型从 boolean 改为 Boolean
…SL and enhance them for Dify DSL (alibaba#2336)

* refactor: use default upload-method (local-file)

* feat: support studio dsl LLMNode

* fix: fix studio LLM Node

* fix: fix dify LLM Node

* feat: init KnowledgeRetrievalNode for Studio DSL

* feat: add NodeSection.resourceFiles Method

* fix bugs

* feat: support generate KnowledgeRetrievalNode code with local Resources

* fix styles

* fix front end bugs

* fix bugs

* fix bugs
@github-actions github-actions bot added area/graph SAA Grpah module area/core SAA Core module area/studio SAA Studio module area/nl2sql SAA nl2sql module labels Sep 9, 2025
ChatOptions options = context.getRequest().getOptions();
if (options instanceof OpenAiChatOptions) {
Map<String, String> metadata = ((OpenAiChatOptions) options).getMetadata();
if (metadata != null) {
Copy link
Contributor

Choose a reason for hiding this comment

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

这里是写死的OpenAIChatOptions?是否应该抽象出一个MetaDataAwareOption

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

临时方案 cc @chickenlj

@chickenlj chickenlj merged commit 7666da0 into alibaba:studio-dev Sep 9, 2025
10 of 13 checks passed
chickenlj pushed a commit to chickenlj/spring-ai-alibaba that referenced this pull request Sep 17, 2025
co63oc pushed a commit to co63oc/spring-ai-alibaba that referenced this pull request Sep 19, 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/graph SAA Grpah module area/nl2sql SAA nl2sql module area/studio SAA Studio module
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants