Skip to content

Conversation

@478320
Copy link
Contributor

@478320 478320 commented Jun 3, 2025

Make sure that:

  • You have read the contribution guidelines.
  • You submit test cases (unit or integration tests) that back your changes.
  • Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

@478320
Copy link
Contributor Author

478320 commented Jun 3, 2025

#5973

@Aias00 Aias00 changed the title fea plugin ai-request-transformer : first add plugin feat: ai-request-transformer plugin Jun 4, 2025
/**
* sysContent.
*/
public static final String SYS_CONTENT = "你是HTTP/1.1协议专家,回答只包含标准HTTP/1.1报文内容。 \n"
Copy link
Contributor

Choose a reason for hiding this comment

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

english pls

*/
public ChatModel initAiModel(final AiRequestTransformerConfig aiRequestTransformerConfig) {
switch (Objects.requireNonNull(AiModelProviderEnum.getByName(aiRequestTransformerConfig.getProvider()))) {
case DEEP_SEEK :
Copy link
Contributor

Choose a reason for hiding this comment

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

use Factory Design Model, not switch case

@Aias00
Copy link
Contributor

Aias00 commented Jun 4, 2025

  1. fix ci
  2. add other sql type
  3. add upgrade sql

Copy link
Contributor

@Aias00 Aias00 left a comment

Choose a reason for hiding this comment

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

LGTM

@Aias00 Aias00 merged commit 1df7ac8 into apache:master Jun 9, 2025
77 of 118 checks passed
478320 added a commit to 478320/shenyu that referenced this pull request Jul 17, 2025
* fea plugin ai-request-transformer:first add plugin

* update sql file

* fix : config based on rule

* add ai model registry

* add sql flie and upgrade sql file

* trigger ci

* fix handler destory logic

* trigger ci

* trigger ci

* trigger ci

* trigger ci

* fix ci

* remove ai model autoconfig

* trigger ci

* fix ci

* trigger ci

* fix ci

* trigger ci
@Aias00 Aias00 added this to the 2.7.0.2 milestone Jul 19, 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