Skip to content

Commit 214986e

Browse files
seanzhougooglecopybara-github
authored andcommitted
chore: Adjust the instruction about default model
PiperOrigin-RevId: 818765464
1 parent 348e552 commit 214986e

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

contributing/samples/adk_agent_builder_assistant/instruction_embedded.template

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,9 @@ Always reference this schema when creating configurations to ensure compliance.
6464
- **MANDATORY CONFIRMATION**: Say "Please confirm what model you want to use" - do NOT assume or suggest defaults
6565
- **EXAMPLES**: "gemini-2.5-flash", "gemini-2.5-pro", etc.
6666
- **RATIONALE**: Only LlmAgent requires model specification; workflow agents do not
67-
- **DEFAULT ONLY**: Use "{default_model}" only if user explicitly says "use default" or similar
67+
- **DEFAULT MODEL**: If user says "use default" or "proceed with default model", use: {default_model}
68+
* This is the actual model name, NOT the literal string "default"
69+
* The default model for this session is: {default_model}
6870
- **WORKFLOW**: Complete all Discovery steps (including this model selection) → Then proceed to Design Phase with model already chosen
6971

7072
### 2. Design Phase
@@ -369,7 +371,7 @@ def log_tool_result(tool: BaseTool, args: Dict[str, Any], context: ToolContext,
369371
**ADK AgentConfig Schema Compliance:**
370372
- Always reference the embedded ADK AgentConfig schema to verify field requirements
371373
- **MODEL FIELD RULES**:
372-
* **LlmAgent**: `model` field is REQUIRED (unless inherited from ancestor) - Ask user for preference only when LlmAgent is needed, use "{default_model}" if not specified
374+
* **LlmAgent**: `model` field is REQUIRED (unless inherited from ancestor) - Ask user for preference only when LlmAgent is needed, use {default_model} if user says to use default
373375
* **Workflow Agents**: `model` field is FORBIDDEN - Remove model field entirely for Sequential/Parallel/Loop agents
374376
- Optional fields: description, instruction, tools, sub_agents as defined in ADK AgentConfig schema
375377

0 commit comments

Comments
 (0)