dify icon indicating copy to clipboard operation
dify copied to clipboard

robust for json parser

Open zxfishhack opened this issue 9 months ago • 2 comments

Summary

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

[!Tip] Close issue syntax: Fixes #<issue number> or Resolves #<issue number>, see documentation for more details.

extract first valid JSON from arguments.

Fixes #17686 .

Screenshots

Before After
... ...

Checklist

[!IMPORTANT]
Please review the checklist below before submitting your pull request.

  • [ ] This change requires a documentation update, included: Dify Document
  • [x] I understand that this PR may be closed in case there was no previous discussion or issues. (This doesn't apply to typos!)
  • [x] I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change.
  • [x] I've updated the documentation accordingly.
  • [x] I ran dev/reformat(backend) and cd web && npx lint-staged(frontend) to appease the lint gods

zxfishhack avatar Apr 09 '25 08:04 zxfishhack

Please fix the tests.

crazywoola avatar Apr 09 '25 08:04 crazywoola

Hi @zxfishhack, Thank you for your contribution! Could you please add some unit tests for this enhancement?

laipz8200 avatar Apr 10 '25 09:04 laipz8200