Skip to content

fix: load artifacts from workflow text responses#5969

Open
he-yufeng wants to merge 1 commit into
google:mainfrom
he-yufeng:fix/load-artifacts-text-response
Open

fix: load artifacts from workflow text responses#5969
he-yufeng wants to merge 1 commit into
google:mainfrom
he-yufeng:fix/load-artifacts-text-response

Conversation

@he-yufeng
Copy link
Copy Markdown
Contributor

Summary

  • support load_artifacts results that were converted into workflow context text
  • scan all parts in the latest content turn instead of assuming the first part is the function response
  • add regressions for workflow text responses and non-first function responses

Fixes #5967.

To verify

  • .\.venv\Scripts\python -m pytest tests\unittests\tools\test_load_artifacts_tool.py -q
  • .\.venv\Scripts\python -m pyink --check src\google\adk\tools\load_artifacts_tool.py tests\unittests\tools\test_load_artifacts_tool.py
  • .\.venv\Scripts\python -m py_compile src\google\adk\tools\load_artifacts_tool.py tests\unittests\tools\test_load_artifacts_tool.py
  • git diff --check

@adk-bot adk-bot added the tools [Component] This issue is related to tools label Jun 4, 2026
@adk-bot
Copy link
Copy Markdown
Collaborator

adk-bot commented Jun 4, 2026

Response from ADK Triaging Agent

Hello @he-yufeng, thank you for creating this PR!

We noticed that a few requirements from our contribution guidelines are missing:

  1. Summary of passed unit tests: Please run your tests and include a summary of the passed pytest results in your PR description.
  2. Logs or screenshots: Please provide execution logs or screenshots showing the tool in action after applying the fix to help reviewers better understand the change.

Providing these details will help the maintainers review and merge your PR more efficiently. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tools [Component] This issue is related to tools

Projects

None yet

Development

Successfully merging this pull request may close these issues.

load_artifacts is not working as expected in adk 2.0

2 participants