chore: Remove experimental tag from SkillToolset

Co-authored-by: Kathy Wu <wukathy@google.com>
PiperOrigin-RevId: 916198410

Author: wukath

src/google/adk/features/_feature_registry.py

@@ -150,7 +150,7 @@ class FeatureConfig:
         FeatureStage.EXPERIMENTAL, default_on=True
     ),
     FeatureName.SKILL_TOOLSET: FeatureConfig(
-        FeatureStage.EXPERIMENTAL, default_on=True
+        FeatureStage.STABLE, default_on=True
     ),
     FeatureName.SPANNER_ADMIN_TOOLSET: FeatureConfig(
         FeatureStage.EXPERIMENTAL, default_on=True

src/google/adk/tools/bash_tool.py

@@ -29,7 +29,6 @@
 
 from google.genai import types
 
-from .. import features
 from .base_tool import BaseTool
 from .tool_context import ToolContext
 
@@ -99,7 +98,6 @@ def _set_resource_limits(policy: BashToolPolicy) -> None:
     logger.warning("Failed to set resource limits: %s", e)
 
 
-@features.experimental(features.FeatureName.SKILL_TOOLSET)
 class ExecuteBashTool(BaseTool):
   """Tool to execute a validated bash command within a workspace directory."""
 

src/google/adk/tools/skill_toolset.py

@@ -25,16 +25,13 @@
 import mimetypes
 from typing import Any
 from typing import TYPE_CHECKING
-import warnings
 
 from google.genai import types
 from typing_extensions import override
 
 from ..agents.readonly_context import ReadonlyContext
 from ..code_executors.base_code_executor import BaseCodeExecutor
 from ..code_executors.code_execution_utils import CodeExecutionInput
-from ..features import experimental
-from ..features import FeatureName
 from ..skills import models
 from ..skills import prompt
 from ..skills import SkillRegistry
@@ -58,7 +55,7 @@
     " conversation history for you to analyze."
 )
 
-_DEFAULT_SKILL_SYSTEM_INSTRUCTION = (
+DEFAULT_SKILL_SYSTEM_INSTRUCTION = (
     "You can use specialized 'skills' to help you with complex tasks. "
     "You MUST use the skill tools to interact with these skills.\n\n"
     "Skills are folders of instructions and resources that extend your "
@@ -88,7 +85,6 @@
 )
 
 
-@experimental(FeatureName.SKILL_TOOLSET)
 class ListSkillsTool(BaseTool):
   """Tool to list all available skills."""
 
@@ -118,7 +114,6 @@ async def run_async(
     return prompt.format_skills_as_xml(skills)
 
 
-@experimental(FeatureName.SKILL_TOOLSET)
 class SearchSkillsTool(BaseTool):
   """Tool to search for relevant skills in the registry."""
 
@@ -181,7 +176,6 @@ async def run_async(
       }
 
 
-@experimental(FeatureName.SKILL_TOOLSET)
 class LoadSkillTool(BaseTool):
   """Tool to load a skill's instructions."""
 
@@ -250,7 +244,6 @@ async def run_async(
     }
 
 
-@experimental(FeatureName.SKILL_TOOLSET)
 class LoadSkillResourceTool(BaseTool):
   """Tool to load resources (references, assets, or scripts) from a skill."""
 
@@ -710,7 +703,6 @@ def _build_wrapper_code(
     return "\n".join(code_lines)
 
 
-@experimental(FeatureName.SKILL_TOOLSET)
 class RunSkillScriptTool(BaseTool):
   """Tool to execute scripts from a skill's scripts/ directory."""
 
@@ -874,7 +866,6 @@ async def run_async(
     )
 
 
-@experimental(FeatureName.SKILL_TOOLSET)
 class SkillToolset(BaseToolset):
   """A toolset for managing and interacting with agent skills."""
 
@@ -1063,7 +1054,7 @@ async def process_llm_request(
       self, *, tool_context: ToolContext, llm_request: LlmRequest
   ) -> None:
     """Processes the outgoing LLM request to include available skills."""
-    instructions = [_DEFAULT_SKILL_SYSTEM_INSTRUCTION]
+    instructions = [DEFAULT_SKILL_SYSTEM_INSTRUCTION]
 
     has_list_skills = any(isinstance(t, ListSkillsTool) for t in self._tools)
 
@@ -1090,16 +1081,3 @@ async def close(self) -> None:
           cached.cancel()
     self._fetched_skill_cache.clear()
     await super().close()
-
-
-def __getattr__(name: str) -> Any:
-  if name == "DEFAULT_SKILL_SYSTEM_INSTRUCTION":
-    warnings.warn(
-        "DEFAULT_SKILL_SYSTEM_INSTRUCTION is experimental. Its content "
-        "is internal implementation and will change in minor/patch releases "
-        "to tune agent performance.",
-        UserWarning,
-        stacklevel=2,
-    )
-    return _DEFAULT_SKILL_SYSTEM_INSTRUCTION
-  raise AttributeError(f"module {__name__} has no attribute {name}")

tests/unittests/tools/test_skill_toolset.py

@@ -471,14 +471,6 @@ async def test_process_llm_request_without_list_skills_tool(
   assert "skill2" in instructions[1]
 
 
-def test_default_skill_system_instruction_warning():
-  with pytest.warns(
-      UserWarning, match="DEFAULT_SKILL_SYSTEM_INSTRUCTION is experimental"
-  ):
-    instruction = skill_toolset.DEFAULT_SKILL_SYSTEM_INSTRUCTION
-    assert "specialized 'skills'" in instruction
-
-
 def test_duplicate_skill_name_raises(mock_skill1):
   skill_dup = mock.create_autospec(models.Skill, instance=True)
   skill_dup.name = "skill1"