Skip to content

Update ADK Docs Path to adk.dev and rewrite instructions to be more explicit/clear to the model.#3

Open
holtskinner wants to merge 2 commits into
derailed-dash:mainfrom
holtskinner:main
Open

Update ADK Docs Path to adk.dev and rewrite instructions to be more explicit/clear to the model.#3
holtskinner wants to merge 2 commits into
derailed-dash:mainfrom
holtskinner:main

Conversation

@holtskinner

@holtskinner holtskinner commented Apr 28, 2026

Copy link
Copy Markdown
  • Fix name of ADK
  • Add .gitignore

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request updates the Agent Development Kit (ADK) documentation and configuration, including a new URL for the documentation source and improved instructions in GEMINI.md. Review feedback suggests explicitly naming the documentation source in the configuration for better model identification, ensuring consistency in example source names across files, and prioritizing the use of the built-in fetch_docs tool over external commands like curl for reliability.

Comment thread gemini-extension.json
"mcpdoc",
"--urls",
"https://google.github.io/adk-docs/llms.txt",
"https://adk.dev/llms.txt",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

Naming the documentation source explicitly (e.g., ADK:https://adk.dev/llms.txt) is a best practice when using mcpdoc. This ensures that the source name matches the examples provided in the GEMINI.md instructions (e.g., "ADK"), making it easier for the model to identify and use the correct source.

Suggested change
"https://adk.dev/llms.txt",
"ADK:https://adk.dev/llms.txt",

Comment thread GEMINI.md
### Workflow

# Accessing off-site content
1. **Discover**: Call `list_doc_sources` to identify available documentation sources (e.g., "ADK", "Sample").

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

The example source name "Sample" does not match the example provided in the README.md (which uses Local_ADK_Docs on line 85). Updating this example ensures consistency across the documentation and helps the model understand the expected source naming patterns.

Suggested change
1. **Discover**: Call `list_doc_sources` to identify available documentation sources (e.g., "ADK", "Sample").
1. **Discover**: Call `list_doc_sources` to identify available documentation sources (e.g., "ADK", "Local_ADK_Docs").

Comment thread GEMINI.md Outdated
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
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.

1 participant