Skip to content

docs: clarify function tool defaults#1731

Merged
koverholt merged 2 commits intogoogle:mainfrom
EfeDurmaz16:efe/clarify-function-tool-defaults
May 6, 2026
Merged

docs: clarify function tool defaults#1731
koverholt merged 2 commits intogoogle:mainfrom
EfeDurmaz16:efe/clarify-function-tool-defaults

Conversation

@EfeDurmaz16
Copy link
Copy Markdown
Contributor

Summary

  • clarify that defaults should not hide required user or business inputs from tool schemas
  • preserve guidance that defaults are valid for genuinely optional tuning parameters
  • align the parent custom tools guidance with the function tools optional parameter section

Fixes #1429

Verification

  • git diff --check
  • python3 -m mkdocs build --strict

@netlify
Copy link
Copy Markdown

netlify Bot commented May 5, 2026

Deploy Preview for adk-docs-preview ready!

Name Link
🔨 Latest commit e9471f2
🔍 Latest deploy log https://app.netlify.com/projects/adk-docs-preview/deploys/69fb8d90a83ede00095b43e3
😎 Deploy Preview https://deploy-preview-1731--adk-docs-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Copy Markdown
Collaborator

@koverholt koverholt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verified these updates against adk-python source. LGTM, thanks!

@koverholt koverholt merged commit 00c0455 into google:main May 6, 2026
4 of 6 checks passed
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.

Inconsistent advice in documentation regarding the use of default values in function tools

2 participants