fix: suppress repetitive A2A experimental mode warnings by optimus-fulcria · Pull Request #1441 · kagent-dev/kagent

optimus-fulcria · 2026-03-05T15:39:50Z

Summary

Suppresses the repetitive [EXPERIMENTAL] RemoteA2aAgent UserWarning that floods agent logs during A2A operations
Adds warnings.filterwarnings() in both types.py and _agent_executor.py to filter upstream ADK experimental warnings at module import time
Uses a regex pattern (\[EXPERIMENTAL\].*A2A) to catch all A2A-related experimental warnings

Fixes #1379

Test plan

Deploy agents with A2A tools and verify logs no longer contain experimental mode warnings
Verify A2A functionality still works correctly (warnings are suppressed, not the functionality)

🤖 Generated with Claude Code

Copilot

Pull request overview

Suppresses repetitive upstream ADK A2A “experimental mode” warnings that spam logs during A2A operations (fixes #1379).

Changes:

Adds warnings.filterwarnings() in kagent.adk.types to ignore A2A-related [EXPERIMENTAL] warnings.
Adds the same warning filter in kagent.adk._agent_executor prior to importing upstream A2A executor components.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File	Description
`python/packages/kagent-adk/src/kagent/adk/types.py`	Installs a module-import-time warnings filter intended to suppress ADK A2A experimental warnings.
`python/packages/kagent-adk/src/kagent/adk/_agent_executor.py`	Installs the same warnings filter before importing upstream `google.adk.a2a` executor types.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-03-05T15:44:34Z

python/packages/kagent-adk/src/kagent/adk/types.py

+# Suppress repetitive experimental mode warnings from upstream ADK A2A support (#1379)
+warnings.filterwarnings("ignore", message=r"\[EXPERIMENTAL\].*A2A")
 from google.adk.agents.remote_a2a_agent import AGENT_CARD_WELL_KNOWN_PATH, DEFAULT_TIMEOUT, RemoteA2aAgent


warnings.filterwarnings() runs at module import time and mutates the global warnings filter for any consumer that imports kagent.adk.types (including via kagent.adk.AgentConfig). This can unexpectedly suppress warnings outside of A2A usage and makes behavior hard to opt out of. Consider scoping the filter (e.g., category=UserWarning + module= for the upstream ADK A2A modules) and/or gating it behind an explicit opt-in (env var / config) or moving it into A2A-only entrypoints instead of a widely imported types module.

Copilot · 2026-03-05T15:44:35Z

python/packages/kagent-adk/src/kagent/adk/types.py

 from google.adk.agents.readonly_context import ReadonlyContext
+
+# Suppress repetitive experimental mode warnings from upstream ADK A2A support (#1379)
+warnings.filterwarnings("ignore", message=r"\[EXPERIMENTAL\].*A2A")


Using action="ignore" fully hides the upstream experimental warning. If the goal is to stop log flooding while still informing users, action="once" (or default + a de-dup strategy) would prevent repetition without completely silencing the warning.

Suggested change

warnings.filterwarnings("ignore", message=r"\[EXPERIMENTAL\].*A2A")

warnings.filterwarnings("once", message=r"\[EXPERIMENTAL\].*A2A")

Copilot · 2026-03-05T15:44:36Z

python/packages/kagent-adk/src/kagent/adk/_agent_executor.py

+# Suppress repetitive experimental mode warnings from upstream ADK A2A support (#1379)
+warnings.filterwarnings("ignore", message=r"\[EXPERIMENTAL\].*A2A")


This installs the same global warnings filter as kagent.adk.types. Because warnings.filterwarnings() prepends by default, importing both modules will add duplicate entries to warnings.filters (minor overhead and harder to reason about). Consider centralizing this in a single helper that is called once, or ensure one module is the single place that configures warning suppression.

Copilot · 2026-03-05T15:44:36Z

python/packages/kagent-adk/src/kagent/adk/types.py

 from google.adk.agents.readonly_context import ReadonlyContext
+
+# Suppress repetitive experimental mode warnings from upstream ADK A2A support (#1379)
+warnings.filterwarnings("ignore", message=r"\[EXPERIMENTAL\].*A2A")


There’s no regression test covering this new behavior (warning suppression). Since types.py is heavily imported in tests, adding a small unit test that asserts the targeted [EXPERIMENTAL] ... A2A UserWarning is not emitted repeatedly (or that the expected filter is installed) would help prevent reintroducing log spam or accidentally over-broad suppression.

Suggested change

warnings.filterwarnings("ignore", message=r"\[EXPERIMENTAL\].*A2A")

warnings.filterwarnings(

"ignore",

message=r"\[EXPERIMENTAL\].*A2A",

category=UserWarning,

module=r"^google\.adk\.agents\.remote_a2a_agent$",

)

The upstream google-adk A2A components emit UserWarning messages about experimental mode on every operation, flooding agent logs. This adds warnings.filterwarnings() to suppress these known upstream warnings at module import time. Fixes kagent-dev#1379 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Optimus (AI Agent) <agent@fulcria.com>

- Change filterwarnings from "ignore" to "once" so users see the experimental warning once per process instead of never - Remove duplicate filter from _agent_executor.py since types.py (imported earlier) already installs it - Add unit test verifying duplicate warnings are suppressed Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Optimus (AI Agent) <agent@fulcria.com>

Address remaining Copilot review feedback: - Add category=UserWarning and module=r"^google\.adk\." to filterwarnings() so the filter only affects A2A warnings from upstream ADK modules, not unrelated warnings from other packages - Add isort:off/on guards to prevent import reordering by ruff - Update tests to verify scoping: warnings from google.adk are deduplicated while warnings from other modules pass through Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Optimus (AI Agent) <agent@fulcria.com>

Address Copilot review: scope the module regex from the broad `^google\.adk\.` prefix to the exact `^google\.adk\.agents\.remote_a2a_agent$` module where the warning originates. This prevents accidentally suppressing experimental warnings from other google.adk submodules. Also adds a test verifying that warnings from other google.adk modules are not affected by the filter. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Optimus (AI Agent) <agent@fulcria.com>

Copilot AI review requested due to automatic review settings March 5, 2026 15:39

optimus-fulcria requested review from EItanya, peterj and yuval-k as code owners March 5, 2026 15:39

Copilot started reviewing on behalf of optimus-fulcria March 5, 2026 15:41 View session

Copilot AI reviewed Mar 5, 2026

View reviewed changes

optimus-fulcria and others added 2 commits March 11, 2026 15:41

optimus-fulcria force-pushed the fix/suppress-a2a-experimental-warning branch from a429bb4 to e9c401f Compare March 11, 2026 15:41

optimus-fulcria and others added 2 commits March 11, 2026 19:18

optimus-fulcria force-pushed the fix/suppress-a2a-experimental-warning branch from 8a99a8e to d042ecc Compare March 11, 2026 19:34

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: suppress repetitive A2A experimental mode warnings#1441

fix: suppress repetitive A2A experimental mode warnings#1441
optimus-fulcria wants to merge 4 commits intokagent-dev:mainfrom
optimus-fulcria:fix/suppress-a2a-experimental-warning

optimus-fulcria commented Mar 5, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Mar 5, 2026

Uh oh!

Copilot AI Mar 5, 2026

Uh oh!

Copilot AI Mar 5, 2026

Uh oh!

Copilot AI Mar 5, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

	warnings.filterwarnings("ignore", message=r"\[EXPERIMENTAL\].*A2A")
	warnings.filterwarnings("once", message=r"\[EXPERIMENTAL\].*A2A")

		# Suppress repetitive experimental mode warnings from upstream ADK A2A support (#1379)
		warnings.filterwarnings("ignore", message=r"\[EXPERIMENTAL\].*A2A")

Conversation

optimus-fulcria commented Mar 5, 2026

Summary

Test plan

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Mar 5, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 5, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 5, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 5, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants