Repository navigation
Conversation
|
Hi @he-yufeng , Thank you for your contribution! We appreciate you taking the time to submit this pull request. Can you please fix the failing unit tests before we can proceed with the review. |
3dca9bd to
b519da0
Compare
b519da0 to
6ec8fca
Compare
|
@rohityan Done, and sorry for the long silence on your June ask. Rebased onto current main (the test file got a union merge with the branch-coverage tests landed since) and the workflow unit suite is green locally: 735 passed, 11 skipped. The one thing I could not replay is the original CI failure, since the June run logs are gone; if a specific test still fails on your side, name it and I'll fix that one directly. |
|
Still applies on main: |
|
Thank you @he-yufeng for your contribution! 🎉 Your changes have been successfully imported and merged via Copybara in commit 6b96ba1. Closing this PR as the changes are now in the main branch. |
Fixes #6089.
Summary
message_as_outputfor plain single-turn agent text output.output_schemaresults asevent.outputinstead of marking them as message output.To verify
$env:PYTHONPATH=(Resolve-Path .\src).Path; python -m pytest tests\unittests\workflow\test_llm_agent_as_node.py -qpython -m py_compile src\google\adk\workflow\_llm_agent_wrapper.py tests\unittests\workflow\test_llm_agent_as_node.pypython -m ruff check src\google\adk\workflow\_llm_agent_wrapper.pygit diff --check