fix: The events array after splitting may contain empty strings, w…

[vislee]

Description

The ctx.ai_token_usage may not get the correct value.
The events array after splitting may contain empty strings, which can cause subsequent event to be skipped.

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)

[shreemaan-abhishek]

please add test cases too.

[Baoyuantop]

Please fix failed lint CI

[Baoyuantop]

Hi @vislee, is there time to continue with this PR?

[vislee]

I've pushed the fix failed lint CI.
Btw, how to run tests locally, Are there docker images?

Hi @vislee, is there time to continue with this PR?

[Baoyuantop]

You can refer to https://github.com/apache/apisix/blob/master/docs/en/latest/build-apisix-dev-environment-devcontainers.md

[Baoyuantop]

Hi @vislee, any updates?

[Baoyuantop]

I'll convert this PR to draft first, if the changes are done please request review again.

[github-actions]

This pull request has been marked as stale due to 60 days of inactivity. It will be closed in 4 weeks if no further activity occurs. If you think that's incorrect or this pull request should instead be reviewed, please simply write any comment. Even if closed, you can still revive the PR at any time or discuss it on the dev@apisix.apache.org list. Thank you for your contributions.

[github-actions]

This pull request/issue has been closed due to lack of activity. If you think that is incorrect, or the pull request requires review, you can revive the PR at any time.

[Baoyuantop]

The issue described in this PR has been fixed in #12530. If you still have problems, please submit an issue.