Skip to content

feat(agent-testing): let output assertions and llmJudge read rich content - #1466

Merged
yileicn merged 1 commit into
SciSharp:masterfrom
yileicn:master
Oct 28, 2026
Merged

yileicn merged 1 commit into
SciSharp:masterfrom
yileicn:master

Conversation

@yileicn

@yileicn yileicn commented Oct 28, 2026

Copy link
Copy Markdown
Member

feat(agent-testing): let output assertions and llmJudge read rich content

  • Some replies keep their substance only in the rich message (e.g. an
    upload card's photo and video lists), so text-only assertions could
    not check them. The driver now also records the rich message as JSON.
  • Opt-in through target "richContent"; the default still reads the reply
    text, so button titles never leak into existing outputNotContains checks.
  • An unknown target fails the assertion instead of falling back to the
    reply text.

Co-Authored-By: Claude Opus 5.5 [email protected]

…tent

- Some replies keep their substance only in the rich message (e.g. an
  upload card's photo and video lists), so text-only assertions could
  not check them. The driver now also records the rich message as JSON.
- Opt-in through target "richContent"; the default still reads the reply
  text, so button titles never leak into existing outputNotContains checks.
- An unknown target fails the assertion instead of falling back to the
  reply text.

Co-Authored-By: Claude Opus 5.5 <[email protected]>
@qodo-code-review

Copy link
Copy Markdown
Contributor

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@yuyixg

yuyixg commented Oct 28, 2026

Copy link
Copy Markdown
Contributor

reviewed

@yileicn
yileicn merged commit d3a7f96 into SciSharp:master Oct 28, 2026
4 checks passed
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