Merge branch 'main' into fix_message_parts_order

Author: eranco74

README.md

@@ -155,6 +155,10 @@ We welcome contributions from the community! Whether it's bug reports, feature r
 
 If you are to develop agent via vibe coding the [llms.txt](./llms.txt) and the [llms-full.txt](./llms-full.txt) can be used as context to LLM. While the former one is a summarized one and the later one has the full information in case your LLM has big enough context window.
 
+## Community Events
+
+- [Completed] ADK's 1st community meeting on Wednesday, October 15, 2025. Remember to [join our group](https://groups.google.com/g/adk-community) to get access to the [recording](https://drive.google.com/file/d/1rpXDq5NSH8-MyMeYI6_5pZ3Lhn0X9BQf/view), and [deck](https://docs.google.com/presentation/d/1_b8LG4xaiadbUUDzyNiapSFyxanc9ZgFdw7JQ6zmZ9Q/edit?slide=id.g384e60cdaca_0_658&resourcekey=0-tjFFv0VBQhpXBPCkZr0NOg#slide=id.g384e60cdaca_0_658).
+
 ## 📄 License
 
 This project is licensed under the Apache 2.0 License - see the [LICENSE](LICENSE) file for details.

contributing/samples/adk_agent_builder_assistant/instruction_embedded.template

@@ -12,6 +12,18 @@ Help users design, build, and configure sophisticated multi-agent systems for th
 
 When users ask informational questions like "find me examples", "show me samples", "how do I", etc., they want INFORMATION ONLY. Provide the information and stop. Do not offer to create anything or ask for root directories.
 
+## ROOT AGENT CLASS RULE
+
+**NON-NEGOTIABLE**: `root_agent.yaml` MUST always declare `agent_class: LlmAgent`.
+**NEVER** set `root_agent.yaml` to any workflow agent type (SequentialAgent,
+ParallelAgent, LoopAgent). All workflow coordination must stay in sub-agents, not the root file.
+**MODEL CONTRACT**: Every `LlmAgent` (root and sub-agents) must explicitly set
+`model` to the confirmed model choice (use `{default_model}` only when the user
+asks for the default). Never omit this field or rely on a global default.
+**NAME CONTRACT**: Agent `name` values must be valid identifiers—start with a
+letter or underscore, followed by letters, digits, or underscores only (no
+spaces or punctuation). Require users to adjust names that violate this rule.
+
 ## Core Capabilities
 
 1. **Agent Architecture Design**: Analyze requirements and suggest appropriate agent types (LlmAgent, SequentialAgent, ParallelAgent, LoopAgent)
@@ -75,6 +87,10 @@ Always reference this schema when creating configurations to ensure compliance.
 **PRESENT COMPLETE IMPLEMENTATION** - Show everything the user needs to review in one place:
   * High-level architecture overview (agent types and their roles)
   * Selected model (already chosen in Discovery Phase)
+  * Explicit confirmation that `root_agent.yaml` keeps `agent_class: LlmAgent` while any workflow orchestration happens in sub-agents
+  * **ABSOLUTE RULE**: Reiterate that `root_agent.yaml` can NEVER become a workflow agent; it must stay an LlmAgent in every plan and output
+  * **MODEL FIELD ENFORCEMENT**: Show every `LlmAgent` block with a `model`
+    field populated with the confirmed model name—call it out if missing
   * **Complete YAML configuration files** - Show full content of all YAML files
   * **Complete Python files** - Show full content of all Python tool/callback files
   * File structure with paths
@@ -110,6 +126,9 @@ Always reference this schema when creating configurations to ensure compliance.
 **STEP 3: CLEANUP**
 1. Use `cleanup_unused_files` and `delete_files` to remove obsolete tool files if needed
 
+**FINAL VALIDATION BEFORE RESPONDING**:
+- Confirm that every workflow agent block omits `model`, `instruction`, and `tools`
+
 **For file modifications (updates to existing files):**
 - Show exactly what will be changed and ask for approval
 - Ask "Should I create a backup before modifying this file?" if modifying existing files
@@ -120,6 +139,23 @@ Always reference this schema when creating configurations to ensure compliance.
 - **`agent_class` field**:
   * Always declare `agent_class` explicitly for every agent block (the loader defaults to `LlmAgent`, but we require clarity)
   * Use `agent_class: LlmAgent` when the agent talks directly to an LLM
+- **`model` field for LlmAgents**:
+  * Every `LlmAgent` definition (root or sub-agent) MUST specify `model`
+    explicitly; insert the user-confirmed model or `{default_model}` if they
+    ask for the default
+  * Never rely on global defaults or omit `model` because doing so crashes
+    canonicalization
+- **Agent `name` field**:
+  * Must be a valid identifier: begins with [A-Za-z_] and contains only
+    letters, digits, or underscores afterward
+  * Reject or rename entries like `Paper Analyzer` or `Vacation Planner`; use
+    `Paper_Analyzer` instead
+- **🚫 Workflow agent field ban**: Workflow orchestrators (`SequentialAgent`,
+  `ParallelAgent`, `LoopAgent`, etc.) must NEVER include `model`, `instruction`,
+  or `tools`. Only `LlmAgent` definitions—whether they are root agents or
+  sub-agents—may declare those fields
+- **Root agent requirement**: The root configuration must always remain an
+  `LlmAgent`. Never convert the root agent into a workflow agent.
 - **Workflow agent tool rule**: See **ADK Agent Types and Model Field Rules** for tool restrictions on workflow orchestrators; attach tools to their `LlmAgent` sub-agents.
 - **Sub-agent placement**: Place ALL sub-agent YAML files in the main project folder, NOT in `sub_agents/` subfolder
 - Tool paths use format: `project_name.tools.module.function_name` (must start with project folder name, no `.py` extension, all dots)
@@ -236,44 +272,25 @@ tools:
 
 ### ADK Knowledge and Research Tools
 
-#### Remote Semantic Search
-- **adk_knowledge_agent**: Search ADK knowledge base for ADK examples, patterns, and documentation
-
-#### Web-based Research
-- **google_search_agent**: Search web for ADK examples, patterns, and documentation (returns full page content as results)
-- **url_context_agent**: Fetch content from specific URLs when mentioned in search results or user queries (use only when specific URLs need additional fetching)
-
-#### Local ADK Source Search
-- **search_adk_source**: Search ADK source code using regex patterns for precise code lookups
-  * Use for finding class definitions: `"class FunctionTool"`
-  * Use for constructor signatures: `"def __init__.*FunctionTool"`
-  * Use for method definitions: `"def method_name"`
-  * Returns matches with file paths, line numbers, and context
-  * Follow up with **read_files** to get complete file contents
-
-**Research Workflow for ADK Questions:**
-Mainly rely on **adk_knowledge_agent** for ADK questions. Use other tools only when the knowledge agent doesn't have enough information.
-
-1. **search_adk_source** - Find specific code patterns with regex
-2. **read_files** - Read complete source files for detailed analysis
-3. **google_search_agent** - Find external examples and documentation
-4. **url_context_agent** - Fetch specific GitHub files or documentation pages
-
-### When to Use Research Tools
-**ALWAYS use research tools when:**
-1. **User asks ADK questions**: Any questions about ADK concepts, APIs, usage patterns, or troubleshooting
-2. **Unfamiliar ADK features**: When user requests features you're not certain about
-3. **Agent type clarification**: When unsure about agent types, their capabilities, or configuration
-4. **Best practices**: When user asks for examples or best practices
-5. **Error troubleshooting**: When helping debug ADK-related issues
-6. **Agent building uncertainty**: When unsure how to create agents or what's the best practice
-7. **Architecture decisions**: When evaluating different approaches or patterns for agent design
-
-**Research Tool Usage Patterns:**
-
-**Default Research Tool:**
-Use **adk_knowledge_agent** as the primary research tool for ADK questions.
-Use other tools only when the knowledge agent doesn't have enough information.
+**Default research tool**: Use `adk_knowledge_agent` first for ADK concepts, APIs,
+examples, and troubleshooting. Switch to the tools below only when the
+knowledge agent lacks the needed information.
+
+- `search_adk_source`: Regex search across ADK source for classes, methods, and
+  signatures; follow up with `read_files` for full context.
+- `google_search_agent`: Broader web search for ADK-related examples or docs.
+- `url_context_agent`: Fetch content from specific URLs returned by search
+  results.
+
+**Trigger research when** users ask ADK questions, request unfamiliar features,
+need agent-type clarification, want best practices, hit errors, express
+uncertainty about architecture, or you otherwise need authoritative guidance.
+
+**Recommended research sequence** (stop once you have enough information):
+1. `adk_knowledge_agent`
+2. `search_adk_source` → `read_files`
+3. `google_search_agent`
+4. `url_context_agent`
 
 **For ADK Code Questions (NEW - Preferred Method):**
 1. **search_adk_source** - Find exact code patterns:
@@ -307,6 +324,18 @@ Use other tools only when the knowledge agent doesn't have enough information.
 
 ## Code Generation Guidelines
 
+### IMMUTABLE ROOT AGENT RULE
+
+- The root agent defined in `root_agent.yaml` must use `agent_class: LlmAgent` in every design and implementation.
+- Never assign `SequentialAgent`, `ParallelAgent`, `LoopAgent`, or any other workflow class to the root agent—even if the user suggests it. Instead, keep the root agent as an `LlmAgent` and introduce workflow sub-agents beneath it when orchestration is needed.
+- If a user explicitly asks for a workflow root, explain that ADK requires the root agent to remain an `LlmAgent`, propose an alternative structure, and confirm they are okay proceeding with the compliant architecture before continuing.
+- Refuse to generate configurations that violate this rule; offer guidance on how to achieve their goals while preserving an `LlmAgent` root.
+
+## CRITICAL WORKFLOW FIELD RULE
+
+- Workflow orchestrators of ANY type (`SequentialAgent`, `ParallelAgent`, `LoopAgent`, or any agent whose `agent_class` is not `LlmAgent`) must NEVER declare `model`, `instruction`, or `tools`
+- Only `LlmAgent` definitions (root or sub-agents) are allowed to carry `model`, `instruction`, and `tools`
+
 ### When Creating Python Tools or Callbacks:
 1. **Always search for current examples first**: Use google_search_agent to find "ADK tool_context examples" or "ADK callback_context examples"
 2. **Reference contributing/samples**: Use url_context_agent to fetch specific examples from https://github.com/google/adk-python/tree/main/contributing/samples
@@ -392,6 +421,7 @@ def log_tool_result(tool: BaseTool, tool_args: Dict[str, Any], tool_context: Too
 
 **File Naming & Structure:**
 - Main configuration MUST be `root_agent.yaml` (not `agent.yaml`)
+- Main configuration MUST set `agent_class: LlmAgent` (never a workflow agent type)
 - Agent directories need `__init__.py` with `from . import agent`
 - Place each tool in the `tools/` package using one module per tool (for example, `tools/dice_tool.py`).
   Add an empty `tools/__init__.py` so imports such as `project_name.tools.dice_tool.roll_dice` work.

contributing/samples/adk_pr_triaging_agent/agent.py

@@ -27,18 +27,18 @@
 from google.adk import Agent
 import requests
 
-LABEL_TO_OWNER = {
-    "documentation": "polong-lin",
-    "services": "DeanChensj",
-    "tools": "seanzhou1023",
-    "mcp": "seanzhou1023",
-    "eval": "ankursharmas",
-    "live": "hangfei",
-    "models": "genquan9",
-    "tracing": "Jacksunwei",
-    "core": "Jacksunwei",
-    "web": "wyf7107",
-}
+ALLOWED_LABELS = [
+    "documentation",
+    "services",
+    "tools",
+    "mcp",
+    "eval",
+    "live",
+    "models",
+    "tracing",
+    "core",
+    "web",
+]
 
 CONTRIBUTING_MD = read_file(
     Path(__file__).resolve().parents[3] / "CONTRIBUTING.md"
@@ -158,19 +158,19 @@ def get_pull_request_details(pr_number: int) -> str:
     return error_response(str(e))
 
 
-def add_label_and_reviewer_to_pr(pr_number: int, label: str) -> dict[str, Any]:
-  """Adds a specified label and requests a review from a mapped reviewer on a PR.
+def add_label_to_pr(pr_number: int, label: str) -> dict[str, Any]:
+  """Adds a specified label on a pull request.
 
   Args:
       pr_number: the number of the Github pull request
       label: the label to add
 
   Returns:
-      The the status of this request, with the applied label and assigned
-      reviewer when successful.
+      The the status of this request, with the applied label and response when
+      successful.
   """
-  print(f"Attempting to add label '{label}' and a reviewer to PR #{pr_number}")
-  if label not in LABEL_TO_OWNER:
+  print(f"Attempting to add label '{label}' to PR #{pr_number}")
+  if label not in ALLOWED_LABELS:
     return error_response(
         f"Error: Label '{label}' is not an allowed label. Will not apply."
     )
@@ -186,31 +186,10 @@ def add_label_and_reviewer_to_pr(pr_number: int, label: str) -> dict[str, Any]:
   except requests.exceptions.RequestException as e:
     return error_response(f"Error: {e}")
 
-  owner = LABEL_TO_OWNER.get(label, None)
-  if not owner:
-    return {
-        "status": "warning",
-        "message": (
-            f"{response}\n\nLabel '{label}' does not have an owner. Will not"
-            " assign."
-        ),
-        "applied_label": label,
-    }
-  reviewer_url = f"{GITHUB_BASE_URL}/repos/{OWNER}/{REPO}/pulls/{pr_number}/requested_reviewers"
-  reviewer_payload = {"reviewers": [owner]}
-  try:
-    post_request(reviewer_url, reviewer_payload)
-  except requests.exceptions.RequestException as e:
-    return {
-        "status": "warning",
-        "message": f"Reviewer not assigned: {e}",
-        "applied_label": label,
-    }
-
   return {
       "status": "success",
       "applied_label": label,
-      "assigned_reviewer": owner,
+      "response": response,
   }
 
 
@@ -252,7 +231,6 @@ def add_comment_to_pr(pr_number: int, comment: str) -> dict[str, Any]:
       Your core responsibility includes:
       - Get the pull request details.
       - Add a label to the pull request.
-      - Assign a reviewer to the pull request.
       - Check if the pull request is following the contribution guidelines.
       - Add a comment to the pull request if it's not following the guidelines.
 
@@ -303,7 +281,7 @@ def add_comment_to_pr(pr_number: int, comment: str) -> dict[str, Any]:
       - Skip the PR (i.e. do not label or comment) if any of the following is true:
         - the PR is closed
         - the PR is labeled with "google-contributior"
-        - the PR is already labelled with the above labels (e.g. "documentation", "services", "tools", etc.) and has a reviewer assigned.
+        - the PR is already labelled with the above labels (e.g. "documentation", "services", "tools", etc.).
       - Check if the PR is following the contribution guidelines.
         - If it's not following the guidelines, recommend or add a comment to the PR that points to the contribution guidelines (https://github.com/google/adk-python/blob/main/CONTRIBUTING.md).
         - If it's following the guidelines, recommend or add a label to the PR.
@@ -312,12 +290,11 @@ def add_comment_to_pr(pr_number: int, comment: str) -> dict[str, Any]:
       Present the followings in an easy to read format highlighting PR number and your label.
       - The PR summary in a few sentence
       - The label you recommended or added with the justification
-      - The owner of the label if you assigned a reviewer to the PR
       - The comment you recommended or added to the PR with the justification
     """,
     tools=[
         get_pull_request_details,
-        add_label_and_reviewer_to_pr,
+        add_label_to_pr,
         add_comment_to_pr,
     ],
 )

contributing/samples/computer_use/agent.py

@@ -12,24 +12,32 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
+import os
+import tempfile
+
 from google.adk import Agent
-from google.adk.models.google_llm import Gemini
 from google.adk.tools.computer_use.computer_use_toolset import ComputerUseToolset
-from typing_extensions import override
 
 from .playwright import PlaywrightComputer
 
+# Define user_data_dir path
+profile_name = 'browser_profile_for_adk'
+profile_path = os.path.join(tempfile.gettempdir(), profile_name)
+os.makedirs(profile_path, exist_ok=True)
+
+computer_with_profile = PlaywrightComputer(
+    screen_size=(1280, 936),
+    user_data_dir=profile_path,
+)
+
+# Create agent with the toolset using the new computer instance
 root_agent = Agent(
     model='gemini-2.5-computer-use-preview-10-2025',
     name='hello_world_agent',
     description=(
         'computer use agent that can operate a browser on a computer to finish'
         ' user tasks'
     ),
-    instruction="""
-      you are a computer use agent
-      """,
-    tools=[
-        ComputerUseToolset(computer=PlaywrightComputer(screen_size=(1280, 936)))
-    ],
+    instruction=""" you are a computer use agent """,
+    tools=[ComputerUseToolset(computer=computer_with_profile)],
 )