Skip to content

Conversation

@sihyeonn
Copy link

@sihyeonn sihyeonn commented Jan 20, 2026

Description

Fix incorrect priority comments for AI plugins in config.yaml.example.

The following plugins had wrong priority values in comments:

  • ai-proxy: 999 → 1040
  • ai-proxy-multi: 998 → 1041
  • ai-aws-content-moderation: 1040 → 1050

Also reordered these plugins to match their actual priority values (sorted by descending priority).

Which issue(s) this PR fixes:

Fixes #

Checklist

  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible (If not, please discuss on the APISIX mailing list first)

- ai-proxy: 999 → 1040
- ai-proxy-multi: 998 → 1041
- ai-aws-content-moderation: 1040 → 1050

Also reorder plugins to match actual priority values.

Signed-off-by: Sihyeon Jang <[email protected]>
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jan 20, 2026
Copy link
Contributor

@Baoyuantop Baoyuantop left a comment

Choose a reason for hiding this comment

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

We also need to add the ai-aliyun-content-moderation and ai-request-rewrite plugins.

@Baoyuantop Baoyuantop added the wait for update wait for the author's response in this issue/PR label Jan 26, 2026
@sihyeonn
Copy link
Author

@Baoyuantop Thanks for the suggestion! This PR focuses on fixing the existing issue. I'll add those plugins in a separate PR.

@Baoyuantop
Copy link
Contributor

Hi @sihyeonn, I believe this falls under a category of issues that should be addressed collectively in the current PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS This PR changes 0-9 lines, ignoring generated files. wait for update wait for the author's response in this issue/PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants