Skip to content

fix: filter hidden artifacts from LLM context - #6875

Open
Hughhhhcoder wants to merge 1 commit into
google:mainfrom
Hughhhhcoder:fix/hide-artifacts-from-llm
Open

fix: filter hidden artifacts from LLM context#6875
Hughhhhcoder wants to merge 1 commit into
google:mainfrom
Hughhhhcoder:fix/hide-artifacts-from-llm

Conversation

@Hughhhhcoder

Copy link
Copy Markdown

Fixes #6873

Summary

  • add an optional sync/async artifact-name filter to LoadArtifactsTool
  • apply the filter to the tool response, dynamic instructions, and artifact content loading
  • preserve existing behavior when no filter is configured

Tests

  • uv run pytest tests/unittests/tools/test_load_artifacts_tool.py -q
  • uv run pre-commit run --files src/google/adk/tools/load_artifacts_tool.py tests/unittests/tools/test_load_artifacts_tool.py

@google-cla

google-cla Bot commented Aug 24, 2026

Copy link
Copy Markdown

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@Hughhhhcoder

Copy link
Copy Markdown
Author

Thanks for the update. The code and repository checks are passing; the only failing check is the Google CLA check, which requires the contributor to complete the CLA at the link provided by the bot before maintainers can review. No code change is needed for this CI failure.

@Hughhhhcoder

Copy link
Copy Markdown
Author

Thanks — I’ve completed the Google CLA and triggered the rescan. The cla/google check is now passing, and all remaining checks are passing or skipped. The PR is ready for maintainer review.

@Hughhhhcoder

Copy link
Copy Markdown
Author

The latest Continuous Integration and Do Not Merge on GitHub pull-request runs are completed with action_required, while cla/google and the target checks pass or are skipped. Could a maintainer approve these workflows when convenient so the PR can proceed to review?

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.

Hide certain artifacts from the LLM in LoadArtifactsTool

2 participants