Skip to content

feat: add support for complete_result and final_result artifacts#32

Merged
sriaradhyula merged 1 commit intomainfrom
feat/complete-result-and-final-result-support-new
Nov 20, 2025
Merged

feat: add support for complete_result and final_result artifacts#32
sriaradhyula merged 1 commit intomainfrom
feat/complete-result-and-final-result-support-new

Conversation

@sriaradhyula
Copy link
Copy Markdown
Member

Added handling for complete_result and final_result artifacts to replace streamed text when partial_result is not available. This ensures authoritative backend-provided results take precedence over accumulated streaming chunks.

Changes:

  • Added complete_result_text and final_result_text variables
  • Added handlers for complete_result and final_result artifacts
  • Updated priority order: UserInputMetaData > partial_result > complete_result > final_result > streaming buffer
  • Enhanced debug logging to include all result types

This matches the pattern used for partial_result and ensures consistent behavior across all authoritative result artifacts.

Closes #

Added handling for complete_result and final_result artifacts to replace
streamed text when partial_result is not available. This ensures
authoritative backend-provided results take precedence over accumulated
streaming chunks.

Changes:
- Added complete_result_text and final_result_text variables
- Added handlers for complete_result and final_result artifacts
- Updated priority order: UserInputMetaData > partial_result > complete_result > final_result > streaming buffer
- Enhanced debug logging to include all result types

This matches the pattern used for partial_result and ensures consistent
behavior across all authoritative result artifacts.

Closes #<issue-number>

Signed-off-by: Sri Aradhyula <sraradhy@cisco.com>
@sriaradhyula sriaradhyula merged commit 4a87f70 into main Nov 20, 2025
11 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.

1 participant